riskrails 3.2.1 → 4.0.1

package/skills/project-update/history/blobs/235fb88393cfe20dd58b35fe9785cc45baf5ffe21a13d926652d76ec68cea8b7

@@ -0,0 +1,35 @@
+# Workflow: bugfix
+
+Something behaves wrongly. The goal is the smallest change that fixes it without
+changing anything else.
+
+> Who runs a stage is set by `pipeline_profile` in `policies/risk-tiers.json`.
+> The table names the agent for when a stage is delegated. In the default `solo`
+> profile T0–T2 run in direct mode — the session does every stage itself, in a
+> few lines, with cheap readers and one `sonnet` review at T2 — and T3+ run the
+> full pipeline with the delegations below.
+
+| Stage | Who | Notes |
+|---|---|---|
+| DISCOVERY | `ai-discovery` | reproduce first: find the exact path that produces the symptom |
+| CONTEXT | `ai-context` | include the *correct* behaviour and where it is defined |
+| IMPACT ANALYSIS | `ai-discovery` | who else depends on the current, wrong behaviour |
+| RISK CLASSIFICATION | `ai-risk` | a bug in a payment path is still T4 |
+| PLAN | `ai-planner` | failing test first, then the fix |
+| IMPLEMENTATION | the session | the failing test is its own step |
+| TEST | the session (`ai-tester` in `team`) | the new test is shown failing before the fix; each step runs only its own scoped tests; the full verification command runs once after the last step, to the end, then the e2e suite once; every failure fixed as one batch |
+| ADVERSARIAL REVIEW | `ai-reviewer` | T2 and above |
+| SECURITY REVIEW | `ai-security` | if the bug was a security bug, always |
+| RELEASE REPORT | `ai-release` | |
+| HUMAN APPROVAL | the human | |
+
+## Specific to this workflow
+
+- **Write the failing test first.** A bugfix without a test that failed before it
+  is a bugfix nobody can prove.
+- Find out *why* the bug exists before fixing it. A bug that is deliberate
+  behaviour somebody depends on is not a bug; it is a requirements conflict, and
+  it goes back to the human.
+- Resist the urge to fix the surrounding code. Note it, fix the bug.
+- If the same bug exists in three places, fix the one that was reported and list
+  the other two in `project/known-risks.md`.

package/skills/project-update/history/blobs/3918b4ee011d9477b9d2779e2893ab599749c406d4b09439a9748b64fbff58f3

@@ -0,0 +1,18 @@
+# Verification — this project's commands
+<!-- Yours. Read by state.py verify and close, setup.py check, update.py hints; /rr-setup --update never rewrites it.
+     Each line is `field: value`; a line left empty reads as "not written down yet", `none` says the project has none.
+     What each command must do is in policies/testing.md (Verification). -->
+
+verify_command:      # the full fast suite, e2e excluded, no fail-fast — e.g. make test | composer qa && vendor/bin/phpunit --exclude-group e2e
+step_test_command:   # the same runner scoped to a path or filter, `{files}` replaced by the step's paths — e.g. vendor/bin/phpunit {files} | pytest {files}
+e2e_command:         # the end-to-end suite, run once at the end of a task — e.g. make e2e | vendor/bin/behat | npx playwright test — `none` if the project has none
+lint_command:        # e.g. vendor/bin/php-cs-fixer fix --dry-run | ruff check . | npm run lint — `none` if the project has none
+typecheck_command:   # e.g. vendor/bin/phpstan analyse --no-progress | mypy . | npx tsc --noEmit — `none` if the project has none
+single_test:         # how to run one test, for the failing-test-first step of a bugfix
+doc_tests:           # the suites a documents-only edit runs after a green suite — e.g. tests/test_docs.py; `none` if the project has none
+bite_codes:          # exit codes that mean "a test failed" (default 1) — e.g. 1,2 for pytest and phpunit, 101 for cargo
+healthy_output:      # two or three lines of what a green run ends with
+
+## Project specifics
+
+<!-- How to run unit tests, integration tests, a single test. Where fixtures live. What the CI runs. Cite the config. -->

package/skills/project-update/history/blobs/449273200dfec2a56885c881ff34d40e8ecc00da53c24c2d09b7fbeb1004d255

@@ -0,0 +1,40 @@
+# Architecture
+
+<!-- Filled by /rr-setup. Describe what the code does, not what the framework's
+     documentation says it should do. -->
+
+## Style
+
+<!-- Layered, hexagonal, modular monolith, service-oriented, or — most often —
+     several of these in different parts of the code. Say which is where. -->
+
+## Entry points
+
+<!-- HTTP routes, console commands, message consumers, cron jobs, webhooks.
+     KNOWN FACT with the file that defines each. -->
+
+## Request and message flow
+
+<!-- One paragraph per major flow: what comes in, what it touches, what it
+     writes, what goes out. -->
+
+## Module boundaries
+
+<!-- Where the seams are, and where they leak. -->
+
+## Where state lives
+
+<!-- Database, cache, queue, session, external systems, filesystem. -->
+
+## Cross-cutting mechanisms
+
+<!-- Events, the command bus, state machines, workflows, the DI container,
+     configuration and how it is layered per environment. -->
+
+## Contradictions found
+
+<!-- Documented behaviour: …
+     Observed behaviour: …
+     Evidence: …
+     Risk: …
+     One block per contradiction. Do not resolve it here. -->

package/skills/project-update/history/blobs/4c45aee9b9fc5f6b4e1d9739e434c4094975a5a8dec6b9703fb25f3080680c77

@@ -0,0 +1,124 @@
+# Testing policy
+
+Tests describe behaviour. Coverage percentage is a side effect, not a goal.
+
+## Characterization first
+
+Before changing behaviour that is not covered by a test — which is most legacy
+behaviour — write a test that passes against the code **as it is today**. That
+test is the definition of what must not break. Only then change the code.
+
+```
+characterization test → refactor → verify identical behaviour → feature change
+```
+
+Never do those in one step, and never in one commit.
+
+## What to consider, by kind of change
+
+| Change touches | Also test |
+|---|---|
+| a shared service | the existing callers, not only the new one |
+| money, tax, fiscal | rounding, currency, negative and zero amounts, refunds |
+| a state machine | every transition that is now reachable, and the ones that must stay unreachable |
+| an async consumer | retry, double delivery, out-of-order delivery, poison message |
+| an external integration | timeout, 500, malformed response, partial write |
+| a write path | idempotency, concurrency, transaction boundaries |
+| authorization | the negative case, for every role |
+| a public API or event | the old shape still works |
+
+## Three scopes, three moments
+
+A task runs tests three times at most, and each run has its own scope. Nothing
+else runs in between.
+
+| When | What runs | Command |
+|---|---|---|
+| after each implementation step, before `step-done` | **only the tests the plan named for that step** — the files it touched, their direct callers' tests | `step_test_command` |
+| once, after the **last** step | the full fast suite — unit and integration, no e2e | `verify_command` |
+| once, at the **end of the task**, after the full suite is green | the end-to-end suite | `e2e_command` |
+
+- A step never runs the full suite, and **never** runs e2e. If the plan named
+  no test for a step, nothing runs for it.
+- A step's tests are scoped by path or filter, not by `--stop-on-failure`: the
+  scoped run still goes to the end, and its failures are fixed inside that step.
+- The e2e suite runs **once per task**, at the very end, never per step. Skip it
+  at T0/T1, and when the change cannot reach a flow the e2e suite covers — say
+  which, in one line; silence is not a verdict.
+- A failure in a step's own tests is fixed in that step, before `step-done`;
+  the batch rule below is for the two end-of-task runs.
+- After a green suite, a documents-only edit runs `doc_tests` (step scope), not the suite.
+
+## Reading a failure
+
+**Run to the end, then fix as one batch.** The verification command runs with
+no fail-fast flag and is never interrupted at the first failure. Collect every
+failure, classify each one, fix all the new regressions in one batch
+(`state.py start --amend` widens the task when a fix needs more files), and run the
+command once more. One failure never
+sends the task back to the start: the plan, the tier and the finished steps
+stand. At most two remediation rounds; a third means the human decides.
+
+Classify before fixing. The four answers are:
+
+- **EXISTING TEST FAILURE** — it failed before this change too. Say so, do not
+  fix it inside this task.
+- **NEW REGRESSION** — this change broke it. Fix the code, not the test.
+- **TEST ENVIRONMENT FAILURE** — database, fixtures, network, container. Say what
+  is missing.
+- **UNKNOWN** — you could not tell. Say what you tried.
+
+A review's MEDIUM and LOW findings are recorded with `state.py memory risk add`, not
+fixed in the task; only a BLOCKER, a HIGH or a regression buys an `--after-fix` run.
+
+**Never edit a test to make it pass when production behaviour changed
+unexpectedly.** A test that suddenly disagrees with the code is evidence, and
+deleting evidence is the worst available option.
+
+## Verification
+
+Three commands, one per scope. `verify_command` proves the project is healthy;
+`step_test_command` proves one step is; `e2e_command` proves the whole flow
+still works. Keep each to one command — chain the pieces in a Makefile or
+composer script if there are several — and make sure none of them stops at the
+first failure (`--stop-on-failure`, `-x`, `--bail`, `failfast` are off).
+
+`verify_command` must **exclude** the e2e suite: it is the fast loop run after
+the last step and again after a remediation batch. `e2e_command` runs once, at
+the end of the task. A bugfix still shows its new failing test first — that one
+test only, through `single_test`.
+
+The commands themselves are written in `policies/verification.md` (`verify_command`,
+`step_test_command`, `e2e_command`, `lint_command`, `typecheck_command`, `single_test`,
+`doc_tests`, `bite_codes`, `healthy_output`): a human fills them in, `state.py` reads them from
+there, and `/rr-setup --update` never rewrites that file.
+
+`step_test_command` may contain `{files}`, which is replaced by the paths the
+step names; without it the paths are appended. The placeholder is what lets
+`sensors.py` run a step's own tests for the "test must bite" check, so a runner
+that only takes a `--filter` gets `bite: unavailable` rather than a wrong answer.
+
+`bite_codes` (optional, in `policies/verification.md`) lists the exit codes that mean "a test failed" when a
+test is shown failing before its fix; the default is `1`. Runners that exit
+differently say so here — `bite_codes: 1,2` for pytest (2 is a collection
+error, such as a missing import), `1,2` for phpunit, `101` for cargo. Any other
+code is the runner's own usage error and does not count as a bite.
+
+`lint_command` and `typecheck_command` are read by the deterministic sensors,
+never by a model. They decide, together with the test run, whether a light-mode change
+can skip its model review (`state.py close` measures them). A **missing** line is
+`unavailable` and keeps the review; only the literal `none`, written by a human,
+says the project has no such tool. Read the exact line from the project's
+config files (`package.json`, `composer.json`, `pyproject.toml`, `Makefile`), and
+never run a command just to find it out.
+
+If the project has no e2e suite, write `e2e_command: none` and say so — an
+empty line is read as "not written down yet" and the next task will go looking
+for it. Say which group, tag or directory marks the e2e tests and how
+`verify_command` excludes them, so a step's scoped run and the fast suite stay
+free of them.
+
+## Project specifics
+
+<!-- The real commands: how to run unit tests, integration tests, a single test.
+     Where fixtures live. What the CI runs. Cite the config. -->

package/skills/project-update/history/blobs/4dd3fa12bc45acc8c836ab65d790248ca7504a021bca6677499d17bb3d798ed9

@@ -0,0 +1,64 @@
+# Manager
+
+The manager is the main session, not a subagent. It orchestrates; it does not
+collect facts itself, and it keeps its own context small so it can stay in the
+task until the end.
+
+The role is the same whichever runtime the session is: Claude Code or Codex. The
+agents it delegates to carry the same names in both, and the tier each runs at is
+the same; only the model behind that tier differs.
+
+## Responsibilities
+
+- Understand the request and restate it in one sentence the human can correct.
+- Classify the task: which workflow, which risk tier.
+- Establish scope, and defend it.
+- Choose the stages the tier requires, and the agent for each.
+- Do inline what the pipeline profile leaves inline — in a few lines — and
+  delegate the rest. In the `solo` profile T0–T2 are direct mode: the
+  developer's request and `grep -n` are the context, the plan is a few lines in
+  the conversation, and a subagent is spawned only as a cheap reader, for the
+  one BALANCED review at T2, or on a trigger from `policies/risk-tiers.json`.
+  From T3 the full pipeline runs, stage by stage. Never read logs, test output
+  or large files into its own context.
+- Run the tests once, to the end, and fix every failure as one batch in a
+  `state.py remediate` step. One failure never restarts the task.
+- Split the work into steps small enough that each one names the files it touches.
+- Track state through `state.py` after every stage. The state file, not the
+  conversation, is what survives.
+- Decide when to escalate a model tier, one task at a time, against the triggers
+  in `policies/risk-tiers.json`, and name the trigger when it does.
+- Record, in every artifact, the tier **and the model that actually ran** — see
+  `policies/model-routing.md`. A gate may have moved an agent down a tier, and a
+  Codex agent file may override the model asked for at spawn time.
+- Write the final summary the human reads.
+
+## What the manager must not do
+
+- Skip a stage because the task looks easy. Make it lightweight instead, and
+  record that it happened.
+- Expand scope. A discovered problem goes into `project/known-risks.md`.
+- Delegate the conclusion. The synthesis a human reads is written by the session.
+- Commit, push, merge or deploy. The pipeline ends at human approval.
+
+## Implementation
+
+The manager implements approved steps itself, under this policy and under
+`ai-scope-guard`. `ai-implementer` exists for mechanical, pattern-copying steps
+and for when the human asks for it — not as the default.
+
+## Questions, and who may answer them
+
+The manager is the only role that asks the human, and `state.py` is the only
+thing that writes the questions file — the path guard denies an edit to it.
+
+- A subagent that cannot continue returns a `QUESTIONS_NEEDED` section instead of
+  guessing. Convert it, unchanged in meaning, with `state.py ask --batch`, and
+  render it for the human with `state.py questions --pending --format md`.
+- Record the human's reply with `state.py answer Q1=B Q2:"free text"` (or
+  `--prose "1B 2A"`). Never hand-edit the file, and never answer on their behalf.
+- While a question is pending, every stage-moving command exits 4. That is the
+  point: the pipeline stops rather than drifting on an assumption.
+- Human approval is not the manager's to grant. `state.py approve` refuses
+  without a terminal, an `AI_UNATTENDED` launcher, or an `[Answer]:` filled into
+  the gate question by the human. Print the exact command and stop.

package/skills/project-update/history/blobs/51e3f15095d7123ea8566b512479ee10a6cb2f336b7352a259a083a887bd7116

@@ -0,0 +1,33 @@
+# Workflow: feature
+
+New behaviour that someone asked for. The default workflow.
+
+> Who runs a stage is set by `pipeline_profile` in `policies/risk-tiers.json`.
+> The table names the agent for when a stage is delegated. In the default `solo`
+> profile T0–T2 run in direct mode — the session does every stage itself, in a
+> few lines, with cheap readers and one BALANCED review at T2 — and T3+ run the
+> full pipeline with the delegations below.
+
+| Stage | Who | Notes |
+|---|---|---|
+| DISCOVERY | `ai-indexer` then `ai-discovery` | where the feature lands, what it will touch |
+| CONTEXT | `ai-context` | the structured summary everything downstream reads |
+| IMPACT ANALYSIS | `ai-discovery` (second pass) | callers, data, contracts, other environments |
+| RISK CLASSIFICATION | `ai-risk` | one tier, from `policies/risk-tiers.json` |
+| PLAN | none at T0/T1; the session at T2; `ai-planner` from T3 | steps with `allowed_files`; STRONG for T3/T4, `ai-expert` for T5 |
+| PLAN REVIEW | `ai-reviewer` | T3 and above |
+| IMPLEMENTATION | the session, one step at a time | scope-guarded |
+| TEST | the session (`ai-tester` in `team`) | per step, only that step's tests; the full suite once after the last step, to the end, then the e2e suite once at the end of the task; every failure fixed as one batch, then one more run |
+| ADVERSARIAL REVIEW | `ai-reviewer` | T2 and above |
+| SECURITY REVIEW | `ai-security` | T4, T5, and anything touching auth or personal data |
+| RELEASE REPORT | `ai-release` | full report from T2 up |
+| HUMAN APPROVAL | the human | the pipeline stops here |
+
+## Specific to this workflow
+
+- The plan states what stays the same, not only what changes. A feature that
+  quietly alters an existing behaviour is the most common source of regressions.
+- New code paths need their negative cases tested: not authorized, not found,
+  invalid input, external system down.
+- If the feature needs a new dependency, that is a decision worth its own line in
+  the plan.

package/skills/project-update/history/blobs/5497a204967bea7dab29079a6e1dfa3d8713b2d2cb294c2779378039f03a55fc

@@ -0,0 +1,36 @@
+# Workflow: bugfix
+
+Something behaves wrongly. The goal is the smallest change that fixes it without
+changing anything else.
+
+> Who runs a stage is set by `pipeline_profile` in `policies/risk-tiers.json`.
+> The table names the agent for when a stage is delegated. In the default `solo`
+> profile T0–T4 run in direct mode — the session does every stage itself, in a
+> few lines, with cheap readers and one review (BALANCED at T2/T3, and only
+> when a sensor is red; STRONG at T4, whose plan the human approves in plan
+> mode) — and T5+ run the full pipeline with the delegations below.
+
+| Stage | Who | Notes |
+|---|---|---|
+| DISCOVERY | `ai-discovery` | reproduce first: find the exact path that produces the symptom |
+| CONTEXT | `ai-context` | include the *correct* behaviour and where it is defined |
+| IMPACT ANALYSIS | `ai-discovery` | who else depends on the current, wrong behaviour |
+| RISK CLASSIFICATION | `ai-risk` | a bug in a payment path is still T5 |
+| PLAN | `ai-planner` | failing test first, then the fix |
+| IMPLEMENTATION | the session | the failing test is its own step |
+| TEST | the session (`ai-tester` in `team`) | the new test is shown failing before the fix; each step runs only its own scoped tests; the full verification command runs once after the last step, to the end, then the e2e suite once; every failure fixed as one batch |
+| ADVERSARIAL REVIEW | `ai-reviewer` | T2 and above; up to T3 the sensors stand in when all are green |
+| SECURITY REVIEW | `ai-security` | if the bug was a security bug, always |
+| RELEASE REPORT | `ai-release` | |
+| HUMAN APPROVAL | the human | |
+
+## Specific to this workflow
+
+- **Write the failing test first.** A bugfix without a test that failed before it
+  is a bugfix nobody can prove.
+- Find out *why* the bug exists before fixing it. A bug that is deliberate
+  behaviour somebody depends on is not a bug; it is a requirements conflict, and
+  it goes back to the human.
+- Resist the urge to fix the surrounding code. Note it, fix the bug.
+- If the same bug exists in three places, fix the one that was reported and list
+  the other two in `project/known-risks.md`.

package/skills/project-update/history/blobs/60c4d7e7fde7049d92d84295fc13ca9a91eb2de423be08f093b3ee5d5e817ff8

@@ -0,0 +1,64 @@
+# Manager
+
+The manager is the main session, not a subagent. It orchestrates; it does not
+collect facts itself, and it keeps its own context small so it can stay in the
+task until the end.
+
+The role is the same whichever runtime the session is: Claude Code or Codex. The
+agents it delegates to carry the same names in both, and the tier each runs at is
+the same; only the model behind that tier differs.
+
+## Responsibilities
+
+- Understand the request and restate it in one sentence the human can correct.
+- Classify the task: which workflow, which risk tier.
+- Establish scope, and defend it.
+- Choose the stages the tier requires, and the agent for each.
+- Do inline what the pipeline profile leaves inline — in a few lines — and
+  delegate the rest. In the `solo` profile T0–T2 are direct mode: the
+  developer's request and `grep -n` are the context, the plan is a few lines in
+  the conversation, and a subagent is spawned only as a cheap reader, for the
+  one `sonnet` review at T2, or on a trigger from `policies/risk-tiers.json`.
+  From T3 the full pipeline runs, stage by stage. Never read logs, test output
+  or large files into its own context.
+- Run the tests once, to the end, and fix every failure as one batch in a
+  `state.py remediate` step. One failure never restarts the task.
+- Split the work into steps small enough that each one names the files it touches.
+- Track state through `state.py` after every stage. The state file, not the
+  conversation, is what survives.
+- Decide when to escalate a model tier, one task at a time, against the triggers
+  in `policies/risk-tiers.json`, and name the trigger when it does.
+- Record, in every artifact, the tier **and the model that actually ran** — see
+  `policies/model-routing.md`. A gate may have moved an agent down a tier, and a
+  Codex agent file may override the model asked for at spawn time.
+- Write the final summary the human reads.
+
+## What the manager must not do
+
+- Skip a stage because the task looks easy. Make it lightweight instead, and
+  record that it happened.
+- Expand scope. A discovered problem goes into `project/known-risks.md`.
+- Delegate the conclusion. The synthesis a human reads is written by the session.
+- Commit, push, merge or deploy. The pipeline ends at human approval.
+
+## Implementation
+
+The manager implements approved steps itself, under this policy and under
+`ai-scope-guard`. `ai-implementer` exists for mechanical, pattern-copying steps
+and for when the human asks for it — not as the default.
+
+## Questions, and who may answer them
+
+The manager is the only role that asks the human, and `state.py` is the only
+thing that writes the questions file — the path guard denies an edit to it.
+
+- A subagent that cannot continue returns a `QUESTIONS_NEEDED` section instead of
+  guessing. Convert it, unchanged in meaning, with `state.py ask --batch`, and
+  render it for the human with `state.py questions --pending --format md`.
+- Record the human's reply with `state.py answer Q1=B Q2:"free text"` (or
+  `--prose "1B 2A"`). Never hand-edit the file, and never answer on their behalf.
+- While a question is pending, every stage-moving command exits 4. That is the
+  point: the pipeline stops rather than drifting on an assumption.
+- Human approval is not the manager's to grant. `state.py approve` refuses
+  without a terminal, an `AI_UNATTENDED` launcher, or an `[Answer]:` filled into
+  the gate question by the human. Print the exact command and stop.

package/skills/project-update/history/blobs/6221b7c14cc04634e485e562d464c6860d45666a7f63e293c06bb2944ff556cc

@@ -0,0 +1,27 @@
+# Project overview
+
+<!-- Filled by /rr-setup from the repository. Every statement carries a label:
+     KNOWN FACT (with file:line) · INFERENCE (with what it rests on) ·
+     UNKNOWN (with what would answer it) · RISK (with what could go wrong). -->
+
+## What this system does
+
+## Stack
+
+<!-- Languages, frameworks and their versions, package managers. KNOWN FACT with
+     the manifest that proves each one. -->
+
+## How it runs
+
+<!-- Local: containers, services, entry points. Production: where, how deployed. -->
+
+## Repository layout
+
+<!-- The directories that matter and what lives in each. Skip the obvious. -->
+
+## Team and process context
+
+<!-- Supplied by the human, not discoverable from the code: team size, who owns
+     what, the tracker, regulatory constraints, release cadence. -->
+
+## Unknowns

package/skills/project-update/history/blobs/637971cfb844ef4285e2886a4b1c834fb2ed5c46e773adb88c2c147fea626ad3

@@ -0,0 +1,33 @@
+# Workflow: hotfix
+
+Production is broken now. Stages get shorter; none of them disappears.
+
+> Who runs a stage is set by `pipeline_profile` in `policies/risk-tiers.json`.
+> The table names the agent for when a stage is delegated; in the default `solo`
+> profile the session does the stage inline up to the tier where the profile
+> delegates it.
+
+| Stage | Who | Notes |
+|---|---|---|
+| DISCOVERY | `ai-discovery` | narrow: the failing path only |
+| CONTEXT | the session, inline | a paragraph, recorded in the state file |
+| RISK CLASSIFICATION | `ai-risk` | the tier of the code being touched, not of the urgency |
+| PLAN | `ai-planner` or the session | the smallest change that stops the bleeding |
+| IMPLEMENTATION | the session | one step |
+| TEST | `ai-tester` | at minimum, a test that reproduces the incident |
+| ADVERSARIAL REVIEW | `ai-reviewer` | short, focused on "what else does this touch" |
+| SECURITY REVIEW | `ai-security` | if the incident is a security incident |
+| RELEASE REPORT | `ai-release` | short form, but rollback is mandatory |
+| HUMAN APPROVAL | the human | always, and usually faster than the rest |
+
+## Rules
+
+- Urgency lowers the *depth* of a stage, never its existence. The risk tier is a
+  property of the code, not of how loudly someone is asking.
+- A hotfix is the smallest possible change. Everything else — the real fix, the
+  cleanup, the missing test — becomes a follow-up task, written down before the
+  hotfix is approved so it does not evaporate when the incident closes.
+- Record what actually happened in `project/known-risks.md`: the trigger, the
+  detection, and what would have caught it earlier.
+- Never disable a test or a check to get a hotfix out. If CI is the obstacle, a
+  human decides that, explicitly.

package/skills/project-update/history/blobs/65af2282eea2a51fcc27a4acf2c26c727b0961c30e1be13176369200c460c219

@@ -0,0 +1,34 @@
+# Workflow: feature
+
+New behaviour that someone asked for. The default workflow.
+
+> Who runs a stage is set by `pipeline_profile` in `policies/risk-tiers.json`.
+> The table names the agent for when a stage is delegated. In the default `solo`
+> profile T0–T4 run in direct mode — the session does every stage itself, in a
+> few lines, with cheap readers and one review (BALANCED at T2/T3, and only
+> when a sensor is red; STRONG at T4, whose plan the human approves in plan
+> mode) — and T5+ run the full pipeline with the delegations below.
+
+| Stage | Who | Notes |
+|---|---|---|
+| DISCOVERY | `ai-indexer` then `ai-discovery` | where the feature lands, what it will touch |
+| CONTEXT | `ai-context` | the structured summary everything downstream reads |
+| IMPACT ANALYSIS | `ai-discovery` (second pass) | callers, data, contracts, other environments |
+| RISK CLASSIFICATION | `ai-risk` | one tier, from `policies/risk-tiers.json` |
+| PLAN | none at T0/T1; the session at T2/T3, in plan mode at T4; `ai-planner` from T5 | steps with `allowed_files`; STRONG for T5, `ai-expert` for T6 |
+| PLAN REVIEW | the human at T4 (plan mode); `ai-reviewer` from T5 | |
+| IMPLEMENTATION | the session, one step at a time | scope-guarded |
+| TEST | the session (`ai-tester` in `team`) | per step, only that step's tests; the full suite once after the last step, to the end, then the e2e suite once at the end of the task; every failure fixed as one batch, then one more run |
+| ADVERSARIAL REVIEW | `ai-reviewer` | T2 and above; up to T3 the sensors stand in when all are green |
+| SECURITY REVIEW | `ai-security` | T5, T6, and anything touching auth or personal data |
+| RELEASE REPORT | `ai-release` | full report from T2 up |
+| HUMAN APPROVAL | the human | the pipeline stops here |
+
+## Specific to this workflow
+
+- The plan states what stays the same, not only what changes. A feature that
+  quietly alters an existing behaviour is the most common source of regressions.
+- New code paths need their negative cases tested: not authorized, not found,
+  invalid input, external system down.
+- If the feature needs a new dependency, that is a decision worth its own line in
+  the plan.

package/skills/project-update/history/blobs/66ecd73e303e7bbc1aa8ee72435d0d10812ff7b2eff0adac59bbf0a303e2cc01

@@ -0,0 +1,41 @@
+# Manager
+
+The manager is the main session, not a subagent. It orchestrates; it does not
+collect facts itself, and it keeps its own context small so it can stay in the
+task until the end.
+
+The role is the same whichever runtime the session is: Claude Code or Codex. The
+agents it delegates to carry the same names in both, and the tier each runs at is
+the same; only the model behind that tier differs.
+
+## Responsibilities
+
+- Understand the request and restate it in one sentence the human can correct.
+- Classify the task: which workflow, which risk tier.
+- Establish scope, and defend it.
+- Choose the stages the tier requires, and the agent for each.
+- Delegate every reading and analysis stage. Never read logs, test output or
+  large files into its own context.
+- Split the work into steps small enough that each one names the files it touches.
+- Track state through `state.py` after every stage. The state file, not the
+  conversation, is what survives.
+- Decide when to escalate a model tier, one task at a time, against the triggers
+  in `policies/risk-tiers.json`, and name the trigger when it does.
+- Record, in every artifact, the tier **and the model that actually ran** — see
+  `policies/model-routing.md`. A gate may have moved an agent down a tier, and a
+  Codex agent file may override the model asked for at spawn time.
+- Write the final summary the human reads.
+
+## What the manager must not do
+
+- Skip a stage because the task looks easy. Make it lightweight instead, and
+  record that it happened.
+- Expand scope. A discovered problem goes into `project/known-risks.md`.
+- Delegate the conclusion. The synthesis a human reads is written by the session.
+- Commit, push, merge or deploy. The pipeline ends at human approval.
+
+## Implementation
+
+The manager implements approved steps itself, under this policy and under
+`ai-scope-guard`. `ai-implementer` exists for mechanical, pattern-copying steps
+and for when the human asks for it — not as the default.

package/skills/project-update/history/blobs/6a43bf32461b22b87fafe2865529dc04fa66cd00182a2a7598aa071667ca255a

@@ -0,0 +1,35 @@
+# Workflow: hotfix
+
+Production is broken now. Stages get shorter; none of them disappears.
+
+> Who runs a stage is set by `pipeline_profile` in `policies/risk-tiers.json`.
+> The table names the agent for when a stage is delegated. In the default `solo`
+> profile T0–T4 run in direct mode — the session does every stage itself, in a
+> few lines, with cheap readers and one review (BALANCED at T2/T3, and only
+> when a sensor is red; STRONG at T4, whose plan the human approves in plan
+> mode) — and T5+ run the full pipeline with the delegations below.
+
+| Stage | Who | Notes |
+|---|---|---|
+| DISCOVERY | `ai-discovery` | narrow: the failing path only |
+| CONTEXT | the session, inline | a paragraph, recorded in the state file |
+| RISK CLASSIFICATION | `ai-risk` | the tier of the code being touched, not of the urgency |
+| PLAN | `ai-planner` or the session | the smallest change that stops the bleeding |
+| IMPLEMENTATION | the session | one step |
+| TEST | the session (`ai-tester` in `team`) | at minimum, a test that reproduces the incident, run scoped; the full command once after the fix, then e2e only if the incident is in a flow it covers; failures fixed as one batch |
+| ADVERSARIAL REVIEW | `ai-reviewer` | short, focused on "what else does this touch" |
+| SECURITY REVIEW | `ai-security` | if the incident is a security incident |
+| RELEASE REPORT | `ai-release` | short form, but rollback is mandatory |
+| HUMAN APPROVAL | the human | always, and usually faster than the rest |
+
+## Rules
+
+- Urgency lowers the *depth* of a stage, never its existence. The risk tier is a
+  property of the code, not of how loudly someone is asking.
+- A hotfix is the smallest possible change. Everything else — the real fix, the
+  cleanup, the missing test — becomes a follow-up task, written down before the
+  hotfix is approved so it does not evaporate when the incident closes.
+- Record what actually happened in `project/known-risks.md`: the trigger, the
+  detection, and what would have caught it earlier.
+- Never disable a test or a check to get a hotfix out. If CI is the obstacle, a
+  human decides that, explicitly.

package/skills/project-update/history/blobs/6d6dee8634abf9ac057bb206828b83a893a2afcf4f86bbe69e839a3b47d0adde

@@ -0,0 +1,48 @@
+{
+  "version": 1,
+  "axes": {
+    "scope": {"measure": "references", "bands": [0, 5], "hot_paths": ["hooks/**", "src/Shared/**"]},
+    "behaviour": {"measure": "session",
+                  "keywords": ["workflow", "transition", "state machine", "messenger", "queue", "async", "transaction"]},
+    "reversibility": {"measure": "paths+content",
+                      "needs_step": ["**/migrations/**", "**/Migrations/**", "config/**", "k8s/**", "**/env/**"],
+                      "irreversible": ["**/prod/**", "**/production/**", "deploy/**", "**/*.deploy.*"],
+                      "irreversible_content": ["DROP ", "TRUNCATE", "DELETE FROM", "ALTER .* DROP", "RENAME"]},
+    "domain": {"measure": "modules+session",
+               "modules": ["**/Payment/**", "**/Tax/**", "**/Invoice/**", "**/Security/**", "**/account/**"],
+               "customer_visible": ["templates/**", "**/Shop/**", "**/Resources/views/**"]},
+    "novelty": {"measure": "session",
+                "manifests": ["composer.json", "requirements*.txt", "package.json", "pyproject.toml", "Gemfile"]},
+    "coverage": {"measure": "tests-map",
+                 "code_globs": ["**/*.php", "**/*.py", "**/*.js", "**/*.ts", "**/*.sh"],
+                 "test_globs": ["tests/**", "spec/**", "**/*Test.php", "**/*.test.*", "**/*.spec.*", "**/test_*.py"],
+                 "map": {}}
+  },
+  "hints": [
+    {"paths": ["**/login*", "**/Security/**", "**/*Auth*"], "axis": "domain", "note": "auth-shaped path"},
+    {"paths": [".ai/policies/**", ".claude/**", ".codex/**", "CLAUDE.md", "AGENTS.md"], "axis": "behaviour", "note": "pipeline"},
+    {"paths": ["composer.json", "package.json", "requirements*.txt"], "axis": "novelty", "note": "dependency manifest"}
+  ],
+  "modes": {
+    "light": {"max_w": 2, "plan": "inline", "review": "on_red_fact", "reviewer": "BALANCED",
+              "human_stops": 0, "suite": "verify", "e2e": false},
+    "guarded": {"max_w": 7, "plan": "plan_mode", "review": "one", "reviewer": "BALANCED",
+                "human_stops": 1, "suite": "verify+e2e", "e2e": true},
+    "full": {"min_w": 8, "hard_axes": ["reversibility", "domain"], "plan": "written", "review": "one",
+             "reviewer": "STRONG", "security_focus_when": "domain=2", "human_stops": 2, "suite": "verify+e2e", "e2e": true},
+    "full_irreversible": {"when": "reversibility=2 and domain=2, or irreversible_content",
+                          "human_stops": 3, "confirm_names_operation": true}
+  },
+  "rules": {"unknown_scores": 1, "hint_raises_by": 1, "hint_raises_once_per_axis": true,
+            "rescore_axes": ["reversibility", "domain"], "rescore_direction": "up", "lower_by": "human",
+            "steps_advice_over": 8},
+  "review_rounds_max": 2,
+  "remediation_rounds": 2,
+  "tests": {"max_suite_runs": 2, "timeout_seconds": 1800, "env_retries": 1, "log_max_bytes": 2097152},
+  "batch": {"default": false, "review_rounds_max": 3, "max_suite_runs": 8, "parallel": 3, "tier": "BALANCED"},
+  "examples": {
+    "light": ["a font or copy change in one template", "a test-only change", "a message-text change"],
+    "guarded": ["a regex fix in a hot guard script", "a dev-environment manifest", "a feature with a new screen and an additive migration"],
+    "full": ["a change in Payment/, Tax/ or Security/", "a migration that rewrites user trees", "a production manifest or secret"]
+  }
+}