@shrkcrft/quality-gates 0.1.0-alpha.24 → 0.1.0-alpha.25

package/dist/gates/impact-gate.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"impact-gate.d.ts","sourceRoot":"","sources":["../../src/gates/impact-gate.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,2BAA2B,CAAC;AAE7D,MAAM,WAAW,kBAAkB;IACjC,sEAAsE;IACtE,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,4DAA4D;IAC5D,MAAM,CAAC,EAAE,SAAS,CAAC,MAAM,GAAG,UAAU,CAAC,EAAE,CAAC;IAC1C;;;;OAIG;IACH,KAAK,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;CAC3B;AAcD;;;;;;;;;;;GAWG;AACH,wBAAgB,UAAU,CAAC,WAAW,EAAE,MAAM,EAAE,OAAO,GAAE,kBAAuB,GAAG,WAAW,CAyD7F"}
+{"version":3,"file":"impact-gate.d.ts","sourceRoot":"","sources":["../../src/gates/impact-gate.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,2BAA2B,CAAC;AAE7D,MAAM,WAAW,kBAAkB;IACjC,sEAAsE;IACtE,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,4DAA4D;IAC5D,MAAM,CAAC,EAAE,SAAS,CAAC,MAAM,GAAG,UAAU,CAAC,EAAE,CAAC;IAC1C;;;;OAIG;IACH,KAAK,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;CAC3B;AAcD;;;;;;;;;;;GAWG;AACH,wBAAgB,UAAU,CAAC,WAAW,EAAE,MAAM,EAAE,OAAO,GAAE,kBAAuB,GAAG,WAAW,CAiE7F"}

package/dist/gates/impact-gate.js

@@ -23,8 +23,14 @@ function rankOf(risk) {
  */
 export function impactGate(projectRoot, options = {}) {
     const start = Date.now();
-    const failOn = options.failOn && options.failOn.length > 0 ? options.failOn : ['critical'];
-    const failThreshold = Math.min(...failOn.map((f) => rankOf(f)));
+    // `undefined` failOn → the standalone default (`['critical']` fails). An
+    // EXPLICIT empty array → ADVISORY mode: blast-radius risk is inherently the
+    // pre-existing structure (touching a hub is risky but not a NEW failure this
+    // change introduced), so it warns rather than reds. The composite `gate`
+    // passes `[]` by default so its verdict clears for a clean inline change.
+    const failOn = options.failOn === undefined ? ['critical'] : options.failOn;
+    const advisoryOnly = failOn.length === 0;
+    const failThreshold = advisoryOnly ? Number.POSITIVE_INFINITY : Math.min(...failOn.map((f) => rankOf(f)));
     // Scope: an explicit changed-file set (when no gitref is given) is analyzed
     // directly; otherwise diff against the gitref (default 'main').
     const useFiles = !options.sinceRef && options.files !== undefined;
@@ -52,11 +58,14 @@ export function impactGate(projectRoot, options = {}) {
             durationMs: Date.now() - start,
         };
     }
-    const status = rankOf(analysis.risk) >= failThreshold ? 'fail' : analysis.risk === 'high' ? 'warn' : 'pass';
+    const rank = rankOf(analysis.risk);
+    const status = rank >= failThreshold ? 'fail' : rank >= (RISK_RANK['high'] ?? 2) ? 'warn' : 'pass';
     const message = status === 'fail'
         ? `Risk: ${analysis.risk}. ${analysis.directDependents.length} direct dependents.`
         : status === 'warn'
-            ? `Risk: ${analysis.risk}. Review the validation scope.`
+            ? advisoryOnly
+                ? `Risk: ${analysis.risk} (advisory — pre-existing blast radius, ${analysis.directDependents.length} direct dependents). Validate broadly; \`--fail-on ${analysis.risk === 'critical' ? 'critical' : 'high'}\` to block.`
+                : `Risk: ${analysis.risk}. Review the validation scope.`
             : `Risk: ${analysis.risk}. Looks safe.`;
     return {
         id: 'impact',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@shrkcrft/quality-gates",
-  "version": "0.1.0-alpha.24",
+  "version": "0.1.0-alpha.25",
   "description": "SharkCraft quality gates: single aggregator that runs the code-intelligence checks (graph index, boundaries, architecture, impact summary) and emits one pass/fail report. The pre-merge gate for AI-agent-authored changes.",
   "license": "MIT",
   "author": "SharkCraft contributors",
@@ -43,15 +43,15 @@
     "typecheck": "tsc --noEmit -p tsconfig.json"
   },
   "dependencies": {
-    "@shrkcrft/core": "^0.1.0-alpha.24",
-    "@shrkcrft/graph": "^0.1.0-alpha.24",
-    "@shrkcrft/architecture-guard": "^0.1.0-alpha.24",
-    "@shrkcrft/impact-engine": "^0.1.0-alpha.24",
-    "@shrkcrft/api-surface-diff": "^0.1.0-alpha.24",
-    "@shrkcrft/boundaries": "^0.1.0-alpha.24",
-    "@shrkcrft/structural-search": "^0.1.0-alpha.24",
-    "@shrkcrft/context-planner": "^0.1.0-alpha.24",
-    "@shrkcrft/inspector": "^0.1.0-alpha.24"
+    "@shrkcrft/core": "^0.1.0-alpha.25",
+    "@shrkcrft/graph": "^0.1.0-alpha.25",
+    "@shrkcrft/architecture-guard": "^0.1.0-alpha.25",
+    "@shrkcrft/impact-engine": "^0.1.0-alpha.25",
+    "@shrkcrft/api-surface-diff": "^0.1.0-alpha.25",
+    "@shrkcrft/boundaries": "^0.1.0-alpha.25",
+    "@shrkcrft/structural-search": "^0.1.0-alpha.25",
+    "@shrkcrft/context-planner": "^0.1.0-alpha.25",
+    "@shrkcrft/inspector": "^0.1.0-alpha.25"
   },
   "publishConfig": {
     "access": "public"