Skip to content

fix(framework): leave model selection to the user - #989

Merged
blafourcade merged 3 commits into
nextfrom
codex/remove-model-prescriptions
Oct 10, 2026
Merged

blafourcade merged 3 commits into
nextfrom
codex/remove-model-prescriptions

Conversation

@alexsoyes

@alexsoyes alexsoyes commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

🎯 What & why

Bundled skills and agents override the user’s model with Sonnet or Opus. Remove those presets so execution uses the host’s model configuration.

🛠️ How it works

Remove model fields from five dev skills, both dev agents and the agent scaffold. Agent generation accepts a model only when the user requests one. Remove the marketplace tier table. General guidance on selecting lightweight agents and the token-optimization recipe stay unchanged. The evaluation harness passes --model only when explicitly supplied.

Telemetry records, provider schemas and compatibility fixtures retain model identifiers because they describe data or supported formats.

🧪 How to verify

  • node --test scripts/__tests__/model-selection.test.js: 3 passing tests; two reproduced the forced defaults before removal. The harness tests use a stub CLI and spend no tokens.
  • pnpm exec lefthook run pre-commit: 557 passing tests; documentation checks pass.
  • node scripts/check-architecture-rules.js: 352 governed files, zero violations.
  • Push hooks: CLI knip passes; CLI suite has 6,836 passing tests and 1 skipped.

✅ I certify

  • I DO CERTIFY I READ EACH LINE OF THE PULL REQUEST BECAUSE I AM A SOFTWARE ENGINEER, NOT A AI PUPPY.

@alexsoyes
alexsoyes marked this pull request as ready for review October 10, 2026 05:29
@alexsoyes
alexsoyes requested a review from a team as a code owner October 10, 2026 05:29
@blafourcade
blafourcade merged commit 41e9169 into next Oct 10, 2026
25 checks passed
@blafourcade
blafourcade deleted the codex/remove-model-prescriptions branch October 10, 2026 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants