Skip to content

[instructions] Sync checkout and model-routing instruction docs with current schema - #68999

Merged
pelikhan merged 4 commits into
mainfrom
instructions-janitor-sync-v0.91.12-89e8f9f5a9374d32
Oct 15, 2026
Merged

pelikhan merged 4 commits into
mainfrom
instructions-janitor-sync-v0.91.12-89e8f9f5a9374d32

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Summary

Routine instructions-janitor pass against release v0.91.12 (published 2026-10-14T04:49:11Z). Repository is a shallow clone (1 local commit), so the release/change audit used GitHub MCP list_commits/get_commit reads scoped to docs/ and .github/aw/ instead of local git diff.

Findings

All .github/aw/*.md files that were directly touched by post-release commits (mcp-clis.md via #68716, work-queue.md via #68582, agentic-workflows-mcp.md via #68625) were already updated by their own PRs and are current. The accuracy audit against pkg/parser/schemas/main_workflow_schema.json found two real gaps that predate those commits:

  1. checkout: schema drift — the checkoutConfig schema has long had multi, allowed-repos, allow-private-repos, max-repos (runtime multi-repository checkout allowlisting, Add allowlisted dynamic repository checkout lists #68593/Require explicit multi: true for multi-repository checkout #68816), plus repo alias, current, github-app, safe-outputs-github-app, pull-request, force-clean-git-credentials, none of which were documented anywhere in .github/aw/.
  2. engine.model-routing.allowed-models described as required — the schema only requires goal/mode (allowed-models is optional per Model routing: make models.allowed/models.blocked the only request policy, and allowed-models optional router candidates #68495, a breaking change already in v0.91.12 itself), and the omitted-value / import-merge semantics from Model routing: an omitted allowed-models means any model when merging imports #68883 were undocumented.

Changes

  • Added .github/aw/syntax-checkout.md (87 lines) with the full checkout: field reference, including the multi-repository allowlist section.
  • Trimmed the inline checkout: block out of syntax-agentic.md down to a short pointer — this also pulled it back from 397 lines (near the 400-line limit) to 345.
  • Added a checkout: row to the syntax.md index.
  • Fixed syntax-engine.md's engine.model-routing entry: allowed-models is no longer listed as required, and now documents the omitted-value/model-family default and cross-import merge behavior.

Before/after line counts

File Before After
syntax-agentic.md 397 345
syntax-checkout.md — 87 (new)
syntax-engine.md 98 98 (content replaced, same line count)
syntax.md 24 25

All edited/new files are under their target limits (syntax-*.md < 400 lines, syntax.md < 100 lines).

Duplication audit

No duplicate checkout: field reference existed elsewhere — other files (create-agentic-workflow.md, reuse.md, safe-outputs-management.md, syntax-tools-imports.md) only contain brief usage mentions/examples, left as-is. Confirmed no other .github/aw/*.md file repeats the stale "required allowed-models" wording after the fix.

Edge cases not pursued

Many other post-release commits touched docs/ (model-routing reference rewrite, work-queue glossary terms, linter-factory doc unbloat, etc.) but those are documentation-site content, not .github/aw/ agent instructions, and didn't surface additional accuracy gaps on inspection.

🤖 Generated with [Claude Code]((claude.com/redacted)

Generated by 🧹 Instructions Janitor · claude · sonnet50 · 164.9 AIC · ⌖ 21.1 AIC · ⊞ 6.7K · ◷

  • expires on Oct 17, 2026, 12:49 AM UTC-08:00

- Document the new multi-repository checkout allowlist (`multi`,
  `allowed-repos`, `allow-private-repos`, `max-repos`) added in #68593/#68816,
  extracted into a dedicated syntax-checkout.md to keep syntax-agentic.md
  under its line budget and add it to the syntax.md index.
- Fix engine.model-routing docs: allowed-models is optional (not required),
  and document the omitted-value and import-merge semantics from #68495/#68883.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@pelikhan

Copy link
Copy Markdown
Collaborator

@copilot merge main and recompile

…sync-v0.91.12-89e8f9f5a9374d32

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Copilot AI commented Oct 15, 2026

Copy link
Copy Markdown
Contributor

@copilot merge main and recompile

Merged main and recompiled all 344 workflows successfully. Merge commit: fa669a8.

Copilot AI requested a review from pelikhan October 15, 2026 13:16
@pelikhan

Copy link
Copy Markdown
Collaborator

@copilot run the script that updates the list of instructions files

Copilot AI and others added 2 commits October 15, 2026 13:28
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Copilot AI commented Oct 15, 2026

Copy link
Copy Markdown
Contributor

@copilot run the script that updates the list of instructions files

Ran the updater via make recompile; synced the embedded fallback and dispatcher list with syntax-checkout.md. All 344 workflows compiled successfully. Commit: a00b6aa.

@pelikhan
pelikhan merged commit fed828c into main Oct 15, 2026
24 checks passed
@pelikhan
pelikhan deleted the instructions-janitor-sync-v0.91.12-89e8f9f5a9374d32 branch October 15, 2026 17:32
@github-actions

Copy link
Copy Markdown
Contributor Author

🎉 This pull request is included in a new release.

Release: v0.92.1

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

Labels

automation documentation Improvements or additions to documentation instructions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants