@codyswann/lisa 2.342.4 → 2.342.6
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/all/copy-overwrite/scripts/lisa-hooks/block-no-verify.sh +35 -1
- package/all/copy-overwrite/scripts/lisa-work-item.mjs +34 -4
- package/dist/codex/scripts/block-no-verify.sh +24 -1
- package/dist/core/upstream-evidence-manifest.js +7 -7
- package/package.json +1 -1
- package/plugins/lisa/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa/hooks/block-no-verify.agy.sh +24 -1
- package/plugins/lisa/hooks/block-no-verify.sh +35 -1
- package/plugins/lisa/hooks/threshold-ratchet.mjs +24 -0
- package/plugins/lisa-agy/hooks/block-no-verify.agy.sh +24 -1
- package/plugins/lisa-agy/plugin.json +1 -1
- package/plugins/lisa-cdk/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-cdk/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-cdk-agy/plugin.json +1 -1
- package/plugins/lisa-cdk-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-cdk-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-copilot/hooks/block-no-verify.sh +35 -1
- package/plugins/lisa-copilot/hooks/threshold-ratchet.mjs +24 -0
- package/plugins/lisa-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-cursor/hooks/block-no-verify.sh +35 -1
- package/plugins/lisa-cursor/hooks/threshold-ratchet.mjs +24 -0
- package/plugins/lisa-expo/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-expo/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-expo-agy/plugin.json +1 -1
- package/plugins/lisa-expo-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-expo-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-harper-fabric/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-harper-fabric/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-harper-fabric-agy/plugin.json +1 -1
- package/plugins/lisa-harper-fabric-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-harper-fabric-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-nestjs/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-nestjs/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-nestjs-agy/plugin.json +1 -1
- package/plugins/lisa-nestjs-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-nestjs-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-openclaw/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-openclaw/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-openclaw-agy/plugin.json +1 -1
- package/plugins/lisa-openclaw-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-openclaw-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-phaser/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-phaser/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-phaser-agy/plugin.json +1 -1
- package/plugins/lisa-phaser-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-phaser-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-rails/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-rails/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-rails-agy/plugin.json +1 -1
- package/plugins/lisa-rails-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-rails-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-typescript/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-typescript/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-typescript-agy/plugin.json +1 -1
- package/plugins/lisa-typescript-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-typescript-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-wiki/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-wiki/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-wiki-agy/plugin.json +1 -1
- package/plugins/lisa-wiki-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-wiki-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/src/base/hooks/block-no-verify.agy.sh +24 -1
- package/plugins/src/base/hooks/block-no-verify.sh +35 -1
- package/plugins/src/base/hooks/threshold-ratchet.mjs +24 -0
- package/rails/copy-overwrite/scripts/check-threshold-ratchet.mjs +24 -0
- package/typescript/copy-overwrite/scripts/check-threshold-ratchet.mjs +24 -0
|
@@ -118,8 +118,42 @@ def is_permitted_hooks_path(value):
|
|
|
118
118
|
cleaned = cleaned[2:]
|
|
119
119
|
return cleaned.rstrip("/") in PERMITTED_HOOKS_PATHS
|
|
120
120
|
|
|
121
|
+
|
|
122
|
+
# Git resolves any UNAMBIGUOUS abbreviation of a long option, so `--no-verify`
|
|
123
|
+
# is only the longest of the spellings that disable verification: `git commit
|
|
124
|
+
# --no-veri` skips hooks exactly as completely. An equality check therefore
|
|
125
|
+
# enforced the guard against the one spelling nobody in a hurry types.
|
|
126
|
+
#
|
|
127
|
+
# Matched as "a prefix of --no-verify" rather than by listing abbreviations,
|
|
128
|
+
# because the set of accepted abbreviations is a property of git's parser and
|
|
129
|
+
# changes with the surrounding options. `--no-verbose` is NOT caught, and must
|
|
130
|
+
# not be: it diverges from `--no-verify` at the character after `--no-ver`, so
|
|
131
|
+
# it fails the prefix test.
|
|
132
|
+
#
|
|
133
|
+
# The floor is `--no-v`. Shorter is refused anyway — bare `--no-` is not a flag
|
|
134
|
+
# — and blocking an abbreviation git would reject as ambiguous costs nothing,
|
|
135
|
+
# while missing one git accepts costs the whole guard.
|
|
136
|
+
NO_VERIFY = "--no-verify"
|
|
137
|
+
NO_VERIFY_MIN_PREFIX = len("--no-v")
|
|
138
|
+
|
|
139
|
+
|
|
140
|
+
def disables_verification(token):
|
|
141
|
+
"""Whether a token is `--no-verify` or an abbreviation git would accept.
|
|
142
|
+
|
|
143
|
+
Args:
|
|
144
|
+
token: A single shell token from the command line.
|
|
145
|
+
|
|
146
|
+
Returns:
|
|
147
|
+
True if git would read this token as --no-verify.
|
|
148
|
+
"""
|
|
149
|
+
return (
|
|
150
|
+
len(token) >= NO_VERIFY_MIN_PREFIX
|
|
151
|
+
and len(token) <= len(NO_VERIFY)
|
|
152
|
+
and NO_VERIFY.startswith(token)
|
|
153
|
+
)
|
|
154
|
+
|
|
121
155
|
for i, token in enumerate(normalized_tokens):
|
|
122
|
-
if token
|
|
156
|
+
if disables_verification(token):
|
|
123
157
|
sys.exit(1)
|
|
124
158
|
if token == "HUSKY=0" or token.startswith("HUSKY_SKIP_HOOKS="):
|
|
125
159
|
sys.exit(1)
|
|
@@ -21,6 +21,36 @@ const RELEASE_SUBJECT =
|
|
|
21
21
|
/^chore\(release\): \d+\.\d+\.\d+(?:-[0-9A-Za-z.-]+)? \[skip ci\]$/;
|
|
22
22
|
const ZERO_OID = /^0+$/;
|
|
23
23
|
const MARKER = "[lisa-pr-link]";
|
|
24
|
+
/**
|
|
25
|
+
* The prefix only — an anchored, fixed-length literal with no quantifier, so
|
|
26
|
+
* it cannot backtrack. The ReDoS was never here; it was in the `\s*(.+?)\s*$`
|
|
27
|
+
* tail, which is now string work.
|
|
28
|
+
*
|
|
29
|
+
* Deliberately still a regex rather than `line.toLowerCase().startsWith(…)`.
|
|
30
|
+
* `toLowerCase()` applies FULL Unicode case mapping, so U+212A KELVIN SIGN
|
|
31
|
+
* folds to "k" and `WorK-Item:` written with it would be accepted — a format
|
|
32
|
+
* this has never accepted, because `/i` on a non-unicode pattern canonicalizes
|
|
33
|
+
* ASCII only.
|
|
34
|
+
*/
|
|
35
|
+
const WORK_ITEM_PREFIX = /^work-item:/i;
|
|
36
|
+
|
|
37
|
+
/**
|
|
38
|
+
* The value of a `Work-Item:` line, or null if this is not one.
|
|
39
|
+
*
|
|
40
|
+
* Replaces `/^Work-Item:\s*(.+?)\s*$/i`, which put a lazy quantifier between
|
|
41
|
+
* two `\s*` groups and backtracked super-linearly on a line that is mostly
|
|
42
|
+
* whitespace — over commit messages and PR bodies, which are not this
|
|
43
|
+
* script's to trust. Acceptance is unchanged: ASCII-case-insensitive prefix at
|
|
44
|
+
* the start of the line, non-empty value, surrounding whitespace ignored.
|
|
45
|
+
* @param {string} line One line of a commit message or PR body.
|
|
46
|
+
* @returns {string | null} The trimmed value, or null.
|
|
47
|
+
*/
|
|
48
|
+
function workItemLineValue(line) {
|
|
49
|
+
const match = WORK_ITEM_PREFIX.exec(line);
|
|
50
|
+
if (!match) return null;
|
|
51
|
+
const value = line.slice(match[0].length).trim();
|
|
52
|
+
return value === "" ? null : value;
|
|
53
|
+
}
|
|
24
54
|
const GUIDANCE = [
|
|
25
55
|
"Mention the ticket this work relates to, or ask Lisa to create one:",
|
|
26
56
|
" Work-Item: <configured-project-ticket>",
|
|
@@ -493,8 +523,8 @@ function parseTrailers(message) {
|
|
|
493
523
|
.split("\n")
|
|
494
524
|
.filter(Boolean)
|
|
495
525
|
.flatMap(line => {
|
|
496
|
-
const
|
|
497
|
-
return
|
|
526
|
+
const value = workItemLineValue(line);
|
|
527
|
+
return value === null ? [] : [value];
|
|
498
528
|
});
|
|
499
529
|
}
|
|
500
530
|
|
|
@@ -1179,8 +1209,8 @@ function prWorkItem(body, contract) {
|
|
|
1179
1209
|
const matches = String(body ?? "")
|
|
1180
1210
|
.split(/\r?\n/)
|
|
1181
1211
|
.flatMap(line => {
|
|
1182
|
-
const
|
|
1183
|
-
return
|
|
1212
|
+
const value = workItemLineValue(line);
|
|
1213
|
+
return value === null ? [] : [value];
|
|
1184
1214
|
});
|
|
1185
1215
|
if (matches.length !== 1)
|
|
1186
1216
|
throw new TrackingError(
|
|
@@ -62,8 +62,31 @@ except ValueError:
|
|
|
62
62
|
|
|
63
63
|
normalized_tokens = [token.strip("();|&") for token in tokens]
|
|
64
64
|
|
|
65
|
+
# Git resolves any UNAMBIGUOUS abbreviation of a long option, so `--no-veri`
|
|
66
|
+
# skips hooks exactly as completely as `--no-verify`. Matched as a prefix
|
|
67
|
+
# rather than by listing abbreviations; `--no-verbose` diverges after
|
|
68
|
+
# `--no-ver` and so is correctly not caught.
|
|
69
|
+
NO_VERIFY = "--no-verify"
|
|
70
|
+
NO_VERIFY_MIN_PREFIX = len("--no-v")
|
|
71
|
+
|
|
72
|
+
|
|
73
|
+
def disables_verification(token):
|
|
74
|
+
"""Whether a token is `--no-verify` or an abbreviation git would accept.
|
|
75
|
+
|
|
76
|
+
Args:
|
|
77
|
+
token: A single shell token from the command line.
|
|
78
|
+
|
|
79
|
+
Returns:
|
|
80
|
+
True if git would read this token as --no-verify.
|
|
81
|
+
"""
|
|
82
|
+
return (
|
|
83
|
+
len(token) >= NO_VERIFY_MIN_PREFIX
|
|
84
|
+
and len(token) <= len(NO_VERIFY)
|
|
85
|
+
and NO_VERIFY.startswith(token)
|
|
86
|
+
)
|
|
87
|
+
|
|
65
88
|
for i, token in enumerate(normalized_tokens):
|
|
66
|
-
if token
|
|
89
|
+
if disables_verification(token):
|
|
67
90
|
sys.exit(1)
|
|
68
91
|
if token == "HUSKY=0" or token.startswith("HUSKY_SKIP_HOOKS="):
|
|
69
92
|
sys.exit(1)
|
|
@@ -6,11 +6,11 @@ export const UPSTREAM_EVIDENCE_MANIFEST = Object.freeze({
|
|
|
6
6
|
"all/copy-overwrite/scripts/lisa-enforcement-fallback.sh": "1bec5f3c284b3febdc471487ee6a23ffb72bd4e7b293f9e26b0188f32318c722",
|
|
7
7
|
"all/copy-overwrite/scripts/lisa-floor-collisions.mjs": "3c7a0234b4f5609fce49e44711d2ec86af8cf7fb82b0831e4450e3347ecefe45",
|
|
8
8
|
"all/copy-overwrite/scripts/lisa-hooks/block-instruction-file-edits.sh": "850cfc23852eb752114ae1e938a1c7bcf34bc50c85d3aaca0c02311cc8c0ef70",
|
|
9
|
-
"all/copy-overwrite/scripts/lisa-hooks/block-no-verify.sh": "
|
|
9
|
+
"all/copy-overwrite/scripts/lisa-hooks/block-no-verify.sh": "031ef7a53fc49cb18665105b834b7b50ad6a43317e0821949809e877e0562ce4",
|
|
10
10
|
"all/copy-overwrite/scripts/lisa-hooks/block-shell-json-parsing.sh": "1934a15e154fda3c2a40979a5cd6225661b14fe7bc77328a69ce499bea85dba7",
|
|
11
11
|
"all/copy-overwrite/scripts/lisa-hooks/parity-safety-net.sh": "d9ea58150aa27e7f0aa9a4ef155ed446d30bdfc932d01c64e2b8ca211aa0de7b",
|
|
12
12
|
"all/copy-overwrite/scripts/lisa-hooks/sonar-secrets.sh": "bf4132e49ba18e2f7e941520c299c15aeeacfd220e489f3d2eb8397f2048157b",
|
|
13
|
-
"all/copy-overwrite/scripts/lisa-work-item.mjs": "
|
|
13
|
+
"all/copy-overwrite/scripts/lisa-work-item.mjs": "e30016a49deff1c3cd3ae7951262aeb2ef3d472bce76bbbe2a177db6017c7cad",
|
|
14
14
|
"all/create-only/.claude/rules/PROJECT_RULES.md": "6e1c7923ad2a15356babc60c97e7f97be728d790af285b6a7cd7d1b4d39cd97f",
|
|
15
15
|
"all/create-only/.lisaignore": "3c9827e7e98daa46510b5357cd6dd9406da1821e64cc46d2eea9f311cd6d06b9",
|
|
16
16
|
"all/create-only/scripts/remote-agent-aws-setup.sh": "12d2204d34195ad4f81082e90a25dbbf7d2493debcc50c1cd37b32dc4648fc17",
|
|
@@ -297,8 +297,8 @@ export const UPSTREAM_EVIDENCE_MANIFEST = Object.freeze({
|
|
|
297
297
|
"plugins/src/base/commands/wiki/install.md": "51392cf053c17edd549bfbaba5d19ce669dc79021aafa7cfa100324ad38de11f",
|
|
298
298
|
"plugins/src/base/hooks/block-instruction-file-edits.agy.sh": "aa249cae53caeb3e0fb6d6af114e2756084f45a2896332fb063a9f20e4902125",
|
|
299
299
|
"plugins/src/base/hooks/block-instruction-file-edits.sh": "850cfc23852eb752114ae1e938a1c7bcf34bc50c85d3aaca0c02311cc8c0ef70",
|
|
300
|
-
"plugins/src/base/hooks/block-no-verify.agy.sh": "
|
|
301
|
-
"plugins/src/base/hooks/block-no-verify.sh": "
|
|
300
|
+
"plugins/src/base/hooks/block-no-verify.agy.sh": "c549f1dadd6ac3fab72c0b670524445a08bee732a05b868954a438970d4d5bcc",
|
|
301
|
+
"plugins/src/base/hooks/block-no-verify.sh": "031ef7a53fc49cb18665105b834b7b50ad6a43317e0821949809e877e0562ce4",
|
|
302
302
|
"plugins/src/base/hooks/block-shell-json-parsing.agy.sh": "dc688efe382e7b8fe6f6c88bb0fde851527128cae778599559f23a93f1c9ec86",
|
|
303
303
|
"plugins/src/base/hooks/block-shell-json-parsing.sh": "1934a15e154fda3c2a40979a5cd6225661b14fe7bc77328a69ce499bea85dba7",
|
|
304
304
|
"plugins/src/base/hooks/cleanup-stale-worktrees.sh": "c9b2fcca1b34d6f636b5289ece11b254cbb8ee2c2bac5c2e426b6f4dcfa972f2",
|
|
@@ -316,7 +316,7 @@ export const UPSTREAM_EVIDENCE_MANIFEST = Object.freeze({
|
|
|
316
316
|
"plugins/src/base/hooks/sonar-secrets.sh": "bf4132e49ba18e2f7e941520c299c15aeeacfd220e489f3d2eb8397f2048157b",
|
|
317
317
|
"plugins/src/base/hooks/threshold-ratchet-compare.mjs": "4535a145c5c5cbee13bdf970bc91d390a264043604e607288cdd6bf28de56192",
|
|
318
318
|
"plugins/src/base/hooks/threshold-ratchet-families.mjs": "db14245ed89b483912af287bdee917a7a921dc21bb29b49be37fc799c05c413f",
|
|
319
|
-
"plugins/src/base/hooks/threshold-ratchet.mjs": "
|
|
319
|
+
"plugins/src/base/hooks/threshold-ratchet.mjs": "e1e3f22b33267212b90f915f83821559d1a45cab5b16c7164b86bd6c7b53549b",
|
|
320
320
|
"plugins/src/base/hooks/threshold-ratchet.sh": "b86f4d0b554e44fd119ad8a8920cd0ba6c9dbe779f7ee219d381cf0629453990",
|
|
321
321
|
"plugins/src/base/hooks/ticket-sync-reminder.sh": "212f80bcdfd9e3ec2254987c5a7186d1d2d38065b330ddcdc631f76df7c4a6c0",
|
|
322
322
|
"plugins/src/base/hooks/track-plan-sessions.sh": "8bc47dcad3aed628df0c9efbe7b6f3df87635b94b0c54c71e0bd52eceace563a",
|
|
@@ -947,7 +947,7 @@ export const UPSTREAM_EVIDENCE_MANIFEST = Object.freeze({
|
|
|
947
947
|
"rails/copy-overwrite/ast-grep/utils/.gitkeep": "e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855",
|
|
948
948
|
"rails/copy-overwrite/config/initializers/version.rb": "fa664a84f407877b5ddbd3709a2352b30e4dc6fbce538a8503bad6c52448dc86",
|
|
949
949
|
"rails/copy-overwrite/lefthook.yml": "28bb382b9171a04fb92ebafc1f1522b8639f5edc4fcffcc6940e7fba3460fb4e",
|
|
950
|
-
"rails/copy-overwrite/scripts/check-threshold-ratchet.mjs": "
|
|
950
|
+
"rails/copy-overwrite/scripts/check-threshold-ratchet.mjs": "e1e3f22b33267212b90f915f83821559d1a45cab5b16c7164b86bd6c7b53549b",
|
|
951
951
|
"rails/copy-overwrite/scripts/lisa-clean-git-env.sh": "e7121a0ee9e1bf7c01cd2ab55f563dd6d9ab75990739bb6ddf571a513efa10e9",
|
|
952
952
|
"rails/copy-overwrite/scripts/threshold-ratchet-compare.mjs": "4535a145c5c5cbee13bdf970bc91d390a264043604e607288cdd6bf28de56192",
|
|
953
953
|
"rails/copy-overwrite/scripts/threshold-ratchet-families.mjs": "db14245ed89b483912af287bdee917a7a921dc21bb29b49be37fc799c05c413f",
|
|
@@ -1070,7 +1070,7 @@ export const UPSTREAM_EVIDENCE_MANIFEST = Object.freeze({
|
|
|
1070
1070
|
"typescript/copy-overwrite/eslint.ignore.config.json": "04dcd754829cf93bf0ba0d76e4031d28cb7149afa658575d02bb13bf73f7880e",
|
|
1071
1071
|
"typescript/copy-overwrite/eslint.slow.config.ts": "f66d04381940936544931d7f20ffd4e43c38b8a907d56ffa101e2236371d8c94",
|
|
1072
1072
|
"typescript/copy-overwrite/knip.json": "6eb93d705a2d645332fbae1dd42cf2d48f85978ebc265451a53790912f277d12",
|
|
1073
|
-
"typescript/copy-overwrite/scripts/check-threshold-ratchet.mjs": "
|
|
1073
|
+
"typescript/copy-overwrite/scripts/check-threshold-ratchet.mjs": "e1e3f22b33267212b90f915f83821559d1a45cab5b16c7164b86bd6c7b53549b",
|
|
1074
1074
|
"typescript/copy-overwrite/scripts/check-verification-coverage.mjs": "3c1d27d0668fc66a18cb014f2b607878f0a88c178a773fbe964b8e46b19e86d6",
|
|
1075
1075
|
"typescript/copy-overwrite/scripts/lisa-mutation.mjs": "7d8d24dea151ef2178807046e97aea7ebfadd9c88eb50347feb7bb4bb190227d",
|
|
1076
1076
|
"typescript/copy-overwrite/scripts/threshold-ratchet-compare.mjs": "4535a145c5c5cbee13bdf970bc91d390a264043604e607288cdd6bf28de56192",
|
package/package.json
CHANGED
|
@@ -120,7 +120,7 @@
|
|
|
120
120
|
}
|
|
121
121
|
},
|
|
122
122
|
"name": "@codyswann/lisa",
|
|
123
|
-
"version": "2.342.
|
|
123
|
+
"version": "2.342.6",
|
|
124
124
|
"description": "Claude Code governance framework that applies guardrails, guidance, and automated enforcement to projects",
|
|
125
125
|
"main": "dist/index.js",
|
|
126
126
|
"exports": {
|
|
@@ -79,8 +79,31 @@ except ValueError:
|
|
|
79
79
|
|
|
80
80
|
normalized_tokens = [token.strip("();|&") for token in tokens]
|
|
81
81
|
|
|
82
|
+
# Git resolves any UNAMBIGUOUS abbreviation of a long option, so `--no-veri`
|
|
83
|
+
# skips hooks exactly as completely as `--no-verify`. Matched as a prefix
|
|
84
|
+
# rather than by listing abbreviations; `--no-verbose` diverges after
|
|
85
|
+
# `--no-ver` and so is correctly not caught.
|
|
86
|
+
NO_VERIFY = "--no-verify"
|
|
87
|
+
NO_VERIFY_MIN_PREFIX = len("--no-v")
|
|
88
|
+
|
|
89
|
+
|
|
90
|
+
def disables_verification(token):
|
|
91
|
+
"""Whether a token is `--no-verify` or an abbreviation git would accept.
|
|
92
|
+
|
|
93
|
+
Args:
|
|
94
|
+
token: A single shell token from the command line.
|
|
95
|
+
|
|
96
|
+
Returns:
|
|
97
|
+
True if git would read this token as --no-verify.
|
|
98
|
+
"""
|
|
99
|
+
return (
|
|
100
|
+
len(token) >= NO_VERIFY_MIN_PREFIX
|
|
101
|
+
and len(token) <= len(NO_VERIFY)
|
|
102
|
+
and NO_VERIFY.startswith(token)
|
|
103
|
+
)
|
|
104
|
+
|
|
82
105
|
for i, token in enumerate(normalized_tokens):
|
|
83
|
-
if token
|
|
106
|
+
if disables_verification(token):
|
|
84
107
|
sys.exit(1)
|
|
85
108
|
if token == "HUSKY=0" or token.startswith("HUSKY_SKIP_HOOKS="):
|
|
86
109
|
sys.exit(1)
|
|
@@ -118,8 +118,42 @@ def is_permitted_hooks_path(value):
|
|
|
118
118
|
cleaned = cleaned[2:]
|
|
119
119
|
return cleaned.rstrip("/") in PERMITTED_HOOKS_PATHS
|
|
120
120
|
|
|
121
|
+
|
|
122
|
+
# Git resolves any UNAMBIGUOUS abbreviation of a long option, so `--no-verify`
|
|
123
|
+
# is only the longest of the spellings that disable verification: `git commit
|
|
124
|
+
# --no-veri` skips hooks exactly as completely. An equality check therefore
|
|
125
|
+
# enforced the guard against the one spelling nobody in a hurry types.
|
|
126
|
+
#
|
|
127
|
+
# Matched as "a prefix of --no-verify" rather than by listing abbreviations,
|
|
128
|
+
# because the set of accepted abbreviations is a property of git's parser and
|
|
129
|
+
# changes with the surrounding options. `--no-verbose` is NOT caught, and must
|
|
130
|
+
# not be: it diverges from `--no-verify` at the character after `--no-ver`, so
|
|
131
|
+
# it fails the prefix test.
|
|
132
|
+
#
|
|
133
|
+
# The floor is `--no-v`. Shorter is refused anyway — bare `--no-` is not a flag
|
|
134
|
+
# — and blocking an abbreviation git would reject as ambiguous costs nothing,
|
|
135
|
+
# while missing one git accepts costs the whole guard.
|
|
136
|
+
NO_VERIFY = "--no-verify"
|
|
137
|
+
NO_VERIFY_MIN_PREFIX = len("--no-v")
|
|
138
|
+
|
|
139
|
+
|
|
140
|
+
def disables_verification(token):
|
|
141
|
+
"""Whether a token is `--no-verify` or an abbreviation git would accept.
|
|
142
|
+
|
|
143
|
+
Args:
|
|
144
|
+
token: A single shell token from the command line.
|
|
145
|
+
|
|
146
|
+
Returns:
|
|
147
|
+
True if git would read this token as --no-verify.
|
|
148
|
+
"""
|
|
149
|
+
return (
|
|
150
|
+
len(token) >= NO_VERIFY_MIN_PREFIX
|
|
151
|
+
and len(token) <= len(NO_VERIFY)
|
|
152
|
+
and NO_VERIFY.startswith(token)
|
|
153
|
+
)
|
|
154
|
+
|
|
121
155
|
for i, token in enumerate(normalized_tokens):
|
|
122
|
-
if token
|
|
156
|
+
if disables_verification(token):
|
|
123
157
|
sys.exit(1)
|
|
124
158
|
if token == "HUSKY=0" or token.startswith("HUSKY_SKIP_HOOKS="):
|
|
125
159
|
sys.exit(1)
|
|
@@ -172,6 +172,30 @@ function run(mode, baseRef, onlyFiles) {
|
|
|
172
172
|
const watched = plan.files.filter(f => familyFor(f));
|
|
173
173
|
if (watched.length === 0) return 0;
|
|
174
174
|
|
|
175
|
+
// A null baseline means one of two opposite things, and they must not be
|
|
176
|
+
// conflated: the file is NEW (nothing to weaken — pass), or it exists at the
|
|
177
|
+
// baseline and could not be read (nothing could be COMPARED — the one case
|
|
178
|
+
// where the ratchet cannot do its job). Both arrive here as null because
|
|
179
|
+
// `git()` swallows every failure, so the ratchet passed exactly when it had
|
|
180
|
+
// no evidence — failing open in its blind spot.
|
|
181
|
+
//
|
|
182
|
+
// `cat-file -e` answers the question `git show` cannot: does this path exist
|
|
183
|
+
// at that ref? Absent means new; present-but-unreadable means undeterminable,
|
|
184
|
+
// and an undeterminable ratchet must refuse rather than wave the change on.
|
|
185
|
+
const unreadable = watched.filter(
|
|
186
|
+
f =>
|
|
187
|
+
git(["show", `${plan.baselineRef}:${f}`], root) === null &&
|
|
188
|
+
git(["cat-file", "-e", `${plan.baselineRef}:${f}`], root) !== null
|
|
189
|
+
);
|
|
190
|
+
if (unreadable.length > 0) {
|
|
191
|
+
return undeterminable(
|
|
192
|
+
mode,
|
|
193
|
+
`could not read the baseline for ${unreadable.join(", ")} — ` +
|
|
194
|
+
`the file exists at ${plan.baselineRef} but its contents could not be ` +
|
|
195
|
+
`retrieved, so a loosened threshold could not be detected`
|
|
196
|
+
);
|
|
197
|
+
}
|
|
198
|
+
|
|
175
199
|
const findings = watched.flatMap(f =>
|
|
176
200
|
compareFile(
|
|
177
201
|
f,
|
|
@@ -79,8 +79,31 @@ except ValueError:
|
|
|
79
79
|
|
|
80
80
|
normalized_tokens = [token.strip("();|&") for token in tokens]
|
|
81
81
|
|
|
82
|
+
# Git resolves any UNAMBIGUOUS abbreviation of a long option, so `--no-veri`
|
|
83
|
+
# skips hooks exactly as completely as `--no-verify`. Matched as a prefix
|
|
84
|
+
# rather than by listing abbreviations; `--no-verbose` diverges after
|
|
85
|
+
# `--no-ver` and so is correctly not caught.
|
|
86
|
+
NO_VERIFY = "--no-verify"
|
|
87
|
+
NO_VERIFY_MIN_PREFIX = len("--no-v")
|
|
88
|
+
|
|
89
|
+
|
|
90
|
+
def disables_verification(token):
|
|
91
|
+
"""Whether a token is `--no-verify` or an abbreviation git would accept.
|
|
92
|
+
|
|
93
|
+
Args:
|
|
94
|
+
token: A single shell token from the command line.
|
|
95
|
+
|
|
96
|
+
Returns:
|
|
97
|
+
True if git would read this token as --no-verify.
|
|
98
|
+
"""
|
|
99
|
+
return (
|
|
100
|
+
len(token) >= NO_VERIFY_MIN_PREFIX
|
|
101
|
+
and len(token) <= len(NO_VERIFY)
|
|
102
|
+
and NO_VERIFY.startswith(token)
|
|
103
|
+
)
|
|
104
|
+
|
|
82
105
|
for i, token in enumerate(normalized_tokens):
|
|
83
|
-
if token
|
|
106
|
+
if disables_verification(token):
|
|
84
107
|
sys.exit(1)
|
|
85
108
|
if token == "HUSKY=0" or token.startswith("HUSKY_SKIP_HOOKS="):
|
|
86
109
|
sys.exit(1)
|
|
@@ -118,8 +118,42 @@ def is_permitted_hooks_path(value):
|
|
|
118
118
|
cleaned = cleaned[2:]
|
|
119
119
|
return cleaned.rstrip("/") in PERMITTED_HOOKS_PATHS
|
|
120
120
|
|
|
121
|
+
|
|
122
|
+
# Git resolves any UNAMBIGUOUS abbreviation of a long option, so `--no-verify`
|
|
123
|
+
# is only the longest of the spellings that disable verification: `git commit
|
|
124
|
+
# --no-veri` skips hooks exactly as completely. An equality check therefore
|
|
125
|
+
# enforced the guard against the one spelling nobody in a hurry types.
|
|
126
|
+
#
|
|
127
|
+
# Matched as "a prefix of --no-verify" rather than by listing abbreviations,
|
|
128
|
+
# because the set of accepted abbreviations is a property of git's parser and
|
|
129
|
+
# changes with the surrounding options. `--no-verbose` is NOT caught, and must
|
|
130
|
+
# not be: it diverges from `--no-verify` at the character after `--no-ver`, so
|
|
131
|
+
# it fails the prefix test.
|
|
132
|
+
#
|
|
133
|
+
# The floor is `--no-v`. Shorter is refused anyway — bare `--no-` is not a flag
|
|
134
|
+
# — and blocking an abbreviation git would reject as ambiguous costs nothing,
|
|
135
|
+
# while missing one git accepts costs the whole guard.
|
|
136
|
+
NO_VERIFY = "--no-verify"
|
|
137
|
+
NO_VERIFY_MIN_PREFIX = len("--no-v")
|
|
138
|
+
|
|
139
|
+
|
|
140
|
+
def disables_verification(token):
|
|
141
|
+
"""Whether a token is `--no-verify` or an abbreviation git would accept.
|
|
142
|
+
|
|
143
|
+
Args:
|
|
144
|
+
token: A single shell token from the command line.
|
|
145
|
+
|
|
146
|
+
Returns:
|
|
147
|
+
True if git would read this token as --no-verify.
|
|
148
|
+
"""
|
|
149
|
+
return (
|
|
150
|
+
len(token) >= NO_VERIFY_MIN_PREFIX
|
|
151
|
+
and len(token) <= len(NO_VERIFY)
|
|
152
|
+
and NO_VERIFY.startswith(token)
|
|
153
|
+
)
|
|
154
|
+
|
|
121
155
|
for i, token in enumerate(normalized_tokens):
|
|
122
|
-
if token
|
|
156
|
+
if disables_verification(token):
|
|
123
157
|
sys.exit(1)
|
|
124
158
|
if token == "HUSKY=0" or token.startswith("HUSKY_SKIP_HOOKS="):
|
|
125
159
|
sys.exit(1)
|
|
@@ -172,6 +172,30 @@ function run(mode, baseRef, onlyFiles) {
|
|
|
172
172
|
const watched = plan.files.filter(f => familyFor(f));
|
|
173
173
|
if (watched.length === 0) return 0;
|
|
174
174
|
|
|
175
|
+
// A null baseline means one of two opposite things, and they must not be
|
|
176
|
+
// conflated: the file is NEW (nothing to weaken — pass), or it exists at the
|
|
177
|
+
// baseline and could not be read (nothing could be COMPARED — the one case
|
|
178
|
+
// where the ratchet cannot do its job). Both arrive here as null because
|
|
179
|
+
// `git()` swallows every failure, so the ratchet passed exactly when it had
|
|
180
|
+
// no evidence — failing open in its blind spot.
|
|
181
|
+
//
|
|
182
|
+
// `cat-file -e` answers the question `git show` cannot: does this path exist
|
|
183
|
+
// at that ref? Absent means new; present-but-unreadable means undeterminable,
|
|
184
|
+
// and an undeterminable ratchet must refuse rather than wave the change on.
|
|
185
|
+
const unreadable = watched.filter(
|
|
186
|
+
f =>
|
|
187
|
+
git(["show", `${plan.baselineRef}:${f}`], root) === null &&
|
|
188
|
+
git(["cat-file", "-e", `${plan.baselineRef}:${f}`], root) !== null
|
|
189
|
+
);
|
|
190
|
+
if (unreadable.length > 0) {
|
|
191
|
+
return undeterminable(
|
|
192
|
+
mode,
|
|
193
|
+
`could not read the baseline for ${unreadable.join(", ")} — ` +
|
|
194
|
+
`the file exists at ${plan.baselineRef} but its contents could not be ` +
|
|
195
|
+
`retrieved, so a loosened threshold could not be detected`
|
|
196
|
+
);
|
|
197
|
+
}
|
|
198
|
+
|
|
175
199
|
const findings = watched.flatMap(f =>
|
|
176
200
|
compareFile(
|
|
177
201
|
f,
|
|
@@ -118,8 +118,42 @@ def is_permitted_hooks_path(value):
|
|
|
118
118
|
cleaned = cleaned[2:]
|
|
119
119
|
return cleaned.rstrip("/") in PERMITTED_HOOKS_PATHS
|
|
120
120
|
|
|
121
|
+
|
|
122
|
+
# Git resolves any UNAMBIGUOUS abbreviation of a long option, so `--no-verify`
|
|
123
|
+
# is only the longest of the spellings that disable verification: `git commit
|
|
124
|
+
# --no-veri` skips hooks exactly as completely. An equality check therefore
|
|
125
|
+
# enforced the guard against the one spelling nobody in a hurry types.
|
|
126
|
+
#
|
|
127
|
+
# Matched as "a prefix of --no-verify" rather than by listing abbreviations,
|
|
128
|
+
# because the set of accepted abbreviations is a property of git's parser and
|
|
129
|
+
# changes with the surrounding options. `--no-verbose` is NOT caught, and must
|
|
130
|
+
# not be: it diverges from `--no-verify` at the character after `--no-ver`, so
|
|
131
|
+
# it fails the prefix test.
|
|
132
|
+
#
|
|
133
|
+
# The floor is `--no-v`. Shorter is refused anyway — bare `--no-` is not a flag
|
|
134
|
+
# — and blocking an abbreviation git would reject as ambiguous costs nothing,
|
|
135
|
+
# while missing one git accepts costs the whole guard.
|
|
136
|
+
NO_VERIFY = "--no-verify"
|
|
137
|
+
NO_VERIFY_MIN_PREFIX = len("--no-v")
|
|
138
|
+
|
|
139
|
+
|
|
140
|
+
def disables_verification(token):
|
|
141
|
+
"""Whether a token is `--no-verify` or an abbreviation git would accept.
|
|
142
|
+
|
|
143
|
+
Args:
|
|
144
|
+
token: A single shell token from the command line.
|
|
145
|
+
|
|
146
|
+
Returns:
|
|
147
|
+
True if git would read this token as --no-verify.
|
|
148
|
+
"""
|
|
149
|
+
return (
|
|
150
|
+
len(token) >= NO_VERIFY_MIN_PREFIX
|
|
151
|
+
and len(token) <= len(NO_VERIFY)
|
|
152
|
+
and NO_VERIFY.startswith(token)
|
|
153
|
+
)
|
|
154
|
+
|
|
121
155
|
for i, token in enumerate(normalized_tokens):
|
|
122
|
-
if token
|
|
156
|
+
if disables_verification(token):
|
|
123
157
|
sys.exit(1)
|
|
124
158
|
if token == "HUSKY=0" or token.startswith("HUSKY_SKIP_HOOKS="):
|
|
125
159
|
sys.exit(1)
|
|
@@ -172,6 +172,30 @@ function run(mode, baseRef, onlyFiles) {
|
|
|
172
172
|
const watched = plan.files.filter(f => familyFor(f));
|
|
173
173
|
if (watched.length === 0) return 0;
|
|
174
174
|
|
|
175
|
+
// A null baseline means one of two opposite things, and they must not be
|
|
176
|
+
// conflated: the file is NEW (nothing to weaken — pass), or it exists at the
|
|
177
|
+
// baseline and could not be read (nothing could be COMPARED — the one case
|
|
178
|
+
// where the ratchet cannot do its job). Both arrive here as null because
|
|
179
|
+
// `git()` swallows every failure, so the ratchet passed exactly when it had
|
|
180
|
+
// no evidence — failing open in its blind spot.
|
|
181
|
+
//
|
|
182
|
+
// `cat-file -e` answers the question `git show` cannot: does this path exist
|
|
183
|
+
// at that ref? Absent means new; present-but-unreadable means undeterminable,
|
|
184
|
+
// and an undeterminable ratchet must refuse rather than wave the change on.
|
|
185
|
+
const unreadable = watched.filter(
|
|
186
|
+
f =>
|
|
187
|
+
git(["show", `${plan.baselineRef}:${f}`], root) === null &&
|
|
188
|
+
git(["cat-file", "-e", `${plan.baselineRef}:${f}`], root) !== null
|
|
189
|
+
);
|
|
190
|
+
if (unreadable.length > 0) {
|
|
191
|
+
return undeterminable(
|
|
192
|
+
mode,
|
|
193
|
+
`could not read the baseline for ${unreadable.join(", ")} — ` +
|
|
194
|
+
`the file exists at ${plan.baselineRef} but its contents could not be ` +
|
|
195
|
+
`retrieved, so a loosened threshold could not be detected`
|
|
196
|
+
);
|
|
197
|
+
}
|
|
198
|
+
|
|
175
199
|
const findings = watched.flatMap(f =>
|
|
176
200
|
compareFile(
|
|
177
201
|
f,
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "lisa-openclaw",
|
|
3
|
-
"version": "2.342.
|
|
3
|
+
"version": "2.342.6",
|
|
4
4
|
"description": "Connect staff roles to Telegram or Slack via OpenClaw — facilitator/specialist hub-and-spoke routing and repo-coding topics, for Claude Code and Codex",
|
|
5
5
|
"author": {
|
|
6
6
|
"name": "Cody Swann"
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "lisa-openclaw",
|
|
3
|
-
"version": "2.342.
|
|
3
|
+
"version": "2.342.6",
|
|
4
4
|
"description": "Connect staff roles to Telegram or Slack via OpenClaw — facilitator/specialist hub-and-spoke routing and repo-coding topics, across Claude and Codex.",
|
|
5
5
|
"author": {
|
|
6
6
|
"name": "Cody Swann"
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "lisa-openclaw",
|
|
3
|
-
"version": "2.342.
|
|
3
|
+
"version": "2.342.6",
|
|
4
4
|
"description": "Connect staff roles to Telegram or Slack via OpenClaw — facilitator/specialist hub-and-spoke routing and repo-coding topics, for Claude Code and Codex",
|
|
5
5
|
"author": {
|
|
6
6
|
"name": "Cody Swann"
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "lisa-openclaw",
|
|
3
|
-
"version": "2.342.
|
|
3
|
+
"version": "2.342.6",
|
|
4
4
|
"description": "Connect staff roles to Telegram or Slack via OpenClaw — facilitator/specialist hub-and-spoke routing and repo-coding topics, for Claude Code and Codex",
|
|
5
5
|
"author": {
|
|
6
6
|
"name": "Cody Swann"
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "lisa-openclaw",
|
|
3
|
-
"version": "2.342.
|
|
3
|
+
"version": "2.342.6",
|
|
4
4
|
"description": "Connect staff roles to Telegram or Slack via OpenClaw — facilitator/specialist hub-and-spoke routing and repo-coding topics, for Claude Code and Codex",
|
|
5
5
|
"author": {
|
|
6
6
|
"name": "Cody Swann"
|
|
@@ -79,8 +79,31 @@ except ValueError:
|
|
|
79
79
|
|
|
80
80
|
normalized_tokens = [token.strip("();|&") for token in tokens]
|
|
81
81
|
|
|
82
|
+
# Git resolves any UNAMBIGUOUS abbreviation of a long option, so `--no-veri`
|
|
83
|
+
# skips hooks exactly as completely as `--no-verify`. Matched as a prefix
|
|
84
|
+
# rather than by listing abbreviations; `--no-verbose` diverges after
|
|
85
|
+
# `--no-ver` and so is correctly not caught.
|
|
86
|
+
NO_VERIFY = "--no-verify"
|
|
87
|
+
NO_VERIFY_MIN_PREFIX = len("--no-v")
|
|
88
|
+
|
|
89
|
+
|
|
90
|
+
def disables_verification(token):
|
|
91
|
+
"""Whether a token is `--no-verify` or an abbreviation git would accept.
|
|
92
|
+
|
|
93
|
+
Args:
|
|
94
|
+
token: A single shell token from the command line.
|
|
95
|
+
|
|
96
|
+
Returns:
|
|
97
|
+
True if git would read this token as --no-verify.
|
|
98
|
+
"""
|
|
99
|
+
return (
|
|
100
|
+
len(token) >= NO_VERIFY_MIN_PREFIX
|
|
101
|
+
and len(token) <= len(NO_VERIFY)
|
|
102
|
+
and NO_VERIFY.startswith(token)
|
|
103
|
+
)
|
|
104
|
+
|
|
82
105
|
for i, token in enumerate(normalized_tokens):
|
|
83
|
-
if token
|
|
106
|
+
if disables_verification(token):
|
|
84
107
|
sys.exit(1)
|
|
85
108
|
if token == "HUSKY=0" or token.startswith("HUSKY_SKIP_HOOKS="):
|
|
86
109
|
sys.exit(1)
|
|
@@ -118,8 +118,42 @@ def is_permitted_hooks_path(value):
|
|
|
118
118
|
cleaned = cleaned[2:]
|
|
119
119
|
return cleaned.rstrip("/") in PERMITTED_HOOKS_PATHS
|
|
120
120
|
|
|
121
|
+
|
|
122
|
+
# Git resolves any UNAMBIGUOUS abbreviation of a long option, so `--no-verify`
|
|
123
|
+
# is only the longest of the spellings that disable verification: `git commit
|
|
124
|
+
# --no-veri` skips hooks exactly as completely. An equality check therefore
|
|
125
|
+
# enforced the guard against the one spelling nobody in a hurry types.
|
|
126
|
+
#
|
|
127
|
+
# Matched as "a prefix of --no-verify" rather than by listing abbreviations,
|
|
128
|
+
# because the set of accepted abbreviations is a property of git's parser and
|
|
129
|
+
# changes with the surrounding options. `--no-verbose` is NOT caught, and must
|
|
130
|
+
# not be: it diverges from `--no-verify` at the character after `--no-ver`, so
|
|
131
|
+
# it fails the prefix test.
|
|
132
|
+
#
|
|
133
|
+
# The floor is `--no-v`. Shorter is refused anyway — bare `--no-` is not a flag
|
|
134
|
+
# — and blocking an abbreviation git would reject as ambiguous costs nothing,
|
|
135
|
+
# while missing one git accepts costs the whole guard.
|
|
136
|
+
NO_VERIFY = "--no-verify"
|
|
137
|
+
NO_VERIFY_MIN_PREFIX = len("--no-v")
|
|
138
|
+
|
|
139
|
+
|
|
140
|
+
def disables_verification(token):
|
|
141
|
+
"""Whether a token is `--no-verify` or an abbreviation git would accept.
|
|
142
|
+
|
|
143
|
+
Args:
|
|
144
|
+
token: A single shell token from the command line.
|
|
145
|
+
|
|
146
|
+
Returns:
|
|
147
|
+
True if git would read this token as --no-verify.
|
|
148
|
+
"""
|
|
149
|
+
return (
|
|
150
|
+
len(token) >= NO_VERIFY_MIN_PREFIX
|
|
151
|
+
and len(token) <= len(NO_VERIFY)
|
|
152
|
+
and NO_VERIFY.startswith(token)
|
|
153
|
+
)
|
|
154
|
+
|
|
121
155
|
for i, token in enumerate(normalized_tokens):
|
|
122
|
-
if token
|
|
156
|
+
if disables_verification(token):
|
|
123
157
|
sys.exit(1)
|
|
124
158
|
if token == "HUSKY=0" or token.startswith("HUSKY_SKIP_HOOKS="):
|
|
125
159
|
sys.exit(1)
|
|
@@ -172,6 +172,30 @@ function run(mode, baseRef, onlyFiles) {
|
|
|
172
172
|
const watched = plan.files.filter(f => familyFor(f));
|
|
173
173
|
if (watched.length === 0) return 0;
|
|
174
174
|
|
|
175
|
+
// A null baseline means one of two opposite things, and they must not be
|
|
176
|
+
// conflated: the file is NEW (nothing to weaken — pass), or it exists at the
|
|
177
|
+
// baseline and could not be read (nothing could be COMPARED — the one case
|
|
178
|
+
// where the ratchet cannot do its job). Both arrive here as null because
|
|
179
|
+
// `git()` swallows every failure, so the ratchet passed exactly when it had
|
|
180
|
+
// no evidence — failing open in its blind spot.
|
|
181
|
+
//
|
|
182
|
+
// `cat-file -e` answers the question `git show` cannot: does this path exist
|
|
183
|
+
// at that ref? Absent means new; present-but-unreadable means undeterminable,
|
|
184
|
+
// and an undeterminable ratchet must refuse rather than wave the change on.
|
|
185
|
+
const unreadable = watched.filter(
|
|
186
|
+
f =>
|
|
187
|
+
git(["show", `${plan.baselineRef}:${f}`], root) === null &&
|
|
188
|
+
git(["cat-file", "-e", `${plan.baselineRef}:${f}`], root) !== null
|
|
189
|
+
);
|
|
190
|
+
if (unreadable.length > 0) {
|
|
191
|
+
return undeterminable(
|
|
192
|
+
mode,
|
|
193
|
+
`could not read the baseline for ${unreadable.join(", ")} — ` +
|
|
194
|
+
`the file exists at ${plan.baselineRef} but its contents could not be ` +
|
|
195
|
+
`retrieved, so a loosened threshold could not be detected`
|
|
196
|
+
);
|
|
197
|
+
}
|
|
198
|
+
|
|
175
199
|
const findings = watched.flatMap(f =>
|
|
176
200
|
compareFile(
|
|
177
201
|
f,
|
|
@@ -172,6 +172,30 @@ function run(mode, baseRef, onlyFiles) {
|
|
|
172
172
|
const watched = plan.files.filter(f => familyFor(f));
|
|
173
173
|
if (watched.length === 0) return 0;
|
|
174
174
|
|
|
175
|
+
// A null baseline means one of two opposite things, and they must not be
|
|
176
|
+
// conflated: the file is NEW (nothing to weaken — pass), or it exists at the
|
|
177
|
+
// baseline and could not be read (nothing could be COMPARED — the one case
|
|
178
|
+
// where the ratchet cannot do its job). Both arrive here as null because
|
|
179
|
+
// `git()` swallows every failure, so the ratchet passed exactly when it had
|
|
180
|
+
// no evidence — failing open in its blind spot.
|
|
181
|
+
//
|
|
182
|
+
// `cat-file -e` answers the question `git show` cannot: does this path exist
|
|
183
|
+
// at that ref? Absent means new; present-but-unreadable means undeterminable,
|
|
184
|
+
// and an undeterminable ratchet must refuse rather than wave the change on.
|
|
185
|
+
const unreadable = watched.filter(
|
|
186
|
+
f =>
|
|
187
|
+
git(["show", `${plan.baselineRef}:${f}`], root) === null &&
|
|
188
|
+
git(["cat-file", "-e", `${plan.baselineRef}:${f}`], root) !== null
|
|
189
|
+
);
|
|
190
|
+
if (unreadable.length > 0) {
|
|
191
|
+
return undeterminable(
|
|
192
|
+
mode,
|
|
193
|
+
`could not read the baseline for ${unreadable.join(", ")} — ` +
|
|
194
|
+
`the file exists at ${plan.baselineRef} but its contents could not be ` +
|
|
195
|
+
`retrieved, so a loosened threshold could not be detected`
|
|
196
|
+
);
|
|
197
|
+
}
|
|
198
|
+
|
|
175
199
|
const findings = watched.flatMap(f =>
|
|
176
200
|
compareFile(
|
|
177
201
|
f,
|
|
@@ -172,6 +172,30 @@ function run(mode, baseRef, onlyFiles) {
|
|
|
172
172
|
const watched = plan.files.filter(f => familyFor(f));
|
|
173
173
|
if (watched.length === 0) return 0;
|
|
174
174
|
|
|
175
|
+
// A null baseline means one of two opposite things, and they must not be
|
|
176
|
+
// conflated: the file is NEW (nothing to weaken — pass), or it exists at the
|
|
177
|
+
// baseline and could not be read (nothing could be COMPARED — the one case
|
|
178
|
+
// where the ratchet cannot do its job). Both arrive here as null because
|
|
179
|
+
// `git()` swallows every failure, so the ratchet passed exactly when it had
|
|
180
|
+
// no evidence — failing open in its blind spot.
|
|
181
|
+
//
|
|
182
|
+
// `cat-file -e` answers the question `git show` cannot: does this path exist
|
|
183
|
+
// at that ref? Absent means new; present-but-unreadable means undeterminable,
|
|
184
|
+
// and an undeterminable ratchet must refuse rather than wave the change on.
|
|
185
|
+
const unreadable = watched.filter(
|
|
186
|
+
f =>
|
|
187
|
+
git(["show", `${plan.baselineRef}:${f}`], root) === null &&
|
|
188
|
+
git(["cat-file", "-e", `${plan.baselineRef}:${f}`], root) !== null
|
|
189
|
+
);
|
|
190
|
+
if (unreadable.length > 0) {
|
|
191
|
+
return undeterminable(
|
|
192
|
+
mode,
|
|
193
|
+
`could not read the baseline for ${unreadable.join(", ")} — ` +
|
|
194
|
+
`the file exists at ${plan.baselineRef} but its contents could not be ` +
|
|
195
|
+
`retrieved, so a loosened threshold could not be detected`
|
|
196
|
+
);
|
|
197
|
+
}
|
|
198
|
+
|
|
175
199
|
const findings = watched.flatMap(f =>
|
|
176
200
|
compareFile(
|
|
177
201
|
f,
|