Add --no-ask-user flag to copilot agentic engine for fully autonomous runs#25637
Add --no-ask-user flag to copilot agentic engine for fully autonomous runs#25637
Conversation
… runs Agent-Logs-Url: https://github.com/github/gh-aw/sessions/13f8c916-39b7-4aa8-9c23-eec64d37d3f0 Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Updates the Copilot agentic engine to include the --no-ask-user Copilot CLI flag (when supported) so agent runs can execute fully autonomously without interactive prompts.
Changes:
- Added a version-gated helper (
copilotSupportsNoAskUser) and wired--no-ask-userinto Copilot execution args for agent jobs only. - Introduced a new minimum-version constant for the flag (
CopilotNoAskUserMinVersion = 1.0.19). - Regenerated numerous workflow lockfiles to reflect the new Copilot CLI invocation.
Show a summary per file
| File | Description |
|---|---|
| pkg/constants/version_constants.go | Adds CopilotNoAskUserMinVersion to document the minimum Copilot CLI version that supports --no-ask-user. |
| pkg/workflow/copilot_engine_execution.go | Adds version-gated emission of --no-ask-user for agent jobs (not detection jobs). |
| .github/workflows/workflow-skill-extractor.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/workflow-normalizer.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/workflow-health-manager.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/workflow-generator.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/weekly-safe-outputs-spec-review.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/weekly-issue-summary.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/weekly-editors-health-check.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/weekly-blog-post-writer.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/video-analyzer.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/update-astro.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/tidy.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/test-quality-sentinel.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/test-project-url-default.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/test-dispatcher.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/terminal-stylist.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/technical-doc-writer.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/super-linter.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/sub-issue-closer.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/stale-repo-identifier.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/smoke-workflow-call.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/smoke-workflow-call-with-inputs.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/smoke-update-cross-repo-pr.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/smoke-test-tools.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/smoke-temporary-id.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/smoke-service-ports.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/smoke-project.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/smoke-multi-pr.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/smoke-create-cross-repo-pr.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/smoke-copilot.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/smoke-copilot-arm.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/security-review.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/security-compliance.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/research.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/repository-quality-improver.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/repo-tree-map.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/repo-audit-analyzer.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/release.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/refiner.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/refactoring-cadence.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/q.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/python-data-charts.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/pr-triage-agent.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/pr-nitpick-reviewer.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/portfolio-analyst.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/poem-bot.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/plan.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/pdf-summary.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/org-health-report.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/notion-issue-summary.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/mcp-inspector.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/jsweep.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/issue-triage-agent.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/issue-monster.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/gpclean.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/glossary-maintainer.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/github-remote-mcp-auth-test.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/functional-pragmatist.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/firewall-escape.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/draft-pr-cleanup.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/docs-noob-tester.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/discussion-task-miner.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/dictation-prompt.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/dev.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/dev-hawk.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/dependabot-go-checker.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/dependabot-burner.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/delight.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/dead-code-remover.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/daily-workflow-updater.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/daily-team-status.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/daily-semgrep-scan.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/daily-secrets-analysis.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/daily-repo-chronicle.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/daily-regulatory.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/daily-performance-summary.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/daily-news.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/daily-malicious-code-scan.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/daily-issues-report.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/daily-integrity-analysis.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/daily-firewall-report.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/daily-cli-tools-tester.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/daily-cli-performance.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/daily-assign-issue-to-user.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/daily-architecture-diagram.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/craft.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/copilot-token-optimizer.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/copilot-token-audit.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/copilot-pr-prompt-analysis.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/copilot-pr-nlp-analysis.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/copilot-pr-merged-report.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/copilot-cli-deep-research.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/contribution-check.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/constraint-solving-potd.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/code-simplifier.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/code-scanning-fixer.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/cli-consistency-checker.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/ci-coach.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/breaking-change-checker.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/brave.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/bot-detection.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/auto-triage-issues.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/artifacts-summary.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/architecture-guardian.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/archie.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/agentic-observability-kit.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/agent-persona-explorer.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
| .github/workflows/agent-performance-analyzer.lock.yml | Updates compiled Copilot CLI invocation to include --no-ask-user. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 121/121 changed files
- Comments generated: 0
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
|
📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing... |
|
✅ All tools validated successfully! Agent Container Smoke Test confirms agent container is ready. |
🧪 Test Quality Sentinel ReportTest Quality Score: 77/100
Test Classification Details
Flagged Tests — Requires Review
|
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
❌ Test Quality Sentinel: 77/100. 33.3% of new tests are classified as low-value implementation tests, just exceeding the 30% threshold. TestCopilotSupportsNoAskUser tests an unexported helper directly rather than observable behavior — its unique edge cases (LATEST uppercase, v-prefix, non-semver) should be folded into the existing behavioral test TestCopilotEngineNoAskUser as additional table rows, then the helper test removed. See the comment above for specific suggested rows. The overall test quality is high; this is a minor restructuring to align with the project's behavioral-contract testing philosophy.
Agent Container Tool Check
Result: 12/12 tools available ✅ Overall Status: PASS
|
There was a problem hiding this comment.
💥 Automated smoke test review - all systems nominal! (Run 24242990842)
💥 [THE END] — Illustrated by Smoke Claude · ● 250.8K
| # shellcheck disable=SC1003 | ||
| sudo -E awf --container-workdir "${GITHUB_WORKSPACE}" --mount "${RUNNER_TEMP}/gh-aw:${RUNNER_TEMP}/gh-aw:ro" --mount "${RUNNER_TEMP}/gh-aw:/host${RUNNER_TEMP}/gh-aw:ro" --env-all --exclude-env COPILOT_GITHUB_TOKEN --exclude-env GITHUB_MCP_SERVER_TOKEN --exclude-env MCP_GATEWAY_API_KEY --allow-domains api.business.githubcopilot.com,api.enterprise.githubcopilot.com,api.github.com,api.githubcopilot.com,api.individual.githubcopilot.com,api.snapcraft.io,archive.ubuntu.com,azure.archive.ubuntu.com,crl.geotrust.com,crl.globalsign.com,crl.identrust.com,crl.sectigo.com,crl.thawte.com,crl.usertrust.com,crl.verisign.com,crl3.digicert.com,crl4.digicert.com,crls.ssl.com,github.com,host.docker.internal,json-schema.org,json.schemastore.org,keyserver.ubuntu.com,ocsp.digicert.com,ocsp.geotrust.com,ocsp.globalsign.com,ocsp.identrust.com,ocsp.sectigo.com,ocsp.ssl.com,ocsp.thawte.com,ocsp.usertrust.com,ocsp.verisign.com,packagecloud.io,packages.cloud.google.com,packages.microsoft.com,ppa.launchpad.net,raw.githubusercontent.com,registry.npmjs.org,s.symcb.com,s.symcd.com,security.ubuntu.com,telemetry.enterprise.githubcopilot.com,ts-crl.ws.symantec.com,ts-ocsp.ws.symantec.com,www.googleapis.com --log-level info --proxy-logs-dir /tmp/gh-aw/sandbox/firewall/logs --audit-dir /tmp/gh-aw/sandbox/firewall/audit --enable-host-access --image-tag 0.25.18 --skip-pull --enable-api-proxy \ | ||
| -- /bin/bash -c 'node ${RUNNER_TEMP}/gh-aw/actions/copilot_driver.cjs /usr/local/bin/copilot --add-dir /tmp/gh-aw/ --log-level all --log-dir /tmp/gh-aw/sandbox/agent/logs/ --disable-builtin-mcps --allow-all-tools --allow-all-paths --add-dir "${GITHUB_WORKSPACE}" --prompt "$(cat /tmp/gh-aw/aw-prompts/prompt.txt)"' 2>&1 | tee -a /tmp/gh-aw/agent-stdio.log | ||
| -- /bin/bash -c 'node ${RUNNER_TEMP}/gh-aw/actions/copilot_driver.cjs /usr/local/bin/copilot --add-dir /tmp/gh-aw/ --log-level all --log-dir /tmp/gh-aw/sandbox/agent/logs/ --disable-builtin-mcps --no-ask-user --allow-all-tools --allow-all-paths --add-dir "${GITHUB_WORKSPACE}" --prompt "$(cat /tmp/gh-aw/aw-prompts/prompt.txt)"' 2>&1 | tee -a /tmp/gh-aw/agent-stdio.log |
There was a problem hiding this comment.
✅ Smoke test review comment #1 (Run 24242990842): The --no-ask-user flag addition looks correct here — this enables autonomous operation for the agent, consistent with the PR's stated goal of suppressing interactive prompts in agentic runs.
| # shellcheck disable=SC1003 | ||
| sudo -E awf --container-workdir "${GITHUB_WORKSPACE}" --mount "${RUNNER_TEMP}/gh-aw:${RUNNER_TEMP}/gh-aw:ro" --mount "${RUNNER_TEMP}/gh-aw:/host${RUNNER_TEMP}/gh-aw:ro" --env-all --exclude-env COPILOT_GITHUB_TOKEN --exclude-env GITHUB_MCP_SERVER_TOKEN --exclude-env MCP_GATEWAY_API_KEY --allow-domains api.business.githubcopilot.com,api.enterprise.githubcopilot.com,api.github.com,api.githubcopilot.com,api.individual.githubcopilot.com,api.snapcraft.io,archive.ubuntu.com,azure.archive.ubuntu.com,crl.geotrust.com,crl.globalsign.com,crl.identrust.com,crl.sectigo.com,crl.thawte.com,crl.usertrust.com,crl.verisign.com,crl3.digicert.com,crl4.digicert.com,crls.ssl.com,github.com,host.docker.internal,json-schema.org,json.schemastore.org,keyserver.ubuntu.com,ocsp.digicert.com,ocsp.geotrust.com,ocsp.globalsign.com,ocsp.identrust.com,ocsp.sectigo.com,ocsp.ssl.com,ocsp.thawte.com,ocsp.usertrust.com,ocsp.verisign.com,packagecloud.io,packages.cloud.google.com,packages.microsoft.com,ppa.launchpad.net,raw.githubusercontent.com,registry.npmjs.org,s.symcb.com,s.symcd.com,security.ubuntu.com,telemetry.enterprise.githubcopilot.com,ts-crl.ws.symantec.com,ts-ocsp.ws.symantec.com,www.googleapis.com --log-level info --proxy-logs-dir /tmp/gh-aw/sandbox/firewall/logs --audit-dir /tmp/gh-aw/sandbox/firewall/audit --enable-host-access --image-tag 0.25.18 --skip-pull --enable-api-proxy \ | ||
| -- /bin/bash -c 'node ${RUNNER_TEMP}/gh-aw/actions/copilot_driver.cjs /usr/local/bin/copilot --add-dir /tmp/gh-aw/ --log-level all --log-dir /tmp/gh-aw/sandbox/agent/logs/ --disable-builtin-mcps --allow-all-tools --add-dir /tmp/gh-aw/cache-memory/ --allow-all-paths --add-dir "${GITHUB_WORKSPACE}" --prompt "$(cat /tmp/gh-aw/aw-prompts/prompt.txt)"' 2>&1 | tee -a /tmp/gh-aw/agent-stdio.log | ||
| -- /bin/bash -c 'node ${RUNNER_TEMP}/gh-aw/actions/copilot_driver.cjs /usr/local/bin/copilot --add-dir /tmp/gh-aw/ --log-level all --log-dir /tmp/gh-aw/sandbox/agent/logs/ --disable-builtin-mcps --no-ask-user --allow-all-tools --add-dir /tmp/gh-aw/cache-memory/ --allow-all-paths --add-dir "${GITHUB_WORKSPACE}" --prompt "$(cat /tmp/gh-aw/aw-prompts/prompt.txt)"' 2>&1 | tee -a /tmp/gh-aw/agent-stdio.log |
There was a problem hiding this comment.
✅ Smoke test review comment #2 (Run 24242990842): Consistent --no-ask-user flag applied here too. The version-gate logic in copilot_engine_execution.go ensures this only emits for Copilot CLI >= 1.0.19, which is a safe approach.
|
Smoke test run 24243208379
|
|
@copilot merge main and recompile |
|
🤖 Smoke test results for run §24243208396:\n\n| Test | Status |\n|------|--------|\n| GitHub MCP | ✅ |\n| MCP Scripts GH CLI | ✅ |\n| Serena MCP | ❌ |\n| Playwright | ✅ |\n| Web Fetch | ✅ |\n| File Writing | ✅ |\n| Bash Tool | ✅ |\n| Discussion Interaction | ✅ |\n| Build gh-aw | ✅ |\n| Upload artifact | ✅ |\n| Discussion Creation | ✅ |\n| Workflow Dispatch | ✅ |\n| PR Review | ✅ |\n\nOverall:
|
There was a problem hiding this comment.
Smoke test code review: PR adds --no-ask-user flag to Copilot engine for fully autonomous agentic runs. Version gating logic is clean and follows established patterns. 187 lock files recompiled. LGTM overall.
📰 BREAKING: Report filed by Smoke Copilot · ● 1.6M
| // CopilotNoAskUserMinVersion is the minimum Copilot CLI version that supports the --no-ask-user | ||
| // flag, which enables fully autonomous agentic runs by suppressing interactive prompts. | ||
| // Workflows using an older Copilot CLI version must not emit --no-ask-user or the run will fail. | ||
| const CopilotNoAskUserMinVersion Version = "1.0.19" |
There was a problem hiding this comment.
💡 Consider adding a comment linking this constant to the GitHub release notes for Copilot CLI v1.0.19 so future maintainers can trace the origin of this version boundary.
| isDetectionJob := workflowData.SafeOutputs == nil | ||
| if !isDetectionJob && copilotSupportsNoAskUser(workflowData.EngineConfig) { | ||
| copilotExecLog.Print("Adding --no-ask-user for fully autonomous agentic run") | ||
| copilotArgs = append(copilotArgs, "--no-ask-user") |
There was a problem hiding this comment.
✅ Clean version-gated flag injection. The copilotSupportsNoAskUser check ensures backward compatibility with older pinned Copilot CLI versions — good defensive pattern consistent with how awfSupportsCliProxy is handled.
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
|
Commit pushed:
|
|
@copilot merge main and recompile |
|
Smoke Test Run 24243208388 — PARTIAL ✅
Overall: PARTIAL — Serena
|
There was a problem hiding this comment.
💥 Automated smoke test review - all systems nominal! (Run 24243208388)
💥 [THE END] — Illustrated by Smoke Claude · ● 233.1K
| # shellcheck disable=SC1003 | ||
| sudo -E awf --container-workdir "${GITHUB_WORKSPACE}" --mount "${RUNNER_TEMP}/gh-aw:${RUNNER_TEMP}/gh-aw:ro" --mount "${RUNNER_TEMP}/gh-aw:/host${RUNNER_TEMP}/gh-aw:ro" --env-all --exclude-env COPILOT_GITHUB_TOKEN --exclude-env GITHUB_MCP_SERVER_TOKEN --exclude-env MCP_GATEWAY_API_KEY --allow-domains api.business.githubcopilot.com,api.enterprise.githubcopilot.com,api.github.com,api.githubcopilot.com,api.individual.githubcopilot.com,api.snapcraft.io,archive.ubuntu.com,azure.archive.ubuntu.com,crl.geotrust.com,crl.globalsign.com,crl.identrust.com,crl.sectigo.com,crl.thawte.com,crl.usertrust.com,crl.verisign.com,crl3.digicert.com,crl4.digicert.com,crls.ssl.com,github.com,host.docker.internal,json-schema.org,json.schemastore.org,keyserver.ubuntu.com,ocsp.digicert.com,ocsp.geotrust.com,ocsp.globalsign.com,ocsp.identrust.com,ocsp.sectigo.com,ocsp.ssl.com,ocsp.thawte.com,ocsp.usertrust.com,ocsp.verisign.com,packagecloud.io,packages.cloud.google.com,packages.microsoft.com,ppa.launchpad.net,raw.githubusercontent.com,registry.npmjs.org,s.symcb.com,s.symcd.com,security.ubuntu.com,telemetry.enterprise.githubcopilot.com,ts-crl.ws.symantec.com,ts-ocsp.ws.symantec.com,www.googleapis.com --log-level info --proxy-logs-dir /tmp/gh-aw/sandbox/firewall/logs --audit-dir /tmp/gh-aw/sandbox/firewall/audit --enable-host-access --image-tag 0.25.18 --skip-pull --enable-api-proxy \ | ||
| -- /bin/bash -c 'node ${RUNNER_TEMP}/gh-aw/actions/copilot_driver.cjs /usr/local/bin/copilot --add-dir /tmp/gh-aw/ --log-level all --log-dir /tmp/gh-aw/sandbox/agent/logs/ --disable-builtin-mcps --allow-all-tools --allow-all-paths --add-dir "${GITHUB_WORKSPACE}" --prompt "$(cat /tmp/gh-aw/aw-prompts/prompt.txt)"' 2>&1 | tee -a /tmp/gh-aw/agent-stdio.log | ||
| -- /bin/bash -c 'node ${RUNNER_TEMP}/gh-aw/actions/copilot_driver.cjs /usr/local/bin/copilot --add-dir /tmp/gh-aw/ --log-level all --log-dir /tmp/gh-aw/sandbox/agent/logs/ --disable-builtin-mcps --no-ask-user --allow-all-tools --allow-all-paths --add-dir "${GITHUB_WORKSPACE}" --prompt "$(cat /tmp/gh-aw/aw-prompts/prompt.txt)"' 2>&1 | tee -a /tmp/gh-aw/agent-stdio.log |
There was a problem hiding this comment.
✅ Smoke test review comment #1 (Run 24243208388): The --no-ask-user flag correctly enables fully autonomous operation for this workflow. Consistent with the PR goal of suppressing interactive prompts.
| # shellcheck disable=SC1003 | ||
| sudo -E awf --container-workdir "${GITHUB_WORKSPACE}" --mount "${RUNNER_TEMP}/gh-aw:${RUNNER_TEMP}/gh-aw:ro" --mount "${RUNNER_TEMP}/gh-aw:/host${RUNNER_TEMP}/gh-aw:ro" --env-all --exclude-env COPILOT_GITHUB_TOKEN --exclude-env GITHUB_MCP_SERVER_TOKEN --exclude-env MCP_GATEWAY_API_KEY --allow-domains api.business.githubcopilot.com,api.enterprise.githubcopilot.com,api.github.com,api.githubcopilot.com,api.individual.githubcopilot.com,api.snapcraft.io,archive.ubuntu.com,azure.archive.ubuntu.com,crl.geotrust.com,crl.globalsign.com,crl.identrust.com,crl.sectigo.com,crl.thawte.com,crl.usertrust.com,crl.verisign.com,crl3.digicert.com,crl4.digicert.com,crls.ssl.com,github.com,host.docker.internal,json-schema.org,json.schemastore.org,keyserver.ubuntu.com,ocsp.digicert.com,ocsp.geotrust.com,ocsp.globalsign.com,ocsp.identrust.com,ocsp.sectigo.com,ocsp.ssl.com,ocsp.thawte.com,ocsp.usertrust.com,ocsp.verisign.com,packagecloud.io,packages.cloud.google.com,packages.microsoft.com,ppa.launchpad.net,raw.githubusercontent.com,registry.npmjs.org,s.symcb.com,s.symcd.com,security.ubuntu.com,telemetry.enterprise.githubcopilot.com,ts-crl.ws.symantec.com,ts-ocsp.ws.symantec.com,www.googleapis.com --log-level info --proxy-logs-dir /tmp/gh-aw/sandbox/firewall/logs --audit-dir /tmp/gh-aw/sandbox/firewall/audit --enable-host-access --image-tag 0.25.18 --skip-pull --enable-api-proxy \ | ||
| -- /bin/bash -c 'node ${RUNNER_TEMP}/gh-aw/actions/copilot_driver.cjs /usr/local/bin/copilot --add-dir /tmp/gh-aw/ --log-level all --log-dir /tmp/gh-aw/sandbox/agent/logs/ --disable-builtin-mcps --allow-all-tools --allow-all-paths --add-dir "${GITHUB_WORKSPACE}" --prompt "$(cat /tmp/gh-aw/aw-prompts/prompt.txt)"' 2>&1 | tee -a /tmp/gh-aw/agent-stdio.log | ||
| -- /bin/bash -c 'node ${RUNNER_TEMP}/gh-aw/actions/copilot_driver.cjs /usr/local/bin/copilot --add-dir /tmp/gh-aw/ --log-level all --log-dir /tmp/gh-aw/sandbox/agent/logs/ --disable-builtin-mcps --no-ask-user --allow-all-tools --allow-all-paths --add-dir "${GITHUB_WORKSPACE}" --prompt "$(cat /tmp/gh-aw/aw-prompts/prompt.txt)"' 2>&1 | tee -a /tmp/gh-aw/agent-stdio.log |
There was a problem hiding this comment.
✅ Smoke test review comment #2 (Run 24243208388): --no-ask-user flag applied uniformly across all copilot engine invocations. Good consistency throughout the PR changes.
…t-agentic-engine # Please enter a commit message to explain why this merge is necessary, # especially if it merges an updated upstream into a topic branch. # # Lines starting with '#' will be ignored, and an empty message aborts # the commit. Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Done — merged main and recompiled all 187 workflows (c7f3e50). 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:
If you need me to access, download, or install something from one of these locations, you can either:
|
🟢 Contribution Check — AlignedHey This PR looks ready for maintainer review. 🎉
|
Summary
Updates the Copilot agentic engine to emit
--no-ask-userwhen invoking the Copilot CLI, enabling fully autonomous agentic runs by suppressing interactive user prompts.Changes
pkg/constants/version_constants.go: AddedCopilotNoAskUserMinVersion = "1.0.19"constant documenting the minimum Copilot CLI version that supports the flag.pkg/workflow/copilot_engine_execution.go:copilotSupportsNoAskUser(engineConfig *EngineConfig) boolhelper following the same version-gate pattern asawfSupportsExcludeEnvandawfSupportsCliProxyGetExecutionSteps, emits--no-ask-userfor agent jobs when the Copilot CLI version supports itpkg/workflow/copilot_engine_test.go: AddedTestCopilotEngineNoAskUserandTestCopilotSupportsNoAskUserwith comprehensive coverage187 lock files recompiled to include the new flag
Version-gate logic
--no-ask-useremitted?1.0.20)latest>= 1.0.19< 1.0.19✨ PR Review Safe Output Test - Run 24242990842
Changeset
--no-ask-userfor Copilot agent jobs when the configured Copilot CLI version supports it (>= 1.0.19), keeping conservative behavior for unsupported or non-semver versions.✨ PR Review Safe Output Test - Run 24243208388