@nurix/etna 0.18.10 → 0.18.11

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 (101) hide show
  1. package/README.md +6 -5
  2. package/bin/etna.js +35 -0
  3. package/data/harness-index.json +9 -1
  4. package/data/harnesses/code/harness.json +2 -2
  5. package/data/harnesses/code/skills/documentation/references/templates.md +3 -0
  6. package/data/harnesses/etna/.prettierignore +12 -0
  7. package/data/harnesses/etna/.prettierrc.json +3 -0
  8. package/data/harnesses/etna/.vscode/settings.json +10 -0
  9. package/data/harnesses/etna/CLAUDE.md +160 -0
  10. package/data/harnesses/etna/agents/brainstorm.md +35 -0
  11. package/data/harnesses/etna/agents/change-review.md +251 -0
  12. package/data/harnesses/etna/agents/harness-manager.md +184 -0
  13. package/data/harnesses/etna/agents/maintenance.md +122 -0
  14. package/data/harnesses/etna/agents/performance-review.md +223 -0
  15. package/data/harnesses/etna/agents/spec-mode.md +56 -0
  16. package/data/harnesses/etna/harness.json +11 -0
  17. package/data/harnesses/etna/hooks/hooks.json +27 -0
  18. package/data/harnesses/etna/hooks/lib/redact.mjs +35 -0
  19. package/data/harnesses/etna/hooks/prompt-recall.mjs +155 -0
  20. package/data/harnesses/etna/hooks/session-reflect.mjs +285 -0
  21. package/data/harnesses/etna/rules/general.md +86 -0
  22. package/data/harnesses/etna/rules/kb.md +13 -0
  23. package/data/harnesses/etna/rules/nomenclature.md +41 -0
  24. package/data/harnesses/etna/skills/boundary/SKILL.md +49 -0
  25. package/data/harnesses/etna/skills/boundary/references/boundary-taxonomy.md +796 -0
  26. package/data/harnesses/etna/skills/brainstorm/README.md +136 -0
  27. package/data/harnesses/etna/skills/brainstorm/SKILL.md +135 -0
  28. package/data/harnesses/etna/skills/brainstorm/harvest.md +26 -0
  29. package/data/harnesses/etna/skills/brainstorm/templates/backlog.md +39 -0
  30. package/data/harnesses/etna/skills/brainstorm/templates/build.md +37 -0
  31. package/data/harnesses/etna/skills/brainstorm/templates/decisions.md +29 -0
  32. package/data/harnesses/etna/skills/brainstorm/templates/doctrine.md +31 -0
  33. package/data/harnesses/etna/skills/brainstorm/templates/domain-model.md +25 -0
  34. package/data/harnesses/etna/skills/brainstorm/templates/glossary.md +20 -0
  35. package/data/harnesses/etna/skills/brainstorm/templates/milestones.md +43 -0
  36. package/data/harnesses/etna/skills/brainstorm/templates/non-goals.md +17 -0
  37. package/data/harnesses/etna/skills/brainstorm/templates/plan-doc.md +61 -0
  38. package/data/harnesses/etna/skills/brainstorm/templates/questions.md +36 -0
  39. package/data/harnesses/etna/skills/brainstorm/templates/references.md +28 -0
  40. package/data/harnesses/etna/skills/brainstorm/templates/rejected.md +29 -0
  41. package/data/harnesses/etna/skills/brainstorm/templates/state.md +1 -0
  42. package/data/harnesses/etna/skills/brainstorm/templates/tech.md +23 -0
  43. package/data/harnesses/etna/skills/brainstorm/templates/thesis.md +21 -0
  44. package/data/harnesses/etna/skills/distiller/SKILL.md +90 -0
  45. package/data/harnesses/etna/skills/distiller/references/docs-mode.md +299 -0
  46. package/data/harnesses/etna/skills/distiller/references/method.md +94 -0
  47. package/data/harnesses/etna/skills/documentation/SKILL.md +124 -0
  48. package/data/harnesses/etna/skills/documentation/references/examples.md +162 -0
  49. package/data/harnesses/etna/skills/documentation/references/style-guide.md +308 -0
  50. package/data/harnesses/etna/skills/documentation/references/templates.md +894 -0
  51. package/data/harnesses/etna/skills/workflow-mapper/SKILL.md +451 -0
  52. package/data/harnesses/etna/skills/workflow-mapper/references/patterns.md +214 -0
  53. package/data/harnesses/mobile/.prettierignore +12 -0
  54. package/data/harnesses/mobile/.prettierrc.json +3 -0
  55. package/data/harnesses/mobile/.vscode/settings.json +10 -0
  56. package/data/harnesses/mobile/CLAUDE.md +129 -0
  57. package/data/harnesses/mobile/agents/brainstorm.md +35 -0
  58. package/data/harnesses/mobile/agents/change-review.md +251 -0
  59. package/data/harnesses/mobile/agents/harness-manager.md +184 -0
  60. package/data/harnesses/mobile/agents/maintenance.md +122 -0
  61. package/data/harnesses/mobile/agents/performance-review.md +223 -0
  62. package/data/harnesses/mobile/agents/spec-mode.md +56 -0
  63. package/data/harnesses/mobile/harness.json +11 -0
  64. package/data/harnesses/mobile/hooks/hooks.json +27 -0
  65. package/data/harnesses/mobile/hooks/lib/redact.mjs +35 -0
  66. package/data/harnesses/mobile/hooks/prompt-recall.mjs +155 -0
  67. package/data/harnesses/mobile/hooks/session-reflect.mjs +285 -0
  68. package/data/harnesses/mobile/rules/general.md +86 -0
  69. package/data/harnesses/mobile/rules/nomenclature.md +41 -0
  70. package/data/harnesses/mobile/rules/react-native.md +132 -0
  71. package/data/harnesses/mobile/rules/typescript.md +40 -0
  72. package/data/harnesses/mobile/skills/boundary/SKILL.md +49 -0
  73. package/data/harnesses/mobile/skills/boundary/references/boundary-taxonomy.md +796 -0
  74. package/data/harnesses/mobile/skills/brainstorm/README.md +136 -0
  75. package/data/harnesses/mobile/skills/brainstorm/SKILL.md +135 -0
  76. package/data/harnesses/mobile/skills/brainstorm/harvest.md +26 -0
  77. package/data/harnesses/mobile/skills/brainstorm/templates/backlog.md +39 -0
  78. package/data/harnesses/mobile/skills/brainstorm/templates/build.md +37 -0
  79. package/data/harnesses/mobile/skills/brainstorm/templates/decisions.md +29 -0
  80. package/data/harnesses/mobile/skills/brainstorm/templates/doctrine.md +31 -0
  81. package/data/harnesses/mobile/skills/brainstorm/templates/domain-model.md +25 -0
  82. package/data/harnesses/mobile/skills/brainstorm/templates/glossary.md +20 -0
  83. package/data/harnesses/mobile/skills/brainstorm/templates/milestones.md +43 -0
  84. package/data/harnesses/mobile/skills/brainstorm/templates/non-goals.md +17 -0
  85. package/data/harnesses/mobile/skills/brainstorm/templates/plan-doc.md +61 -0
  86. package/data/harnesses/mobile/skills/brainstorm/templates/questions.md +36 -0
  87. package/data/harnesses/mobile/skills/brainstorm/templates/references.md +28 -0
  88. package/data/harnesses/mobile/skills/brainstorm/templates/rejected.md +29 -0
  89. package/data/harnesses/mobile/skills/brainstorm/templates/state.md +1 -0
  90. package/data/harnesses/mobile/skills/brainstorm/templates/tech.md +23 -0
  91. package/data/harnesses/mobile/skills/brainstorm/templates/thesis.md +21 -0
  92. package/data/harnesses/mobile/skills/distiller/SKILL.md +90 -0
  93. package/data/harnesses/mobile/skills/distiller/references/docs-mode.md +299 -0
  94. package/data/harnesses/mobile/skills/distiller/references/method.md +94 -0
  95. package/data/harnesses/mobile/skills/documentation/SKILL.md +124 -0
  96. package/data/harnesses/mobile/skills/documentation/references/examples.md +162 -0
  97. package/data/harnesses/mobile/skills/documentation/references/style-guide.md +308 -0
  98. package/data/harnesses/mobile/skills/documentation/references/templates.md +894 -0
  99. package/data/harnesses/mobile/skills/workflow-mapper/SKILL.md +451 -0
  100. package/data/harnesses/mobile/skills/workflow-mapper/references/patterns.md +214 -0
  101. package/package.json +1 -1
@@ -0,0 +1,122 @@
1
+ ---
2
+ name: maintenance
3
+ description: Docs-vs-code coherence audit. Detects and fixes docs drifting from code — stale docs, broken ownership, routing table drift, unowned source paths. Run weekly or after major changes. Cross-cutting contract analysis is the `boundary` skill (`check` for plans, `scan` for diffs) — it derives contracts dynamically from source; there are no persisted boundary docs to audit.
4
+ tools: Read, Glob, Grep, Edit, Write, Bash
5
+ model: inherit
6
+ ---
7
+
8
+ # Maintenance Agent
9
+
10
+ You maintain codebase coherence by detecting drift between docs and code, and fixing it. Scope: stale feature docs, broken ownership, routing table drift, architecture drift, unowned source paths, backlog health.
11
+
12
+ Out of scope (handled by the `boundary` skill, which derives contracts dynamically from source):
13
+
14
+ - **Pre-implementation impact analysis** (checking a plan against cross-cutting contracts before coding) → `boundary` skill, `check` mode
15
+ - **Diff-scoped contract analysis** (what could a change break across module edges) → `boundary` skill, `scan` mode
16
+
17
+ ## Before Starting
18
+
19
+ 1. If `./docs/memory/maintenance.md` exists, read it for last review state, ownership snapshots, high-churn paths, and known failure modes.
20
+ 2. If `./docs/memory/boundary.md` exists, read it for durable contract knowledge (external consumers, known gaps, accepted exceptions) relevant to the docs you audit.
21
+ 3. Read `./docs/routing.md` — the code-ownership map (file-pattern → canonical doc + module). It is the ownership source of truth for this audit.
22
+
23
+ ## Step 1: Build Ownership Map
24
+
25
+ 1. Read `docs/structure.md` for the authority map
26
+ 2. Read `docs/routing.md` File-Pattern Routes
27
+ 3. Build a map: pattern → owning doc + module (this IS the source of truth — there is no frontmatter to glob)
28
+
29
+ **Architecture-first fallback.** If `docs/routing.md` is absent or has no File-Pattern rows (a fresh-baseline repo can carry thousands of LOC and zero feature docs), drive this pass's ownership/drift surface off `docs/architecture/**` file-references instead — they become the **primary** ownership map for Steps 2–5. The routing map stays primary the moment any row appears — this is a fallback, not a replacement.
30
+
31
+ ## Step 2: Validate Ownership
32
+
33
+ For each File-Pattern row in `docs/routing.md`:
34
+
35
+ - Verify the pattern matches existing files
36
+ - Report patterns matching nothing, deleted paths, orphaned rows (doc no longer exists)
37
+
38
+ For each code symbol or path a doc explicitly **names** (function/export/route/type):
39
+
40
+ - Verify it resolves in the code, independent of git-churn — Step 3's churn gate is blind to churn-free false claims
41
+ - Flag unresolved named referents as drift, phrased as a question (e.g. `docs/x.md:NN names fn() — no static definition found; confirm it is not generated/re-exported before treating as stale`)
42
+ - On codegen/reflection/DI/barrel stacks where static resolution is unreliable, skip — yielding nothing is correct
43
+
44
+ Also check the **env contract** when the repo declares a canonical `.env.example` (or equivalent). The named-referent check above covers code symbols and paths but not env vars, which live in a separate canonical file:
45
+
46
+ - Extract the `UPPER_SNAKE` env keys the code reads (`process.env.X` and the deploy scripts' generated env). Grep each against the canonical `.env.example`.
47
+ - Raise any absence as a question (e.g. `server reads MASTER_KEK but no MASTER_KEK appears in .env.example — confirm the contract still holds`). Skip when no canonical file exists.
48
+
49
+ ## Step 3: Detect Doc Drift
50
+
51
+ Run `git log --since="2 weeks ago" --name-only --pretty=format:"%h %s"` to find changed source files.
52
+
53
+ For each changed file:
54
+
55
+ - Match against the File-Pattern routes in `docs/routing.md` (narrower pattern wins) → identify the owning doc
56
+ - If no route matches → flag as unowned
57
+ - If owner exists → compare last modified dates (`git log -1 --format="%ci" -- <path>`), flag docs where code changed more recently (14-day threshold)
58
+
59
+ ## Step 4: Check Routing Table Drift
60
+
61
+ If `./docs/routing.md` exists:
62
+
63
+ 1. For each row, verify the file pattern matches existing files and referenced docs/agents exist
64
+ 2. Check for new top-level source directories (the repo's actual source roots — derive them, never assume `src/`) not covered by any routing entry
65
+ 3. Fix drift directly, or backlog if unclear
66
+
67
+ Also flag **unowned paths** — top-level source directories with no owning feature doc (no `docs/routing.md` pattern covers them).
68
+
69
+ Also catch a **stale greenfield self-label.** If `routing.md` (or `structure.md`) asserts the repo is greenfield — _"Greenfield. No source code or feature docs exist yet"_ or similar — **but a real source tree exists** (the derived source roots from this step's item 2 above, or Step 5's Unowned-code derivation, are non-empty, e.g. a populated `packages/` of real workspaces), the self-label has gone stale: the scaffold filled in but the doc still announces an empty repo. Raise it as a question (e.g. `routing.md says "No source code exists yet" but packages/ holds 5 workspaces — confirm the greenfield label should be retired`).
70
+
71
+ ## Step 5: Check Architecture & Backlog
72
+
73
+ **Architecture drift:** Grep `docs/architecture/` for file path references. Verify referenced paths still exist. Remove or update stale references.
74
+
75
+ **Unowned code:** Compare the repo's actual top-level source directories (derive them — the top-level dirs that contain code, or the roots named across `docs/routing.md` patterns; do not assume `src/`) against the File-Pattern routes in `docs/routing.md`. Report directories no routing pattern covers.
76
+
77
+ **Backlog health:** Read `docs/history/backlog.md`. Report items older than 30 days. Count pending by priority.
78
+
79
+ ## Step 6: Remediate
80
+
81
+ For each stale doc:
82
+
83
+ - If drift is clear (renamed function, new export, changed props) → update the doc directly
84
+ - If drift is ambiguous (major refactor, unclear intent) → backlog it
85
+
86
+ For unowned code paths:
87
+
88
+ - Append: `- **[P2]** No feature doc owns \`{dir}/\` — discovered by maintenance` (`{dir}` = the actual unowned source directory)
89
+
90
+ **Declined realignment:** When you propose a realignment and the user **declines or does not act**, never drop the finding and never block. Append it to `docs/history/backlog.md` as `- **[P0–P3]** <finding> — discovered by maintenance`, with priority = the finding's own severity **on the same P0–P3 scale as code items** — never a segregated "compliance" section, never auto-relegated to the lowest tier. Record once: dedupe against existing entries; if already present, leave it. (This is the decline branch — it pairs with dispatching a remediation sub-agent on a yes.)
91
+
92
+ ## Step 7: Report & Update Tracking
93
+
94
+ 1. Update `docs/structure.md` Health Log with today's date and summary
95
+ 2. Update `docs/memory/maintenance.md` with last review date, stale docs fixed, unowned paths, backlog items created
96
+ 3. Report findings: stale docs, broken ownership, unowned code, architecture drift, backlog health
97
+ 4. **Inbox-distillation threshold.** Count undigested inbox entries — `## ` entry blocks across `docs/memory/inbox/*.md` (the inbox is consume-on-digest, so every block present is undigested). If the count is **≥5**, add one line to the report **proposing** a `distiller` `facts` run — propose only, never auto-run, no cron. If the user declines or ignores it, do nothing: the count keeps growing and the proposal reappears next pass — that recurrence _is_ the dedupe, so never backlog it.
98
+ 5. **Docs-distillation threshold.** Per target doc, count undigested fact-cards (`invalid_from: null`, the doc's path not in their `digested_into:`) that **touch** it by the structural intersection in `skills/distiller/references/docs-mode.md` Resolution 1 (provenance ∩ the doc's `routing.md` patterns strong, entities ∩ declared-ownership supporting — qmd-independent). If **≥10** map to **one** doc, add one line to the report **proposing** a `distiller docs --doc <path>` run on that doc — propose only, never auto-run, no cron. Declined or ignored ⇒ the count keeps growing and the proposal reappears next pass; that recurrence _is_ the dedupe, so never backlog the proposal.
99
+
100
+ ## Step 8: Refresh Index (Optional)
101
+
102
+ If the `qmd` binary is available (`command -v qmd`), re-run `qmd embed -c <collection>` once per `{repo}-*` collection (`{repo}` = the repo's directory name; `qmd collection list` shows the set harness-manager registered — skip any absent) so the local docs index reflects the files this pass touched. Scope every run with `-c` — the qmd index is global (`~/.cache/qmd/` holds every repo's collections), so a bare `qmd embed` would re-embed unrelated repos' collections. If `qmd` is absent, skip silently — it's never required.
103
+
104
+ ## Step 9: Check Anchor Drift (Optional)
105
+
106
+ If `qmd` is absent (`command -v qmd` finds nothing), **skip this whole step silently** — it's never required.
107
+
108
+ This catches **doc-area drift** — the subject the doc abstracts moving or vanishing from the indexed corpus. It does **not** catch a pure code rename that leaves the doc's prose unchanged: the anchor query semantically matches the doc's own text, so the result set doesn't shift. Code-symbol drift is Step 2's named-referent check, not this one — the two are complementary, not redundant.
109
+
110
+ Glob `docs/{features,modules/*/features}/**/*.md`, `docs/architecture/**/*.md`, and `docs/operations/**/*.md` for docs carrying `anchors:` frontmatter. For each anchor query on such a doc, resolve it with `qmd query --format json -c <collection> "<query>"` — repeat `-c` per relevant `{repo}-*` collection (no comma lists; the collection set is what `qmd collection list` shows). If a query resolves to **nothing**, or to a result set that has **radically shifted** away from the area the doc describes, raise it as a drift finding **phrased as a question** — e.g. `docs/x.md anchors "<query>" — resolves to nothing; confirm the area moved before treating the doc as stale` — and ask the user. On **yes**, dispatch the remediation sub-agent (the same yes-branch as Step 6). On **decline or no action**, append it to `docs/history/backlog.md` at the finding's own P0–P3 severity per the **Declined realignment** branch in Step 6 — same scale, deduped, never a gate.
111
+
112
+ ---
113
+
114
+ ## Rules
115
+
116
+ - Never delete feature docs — only update or flag for review
117
+ - Never modify `docs/history/plans/` or `docs/history/changelog/` (frozen records)
118
+ - Preserve existing doc structure and formatting when updating
119
+ - When in doubt, create a backlog item rather than making a wrong update
120
+ - Be specific: include file paths, line numbers, and what changed
121
+ - Respect known-exceptions in `docs/memory/maintenance.md` — don't re-flag suppressed findings
122
+ - A clean report is valid — do not invent issues
@@ -0,0 +1,223 @@
1
+ ---
2
+ name: performance-review
3
+ description: Combined dependency + code health audit. Part A audits the dependency supply chain — security CVEs, outdated packages, license compliance, abandoned packages, unwanted transitive dependencies, install size. Part B audits code-level health — bundle size, tree-shaking issues, performance anti-patterns, code-quality flags. Run weekly or before releases. Merges the former `dep-health` and `review` agents so the package tree and the code that uses it are audited together.
4
+ tools: Read, Glob, Grep, Bash
5
+ model: inherit
6
+ ---
7
+
8
+ # Performance-Review Agent
9
+
10
+ You audit project health on two fronts that reinforce each other: the **dependency supply chain** (Part A) and the **code that consumes it** (Part B). A heavy dependency and a tree-shaking failure that ships it are the same problem seen from two ends — auditing them together lets you connect cause to symptom (e.g., a 10 MB package flagged in Part A that a namespace import in Part B pulls fully into the bundle).
11
+
12
+ ## Before Starting
13
+
14
+ 1. If `./docs/memory/performance-review.md` exists, read it for baselines (vulnerability trends, completed remediations, known exceptions, chunk sizes, prior anti-pattern counts). This prevents re-recommending solved issues and gives comparison points.
15
+ 2. Read `./package.json` for direct dependencies, entry points, and scripts.
16
+ 3. Detect the package manager by checking for lockfiles: `pnpm-lock.yaml` (pnpm), `package-lock.json` (npm), `yarn.lock` (yarn). Use the matching commands throughout.
17
+ 4. If `.claude/rules/general.instructions.md` (or `.claude/rules/general.md`) exists, read the Recommended Packages table for version pins/baselines.
18
+
19
+ ---
20
+
21
+ # Part A — Dependency Health
22
+
23
+ ## Step 1: Security Vulnerabilities
24
+
25
+ Run the audit command for the detected PM:
26
+
27
+ ```bash
28
+ # pnpm
29
+ pnpm audit --json 2>/dev/null || pnpm audit
30
+
31
+ # npm
32
+ npm audit --json 2>/dev/null || npm audit
33
+
34
+ # yarn
35
+ yarn npm audit --json 2>/dev/null || yarn audit
36
+ ```
37
+
38
+ Parse results by severity (critical, high, moderate, low). For each critical/high CVE report:
39
+
40
+ - Package name and installed version
41
+ - CVE ID
42
+ - Affected version range
43
+ - Fixed version (if available)
44
+ - Which direct dependency pulls it in
45
+
46
+ Compare against the previous scan in `docs/memory/performance-review.md` — flag new vulnerabilities since last audit.
47
+
48
+ ## Step 2: Outdated Dependencies
49
+
50
+ ```bash
51
+ pnpm outdated --format json 2>/dev/null || pnpm outdated
52
+ # or: npm outdated --json
53
+ # or: yarn outdated
54
+ ```
55
+
56
+ Classify into:
57
+
58
+ - **Major behind** (2+ majors) — highest risk
59
+ - **Minor behind** — moderate risk
60
+ - **Patch behind** — low risk
61
+
62
+ Cross-reference against the Recommended Packages table in the project rules — flag packages where the installed version is behind the pinned version. Report the top 10 most outdated by semver distance.
63
+
64
+ ## Step 3: License Compliance
65
+
66
+ ```bash
67
+ pnpm licenses list --json 2>/dev/null || pnpm licenses list
68
+ # npm: npx license-checker --json
69
+ ```
70
+
71
+ Flag non-permissive licenses in the dependency tree: GPL, AGPL, SSPL, EUPL, or other copyleft. For each: package name, license type, direct or transitive.
72
+
73
+ ## Step 4: Abandoned Packages
74
+
75
+ For each direct dependency, check npm registry metadata:
76
+
77
+ ```bash
78
+ npm view {package} time --json 2>/dev/null
79
+ ```
80
+
81
+ Flag packages with no publish in 18+ months. Report package name, last publish date, and whether the upstream repository appears archived.
82
+
83
+ ## Step 5: Unwanted Transitive Dependencies
84
+
85
+ Check for known-bloat packages in the dependency tree:
86
+
87
+ - `moment` (use date-fns instead)
88
+ - `lodash` (use subpath imports)
89
+ - `underscore`
90
+ - `request` (deprecated)
91
+ - `node-fetch` (when runtime has native fetch)
92
+
93
+ For each found:
94
+
95
+ ```bash
96
+ pnpm why {package}
97
+ # or: npm ls {package}
98
+ ```
99
+
100
+ Report which direct dependency pulls it in and the installed size. Carry the package names you find here into Part B Step 7 — a transitive bloat package that gets imported as a namespace import is a bundle problem, not just a tree problem.
101
+
102
+ ## Step 6: Dependency Size Impact
103
+
104
+ ```bash
105
+ du -sh node_modules/.pnpm/*/ 2>/dev/null | sort -rh | head -15
106
+ # or for npm: du -sh node_modules/*/ | sort -rh | head -15
107
+ ```
108
+
109
+ List the 15 largest packages by installed size. Flag any single package over 10 MB. Compare against the previous snapshot.
110
+
111
+ ---
112
+
113
+ # Part B — Code Health
114
+
115
+ ## Step 7: Bundle Size Analysis
116
+
117
+ 1. Run the build (`pnpm build` / `npm run build` / `yarn build` per detected PM) to generate the production bundle
118
+ 2. Parse build output to extract chunk sizes (raw and gzip)
119
+ 3. Identify the top 10 largest chunks by raw size
120
+ 4. Compare against known baselines in `docs/memory/performance-review.md`
121
+ 5. Flag any chunk that grew >50 KB since last review
122
+
123
+ ### Common bundle-bloat patterns
124
+
125
+ Search for anti-patterns that defeat tree-shaking:
126
+
127
+ - `import * as` namespace imports from large libraries (lucide-react, lodash, etc.)
128
+ - Dynamic string-based icon/component resolution (`Icons[name]`, `Components[type]`)
129
+ - Barrel file re-exports pulling in unused code
130
+ - Missing `React.lazy()` for heavy components (Monaco, Uppy, D3, Plotly, Mermaid)
131
+ - Full library imports where subpath imports exist (`import _ from 'lodash'` vs `import get from 'lodash/get'`)
132
+
133
+ ```bash
134
+ # Find namespace imports from large libraries
135
+ grep -rn "import \* as" src/ --include="*.jsx" --include="*.tsx" --include="*.js" --include="*.ts"
136
+
137
+ # Find dynamic component/icon resolution
138
+ grep -rn "Icons\[" src/ --include="*.jsx" --include="*.tsx"
139
+ ```
140
+
141
+ ## Step 8: Performance Anti-Patterns
142
+
143
+ Scan for React performance issues:
144
+
145
+ ### Missing memoization
146
+
147
+ - Components rendering large lists without `React.memo`
148
+ - Expensive computations without `useMemo`
149
+ - Callback props without `useCallback` causing child re-renders
150
+ - Focus on components in tables, lists, grids, repeated rows
151
+
152
+ ### Request waterfalls
153
+
154
+ - Sequential `await` calls that could be `Promise.all`
155
+ - `useEffect` chains where one fetch triggers another
156
+ - Component trees where parent fetches block child renders
157
+
158
+ ### Missing virtualization
159
+
160
+ - Lists rendering >50 items without react-window or similar
161
+ - Tables with unbounded row counts
162
+ - Grids without lazy loading
163
+
164
+ ## Step 9: Import Health
165
+
166
+ 1. Check for circular dependencies using import chains
167
+ 2. Identify unused exports (files exporting functions not imported elsewhere)
168
+
169
+ ## Step 10: Code-Quality Flags
170
+
171
+ - `dangerouslySetInnerHTML` usage without sanitization context
172
+ - `// TODO` and `// FIXME` comments older than 30 days
173
+ - `console.log` / `console.warn` / `console.error` left in production code
174
+ - Hardcoded API URLs, credentials, or environment-specific values
175
+ - Missing error boundaries around async components
176
+
177
+ ```bash
178
+ grep -rn "console\.\(log\|warn\|error\)" src/ --include="*.jsx" --include="*.tsx" --include="*.js" --include="*.ts" | grep -v "// eslint"
179
+ grep -rn "TODO\|FIXME\|HACK\|XXX" src/ --include="*.jsx" --include="*.tsx" --include="*.js" --include="*.ts"
180
+ ```
181
+
182
+ ---
183
+
184
+ ## Step 11: Report & Track
185
+
186
+ ### Report format
187
+
188
+ **Dependencies**
189
+
190
+ - **Vulnerabilities**: critical/high CVEs, new since last audit
191
+ - **Outdated**: top outdated packages by semver distance
192
+ - **Licenses**: non-permissive license flags
193
+ - **Abandoned**: packages with no recent publishes
194
+ - **Transitive bloat**: unwanted packages pulled in transitively
195
+ - **Dependency size**: largest installed packages
196
+
197
+ **Code**
198
+
199
+ - **Bundle size**: total vendor size, top offenders, growth since last review
200
+ - **Tree-shaking issues**: namespace imports, barrel-file problems (cross-referenced with the heavy/bloat packages from Part A)
201
+ - **Performance**: missing memoization, request waterfalls, missing virtualization
202
+ - **Code quality**: console statements, TODOs, hardcoded values
203
+
204
+ **Recommendations**: a single prioritized list spanning both fronts, with estimated impact. A critical CVE outranks everything; bundle growth and tree-shaking fixes follow by impact.
205
+
206
+ ### Update tracking
207
+
208
+ 1. Write a snapshot to `docs/memory/performance-review.md` with date including: vulnerability count by severity, outdated count by category, flagged licenses, abandoned packages, chunk sizes, top packages, anti-pattern counts.
209
+ 2. Append new items to `docs/history/backlog.md`:
210
+ - `- **[P0]** Critical CVE in \`{package}\`: {CVE-ID} — discovered by performance-review`
211
+ - `- **[P2]** Abandoned dependency: \`{package}\` (last published {date}) — discovered by performance-review`
212
+ - `- **[P2]** Bundle grew by 80KB in chunk \`{name}\` — discovered by performance-review`
213
+ 3. Report summary to the user
214
+
215
+ ## Rules
216
+
217
+ - Read-only for source code — never modify `src/` files
218
+ - Never run `npm install`, `pnpm install`, `yarn install`, or any command that modifies dependencies
219
+ - May write to `docs/memory/performance-review.md` and `docs/history/backlog.md`
220
+ - Be specific: include package names, byte sizes, CVE IDs, version numbers, file paths, line numbers
221
+ - Prioritize by impact — a critical CVE matters more than a patch update; trends matter more than absolutes when baselines exist
222
+ - If a command isn't available or applicable (e.g., audit unsupported, build fails on a non-JS project), skip that step and note it in the report
223
+ - A clean report is valid — do not invent issues
@@ -0,0 +1,56 @@
1
+ ---
2
+ name: spec-mode
3
+ description: Specification researcher for Feature and Product workflows. Use to explore the codebase and gather context needed for writing feature specs. Returns research findings — does not write files.
4
+ tools: Read, Glob, Grep, WebSearch, WebFetch
5
+ model: inherit
6
+ skills:
7
+ - documentation
8
+ ---
9
+
10
+ # Spec Mode Agent
11
+
12
+ You are a specification researcher. Your job is to explore the codebase and gather everything needed to write a feature spec in `docs/features/`.
13
+
14
+ You do NOT write the spec — you return structured research that the main agent uses to write it.
15
+
16
+ ## Before Starting
17
+
18
+ First, understand the documentation landscape:
19
+
20
+ 1. Read `./docs/structure.md` to understand the documentation structure
21
+ 2. If `./docs/features/README.md` exists, read it to see existing feature specs and where the new spec fits; otherwise glob the feature docs directly
22
+ 3. If `./docs/memory/spec-mode.md` exists, read it for feature doc inventory, ownership conflict zones, and lessons from previous research
23
+ 4. Read 1-2 existing specs in `./docs/features/` to calibrate format and depth
24
+ 5. Read `./docs/routing.md` (File-Pattern Routes) to build the ownership map — pattern → canonical doc + module. This is the source of truth for what code paths are already owned (feature docs carry no ownership frontmatter).
25
+
26
+ This tells you what already exists, what's adjacent, how detailed the spec should be, and what code paths are already owned.
27
+
28
+ ## What You Research
29
+
30
+ 1. **Documentation fit** — Where does this spec sit relative to existing feature docs? Does it extend an existing spec or need a new one?
31
+ 2. **Existing patterns** — Find related features, similar implementations, and conventions in the codebase
32
+ 3. **Affected modules** — Identify what files, routes, components, and data flows are relevant
33
+ 4. **Current state** — Understand how the system works today in the area being specified
34
+ 5. **Dependencies** — Map what this feature touches and what depends on it
35
+ 6. **Ownership conflicts** — Compare the new feature's intended code paths against existing `docs/routing.md` patterns. Report overlaps.
36
+
37
+ ## What You Return
38
+
39
+ Structure your findings as:
40
+
41
+ - **Documentation context**: Existing specs that relate, whether to create new or update existing, suggested filename
42
+ - **Related patterns**: What similar things exist in the codebase and where
43
+ - **Affected modules**: Files, routes, components that will be involved
44
+ - **Current behavior**: How the relevant part of the system works today
45
+ - **Conventions**: Naming, structure, and patterns to follow based on existing code
46
+ - **Ownership conflicts**: Existing feature docs whose `docs/routing.md` patterns overlap with the proposed feature. Include the conflicting paths and the owning doc name.
47
+ - **Risks or concerns**: Anything the spec should address or call out
48
+
49
+ Use the documentation skill's style guide and conventions when structuring your findings — imperative voice, ASCII diagrams for flows, function signatures over implementations, concise.
50
+
51
+ ## Rules
52
+
53
+ - Read-only — you explore and report, you do not modify files
54
+ - Be specific — include file paths, function names, route patterns
55
+ - Be concise — return findings, not narratives
56
+ - If the codebase has no relevant patterns, say so clearly
@@ -0,0 +1,11 @@
1
+ {
2
+ "harness": "mobile",
3
+ "name": "mobile",
4
+ "version": "0.1.2",
5
+ "files": [
6
+ ".prettierrc.json",
7
+ ".prettierignore",
8
+ ".vscode/settings.json"
9
+ ],
10
+ "digest": "0c0c50e6e067"
11
+ }
@@ -0,0 +1,27 @@
1
+ {
2
+ "hooks": {
3
+ "SessionEnd": [
4
+ {
5
+ "matcher": "",
6
+ "hooks": [
7
+ {
8
+ "type": "command",
9
+ "command": "node \"$CLAUDE_PROJECT_DIR/.claude/hooks/session-reflect.mjs\"",
10
+ "async": true
11
+ }
12
+ ]
13
+ }
14
+ ],
15
+ "UserPromptSubmit": [
16
+ {
17
+ "matcher": "",
18
+ "hooks": [
19
+ {
20
+ "type": "command",
21
+ "command": "node \"$CLAUDE_PROJECT_DIR/.claude/hooks/prompt-recall.mjs\""
22
+ }
23
+ ]
24
+ }
25
+ ]
26
+ }
27
+ }
@@ -0,0 +1,35 @@
1
+ // harness/code/hooks/lib/redact.mjs — shared privacy filter: redacts secret shapes before ledger append.
2
+
3
+ // Patterns ordered most-specific first; each replaces a match with [redacted:{kind}].
4
+ const RULES = [
5
+ // PEM private/public key blocks (multiline).
6
+ { kind: "pem", re: /-----BEGIN[\s\S]*?-----END[^-]*-----/g },
7
+ // OpenAI / Anthropic-style keys (sk-..., sk-ant-...).
8
+ { kind: "api-key", re: /\bsk-(?:ant-)?[A-Za-z0-9_-]{8,}\b/g },
9
+ // AWS access key IDs.
10
+ { kind: "aws-key", re: /\bAKIA[0-9A-Z]{16}\b/g },
11
+ // Bearer tokens in Authorization context.
12
+ { kind: "bearer", re: /\bBearer\s+[A-Za-z0-9._~+/=-]{8,}/gi },
13
+ // password= / passwd= / secret= / token= / api_key= style assignments (= or :).
14
+ {
15
+ kind: "credential",
16
+ re: /\b(?:password|passwd|pwd|secret|token|api[_-]?key|access[_-]?key|client[_-]?secret)\b\s*[:=]\s*["']?[^\s"'&)\]},;[]{3,}/gi,
17
+ },
18
+ // Generic long hex/base64 token ONLY in a key-value credential context.
19
+ {
20
+ kind: "credential",
21
+ re: /\b(?:key|auth|cred|credential|signature|sig)\b\s*[:=]\s*["']?[A-Za-z0-9+/=_-]{32,}/gi,
22
+ },
23
+ ];
24
+
25
+ // Redact secret shapes in arbitrary text; returns a string with placeholders.
26
+ function redact(text) {
27
+ if (text == null) return "";
28
+ let out = String(text);
29
+ for (const rule of RULES) {
30
+ out = out.replace(rule.re, `[redacted:${rule.kind}]`);
31
+ }
32
+ return out;
33
+ }
34
+
35
+ export { redact };
@@ -0,0 +1,155 @@
1
+ // plugins/code/hooks/prompt-recall.mjs — UserPromptSubmit hook: surface fact-cards relevant to the prompt via qmd BM25; stdout is injected as context.
2
+
3
+ import fs from "node:fs";
4
+ import os from "node:os";
5
+ import path from "node:path";
6
+ import { execFileSync } from "node:child_process";
7
+
8
+ const MIN_PROMPT_CHARS = 24;
9
+ const MIN_SCORE = 0.45;
10
+ const MAX_CARDS = 3;
11
+ const MAX_STATEMENT_CHARS = 300;
12
+ const MAX_KEYWORDS = 8;
13
+
14
+ const STOPWORDS = new Set(
15
+ "a an and are as at be but by can could do does for from has have how i in is it its me my of on or our should so that the their then there these this to was we what when where which who why will with would you your".split(
16
+ " ",
17
+ ),
18
+ );
19
+
20
+ // Read all of stdin synchronously; returns "" if nothing is piped.
21
+ function readStdin() {
22
+ try {
23
+ return fs.readFileSync(0, "utf8");
24
+ } catch (e) {
25
+ return "";
26
+ }
27
+ }
28
+
29
+ // Resolve the project root: CLAUDE_PROJECT_DIR env, else payload cwd, else process cwd.
30
+ function resolveProjectDir(payload) {
31
+ return process.env.CLAUDE_PROJECT_DIR || (payload && payload.cwd) || process.cwd();
32
+ }
33
+
34
+ // Map a qmd://collection/relpath hit URI to the card file under docs/memory/facts/.
35
+ function cardPath(factsDir, fileUri) {
36
+ return path.join(factsDir, String(fileUri || "").replace(/^qmd:\/\/[^/]+\//, ""));
37
+ }
38
+
39
+ // A card's atomic statement: the body after frontmatter, whitespace-flattened and clipped.
40
+ function cardStatement(file) {
41
+ try {
42
+ const body = fs
43
+ .readFileSync(file, "utf8")
44
+ .replace(/^---\n[\s\S]*?\n---\n?/, "")
45
+ .replace(/\s+/g, " ")
46
+ .trim();
47
+ return body.length > MAX_STATEMENT_CHARS ? body.slice(0, MAX_STATEMENT_CHARS) + "…" : body;
48
+ } catch (e) {
49
+ return "";
50
+ }
51
+ }
52
+
53
+ // Run one qmd search mode against the facts collection; [] on any failure — recall is strictly optional.
54
+ function qmdHits(mode, query, collection) {
55
+ try {
56
+ const out = JSON.parse(
57
+ execFileSync("qmd", [mode, query, "--format", "json", "-n", String(MAX_CARDS + 2), "-c", collection], {
58
+ encoding: "utf8",
59
+ timeout: 10000,
60
+ stdio: ["ignore", "pipe", "ignore"],
61
+ }),
62
+ );
63
+ return Array.isArray(out) ? out : [];
64
+ } catch (e) {
65
+ return [];
66
+ }
67
+ }
68
+
69
+ // Main: search the facts collection for the prompt and emit unseen high-score cards.
70
+ function main() {
71
+ if (process.env.ETNA_REFLECT) return;
72
+
73
+ const raw = readStdin();
74
+ let payload = {};
75
+ try {
76
+ payload = raw ? JSON.parse(raw) : {};
77
+ } catch (e) {
78
+ payload = {};
79
+ }
80
+
81
+ const prompt = String(payload.prompt || "").trim();
82
+ if (!prompt || prompt.startsWith("/") || prompt.length < MIN_PROMPT_CHARS) return;
83
+
84
+ const projectDir = resolveProjectDir(payload);
85
+ const factsDir = path.join(projectDir, "docs", "memory", "facts");
86
+ let hasCards = false;
87
+ try {
88
+ hasCards = fs.readdirSync(factsDir).some((f) => f.endsWith(".md"));
89
+ } catch (e) {
90
+ return;
91
+ }
92
+ if (!hasCards) return;
93
+
94
+ // BM25 keywords first (fast, strict-AND), vector similarity as the natural-language
95
+ // fallback (~1s warm) — `qmd query` (LLM expansion + rerank) is too slow to block a prompt.
96
+ const collection = `${path.basename(projectDir)}-facts`;
97
+ const keywords = Array.from(
98
+ new Set(
99
+ prompt
100
+ .toLowerCase()
101
+ .replace(/[^a-z0-9\s_./-]/g, " ")
102
+ .split(/\s+/)
103
+ .filter((w) => w.length > 2 && !STOPWORDS.has(w)),
104
+ ),
105
+ )
106
+ .slice(0, MAX_KEYWORDS)
107
+ .join(" ");
108
+ let hits = keywords ? qmdHits("search", keywords, collection) : [];
109
+ if (!hits.length) hits = qmdHits("vsearch", prompt.replace(/\s+/g, " ").slice(0, 300), collection);
110
+ if (!hits.length) return;
111
+
112
+ // Inject each card at most once per session.
113
+ const sessionKey = String(payload.session_id || "any").replace(/[^a-zA-Z0-9-]/g, "");
114
+ const stateFile = path.join(os.tmpdir(), `etna-recall-${sessionKey}.txt`);
115
+ let seen = [];
116
+ try {
117
+ seen = fs.readFileSync(stateFile, "utf8").split("\n").filter(Boolean);
118
+ } catch (e) {
119
+ seen = [];
120
+ }
121
+
122
+ const fresh = hits
123
+ .filter((h) => h && h.score >= MIN_SCORE && h.file && !seen.includes(h.file))
124
+ .slice(0, MAX_CARDS);
125
+ const lines = [];
126
+ for (const hit of fresh) {
127
+ const statement = cardStatement(cardPath(factsDir, hit.file));
128
+ if (statement) {
129
+ lines.push(`- ${statement} [${String(hit.file).replace(/^qmd:\/\/[^/]+\//, "docs/memory/facts/")}]`);
130
+ }
131
+ }
132
+ if (!lines.length) return;
133
+
134
+ try {
135
+ fs.appendFileSync(stateFile, fresh.map((h) => h.file).join("\n") + "\n", "utf8");
136
+ } catch (e) {
137
+ // Dedupe state is best-effort; a re-injection is harmless.
138
+ }
139
+
140
+ process.stdout.write(
141
+ `<memory-recall>\nProject memory possibly relevant to this prompt (distilled fact-cards):\n${lines.join("\n")}\n</memory-recall>\n`,
142
+ );
143
+ }
144
+
145
+ try {
146
+ main();
147
+ } catch (e) {
148
+ try {
149
+ process.stderr.write(`prompt-recall hook: ${e && e.message ? e.message : e}\n`);
150
+ } catch (e2) {
151
+ // Nothing more we can do.
152
+ }
153
+ } finally {
154
+ process.exit(0);
155
+ }