chore(ai-tooling): standardize agent model policy

This commit is contained in:
Tommaso Casaburi
2026-04-08 18:35:08 +07:00
parent f14d0562cd
commit d5e00a241e
10 changed files with 20 additions and 8 deletions
+1 -1
View File
@@ -1,4 +1,4 @@
model = "gpt-5.3-codex-spark" model = "gpt-5.4"
model_reasoning_effort = "medium" model_reasoning_effort = "medium"
sandbox_mode = "read-only" sandbox_mode = "read-only"
developer_instructions = """ developer_instructions = """
+1 -1
View File
@@ -1,4 +1,4 @@
model = "gpt-5.3-codex" model = "gpt-5.4"
model_reasoning_effort = "medium" model_reasoning_effort = "medium"
developer_instructions = """ developer_instructions = """
Run the repo's required verification commands for the files or feature area the parent agent just changed, then fix only issues surfaced by those checks. Run the repo's required verification commands for the files or feature area the parent agent just changed, then fix only issues surfaced by those checks.
+1 -1
View File
@@ -1,4 +1,4 @@
model = "gpt-5.3-codex" model = "gpt-5.4"
model_reasoning_effort = "medium" model_reasoning_effort = "medium"
developer_instructions = """ developer_instructions = """
Implement only the concrete task or task slice assigned by the parent agent. Implement only the concrete task or task slice assigned by the parent agent.
+1 -1
View File
@@ -1,4 +1,4 @@
model = "gpt-5.3-codex-spark" model = "gpt-5.4"
model_reasoning_effort = "medium" model_reasoning_effort = "medium"
sandbox_mode = "read-only" sandbox_mode = "read-only"
developer_instructions = """ developer_instructions = """
+1 -1
View File
@@ -1,4 +1,4 @@
model = "gpt-5.3-codex" model = "gpt-5.4"
model_reasoning_effort = "medium" model_reasoning_effort = "medium"
developer_instructions = """ developer_instructions = """
Only act when the parent agent provides both the exact React Doctor diagnostic and a concrete fix plan. Only act when the parent agent provides both the exact React Doctor diagnostic and a concrete fix plan.
+1 -1
View File
@@ -1,4 +1,4 @@
model = "gpt-5.3-codex" model = "gpt-5.4"
model_reasoning_effort = "medium" model_reasoning_effort = "medium"
developer_instructions = """ developer_instructions = """
Review only the recently changed React files or the file set the parent agent names. Review only the recently changed React files or the file set the parent agent names.
+1 -1
View File
@@ -1,4 +1,4 @@
model = "gpt-5.3-codex" model = "gpt-5.4"
model_reasoning_effort = "medium" model_reasoning_effort = "medium"
developer_instructions = """ developer_instructions = """
Test only the Android workflow the parent agent asks about on the local emulator, leaving the emulator running when you finish unless the parent agent says otherwise. Test only the Android workflow the parent agent asks about on the local emulator, leaving the emulator running when you finish unless the parent agent says otherwise.
+1 -1
View File
@@ -1,4 +1,4 @@
model = "gpt-5.3-codex-spark" model = "gpt-5.4"
model_reasoning_effort = "medium" model_reasoning_effort = "medium"
developer_instructions = """ developer_instructions = """
Translate exactly one i18next key at a time into every supported language. Translate exactly one i18next key at a time into every supported language.
+2
View File
@@ -138,6 +138,8 @@ src/
- Treat `.codex/`, `.cursor/`, and `.claude/` as repo-managed contributor tooling, not private scratch space. - Treat `.codex/`, `.cursor/`, and `.claude/` as repo-managed contributor tooling, not private scratch space.
- Keep equivalent workflow files aligned across all toolchains when their directories contain the same skill, hook, or agent. - Keep equivalent workflow files aligned across all toolchains when their directories contain the same skill, hook, or agent.
- Do not configure `.claude` agents to use `composer-2`; that model is Cursor-only in this repo. Keep `.claude` agent models on Claude-supported options.
- Do not configure `.codex/agents/*.toml` with `gpt-5.3-codex` or `gpt-5.3-codex-spark`; standardize Codex agents on `gpt-5.4` unless the user explicitly requests a different model.
- When changing shared agent behavior, update the relevant files in `.codex/skills/`, `.cursor/skills/`, `.claude/skills/`, `.codex/agents/`, `.cursor/agents/`, `.claude/agents/`, `.codex/hooks/`, `.cursor/hooks/`, `.claude/hooks/`, and their `hooks.json` or config entry points as needed. - When changing shared agent behavior, update the relevant files in `.codex/skills/`, `.cursor/skills/`, `.claude/skills/`, `.codex/agents/`, `.cursor/agents/`, `.claude/agents/`, `.codex/hooks/`, `.cursor/hooks/`, `.claude/hooks/`, and their `hooks.json` or config entry points as needed.
- If `AGENTS.md` references a skill, agent, or hook, prefer a tracked file under `.codex/`, `.cursor/`, or `.claude/` rather than an untracked local-only instruction. - If `AGENTS.md` references a skill, agent, or hook, prefer a tracked file under `.codex/`, `.cursor/`, or `.claude/` rather than an untracked local-only instruction.
- Review `.codex/config.toml`, `.cursor/hooks.json`, and `.claude/hooks.json` before changing agent orchestration or hook behavior, because they are the entry points contributors will actually load. - Review `.codex/config.toml`, `.cursor/hooks.json`, and `.claude/hooks.json` before changing agent orchestration or hook behavior, because they are the entry points contributors will actually load.
+10
View File
@@ -77,3 +77,13 @@ If uncertain, ask the developer before adding an entry.
- **Impact:** Parallel 5chan branches can block each other even though Portless is meant to let them coexist safely. - **Impact:** Parallel 5chan branches can block each other even though Portless is meant to let them coexist safely.
- **Mitigation:** Keep Portless startup behind `scripts/start-dev.js`, which now uses a branch-scoped `*.5chan.localhost:1355` route outside the canonical case and automatically increments a `-2`, `-3`, ... suffix when that branch-scoped route is already occupied. - **Mitigation:** Keep Portless startup behind `scripts/start-dev.js`, which now uses a branch-scoped `*.5chan.localhost:1355` route outside the canonical case and automatically increments a `-2`, `-3`, ... suffix when that branch-scoped route is already occupied.
- **Status:** confirmed - **Status:** confirmed
### Toolchain model names are not interchangeable
- **Date:** 2026-04-08
- **Observed by:** contributor + Codex
- **Context:** Reviewing repo-managed agent configs under `.codex/agents`, `.cursor/agents`, and `.claude/agents`
- **What was surprising:** `composer-2` is only available for Cursor in this repo, while Codex agents using `gpt-5.3-codex` or `gpt-5.3-codex-spark` perform poorly enough that they should not be configured by default.
- **Impact:** Agents can silently inherit invalid or weak model settings, leading to broken subagent runs or degraded implementation quality.
- **Mitigation:** Keep `.cursor` agent configs on Cursor-supported models only, never use `composer-2` in `.claude`, and standardize `.codex/agents/*.toml` on `gpt-5.4` unless a contributor explicitly requests an override.
- **Status:** confirmed