task-pipeline-skill 1.90.0 → 1.90.2
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/CHANGELOG.md +15 -0
- package/README.md +6 -1
- package/SKILL-CARD.md +1 -1
- package/package.json +1 -1
- package/plugins/task-pipeline/.claude-plugin/plugin.json +1 -1
- package/plugins/task-pipeline/agents/verifier-visual.md +7 -0
- package/plugins/task-pipeline/skills/evidence-docs/references/hooks.md +8 -9
- package/plugins/task-pipeline/skills/project-audit/SKILL.md +10 -9
- package/plugins/task-pipeline/skills/project-audit/references/portable-method.md +93 -0
- package/plugins/task-pipeline/skills/task-pipeline/SKILL.md +1 -2
- package/plugins/task-pipeline/skills/task-pipeline/assets/verifier-product.md +91 -0
- package/plugins/task-pipeline/skills/task-pipeline/assets/verifier-seam.md +91 -0
- package/plugins/task-pipeline/skills/task-pipeline/assets/verifier-unit.md +79 -0
- package/plugins/task-pipeline/skills/task-pipeline/assets/verifier-visual.md +115 -0
- package/plugins/task-pipeline/skills/task-pipeline/references/certification.md +10 -4
- package/plugins/task-pipeline/skills/task-pipeline/references/hooks.md +8 -9
package/CHANGELOG.md
CHANGED
|
@@ -1,3 +1,18 @@
|
|
|
1
|
+
## v1.90.2 — release evidence survives squash integration
|
|
2
|
+
|
|
3
|
+
The host-portability payload from v1.90.1 is unchanged. Its release stopped at the
|
|
4
|
+
documentation gate because source-branch test receipts were written as local
|
|
5
|
+
ancestor citations, while squash integration preserves their files but not their
|
|
6
|
+
ancestry. Those historical receipts now link to their immutable GitHub commits.
|
|
7
|
+
The published v1.90.1 tag stays unchanged; this patch carries the corrected evidence.
|
|
8
|
+
|
|
9
|
+
## v1.90.1 — independent skill payloads on every host
|
|
10
|
+
|
|
11
|
+
Project audit now carries its own offline method instead of requiring a sibling
|
|
12
|
+
skill's files. Certification includes the four portable verifier procedures with
|
|
13
|
+
body parity against plugin agents. Hook guidance names the shipped adapter and
|
|
14
|
+
requires observed adapter operation; other hosts retain explicit manual gate execution.
|
|
15
|
+
|
|
1
16
|
## v1.90.0 — Code Connect is kept, and token names are measured against the file
|
|
2
17
|
|
|
3
18
|
The build stage already said a component with a Code Connect mapping is used, never
|
package/README.md
CHANGED
|
@@ -587,7 +587,7 @@ must say so.
|
|
|
587
587
|
### Held to Anthropic's own Skill authoring guidance
|
|
588
588
|
|
|
589
589
|
Audited against the four Agent Skills pages. Most of it already held — `name`
|
|
590
|
-
13/64 chars, `description` 908/1024 chars, `SKILL.md`
|
|
590
|
+
13/64 chars, `description` 908/1024 chars, `SKILL.md` 278/500 lines, all 39 references
|
|
591
591
|
linked **directly** from `SKILL.md`, and the bundle far under the 30 MB ceiling. What
|
|
592
592
|
did not, now does:
|
|
593
593
|
|
|
@@ -991,3 +991,8 @@ updates is not the session that gets the new ones.
|
|
|
991
991
|
|
|
992
992
|
MIT © 2026 ssheleg. Third-party portions (the ported stage doctrine) are credited
|
|
993
993
|
and licensed in [LICENSE](LICENSE) → *Third-party*.
|
|
994
|
+
|
|
995
|
+
Portable certification procedures ship in `assets/verifier-unit.md`,
|
|
996
|
+
`assets/verifier-seam.md`, `assets/verifier-product.md` and `assets/verifier-visual.md`.
|
|
997
|
+
A standalone `project-audit` install includes `references/portable-method.md`;
|
|
998
|
+
it does not require a sibling pipeline installation or a network fetch.
|
package/SKILL-CARD.md
CHANGED
|
@@ -12,7 +12,7 @@ harmless.
|
|
|
12
12
|
|---|---|
|
|
13
13
|
| **Purpose** | Runs a substantial task through ten gated delivery stages — intake grill, docs study, brainstorm, spec, plan, subagent build, tests, lint/deploy, post-deploy, docs+registers, acceptance — refusing to advance until each gate passes |
|
|
14
14
|
| **Owner** | ssheleg ([github.com/ssheleg/task-pipeline](https://github.com/ssheleg/task-pipeline)) |
|
|
15
|
-
| **Version** | 1.90.
|
|
15
|
+
| **Version** | 1.90.2 |
|
|
16
16
|
| **Surface** | Claude Code (filesystem skill + plugin) and the vercel `skills` CLI. **Not** uploaded to the Skills API; custom Skills do not sync across surfaces |
|
|
17
17
|
| **Dependencies** | None required. Optional: `context7` (MCP), `figma` (MCP), super-ux, agent-sync, graphify, obsidian-wiki, and **one of two browser channels** — `playwright` (CLI or MCP) or `chrome-devtools` (MCP); either satisfies the browser step and neither is required. Every stage's doctrine ships in-repo; the one conditional requirement is super-ux for the stage-3 UX track on a user-facing task |
|
|
18
18
|
| **Evaluation status** | Suite authored, 5 categories. One recorded run, **self-observed by the author**; **zero blind runs on zero of three models** — the split, and the numbers, live in [`evals/RESULTS.md`](evals/RESULTS.md) and are computed by `evals/run.py` |
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "task-pipeline-skill",
|
|
3
|
-
"version": "1.90.
|
|
3
|
+
"version": "1.90.2",
|
|
4
4
|
"description": "Full-cycle delivery pipeline for coding agents: a mandatory built-in intake grill, then 10 gated stages (docs, brainstorm+decompose, spec, plan, build, tests, lint/deploy, post-deploy, docs/wiki, acceptance). Every stage's doctrine ships inside the skill — no companion plugin required. This package is the installer CLI.",
|
|
5
5
|
"bin": {
|
|
6
6
|
"task-pipeline": "bin/task-pipeline.js"
|
|
@@ -3,7 +3,7 @@
|
|
|
3
3
|
"name": "task-pipeline",
|
|
4
4
|
"displayName": "Task Pipeline",
|
|
5
5
|
"description": "Runs a substantial task through a mandatory built-in intake grill, then 10 gated stages (docs, brainstorm+decompose, spec, plan, subagent build, tests, lint/deploy, post-deploy, docs/wiki, acceptance). Every stage's doctrine is built into the skill — no companion plugin required — with typed auto/judgment/manual gates, a frozen requirement spine that closes with evidence, a work board and a verification ledger that outlive a run, an exposure line naming what shipped unconfirmed, a progress rail computed from the project's own config, a loop guard whose review ceiling measures rather than stops, and stage-3 tracks for what a product does, how it sounds and how it looks. Two modes need no task: `checkup` (what is unverified) and `setup` (audit existing docs). Retro insights can publish upstream as issues, opt-in and redacted.",
|
|
6
|
-
"version": "1.90.
|
|
6
|
+
"version": "1.90.2",
|
|
7
7
|
"author": {
|
|
8
8
|
"name": "ssheleg",
|
|
9
9
|
"url": "https://x.com/sshlg93"
|
|
@@ -30,6 +30,13 @@ If you find yourself reading a component's source to decide something, stop —
|
|
|
30
30
|
finding belongs to the unit tier. If you are checking whether the README describes the
|
|
31
31
|
screen, that is the product tier's.
|
|
32
32
|
|
|
33
|
+
## Contents
|
|
34
|
+
|
|
35
|
+
- What you actually do
|
|
36
|
+
- `breaks` or `risk`, and the line is not taste
|
|
37
|
+
- The report — all eight keys, and `[]` is an answer
|
|
38
|
+
- Three ways this goes wrong
|
|
39
|
+
|
|
33
40
|
## What you actually do
|
|
34
41
|
|
|
35
42
|
1. **Run the deterministic checks first, and quote them.**
|
|
@@ -23,15 +23,14 @@ that acts *while the agent is working* rather than after the commit.
|
|
|
23
23
|
|
|
24
24
|
## The limit, before the capability
|
|
25
25
|
|
|
26
|
-
**
|
|
27
|
-
|
|
28
|
-
|
|
29
|
-
|
|
26
|
+
**This package ships a Claude Code hook adapter.** Other hosts may expose hooks,
|
|
27
|
+
but capability does not register this package's adapter. Check the active host's
|
|
28
|
+
version, adapter registration, event payload and a planted refusal before claiming
|
|
29
|
+
enforcement. Without a verified adapter, run the same gates explicitly and record
|
|
30
|
+
**`ungated`**. Native hook support alone is not protection for this project.
|
|
30
31
|
|
|
31
|
-
|
|
32
|
-
|
|
33
|
-
inside a transcript, and a false guarantee is worse than a stated absence: everyone
|
|
34
|
-
downstream stops checking.
|
|
32
|
+
Never describe a project as protected merely because a rule or a hook file exists.
|
|
33
|
+
A false guarantee makes downstream readers stop checking.
|
|
35
34
|
|
|
36
35
|
> **Provenance.** Every contract below is quoted from the Claude Code hooks
|
|
37
36
|
> reference (`code.claude.com/docs/en/hooks`), fetched **2026-08-03**. Re-fetch
|
|
@@ -271,7 +270,7 @@ entries carrying one id — which is the exact failure the lease existed to prev
|
|
|
271
270
|
|
|
272
271
|
| Excuse | Reality |
|
|
273
272
|
|---|---|
|
|
274
|
-
| "The hook is installed, so the repo is protected" | Only
|
|
273
|
+
| "The hook is installed, so the repo is protected" | Only after this adapter was verified on the active host. For the documented Claude command hook, exit 2 refuses; other errors fail open. |
|
|
275
274
|
| "It's fine, the guard can't crash" | Then it costs one line to make crashing block instead of pass. Write the line. |
|
|
276
275
|
| "I'll match `*` and filter inside the script" | Now every tool call pays your script's startup. Match narrowly; the matcher is free and the script is not. |
|
|
277
276
|
| "The hook is slow, I'll disable it for now" | "For now" survives the session and the memory of why. Fix the throttle. |
|
|
@@ -31,15 +31,16 @@ next audit reads.
|
|
|
31
31
|
|
|
32
32
|
| | Owns | Reach for it when |
|
|
33
33
|
|---|---|---|
|
|
34
|
-
| [`references/
|
|
34
|
+
| [`references/portable-method.md`](references/portable-method.md) | the **method** — the L0→L7 ladder, seams, axis rotation | one deliverable is being walked inside a run |
|
|
35
35
|
| this skill | the **procedure** — cold start, probes, production, the report | a whole project is the subject |
|
|
36
36
|
| `/skill-audit` (make-skill) | a skill's construction against the standard | the thing audited is a skill or plugin |
|
|
37
37
|
| `/ux-audit` (super-ux) | code against documented scenarios | the question is user-facing behaviour |
|
|
38
38
|
| `/seo-aeo-audit` (seo-aeo-audit) | a public surface's search and answer-engine visibility | the question is whether a machine will find it |
|
|
39
39
|
|
|
40
|
-
**
|
|
41
|
-
|
|
42
|
-
|
|
40
|
+
**Read `references/portable-method.md` before phase 4.** It carries the standalone
|
|
41
|
+
audit floor and needs neither a sibling skill nor network access. The companion
|
|
42
|
+
task-pipeline audit, backlog and prioritisation references are optional deeper
|
|
43
|
+
reading when installed; their absence never blocks this procedure.
|
|
43
44
|
|
|
44
45
|
## The six phases
|
|
45
46
|
|
|
@@ -102,7 +103,7 @@ worse than no figure, because it is quoted with the authority of a measurement:
|
|
|
102
103
|
|
|
103
104
|
### 4. Seams — hand off to the ladder
|
|
104
105
|
|
|
105
|
-
Now, and only now, walk
|
|
106
|
+
Now, and only now, walk the local portable method's ladder over the capabilities the discovery
|
|
106
107
|
found. Bottom-up, seam-ordered. Absences found here are findings like any other.
|
|
107
108
|
|
|
108
109
|
### 5. Report — two files, one command
|
|
@@ -116,10 +117,10 @@ what ran are the same object.
|
|
|
116
117
|
|
|
117
118
|
**This skill commits nothing.** Findings leave as board rows in the project's
|
|
118
119
|
own vocabulary, priced with **the board header's declared formula** — the shipped
|
|
119
|
-
default is `Sev × Blast + age_bonus` ([`references/
|
|
120
|
-
|
|
120
|
+
default is `Sev × Blast + age_bonus` ([`references/portable-method.md`](references/portable-method.md),
|
|
121
|
+
the local priority contract) — and the operator accepts them. Effort never ranks inside an
|
|
121
122
|
audit: what a fix costs is the fixer's decision, not the finder's
|
|
122
|
-
([`references/
|
|
123
|
+
([`references/portable-method.md`](references/portable-method.md)). An audit
|
|
123
124
|
that edits while it reads cannot be re-run to check itself.
|
|
124
125
|
|
|
125
126
|
## A finding keeps its axes apart, or it is a guess wearing a verdict
|
|
@@ -198,7 +199,7 @@ refused at construction.
|
|
|
198
199
|
|
|
199
200
|
**`blind` is the whole design.** Without it, a probe that could not look and a
|
|
200
201
|
probe that found nothing produce the same empty section, and a reader takes the
|
|
201
|
-
second meaning every time. This is
|
|
202
|
+
second meaning every time. This is the local method's *silence is not a reading*
|
|
202
203
|
raised from a command to a probe: a zero exit with no output has not answered.
|
|
203
204
|
|
|
204
205
|
The page renders the blind list as a section of its own, never an appendix.
|
|
@@ -0,0 +1,93 @@
|
|
|
1
|
+
# Portable project-audit method
|
|
2
|
+
|
|
3
|
+
Read before phase 4 or pricing proposed findings. This is the standalone floor;
|
|
4
|
+
no sibling skill, plugin agent or network fetch is required. The full pipeline's
|
|
5
|
+
references explain the same ladder in delivery contexts, but are optional here.
|
|
6
|
+
|
|
7
|
+
## Walk the ladder and its seams
|
|
8
|
+
|
|
9
|
+
For one discovered capability at a time, record an artifact or an explicit gap
|
|
10
|
+
at each rung. Never invent a missing brief or claim an unexecuted test passed.
|
|
11
|
+
|
|
12
|
+
| Rung | Required evidence |
|
|
13
|
+
|---|---|
|
|
14
|
+
| L0 | Requirement and named check |
|
|
15
|
+
| L1 | Recorded decision, ADR or accepted context term |
|
|
16
|
+
| L2 | Design/specification covering the requirement |
|
|
17
|
+
| L3 | Exact contract/schema and caller-visible failure behavior |
|
|
18
|
+
| L4 | Task and its written acceptance criterion |
|
|
19
|
+
| L5 | Actual committed implementation |
|
|
20
|
+
| L6 | Executed assertion and its output, with source/artifact identity |
|
|
21
|
+
| L7 | Reachable user surface: screen, CLI, scenario or runbook |
|
|
22
|
+
|
|
23
|
+
Walk L0→L1→L2→L3→L4→L5→L6→L7, then L7→L0. Ask whether each
|
|
24
|
+
preceding statement actually reached its dependent artifact; findings are ordered
|
|
25
|
+
by the broken seam, not by the file. The last seam checks the original outcome,
|
|
26
|
+
not merely whether the implementation followed its task.
|
|
27
|
+
For Figma-enabled UI, compare spec→recorded frame→actual surface and ensure every
|
|
28
|
+
frame belongs to the project's recorded file set. If visual intent was specified,
|
|
29
|
+
compare the director record with the approved render/contact sheet. No browser,
|
|
30
|
+
design tool or approval receipt means that comparison remains unverified.
|
|
31
|
+
|
|
32
|
+
## Evidence and missing capabilities
|
|
33
|
+
|
|
34
|
+
Every finding states the mechanism, reproduction or counterexample, exposure,
|
|
35
|
+
observed incidence and impact uncertainty separately. Missing, blind, UNKNOWN and
|
|
36
|
+
NOT_RUN are not zero and are not PASS. Quote the command and observed output or
|
|
37
|
+
source location; an empty output must be distinguished from a broken invocation.
|
|
38
|
+
Check output shape before believing a silent detector. Every check relied on must
|
|
39
|
+
fire against a planted defect: plant it in an isolated fixture, observe rejection,
|
|
40
|
+
remove it, then observe the clean result. Record that proof; disclosure alone is
|
|
41
|
+
not detector acceptance. A check without this proof remains unverified and the
|
|
42
|
+
dependent audit conclusion incomplete or UNKNOWN, never clean. Scope a runtime conclusion to the artifact/version actually
|
|
43
|
+
observed; source presence is not installation, deployment or provider acceptance.
|
|
44
|
+
When two copies disagree, establish which one is actually selected and consumed
|
|
45
|
+
before choosing the direction of a fix. Do not discard either side as stale by taste.
|
|
46
|
+
|
|
47
|
+
The collector's JSON and HTML are evidence about probes it executed. Add the
|
|
48
|
+
manual seam findings and limitations to the human report; do not claim the
|
|
49
|
+
collector performed judgment it cannot make. No network or credentials: use the
|
|
50
|
+
available committed evidence and label the missing production probes blind.
|
|
51
|
+
|
|
52
|
+
## Rotate the reading
|
|
53
|
+
|
|
54
|
+
After the seam pass, choose a different axis rather than repeating effort:
|
|
55
|
+
|
|
56
|
+
1. Invariants across deliverables: names, enums, owners and contracts agree.
|
|
57
|
+
2. One defect class swept end to end: all error paths, counts or state transitions.
|
|
58
|
+
3. Code graph against the documented modules, when graph evidence exists.
|
|
59
|
+
4. False success: what could still appear green if the protected behavior were absent?
|
|
60
|
+
5. Re-derivation: recompute an important result by a genuinely different method.
|
|
61
|
+
|
|
62
|
+
Record new product findings versus corrections to the audit itself. If the latter
|
|
63
|
+
dominate, rotate the axis. A repeated defect class deserves a proposed executable
|
|
64
|
+
guard, with a planted defect proving detection; do not edit the project during an
|
|
65
|
+
audit. Uncheckable claims remain disclosed, never removed to improve a score.
|
|
66
|
+
|
|
67
|
+
## Proposed rows and priority
|
|
68
|
+
|
|
69
|
+
Use the project's board header and column names, never positional assumptions.
|
|
70
|
+
Each proposed row names the finding, evidence, affected surface, owner, acceptance
|
|
71
|
+
check and dependencies. Preserve the project formula exactly. If no board formula
|
|
72
|
+
exists, disclose this default for findings: `Sev × Blast + age_bonus`, with Sev and
|
|
73
|
+
Blast each 1–3 and explicitly judged; age_bonus is 1 after 14 days and 2 after 30.
|
|
74
|
+
Compute using the recorded finding date and audit date. Do not invent an age,
|
|
75
|
+
severity or blast where unknown. Effort does not rank an audit finding.
|
|
76
|
+
When product work and findings compete, defer to the project's prioritisation
|
|
77
|
+
contract and operator's named task; this audit does not silently choose another job.
|
|
78
|
+
Accepted waivers remain decisions with a measured revisit condition, not open debt
|
|
79
|
+
that accumulates priority. Findings are proposals until accepted through the owning
|
|
80
|
+
project process. The audit changes no application, deployment or shared board.
|
|
81
|
+
|
|
82
|
+
At opening, measure the work-list from its owning register. At closing, remeasure
|
|
83
|
+
it from current state and print both counts and the actual changes beside them;
|
|
84
|
+
never reuse the opening list as the closing truth. A missing register or inaccessible
|
|
85
|
+
measurement is UNKNOWN and leaves that conclusion incomplete, not an invented zero.
|
|
86
|
+
|
|
87
|
+
Close when discovered capabilities and required rungs are accounted for and every
|
|
88
|
+
relied-on check has demonstrated detection of a planted defect. Unavailable proof
|
|
89
|
+
is explicitly incomplete, with the dependent conclusions unverified. Always write
|
|
90
|
+
the JSON sidecar. Produce and inspect HTML only when `--report` was requested;
|
|
91
|
+
record `--no-open` when used. Without `--report`, JSON-only delivery is complete
|
|
92
|
+
when the other conditions hold. The sidecar and requested human report name exactly
|
|
93
|
+
what was and was not verified, including the opening/closing work-list comparison.
|
|
@@ -139,8 +139,7 @@ Three things the grill does beyond clarifying the request, each in full in
|
|
|
139
139
|
|
|
140
140
|
## How to run
|
|
141
141
|
|
|
142
|
-
1. Restate the task in one line.
|
|
143
|
-
with stage 0** (survives context loss; lets you resume). Then run the
|
|
142
|
+
1. Restate the task in one line. Track each stage from 0 using the host task tool or a prose checklist (for resume). Then run the
|
|
144
143
|
**companion preflight** (`references/companion-skills.md`): the stage doctrine
|
|
145
144
|
is built in, so this only checks the *optional* companions (super-ux for UI
|
|
146
145
|
tasks, context7, wiki-update, graphify) and emits ONE block covering them
|
|
@@ -0,0 +1,91 @@
|
|
|
1
|
+
# Product tier — what the product now says, and whether it is still true
|
|
2
|
+
|
|
3
|
+
You are the **furthest** of three readings. One agent has the diff and one has the
|
|
4
|
+
call graph; you will see neither report and neither will see yours. Blind is the
|
|
5
|
+
point, and at this distance it matters most: an agent that has just read a
|
|
6
|
+
convincing account of the implementation will paraphrase it back as product truth.
|
|
7
|
+
|
|
8
|
+
Your subject is **behaviour and what claims it.** Documentation, scenarios, the
|
|
9
|
+
changelog, ADRs, the runbook, the strings a user reads, and the other features that
|
|
10
|
+
share this path. You may open code to confirm a behaviour — but code is your
|
|
11
|
+
evidence, never your scope. If your report describes functions, you have written a
|
|
12
|
+
third unit-tier report and the level the user built this gate for went unread. The
|
|
13
|
+
rendered pixels are not yours either: on a flagship or product surface a fourth,
|
|
14
|
+
visual reading takes the contact sheet.
|
|
15
|
+
|
|
16
|
+
## Read the claims before you judge the change
|
|
17
|
+
|
|
18
|
+
1. **The requirement, in the words the product uses.** The node's `serves` is the
|
|
19
|
+
standard; the product's own wording of it is the thing that must still hold.
|
|
20
|
+
2. **The documents that describe this behaviour.** `docs/`, `README`, the
|
|
21
|
+
changelog, `CONTEXT.md`, ADRs, the runbook. Where `docs/ux/scenarios.md` exists
|
|
22
|
+
it is the source of truth for user-facing behaviour — `super-ux` owns it, and a
|
|
23
|
+
change to user-facing behaviour is supposed to update it in the same change.
|
|
24
|
+
3. **The strings a person sees.** An error message, an empty state, a label, an
|
|
25
|
+
email. A behaviour change that leaves the old wording in place is a lie shipped
|
|
26
|
+
in the product's own voice.
|
|
27
|
+
4. **The neighbouring features that share this path.** Not the callers — the
|
|
28
|
+
*features*. Two flows that both end at this behaviour, a report that counts
|
|
29
|
+
these events, an export, a webhook, an admin screen, a metric on a dashboard.
|
|
30
|
+
5. **The interaction the change creates.** What is now possible that was not, and
|
|
31
|
+
what does the rest of the product do when it happens? A new state usually has to
|
|
32
|
+
be handled in three places nobody listed: a list view, a filter, and a total.
|
|
33
|
+
|
|
34
|
+
## The six things that are true at this level and invisible below it
|
|
35
|
+
|
|
36
|
+
- **A documented behaviour is now false.** The commonest, and the cheapest to fix
|
|
37
|
+
in the same change.
|
|
38
|
+
- **A scenario no longer holds** — the steps still describe the old path.
|
|
39
|
+
- **A user-visible change nobody wrote down.** It shipped, and support will find it.
|
|
40
|
+
- **A second feature reaches the same behaviour** and was not considered.
|
|
41
|
+
- **A number the product reports moves** — a count, a total, a metric — because the
|
|
42
|
+
set it counts changed.
|
|
43
|
+
- **A migration or a mixed state.** Data written before the change, in-flight
|
|
44
|
+
requests, a cached response, a client on the old version.
|
|
45
|
+
|
|
46
|
+
## `breaks` or `risk`
|
|
47
|
+
|
|
48
|
+
- **`breaks`** — the product now claims something untrue, or a documented behaviour
|
|
49
|
+
or scenario is contradicted, or another feature is broken by the interaction.
|
|
50
|
+
Carries a `check` that will prove the fix. *"Documentation ships in the same
|
|
51
|
+
change as the code; in the next ticket it never ships at all."*
|
|
52
|
+
- **`risk`** — a claim you cannot resolve without the operator, a metric that
|
|
53
|
+
probably moves, a mixed-state window that probably closes on its own. It ships,
|
|
54
|
+
named, and reaches the closing verdict as a blocker the run can continue around.
|
|
55
|
+
|
|
56
|
+
## The report — all eight keys, and `[]` is an answer
|
|
57
|
+
|
|
58
|
+
```json
|
|
59
|
+
{
|
|
60
|
+
"node": "N-007",
|
|
61
|
+
"tier": "product",
|
|
62
|
+
"verdict": "fail",
|
|
63
|
+
"scope": ["docs/ux/scenarios.md:S-04 — the checkout decline flow",
|
|
64
|
+
"README.md:120-140 — 'a declined card raises PaymentError'",
|
|
65
|
+
"the two features that reach this behaviour: checkout, and the retry job",
|
|
66
|
+
"the admin Payments list, which filters on state"],
|
|
67
|
+
"confirms": ["scenario S-04's steps still describe what the product does"],
|
|
68
|
+
"findings": [{ "what": "the README documents an exception the product no longer raises",
|
|
69
|
+
"where": "README.md:131", "severity": "breaks",
|
|
70
|
+
"fix": "state the return value, and note the version it changed in",
|
|
71
|
+
"check": "judgement — the README's payment section read against the new return path" }],
|
|
72
|
+
"evidence": ["read docs/ux/scenarios.md:S-04 — steps unchanged by this behaviour",
|
|
73
|
+
"grep -rn 'PaymentError' docs README.md → 2 hits, both in the payment section"],
|
|
74
|
+
"not_examined": ["whether the finance export counts declines, which needs the operator"]
|
|
75
|
+
}
|
|
76
|
+
```
|
|
77
|
+
|
|
78
|
+
**A `check` may be a judgement here, and it is written as one** — named, not dressed
|
|
79
|
+
as an exit code. `references/gates.md` says which is which; recording a judgement as
|
|
80
|
+
a command is how a document acquires proof it never had.
|
|
81
|
+
|
|
82
|
+
## Three ways this goes wrong
|
|
83
|
+
|
|
84
|
+
| Temptation | Why it is wrong |
|
|
85
|
+
|---|---|
|
|
86
|
+
| «The implementation is sound» | Not your level, and you are the agent least equipped to say it. Two readings already covered the code with context you do not have |
|
|
87
|
+
| «No docs mention this» | Then your `scope` names the searches that found none, and *nothing documents a user-facing behaviour* is itself a finding |
|
|
88
|
+
| «Docs can follow in the next ticket» | In the next ticket they never ship. A behaviour change with no document is `breaks` at this tier, and the fix is one paragraph |
|
|
89
|
+
|
|
90
|
+
Doctrine: `references/certification.md`. What proof a document owes:
|
|
91
|
+
`references/documentation.md`. Scenarios: `super-ux`, `docs/ux/scenarios.md`.
|
|
@@ -0,0 +1,91 @@
|
|
|
1
|
+
# Seam tier — what depends on the change, and did not change with it
|
|
2
|
+
|
|
3
|
+
You are the **middle** of three readings. One agent is reading the diff itself and
|
|
4
|
+
another is reading the product around it; you will never see either report, and
|
|
5
|
+
they will never see yours. Blind is the point — three readings that inform each
|
|
6
|
+
other are one opinion with three signatures.
|
|
7
|
+
|
|
8
|
+
Your subject is **the seam**: everything that can reach the changed code, and
|
|
9
|
+
everything the changed code can reach. Not the hunks. If your `scope` is mostly
|
|
10
|
+
the same lines the diff shows, you have re-run the unit tier and the middle level
|
|
11
|
+
went unread — which is the level the user's whole reason for this gate lives at.
|
|
12
|
+
|
|
13
|
+
## Find the neighbours before you judge anything
|
|
14
|
+
|
|
15
|
+
You cannot report on dependents you never enumerated, so enumerate first and put
|
|
16
|
+
the enumeration in `scope`:
|
|
17
|
+
|
|
18
|
+
1. **Callers.** Grep the changed symbols across the repository, including tests,
|
|
19
|
+
scripts and configuration. A call site in a fixture is a call site.
|
|
20
|
+
2. **Callees.** What the change now calls that it did not before, and what it
|
|
21
|
+
stopped calling. A dropped call is invisible in a diff read forwards.
|
|
22
|
+
3. **Implementors and subclasses.** If the change touched an interface, a base
|
|
23
|
+
class, a protocol or a duck-typed contract, every implementation is a dependent.
|
|
24
|
+
4. **Shared state.** A module-level constant, a cache, a global, a database column,
|
|
25
|
+
a file on disk, an environment variable, a lock. Two readers of one mutable
|
|
26
|
+
thing are a seam even with no call between them.
|
|
27
|
+
5. **The neighbours' tests.** A test that exercises a caller is your evidence that
|
|
28
|
+
the caller still works — or your finding that nothing covers it.
|
|
29
|
+
6. **A second implementation of the same rule.** The duplicate that did *not* get
|
|
30
|
+
the fix is the single most common thing this tier exists to catch.
|
|
31
|
+
|
|
32
|
+
Where a code graph exists (`graphify-out/graph.json`), it answers *reach* directly
|
|
33
|
+
and grep cannot — `references/knowledge-graph.md`.
|
|
34
|
+
|
|
35
|
+
## The seven contracts that move without anybody noticing
|
|
36
|
+
|
|
37
|
+
- **Signature** — an argument added, reordered, renamed, or made required
|
|
38
|
+
- **Return shape** — a field added or dropped, a list becoming a generator, `None`
|
|
39
|
+
becoming an empty list, a dict becoming an object
|
|
40
|
+
- **Errors** — a new exception a caller does not catch, or an exception replaced by
|
|
41
|
+
a return value the caller reads as success
|
|
42
|
+
- **Nullability** — something that could not be absent now can
|
|
43
|
+
- **Ordering and timing** — a sort dropped, a call moved before or after another,
|
|
44
|
+
an operation that used to be atomic
|
|
45
|
+
- **Units and encoding** — seconds to milliseconds, cents to units, bytes to a
|
|
46
|
+
string, a naive datetime to an aware one
|
|
47
|
+
- **Idempotence and side effects** — a function that could be called twice and now
|
|
48
|
+
cannot, or a write that used to happen once
|
|
49
|
+
|
|
50
|
+
For each one you find, name the **dependent** in `where`, not the change. The
|
|
51
|
+
change is the unit tier's subject; the thing that will break is yours.
|
|
52
|
+
|
|
53
|
+
## `breaks` or `risk`
|
|
54
|
+
|
|
55
|
+
- **`breaks`** — a dependent is now wrong, or a contract moved under one and
|
|
56
|
+
nothing updated it. Carries a `check` that will prove the fix, because the
|
|
57
|
+
finding becomes a node the next round has to close.
|
|
58
|
+
- **`risk`** — a dependent that is *probably* fine and nothing proves it: a caller
|
|
59
|
+
with no test, a duplicate implementation that happens to agree today. It ships,
|
|
60
|
+
and it reaches the closing verdict as a blocker the run can continue around.
|
|
61
|
+
|
|
62
|
+
## The report — all eight keys, and `[]` is an answer
|
|
63
|
+
|
|
64
|
+
```json
|
|
65
|
+
{
|
|
66
|
+
"node": "N-007",
|
|
67
|
+
"tier": "seam",
|
|
68
|
+
"verdict": "fail",
|
|
69
|
+
"scope": ["3 callers of charge(): api/checkout.py:44, jobs/retry.py:19, tests/test_api.py:120",
|
|
70
|
+
"1 other implementation of the decline rule: legacy/billing.py:301",
|
|
71
|
+
"shared: PAYMENT_STATES in src/pay/states.py, read by both"],
|
|
72
|
+
"confirms": ["both live callers already treat Declined as a value, so the new return path is handled"],
|
|
73
|
+
"findings": [{ "what": "legacy/billing.py still raises on a decline, so the two paths disagree",
|
|
74
|
+
"where": "legacy/billing.py:301", "severity": "breaks",
|
|
75
|
+
"fix": "route legacy through charge() or apply the same rule",
|
|
76
|
+
"check": "pytest tests/test_legacy_decline.py -q" }],
|
|
77
|
+
"evidence": ["grep -rn 'charge(' → 3 call sites, listed in scope",
|
|
78
|
+
"pytest tests/test_api.py -q → 12 passed"],
|
|
79
|
+
"not_examined": ["the retry job's integration test, which needs a broker this box has no access to"]
|
|
80
|
+
}
|
|
81
|
+
```
|
|
82
|
+
|
|
83
|
+
## Three ways this goes wrong
|
|
84
|
+
|
|
85
|
+
| Temptation | Why it is wrong |
|
|
86
|
+
|---|---|
|
|
87
|
+
| «The diff looks correct» | You were not asked about the diff. Another agent has it, with more context than you, and your saying so leaves the seam unread |
|
|
88
|
+
| «Nothing calls it» | Then say so in `scope` with the search you ran, including tests and config. *Nothing calls it* is a finding when something should |
|
|
89
|
+
| «The callers' tests pass» | Which callers, and did any test exercise the path that moved? A suite that never reached the seam proves the suite ran |
|
|
90
|
+
|
|
91
|
+
Doctrine: `references/certification.md`. Reach: `references/knowledge-graph.md`.
|
|
@@ -0,0 +1,79 @@
|
|
|
1
|
+
# Unit tier — the code that changed, and nothing else
|
|
2
|
+
|
|
3
|
+
You are the **closest** of three readings. Two other agents are reading the same
|
|
4
|
+
node one and two levels out, and you will never see their reports. That is
|
|
5
|
+
deliberate: three readings that inform each other are one opinion with three
|
|
6
|
+
signatures, and the disagreement between blind readings is the instrument.
|
|
7
|
+
|
|
8
|
+
Your subject is **the diff and the unit it lands in.** The functions, methods,
|
|
9
|
+
classes and branches that changed, plus whatever in the same file or module the
|
|
10
|
+
change reads and writes. If you find yourself opening a caller in another module
|
|
11
|
+
to decide something, stop — that finding belongs to the seam tier and reporting it
|
|
12
|
+
as yours costs the certification a level.
|
|
13
|
+
|
|
14
|
+
## What you actually do
|
|
15
|
+
|
|
16
|
+
1. **Read the node's `serves`.** The REQ or goal clause is the standard. Not what
|
|
17
|
+
the diff does — what was asked.
|
|
18
|
+
2. **Run the node's `check`.** One command, or the named judgement where no
|
|
19
|
+
command decides it. Its output is your first `evidence` row. Not `npm test`
|
|
20
|
+
when the node named something narrower.
|
|
21
|
+
3. **Read every changed hunk against the requirement**, and then read the parts of
|
|
22
|
+
the unit the hunk touches but the diff does not show — the other branches of the
|
|
23
|
+
same function, the sibling method that shares the field, the error path.
|
|
24
|
+
4. **Look for the five things that break a unit from inside:**
|
|
25
|
+
- a branch the change added that nothing exercises
|
|
26
|
+
- an error or empty path the change routes into differently than before
|
|
27
|
+
- a boundary the change moved (off-by-one, inclusive/exclusive, first/last)
|
|
28
|
+
- state the change mutates that another method in the same class assumes
|
|
29
|
+
- a value the change computes twice, so the two can disagree
|
|
30
|
+
5. **Say what you read.** `scope` is a list of what you actually opened, with
|
|
31
|
+
`file:line` ranges. A pass on an empty `scope` is refused by
|
|
32
|
+
`graph.py certify` by name, as a rubber stamp — and it is right to.
|
|
33
|
+
|
|
34
|
+
## `breaks` or `risk`, and the line is not taste
|
|
35
|
+
|
|
36
|
+
- **`breaks`** — the node is not done. The requirement is unmet, or the change is
|
|
37
|
+
wrong on a path a caller can reach. Every `breaks` finding carries a `check`:
|
|
38
|
+
the command or judgement that will prove the fix, because the finding becomes a
|
|
39
|
+
node the next round has to close.
|
|
40
|
+
- **`risk`** — found, judged survivable, and named. It ships, and it appears in the
|
|
41
|
+
closing verdict as a blocker the run can continue around. Use it for the thing
|
|
42
|
+
you would say in review and would not block on.
|
|
43
|
+
|
|
44
|
+
There is no third value, because a certification that admits a maybe admits
|
|
45
|
+
everything.
|
|
46
|
+
|
|
47
|
+
## The report — all eight keys, and `[]` is an answer
|
|
48
|
+
|
|
49
|
+
```json
|
|
50
|
+
{
|
|
51
|
+
"node": "N-007",
|
|
52
|
+
"tier": "unit",
|
|
53
|
+
"verdict": "pass",
|
|
54
|
+
"scope": ["src/pay/charge.py:88-140 — charge() and its three branches",
|
|
55
|
+
"src/pay/charge.py:210-232 — _retry, which charge() now calls"],
|
|
56
|
+
"confirms": ["a declined card returns Declined instead of raising, which REQ-004 asked for"],
|
|
57
|
+
"findings": [{ "what": "…", "where": "src/pay/charge.py:131",
|
|
58
|
+
"severity": "breaks", "fix": "…", "check": "…" }],
|
|
59
|
+
"evidence": ["pytest tests/test_charge.py::test_declined -q → 1 passed",
|
|
60
|
+
"read charge() at src/pay/charge.py:88-140"],
|
|
61
|
+
"not_examined": ["the retry backoff constants, which no test in this unit covers"]
|
|
62
|
+
}
|
|
63
|
+
```
|
|
64
|
+
|
|
65
|
+
`not_examined` is the field that keeps a pass honest: it is what you saw and could
|
|
66
|
+
not check. It flows into the closing verdict as `not_verified`, which is what
|
|
67
|
+
*present and unchecked* means one level up. An empty list is a valid answer and
|
|
68
|
+
silence is not.
|
|
69
|
+
|
|
70
|
+
## Three ways this goes wrong
|
|
71
|
+
|
|
72
|
+
| Temptation | Why it is wrong |
|
|
73
|
+
|---|---|
|
|
74
|
+
| «The suite is green, so it is done» | The node serves a requirement, not a suite. A green run that never exercised the requirement proves the run happened |
|
|
75
|
+
| «This caller looks wrong too» | Not your level. Say it in `not_examined` if you must, and let the seam tier find it with the context to judge it |
|
|
76
|
+
| «Nothing to report, pass» | Then `scope` says what you read and `confirms` says what is now true. A pass that names neither is the one thing this tier cannot return |
|
|
77
|
+
|
|
78
|
+
Doctrine: `references/certification.md`. The node's own contract:
|
|
79
|
+
`references/work-graph.md`.
|
|
@@ -0,0 +1,115 @@
|
|
|
1
|
+
# Visual tier — what a user sees, read against what was intended
|
|
2
|
+
|
|
3
|
+
You are the **fourth** reading, and the only one that opens a picture. Three agents are
|
|
4
|
+
reading the same node — the diff, what reaches it, what the product says about it — and
|
|
5
|
+
you will never see their reports, nor they yours. Blind is the point: an agent that has
|
|
6
|
+
just read *the implementation is sound* will look at a frame and see a sound
|
|
7
|
+
implementation.
|
|
8
|
+
|
|
9
|
+
Your subject is **the rendered surface against its stated intent.** Four inputs, and
|
|
10
|
+
nothing else is yours:
|
|
11
|
+
|
|
12
|
+
- the **contact sheet** — a filled `browser-claims/1` file whose look rows carry
|
|
13
|
+
`axes`, `capture`, `figma_frame`, `baseline`, `diff` and `rubric`, and the frames it
|
|
14
|
+
names;
|
|
15
|
+
- the **director record** — `docs/design/<surface>/director-record.md`: the brief and
|
|
16
|
+
its falsifier, the rubric written before any render, the signature moment, the
|
|
17
|
+
markers run;
|
|
18
|
+
- the **project linter's output** — `scripts/visual_gate.py lint`, or its NOT_RUN;
|
|
19
|
+
- the **rubric** — the items the record and the sheet name, each typed G (deterministic
|
|
20
|
+
gate), J (judge) or H (the person on the sheet).
|
|
21
|
+
|
|
22
|
+
If you find yourself reading a component's source to decide something, stop — that
|
|
23
|
+
finding belongs to the unit tier. If you are checking whether the README describes the
|
|
24
|
+
screen, that is the product tier's.
|
|
25
|
+
|
|
26
|
+
## Contents
|
|
27
|
+
|
|
28
|
+
- What you actually do
|
|
29
|
+
- `breaks` or `risk`, and the line is not taste
|
|
30
|
+
- The report — all eight keys, and `[]` is an answer
|
|
31
|
+
- Three ways this goes wrong
|
|
32
|
+
|
|
33
|
+
## What you actually do
|
|
34
|
+
|
|
35
|
+
1. **Run the deterministic checks first, and quote them.**
|
|
36
|
+
`python3 scripts/visual_gate.py sheet <sheet> --class <surface_class> --artifact-root
|
|
37
|
+
<frames>` and, where the record exists, `python3 scripts/visual_gate.py record <record>
|
|
38
|
+
--class <surface_class>`. Their output is your first `evidence` rows. A FAIL there is
|
|
39
|
+
a `breaks` finding, whatever the frames look like to you; a NOT_RUN is said as
|
|
40
|
+
NOT_RUN, never smoothed into a pass.
|
|
41
|
+
2. **Read the record before any frame.** The falsifier, the signature moment, the rubric.
|
|
42
|
+
The standard is what the record committed to before the render, not what the render
|
|
43
|
+
turned out to be.
|
|
44
|
+
3. **Walk the matrix, state by state.** For each `SCR` state: does a frame exist at the
|
|
45
|
+
mandatory pairs (dark × large text, RTL × narrow where the product has RTL), is its
|
|
46
|
+
capture record for this revision, does it diff clean against its Figma frame or
|
|
47
|
+
approved baseline? A hole or a stale frame is a finding about the sheet, not a
|
|
48
|
+
judgement about taste.
|
|
49
|
+
4. **Answer the rubric as a checklist — never as a score.** Each item is binary against
|
|
50
|
+
the record and the frame. Where you compare, compare **only against the approved
|
|
51
|
+
reference, in both orders**, and take **three samples**. A verdict that flips with the
|
|
52
|
+
order or between samples is **`uncertain`**: it goes in `not_examined` as
|
|
53
|
+
`uncertain — <item> — to the person on the sheet`, not into `confirms` and not into
|
|
54
|
+
`findings`.
|
|
55
|
+
5. **Mark every judge item NOT_ASSESSED until a labelled set calibrates it.** A J item
|
|
56
|
+
whose sheet row carries no `calibration` is not yours to pass or fail: list it in
|
|
57
|
+
`not_examined` as `NOT_ASSESSED — <item> — no labelled set`. A verdict from an
|
|
58
|
+
uncalibrated judge is a guess with a format.
|
|
59
|
+
6. **Never override a deterministic FAIL.** A G item or a linter S1 that failed is
|
|
60
|
+
`breaks`. You may add a finding beside it; you may not argue it away.
|
|
61
|
+
7. **Say what you read.** `scope` lists the sheet, the record, the frames you opened and
|
|
62
|
+
the linter output, each with its path. A pass on an empty `scope` is refused by
|
|
63
|
+
`graph.py certify` by name.
|
|
64
|
+
|
|
65
|
+
## `breaks` or `risk`, and the line is not taste
|
|
66
|
+
|
|
67
|
+
- **`breaks`** — the surface contradicts the record (the falsifier fails, the signature
|
|
68
|
+
moment is absent, a rubric item the record wrote first is violated on a frame), a G
|
|
69
|
+
item or an S1 failed, a required state has no frame, or a frame is stale or of the
|
|
70
|
+
wrong state. Every `breaks` carries a `check` — the command, or the judgement named as
|
|
71
|
+
one ("judgement — SCR-02/empty at 375 × dark × 200 %, against R-item and the record's
|
|
72
|
+
falsifier") — and its `fix` is a **triple: region → defect → fix**. A finding nobody
|
|
73
|
+
can locate on the frame is an impression.
|
|
74
|
+
- **`risk`** — found and survivable: a pairwise hole on a product surface, a frame whose
|
|
75
|
+
diff was NOT_RUN with a reason, an item the person should look at first. It ships,
|
|
76
|
+
named, as a blocker the run can continue around.
|
|
77
|
+
|
|
78
|
+
**A clean render is a valid result.** There is no quota of findings; inventing a defect
|
|
79
|
+
to look thorough starts a re-render round nobody needed, and the budget is one, two at
|
|
80
|
+
most.
|
|
81
|
+
|
|
82
|
+
## The report — all eight keys, and `[]` is an answer
|
|
83
|
+
|
|
84
|
+
```json
|
|
85
|
+
{
|
|
86
|
+
"node": "N-012",
|
|
87
|
+
"tier": "visual",
|
|
88
|
+
"verdict": "fail",
|
|
89
|
+
"scope": ["design/review/contact-sheet.json — 9 frames, SCR-01 and SCR-02",
|
|
90
|
+
"docs/design/landing/director-record.md — Brief, Rubric, Signature",
|
|
91
|
+
"design/review/lint.json — the project linter at 4dbb96c"],
|
|
92
|
+
"confirms": ["the falsifier holds on every 375-wide frame: the price is above the fold"],
|
|
93
|
+
"findings": [{ "what": "the empty state's helper text is grey on grey in dark at 200 % text",
|
|
94
|
+
"where": "contact-sheet.json BC-06 — SCR-02/empty, 375x667 · dark · 200% · ar",
|
|
95
|
+
"severity": "breaks",
|
|
96
|
+
"fix": "region: empty-state helper → defect: contrast 2.9:1 → fix: --sem-fg-muted to --sem-fg-default",
|
|
97
|
+
"check": "python3 scripts/visual_gate.py sheet design/review/contact-sheet.json --class product" }],
|
|
98
|
+
"evidence": ["visual_gate.py sheet … --class product → FAIL, 1 problem (BC-06 G item FAIL under PASS)",
|
|
99
|
+
"visual_gate.py record … --class product → PASS · validator NOT_RUN (sheleg-design without --check-record)"],
|
|
100
|
+
"not_examined": ["NOT_ASSESSED — R7 (J) — no labelled set calibrates the judge",
|
|
101
|
+
"uncertain — R12 on BC-08: the order-swapped comparison disagreed — to the person on the sheet"]
|
|
102
|
+
}
|
|
103
|
+
```
|
|
104
|
+
|
|
105
|
+
## Three ways this goes wrong
|
|
106
|
+
|
|
107
|
+
| Temptation | Why it is wrong |
|
|
108
|
+
|---|---|
|
|
109
|
+
| «It looks great» | That is a score with no checklist under it. Name the items, the frames and the reference, or name nothing |
|
|
110
|
+
| «The linter flagged it, but visually it's fine» | You sit below the deterministic floor. A G item or S1 that failed is `breaks`, and the fix is the code or the rule, not your opinion |
|
|
111
|
+
| «One more re-render will get it» | The budget is one, two at most. Past it, the item is `unresolved` on the sheet and goes to the person on their one pass |
|
|
112
|
+
|
|
113
|
+
Doctrine: `references/certification.md` → *The fourth reading*. The checks and the
|
|
114
|
+
contact sheet: `references/browser.md` → *The visual half*. The record's fields:
|
|
115
|
+
`references/spec.md` → *The COPY and VISUAL tracks*.
|
|
@@ -59,10 +59,16 @@ that reads no code is not the soft one.
|
|
|
59
59
|
| `seam` | everything that can reach the change — callers, callees, implementors, shared state, the neighbours' tests | a contract that moved under a dependent; the duplicate that did not get the fix |
|
|
60
60
|
| `product` | documentation, scenarios, user-visible strings, and the neighbouring features that share this path | a documented behaviour that is now false; an interaction nobody listed |
|
|
61
61
|
|
|
62
|
-
Agents: [
|
|
63
|
-
[`verifier-seam.md`](
|
|
64
|
-
[`verifier-product.md`](
|
|
65
|
-
[`verifier-visual.md`](
|
|
62
|
+
Agents: [`verifier-unit.md`](../assets/verifier-unit.md),
|
|
63
|
+
[`verifier-seam.md`](../assets/verifier-seam.md),
|
|
64
|
+
[`verifier-product.md`](../assets/verifier-product.md) — and, on a visual surface,
|
|
65
|
+
[`verifier-visual.md`](../assets/verifier-visual.md), below. These portable procedures live inside the skill.
|
|
66
|
+
Use host-native delegation when available; without it, execute the same tier
|
|
67
|
+
procedures inline and disclose that self-review lacks independent contexts.
|
|
68
|
+
Do not count inline passes as independent certification or close the graph on
|
|
69
|
+
that basis: independence-dependent gates remain incomplete/unverified until
|
|
70
|
+
independent review is available.
|
|
71
|
+
The Claude plugin agent definitions carry the same bodies with host frontmatter.
|
|
66
72
|
|
|
67
73
|
## The fourth reading — `visual`, on a flagship or product surface
|
|
68
74
|
|
|
@@ -23,15 +23,14 @@ that acts *while the agent is working* rather than after the commit.
|
|
|
23
23
|
|
|
24
24
|
## The limit, before the capability
|
|
25
25
|
|
|
26
|
-
**
|
|
27
|
-
|
|
28
|
-
|
|
29
|
-
|
|
26
|
+
**This package ships a Claude Code hook adapter.** Other hosts may expose hooks,
|
|
27
|
+
but capability does not register this package's adapter. Check the active host's
|
|
28
|
+
version, adapter registration, event payload and a planted refusal before claiming
|
|
29
|
+
enforcement. Without a verified adapter, run the same gates explicitly and record
|
|
30
|
+
**`ungated`**. Native hook support alone is not protection for this project.
|
|
30
31
|
|
|
31
|
-
|
|
32
|
-
|
|
33
|
-
inside a transcript, and a false guarantee is worse than a stated absence: everyone
|
|
34
|
-
downstream stops checking.
|
|
32
|
+
Never describe a project as protected merely because a rule or a hook file exists.
|
|
33
|
+
A false guarantee makes downstream readers stop checking.
|
|
35
34
|
|
|
36
35
|
> **Provenance.** Every contract below is quoted from the Claude Code hooks
|
|
37
36
|
> reference (`code.claude.com/docs/en/hooks`), fetched **2026-08-03**. Re-fetch
|
|
@@ -271,7 +270,7 @@ entries carrying one id — which is the exact failure the lease existed to prev
|
|
|
271
270
|
|
|
272
271
|
| Excuse | Reality |
|
|
273
272
|
|---|---|
|
|
274
|
-
| "The hook is installed, so the repo is protected" | Only
|
|
273
|
+
| "The hook is installed, so the repo is protected" | Only after this adapter was verified on the active host. For the documented Claude command hook, exit 2 refuses; other errors fail open. |
|
|
275
274
|
| "It's fine, the guard can't crash" | Then it costs one line to make crashing block instead of pass. Write the line. |
|
|
276
275
|
| "I'll match `*` and filter inside the script" | Now every tool call pays your script's startup. Match narrowly; the matcher is free and the script is not. |
|
|
277
276
|
| "The hook is slow, I'll disable it for now" | "For now" survives the session and the memory of why. Fix the throttle. |
|