feat(workflows): collapse oversized model catalogs in config/validate + config --models; ACP deps currency - #373
Merged
Conversation
Co-authored-by: acp-release-bot[bot] <246668977+acp-release-bot[bot]@users.noreply.github.com>
…e; add config --models Harnesses with large model catalogs (pi, opencode advertise hundreds) dumped every model id inline in the config/validate option tables and in --json, flooding an authoring agent's context. Any select above 24 leaf choices (in practice the model option) now renders as a grouped summary (total + per-provider/group counts) on both the human table and --json; small catalogs (claude, codex) are unchanged. The full list is reachable only via the new `config <harness> --models[=<filter>]`: bare prints the provider/group breakdown, =<provider|substring|/regex/> prints matching leaf ids. No unfiltered full-leaf dump on any surface. The collapse is applied only at the CLI print boundary; the in-memory report and probeHarnessConfig()/validateWorkflowScript() returns stay complete, so configOptions validation still checks every advertised model.
…d56827) into packages/codex-acp # Conflicts: # packages/codex-acp/CHANGELOG.md # packages/codex-acp/package-lock.json # packages/codex-acp/package.json
…eign-heads + changeset
…-sdk 0.3.235 override
VikashLoomba
enabled auto-merge
August 19, 2026 00:41
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Feature — config/validate model-list collapse (the ask)
Harnesses with large model catalogs (pi, opencode advertise hundreds) dumped every model id inline in the
config/validateoption tables and in--json, flooding an authoring agent's context. Now any select option above 24 leaf choices (in practice themodeloption) renders as a grouped summary (total + per-provider/group counts) on both the human table and--json; small catalogs (claude, codex, and every effort/mode/boolean option) are unchanged and print verbatim.The full list is reachable only through the new
config <harness> --models[=<filter>]: bare prints the provider/group breakdown (no leaf ids),=<provider|substring|/regex/>prints the matching leaf ids. There is deliberately no unfiltered full-leaf dump on any surface — neither--jsonnor--modelscan flood context. The collapse happens only at the CLI print boundary; the in-memory report and the programmaticprobeHarnessConfig()/validateWorkflowScript()returns stay complete, soconfigOptionsvalidation still checks against every advertised model.Docs moved with it: the authoring skill reference (+ regenerated
author-workflowprompt),docs/specs/config-options.md, the workflows README, and CLI help. New tests inpackages/workflows/test/config-model-collapse.test.ts(+ updatedconfig.test.tsfor the new pi--jsoncontract). Live-verified: opencode's 601-model catalog collapses to a one-line summary;--models=deepseekreturns the matching leaves; claude/codex unchanged.ACP dependency currency (rode along — the gate blocks all merges when stale)
Mechanical bumps, each classified against the runbook's surface map:
providers/*wire we consume is SDK-defined (unchanged at 1.3.0) and our client already re-issuesproviders/seton reconstructed connections. Docs citations moved.release 1.5.0(#409), a release-please version/CHANGELOG bump only, no source change.Dependency gate, attribution gate, manifest check, build, typecheck all green; full test matrix green (every package 0 fail); codex-acp fork suite 914 pass; pre-push live 4-backend e2e green.