fix(demo): end the startup terminal on the report, hide the chat sidebar - #26
Merged
Merged
Conversation
Tested end to end in a fresh npm Codespace: - `postAttachCommand` ran `drift analyze` in a real terminal, so after the report it asked whether to file a GitHub issue or cut a branch. The startup terminal sat on that question with a needs-attention flag, one keypress from opening an issue on this repository. Stdin is now /dev/null, which the CLI treats as non-interactive: the report is where the command ends. - Codespaces opened the chat secondary sidebar by default, taking a third of the window and wrapping the terminal report mid-word. `workbench.secondarySideBar.defaultVisibility: hidden` keeps it closed. Also corrects the `--dir` comment: the subdirectory bug it described was fixed in @usedrift/cli 0.1.2. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Found by testing the npm Codespace end to end in a browser.
postAttachCommand: drift analyzeruns in a real terminal, so after the report the CLI offered File a GitHub issue / Create a local branch / … and the terminal sat there flagged ⚠ — one keypress from filing an issue on this repo. Nowdrift analyze < /dev/null: non-interactive, ends on the report. Works with every published CLI version."workbench.secondarySideBar.defaultVisibility": "hidden".--dircomment (that bug was fixed in@usedrift/cli0.1.2).All 16 devcontainers regenerated with
node scripts/sync-devcontainers.mjs;node scripts/check-demos.mjs→ All demos OK.The blank Drift panel seen in the same test is an extension bug, fixed in trydrift/drift#272.
🤖 Generated with Claude Code