-
Notifications
You must be signed in to change notification settings - Fork 46
docs: Clarify engine selection in Quick Start guide #12135
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
+11
−5
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
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Document AI engine selection in Quick Start flow
docs: Clarify engine selection in Quick Start guide
Jan 28, 2026
Contributor
🔍 PR Triage ResultsCategory: docs | Risk: low | Priority: 43/100 Scores Breakdown
📋 Recommended Action: auto_mergeThis PR is an excellent auto-merge candidate:
This documentation improvement clarifies engine selection options in the Quick Start guide, making it clear that Claude, Copilot, and Codex are all first-class options. Triaged by PR Triage Agent on 2026-01-28T00:34:49Z
|
This was referenced Jan 28, 2026
pelikhan
approved these changes
Jan 28, 2026
Mossaka
pushed a commit
that referenced
this pull request
Jan 28, 2026
This was referenced Jan 28, 2026
Mossaka
added a commit
that referenced
this pull request
Jan 28, 2026
* Initial plan * Add build and test tasks to all smoke workflows - Add 'make build' and 'make test' task to smoke-copilot.md - Add 'make build' and 'make test' task to smoke-codex.md - Add 'make test' task to smoke-claude.md (already has build test) - Add 'make build' and 'make test' task to smoke-opencode.md - Recompile all workflows to update .lock.yml files - Minimal changes following update-agentic-workflow.md guidelines - No UI changes CI note: lint-go failure in run 21378132566 is unrelated (unused funcs in add_command.go and add_interactive.go) * Add changeset [skip-ci] * Merge main and regenerate workflows * Regenerate workflows after merge with main Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * Add changeset [skip-ci] * chore: increase smoke workflow timeouts and remove make test - Increase timeout from 5/10 minutes to 15 minutes for all smoke workflows - Remove 'make test' from smoke test steps, keeping only 'make build' - Affects: smoke-copilot, smoke-codex, smoke-claude, smoke-opencode - All workflows already have 'go' runtime configured (no changes needed) * Add runtimes: go to smoke-opencode workflow Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * Add changeset [skip-ci] * Recompile workflows after merging origin/main Updates lock files for security-alert-burndown and security-guard workflows to incorporate environment variable mirroring changes from main branch. Changes: - Add GH_AW_TOOL_BINS environment variable construction - Pass additional runner environment variables (ANDROID_*, CARGO_HOME, etc.) - Update PATH construction to use GH_AW_TOOL_BINS Related to env-mirror feature from origin/main. * Update artifact usage counts for security-guard workflow * Add changeset [skip-ci] * Fix shell escaping for environment variable expansion in AWF Pre-wrap environment variable values in double quotes so they're properly expanded inside AWF containers. This fixes the PATH priority mechanism for tools from actions/setup-*. Changes: - env_mirror.go: Wrap mirrored env vars in double quotes - engine_helpers.go: Wrap GH_AW_TOOL_BINS in double quotes - Updated tests to expect the new format - Regenerated all lock files Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * Regenerate lock files after merge * Add changeset [skip-ci] * docs: Clarify engine selection in Quick Start guide (#12135) * Use go env GOROOT instead of GOROOT env var for Go bin path actions/setup-go doesn't export GOROOT as an environment variable (only adds Go to PATH). This caused GH_AW_TOOL_BINS to not include the Go bin path, resulting in older Go versions from hostedtoolcache being used due to alphabetical find ordering. Fix: Use `command -v go && $(go env GOROOT)/bin` to dynamically determine GOROOT from the Go toolchain itself. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * Add changeset [skip-ci] * Merge main and regenerate agentic workflows Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * Fix Go build cache permission error in AWF sandbox Add $HOME/.cache mount to AWF container configuration for all engines (Claude, Codex, Copilot). This allows Go's build cache and other tool caches (npm, etc.) to be writable inside the sandbox container. The directory is created with `mkdir -p "$HOME/.cache"` before AWF runs to ensure the mount path exists on the host. Also added /home/runner/.cache to the SRT sandbox's default AllowWrite list for consistency. Fixes permission denied errors when running `make build` inside the agent container (e.g., smoke-codex workflow). Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * Add changeset [skip-ci] * Use --dangerously-bypass-approvals-and-sandbox for Codex engine Replace --full-auto and --sandbox danger-full-access flags with the single --dangerously-bypass-approvals-and-sandbox flag which combines both behaviors. This simplifies the command and is the recommended approach for externally sandboxed environments like AWF. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * Add changeset [skip-ci] --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: Jiaxiao Zhou <duibao55328@gmail.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
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.
Quick Start guide referenced engine selection vaguely, obscuring that Claude, Copilot, and Codex are all first-class options. Users may assume Copilot is required.
Changes
Prerequisites section - Expanded AI Account requirement to explicit list:
Interactive process description - Changed from:
To:
Added TIP callout after interactive process:
All language treats engines neutrally without suggesting Copilot primacy.
Warning
Firewall rules blocked me from connecting to one or more addresses (expand for details)
I tried to connect to the following addresses, but was blocked by firewall rules:
telemetry.astro.build/opt/hostedtoolcache/node/24.13.0/x64/bin/node node /home/REDACTED/work/gh-aw/gh-aw/docs/node_modules/.bin/astro build(dns block)If you need me to access, download, or install something from one of these locations, you can either:
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.