continuous-improvement 3.20.4 → 3.22.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 (37) hide show
  1. package/.claude-plugin/marketplace.json +2 -2
  2. package/CHANGELOG.md +23 -0
  3. package/QUICKSTART.md +1 -1
  4. package/README.md +7 -6
  5. package/bin/check-reconcile-parity.mjs +168 -0
  6. package/bin/generate-plugin-manifests.mjs +2 -0
  7. package/bin/install.mjs +30 -69
  8. package/bin/reconcile.mjs +259 -0
  9. package/commands/production-readiness-review.md +5 -4
  10. package/commands/reconcile.md +30 -6
  11. package/commands/simplicity-review.md +35 -0
  12. package/commands/verify-install.md +2 -2
  13. package/hooks/session.mjs +85 -0
  14. package/lib/git-state.mjs +411 -0
  15. package/lib/plugin-metadata.mjs +18 -20
  16. package/llms.txt +1 -1
  17. package/package.json +6 -4
  18. package/plugins/beginner.json +1 -1
  19. package/plugins/continuous-improvement/.claude-plugin/marketplace.json +2 -2
  20. package/plugins/continuous-improvement/.claude-plugin/plugin.json +2 -2
  21. package/plugins/continuous-improvement/bin/reconcile.mjs +259 -0
  22. package/plugins/continuous-improvement/commands/production-readiness-review.md +5 -4
  23. package/plugins/continuous-improvement/commands/reconcile.md +30 -6
  24. package/plugins/continuous-improvement/commands/simplicity-review.md +35 -0
  25. package/plugins/continuous-improvement/commands/verify-install.md +2 -2
  26. package/plugins/continuous-improvement/hooks/hooks.json +15 -16
  27. package/plugins/continuous-improvement/hooks/session.mjs +85 -0
  28. package/plugins/continuous-improvement/lib/git-state.mjs +411 -0
  29. package/plugins/continuous-improvement/lib/plugin-metadata.mjs +18 -20
  30. package/plugins/continuous-improvement/skills/README.md +1 -0
  31. package/plugins/continuous-improvement/skills/proceed-with-the-recommendation/SKILL.md +1 -0
  32. package/plugins/continuous-improvement/skills/reconcile/SKILL.md +62 -12
  33. package/plugins/continuous-improvement/skills/simplicity-review/SKILL.md +80 -0
  34. package/plugins/expert.json +1 -1
  35. package/skills/proceed-with-the-recommendation.md +1 -0
  36. package/skills/reconcile.md +62 -12
  37. package/skills/simplicity-review.md +80 -0
@@ -0,0 +1,411 @@
1
+ /**
2
+ * git-state: pure classifiers for the `reconcile` ground-truth contract.
3
+ *
4
+ * No I/O. `src/bin/reconcile.mts` does the actual `git` spawning and feeds raw
5
+ * stdout plus exit codes in here; keeping the logic pure lets the unit tests
6
+ * cover every boundary (no configured upstream, detached HEAD, linked worktree,
7
+ * failed remote probe, autocrlf phantom drift) without building a repo per case.
8
+ *
9
+ * This module is also the single source of truth for the ground-truth command
10
+ * set: `GROUND_TRUTH_PROBES` is what `bin/reconcile.mjs` runs and what
11
+ * `bin/check-reconcile-parity.mjs` asserts the reconcile skill and command docs
12
+ * still document. Prose that drifts from this array fails `verify:all`.
13
+ *
14
+ * Every classifier fails closed. "We could not tell" is reported as unknown or
15
+ * unverified, never as clean, even, absent, or landed.
16
+ */
17
+ /** Branch names treated as protected unless the caller overrides the list. */
18
+ export const DEFAULT_PROTECTED_BRANCHES = [
19
+ "main",
20
+ "master",
21
+ "release",
22
+ "production",
23
+ ];
24
+ /**
25
+ * The ground-truth command set.
26
+ *
27
+ * Every entry is portable: plain `git` argv with no shell, no `ls`, no
28
+ * coreutils, and no literal `.git/` path — inside a linked worktree `.git` is a
29
+ * *file*, so a `.git/`-relative probe silently reports nothing. Ordering is the
30
+ * order the runner executes and the renderer prints.
31
+ */
32
+ export const GROUND_TRUTH_PROBES = [
33
+ {
34
+ id: "repo-root",
35
+ args: ["rev-parse", "--show-toplevel"],
36
+ purpose: "Confirm we are inside a work tree and learn its root.",
37
+ tolerateFailure: false,
38
+ },
39
+ {
40
+ id: "head-sha",
41
+ args: ["rev-parse", "HEAD"],
42
+ purpose: "Pin the exact commit every later claim is relative to.",
43
+ tolerateFailure: false,
44
+ },
45
+ {
46
+ id: "head-branch",
47
+ args: ["symbolic-ref", "--quiet", "--short", "HEAD"],
48
+ purpose: "Branch name, or a non-zero exit that proves a detached HEAD.",
49
+ tolerateFailure: true,
50
+ },
51
+ {
52
+ id: "upstream-ref",
53
+ args: ["rev-parse", "--abbrev-ref", "--symbolic-full-name", "@{u}"],
54
+ purpose: "Configured upstream, or a non-zero exit that proves there is none.",
55
+ tolerateFailure: true,
56
+ },
57
+ {
58
+ id: "upstream-counts",
59
+ args: ["rev-list", "--left-right", "--count", "@{u}...HEAD"],
60
+ purpose: "Behind/ahead counts; only meaningful once an upstream exists.",
61
+ tolerateFailure: true,
62
+ },
63
+ {
64
+ id: "status-porcelain",
65
+ args: ["status", "--porcelain=v1"],
66
+ purpose: "Reported working-tree changes — inflated by autocrlf on Windows.",
67
+ tolerateFailure: false,
68
+ },
69
+ {
70
+ id: "content-drift",
71
+ args: ["diff", "--name-only", "--ignore-all-space"],
72
+ purpose: "Real content drift — the number to trust when autocrlf is on.",
73
+ tolerateFailure: false,
74
+ },
75
+ {
76
+ id: "stash-list",
77
+ args: ["stash", "list"],
78
+ purpose: "Stashes an earlier session may have left holding real work.",
79
+ tolerateFailure: false,
80
+ },
81
+ {
82
+ id: "worktree-list",
83
+ args: ["worktree", "list", "--porcelain"],
84
+ purpose: "Sibling worktrees another session may be writing to.",
85
+ tolerateFailure: false,
86
+ },
87
+ ];
88
+ /**
89
+ * In-progress operation markers, addressed via `git rev-parse --git-path` so
90
+ * they resolve correctly inside a linked worktree, where `.git` is a file and
91
+ * the real marker lives under `.git/worktrees/<name>/`.
92
+ */
93
+ export const IN_PROGRESS_MARKERS = [
94
+ { id: "merge", gitPath: "MERGE_HEAD", label: "merge in progress" },
95
+ { id: "rebase-merge", gitPath: "rebase-merge", label: "interactive rebase in progress" },
96
+ { id: "rebase-apply", gitPath: "rebase-apply", label: "rebase/am in progress" },
97
+ { id: "cherry-pick", gitPath: "CHERRY_PICK_HEAD", label: "cherry-pick in progress" },
98
+ { id: "revert", gitPath: "REVERT_HEAD", label: "revert in progress" },
99
+ { id: "bisect", gitPath: "BISECT_LOG", label: "bisect in progress" },
100
+ ];
101
+ const SHA = /^[0-9a-f]{7,64}$/i;
102
+ const SAFE_REF = /^[A-Za-z0-9._/-]+$/;
103
+ /** Split git output on either line ending — git on Windows emits CRLF. */
104
+ function toLines(raw) {
105
+ return raw.split(/\r?\n/);
106
+ }
107
+ function isSha(value) {
108
+ return SHA.test(value);
109
+ }
110
+ /**
111
+ * True when `name` is safe to interpolate into a path, an argv entry, or a
112
+ * comparison that authorizes a mutation.
113
+ *
114
+ * Fails closed: null, empty, surrounding whitespace, shell metacharacters,
115
+ * `..`, refspec syntax (`@{`), a leading `-`, a trailing `/` or `.lock`, and
116
+ * anything over 255 characters all return false rather than being sanitized
117
+ * into something that then looks valid.
118
+ */
119
+ export function isSafeRefName(name) {
120
+ if (typeof name !== "string")
121
+ return false;
122
+ if (name.length === 0 || name.length > 255)
123
+ return false;
124
+ if (name.trim() !== name)
125
+ return false;
126
+ if (name.startsWith("-"))
127
+ return false;
128
+ if (name.endsWith("/") || name.endsWith(".lock"))
129
+ return false;
130
+ if (name.includes("..") || name.includes("@{"))
131
+ return false;
132
+ return SAFE_REF.test(name);
133
+ }
134
+ /**
135
+ * Classify HEAD from `git symbolic-ref --quiet --short HEAD`.
136
+ *
137
+ * A detached HEAD makes that command exit non-zero with empty output. This
138
+ * module deliberately never uses `git branch --show-current`, which returns an
139
+ * empty string with exit 0 and so cannot be told apart from a successful read.
140
+ * Returns `{ kind: "unknown", branch: null }` when output is present but is not
141
+ * a usable ref name, so an unparseable branch never reads as a match against an
142
+ * expected one.
143
+ */
144
+ export function classifyHead(rawBranch, exitCode = 0) {
145
+ const value = typeof rawBranch === "string" ? rawBranch.trim() : "";
146
+ if (exitCode !== 0 || value.length === 0) {
147
+ return { kind: "detached", branch: null };
148
+ }
149
+ if (!isSafeRefName(value))
150
+ return { kind: "unknown", branch: null };
151
+ return { kind: "branch", branch: value };
152
+ }
153
+ /**
154
+ * Parse `git rev-list --left-right --count @{u}...HEAD` output, which is
155
+ * `"<behind>\t<ahead>"`.
156
+ *
157
+ * Returns null for empty, malformed, negative, or non-integer output. Callers
158
+ * must treat null as "unknown" — never as zero.
159
+ */
160
+ export function parseRevListCounts(raw) {
161
+ if (typeof raw !== "string")
162
+ return null;
163
+ const first = toLines(raw).find((line) => line.trim().length > 0);
164
+ if (first === undefined)
165
+ return null;
166
+ const parts = first.trim().split(/\s+/);
167
+ if (parts.length !== 2)
168
+ return null;
169
+ const behind = Number(parts[0]);
170
+ const ahead = Number(parts[1]);
171
+ if (!Number.isInteger(behind) || behind < 0)
172
+ return null;
173
+ if (!Number.isInteger(ahead) || ahead < 0)
174
+ return null;
175
+ return { behind, ahead };
176
+ }
177
+ /**
178
+ * Map counts to an upstream relation.
179
+ *
180
+ * Fails closed: no upstream returns `"no-upstream"`, and an upstream that does
181
+ * exist but whose counts would not parse returns `"unknown"` — never `"even"`.
182
+ */
183
+ export function classifyUpstream(counts, hasUpstream) {
184
+ if (!hasUpstream)
185
+ return "no-upstream";
186
+ if (counts === null)
187
+ return "unknown";
188
+ const { behind, ahead } = counts;
189
+ if (behind === 0 && ahead === 0)
190
+ return "even";
191
+ if (behind > 0 && ahead > 0)
192
+ return "diverged";
193
+ return ahead > 0 ? "ahead" : "behind";
194
+ }
195
+ /**
196
+ * Collect a label for every in-progress operation whose marker is present.
197
+ *
198
+ * `present` is keyed by `InProgressMarker.id`. A marker missing from the map is
199
+ * treated as *unprobed*, not absent, and reported as `"<id>: unprobed"` — a
200
+ * probe that never ran must not masquerade as a clean tree. Returns `[]` only
201
+ * when every known marker was probed and none was present.
202
+ */
203
+ export function classifyInProgress(present) {
204
+ const found = [];
205
+ for (const marker of IN_PROGRESS_MARKERS) {
206
+ const state = present[marker.id];
207
+ if (state === undefined) {
208
+ found.push(`${marker.id}: unprobed`);
209
+ }
210
+ else if (state) {
211
+ found.push(marker.label);
212
+ }
213
+ }
214
+ return found;
215
+ }
216
+ /**
217
+ * True when `branch` must not be pushed to or committed onto directly.
218
+ *
219
+ * Fails closed: null, an unparseable name, and a detached HEAD all return true,
220
+ * because an unknown branch is not proof of safety. Supports one trailing `/*`
221
+ * wildcard per pattern (`release/*`).
222
+ */
223
+ export function isProtectedBranch(branch, patterns = DEFAULT_PROTECTED_BRANCHES) {
224
+ if (!isSafeRefName(branch))
225
+ return true;
226
+ const name = branch;
227
+ return patterns.some((pattern) => pattern.endsWith("/*")
228
+ ? name.startsWith(pattern.slice(0, -1))
229
+ : name === pattern);
230
+ }
231
+ /**
232
+ * Decide whether a push actually landed, from `git ls-remote` output.
233
+ *
234
+ * Three outcomes, never collapsed into two: a non-zero exit is `"unverified"`
235
+ * (the network or the remote failed, so absence of the ref is unproven and the
236
+ * push may well have landed), an empty successful probe is `"not-landed"` (the
237
+ * ref is genuinely absent), and a present ref is compared by SHA. An
238
+ * unparseable local HEAD or unparseable remote output is `"unverified"`.
239
+ */
240
+ export function verifyPushLanded(input) {
241
+ const local = (input.localHead ?? "").trim();
242
+ if (!isSha(local)) {
243
+ return { verdict: "unverified", reason: "local HEAD is not a usable sha" };
244
+ }
245
+ if (input.lsRemoteExitCode !== 0) {
246
+ return {
247
+ verdict: "unverified",
248
+ reason: `git ls-remote exited ${input.lsRemoteExitCode}; absence of the ref is unproven`,
249
+ };
250
+ }
251
+ const raw = (input.lsRemoteStdout ?? "").trim();
252
+ if (raw.length === 0) {
253
+ return { verdict: "not-landed", reason: "remote ref does not exist" };
254
+ }
255
+ const remoteSha = ((toLines(raw)[0] ?? "").trim().split(/\s+/)[0] ?? "");
256
+ if (!isSha(remoteSha)) {
257
+ return { verdict: "unverified", reason: "could not parse a sha from ls-remote output" };
258
+ }
259
+ if (remoteSha.toLowerCase() === local.toLowerCase()) {
260
+ return {
261
+ verdict: "landed",
262
+ reason: `remote tip matches local HEAD ${local.slice(0, 12)}`,
263
+ };
264
+ }
265
+ return {
266
+ verdict: "not-landed",
267
+ reason: `remote tip ${remoteSha.slice(0, 12)} differs from local HEAD ${local.slice(0, 12)}`,
268
+ };
269
+ }
270
+ /**
271
+ * Reconcile `git status --porcelain` output against real content drift.
272
+ *
273
+ * On an `autocrlf=true` tree status reports line-ending-only modifications that
274
+ * carry no content change, which is why the reconcile skill says to trust
275
+ * `git diff` instead. A `contentDrift` of null means the drift probe never ran;
276
+ * it is reported as null rather than assumed to be zero.
277
+ */
278
+ export function accountDirty(statusStdout, diffNamesStdout) {
279
+ const count = (raw) => typeof raw === "string"
280
+ ? toLines(raw).filter((line) => line.trim().length > 0).length
281
+ : null;
282
+ const reported = count(statusStdout) ?? 0;
283
+ const contentDrift = count(diffNamesStdout);
284
+ return {
285
+ reported,
286
+ contentDrift,
287
+ phantomSuspected: contentDrift !== null && reported > contentDrift,
288
+ };
289
+ }
290
+ /**
291
+ * True when the repo moved under us since `before` was captured — the check to
292
+ * run immediately before each mutation on a host with concurrent writers.
293
+ *
294
+ * Fails closed: a missing or unparseable sha on either side counts as shifted,
295
+ * because "we could not tell" must never read as "nothing moved".
296
+ */
297
+ export function baselineShifted(before, after) {
298
+ if (before === null || after === null)
299
+ return true;
300
+ const beforeHead = (before.head ?? "").trim();
301
+ const afterHead = (after.head ?? "").trim();
302
+ if (!isSha(beforeHead) || !isSha(afterHead))
303
+ return true;
304
+ if (beforeHead.toLowerCase() !== afterHead.toLowerCase())
305
+ return true;
306
+ return (before.branch ?? null) !== (after.branch ?? null);
307
+ }
308
+ /**
309
+ * Turn a probed state into the ordered finding list the resolved-state block
310
+ * renders. Every finding carries a severity; `blocker` means do not mutate.
311
+ */
312
+ export function assessGitState(input) {
313
+ const findings = [];
314
+ const protectedList = input.protectedBranches ?? DEFAULT_PROTECTED_BRANCHES;
315
+ if (input.head.kind === "branch") {
316
+ const onProtected = isProtectedBranch(input.head.branch, protectedList);
317
+ findings.push({
318
+ id: "head",
319
+ label: "HEAD",
320
+ detail: onProtected
321
+ ? `on protected branch ${input.head.branch} — branch before mutating`
322
+ : `on ${input.head.branch}`,
323
+ severity: onProtected ? "warn" : "ok",
324
+ });
325
+ }
326
+ else if (input.head.kind === "detached") {
327
+ findings.push({
328
+ id: "head",
329
+ label: "HEAD",
330
+ detail: "detached — no branch to commit onto, push, or name in a PR",
331
+ severity: "blocker",
332
+ });
333
+ }
334
+ else {
335
+ findings.push({
336
+ id: "head",
337
+ label: "HEAD",
338
+ detail: "branch name unparseable — treat repo identity as unknown",
339
+ severity: "blocker",
340
+ });
341
+ }
342
+ const relation = classifyUpstream(input.counts, input.upstreamRef !== null);
343
+ const upstreamRows = {
344
+ even: { detail: `even with ${input.upstreamRef}`, severity: "ok" },
345
+ ahead: {
346
+ detail: `${input.counts?.ahead ?? "?"} ahead of ${input.upstreamRef}`,
347
+ severity: "ok",
348
+ },
349
+ behind: {
350
+ detail: `${input.counts?.behind ?? "?"} behind ${input.upstreamRef} — pull --ff-only first`,
351
+ severity: "warn",
352
+ },
353
+ diverged: {
354
+ detail: `diverged from ${input.upstreamRef} ` +
355
+ `(${input.counts?.behind ?? "?"} behind / ${input.counts?.ahead ?? "?"} ahead) — ` +
356
+ "rebase or merge deliberately, never blind --force",
357
+ severity: "warn",
358
+ },
359
+ "no-upstream": {
360
+ detail: "no configured upstream — compare against origin/<default> explicitly before classifying",
361
+ severity: "warn",
362
+ },
363
+ unknown: {
364
+ detail: "upstream exists but counts did not parse — relationship unknown, do not assume even",
365
+ severity: "blocker",
366
+ },
367
+ };
368
+ findings.push({ id: "upstream", label: "upstream", ...upstreamRows[relation] });
369
+ const ops = classifyInProgress(Object.fromEntries(IN_PROGRESS_MARKERS.map((marker) => [marker.id, input.inProgress.includes(marker.id)])));
370
+ findings.push({
371
+ id: "in-progress",
372
+ label: "in-progress op",
373
+ detail: ops.length === 0 ? "none" : `${ops.join(", ")} — another actor may own this tree`,
374
+ severity: ops.length === 0 ? "ok" : "blocker",
375
+ });
376
+ const { reported, contentDrift, phantomSuspected } = input.dirty;
377
+ findings.push({
378
+ id: "dirty",
379
+ label: "working tree",
380
+ detail: contentDrift === null
381
+ ? `${reported} path(s) reported by status; content drift unprobed`
382
+ : phantomSuspected
383
+ ? `${reported} reported by status but only ${contentDrift} with content drift — ` +
384
+ "stage by explicit filename, never git add -A"
385
+ : `${contentDrift} path(s) with content drift`,
386
+ severity: contentDrift === null ? "warn" : "ok",
387
+ });
388
+ return findings;
389
+ }
390
+ /** True when any finding blocks mutation. */
391
+ export function hasBlockers(findings) {
392
+ return findings.some((finding) => finding.severity === "blocker");
393
+ }
394
+ const SEVERITY_TAG = {
395
+ ok: "ok",
396
+ warn: "WARN",
397
+ blocker: "BLOCKER",
398
+ };
399
+ /**
400
+ * Render the resolved-state block an operator reads before mutating. Returns a
401
+ * single "state unknown" line when no probe produced a finding, so an empty
402
+ * result never renders as a clean tree.
403
+ */
404
+ export function renderResolvedState(findings) {
405
+ if (findings.length === 0) {
406
+ return "reconcile (resolved): no probes ran — state unknown";
407
+ }
408
+ const width = Math.max(...findings.map((finding) => finding.label.length));
409
+ const rows = findings.map((finding) => ` ${finding.label.padEnd(width)} ${SEVERITY_TAG[finding.severity].padEnd(7)} ${finding.detail}`);
410
+ return ["reconcile (resolved):", ...rows].join("\n");
411
+ }
@@ -26,7 +26,7 @@ const KEYWORDS = [
26
26
  "transcript-linter",
27
27
  ];
28
28
  const CLAUDE_PLUGIN_CATEGORY = "productivity";
29
- const SHARED_PLUGIN_DESCRIPTION = "The persistent-memory and runtime-discipline layer for Claude Code. It remembers the corrections you already gave, grounds every edit in real facts before it lands, and — through the Mulahazah engine — turns each fix into a reusable instinct, so a lesson learned once is applied automatically next time with no re-teaching. Built on the 7 Laws of AI Agent Discipline (research, plan, verify, reflect, learn) and shipped as 27 bundled skills, instinct-aware hooks, an MCP toolset for recall and reflection, and a GitHub Action transcript linter that feeds real work history back into sharper instincts.";
29
+ const SHARED_PLUGIN_DESCRIPTION = "The persistent-memory and runtime-discipline layer for Claude Code. It remembers the corrections you already gave, grounds every edit in real facts before it lands, and — through the Mulahazah engine — turns each fix into a reusable instinct, so a lesson learned once is applied automatically next time with no re-teaching. Built on the 7 Laws of AI Agent Discipline (research, plan, verify, reflect, learn) and shipped as 28 bundled skills, instinct-aware hooks, an MCP toolset for recall and reflection, and a GitHub Action transcript linter that feeds real work history back into sharper instincts.";
30
30
  // Four vendored upstream companions registered alongside the CI plugin.
31
31
  // Each entry points at a pinned-SHA snapshot under third-party/<name>/.
32
32
  // See third-party/MANIFEST.md for refresh recipes and per-snapshot
@@ -457,76 +457,74 @@ export function getClaudePluginManifest() {
457
457
  };
458
458
  }
459
459
  export function getPluginHooksConfig() {
460
+ // Cold Node startup on loaded Windows hosts has exceeded five seconds.
461
+ const hookTimeoutSeconds = 30;
460
462
  const gateguardCommand = {
461
463
  type: "command",
462
464
  command: "node \"${CLAUDE_PLUGIN_ROOT}/hooks/gateguard.mjs\"",
463
- timeout: 5,
465
+ timeout: hookTimeoutSeconds,
464
466
  };
465
467
  const companionPreferenceCommand = {
466
468
  type: "command",
467
469
  command: "node \"${CLAUDE_PLUGIN_ROOT}/hooks/companion-preference.mjs\"",
468
- timeout: 5,
470
+ timeout: hookTimeoutSeconds,
469
471
  };
470
472
  const hookPackCommand = {
471
473
  type: "command",
472
474
  command: "node \"${CLAUDE_PLUGIN_ROOT}/hooks/hook-pack.mjs\"",
473
- timeout: 5,
475
+ timeout: hookTimeoutSeconds,
474
476
  };
475
477
  const observeCommand = {
476
478
  type: "command",
477
- command: "bash \"${CLAUDE_PLUGIN_ROOT}/hooks/observe.sh\"",
478
- timeout: 5,
479
+ command: "node \"${CLAUDE_PLUGIN_ROOT}/bin/observe.mjs\"",
480
+ timeout: hookTimeoutSeconds,
479
481
  };
480
482
  const sessionCommand = {
481
483
  type: "command",
482
- command: "bash \"${CLAUDE_PLUGIN_ROOT}/hooks/session.sh\"",
483
- timeout: 5,
484
+ command: "node \"${CLAUDE_PLUGIN_ROOT}/hooks/session.mjs\"",
485
+ timeout: hookTimeoutSeconds,
484
486
  };
485
487
  const threeSectionCloseCommand = {
486
488
  type: "command",
487
489
  command: "node \"${CLAUDE_PLUGIN_ROOT}/hooks/three-section-close.mjs\"",
488
- timeout: 5,
490
+ timeout: hookTimeoutSeconds,
489
491
  };
490
492
  const goalDriftStopCommand = {
491
493
  type: "command",
492
494
  command: "node \"${CLAUDE_PLUGIN_ROOT}/hooks/goal-drift-stop.mjs\"",
493
- timeout: 5,
495
+ timeout: hookTimeoutSeconds,
494
496
  };
495
497
  const workflowDistillCommand = {
496
498
  type: "command",
497
499
  command: "node \"${CLAUDE_PLUGIN_ROOT}/hooks/workflow-distill.mjs\"",
498
- timeout: 5,
500
+ timeout: hookTimeoutSeconds,
499
501
  };
500
502
  const typecheckStopCommand = {
501
503
  type: "command",
502
504
  command: "node \"${CLAUDE_PLUGIN_ROOT}/hooks/typecheck-stop.mjs\"",
503
- // Longer than the 5s hooks: tsc is slower. Opt-in via CLAUDE_TYPECHECK_GATE
504
- // (off by default) and near-zero cost when off / no TS file changed; on an
505
- // internal timeout it fails open (allow) rather than blocking.
506
- timeout: 30,
505
+ timeout: hookTimeoutSeconds,
507
506
  };
508
507
  const queryCostNudgeCommand = {
509
508
  type: "command",
510
509
  command: "node \"${CLAUDE_PLUGIN_ROOT}/hooks/query-cost-nudge.mjs\"",
511
- timeout: 5,
510
+ timeout: hookTimeoutSeconds,
512
511
  };
513
512
  const routePromptCommand = {
514
513
  type: "command",
515
514
  command: "node \"${CLAUDE_PLUGIN_ROOT}/hooks/route-prompt.mjs\"",
516
- timeout: 5,
515
+ timeout: hookTimeoutSeconds,
517
516
  };
518
517
  const recallBriefingCommand = {
519
518
  type: "command",
520
519
  command: "node \"${CLAUDE_PLUGIN_ROOT}/hooks/recall-briefing.mjs\"",
521
- timeout: 5,
520
+ timeout: hookTimeoutSeconds,
522
521
  };
523
522
  return {
524
- description: "Gateguard fact-forcing PreToolUse, companion-preference enforcement, observation, session lifecycle, 3-section-close discipline, goal-drift Stop gate, opt-in workflow-distill Stop nudge, opt-in typecheck Stop gate, opt-in query-cost Stop nudge, and UserPromptSubmit lazy-routing plus opt-in proactive recall-briefing hooks for continuous-improvement.",
525
523
  hooks: {
526
524
  // gateguard runs FIRST on PreToolUse so its block decision short-circuits
527
525
  // before companion-preference sees the call. companion-preference runs
528
526
  // second on Skill tool calls; it is a no-op under ci-first (the default)
529
- // and never blocks under companions-first. observe.sh only runs on
527
+ // and never blocks under companions-first. The observer only runs on
530
528
  // PostToolUse: gateguard-blocked calls are intentionally not observed so
531
529
  // PreToolUse stays at two subprocesses on the hot path. route-prompt
532
530
  // fires on UserPromptSubmit and emits a system-reminder when a prompt
package/llms.txt CHANGED
@@ -1,6 +1,6 @@
1
1
  # continuous-improvement
2
2
 
3
- > The persistent-memory and runtime-discipline layer for Claude Code. It remembers the corrections you already gave, grounds every edit in real facts before it lands, and — through the Mulahazah engine — turns each fix into a reusable instinct, so a lesson learned once is applied automatically next time with no re-teaching. Built on the 7 Laws of AI Agent Discipline (research, plan, verify, reflect, learn) and shipped as 27 bundled skills, instinct-aware hooks, an MCP toolset for recall and reflection, and a GitHub Action transcript linter that feeds real work history back into sharper instincts.
3
+ > The persistent-memory and runtime-discipline layer for Claude Code. It remembers the corrections you already gave, grounds every edit in real facts before it lands, and — through the Mulahazah engine — turns each fix into a reusable instinct, so a lesson learned once is applied automatically next time with no re-teaching. Built on the 7 Laws of AI Agent Discipline (research, plan, verify, reflect, learn) and shipped as 28 bundled skills, instinct-aware hooks, an MCP toolset for recall and reflection, and a GitHub Action transcript linter that feeds real work history back into sharper instincts.
4
4
 
5
5
  ## What This Is
6
6
 
package/package.json CHANGED
@@ -1,7 +1,7 @@
1
1
  {
2
2
  "name": "continuous-improvement",
3
- "version": "3.20.4",
4
- "description": "Claude Code that gets sharper every session: the persistent-memory and runtime-discipline layer built on the 7 Laws of AI Agent Discipline. It grounds every edit in real facts before it lands and, through the Mulahazah engine, turns each fix into a reusable instinct, so a lesson learned once is applied automatically next time with no re-teaching. Shipped as 27 bundled skills, instinct-aware hooks, an MCP toolset for recall and reflection, and a GitHub Action transcript linter that feeds real work history back into sharper instincts. Beginner: one /plugin install command. Expert: adds MCP tools and session hooks.",
3
+ "version": "3.22.0",
4
+ "description": "Claude Code that gets sharper every session: the persistent-memory and runtime-discipline layer built on the 7 Laws of AI Agent Discipline. It grounds every edit in real facts before it lands and, through the Mulahazah engine, turns each fix into a reusable instinct, so a lesson learned once is applied automatically next time with no re-teaching. Shipped as 28 bundled skills, instinct-aware hooks, an MCP toolset for recall and reflection, and a GitHub Action transcript linter that feeds real work history back into sharper instincts. Beginner: one /plugin install command. Expert: adds MCP tools and session hooks.",
5
5
  "keywords": [
6
6
  "claude-code",
7
7
  "claude-code-plugin",
@@ -35,7 +35,8 @@
35
35
  "ci": "bin/unified-cli.mjs",
36
36
  "ci-plan-pack": "bin/plan-pack.mjs",
37
37
  "ci-audit-actions": "bin/audit-actions.mjs",
38
- "ci-portfolio-health": "bin/portfolio-health.mjs"
38
+ "ci-portfolio-health": "bin/portfolio-health.mjs",
39
+ "ci-reconcile": "bin/reconcile.mjs"
39
40
  },
40
41
  "scripts": {
41
42
  "build": "tsc -p tsconfig.json && node bin/generate-plugin-manifests.mjs && node -e \"const fs=require('node:fs'); for (const f of fs.readdirSync('bin')) { if (f.endsWith('.mjs')) fs.chmodSync('bin/'+f, 0o755); } for (const f of fs.readdirSync('hooks')) { if (f.endsWith('.mjs')) fs.chmodSync('hooks/'+f, 0o755); } for (const f of fs.readdirSync('lib')) { if (f.endsWith('.mjs')) fs.chmodSync('lib/'+f, 0o755); } for (const f of fs.readdirSync('plugins/continuous-improvement/bin')) { if (f.endsWith('.mjs')) fs.chmodSync('plugins/continuous-improvement/bin/'+f, 0o755); } for (const f of fs.readdirSync('plugins/continuous-improvement/lib')) { if (f.endsWith('.mjs')) fs.chmodSync('plugins/continuous-improvement/lib/'+f, 0o755); } for (const f of fs.readdirSync('plugins/continuous-improvement/hooks')) { if (f.endsWith('.mjs')) fs.chmodSync('plugins/continuous-improvement/hooks/'+f, 0o755); } for (const f of fs.readdirSync('scripts')) { if (f.endsWith('.mjs')) fs.chmodSync('scripts/'+f, 0o755); } for (const f of fs.readdirSync('synthetic-checks')) { if (f.endsWith('.mjs')) fs.chmodSync('synthetic-checks/'+f, 0o755); } \"",
@@ -60,7 +61,8 @@
60
61
  "verify:scripts-citation-drift": "node bin/check-scripts-citation-drift.mjs && node bin/check-scripts-citation-drift.mjs plugins/continuous-improvement",
61
62
  "verify:third-party-shape": "node bin/check-third-party-shape.mjs",
62
63
  "verify:tool-count": "node bin/check-tool-count.mjs",
63
- "verify:all": "npm run verify:skill-mirror && npm run verify:skill-tiers && npm run verify:skill-law-tag && npm run verify:skill-count && npm run verify:skill-count-prose && npm run verify:command-count && npm run verify:docs-substrings && npm run verify:everything-mirror && npm run verify:routing-targets && npm run verify:doc-runtime-claims && npm run verify:test-imports-only && npm run verify:landing-version && npm run verify:scripts-citation-drift && npm run verify:third-party-shape && npm run verify:tool-count && npm run typecheck"
64
+ "verify:reconcile-parity": "node bin/check-reconcile-parity.mjs",
65
+ "verify:all": "npm run verify:skill-mirror && npm run verify:skill-tiers && npm run verify:skill-law-tag && npm run verify:skill-count && npm run verify:skill-count-prose && npm run verify:command-count && npm run verify:docs-substrings && npm run verify:everything-mirror && npm run verify:routing-targets && npm run verify:doc-runtime-claims && npm run verify:test-imports-only && npm run verify:landing-version && npm run verify:scripts-citation-drift && npm run verify:third-party-shape && npm run verify:tool-count && npm run verify:reconcile-parity && npm run typecheck"
64
66
  },
65
67
  "files": [
66
68
  ".claude-plugin/",
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "continuous-improvement",
3
- "version": "3.20.4",
3
+ "version": "3.22.0",
4
4
  "mode": "beginner",
5
5
  "description": "Beginner mode: see what your agent learned, list its instincts, and request a session reflection. Bundles three grounding skills (gateguard, tdd-workflow, verification-loop) so research, memory, tests, and verification happen by default — every edit starts from facts, not guesses.",
6
6
  "tools": [
@@ -7,8 +7,8 @@
7
7
  "plugins": [
8
8
  {
9
9
  "name": "continuous-improvement",
10
- "description": "The persistent-memory and runtime-discipline layer for Claude Code. It remembers the corrections you already gave, grounds every edit in real facts before it lands, and — through the Mulahazah engine — turns each fix into a reusable instinct, so a lesson learned once is applied automatically next time with no re-teaching. Built on the 7 Laws of AI Agent Discipline (research, plan, verify, reflect, learn) and shipped as 27 bundled skills, instinct-aware hooks, an MCP toolset for recall and reflection, and a GitHub Action transcript linter that feeds real work history back into sharper instincts.",
11
- "version": "3.20.4",
10
+ "description": "The persistent-memory and runtime-discipline layer for Claude Code. It remembers the corrections you already gave, grounds every edit in real facts before it lands, and — through the Mulahazah engine — turns each fix into a reusable instinct, so a lesson learned once is applied automatically next time with no re-teaching. Built on the 7 Laws of AI Agent Discipline (research, plan, verify, reflect, learn) and shipped as 28 bundled skills, instinct-aware hooks, an MCP toolset for recall and reflection, and a GitHub Action transcript linter that feeds real work history back into sharper instincts.",
11
+ "version": "3.22.0",
12
12
  "source": "./",
13
13
  "author": {
14
14
  "name": "naimkatiman"
@@ -1,7 +1,7 @@
1
1
  {
2
2
  "name": "continuous-improvement",
3
- "version": "3.20.4",
4
- "description": "The persistent-memory and runtime-discipline layer for Claude Code. It remembers the corrections you already gave, grounds every edit in real facts before it lands, and — through the Mulahazah engine — turns each fix into a reusable instinct, so a lesson learned once is applied automatically next time with no re-teaching. Built on the 7 Laws of AI Agent Discipline (research, plan, verify, reflect, learn) and shipped as 27 bundled skills, instinct-aware hooks, an MCP toolset for recall and reflection, and a GitHub Action transcript linter that feeds real work history back into sharper instincts.",
3
+ "version": "3.22.0",
4
+ "description": "The persistent-memory and runtime-discipline layer for Claude Code. It remembers the corrections you already gave, grounds every edit in real facts before it lands, and — through the Mulahazah engine — turns each fix into a reusable instinct, so a lesson learned once is applied automatically next time with no re-teaching. Built on the 7 Laws of AI Agent Discipline (research, plan, verify, reflect, learn) and shipped as 28 bundled skills, instinct-aware hooks, an MCP toolset for recall and reflection, and a GitHub Action transcript linter that feeds real work history back into sharper instincts.",
5
5
  "author": {
6
6
  "name": "naimkatiman",
7
7
  "url": "https://github.com/naimkatiman"