docs(brainstorming): correct Copilot CLI backgrounding guidance for Windows - #2006
Open
arimu1 wants to merge 1 commit into
Open
docs(brainstorming): correct Copilot CLI backgrounding guidance for Windows#2006arimu1 wants to merge 1 commit into
arimu1 wants to merge 1 commit into
Conversation
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.
Who is submitting this PR? (required)
What problem are you trying to solve?
GitHub Copilot CLI on Windows uses PowerShell as its primary shell environment. This environment does not support bash-specific async tool options (
bashtool,read_bash,stop_bash,mode: "async"). However,skills/brainstorming/visual-companion.mdincorrectly advises Windows/Copilot CLI users to use these nonexistent bash-specific options.This mismatch was originally reported in #1929.
What does this PR change?
This PR updates the Copilot CLI section in
visual-companion.mdto guide users to use the harness's general background execution mechanism instead of bash-specific options, aligning it with the general "Other environments" instructions in the same file.Is this change appropriate for the core library?
Yes, it corrects general documentation for the core brainstorming skill.
What alternatives did you consider?
We considered adding
.ps1backgrounding scripts or native PowerShell adapters, but decided a docs-only fix pointing to the platform's general background execution mechanism is much simpler, safer, and follows the existing "Other environments" precedent.Does this PR contain multiple unrelated changes?
No.
Existing PRs
Environment tested
N/A (documentation-only correction).