Skip to content

fix: propagate config fields to all layers#5809

Draft
github-actions[bot] wants to merge 1 commit into
mainfrom
fix/config-audit-container-mounts-spec-6faf42f774223327
Draft

fix: propagate config fields to all layers#5809
github-actions[bot] wants to merge 1 commit into
mainfrom
fix/config-audit-container-mounts-spec-6faf42f774223327

Conversation

@github-actions

@github-actions github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Config Consistency Fixes

Automated fixes for configuration fields not fully propagated across all required layers.

From PR #5755 — feat: add container.mounts to AWF config file schema

Field Fix Applied
container.mounts Added CLI mapping entry to Section 5 of docs/awf-config-spec.md: maps to -v, --mount (repeatable)

Root cause: PR #5755 correctly updated src/awf-config-schema.json, docs/awf-config.schema.json, src/config-file.ts, and src/config-mapper.ts, but did not add the CLI mapping entry to docs/awf-config-spec.md.

Verification

  • Schemas identical: diff src/awf-config-schema.json docs/awf-config.schema.json produces no output
  • TypeScript compiles: tsc passes with no errors
  • Config-file-mapping tests pass: 118 tests passed across config-file-mapping and schema suites

Warning

Firewall blocked 1 domain

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

  • awmgmcpg

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

network:
  allowed:
    - defaults
    - "awmgmcpg"

See Network Configuration for more information.

Generated by Config Consistency Auditor · 58.1 AIC · ⊞ 6.4K ·

Add missing container.mounts CLI mapping entry to awf-config-spec.md.

PR #5755 added container.mounts to the JSON schema, TypeScript types,
and config-file.ts mapping, but the Section 5 CLI mapping table in
docs/awf-config-spec.md was not updated.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

0 participants