Skip to content

docs: add missing apiProxy CLI mappings and sync schema descriptions#2759

Merged
lpcox merged 3 commits intomainfrom
docs/spec-cli-mapping-sync
May 8, 2026
Merged

docs: add missing apiProxy CLI mappings and sync schema descriptions#2759
lpcox merged 3 commits intomainfrom
docs/spec-cli-mapping-sync

Conversation

@lpcox
Copy link
Copy Markdown
Collaborator

@lpcox lpcox commented May 8, 2026

Summary

Integrates the changes from #2753 (which was closed without merging) into a clean PR.

Changes

docs/awf-config-spec.md — three missing rows added to §5 CLI mapping:

Config path CLI flag / note
apiProxy.anthropicAutoCache --anthropic-auto-cache
apiProxy.anthropicCacheTailTtl --anthropic-cache-tail-ttl
apiProxy.models config-only (model alias rewriting)

src/awf-config-schema.json — synced descriptions to match docs/awf-config.schema.json:

  • apiProxy.enabled — now references §9 credential isolation semantics
  • environment — now references §8–9 merge and credential isolation rules

Supersedes #2753.

Add three apiProxy entries missing from the §5 CLI mapping table in
awf-config-spec.md:
- apiProxy.anthropicAutoCache → --anthropic-auto-cache
- apiProxy.anthropicCacheTailTtl → --anthropic-cache-tail-ttl
- apiProxy.models → config-only (model alias rewriting)

Sync src/awf-config-schema.json descriptions for apiProxy.enabled and
environment to match docs/awf-config.schema.json, which already
references §8–9 credential isolation and merge semantics.

Supersedes #2753.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 8, 2026 19:04
@lpcox lpcox requested a review from Mossaka as a code owner May 8, 2026 19:04
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

Documentation Preview

Documentation build failed for this PR. View logs.

Built from commit 76ed37a

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

✅ Coverage Check Passed

Overall Coverage

Metric Base PR Delta
Lines 87.48% 87.55% 📈 +0.07%
Statements 87.44% 87.51% 📈 +0.07%
Functions 82.66% 82.66% ➡️ +0.00%
Branches 79.65% 79.69% 📈 +0.04%
📁 Per-file Coverage Changes (1 files)
File Lines (Before → After) Statements (Before → After)
src/container-lifecycle.ts 87.1% → 88.2% (+1.14%) 87.5% → 88.6% (+1.11%)

Coverage comparison generated by scripts/ci/compare-coverage.ts

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates AWF configuration documentation and schema metadata to reflect existing CLI/config behavior around the API proxy and environment/credential isolation, incorporating previously unmerged doc fixes from #2753.

Changes:

  • Adds missing apiProxy.* entries to the §5 CLI mapping table in docs/awf-config-spec.md.
  • Updates src/awf-config-schema.json descriptions for apiProxy.enabled and environment to match docs/awf-config.schema.json and reference the spec’s credential-isolation/merge semantics.
Show a summary per file
File Description
docs/awf-config-spec.md Adds missing §5 CLI mapping rows for Anthropic auto-cache flags and notes apiProxy.models as config-only.
src/awf-config-schema.json Aligns schema descriptions with the docs schema, including references to §8–9 environment merge + credential isolation semantics.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 2/2 changed files
  • Comments generated: 2

Comment thread docs/awf-config-spec.md Outdated
Comment thread src/awf-config-schema.json Outdated
@github-actions

This comment has been minimized.

lpcox and others added 2 commits May 8, 2026 12:10
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

Smoke Test: Copilot BYOK (Offline) Mode

Test Result
GitHub MCP (list PRs)
GitHub.com connectivity (HTTP 200)
File write/read ⚠️ template vars unsubstituted
BYOK inference (this response)

Running in BYOK offline mode (COPILOT_OFFLINE=true) via api-proxy → api.githubcopilot.com.

PR by @lpcox · reviewer: @Mossaka

Overall: PASS

🔑 BYOK report filed by Smoke Copilot BYOK

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

Smoke Test Results

✅ GitHub MCP: Last 2 merged PRs retrieved (#2757, #2747)
✅ Playwright: github.com page title verified
✅ File Writing: Test file created successfully
✅ Bash Tool: File verification passed

Overall: PASS

💥 [THE END] — Illustrated by Smoke Claude

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

🔮 Codex smoke oracle: FAIL
PRs: docs: proofread AWF config spec for W3C-style conformance; feat: add daily schema-sync workflow
✅ GitHub PR review, Playwright, file/bash, discussion, build
❌ Safe Inputs GH CLI: safeinputs-gh missing
❌ Tavily web search: no Tavily tools exposed
Overall status: FAIL

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • registry.npmjs.org

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "registry.npmjs.org"

See Network Configuration for more information.

🔮 The oracle has spoken through Smoke Codex

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

🤖 Smoke Test Results

Test Status
GitHub MCP connectivity
GitHub.com HTTP connectivity ⚠️ (pre-step data unavailable — template vars not expanded)
File write/read ⚠️ (file not found at expected path)

PR: docs: add missing apiProxy CLI mappings and sync schema descriptions
Author: @lpcox | Reviewer: @Mossaka

Overall: PARTIAL — MCP ✅, pre-step outputs were not injected into this workflow step.

📰 BREAKING: Report filed by Smoke Copilot

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

🧪 Chroot Version Comparison

Runtime Host Version Chroot Version Match?
Python Python 3.12.13 Python 3.12.3
Node.js v24.14.1 v20.20.2
Go go1.22.12 go1.22.12

Result: Some versions differ between host and chroot. Go matches; Python and Node.js do not.

Tested by Smoke Chroot

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

Smoke Test: GitHub Actions Services Connectivity

Check Result
Redis PING ❌ Timeout (no response)
PostgreSQL pg_isready ❌ No response
PostgreSQL SELECT 1 ❌ Failed

Overall: FAILhost.docker.internal is not reachable from this environment. Service containers may not be running or the host alias is not configured.

🔌 Service connectivity validated by Smoke Services

@lpcox lpcox merged commit bf10d9d into main May 8, 2026
62 of 68 checks passed
@lpcox lpcox deleted the docs/spec-cli-mapping-sync branch May 8, 2026 19:26
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

🏗️ Build Test Suite Results

Ecosystem Project Build/Install Tests Status
Bun elysia 1/1 passed ✅ PASS
Bun hono 1/1 passed ✅ PASS
C++ fmt N/A ✅ PASS
C++ json N/A ✅ PASS
Deno oak N/A 1/1 passed ✅ PASS
Deno std N/A 1/1 passed ✅ PASS
.NET hello-world N/A ✅ PASS
.NET json-parse N/A ✅ PASS
Go color 1/1 passed ✅ PASS
Go env 1/1 passed ✅ PASS
Go uuid 1/1 passed ✅ PASS
Java gson 1/1 passed ✅ PASS
Java caffeine 1/1 passed ✅ PASS
Node.js clsx All passed ✅ PASS
Node.js execa All passed ✅ PASS
Node.js p-limit All passed ✅ PASS
Rust fd 1/1 passed ✅ PASS
Rust zoxide 1/1 passed ✅ PASS

Overall: 8/8 ecosystems passed — ✅ PASS

Generated by Build Test Suite for issue #2759 · ● 541.8K ·

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants