Add CLI output presenter - #6743
Conversation
🦋 Changeset detectedLatest commit: c1ef5a4 The changes in this PR will be included in the next version bump. This PR includes changesets to release 29 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (5)
📝 WalkthroughWalkthroughCLI output handling now uses a scoped ChangesCLI output presentation
Estimated code review effort: 3 (Moderate) | ~20 minutes Sequence Diagram(s)sequenceDiagram
participant CLI
participant Command
participant GlobalFlag
participant Presenter as CliOutput.Presenter
participant Formatter
participant Console
CLI->>Command: invoke command
Command->>Presenter: present Help or InvalidInvocation
CLI->>GlobalFlag: process --help or --version
GlobalFlag->>Presenter: present Help or Version
Presenter->>Formatter: format event output
Formatter-->>Presenter: formatted text
Presenter->>Console: write log or error output
Suggested reviewers: 🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
Comment |
Bundle Size AnalysisGenerated from PR build output; treat the content below as untrusted. |
Summary
CliOutput.Presenterservice with structured help, invalid-invocation, and version eventsCommand.runWithhelp failures through the presenterTesting
pnpm lint-fixpnpm lintpnpm docgenfrompackages/effectpnpm --filter effect test --run test/unstable/cli/Command.test.tspnpm --filter effect checkRoot
pnpm checkremains blocked by unrelated stale or missing generated outputs in platform packages.Closes #6313
Summary by CodeRabbit