From a64163603b2de347a60fda63449fb634cb95390e Mon Sep 17 00:00:00 2001 From: Tim Biggs Date: Sat, 7 Nov 2026 20:06:15 +0000 Subject: [PATCH 1/2] Centralize independent review in console Keep project verification and shipping permissions local. Co-Authored-By: Codex --- AGENTS.md | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/AGENTS.md b/AGENTS.md index b919730ebe749..d97a2c491cd89 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -145,6 +145,19 @@ For mobile verification, a missing or outdated native client is an install step, `evals/` checks how real models behave with T3 Code's model-facing text: agent instructions, MCP tool descriptions and schemas, and generated titles. Before you change that text, read [evals/README.md](evals/README.md) and run the evals it names. When you fix a model behavior bug, add a case that fails before the fix. +## Tim's fork: review process + +Use the console-managed `review` skill for independent code review and its shared +process; use `council` only when explicitly requested. Load the installed skill from +`~/.claude/skills/review/SKILL.md` or `~/.codex/skills/review/SKILL.md`, using this +machine's home directory. The project's checks, product invariants and shipping +permissions in this repository still apply. + +Focus review on contracts, permissions, persistence and the affected clients, +providers and connection modes. Keep `check:changed` and existing CI requirements; +do not add repo-wide local checks. Target only `tebiggs/t3code`. Tim still reviews +and merges ordinary fork PRs; the existing daily-sync exception remains separate. + ## Pull requests - Never make a PR unless the developer explicitly asks you to do so. From c1180d492f58559888b488e0cdda747c4f7399df Mon Sep 17 00:00:00 2001 From: Tim Biggs Date: Sun, 8 Nov 2026 02:21:46 +0000 Subject: [PATCH 2/2] Keep review mechanics in the shared skill Use short project triggers and maintainer roles instead of personal names. Co-Authored-By: Codex --- AGENTS.md | 18 ++++++------------ 1 file changed, 6 insertions(+), 12 deletions(-) diff --git a/AGENTS.md b/AGENTS.md index d97a2c491cd89..eca79ab4baf21 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -145,18 +145,12 @@ For mobile verification, a missing or outdated native client is an install step, `evals/` checks how real models behave with T3 Code's model-facing text: agent instructions, MCP tool descriptions and schemas, and generated titles. Before you change that text, read [evals/README.md](evals/README.md) and run the evals it names. When you fix a model behavior bug, add a case that fails before the fix. -## Tim's fork: review process - -Use the console-managed `review` skill for independent code review and its shared -process; use `council` only when explicitly requested. Load the installed skill from -`~/.claude/skills/review/SKILL.md` or `~/.codex/skills/review/SKILL.md`, using this -machine's home directory. The project's checks, product invariants and shipping -permissions in this repository still apply. - -Focus review on contracts, permissions, persistence and the affected clients, -providers and connection modes. Keep `check:changed` and existing CI requirements; -do not add repo-wide local checks. Target only `tebiggs/t3code`. Tim still reviews -and merges ordinary fork PRs; the existing daily-sync exception remains separate. +## Review + +Before opening a PR, run the `review` skill. + +PRs target `tebiggs/t3code`; the maintainer reviews and merges them. The existing +daily-sync exception remains separate. ## Pull requests