fix: propagate boundedAgents config fields to CLI mapping spec - #6903
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Documents all boundedAgents settings as config-only fields in the CLI mapping specification.
Changes:
- Adds mapping entries for all 15
boundedAgentsfields. - Cross-references the bounded-agent specification in §15.
Show a summary per file
| File | Description |
|---|---|
docs/awf-config-spec.md |
Adds boundedAgents.* entries to the CLI mapping table. |
Review details
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Balanced
|
✅ Copilot review passed with no inline comments. @github-actions[bot] Add the |
|
📰 VERDICT: Smoke Docker Sbx has concluded. All systems operational. This is a developing story. 🎤 |
|
✅ Smoke Gemini completed. All facets verified. 💎 |
|
📡 Smoke OTel Tracing completed. All tracing scenarios validated. ✅ |
|
✅ Build Test Suite completed successfully! |
|
🔌 Smoke Services — All services reachable! ✅ |
|
✅ Contribution Check completed successfully! No contribution-guidelines issues found for PR #6903. The change is docs-only, matches the existing config-only mapping pattern, and the PR description clearly explains the gap and fix. No additional tests or file-organization changes were needed. |
|
✅ Smoke Copilot BYOK AOAI (api-key) completed. Copilot AOAI BYOK (api-key) mode operational. 🔓 |
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
|
✅ Smoke Copilot BYOK completed. Copilot BYOK mode operational. 🔓 |
|
🚀 Security Guard has started processing this pull request |
|
🛡️ Smoke Copilot Network Isolation confirmed the egress allowlist is enforced. ✅ |
|
✅ Smoke Copilot BYOK AOAI (Entra) completed. Copilot AOAI BYOK (Entra) mode operational. 🔓 |
|
✅ Smoke Claude passed |
|
Chroot tests passed! Smoke Chroot - All security and functionality tests succeeded. |
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
Smoke Test: Copilot BYOK (Direct) Mode ✅ PASS✅ GitHub MCP: PR #6898, #6894 retrieved Status: All tests passed. Running in direct BYOK mode (COPILOT_PROVIDER_API_KEY). cc
|
Smoke Test: Claude Engine Validation
Overall result: PASS
|
|
EGRESS_RESULT allow=pass deny=pass ✅ Allowed domain (github.com) reachable: Overall: PASS cc Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "example.com"See Network Configuration for more information.
|
|
Smoke Test: Copilot Engine —
Recent PRs checked: #6898 (Auth Doctor Updater), #6894 (isolate Actions OIDC) Overall: PASS
|
github-actions
Bot
added
the
smoke-copilot-network-isolation
Smoke Test: Services Connectivity
Overall: FAIL —
|
📡 OTel Tracing Smoke Test Results
Overall: All scenarios pass. No regressions detected in OTEL tracing integration.
|
Chroot Version Comparison Results
Overall: FAILED - Node.js version differs between host and chroot environments (smoke-chroot label not added).
|
|
Smoke test summary
Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "registry.npmjs.org"See Network Configuration for more information.
|
|
${{ steps.smoke-data.outputs.SMOKE_PR_DATA }} ✅ GitHub MCP connectivity Running in direct BYOK mode (COPILOT_PROVIDER_API_KEY + COPILOT_PROVIDER_BASE_URL) via api-proxy → Azure OpenAI (Foundry, o4-mini-aw) cc
|
|
feat: add Auth Doctor Updater workflow — ✅
|
Smoke Test: Docker Sbx Validation
Overall: PASS cc
|
🏗️ Build Test Suite Results
Overall: 8/8 ecosystems passed — PASS Notes:
|
Gemini Smoke Test Results
Overall status: FAIL
|
Summary
Config-consistency audit of recently merged PRs found one gap: the
boundedAgentsconfiguration section (introduced in PR #6872, refined in #6874) is fully wired in the JSON Schema (src/awf-config-schema.json/docs/awf-config.schema.json, kept identical) and insrc/config-file.ts(AwfFileConfig.boundedAgents), but Section 5 (CLI Mapping) ofdocs/awf-config-spec.mdhad no rows for anyboundedAgents.*field — unlike its siblingboundedQueries, which has a full set of mapping rows.Fix
Added 16 rows to Section 5 documenting each
boundedAgents.*field as config-only (no CLI flag), consistent with the existingboundedQueries.*rows and cross-referencing §15 where the feature is fully documented.config-file.ts)boundedAgents.*rowsPRs reviewed (no other gaps found)
Verification checklist
tsc --noEmit(docs-only change; no TS files touched)diff src/awf-config-schema.json docs/awf-config.schema.jsonconfirmed identical (verified before and after this change)