@codyswann/lisa 2.166.0 → 2.166.2
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/dist/codex/scripts/block-no-verify.sh +1 -1
- package/dist/configs/vitest/base.d.ts +20 -2
- package/dist/configs/vitest/base.d.ts.map +1 -1
- package/dist/configs/vitest/base.js +23 -4
- package/dist/configs/vitest/base.js.map +1 -1
- package/dist/configs/vitest/nestjs.d.ts +2 -2
- package/dist/configs/vitest/nestjs.d.ts.map +1 -1
- package/dist/configs/vitest/nestjs.js +4 -4
- package/dist/configs/vitest/nestjs.js.map +1 -1
- package/dist/configs/vitest/typescript.d.ts +2 -2
- package/dist/configs/vitest/typescript.d.ts.map +1 -1
- package/dist/configs/vitest/typescript.js +4 -4
- package/dist/configs/vitest/typescript.js.map +1 -1
- 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 +1 -1
- package/plugins/lisa/hooks/block-no-verify.sh +4 -5
- package/plugins/lisa/rules/eager/base-rules.md +1 -0
- package/plugins/lisa/rules/eager/security-audit-handling.md +8 -2
- package/plugins/lisa/rules/reference/base-rules.md +1 -0
- package/plugins/lisa/rules/reference/security-audit-handling.md +8 -2
- package/plugins/lisa-agy/hooks/block-no-verify.agy.sh +1 -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 +4 -5
- package/plugins/lisa-copilot/rules/eager/base-rules.md +1 -0
- package/plugins/lisa-copilot/rules/eager/security-audit-handling.md +8 -2
- package/plugins/lisa-copilot/rules/reference/base-rules.md +1 -0
- package/plugins/lisa-copilot/rules/reference/security-audit-handling.md +8 -2
- package/plugins/lisa-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-cursor/hooks/block-no-verify.sh +4 -5
- package/plugins/lisa-cursor/rules/base-rules-reference.mdc +1 -0
- package/plugins/lisa-cursor/rules/base-rules.mdc +1 -0
- package/plugins/lisa-cursor/rules/security-audit-handling-reference.mdc +8 -2
- package/plugins/lisa-cursor/rules/security-audit-handling.mdc +8 -2
- 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 +1 -1
- package/plugins/src/base/hooks/block-no-verify.sh +4 -5
- package/plugins/src/base/rules/eager/base-rules.md +1 -0
- package/plugins/src/base/rules/eager/security-audit-handling.md +8 -2
- package/plugins/src/base/rules/reference/base-rules.md +1 -0
- package/plugins/src/base/rules/reference/security-audit-handling.md +8 -2
|
@@ -81,7 +81,7 @@ then
|
|
|
81
81
|
"hookSpecificOutput": {
|
|
82
82
|
"hookEventName": "PreToolUse",
|
|
83
83
|
"permissionDecision": "deny",
|
|
84
|
-
"permissionDecisionReason": "Blocked: this command bypasses pre-commit/pre-push hooks (--no-verify, HUSKY=0, or core.hooksPath disabling). Fix the underlying issue
|
|
84
|
+
"permissionDecisionReason": "Blocked: this command bypasses pre-commit/pre-push hooks (--no-verify, HUSKY=0, or core.hooksPath disabling). Fix the underlying issue (security audit, lint, typecheck, tests, formatting) instead. If a fix is genuinely impossible, ask the user to make the risk-acceptance decision and add a specific documented ignore; never bypass the hook."
|
|
85
85
|
}
|
|
86
86
|
}'
|
|
87
87
|
fi
|
|
@@ -53,10 +53,28 @@ export declare const defaultThresholds: PortableThresholds;
|
|
|
53
53
|
export declare const defaultCoverageExclusions: readonly string[];
|
|
54
54
|
/**
|
|
55
55
|
* Default patterns to exclude from test discovery across all stacks.
|
|
56
|
-
*
|
|
57
|
-
*
|
|
56
|
+
*
|
|
57
|
+
* The `.claude/worktrees/` exclusion is intentionally NOT baked in here —
|
|
58
|
+
* it is cwd-conditional and supplied by {@link worktreeExclusions} so that
|
|
59
|
+
* a vitest run launched from INSIDE a worktree can still discover its own
|
|
60
|
+
* tests. Stack factories spread `worktreeExclusions()` alongside this list.
|
|
58
61
|
*/
|
|
59
62
|
export declare const defaultTestExclusions: readonly string[];
|
|
63
|
+
/**
|
|
64
|
+
* Returns the worktree exclusion glob a stack config should add to skip
|
|
65
|
+
* test files / coverage that live inside `.claude/worktrees/`.
|
|
66
|
+
*
|
|
67
|
+
* Lisa manages `.claude/worktrees/` as scratch worktrees for subagents.
|
|
68
|
+
* When vitest runs from the primary checkout, tests inside those worktrees
|
|
69
|
+
* should be skipped — each worktree has its own vitest run. When vitest runs
|
|
70
|
+
* from INSIDE a worktree (the project root *is* the worktree), the same glob
|
|
71
|
+
* matches every path under root and vitest finds zero tests. This returns the
|
|
72
|
+
* glob only when the current working directory is outside a worktree, so each
|
|
73
|
+
* stack factory can spread it into its `exclude` arrays without hand-rolling
|
|
74
|
+
* the conditional. Mirrors jest's `worktreeTestPathIgnorePatterns()`.
|
|
75
|
+
* @returns Single-entry array with the worktree exclude glob, or an empty array when already inside a worktree.
|
|
76
|
+
*/
|
|
77
|
+
export declare function worktreeExclusions(): readonly string[];
|
|
60
78
|
/**
|
|
61
79
|
* Maps portable threshold format (with `global` wrapper) to Vitest's
|
|
62
80
|
* flat threshold format. Projects store thresholds in the portable
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"base.d.ts","sourceRoot":"","sources":["../../../src/configs/vitest/base.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AACH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,eAAe,CAAC;AAEpD,8DAA8D;AAC9D,KAAK,UAAU,GAAG,cAAc,CAAC;AAEjC;;;;GAIG;AACH,MAAM,WAAW,gBAAgB;IAC/B,QAAQ,CAAC,UAAU,CAAC,EAAE,MAAM,CAAC;IAC7B,QAAQ,CAAC,QAAQ,CAAC,EAAE,MAAM,CAAC;IAC3B,QAAQ,CAAC,SAAS,CAAC,EAAE,MAAM,CAAC;IAC5B,QAAQ,CAAC,KAAK,CAAC,EAAE,MAAM,CAAC;CACzB;AAED;;;;GAIG;AACH,MAAM,WAAW,kBAAkB;IACjC,QAAQ,CAAC,MAAM,CAAC,EAAE;QAChB,QAAQ,CAAC,UAAU,CAAC,EAAE,MAAM,CAAC;QAC7B,QAAQ,CAAC,QAAQ,CAAC,EAAE,MAAM,CAAC;QAC3B,QAAQ,CAAC,SAAS,CAAC,EAAE,MAAM,CAAC;QAC5B,QAAQ,CAAC,KAAK,CAAC,EAAE,MAAM,CAAC;KACzB,CAAC;IACF,QAAQ,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC;CAClC;AAED;;;GAGG;AACH,eAAO,MAAM,iBAAiB,EAAE,kBAO/B,CAAC;AAEF;;;;;;GAMG;AACH,eAAO,MAAM,yBAAyB,EAAE,SAAS,MAAM,
|
|
1
|
+
{"version":3,"file":"base.d.ts","sourceRoot":"","sources":["../../../src/configs/vitest/base.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AACH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,eAAe,CAAC;AAEpD,8DAA8D;AAC9D,KAAK,UAAU,GAAG,cAAc,CAAC;AAEjC;;;;GAIG;AACH,MAAM,WAAW,gBAAgB;IAC/B,QAAQ,CAAC,UAAU,CAAC,EAAE,MAAM,CAAC;IAC7B,QAAQ,CAAC,QAAQ,CAAC,EAAE,MAAM,CAAC;IAC3B,QAAQ,CAAC,SAAS,CAAC,EAAE,MAAM,CAAC;IAC5B,QAAQ,CAAC,KAAK,CAAC,EAAE,MAAM,CAAC;CACzB;AAED;;;;GAIG;AACH,MAAM,WAAW,kBAAkB;IACjC,QAAQ,CAAC,MAAM,CAAC,EAAE;QAChB,QAAQ,CAAC,UAAU,CAAC,EAAE,MAAM,CAAC;QAC7B,QAAQ,CAAC,QAAQ,CAAC,EAAE,MAAM,CAAC;QAC3B,QAAQ,CAAC,SAAS,CAAC,EAAE,MAAM,CAAC;QAC5B,QAAQ,CAAC,KAAK,CAAC,EAAE,MAAM,CAAC;KACzB,CAAC;IACF,QAAQ,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC;CAClC;AAED;;;GAGG;AACH,eAAO,MAAM,iBAAiB,EAAE,kBAO/B,CAAC;AAEF;;;;;;GAMG;AACH,eAAO,MAAM,yBAAyB,EAAE,SAAS,MAAM,EAatD,CAAC;AAEF;;;;;;;GAOG;AACH,eAAO,MAAM,qBAAqB,EAAE,SAAS,MAAM,EAGlD,CAAC;AAEF;;;;;;;;;;;;;GAaG;AACH,wBAAgB,kBAAkB,IAAI,SAAS,MAAM,EAAE,CAMtD;AAED;;;;;;GAMG;AACH,eAAO,MAAM,aAAa,GACxB,YAAY,kBAAkB,KAC7B,gBAaD,CAAC;AAEH;;;;;;;;;;GAUG;AACH,eAAO,MAAM,eAAe,GAC1B,UAAU,kBAAkB,EAC5B,WAAW,kBAAkB,KAC5B,kBAOD,CAAC;AAEH;;;;;;;GAOG;AACH,eAAO,MAAM,kBAAkB,GAAI,GAAG,SAAS,UAAU,EAAE,KAAG,UA4C7D,CAAC"}
|
|
@@ -22,7 +22,6 @@ export const defaultCoverageExclusions = [
|
|
|
22
22
|
"**/index.ts",
|
|
23
23
|
"**/node_modules/**",
|
|
24
24
|
"**/dist/**",
|
|
25
|
-
"**/.claude/worktrees/**",
|
|
26
25
|
"**/*.test.ts",
|
|
27
26
|
"**/*.spec.ts",
|
|
28
27
|
"**/*.mock.ts",
|
|
@@ -34,14 +33,34 @@ export const defaultCoverageExclusions = [
|
|
|
34
33
|
];
|
|
35
34
|
/**
|
|
36
35
|
* Default patterns to exclude from test discovery across all stacks.
|
|
37
|
-
*
|
|
38
|
-
*
|
|
36
|
+
*
|
|
37
|
+
* The `.claude/worktrees/` exclusion is intentionally NOT baked in here —
|
|
38
|
+
* it is cwd-conditional and supplied by {@link worktreeExclusions} so that
|
|
39
|
+
* a vitest run launched from INSIDE a worktree can still discover its own
|
|
40
|
+
* tests. Stack factories spread `worktreeExclusions()` alongside this list.
|
|
39
41
|
*/
|
|
40
42
|
export const defaultTestExclusions = [
|
|
41
43
|
"**/node_modules/**",
|
|
42
44
|
"**/dist/**",
|
|
43
|
-
"**/.claude/worktrees/**",
|
|
44
45
|
];
|
|
46
|
+
/**
|
|
47
|
+
* Returns the worktree exclusion glob a stack config should add to skip
|
|
48
|
+
* test files / coverage that live inside `.claude/worktrees/`.
|
|
49
|
+
*
|
|
50
|
+
* Lisa manages `.claude/worktrees/` as scratch worktrees for subagents.
|
|
51
|
+
* When vitest runs from the primary checkout, tests inside those worktrees
|
|
52
|
+
* should be skipped — each worktree has its own vitest run. When vitest runs
|
|
53
|
+
* from INSIDE a worktree (the project root *is* the worktree), the same glob
|
|
54
|
+
* matches every path under root and vitest finds zero tests. This returns the
|
|
55
|
+
* glob only when the current working directory is outside a worktree, so each
|
|
56
|
+
* stack factory can spread it into its `exclude` arrays without hand-rolling
|
|
57
|
+
* the conditional. Mirrors jest's `worktreeTestPathIgnorePatterns()`.
|
|
58
|
+
* @returns Single-entry array with the worktree exclude glob, or an empty array when already inside a worktree.
|
|
59
|
+
*/
|
|
60
|
+
export function worktreeExclusions() {
|
|
61
|
+
const isInsideWorktree = /[/\\]\.claude[/\\]worktrees(?:[/\\]|$)/.test(process.cwd());
|
|
62
|
+
return isInsideWorktree ? [] : ["**/.claude/worktrees/**"];
|
|
63
|
+
}
|
|
45
64
|
/**
|
|
46
65
|
* Maps portable threshold format (with `global` wrapper) to Vitest's
|
|
47
66
|
* flat threshold format. Projects store thresholds in the portable
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"base.js","sourceRoot":"","sources":["../../../src/configs/vitest/base.ts"],"names":[],"mappings":"AA4CA;;;GAGG;AACH,MAAM,CAAC,MAAM,iBAAiB,GAAuB;IACnD,MAAM,EAAE;QACN,UAAU,EAAE,EAAE;QACd,QAAQ,EAAE,EAAE;QACZ,SAAS,EAAE,EAAE;QACb,KAAK,EAAE,EAAE;KACV;CACF,CAAC;AAEF;;;;;;GAMG;AACH,MAAM,CAAC,MAAM,yBAAyB,GAAsB;IAC1D,WAAW;IACX,aAAa;IACb,oBAAoB;IACpB,YAAY;IACZ,
|
|
1
|
+
{"version":3,"file":"base.js","sourceRoot":"","sources":["../../../src/configs/vitest/base.ts"],"names":[],"mappings":"AA4CA;;;GAGG;AACH,MAAM,CAAC,MAAM,iBAAiB,GAAuB;IACnD,MAAM,EAAE;QACN,UAAU,EAAE,EAAE;QACd,QAAQ,EAAE,EAAE;QACZ,SAAS,EAAE,EAAE;QACb,KAAK,EAAE,EAAE;KACV;CACF,CAAC;AAEF;;;;;;GAMG;AACH,MAAM,CAAC,MAAM,yBAAyB,GAAsB;IAC1D,WAAW;IACX,aAAa;IACb,oBAAoB;IACpB,YAAY;IACZ,cAAc;IACd,cAAc;IACd,cAAc;IACd,YAAY;IACZ,aAAa;IACb,iBAAiB;IACjB,iBAAiB;IACjB,qBAAqB;CACtB,CAAC;AAEF;;;;;;;GAOG;AACH,MAAM,CAAC,MAAM,qBAAqB,GAAsB;IACtD,oBAAoB;IACpB,YAAY;CACb,CAAC;AAEF;;;;;;;;;;;;;GAaG;AACH,MAAM,UAAU,kBAAkB;IAChC,MAAM,gBAAgB,GAAG,wCAAwC,CAAC,IAAI,CACpE,OAAO,CAAC,GAAG,EAAE,CACd,CAAC;IAEF,OAAO,gBAAgB,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,yBAAyB,CAAC,CAAC;AAC7D,CAAC;AAED;;;;;;GAMG;AACH,MAAM,CAAC,MAAM,aAAa,GAAG,CAC3B,UAA8B,EACZ,EAAE,CAAC,CAAC;IACtB,GAAG,CAAC,UAAU,CAAC,MAAM,EAAE,UAAU,KAAK,SAAS;QAC7C,CAAC,CAAC,EAAE,UAAU,EAAE,UAAU,CAAC,MAAM,CAAC,UAAU,EAAE;QAC9C,CAAC,CAAC,EAAE,CAAC;IACP,GAAG,CAAC,UAAU,CAAC,MAAM,EAAE,QAAQ,KAAK,SAAS;QAC3C,CAAC,CAAC,EAAE,QAAQ,EAAE,UAAU,CAAC,MAAM,CAAC,QAAQ,EAAE;QAC1C,CAAC,CAAC,EAAE,CAAC;IACP,GAAG,CAAC,UAAU,CAAC,MAAM,EAAE,SAAS,KAAK,SAAS;QAC5C,CAAC,CAAC,EAAE,SAAS,EAAE,UAAU,CAAC,MAAM,CAAC,SAAS,EAAE;QAC5C,CAAC,CAAC,EAAE,CAAC;IACP,GAAG,CAAC,UAAU,CAAC,MAAM,EAAE,KAAK,KAAK,SAAS;QACxC,CAAC,CAAC,EAAE,KAAK,EAAE,UAAU,CAAC,MAAM,CAAC,KAAK,EAAE;QACpC,CAAC,CAAC,EAAE,CAAC;CACR,CAAC,CAAC;AAEH;;;;;;;;;;GAUG;AACH,MAAM,CAAC,MAAM,eAAe,GAAG,CAC7B,QAA4B,EAC5B,SAA6B,EACT,EAAE,CAAC,CAAC;IACxB,GAAG,QAAQ;IACX,GAAG,SAAS;IACZ,MAAM,EAAE;QACN,GAAI,QAAQ,CAAC,MAAiC;QAC9C,GAAI,SAAS,CAAC,MAAiC;KAChD;CACF,CAAC,CAAC;AAEH;;;;;;;GAOG;AACH,MAAM,CAAC,MAAM,kBAAkB,GAAG,CAAC,GAAG,OAAqB,EAAc,EAAE;IACzE,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACzB,OAAO,EAAE,CAAC;IACZ,CAAC;IAED,MAAM,YAAY,GAAG,CACnB,CAA0B,EAC1B,CAA0B,EACD,EAAE,CAC3B,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,MAAM,CACnB,CAAC,GAAG,EAAE,GAAG,EAAE,EAAE;QACX,MAAM,MAAM,GAAG,GAAG,CAAC,GAAG,CAAC,CAAC;QACxB,MAAM,IAAI,GAAG,CAAC,CAAC,GAAG,CAAC,CAAC;QAEpB,MAAM,MAAM,GACV,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,IAAI,KAAK,CAAC,OAAO,CAAC,IAAI,CAAC;YAC1C,CAAC,CAAC,CAAC,GAAG,IAAI,GAAG,CAAC,CAAC,GAAG,MAAM,EAAE,GAAG,IAAI,CAAC,CAAC,CAAC;YACpC,CAAC,CAAC,OAAO,MAAM,KAAK,QAAQ;gBACxB,MAAM,KAAK,IAAI;gBACf,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC;gBACtB,OAAO,IAAI,KAAK,QAAQ;gBACxB,IAAI,KAAK,IAAI;gBACb,CAAC,KAAK,CAAC,OAAO,CAAC,IAAI,CAAC;gBACtB,CAAC,CAAC;oBACE,GAAI,MAAkC;oBACtC,GAAI,IAAgC;iBACrC;gBACH,CAAC,CAAC,IAAI,CAAC;QAEb,OAAO,EAAE,GAAG,GAAG,EAAE,CAAC,GAAG,CAAC,EAAE,MAAM,EAAE,CAAC;IACnC,CAAC,EACD,EAAE,GAAG,CAAC,EAAE,CACT,CAAC;IAEJ,OAAO,OAAO,CAAC,MAAM,CAAC,CAAC,GAAG,EAAE,MAAM,EAAE,EAAE;QACpC,MAAM,OAAO,GAAG,CAAC,GAAG,CAAC,IAAI,IAAI,EAAE,CAA4B,CAAC;QAC5D,MAAM,UAAU,GAAG,CAAC,MAAM,CAAC,IAAI,IAAI,EAAE,CAA4B,CAAC;QAElE,OAAO;YACL,GAAG,GAAG;YACN,GAAG,MAAM;YACT,IAAI,EAAE,YAAY,CAAC,OAAO,EAAE,UAAU,CAAC;SACxC,CAAC;IACJ,CAAC,EAAE,EAAgB,CAAC,CAAC;AACvB,CAAC,CAAC"}
|
|
@@ -13,9 +13,9 @@
|
|
|
13
13
|
import type { ViteUserConfig } from "vitest/config";
|
|
14
14
|
/** Vite UserConfig augmented with Vitest's `test` property */
|
|
15
15
|
type UserConfig = ViteUserConfig;
|
|
16
|
-
import { defaultCoverageExclusions, defaultTestExclusions, defaultThresholds, mapThresholds, mergeThresholds, mergeVitestConfigs } from "./base.js";
|
|
16
|
+
import { defaultCoverageExclusions, defaultTestExclusions, defaultThresholds, mapThresholds, mergeThresholds, mergeVitestConfigs, worktreeExclusions } from "./base.js";
|
|
17
17
|
import type { PortableThresholds } from "./base.js";
|
|
18
|
-
export { defaultCoverageExclusions, defaultTestExclusions, defaultThresholds, mapThresholds, mergeThresholds, mergeVitestConfigs, };
|
|
18
|
+
export { defaultCoverageExclusions, defaultTestExclusions, defaultThresholds, mapThresholds, mergeThresholds, mergeVitestConfigs, worktreeExclusions, };
|
|
19
19
|
export type { PortableThresholds };
|
|
20
20
|
/**
|
|
21
21
|
* Options for configuring the NestJS Vitest config factory.
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"nestjs.d.ts","sourceRoot":"","sources":["../../../src/configs/vitest/nestjs.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AACH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,eAAe,CAAC;AAEpD,8DAA8D;AAC9D,KAAK,UAAU,GAAG,cAAc,CAAC;AAEjC,OAAO,EACL,yBAAyB,EACzB,qBAAqB,EACrB,iBAAiB,EACjB,aAAa,EACb,eAAe,EACf,kBAAkB,EACnB,MAAM,WAAW,CAAC;AAEnB,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,WAAW,CAAC;AAGpD,OAAO,EACL,yBAAyB,EACzB,qBAAqB,EACrB,iBAAiB,EACjB,aAAa,EACb,eAAe,EACf,kBAAkB,GACnB,CAAC;AAEF,YAAY,EAAE,kBAAkB,EAAE,CAAC;AAEnC;;GAEG;AACH,UAAU,mBAAmB;IAC3B,mFAAmF;IACnF,QAAQ,CAAC,UAAU,CAAC,EAAE,kBAAkB,CAAC;CAC1C;AAwBD;;;;;;;;;GASG;AACH,eAAO,MAAM,qBAAqB,GAAI,kBAEnC,mBAAwB,KAAG,UAkB5B,CAAC"}
|
|
1
|
+
{"version":3,"file":"nestjs.d.ts","sourceRoot":"","sources":["../../../src/configs/vitest/nestjs.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AACH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,eAAe,CAAC;AAEpD,8DAA8D;AAC9D,KAAK,UAAU,GAAG,cAAc,CAAC;AAEjC,OAAO,EACL,yBAAyB,EACzB,qBAAqB,EACrB,iBAAiB,EACjB,aAAa,EACb,eAAe,EACf,kBAAkB,EAClB,kBAAkB,EACnB,MAAM,WAAW,CAAC;AAEnB,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,WAAW,CAAC;AAGpD,OAAO,EACL,yBAAyB,EACzB,qBAAqB,EACrB,iBAAiB,EACjB,aAAa,EACb,eAAe,EACf,kBAAkB,EAClB,kBAAkB,GACnB,CAAC;AAEF,YAAY,EAAE,kBAAkB,EAAE,CAAC;AAEnC;;GAEG;AACH,UAAU,mBAAmB;IAC3B,mFAAmF;IACnF,QAAQ,CAAC,UAAU,CAAC,EAAE,kBAAkB,CAAC;CAC1C;AAwBD;;;;;;;;;GASG;AACH,eAAO,MAAM,qBAAqB,GAAI,kBAEnC,mBAAwB,KAAG,UAkB5B,CAAC"}
|
|
@@ -1,6 +1,6 @@
|
|
|
1
|
-
import { defaultCoverageExclusions, defaultTestExclusions, defaultThresholds, mapThresholds, mergeThresholds, mergeVitestConfigs, } from "./base.js";
|
|
1
|
+
import { defaultCoverageExclusions, defaultTestExclusions, defaultThresholds, mapThresholds, mergeThresholds, mergeVitestConfigs, worktreeExclusions, } from "./base.js";
|
|
2
2
|
// Re-export base utilities for entry-point configs
|
|
3
|
-
export { defaultCoverageExclusions, defaultTestExclusions, defaultThresholds, mapThresholds, mergeThresholds, mergeVitestConfigs, };
|
|
3
|
+
export { defaultCoverageExclusions, defaultTestExclusions, defaultThresholds, mapThresholds, mergeThresholds, mergeVitestConfigs, worktreeExclusions, };
|
|
4
4
|
/**
|
|
5
5
|
* NestJS-specific patterns excluded from coverage collection.
|
|
6
6
|
* These are generated or boilerplate files that don't benefit from coverage tracking.
|
|
@@ -41,12 +41,12 @@ export const getNestjsVitestConfig = ({ thresholds = defaultThresholds, } = {})
|
|
|
41
41
|
// on "No test files found" otherwise. See typescript.ts for rationale.
|
|
42
42
|
passWithNoTests: true,
|
|
43
43
|
include: ["**/*.spec.ts"],
|
|
44
|
-
exclude: [...defaultTestExclusions],
|
|
44
|
+
exclude: [...defaultTestExclusions, ...worktreeExclusions()],
|
|
45
45
|
testTimeout: 10000,
|
|
46
46
|
coverage: {
|
|
47
47
|
provider: "v8",
|
|
48
48
|
include: ["**/*.ts"],
|
|
49
|
-
exclude: [...nestjsCoverageExclusions],
|
|
49
|
+
exclude: [...nestjsCoverageExclusions, ...worktreeExclusions()],
|
|
50
50
|
thresholds: mapThresholds(thresholds),
|
|
51
51
|
},
|
|
52
52
|
},
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"nestjs.js","sourceRoot":"","sources":["../../../src/configs/vitest/nestjs.ts"],"names":[],"mappings":"AAiBA,OAAO,EACL,yBAAyB,EACzB,qBAAqB,EACrB,iBAAiB,EACjB,aAAa,EACb,eAAe,EACf,kBAAkB,GACnB,MAAM,WAAW,CAAC;AAInB,mDAAmD;AACnD,OAAO,EACL,yBAAyB,EACzB,qBAAqB,EACrB,iBAAiB,EACjB,aAAa,EACb,eAAe,EACf,kBAAkB,GACnB,CAAC;AAYF;;;GAGG;AACH,MAAM,wBAAwB,GAAsB;IAClD,GAAG,yBAAyB;IAC5B,gBAAgB;IAChB,aAAa;IACb,eAAe;IACf,cAAc;IACd,eAAe;IACf,gBAAgB;IAChB,iBAAiB;IACjB,cAAc;IACd,mBAAmB;IACnB,mBAAmB;IACnB,2BAA2B;IAC3B,sBAAsB;IACtB,eAAe;IACf,YAAY;CACb,CAAC;AAEF;;;;;;;;;GASG;AACH,MAAM,CAAC,MAAM,qBAAqB,GAAG,CAAC,EACpC,UAAU,GAAG,iBAAiB,MACP,EAAE,EAAc,EAAE,CAAC,CAAC;IAC3C,IAAI,EAAE;QACJ,OAAO,EAAE,IAAI;QACb,WAAW,EAAE,MAAM;QACnB,IAAI,EAAE,KAAK;QACX,8EAA8E;QAC9E,uEAAuE;QACvE,eAAe,EAAE,IAAI;QACrB,OAAO,EAAE,CAAC,cAAc,CAAC;QACzB,OAAO,EAAE,CAAC,GAAG,qBAAqB,CAAC;
|
|
1
|
+
{"version":3,"file":"nestjs.js","sourceRoot":"","sources":["../../../src/configs/vitest/nestjs.ts"],"names":[],"mappings":"AAiBA,OAAO,EACL,yBAAyB,EACzB,qBAAqB,EACrB,iBAAiB,EACjB,aAAa,EACb,eAAe,EACf,kBAAkB,EAClB,kBAAkB,GACnB,MAAM,WAAW,CAAC;AAInB,mDAAmD;AACnD,OAAO,EACL,yBAAyB,EACzB,qBAAqB,EACrB,iBAAiB,EACjB,aAAa,EACb,eAAe,EACf,kBAAkB,EAClB,kBAAkB,GACnB,CAAC;AAYF;;;GAGG;AACH,MAAM,wBAAwB,GAAsB;IAClD,GAAG,yBAAyB;IAC5B,gBAAgB;IAChB,aAAa;IACb,eAAe;IACf,cAAc;IACd,eAAe;IACf,gBAAgB;IAChB,iBAAiB;IACjB,cAAc;IACd,mBAAmB;IACnB,mBAAmB;IACnB,2BAA2B;IAC3B,sBAAsB;IACtB,eAAe;IACf,YAAY;CACb,CAAC;AAEF;;;;;;;;;GASG;AACH,MAAM,CAAC,MAAM,qBAAqB,GAAG,CAAC,EACpC,UAAU,GAAG,iBAAiB,MACP,EAAE,EAAc,EAAE,CAAC,CAAC;IAC3C,IAAI,EAAE;QACJ,OAAO,EAAE,IAAI;QACb,WAAW,EAAE,MAAM;QACnB,IAAI,EAAE,KAAK;QACX,8EAA8E;QAC9E,uEAAuE;QACvE,eAAe,EAAE,IAAI;QACrB,OAAO,EAAE,CAAC,cAAc,CAAC;QACzB,OAAO,EAAE,CAAC,GAAG,qBAAqB,EAAE,GAAG,kBAAkB,EAAE,CAAC;QAC5D,WAAW,EAAE,KAAK;QAClB,QAAQ,EAAE;YACR,QAAQ,EAAE,IAAI;YACd,OAAO,EAAE,CAAC,SAAS,CAAC;YACpB,OAAO,EAAE,CAAC,GAAG,wBAAwB,EAAE,GAAG,kBAAkB,EAAE,CAAC;YAC/D,UAAU,EAAE,aAAa,CAAC,UAAU,CAAC;SACtC;KACF;CACF,CAAC,CAAC"}
|
|
@@ -9,9 +9,9 @@
|
|
|
9
9
|
import type { ViteUserConfig } from "vitest/config";
|
|
10
10
|
/** Vite UserConfig augmented with Vitest's `test` property */
|
|
11
11
|
type UserConfig = ViteUserConfig;
|
|
12
|
-
import { defaultCoverageExclusions, defaultTestExclusions, defaultThresholds, mapThresholds, mergeThresholds, mergeVitestConfigs } from "./base.js";
|
|
12
|
+
import { defaultCoverageExclusions, defaultTestExclusions, defaultThresholds, mapThresholds, mergeThresholds, mergeVitestConfigs, worktreeExclusions } from "./base.js";
|
|
13
13
|
import type { PortableThresholds } from "./base.js";
|
|
14
|
-
export { defaultCoverageExclusions, defaultTestExclusions, defaultThresholds, mapThresholds, mergeThresholds, mergeVitestConfigs, };
|
|
14
|
+
export { defaultCoverageExclusions, defaultTestExclusions, defaultThresholds, mapThresholds, mergeThresholds, mergeVitestConfigs, worktreeExclusions, };
|
|
15
15
|
export type { PortableThresholds };
|
|
16
16
|
/**
|
|
17
17
|
* Options for configuring the TypeScript Vitest config factory.
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"typescript.d.ts","sourceRoot":"","sources":["../../../src/configs/vitest/typescript.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AACH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,eAAe,CAAC;AAEpD,8DAA8D;AAC9D,KAAK,UAAU,GAAG,cAAc,CAAC;AAEjC,OAAO,EACL,yBAAyB,EACzB,qBAAqB,EACrB,iBAAiB,EACjB,aAAa,EACb,eAAe,EACf,kBAAkB,EACnB,MAAM,WAAW,CAAC;AAEnB,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,WAAW,CAAC;AAGpD,OAAO,EACL,yBAAyB,EACzB,qBAAqB,EACrB,iBAAiB,EACjB,aAAa,EACb,eAAe,EACf,kBAAkB,GACnB,CAAC;AAEF,YAAY,EAAE,kBAAkB,EAAE,CAAC;AAEnC;;GAEG;AACH,UAAU,uBAAuB;IAC/B,mFAAmF;IACnF,QAAQ,CAAC,UAAU,CAAC,EAAE,kBAAkB,CAAC;CAC1C;AAED;;;;;;;;;GASG;AACH,eAAO,MAAM,yBAAyB,GAAI,kBAEvC,uBAA4B,KAAG,UAmBhC,CAAC"}
|
|
1
|
+
{"version":3,"file":"typescript.d.ts","sourceRoot":"","sources":["../../../src/configs/vitest/typescript.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AACH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,eAAe,CAAC;AAEpD,8DAA8D;AAC9D,KAAK,UAAU,GAAG,cAAc,CAAC;AAEjC,OAAO,EACL,yBAAyB,EACzB,qBAAqB,EACrB,iBAAiB,EACjB,aAAa,EACb,eAAe,EACf,kBAAkB,EAClB,kBAAkB,EACnB,MAAM,WAAW,CAAC;AAEnB,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,WAAW,CAAC;AAGpD,OAAO,EACL,yBAAyB,EACzB,qBAAqB,EACrB,iBAAiB,EACjB,aAAa,EACb,eAAe,EACf,kBAAkB,EAClB,kBAAkB,GACnB,CAAC;AAEF,YAAY,EAAE,kBAAkB,EAAE,CAAC;AAEnC;;GAEG;AACH,UAAU,uBAAuB;IAC/B,mFAAmF;IACnF,QAAQ,CAAC,UAAU,CAAC,EAAE,kBAAkB,CAAC;CAC1C;AAED;;;;;;;;;GASG;AACH,eAAO,MAAM,yBAAyB,GAAI,kBAEvC,uBAA4B,KAAG,UAmBhC,CAAC"}
|
|
@@ -1,6 +1,6 @@
|
|
|
1
|
-
import { defaultCoverageExclusions, defaultTestExclusions, defaultThresholds, mapThresholds, mergeThresholds, mergeVitestConfigs, } from "./base.js";
|
|
1
|
+
import { defaultCoverageExclusions, defaultTestExclusions, defaultThresholds, mapThresholds, mergeThresholds, mergeVitestConfigs, worktreeExclusions, } from "./base.js";
|
|
2
2
|
// Re-export base utilities for stack-specific configs to use
|
|
3
|
-
export { defaultCoverageExclusions, defaultTestExclusions, defaultThresholds, mapThresholds, mergeThresholds, mergeVitestConfigs, };
|
|
3
|
+
export { defaultCoverageExclusions, defaultTestExclusions, defaultThresholds, mapThresholds, mergeThresholds, mergeVitestConfigs, worktreeExclusions, };
|
|
4
4
|
/**
|
|
5
5
|
* Creates a Vitest configuration for TypeScript/Node projects.
|
|
6
6
|
*
|
|
@@ -21,12 +21,12 @@ export const getTypescriptVitestConfig = ({ thresholds = defaultThresholds, } =
|
|
|
21
21
|
// the test-world analog of allowing `files: []` in tsconfig for zero sources.
|
|
22
22
|
passWithNoTests: true,
|
|
23
23
|
include: ["tests/**/*.test.ts", "src/**/*.test.ts"],
|
|
24
|
-
exclude: [...defaultTestExclusions],
|
|
24
|
+
exclude: [...defaultTestExclusions, ...worktreeExclusions()],
|
|
25
25
|
testTimeout: 10000,
|
|
26
26
|
coverage: {
|
|
27
27
|
provider: "v8",
|
|
28
28
|
include: ["src/**/*.ts"],
|
|
29
|
-
exclude: [...defaultCoverageExclusions],
|
|
29
|
+
exclude: [...defaultCoverageExclusions, ...worktreeExclusions()],
|
|
30
30
|
thresholds: mapThresholds(thresholds),
|
|
31
31
|
},
|
|
32
32
|
},
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"typescript.js","sourceRoot":"","sources":["../../../src/configs/vitest/typescript.ts"],"names":[],"mappings":"AAaA,OAAO,EACL,yBAAyB,EACzB,qBAAqB,EACrB,iBAAiB,EACjB,aAAa,EACb,eAAe,EACf,kBAAkB,GACnB,MAAM,WAAW,CAAC;AAInB,6DAA6D;AAC7D,OAAO,EACL,yBAAyB,EACzB,qBAAqB,EACrB,iBAAiB,EACjB,aAAa,EACb,eAAe,EACf,kBAAkB,GACnB,CAAC;AAYF;;;;;;;;;GASG;AACH,MAAM,CAAC,MAAM,yBAAyB,GAAG,CAAC,EACxC,UAAU,GAAG,iBAAiB,MACH,EAAE,EAAc,EAAE,CAAC,CAAC;IAC/C,IAAI,EAAE;QACJ,OAAO,EAAE,IAAI;QACb,WAAW,EAAE,MAAM;QACnB,4EAA4E;QAC5E,uEAAuE;QACvE,2EAA2E;QAC3E,8EAA8E;QAC9E,eAAe,EAAE,IAAI;QACrB,OAAO,EAAE,CAAC,oBAAoB,EAAE,kBAAkB,CAAC;QACnD,OAAO,EAAE,CAAC,GAAG,qBAAqB,CAAC;
|
|
1
|
+
{"version":3,"file":"typescript.js","sourceRoot":"","sources":["../../../src/configs/vitest/typescript.ts"],"names":[],"mappings":"AAaA,OAAO,EACL,yBAAyB,EACzB,qBAAqB,EACrB,iBAAiB,EACjB,aAAa,EACb,eAAe,EACf,kBAAkB,EAClB,kBAAkB,GACnB,MAAM,WAAW,CAAC;AAInB,6DAA6D;AAC7D,OAAO,EACL,yBAAyB,EACzB,qBAAqB,EACrB,iBAAiB,EACjB,aAAa,EACb,eAAe,EACf,kBAAkB,EAClB,kBAAkB,GACnB,CAAC;AAYF;;;;;;;;;GASG;AACH,MAAM,CAAC,MAAM,yBAAyB,GAAG,CAAC,EACxC,UAAU,GAAG,iBAAiB,MACH,EAAE,EAAc,EAAE,CAAC,CAAC;IAC/C,IAAI,EAAE;QACJ,OAAO,EAAE,IAAI;QACb,WAAW,EAAE,MAAM;QACnB,4EAA4E;QAC5E,uEAAuE;QACvE,2EAA2E;QAC3E,8EAA8E;QAC9E,eAAe,EAAE,IAAI;QACrB,OAAO,EAAE,CAAC,oBAAoB,EAAE,kBAAkB,CAAC;QACnD,OAAO,EAAE,CAAC,GAAG,qBAAqB,EAAE,GAAG,kBAAkB,EAAE,CAAC;QAC5D,WAAW,EAAE,KAAK;QAClB,QAAQ,EAAE;YACR,QAAQ,EAAE,IAAI;YACd,OAAO,EAAE,CAAC,aAAa,CAAC;YACxB,OAAO,EAAE,CAAC,GAAG,yBAAyB,EAAE,GAAG,kBAAkB,EAAE,CAAC;YAChE,UAAU,EAAE,aAAa,CAAC,UAAU,CAAC;SACtC;KACF;CACF,CAAC,CAAC"}
|
package/package.json
CHANGED
|
@@ -85,7 +85,7 @@
|
|
|
85
85
|
"lodash": ">=4.18.1"
|
|
86
86
|
},
|
|
87
87
|
"name": "@codyswann/lisa",
|
|
88
|
-
"version": "2.166.
|
|
88
|
+
"version": "2.166.2",
|
|
89
89
|
"description": "Claude Code governance framework that applies guardrails, guidance, and automated enforcement to projects",
|
|
90
90
|
"main": "dist/index.js",
|
|
91
91
|
"exports": {
|
|
@@ -25,7 +25,7 @@ allow() {
|
|
|
25
25
|
}
|
|
26
26
|
|
|
27
27
|
deny() {
|
|
28
|
-
printf '%s\n' '{"decision":"deny","reason":"This command bypasses Lisa pre-commit/pre-push quality gates (--no-verify, HUSKY=0, or core.hooksPath disabling). Fix the underlying issue (lint, tests, formatting)
|
|
28
|
+
printf '%s\n' '{"decision":"deny","reason":"This command bypasses Lisa pre-commit/pre-push quality gates (--no-verify, HUSKY=0, or core.hooksPath disabling). Fix the underlying issue (security audit, lint, typecheck, tests, formatting) instead. If a fix is genuinely impossible, ask the user to make the risk-acceptance decision and add a specific documented ignore; never bypass the hook."}'
|
|
29
29
|
exit 0
|
|
30
30
|
}
|
|
31
31
|
|
|
@@ -91,11 +91,10 @@ PY
|
|
|
91
91
|
then
|
|
92
92
|
cat >&2 <<'EOF'
|
|
93
93
|
Blocked: this command bypasses pre-commit/pre-push hooks (--no-verify, HUSKY=0,
|
|
94
|
-
or core.hooksPath disabling). Fix the underlying issue (
|
|
95
|
-
|
|
96
|
-
|
|
97
|
-
|
|
98
|
-
re-run after they confirm.
|
|
94
|
+
or core.hooksPath disabling). Fix the underlying issue (security audit, lint,
|
|
95
|
+
typecheck, tests, formatting) instead. If a fix is genuinely impossible, ask the
|
|
96
|
+
user to make the risk-acceptance decision and add a specific documented ignore;
|
|
97
|
+
never bypass the hook.
|
|
99
98
|
EOF
|
|
100
99
|
exit 2
|
|
101
100
|
fi
|
|
@@ -25,6 +25,7 @@ Do not begin work if there are blockers, ambiguities, access requirements, or un
|
|
|
25
25
|
## Git Discipline
|
|
26
26
|
|
|
27
27
|
- **Never use `--no-verify`** or bypass any git hook.
|
|
28
|
+
- When a hook or quality gate fails, fix the root cause first. If no fix is genuinely possible, ask the user to make the risk-acceptance decision and add a specific documented ignore; never use a blanket bypass.
|
|
28
29
|
- **Never bypass branch protection** — no `--admin`, `--force`, no merging a PR with failing CI. "Green in CI" is the definition of done.
|
|
29
30
|
- Never commit directly to environment branches (`dev`, `staging`, `main`).
|
|
30
31
|
- Prefix `git push` with `GIT_SSH_COMMAND="ssh -o ServerAliveInterval=30 -o ServerAliveCountMax=5"`.
|
|
@@ -1,6 +1,12 @@
|
|
|
1
1
|
# Security Audit Handling (load-bearing)
|
|
2
2
|
|
|
3
|
-
If `git push` fails because the pre-push hook reports security vulnerabilities, follow the rules below. **Never use `--no-verify
|
|
3
|
+
If `git push` fails because the pre-push hook reports security vulnerabilities, follow the rules below. **Never use `--no-verify`**, `HUSKY=0`, `core.hooksPath`, or any other hook bypass to skip the security audit.
|
|
4
|
+
|
|
5
|
+
## Fix before ignore
|
|
6
|
+
|
|
7
|
+
1. Fix the root cause first: upgrade or override the actually-vulnerable leaf package to a patched compatible version, regenerate the lockfile, and retry the gate.
|
|
8
|
+
2. Only if no safe fix exists, ask the user to make the risk-acceptance decision. Add a narrow documented ignore for the specific advisory, package, and reason.
|
|
9
|
+
3. Never add a blanket audit bypass, lower an audit level, or self-approve a new risk-acceptance entry.
|
|
4
10
|
|
|
5
11
|
## Core rule
|
|
6
12
|
|
|
@@ -17,7 +23,7 @@ Before adding any override, verify:
|
|
|
17
23
|
|
|
18
24
|
1. Note GHSA ID, package, advisory URL.
|
|
19
25
|
2. If a patched version exists: add a resolution AND override in `package.json` for the leaf package, regenerate the lockfile, commit, retry.
|
|
20
|
-
3. If no patch but safe (transitive, no untrusted input, dev/build only): add an exclusion to `audit.ignore.local.json` with `{"id", "package", "reason"}`, commit, retry.
|
|
26
|
+
3. If no patch but safe (transitive, no untrusted input, dev/build only): ask the user to make the risk-acceptance decision, then add an exclusion to `audit.ignore.local.json` with `{"id", "package", "reason"}`, commit, retry.
|
|
21
27
|
|
|
22
28
|
## Rails (bundler-audit)
|
|
23
29
|
|
|
@@ -50,6 +50,7 @@ Git Discipline:
|
|
|
50
50
|
- Prefix git push with `GIT_SSH_COMMAND="ssh -o ServerAliveInterval=30 -o ServerAliveCountMax=5"`.
|
|
51
51
|
- Never commit directly to an environment branch (dev, staging, main).
|
|
52
52
|
- Never use --no-verify or attempt to bypass a git hook.
|
|
53
|
+
- When a pre-commit, pre-push, CI, or other quality gate fails, fix the root cause first: upgrade the vulnerable dependency, fix the lint/type/test failure, remove the secret, or repair the failing check. If a fix is genuinely impossible, ask the user to make the risk-acceptance decision and add a narrow, documented ignore for the specific failing rule or advisory. Never use `--no-verify`, hook environment switches, blanket ignores, or threshold reductions as a substitute for fixing the gate.
|
|
53
54
|
- Never bypass branch protection. Never use `--admin`, `--force`, or any other flag to merge a PR that has failing CI checks. If CI fails, fix it. If you cannot fix it, escalate to the human. There are zero exceptions. "Green in CI" is the definition of done — not "green locally." A PR is not complete until CI passes on the actual PR branch.
|
|
54
55
|
- Never stash changes you cannot commit. Either fix whatever is preventing the commit or fail out and let the human know why.
|
|
55
56
|
- Never add "BREAKING CHANGE" to a commit message unless there is actually a breaking change.
|
|
@@ -1,13 +1,19 @@
|
|
|
1
1
|
# Security Audit Handling
|
|
2
2
|
|
|
3
|
-
If `git push` fails because the pre-push hook reports security vulnerabilities, follow these steps. Never use `--no-verify`
|
|
3
|
+
If `git push` fails because the pre-push hook reports security vulnerabilities, follow these steps. Never use `--no-verify`, `HUSKY=0`, `core.hooksPath`, or any other hook bypass to skip the security audit.
|
|
4
|
+
|
|
5
|
+
## Fix before ignore
|
|
6
|
+
|
|
7
|
+
1. Fix the root cause first: upgrade or override the actually-vulnerable leaf package to a patched compatible version, regenerate the lockfile, and retry the gate.
|
|
8
|
+
2. Only if no safe fix exists, ask the user to make the risk-acceptance decision. Add a narrow documented ignore for the specific advisory, package, and reason.
|
|
9
|
+
3. Never add a blanket audit bypass, lower an audit level, or self-approve a new risk-acceptance entry.
|
|
4
10
|
|
|
5
11
|
## Node.js Projects (GHSA advisories)
|
|
6
12
|
|
|
7
13
|
1. Note the GHSA ID(s), affected package(s), and advisory URL from the error output
|
|
8
14
|
2. Check the advisory URL to determine if a patched version of the vulnerable package exists
|
|
9
15
|
3. If a patched version exists: add a resolution/override in package.json to force the patched version (add to both `resolutions` and `overrides` sections), then run the package manager install command to regenerate the lockfile, commit the changes, and retry the push
|
|
10
|
-
4. If no patched version exists and the vulnerability is safe for this project (e.g., transitive dependency with no untrusted input, devDeps only, or build tool only): add an exclusion entry to `audit.ignore.local.json` with the format `{"id": "GHSA-xxx", "package": "pkg-name", "reason": "why this is safe for this project"}`, then commit and retry the push
|
|
16
|
+
4. If no patched version exists and the vulnerability is safe for this project (e.g., transitive dependency with no untrusted input, devDeps only, or build tool only): ask the user to make the risk-acceptance decision, then add an exclusion entry to `audit.ignore.local.json` with the format `{"id": "GHSA-xxx", "package": "pkg-name", "reason": "why this is safe for this project"}`, then commit and retry the push
|
|
11
17
|
|
|
12
18
|
### Critical: Override the vulnerable package, not its parent
|
|
13
19
|
|
|
@@ -25,7 +25,7 @@ allow() {
|
|
|
25
25
|
}
|
|
26
26
|
|
|
27
27
|
deny() {
|
|
28
|
-
printf '%s\n' '{"decision":"deny","reason":"This command bypasses Lisa pre-commit/pre-push quality gates (--no-verify, HUSKY=0, or core.hooksPath disabling). Fix the underlying issue (lint, tests, formatting)
|
|
28
|
+
printf '%s\n' '{"decision":"deny","reason":"This command bypasses Lisa pre-commit/pre-push quality gates (--no-verify, HUSKY=0, or core.hooksPath disabling). Fix the underlying issue (security audit, lint, typecheck, tests, formatting) instead. If a fix is genuinely impossible, ask the user to make the risk-acceptance decision and add a specific documented ignore; never bypass the hook."}'
|
|
29
29
|
exit 0
|
|
30
30
|
}
|
|
31
31
|
|
|
@@ -91,11 +91,10 @@ PY
|
|
|
91
91
|
then
|
|
92
92
|
cat >&2 <<'EOF'
|
|
93
93
|
Blocked: this command bypasses pre-commit/pre-push hooks (--no-verify, HUSKY=0,
|
|
94
|
-
or core.hooksPath disabling). Fix the underlying issue (
|
|
95
|
-
|
|
96
|
-
|
|
97
|
-
|
|
98
|
-
re-run after they confirm.
|
|
94
|
+
or core.hooksPath disabling). Fix the underlying issue (security audit, lint,
|
|
95
|
+
typecheck, tests, formatting) instead. If a fix is genuinely impossible, ask the
|
|
96
|
+
user to make the risk-acceptance decision and add a specific documented ignore;
|
|
97
|
+
never bypass the hook.
|
|
99
98
|
EOF
|
|
100
99
|
exit 2
|
|
101
100
|
fi
|
|
@@ -25,6 +25,7 @@ Do not begin work if there are blockers, ambiguities, access requirements, or un
|
|
|
25
25
|
## Git Discipline
|
|
26
26
|
|
|
27
27
|
- **Never use `--no-verify`** or bypass any git hook.
|
|
28
|
+
- When a hook or quality gate fails, fix the root cause first. If no fix is genuinely possible, ask the user to make the risk-acceptance decision and add a specific documented ignore; never use a blanket bypass.
|
|
28
29
|
- **Never bypass branch protection** — no `--admin`, `--force`, no merging a PR with failing CI. "Green in CI" is the definition of done.
|
|
29
30
|
- Never commit directly to environment branches (`dev`, `staging`, `main`).
|
|
30
31
|
- Prefix `git push` with `GIT_SSH_COMMAND="ssh -o ServerAliveInterval=30 -o ServerAliveCountMax=5"`.
|
|
@@ -1,6 +1,12 @@
|
|
|
1
1
|
# Security Audit Handling (load-bearing)
|
|
2
2
|
|
|
3
|
-
If `git push` fails because the pre-push hook reports security vulnerabilities, follow the rules below. **Never use `--no-verify
|
|
3
|
+
If `git push` fails because the pre-push hook reports security vulnerabilities, follow the rules below. **Never use `--no-verify`**, `HUSKY=0`, `core.hooksPath`, or any other hook bypass to skip the security audit.
|
|
4
|
+
|
|
5
|
+
## Fix before ignore
|
|
6
|
+
|
|
7
|
+
1. Fix the root cause first: upgrade or override the actually-vulnerable leaf package to a patched compatible version, regenerate the lockfile, and retry the gate.
|
|
8
|
+
2. Only if no safe fix exists, ask the user to make the risk-acceptance decision. Add a narrow documented ignore for the specific advisory, package, and reason.
|
|
9
|
+
3. Never add a blanket audit bypass, lower an audit level, or self-approve a new risk-acceptance entry.
|
|
4
10
|
|
|
5
11
|
## Core rule
|
|
6
12
|
|
|
@@ -17,7 +23,7 @@ Before adding any override, verify:
|
|
|
17
23
|
|
|
18
24
|
1. Note GHSA ID, package, advisory URL.
|
|
19
25
|
2. If a patched version exists: add a resolution AND override in `package.json` for the leaf package, regenerate the lockfile, commit, retry.
|
|
20
|
-
3. If no patch but safe (transitive, no untrusted input, dev/build only): add an exclusion to `audit.ignore.local.json` with `{"id", "package", "reason"}`, commit, retry.
|
|
26
|
+
3. If no patch but safe (transitive, no untrusted input, dev/build only): ask the user to make the risk-acceptance decision, then add an exclusion to `audit.ignore.local.json` with `{"id", "package", "reason"}`, commit, retry.
|
|
21
27
|
|
|
22
28
|
## Rails (bundler-audit)
|
|
23
29
|
|
|
@@ -50,6 +50,7 @@ Git Discipline:
|
|
|
50
50
|
- Prefix git push with `GIT_SSH_COMMAND="ssh -o ServerAliveInterval=30 -o ServerAliveCountMax=5"`.
|
|
51
51
|
- Never commit directly to an environment branch (dev, staging, main).
|
|
52
52
|
- Never use --no-verify or attempt to bypass a git hook.
|
|
53
|
+
- When a pre-commit, pre-push, CI, or other quality gate fails, fix the root cause first: upgrade the vulnerable dependency, fix the lint/type/test failure, remove the secret, or repair the failing check. If a fix is genuinely impossible, ask the user to make the risk-acceptance decision and add a narrow, documented ignore for the specific failing rule or advisory. Never use `--no-verify`, hook environment switches, blanket ignores, or threshold reductions as a substitute for fixing the gate.
|
|
53
54
|
- Never bypass branch protection. Never use `--admin`, `--force`, or any other flag to merge a PR that has failing CI checks. If CI fails, fix it. If you cannot fix it, escalate to the human. There are zero exceptions. "Green in CI" is the definition of done — not "green locally." A PR is not complete until CI passes on the actual PR branch.
|
|
54
55
|
- Never stash changes you cannot commit. Either fix whatever is preventing the commit or fail out and let the human know why.
|
|
55
56
|
- Never add "BREAKING CHANGE" to a commit message unless there is actually a breaking change.
|
|
@@ -1,13 +1,19 @@
|
|
|
1
1
|
# Security Audit Handling
|
|
2
2
|
|
|
3
|
-
If `git push` fails because the pre-push hook reports security vulnerabilities, follow these steps. Never use `--no-verify`
|
|
3
|
+
If `git push` fails because the pre-push hook reports security vulnerabilities, follow these steps. Never use `--no-verify`, `HUSKY=0`, `core.hooksPath`, or any other hook bypass to skip the security audit.
|
|
4
|
+
|
|
5
|
+
## Fix before ignore
|
|
6
|
+
|
|
7
|
+
1. Fix the root cause first: upgrade or override the actually-vulnerable leaf package to a patched compatible version, regenerate the lockfile, and retry the gate.
|
|
8
|
+
2. Only if no safe fix exists, ask the user to make the risk-acceptance decision. Add a narrow documented ignore for the specific advisory, package, and reason.
|
|
9
|
+
3. Never add a blanket audit bypass, lower an audit level, or self-approve a new risk-acceptance entry.
|
|
4
10
|
|
|
5
11
|
## Node.js Projects (GHSA advisories)
|
|
6
12
|
|
|
7
13
|
1. Note the GHSA ID(s), affected package(s), and advisory URL from the error output
|
|
8
14
|
2. Check the advisory URL to determine if a patched version of the vulnerable package exists
|
|
9
15
|
3. If a patched version exists: add a resolution/override in package.json to force the patched version (add to both `resolutions` and `overrides` sections), then run the package manager install command to regenerate the lockfile, commit the changes, and retry the push
|
|
10
|
-
4. If no patched version exists and the vulnerability is safe for this project (e.g., transitive dependency with no untrusted input, devDeps only, or build tool only): add an exclusion entry to `audit.ignore.local.json` with the format `{"id": "GHSA-xxx", "package": "pkg-name", "reason": "why this is safe for this project"}`, then commit and retry the push
|
|
16
|
+
4. If no patched version exists and the vulnerability is safe for this project (e.g., transitive dependency with no untrusted input, devDeps only, or build tool only): ask the user to make the risk-acceptance decision, then add an exclusion entry to `audit.ignore.local.json` with the format `{"id": "GHSA-xxx", "package": "pkg-name", "reason": "why this is safe for this project"}`, then commit and retry the push
|
|
11
17
|
|
|
12
18
|
### Critical: Override the vulnerable package, not its parent
|
|
13
19
|
|
|
@@ -91,11 +91,10 @@ PY
|
|
|
91
91
|
then
|
|
92
92
|
cat >&2 <<'EOF'
|
|
93
93
|
Blocked: this command bypasses pre-commit/pre-push hooks (--no-verify, HUSKY=0,
|
|
94
|
-
or core.hooksPath disabling). Fix the underlying issue (
|
|
95
|
-
|
|
96
|
-
|
|
97
|
-
|
|
98
|
-
re-run after they confirm.
|
|
94
|
+
or core.hooksPath disabling). Fix the underlying issue (security audit, lint,
|
|
95
|
+
typecheck, tests, formatting) instead. If a fix is genuinely impossible, ask the
|
|
96
|
+
user to make the risk-acceptance decision and add a specific documented ignore;
|
|
97
|
+
never bypass the hook.
|
|
99
98
|
EOF
|
|
100
99
|
exit 2
|
|
101
100
|
fi
|
|
@@ -55,6 +55,7 @@ Git Discipline:
|
|
|
55
55
|
- Prefix git push with `GIT_SSH_COMMAND="ssh -o ServerAliveInterval=30 -o ServerAliveCountMax=5"`.
|
|
56
56
|
- Never commit directly to an environment branch (dev, staging, main).
|
|
57
57
|
- Never use --no-verify or attempt to bypass a git hook.
|
|
58
|
+
- When a pre-commit, pre-push, CI, or other quality gate fails, fix the root cause first: upgrade the vulnerable dependency, fix the lint/type/test failure, remove the secret, or repair the failing check. If a fix is genuinely impossible, ask the user to make the risk-acceptance decision and add a narrow, documented ignore for the specific failing rule or advisory. Never use `--no-verify`, hook environment switches, blanket ignores, or threshold reductions as a substitute for fixing the gate.
|
|
58
59
|
- Never bypass branch protection. Never use `--admin`, `--force`, or any other flag to merge a PR that has failing CI checks. If CI fails, fix it. If you cannot fix it, escalate to the human. There are zero exceptions. "Green in CI" is the definition of done — not "green locally." A PR is not complete until CI passes on the actual PR branch.
|
|
59
60
|
- Never stash changes you cannot commit. Either fix whatever is preventing the commit or fail out and let the human know why.
|
|
60
61
|
- Never add "BREAKING CHANGE" to a commit message unless there is actually a breaking change.
|
|
@@ -30,6 +30,7 @@ Do not begin work if there are blockers, ambiguities, access requirements, or un
|
|
|
30
30
|
## Git Discipline
|
|
31
31
|
|
|
32
32
|
- **Never use `--no-verify`** or bypass any git hook.
|
|
33
|
+
- When a hook or quality gate fails, fix the root cause first. If no fix is genuinely possible, ask the user to make the risk-acceptance decision and add a specific documented ignore; never use a blanket bypass.
|
|
33
34
|
- **Never bypass branch protection** — no `--admin`, `--force`, no merging a PR with failing CI. "Green in CI" is the definition of done.
|
|
34
35
|
- Never commit directly to environment branches (`dev`, `staging`, `main`).
|
|
35
36
|
- Prefix `git push` with `GIT_SSH_COMMAND="ssh -o ServerAliveInterval=30 -o ServerAliveCountMax=5"`.
|
|
@@ -5,14 +5,20 @@ alwaysApply: false
|
|
|
5
5
|
|
|
6
6
|
# Security Audit Handling
|
|
7
7
|
|
|
8
|
-
If `git push` fails because the pre-push hook reports security vulnerabilities, follow these steps. Never use `--no-verify`
|
|
8
|
+
If `git push` fails because the pre-push hook reports security vulnerabilities, follow these steps. Never use `--no-verify`, `HUSKY=0`, `core.hooksPath`, or any other hook bypass to skip the security audit.
|
|
9
|
+
|
|
10
|
+
## Fix before ignore
|
|
11
|
+
|
|
12
|
+
1. Fix the root cause first: upgrade or override the actually-vulnerable leaf package to a patched compatible version, regenerate the lockfile, and retry the gate.
|
|
13
|
+
2. Only if no safe fix exists, ask the user to make the risk-acceptance decision. Add a narrow documented ignore for the specific advisory, package, and reason.
|
|
14
|
+
3. Never add a blanket audit bypass, lower an audit level, or self-approve a new risk-acceptance entry.
|
|
9
15
|
|
|
10
16
|
## Node.js Projects (GHSA advisories)
|
|
11
17
|
|
|
12
18
|
1. Note the GHSA ID(s), affected package(s), and advisory URL from the error output
|
|
13
19
|
2. Check the advisory URL to determine if a patched version of the vulnerable package exists
|
|
14
20
|
3. If a patched version exists: add a resolution/override in package.json to force the patched version (add to both `resolutions` and `overrides` sections), then run the package manager install command to regenerate the lockfile, commit the changes, and retry the push
|
|
15
|
-
4. If no patched version exists and the vulnerability is safe for this project (e.g., transitive dependency with no untrusted input, devDeps only, or build tool only): add an exclusion entry to `audit.ignore.local.json` with the format `{"id": "GHSA-xxx", "package": "pkg-name", "reason": "why this is safe for this project"}`, then commit and retry the push
|
|
21
|
+
4. If no patched version exists and the vulnerability is safe for this project (e.g., transitive dependency with no untrusted input, devDeps only, or build tool only): ask the user to make the risk-acceptance decision, then add an exclusion entry to `audit.ignore.local.json` with the format `{"id": "GHSA-xxx", "package": "pkg-name", "reason": "why this is safe for this project"}`, then commit and retry the push
|
|
16
22
|
|
|
17
23
|
### Critical: Override the vulnerable package, not its parent
|
|
18
24
|
|
|
@@ -5,7 +5,13 @@ alwaysApply: true
|
|
|
5
5
|
|
|
6
6
|
# Security Audit Handling (load-bearing)
|
|
7
7
|
|
|
8
|
-
If `git push` fails because the pre-push hook reports security vulnerabilities, follow the rules below. **Never use `--no-verify
|
|
8
|
+
If `git push` fails because the pre-push hook reports security vulnerabilities, follow the rules below. **Never use `--no-verify`**, `HUSKY=0`, `core.hooksPath`, or any other hook bypass to skip the security audit.
|
|
9
|
+
|
|
10
|
+
## Fix before ignore
|
|
11
|
+
|
|
12
|
+
1. Fix the root cause first: upgrade or override the actually-vulnerable leaf package to a patched compatible version, regenerate the lockfile, and retry the gate.
|
|
13
|
+
2. Only if no safe fix exists, ask the user to make the risk-acceptance decision. Add a narrow documented ignore for the specific advisory, package, and reason.
|
|
14
|
+
3. Never add a blanket audit bypass, lower an audit level, or self-approve a new risk-acceptance entry.
|
|
9
15
|
|
|
10
16
|
## Core rule
|
|
11
17
|
|
|
@@ -22,7 +28,7 @@ Before adding any override, verify:
|
|
|
22
28
|
|
|
23
29
|
1. Note GHSA ID, package, advisory URL.
|
|
24
30
|
2. If a patched version exists: add a resolution AND override in `package.json` for the leaf package, regenerate the lockfile, commit, retry.
|
|
25
|
-
3. If no patch but safe (transitive, no untrusted input, dev/build only): add an exclusion to `audit.ignore.local.json` with `{"id", "package", "reason"}`, commit, retry.
|
|
31
|
+
3. If no patch but safe (transitive, no untrusted input, dev/build only): ask the user to make the risk-acceptance decision, then add an exclusion to `audit.ignore.local.json` with `{"id", "package", "reason"}`, commit, retry.
|
|
26
32
|
|
|
27
33
|
## Rails (bundler-audit)
|
|
28
34
|
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "lisa-openclaw",
|
|
3
|
-
"version": "2.166.
|
|
3
|
+
"version": "2.166.2",
|
|
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.166.
|
|
3
|
+
"version": "2.166.2",
|
|
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.166.
|
|
3
|
+
"version": "2.166.2",
|
|
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.166.
|
|
3
|
+
"version": "2.166.2",
|
|
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.166.
|
|
3
|
+
"version": "2.166.2",
|
|
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"
|
|
@@ -25,7 +25,7 @@ allow() {
|
|
|
25
25
|
}
|
|
26
26
|
|
|
27
27
|
deny() {
|
|
28
|
-
printf '%s\n' '{"decision":"deny","reason":"This command bypasses Lisa pre-commit/pre-push quality gates (--no-verify, HUSKY=0, or core.hooksPath disabling). Fix the underlying issue (lint, tests, formatting)
|
|
28
|
+
printf '%s\n' '{"decision":"deny","reason":"This command bypasses Lisa pre-commit/pre-push quality gates (--no-verify, HUSKY=0, or core.hooksPath disabling). Fix the underlying issue (security audit, lint, typecheck, tests, formatting) instead. If a fix is genuinely impossible, ask the user to make the risk-acceptance decision and add a specific documented ignore; never bypass the hook."}'
|
|
29
29
|
exit 0
|
|
30
30
|
}
|
|
31
31
|
|
|
@@ -91,11 +91,10 @@ PY
|
|
|
91
91
|
then
|
|
92
92
|
cat >&2 <<'EOF'
|
|
93
93
|
Blocked: this command bypasses pre-commit/pre-push hooks (--no-verify, HUSKY=0,
|
|
94
|
-
or core.hooksPath disabling). Fix the underlying issue (
|
|
95
|
-
|
|
96
|
-
|
|
97
|
-
|
|
98
|
-
re-run after they confirm.
|
|
94
|
+
or core.hooksPath disabling). Fix the underlying issue (security audit, lint,
|
|
95
|
+
typecheck, tests, formatting) instead. If a fix is genuinely impossible, ask the
|
|
96
|
+
user to make the risk-acceptance decision and add a specific documented ignore;
|
|
97
|
+
never bypass the hook.
|
|
99
98
|
EOF
|
|
100
99
|
exit 2
|
|
101
100
|
fi
|
|
@@ -25,6 +25,7 @@ Do not begin work if there are blockers, ambiguities, access requirements, or un
|
|
|
25
25
|
## Git Discipline
|
|
26
26
|
|
|
27
27
|
- **Never use `--no-verify`** or bypass any git hook.
|
|
28
|
+
- When a hook or quality gate fails, fix the root cause first. If no fix is genuinely possible, ask the user to make the risk-acceptance decision and add a specific documented ignore; never use a blanket bypass.
|
|
28
29
|
- **Never bypass branch protection** — no `--admin`, `--force`, no merging a PR with failing CI. "Green in CI" is the definition of done.
|
|
29
30
|
- Never commit directly to environment branches (`dev`, `staging`, `main`).
|
|
30
31
|
- Prefix `git push` with `GIT_SSH_COMMAND="ssh -o ServerAliveInterval=30 -o ServerAliveCountMax=5"`.
|
|
@@ -1,6 +1,12 @@
|
|
|
1
1
|
# Security Audit Handling (load-bearing)
|
|
2
2
|
|
|
3
|
-
If `git push` fails because the pre-push hook reports security vulnerabilities, follow the rules below. **Never use `--no-verify
|
|
3
|
+
If `git push` fails because the pre-push hook reports security vulnerabilities, follow the rules below. **Never use `--no-verify`**, `HUSKY=0`, `core.hooksPath`, or any other hook bypass to skip the security audit.
|
|
4
|
+
|
|
5
|
+
## Fix before ignore
|
|
6
|
+
|
|
7
|
+
1. Fix the root cause first: upgrade or override the actually-vulnerable leaf package to a patched compatible version, regenerate the lockfile, and retry the gate.
|
|
8
|
+
2. Only if no safe fix exists, ask the user to make the risk-acceptance decision. Add a narrow documented ignore for the specific advisory, package, and reason.
|
|
9
|
+
3. Never add a blanket audit bypass, lower an audit level, or self-approve a new risk-acceptance entry.
|
|
4
10
|
|
|
5
11
|
## Core rule
|
|
6
12
|
|
|
@@ -17,7 +23,7 @@ Before adding any override, verify:
|
|
|
17
23
|
|
|
18
24
|
1. Note GHSA ID, package, advisory URL.
|
|
19
25
|
2. If a patched version exists: add a resolution AND override in `package.json` for the leaf package, regenerate the lockfile, commit, retry.
|
|
20
|
-
3. If no patch but safe (transitive, no untrusted input, dev/build only): add an exclusion to `audit.ignore.local.json` with `{"id", "package", "reason"}`, commit, retry.
|
|
26
|
+
3. If no patch but safe (transitive, no untrusted input, dev/build only): ask the user to make the risk-acceptance decision, then add an exclusion to `audit.ignore.local.json` with `{"id", "package", "reason"}`, commit, retry.
|
|
21
27
|
|
|
22
28
|
## Rails (bundler-audit)
|
|
23
29
|
|
|
@@ -50,6 +50,7 @@ Git Discipline:
|
|
|
50
50
|
- Prefix git push with `GIT_SSH_COMMAND="ssh -o ServerAliveInterval=30 -o ServerAliveCountMax=5"`.
|
|
51
51
|
- Never commit directly to an environment branch (dev, staging, main).
|
|
52
52
|
- Never use --no-verify or attempt to bypass a git hook.
|
|
53
|
+
- When a pre-commit, pre-push, CI, or other quality gate fails, fix the root cause first: upgrade the vulnerable dependency, fix the lint/type/test failure, remove the secret, or repair the failing check. If a fix is genuinely impossible, ask the user to make the risk-acceptance decision and add a narrow, documented ignore for the specific failing rule or advisory. Never use `--no-verify`, hook environment switches, blanket ignores, or threshold reductions as a substitute for fixing the gate.
|
|
53
54
|
- Never bypass branch protection. Never use `--admin`, `--force`, or any other flag to merge a PR that has failing CI checks. If CI fails, fix it. If you cannot fix it, escalate to the human. There are zero exceptions. "Green in CI" is the definition of done — not "green locally." A PR is not complete until CI passes on the actual PR branch.
|
|
54
55
|
- Never stash changes you cannot commit. Either fix whatever is preventing the commit or fail out and let the human know why.
|
|
55
56
|
- Never add "BREAKING CHANGE" to a commit message unless there is actually a breaking change.
|
|
@@ -1,13 +1,19 @@
|
|
|
1
1
|
# Security Audit Handling
|
|
2
2
|
|
|
3
|
-
If `git push` fails because the pre-push hook reports security vulnerabilities, follow these steps. Never use `--no-verify`
|
|
3
|
+
If `git push` fails because the pre-push hook reports security vulnerabilities, follow these steps. Never use `--no-verify`, `HUSKY=0`, `core.hooksPath`, or any other hook bypass to skip the security audit.
|
|
4
|
+
|
|
5
|
+
## Fix before ignore
|
|
6
|
+
|
|
7
|
+
1. Fix the root cause first: upgrade or override the actually-vulnerable leaf package to a patched compatible version, regenerate the lockfile, and retry the gate.
|
|
8
|
+
2. Only if no safe fix exists, ask the user to make the risk-acceptance decision. Add a narrow documented ignore for the specific advisory, package, and reason.
|
|
9
|
+
3. Never add a blanket audit bypass, lower an audit level, or self-approve a new risk-acceptance entry.
|
|
4
10
|
|
|
5
11
|
## Node.js Projects (GHSA advisories)
|
|
6
12
|
|
|
7
13
|
1. Note the GHSA ID(s), affected package(s), and advisory URL from the error output
|
|
8
14
|
2. Check the advisory URL to determine if a patched version of the vulnerable package exists
|
|
9
15
|
3. If a patched version exists: add a resolution/override in package.json to force the patched version (add to both `resolutions` and `overrides` sections), then run the package manager install command to regenerate the lockfile, commit the changes, and retry the push
|
|
10
|
-
4. If no patched version exists and the vulnerability is safe for this project (e.g., transitive dependency with no untrusted input, devDeps only, or build tool only): add an exclusion entry to `audit.ignore.local.json` with the format `{"id": "GHSA-xxx", "package": "pkg-name", "reason": "why this is safe for this project"}`, then commit and retry the push
|
|
16
|
+
4. If no patched version exists and the vulnerability is safe for this project (e.g., transitive dependency with no untrusted input, devDeps only, or build tool only): ask the user to make the risk-acceptance decision, then add an exclusion entry to `audit.ignore.local.json` with the format `{"id": "GHSA-xxx", "package": "pkg-name", "reason": "why this is safe for this project"}`, then commit and retry the push
|
|
11
17
|
|
|
12
18
|
### Critical: Override the vulnerable package, not its parent
|
|
13
19
|
|