fix(cli): implement confirmDestructive prompts

[galligan]

Summary

• Implement confirmDestructive prompt handling in CLI input.
• Extend input tests for destructive confirmations.
• Update CLI package dependencies.

Changes

• packages/cli/src/input.ts
• packages/cli/src/tests/input.test.ts
• packages/cli/package.json
• bun.lock

Testing

• Not run (not requested).

[galligan]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• chore(outfitter): sync templates on pack #108
• chore(changesets): ignore app package #107
• fix(errors): avoid TaggedError extends warnings #106
• chore(release): align package versions to 0.1.0 #105
• docs: add README and CHANGELOG for testing, kit, outfitter #95
• feat(templates): add daemon project template #94
• feat(templates): add MCP server project template #93
• feat(templates): add CLI project template #92
• fix(cli): implement confirmDestructive prompts #104 👈 (View in Graphite)
• fix(cli): implement createCLI and command builder #103
• feat(kit): add @outfitter/kit meta-package #91
• docs(daemon): add README and CHANGELOG #90
• fix(daemon): fix locking module bugs (GREEN phase) #89
• test(daemon): add locking module tests (RED phase) #88
• feat(daemon): add errors, platform, and locking module stubs #87
• feat(outfitter): add doctor command for environment validation #86
• feat(outfitter): add init command with project scaffolding #85
• feat(daemon): add IPC communication and health checks #84
• feat(daemon): add types and lifecycle management #83
• feat(index): add FTS5 full-text search implementation #82
• feat(index): add types for FTS5 full-text search #81
• feat(testing): add MCP and CLI test harnesses #80
• feat(testing): add test fixture utilities #79
• feat(mcp): add MCP server implementation with typed handlers #78
• feat(mcp): add types and interfaces for MCP server #77
• feat(ui): add Shape discriminated union and render() #76
• feat(ui): add createTokens semantic color layer #75
• feat(state): add pagination helpers #74
• feat(logging): add DEFAULT_PATTERNS for common secret redaction #73
• feat(ui): add formatRelative() for human-readable timestamps #72
• feat(state): add encodeCursor/decodeCursor for pagination #71
• feat(types): add hashId() for deterministic short identifiers #70
• feat(contracts): add Logger.child() and fix AssertionError category #69
• feat(config): centralize environment variable access with Zod validation #42
• fix(logging): enable redaction by default when global rules exist #41
• docs(ui): add README, CHANGELOG, and TSDoc comments #39
• docs(state): add README, CHANGELOG, and TSDoc comments #38
• docs(file-ops): add README, CHANGELOG, and TSDoc comments #37
• docs(logging): add README, CHANGELOG, and TSDoc comments #36
• docs(config): add README, CHANGELOG, and TSDoc comments #35
• fix(cli): enforce cursor TTL #114
• fix(tests): remove placeholder tests #113
• chore(gitattributes): prefer ours for bun lockfiles #115
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

[greptile-apps]

Greptile Summary

• Implements confirmDestructive interactive prompt functionality in CLI input module using @clack/prompts for destructive operations
• Adds comprehensive test coverage transitioning from RED to GREEN phase of TDD with proper mocking for prompt interactions
• Updates package dependencies to include @clack/prompts which aligns with project's blessed dependencies

Important Files Changed

Filename	Overview
packages/cli/src/input.ts	Replaces placeholder implementation with functional destructive confirmation prompts using Result pattern
packages/cli/src/tests/input.test.ts	Adds comprehensive test suite for confirmation prompts with mock infrastructure and multiple scenarios

Confidence score: 4/5

• This PR is safe to merge with only minor concerns about test complexity
• Score reflects solid implementation following TDD principles but complexity in mock infrastructure could lead to maintenance challenges
• Pay close attention to packages/cli/src/__tests__/input.test.ts for mock queue management and test isolation

Sequence Diagram

sequenceDiagram
    participant User
    participant CLI
    participant prompts as "@clack/prompts"
    participant process as "Process/TTY"

    User->>CLI: "Invoke destructive command"
    CLI->>CLI: "Check bypassFlag"
    alt bypassFlag is true
        CLI->>User: "Return Ok(true)"
    else bypassFlag is false
        CLI->>process: "Check TTY and TERM"
        alt non-TTY or TERM=dumb
            CLI->>User: "Return Err(CancelledError)"
        else interactive mode
            CLI->>prompts: "confirm({ message })"
            prompts->>User: "Show confirmation prompt"
            User->>prompts: "User response (true/false/cancel)"
            prompts->>CLI: "Return response"
            alt response is true
                CLI->>User: "Return Ok(true)"
            else response is false or cancel
                CLI->>User: "Return Err(CancelledError)"
            end
        end
    end

Loading

[galligan]

Restacked and resubmitted. No additional changes in this PR.