@event4u/agent-config 9.12.0 → 9.13.0

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.
Files changed (130) hide show
  1. package/.claude-plugin/marketplace.json +1 -1
  2. package/CHANGELOG.md +80 -256
  3. package/README.md +1 -1
  4. package/dist/agent-src/rules/active-remediation.md +0 -2
  5. package/dist/agent-src/rules/architecture.md +0 -1
  6. package/dist/agent-src/rules/artifact-drafting-protocol.md +0 -4
  7. package/dist/agent-src/rules/autonomous-execution.md +1 -2
  8. package/dist/agent-src/rules/broken-access-control.md +0 -2
  9. package/dist/agent-src/rules/code-comment-discipline.md +0 -3
  10. package/dist/agent-src/rules/code-provenance.md +0 -3
  11. package/dist/agent-src/rules/communication-through-line.md +2 -3
  12. package/dist/agent-src/rules/content-quoting-floor.md +0 -2
  13. package/dist/agent-src/rules/context-hygiene.md +1 -3
  14. package/dist/agent-src/rules/decision-revisit-gate.md +0 -2
  15. package/dist/agent-src/rules/design-fidelity.md +29 -0
  16. package/dist/agent-src/rules/domain-adoption-policy.md +0 -3
  17. package/dist/agent-src/rules/downstream-changes.md +10 -1
  18. package/dist/agent-src/rules/external-reference-deep-dive.md +0 -7
  19. package/dist/agent-src/rules/fast-path-marker-visibility.md +0 -1
  20. package/dist/agent-src/rules/git-history-discipline.md +0 -7
  21. package/dist/agent-src/rules/guidelines.md +0 -2
  22. package/dist/agent-src/rules/improve-before-implement.md +4 -2
  23. package/dist/agent-src/rules/invite-challenge.md +0 -4
  24. package/dist/agent-src/rules/linked-projects-onboarding-gate.md +0 -2
  25. package/dist/agent-src/rules/markdown-safe-codeblocks.md +0 -1
  26. package/dist/agent-src/rules/minimal-safe-diff.md +2 -2
  27. package/dist/agent-src/rules/missing-tool-handling.md +0 -1
  28. package/dist/agent-src/rules/no-attribution-footers.md +0 -3
  29. package/dist/agent-src/rules/no-decorative-emojis-in-git-surfaces.md +8 -5
  30. package/dist/agent-src/rules/no-pr-progress-comments.md +1 -3
  31. package/dist/agent-src/rules/no-roadmap-references.md +0 -2
  32. package/dist/agent-src/rules/output-discipline.md +0 -1
  33. package/dist/agent-src/rules/prefer-enums-over-literals.md +0 -2
  34. package/dist/agent-src/rules/preservation-guard.md +0 -3
  35. package/dist/agent-src/rules/role-mode-adherence.md +0 -1
  36. package/dist/agent-src/rules/secret-vcs-guard.md +0 -2
  37. package/dist/agent-src/rules/senior-engineering-discipline.md +0 -2
  38. package/dist/agent-src/rules/session-canary.md +0 -2
  39. package/dist/agent-src/rules/size-enforcement.md +0 -5
  40. package/dist/agent-src/rules/source-confidentiality.md +0 -3
  41. package/dist/agent-src/rules/source-discovery-gate.md +0 -2
  42. package/dist/agent-src/rules/think-before-action.md +3 -3
  43. package/dist/agent-src/rules/token-budget-discipline.md +0 -1
  44. package/dist/agent-src/rules/token-efficiency.md +1 -2
  45. package/dist/agent-src/rules/tool-safety.md +0 -1
  46. package/dist/agent-src/rules/user-interaction.md +0 -3
  47. package/dist/agent-src/rules/user-interrupt-priority.md +0 -1
  48. package/dist/agent-src/skills/agent-docs-writing/SKILL.md +8 -0
  49. package/dist/agent-src/skills/analysis-skill-router/SKILL.md +55 -86
  50. package/dist/agent-src/skills/character-consistency/SKILL.md +7 -4
  51. package/dist/agent-src/skills/decision-record/SKILL.md +8 -0
  52. package/dist/agent-src/skills/design-review/SKILL.md +11 -0
  53. package/dist/agent-src/skills/design-system-capture/reference/design-system-json.md +8 -0
  54. package/dist/agent-src/skills/existing-ui-audit/SKILL.md +34 -3
  55. package/dist/agent-src/skills/fe-design/SKILL.md +10 -0
  56. package/dist/agent-src/skills/motion-choreographer/SKILL.md +7 -4
  57. package/dist/agent-src/skills/overbuild-review-lens/SKILL.md +201 -0
  58. package/dist/agent-src/skills/performance/SKILL.md +14 -0
  59. package/dist/agent-src/skills/pixar-storyteller/SKILL.md +8 -5
  60. package/dist/agent-src/skills/scene-expander/SKILL.md +9 -6
  61. package/dist/agent-src/skills/supply-chain-intake/SKILL.md +10 -0
  62. package/dist/agent-src/skills/universal-project-analysis/SKILL.md +14 -37
  63. package/dist/agent-src/skills/video-director/SKILL.md +9 -6
  64. package/dist/agent-src/templates/agents/agent-project-settings.example.yml +1 -1
  65. package/dist/agent-src/templates/scripts/work_engine/directives/ui/apply.ts +154 -13
  66. package/dist/agent-src/templates/scripts/work_engine/directives/ui/design.ts +118 -0
  67. package/dist/agent-src/templates/scripts/work_engine/directives/ui/polish.ts +40 -0
  68. package/dist/agent-src/templates/scripts/work_engine/state.ts +55 -0
  69. package/dist/discovery/deprecation-report.md +1 -1
  70. package/dist/discovery/discovery-manifest.json +89 -67
  71. package/dist/discovery/discovery-manifest.json.sha256 +1 -1
  72. package/dist/discovery/discovery-manifest.summary.md +3 -3
  73. package/dist/discovery/orphan-report.md +1 -1
  74. package/dist/discovery/packs.json +6 -5
  75. package/dist/discovery/trust-report.md +2 -2
  76. package/dist/discovery/workspaces.json +5 -4
  77. package/dist/hooks/dispatch.js +174 -13
  78. package/dist/mcp/registry-manifest.json +3 -3
  79. package/dist/mcp/server.json +2 -2
  80. package/dist/mcp/server.mjs +750 -507
  81. package/dist/router.json +1 -1
  82. package/dist/scripts/condense.js +2 -2
  83. package/docs/MIGRATION.md +1 -1
  84. package/docs/guidelines/agent-infra/agent-interaction-and-decision-quality.md +71 -0
  85. package/docs/guidelines/agent-infra/minimal-safe-diff-mechanics.md +24 -1
  86. package/docs/guidelines/code-clarity.md +9 -2
  87. package/docs/guidelines/design-fidelity-mechanics.md +44 -0
  88. package/llms.txt +63 -27
  89. package/package.json +4 -4
  90. package/src/agent-src/templates/scripts/work_engine/directives/ui/apply.ts +154 -13
  91. package/src/agent-src/templates/scripts/work_engine/directives/ui/design.ts +118 -0
  92. package/src/agent-src/templates/scripts/work_engine/directives/ui/polish.ts +40 -0
  93. package/src/agent-src/templates/scripts/work_engine/state.ts +55 -0
  94. package/src/config/agent-settings.template.yml +8 -0
  95. package/src/scripts/_lib/counted_probe.ts +142 -0
  96. package/src/scripts/_lib/overbuild_lens_contract.ts +196 -0
  97. package/src/scripts/_lib/release_scope.ts +207 -0
  98. package/src/scripts/ai_council/stance_tally.ts +37 -2
  99. package/src/scripts/audit_auto_rules.ts +9 -9
  100. package/src/scripts/audit_likelihood.ts +3 -3
  101. package/src/scripts/audit_overlap.ts +2 -2
  102. package/src/scripts/audit_skill_overlap.ts +233 -95
  103. package/src/scripts/audit_skill_overlap_allowlist.json +4 -0
  104. package/src/scripts/build_proof.ts +15 -1
  105. package/src/scripts/check_dependency_floors.ts +16 -6
  106. package/src/scripts/check_gate_coverage.ts +6 -6
  107. package/src/scripts/check_kernel_rule_bundle.ts +5 -4
  108. package/src/scripts/check_token_quality_golden.ts +0 -18
  109. package/src/scripts/code_graph/loader.ts +35 -6
  110. package/src/scripts/compile_router.ts +3 -8
  111. package/src/scripts/condense.ts +2 -2
  112. package/src/scripts/explain_run.ts +0 -1
  113. package/src/scripts/generate_pack_manifests.ts +3 -20
  114. package/src/scripts/hooks/block_kernel_rule_writes.ts +145 -10
  115. package/src/scripts/hooks/block_no_verify.ts +101 -0
  116. package/src/scripts/lint_agent_security.ts +5 -4
  117. package/src/scripts/lint_framework_leakage_allowlist.json +2 -2
  118. package/src/scripts/lint_regression.ts +92 -1
  119. package/src/scripts/lint_skill_descriptions.ts +93 -5
  120. package/src/scripts/project_thin_rules.ts +2 -2
  121. package/src/scripts/release.ts +107 -3
  122. package/src/scripts/resolve_lint_scope.ts +53 -0
  123. package/src/scripts/router_telemetry.ts +0 -5
  124. package/src/scripts/runtime_dispatcher.ts +40 -0
  125. package/src/scripts/schemas/pack.schema.json +1 -5
  126. package/src/scripts/schemas/rule.schema.json +1 -2
  127. package/src/scripts/self_review_gate.ts +15 -73
  128. package/src/scripts/skill_linter.ts +93 -11
  129. package/src/scripts/sweep_dead_scan_roots.ts +550 -0
  130. package/src/scripts/trigger_coverage.ts +11 -37
@@ -181,12 +181,15 @@ Worked example — `creature`:
181
181
 
182
182
  ## Policies
183
183
 
184
+ Paths, enforcement model, and the full set: the
185
+ [media policy preamble](../../../agents/settings/policies/media/README.md).
186
+
184
187
  When a character lock would identify or render a real person, consult before emitting the JSON:
185
188
 
186
- - [`agents/settings/policies/media/likeness.md`](../../../agents/settings/policies/media/likeness.md) — real-person identity tokens require a cited likeness release.
187
- - [`agents/settings/policies/media/public-figures.md`](../../../agents/settings/policies/media/public-figures.md) — recognised public figures carry the harder gate (publicity rights + transformative-intent).
188
- - [`agents/settings/policies/media/voice-cloning.md`](../../../agents/settings/policies/media/voice-cloning.md) — when `voice_note` references a real person's voice.
189
- - [`agents/settings/policies/media/disclosure.md`](../../../agents/settings/policies/media/disclosure.md) — outputs carrying a real-person lock require the non-removable AI-generation disclosure downstream.
189
+ - **`likeness`** — real-person identity tokens require a cited likeness release.
190
+ - **`public-figures`** — recognised public figures carry the harder gate (publicity rights + transformative-intent).
191
+ - **`voice-cloning`** — when `voice_note` references a real person's voice.
192
+ - **`disclosure`** — outputs carrying a real-person lock require the non-removable AI-generation disclosure downstream.
190
193
 
191
194
  Refuse-and-surface the file path; do not silently sanitise the prompt.
192
195
 
@@ -58,6 +58,14 @@ Do NOT use when:
58
58
  get a one-page record; one-way doors get the full options +
59
59
  consequences + supersession chain. The reversibility row of the
60
60
  matrix decides which template fires. See `mental-models.md` § 10.
61
+ **Corollary — a shortcut through a one-way door is a decision, not a
62
+ defer.** The cheap-and-simple option is *preferred* where the choice can
63
+ be walked back later. Where it cannot — a public API shape, a DB schema,
64
+ a migration, a wire format, a published identifier — "we can simplify
65
+ this later" is false, so it never ships as a deferred note; it lands
66
+ here as a recorded decision with its consequences named. The one-way
67
+ categories are enumerated so the call is not left to a judgement about
68
+ what feels important.
61
69
  - **Team context-spine slot.** Read the **team** slot of the
62
70
  [context-spine](../../../docs/contracts/context-spine.md) to
63
71
  capture who is bound by the decision (review groups, on-call
@@ -352,6 +352,17 @@ structured review phases, add an explicit **Anti-Slop Check** section:
352
352
  3. Run the AI-slop originality self-test on the overall aesthetic direction.
353
353
  Report the result (pass / flag / fail) with one sentence of evidence — this
354
354
  is the human judgment the detector deliberately does not make.
355
+ 4. **Mark the findings a provided artifact already answers.** When the run is a
356
+ port (`state.ui_design.provided_artifact` is set), a finding about a
357
+ decision the artifact makes is correct about the pattern and wrong about the
358
+ action. Keep it in the output, cite it as **"matches provided spec"**, and
359
+ set `artifact_covered: true` on it — the polish gate drops those from the
360
+ round-driving set, so an unmarked finding will send a round at the user's
361
+ own design. The flag is scoped to decisions the artifact actually covers;
362
+ anything it leaves open (a state it never showed) is generative work and
363
+ keeps the full scan. Precedence + scope:
364
+ [`design-fidelity-mechanics`](../../../docs/guidelines/design-fidelity-mechanics.md)
365
+ § Provided-artifact precedence. Regression witness: `daf-slop-vs-provided`.
355
366
 
356
367
  For the **objective quality floors** (WCAG contrast, font-size, line-length,
357
368
  reduced-motion, heading hierarchy, focus indicator), do NOT eyeball them —
@@ -42,6 +42,14 @@
42
42
  - **`bundled_local`** on a font family is a flag, not an instruction: the
43
43
  package never downloads or bundles fonts (out of scope) — it records that the
44
44
  source did.
45
+ - **`motion` has a consumer.** It sits outside the DTCG mapping above, and for
46
+ a long time that meant nothing read it at all: the block was written by
47
+ capture and consumed by nobody, so easing and duration were re-derived from
48
+ taste on every port. On the **Port a provided artifact** branch
49
+ ([`design-artifact-lifecycle`](../../../../../docs/contracts/design-artifact-lifecycle.md))
50
+ the audit reads `motion.durations` and `motion.easings` as the answer. The
51
+ extraction-only marking above still holds: `motion._meta.detected_libs`
52
+ informs the human and never becomes a token.
45
53
 
46
54
  ## Trust posture (mandatory)
47
55
 
@@ -104,9 +104,25 @@ Write into `state.ui_audit.design_tokens` (object, never null — empty object i
104
104
  | `:root { --... }` blocks in `resources/css/`, `app/globals.css`, `src/app/globals.css` | every `--token-name: value` pair |
105
105
  | `theme.json` / `tokens.json` (any depth) | flat or nested token tree |
106
106
  | `app/css/variables.css`, `assets/scss/_tokens.scss` | SCSS `$var: value` and CSS custom properties |
107
+ | **A supplied `design-system.json`** (handed over with a provided artifact) | `colors`, `typography`, `spacing`, `radius`, `shadow`, **and `motion`** (`durations`, `easings`) — read as the answer, never re-derived |
108
+ | **A supplied artifact's own `:root { --… }` / inline `<style>`** (the handover file itself, e.g. `design.html`) | every `--token-name: value` pair, when no `design-system.json` accompanies it |
107
109
 
108
110
  Group output by category: `colors`, `spacing`, `radius`, `font`, `shadow`, `breakpoint`, `other`.
109
111
 
112
+ **Artifact-sourced tokens stay distinguishable from project tokens.** Every
113
+ group carries a `source` — `project` for the first four rows, `artifact` for the
114
+ last two — because the mapping between them is the thing a port has to keep
115
+ visible. Collapsing them loses the answer to "did this value come from what the
116
+ user handed me, or from what the repo already had?", and that is exactly the
117
+ question the apply coverage report has to answer per item. The two sets may
118
+ disagree; when they do, surface the conflict rather than merging it (a supplied
119
+ spec outranks house taste, but not a registered brand token — see
120
+ [`brand-source-of-truth`](../../rules/brand-source-of-truth.md)).
121
+
122
+ `motion` is new here: the block has existed in the `design-system.json` schema
123
+ since capture shipped and **nothing consumed it**. On the port branch it is
124
+ read, so easing and duration stop being values the brief silently regenerates.
125
+
110
126
  ### 4. Detect shadcn inventory (only when `state.stack.frontend == "react-shadcn"`)
111
127
 
112
128
  Read `components.json` for the registered style + base color, then read `package.json` for `@radix-ui/*` and any locally vendored `components/ui/*.tsx` files. Write into `state.ui_audit.shadcn_inventory`:
@@ -265,9 +281,24 @@ scheme, border-radius convention, motion patterns), cross-check the findings
265
281
  against
266
282
  [`docs/guidelines/design-antipatterns.md`](../../../docs/guidelines/design-antipatterns.md).
267
283
  If the existing UI already uses a listed anti-pattern, surface it as a design-debt
268
- finding (separate from the reuse inventory) — flag by entry ID and severity. The
269
- next design step can then decide whether to continue the existing pattern (for
270
- consistency) or introduce a corrective direction change.
284
+ finding (separate from the reuse inventory) — flag by entry ID and severity.
285
+
286
+ **What the design step may then do depends on whose aesthetic it is.** These
287
+ are two different situations and the sentence that used to cover both was
288
+ wrong about one of them:
289
+
290
+ | The anti-pattern lives in… | The design step may… |
291
+ |---|---|
292
+ | **The consumer's own legacy UI** (inventoried from the repo) | continue it for consistency, **or** propose a corrective direction change. Both are legitimate; it is their codebase and their debt. |
293
+ | **A supplied spec** (a handed-over artifact, a `design-system.json`, a registered brand token) | **neither.** Build it as given. The finding is recorded as informational, marked `artifact_covered`, and no polish round acts on it. |
294
+
295
+ A corrective direction change against a supplied spec is not a design
296
+ improvement, it is overriding a decision the user already made — the failure
297
+ [`design-fidelity`](../../rules/design-fidelity.md) exists to prevent, arriving
298
+ through the audit's side door. Precedence and its exact scope:
299
+ [`design-fidelity-mechanics`](../../../docs/guidelines/design-fidelity-mechanics.md)
300
+ § Provided-artifact precedence. Where the two sources disagree, surface the
301
+ conflict rather than picking (fixture `daf-slop-vs-provided`).
271
302
 
272
303
  ## Do NOT
273
304
 
@@ -340,6 +340,16 @@ If a tell was the first impulse, choose a different approach or explicitly
340
340
  justify why this brief calls for it. (The full pre-proposal scan is under
341
341
  *Anti-slop discipline* below.)
342
342
 
343
+ **Carve-out — a supplied artifact is the spec, not a first impulse.** This
344
+ discipline governs what *you* reach for. When the user hands over a finished
345
+ artifact, the choices it makes are not your impulses and the justify-or-change
346
+ burden does not apply to them: build them as given. The carve-out is scoped to
347
+ decisions the artifact actually covers — anything it leaves open (a state it
348
+ never shows, a surface it never had) is your first impulse again and gets the
349
+ full scan. See
350
+ [`design-fidelity-mechanics`](../../../docs/guidelines/design-fidelity-mechanics.md)
351
+ § Provided-artifact precedence.
352
+
343
353
  ## Aesthetic direction
344
354
 
345
355
  Audit-pinned tokens and components always take precedence (see `existing-ui-audit`). When the audit pins an aesthetic, honor it without deviation. When the audit shows **no pinned aesthetic** — greenfield surface, marketing landing page, brand-new feature without design-system precedent — the design brief is allowed (and expected) to commit to a deliberate direction instead of defaulting to safe centered hero + 3-column features + CTA.
@@ -186,12 +186,15 @@ stdin shape. The orchestrator pipes this into the video adapter's
186
186
 
187
187
  ## Policies
188
188
 
189
+ Paths, enforcement model, and the full set: the
190
+ [media policy preamble](../../../agents/settings/policies/media/README.md).
191
+
189
192
  Motion prompts inherit every constraint the upstream blueprint carries. Before emitting provider-tuned prose:
190
193
 
191
- - [`agents/settings/policies/media/disclosure.md`](../../../agents/settings/policies/media/disclosure.md) — every distributed clip carries the non-removable AI-generation disclosure; refuse adapter flags that would suppress it.
192
- - [`agents/settings/policies/media/transparency.md`](../../../agents/settings/policies/media/transparency.md) — provider provenance (C2PA / SynthID) is preserved; refuse re-encode flags whose effect is to strip provenance.
193
- - [`agents/settings/policies/media/voice-cloning.md`](../../../agents/settings/policies/media/voice-cloning.md) — when the motion prompt requests `audio: native` narration in a named voice.
194
- - [`agents/settings/policies/media/brand-impersonation.md`](../../../agents/settings/policies/media/brand-impersonation.md) — when the motion prompt copies a recognised brand's chyron / mascot / signature transition.
194
+ - **`disclosure`** — every distributed clip carries the non-removable AI-generation disclosure; refuse adapter flags that would suppress it.
195
+ - **`transparency`** — provider provenance (C2PA / SynthID) is preserved; refuse re-encode flags whose effect is to strip provenance.
196
+ - **`voice-cloning`** — when the motion prompt requests `audio: native` narration in a named voice.
197
+ - **`brand-impersonation`** — when the motion prompt copies a recognised brand's chyron / mascot / signature transition.
195
198
 
196
199
  Refuse-and-surface; the motion prompt cannot launder a policy gap upstream skills should have caught.
197
200
 
@@ -0,0 +1,201 @@
1
+ ---
2
+ model_tier: high
3
+ name: overbuild-review-lens
4
+ description: "Use when a diff builds more than the task needs — code that should not exist, a dependency the platform already covers, or a clever form where a flat one reads better. Deletion-hunting, not quality."
5
+ personas:
6
+ - senior-engineer
7
+ domain: quality
8
+ workspaces:
9
+ - engineering
10
+ packs:
11
+ - engineering-base
12
+ ---
13
+
14
+ # overbuild-review-lens
15
+
16
+ > A deletion-hunting lens. It asks one question the other judges never
17
+ > ask: **should this code exist at all?** Sibling of
18
+ > [`architecture-review-lens`](../architecture-review-lens/SKILL.md) and
19
+ > the `judge-*` family — never overlaps with them.
20
+
21
+ ## When to use
22
+
23
+ - A diff adds a helper, a wrapper, a config layer, an abstraction, or a
24
+ dependency.
25
+ - A rewrite, v2, or large refactor landed — the peak over-build context
26
+ (see [`minimal-safe-diff-mechanics`](../../../docs/guidelines/agent-infra/minimal-safe-diff-mechanics.md)
27
+ § The sanctioned-rewrite trap).
28
+ - A reviewer asks "is all of this needed?", "could the platform do
29
+ this?", or "why is this so clever?".
30
+
31
+ Do NOT use when:
32
+
33
+ - The concern is whether the code is **correct** — route to
34
+ [`judge-bug-hunter`](../judge-bug-hunter/SKILL.md).
35
+ - The concern is **security** — route to
36
+ [`judge-security-auditor`](../judge-security-auditor/SKILL.md).
37
+ - The concern is **naming, single-responsibility, or convention fit** of
38
+ code whose existence is not in question — route to
39
+ [`judge-code-quality`](../judge-code-quality/SKILL.md). That judge asks
40
+ *is this code malformed*; this lens asks *should it exist*.
41
+ - The concern is **performance** — route to
42
+ [`performance`](../performance/SKILL.md). Speed is out of scope here.
43
+ - The diff is documentation-only or formatting-only.
44
+
45
+ ## The scope fence — read this before the first finding
46
+
47
+ ```
48
+ CORRECTNESS, SECURITY AND PERFORMANCE ARE OUT OF SCOPE FOR THIS LENS.
49
+ THE MINIMUM RUNNABLE CHECK IS NEVER FLAGGED FOR DELETION.
50
+ A SIZE REDUCTION THAT DROPS A GUARD IS A LOSS, NOT A FINDING.
51
+ ```
52
+
53
+ The measured failure mode this fence exists for: a bare "make it
54
+ simpler" critic was the only arm in a controlled comparison that dropped
55
+ a safety guard — the three lines it saved were a path-traversal check.
56
+ Tests, validation, authorization checks, tenant scoping, and error paths
57
+ that can actually occur are **not** over-build. If removing something
58
+ would reduce coverage of a real case, it is not a finding here.
59
+
60
+ ## Procedure
61
+
62
+ ### 1. Establish what the task actually asked for
63
+
64
+ Read the ticket / prompt, not just the diff. Over-build is measured
65
+ against the **requirement**, not against your taste. Anything the diff
66
+ adds beyond the requirement is a candidate; anything the requirement
67
+ demands is not, however large.
68
+
69
+ ### 2. Walk each addition down the solution-size ladder
70
+
71
+ For every new unit (function, class, module, config key, dependency),
72
+ find the highest rung that would have carried it — see
73
+ [`agent-interaction-and-decision-quality`](../../../docs/guidelines/agent-infra/agent-interaction-and-decision-quality.md)
74
+ § 8b-ladder:
75
+
76
+ | Tag | Meaning | The evidence that makes it a finding |
77
+ |---|---|---|
78
+ | `delete:` | This should not exist at all | A fence line (see § 3) — mandatory |
79
+ | `stdlib:` | The language stdlib or framework already does it | Name the exact API and show it is equivalent |
80
+ | `native:` | The OS / runtime / browser / database already does it | Name the platform capability and its availability floor |
81
+ | `yagni:` | Built for a requirement nobody stated | Quote the requirement it exceeds |
82
+ | `shrink:` | Must exist, but smaller | Name the lines that carry no requirement |
83
+ | `flatten:` | Must exist, but in a **simpler form** | Name the cognitive cost, not the line count |
84
+
85
+ `flatten:` is the shape-axis inverse of `shrink:` — same logic, simpler
86
+ form, **even when that costs a line or two**. Without it the lens only
87
+ ever argues downward and becomes a golfing engine. A nested ternary
88
+ replaced by an `if` block is a valid `flatten:` finding even though the
89
+ diff grows.
90
+
91
+ ### 3. Every `delete:` carries a fence line — no exceptions
92
+
93
+ Chesterton's Fence, operationalised. Agents are documented as
94
+ especially fence-blind: complex code reads as an invitation to simplify
95
+ when the complexity may exist for a reason the reader has no context
96
+ for. This does not restate
97
+ [`minimal-safe-diff`](../../../src/rules/minimal-safe-diff.md)'s "never
98
+ delete code that *looks* dead without proof" — it is the output contract
99
+ that makes the proof visible.
100
+
101
+ The fence line has three fields, all required:
102
+
103
+ ```
104
+ Fence: why=<why the code existed — blame, test, issue, or "unknown">
105
+ safe=<the evidence that removal is safe>
106
+ covered=<yes|no|partial — was the removed path under test?>
107
+ ```
108
+
109
+ - `why=unknown` is a legal value and it is **not** a pass. It downgrades
110
+ the finding to a question for the author, never an instruction.
111
+ - `covered=` is the one machine-checkable input to an otherwise
112
+ archaeological judgement (the Beyoncé rule: if you liked it, you put a
113
+ test on it). Deleting *tested* behaviour trips a test and is visible;
114
+ deleting *untested* behaviour breaks silently — so `covered=no` is
115
+ surfaced as **higher** risk, not as equivalent.
116
+
117
+ ### 4. Emit the null when there is nothing to cut
118
+
119
+ ```
120
+ A LENS THAT CANNOT SAY "NOTHING TO CUT" IS A FINDING GENERATOR.
121
+ ```
122
+
123
+ A lean diff must produce the honest null, not an invented finding. This
124
+ is the load-bearing output, not a fallback: a reviewer who sees findings
125
+ on every diff stops reading them.
126
+
127
+ ### 5. Validate before emitting
128
+
129
+ - Every finding has a `file:line` and exactly one tag from § 2.
130
+ - Every `delete:` has all three fence fields.
131
+ - No finding restates a correctness, security, or performance concern.
132
+ - The net-lines summary counts `flatten:` findings honestly, including
133
+ the ones that **add** lines.
134
+ - Zero findings → emit the null block, not an empty issues list.
135
+
136
+ ## Output format
137
+
138
+ The verdict block carries these ordered fields:
139
+
140
+ 1. `Lens:` — fixed value `overbuild-review-lens`
141
+ 2. `Target:` — the diff range or file set reviewed
142
+ 3. `Verdict:` — exactly one of `lean` / `trim` / `overbuilt`
143
+ 4. `Findings:` — one line per finding: tag, `file:line`, one sentence;
144
+ `delete:` findings carry their `Fence:` line directly beneath
145
+ 5. `Net:` — lines removable if every finding is applied, stated as a
146
+ signed number so a line-adding `flatten:` is visible
147
+
148
+ ```
149
+ Lens: overbuild-review-lens
150
+ Target: <branch / diff range>
151
+
152
+ Verdict: lean | trim | overbuilt
153
+
154
+ Findings:
155
+ 1. stdlib: src/util/uuid.ts:1 Hand-rolled v4 generator; crypto.randomUUID covers it.
156
+ 2. delete: src/config/flags.ts:12 Flag read by nothing since the feature shipped.
157
+ Fence: why=guarded the 2026-03 rollout safe=zero readers, grep clean covered=no
158
+ 3. flatten: src/parser.ts:88 Nested ternary; an if/else reads flat (+2 lines).
159
+
160
+ Net: -34 lines
161
+ ```
162
+
163
+ The null, when there is nothing to cut — emit verbatim, do not
164
+ paraphrase:
165
+
166
+ ```
167
+ Lens: overbuild-review-lens
168
+ Target: <branch / diff range>
169
+
170
+ Verdict: lean
171
+
172
+ Findings: none — nothing in this diff exceeds the stated requirement.
173
+
174
+ Net: 0 lines
175
+ ```
176
+
177
+ ## Gotcha
178
+
179
+ - **The requirement is the yardstick, not your preference.** A large
180
+ diff that the ticket demanded is not over-built. Say so and emit the
181
+ null.
182
+ - **`covered=no` makes a deletion riskier, not more attractive.** The
183
+ absence of a test is the absence of a tripwire.
184
+ - **A `flatten:` finding that saves lines is suspicious.** If it is
185
+ shorter *and* simpler it is probably a `shrink:`; `flatten:` exists
186
+ for the cases where simplicity costs length.
187
+ - **Never flag the minimum runnable check.** One happy-path test is not
188
+ over-build; it is the floor.
189
+ - **Do not count `delete:` lines that git already removed.** The net
190
+ figure is about the diff as proposed, not the diff's own deletions.
191
+
192
+ ## Do NOT
193
+
194
+ - Do NOT emit a `delete:` finding without all three fence fields — the
195
+ output contract rejects it.
196
+ - Do NOT invent a finding to avoid an empty report.
197
+ - Do NOT review correctness, security, or performance — other surfaces
198
+ own those, and a size argument against a guard is the canonical
199
+ failure this lens is fenced against.
200
+ - Do NOT report a size number as a score. A size metric is a
201
+ measurement, never a target.
@@ -28,6 +28,20 @@ Do NOT use when:
28
28
  3. Check for N+1 queries on list endpoints.
29
29
  4. Search for existing cache services in the project.
30
30
 
31
+ **Performance complexity is a claim, and a claim needs evidence.** No cache
32
+ layer, no denormalised column, no materialised view, no read replica ships on
33
+ "this will be slow" — it ships on a profile, a query log, or a timing that names
34
+ the bottleneck. The added complexity is permanent (invalidation bugs, stale
35
+ reads, a second source of truth); the speedup is hypothetical until measured.
36
+ State the measurement in the change, or do not add the mechanism.
37
+
38
+ **Carve-out — this does not override the `scale-discipline` floor.** Where that
39
+ pack is installed, R-A2 *mandates* an index on every FK and every WHERE / ORDER
40
+ BY column, and R-A4 governs duplicated data by waiver. Those are structural
41
+ defaults that ship with the query, not optimisations awaiting a profiler; the
42
+ evidence gate above applies to *added mechanisms* (caches, denormalisation,
43
+ replicas, bespoke indexes beyond parity), never to the floor's baseline.
44
+
31
45
  ### Step 1: Apply the right fix
32
46
 
33
47
  | Bottleneck | Fix |
@@ -1,7 +1,7 @@
1
1
  ---
2
2
  model_tier: inherit
3
3
  name: pixar-storyteller
4
- description: "Use when turning an idea into a Pixar-style animation prompt — character sheet, scene, image, video — anchored in emotional beat, want, obstacle. Triggers 'Pixar prompt', 'animated scene'."
4
+ description: "Use when an idea becomes a Pixar-style animation prompt — character sheet, scene, image, video; emotional beat, want, obstacle. Triggers 'Pixar prompt', 'animated scene'. Live-action → video-director."
5
5
  domain: product
6
6
  workspaces:
7
7
  - small-business
@@ -119,12 +119,15 @@ Any "no" → revise that block.
119
119
 
120
120
  ## Policies
121
121
 
122
+ Paths, enforcement model, and the full set: the
123
+ [media policy preamble](../../../agents/settings/policies/media/README.md).
124
+
122
125
  The Pixar-storyteller skill anchors prompts to named films and studios by design — the policy surface is the largest in the video cluster:
123
126
 
124
- - [`agents/settings/policies/media/style.md`](../../../agents/settings/policies/media/style.md) — naming a film + year as the *primary* anchor crosses the "in the style of [STUDIO]" trigger; surface and refuse without a transformative-intent rationale.
125
- - [`agents/settings/policies/media/likeness.md`](../../../agents/settings/policies/media/likeness.md) — when a beat references a named animator's signature character (real-person extension of style).
126
- - [`agents/settings/policies/media/public-figures.md`](../../../agents/settings/policies/media/public-figures.md) — when the storyteller's character is a recognised public figure rendered in Pixar shape.
127
- - [`agents/settings/policies/media/disclosure.md`](../../../agents/settings/policies/media/disclosure.md) — every Pixar-style output ships with the AI-generation disclosure; parody / commentary cases are flagged for human review.
127
+ - **`style`** — naming a film + year as the *primary* anchor crosses the "in the style of [STUDIO]" trigger; surface and refuse without a transformative-intent rationale.
128
+ - **`likeness`** — when a beat references a named animator's signature character (real-person extension of style).
129
+ - **`public-figures`** — when the storyteller's character is a recognised public figure rendered in Pixar shape.
130
+ - **`disclosure`** — every Pixar-style output ships with the AI-generation disclosure; parody / commentary cases are flagged for human review.
128
131
 
129
132
  Refuse-and-surface when style ⇒ primary signature, not one influence among several.
130
133
 
@@ -1,7 +1,7 @@
1
1
  ---
2
2
  model_tier: inherit
3
3
  name: scene-expander
4
- description: "Use when expanding a one-line idea into the 12-block Cinematic Scene Blueprint — provider-agnostic, includes optional dialogue + ambient. Triggers 'expand this scene', 'blueprint for X'."
4
+ description: "Use when expanding a one-line idea into the 12-block Cinematic Scene Blueprint — optional dialogue + ambient. Triggers 'expand this scene', 'blueprint for X'. 11-block refine → video-director."
5
5
  personas:
6
6
  - hollywood-director
7
7
  domain: product
@@ -146,13 +146,16 @@ Any "no" → revise that block.
146
146
 
147
147
  ## Policies
148
148
 
149
+ Paths, enforcement model, and the full set: the
150
+ [media policy preamble](../../../agents/settings/policies/media/README.md).
151
+
149
152
  The 12-block Cinematic Scene Blueprint is the policy choke point — every downstream skill (`motion-choreographer`, `video-director`) inherits whatever the blueprint encodes. Before emitting:
150
153
 
151
- - [`agents/settings/policies/media/likeness.md`](../../../agents/settings/policies/media/likeness.md) — when the SUBJECT block names or visually identifies a real person.
152
- - [`agents/settings/policies/media/public-figures.md`](../../../agents/settings/policies/media/public-figures.md) — when the SUBJECT block is a recognised public figure.
153
- - [`agents/settings/policies/media/brand-impersonation.md`](../../../agents/settings/policies/media/brand-impersonation.md) — when STYLE / ENVIRONMENT references a recognised brand's visual identity.
154
- - [`agents/settings/policies/media/style.md`](../../../agents/settings/policies/media/style.md) — when STYLE anchors to a named living artist or studio as the primary signature.
155
- - [`agents/settings/policies/media/disclosure.md`](../../../agents/settings/policies/media/disclosure.md) — every distributed blueprint output carries the AI-generation disclosure downstream.
154
+ - **`likeness`** — when the SUBJECT block names or visually identifies a real person.
155
+ - **`public-figures`** — when the SUBJECT block is a recognised public figure.
156
+ - **`brand-impersonation`** — when STYLE / ENVIRONMENT references a recognised brand's visual identity.
157
+ - **`style`** — when STYLE anchors to a named living artist or studio as the primary signature.
158
+ - **`disclosure`** — every distributed blueprint output carries the AI-generation disclosure downstream.
156
159
 
157
160
  Refuse-and-surface at the blueprint layer; do not push policy questions down to the adapter.
158
161
 
@@ -32,6 +32,16 @@ PIN IT, LOCK IT, CVE-SCAN IT. NEVER PIPE A REMOTE SCRIPT STRAIGHT TO A SHELL.
32
32
 
33
33
  ## Procedure — intake gate (run in order before adding a dependency)
34
34
 
35
+ 0. **Do you need a dependency at all?** The cheapest supply-chain risk is the
36
+ one never taken. Walk the rungs above "installed dependency" first — is it
37
+ already in the tree (`npm ls <pkg>`, `composer show`, `pip list`), does the
38
+ stdlib or framework carry it, does the **platform** already do it
39
+ (`crypto.randomUUID` before a uuid package, `Intl` before a formatting
40
+ library, `AbortSignal.timeout` before a timeout helper, the database's own
41
+ full-text / JSON support before an application-side index)? Full ordering:
42
+ [`agent-interaction-and-decision-quality` § 8b-ladder](../../../docs/guidelines/agent-infra/agent-interaction-and-decision-quality.md).
43
+ A dependency added for something already present is permanent cost — install
44
+ surface, CVE surface, upgrade surface — bought against a capability you had.
35
45
  1. **Existence** — confirm the exact string resolves on the real registry, published before your session and with real usage:
36
46
  ```bash
37
47
  npm view <pkg> version # non-zero exit = does not exist (hallucination)
@@ -31,15 +31,11 @@ Do NOT use when:
31
31
 
32
32
  ## Mission
33
33
 
34
- Act as the top-level router for deep project investigation.
35
-
36
- This skill must:
37
-
38
- * confirm whether full-project analysis is justified
39
- * identify the stack and framework
40
- * choose the correct analysis mode
41
- * route to the right specialist analysis skills
42
- * define the required output for broad project investigations
34
+ Run the deep multi-pass audit itself, and hand each pass to the right
35
+ specialist. Picking *which* skill a request needs in the first place belongs to
36
+ [`analysis-skill-router`](../analysis-skill-router/SKILL.md) — that chooser and
37
+ this orchestrator are deliberately separate files, and the scope test, the
38
+ detection step, and the decision table live there, not here.
43
39
 
44
40
  This skill must NOT become:
45
41
 
@@ -88,9 +84,9 @@ Goal: identify hot paths, find expensive boundaries, route into architecture or
88
84
 
89
85
  ### 1. Confirm scope
90
86
 
91
- Check whether full-project analysis is really needed.
92
- Use this skill only if the user wants: broad system understanding, architecture reconstruction, deep multi-layer debugging, broad audit across modules or runtime boundaries.
93
- If not: route to the narrower specialist skill directly.
87
+ Invoked through the router, this is already settled — proceed. Invoked
88
+ directly, apply the router's scope test before spending a multi-pass audit on
89
+ something a specialist covers.
94
90
 
95
91
  ### 2. Discover the project
96
92
 
@@ -128,36 +124,17 @@ Check:
128
124
  * uncertainties are marked
129
125
  * conclusions are evidence-based
130
126
 
131
- ## Routing map
132
-
133
- ### Universal analysis skills
134
-
135
- * `project-analysis-core`
136
- * `project-analysis-hypothesis-driven`
137
-
138
- ### Framework-specific deep dives
139
-
140
- * `project-analysis-laravel`
141
- * `project-analysis-symfony`
142
- * `project-analysis-zend-laminas`
143
- * `project-analysis-node-express`
144
- * `project-analysis-react`
145
- * `project-analysis-nextjs`
146
-
147
- ### Optional downstream specialists
148
-
149
- * `bug-analyzer`
150
- * `performance-analysis`
151
- * `security-audit`
152
-
153
- ### Ingestion preprocessor
127
+ ## Ingestion preprocessor
154
128
 
155
129
  * `markitdown` — when the project ships PDFs, DOCX, XLSX, PPTX, EPUB,
156
- images, or audio that need to feed into any of the analysis skills
157
- above. Convert first via the upstream `markitdown-mcp` server, then
130
+ images, or audio that need to feed into any of the routes in steps 3–4.
131
+ Convert first via the upstream `markitdown-mcp` server, then
158
132
  route the resulting Markdown into the relevant deep-dive skill.
159
133
  Never read a binary office format raw.
160
134
 
135
+ > The route lists in steps 3 and 4 are the routing map. A second copy of them
136
+ > lived here until 2026-08-02 and was removed — one list, one place.
137
+
161
138
  ## When to add a new framework analysis skill
162
139
 
163
140
  A framework gets its own `project-analysis-*` skill ONLY if:
@@ -1,7 +1,7 @@
1
1
  ---
2
2
  model_tier: inherit
3
3
  name: video-director
4
- description: "Use when turning a scene idea into the 11-block cinematic prompt for live-action AI video — lens, lighting, blocking, motion, negatives. Triggers 'cinematic prompt', 'film-grade scene'."
4
+ description: "Use when a live-action beat becomes the 11-block cinematic prompt — lens, lighting, negatives. Triggers 'cinematic prompt', 'film-grade scene'. Animated → pixar-storyteller; 12-block → scene-expander."
5
5
  personas:
6
6
  - hollywood-director
7
7
  domain: product
@@ -124,13 +124,16 @@ Any "no" → revise that block before handing off.
124
124
 
125
125
  ## Policies
126
126
 
127
+ Paths, enforcement model, and the full set: the
128
+ [media policy preamble](../../../agents/settings/policies/media/README.md).
129
+
127
130
  The 11-block cinematic prompt is live-action shape — real-person and brand-impersonation risks are the highest in the cluster. Before emitting:
128
131
 
129
- - [`agents/settings/policies/media/likeness.md`](../../../agents/settings/policies/media/likeness.md) — when the prompt names or visually identifies a real person on camera.
130
- - [`agents/settings/policies/media/public-figures.md`](../../../agents/settings/policies/media/public-figures.md) — when the subject is a recognised public figure.
131
- - [`agents/settings/policies/media/brand-impersonation.md`](../../../agents/settings/policies/media/brand-impersonation.md) — when the prompt copies a journalism / broadcaster / regulated-industry visual identity.
132
- - [`agents/settings/policies/media/style.md`](../../../agents/settings/policies/media/style.md) — when LIGHT / LENS choices are anchored to a named living cinematographer's signature.
133
- - [`agents/settings/policies/media/disclosure.md`](../../../agents/settings/policies/media/disclosure.md) — every distributed live-action AI clip carries the non-removable AI-generation disclosure.
132
+ - **`likeness`** — when the prompt names or visually identifies a real person on camera.
133
+ - **`public-figures`** — when the subject is a recognised public figure.
134
+ - **`brand-impersonation`** — when the prompt copies a journalism / broadcaster / regulated-industry visual identity.
135
+ - **`style`** — when LIGHT / LENS choices are anchored to a named living cinematographer's signature.
136
+ - **`disclosure`** — every distributed live-action AI clip carries the non-removable AI-generation disclosure.
134
137
 
135
138
  Refuse-and-surface at the directorial layer; live-action realism amplifies every policy gap downstream.
136
139
 
@@ -39,7 +39,7 @@ schema_version: 1
39
39
  # CI guard: a release bump of `package.json` must update this value
40
40
  # in lockstep — see scripts/check_template_pin_drift.ts (road-to-
41
41
  # portable-runtime-and-update-check P3.3).
42
- agent_config_version: "9.12.0"
42
+ agent_config_version: "9.13.0"
43
43
 
44
44
  # --- Project identity ---
45
45
  project: