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.
- package/.claude-plugin/marketplace.json +2 -2
- package/CHANGELOG.md +23 -0
- package/QUICKSTART.md +1 -1
- package/README.md +7 -6
- package/bin/check-reconcile-parity.mjs +168 -0
- package/bin/generate-plugin-manifests.mjs +2 -0
- package/bin/install.mjs +30 -69
- package/bin/reconcile.mjs +259 -0
- package/commands/production-readiness-review.md +5 -4
- package/commands/reconcile.md +30 -6
- package/commands/simplicity-review.md +35 -0
- package/commands/verify-install.md +2 -2
- package/hooks/session.mjs +85 -0
- package/lib/git-state.mjs +411 -0
- package/lib/plugin-metadata.mjs +18 -20
- package/llms.txt +1 -1
- package/package.json +6 -4
- package/plugins/beginner.json +1 -1
- package/plugins/continuous-improvement/.claude-plugin/marketplace.json +2 -2
- package/plugins/continuous-improvement/.claude-plugin/plugin.json +2 -2
- package/plugins/continuous-improvement/bin/reconcile.mjs +259 -0
- package/plugins/continuous-improvement/commands/production-readiness-review.md +5 -4
- package/plugins/continuous-improvement/commands/reconcile.md +30 -6
- package/plugins/continuous-improvement/commands/simplicity-review.md +35 -0
- package/plugins/continuous-improvement/commands/verify-install.md +2 -2
- package/plugins/continuous-improvement/hooks/hooks.json +15 -16
- package/plugins/continuous-improvement/hooks/session.mjs +85 -0
- package/plugins/continuous-improvement/lib/git-state.mjs +411 -0
- package/plugins/continuous-improvement/lib/plugin-metadata.mjs +18 -20
- package/plugins/continuous-improvement/skills/README.md +1 -0
- package/plugins/continuous-improvement/skills/proceed-with-the-recommendation/SKILL.md +1 -0
- package/plugins/continuous-improvement/skills/reconcile/SKILL.md +62 -12
- package/plugins/continuous-improvement/skills/simplicity-review/SKILL.md +80 -0
- package/plugins/expert.json +1 -1
- package/skills/proceed-with-the-recommendation.md +1 -0
- package/skills/reconcile.md +62 -12
- 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
|
+
}
|
package/lib/plugin-metadata.mjs
CHANGED
|
@@ -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
|
|
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:
|
|
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:
|
|
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:
|
|
475
|
+
timeout: hookTimeoutSeconds,
|
|
474
476
|
};
|
|
475
477
|
const observeCommand = {
|
|
476
478
|
type: "command",
|
|
477
|
-
command: "
|
|
478
|
-
timeout:
|
|
479
|
+
command: "node \"${CLAUDE_PLUGIN_ROOT}/bin/observe.mjs\"",
|
|
480
|
+
timeout: hookTimeoutSeconds,
|
|
479
481
|
};
|
|
480
482
|
const sessionCommand = {
|
|
481
483
|
type: "command",
|
|
482
|
-
command: "
|
|
483
|
-
timeout:
|
|
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:
|
|
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:
|
|
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:
|
|
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
|
-
|
|
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:
|
|
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:
|
|
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:
|
|
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.
|
|
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
|
|
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.
|
|
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
|
|
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:
|
|
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/",
|
package/plugins/beginner.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "continuous-improvement",
|
|
3
|
-
"version": "3.
|
|
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
|
|
11
|
-
"version": "3.
|
|
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.
|
|
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
|
|
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"
|