slopbrick 0.41.0 → 0.42.0

package/dist/engine/worker.cjs

@@ -41124,23 +41124,16 @@ var builtinRules = [
 // src/rules/registry.ts
 var RuleRegistry = class {
   rules = /* @__PURE__ */ new Map();
-  register(ruleOrId, factory) {
-    if (typeof ruleOrId === "string") {
-      if (!factory) {
-        throw new Error("Factory is required when registering by id");
-      }
-      const rule = {
-        id: ruleOrId,
-        category: factory.category,
-        severity: factory.severity,
-        aiSpecific: factory.aiSpecific,
-        create: factory.create,
-        analyze: factory.analyze
-      };
-      this.rules.set(ruleOrId, rule);
-    } else {
-      this.rules.set(ruleOrId.id, ruleOrId);
-    }
+  /**
+   * Register a rule. The 2-arg overload `register(id, factory)` was
+   * deleted in v0.42.0 (architecture review F6): no callers used it,
+   * every `Rule` carries its own `id`, and the per-id registration
+   * was ergonomically redundant with the 1-arg form (`register(rule)`).
+   * External consumers that needed `register-by-id` semantics can
+   * construct a `Rule` directly and pass it in.
+   */
+  register(rule) {
+    this.rules.set(rule.id, rule);
   }
   loadBuiltins(onlyRuleId) {
     for (const rule of builtinRules) {
@@ -41169,6 +41162,15 @@ var RuleRegistry = class {
   all() {
     return Array.from(this.rules.values());
   }
+  /** v0.42.0 (§3b.5): look up a registered rule by id.
+   *  Returns undefined if not registered. */
+  get(id) {
+    return this.rules.get(id);
+  }
+  /** v0.42.0 (§3b.5): check whether a rule id is registered. */
+  has(id) {
+    return this.rules.has(id);
+  }
   createContexts(config, filePath, cwd, hotspotIssues = []) {
     const context = {
       config,
@@ -44863,6 +44865,36 @@ async function scanFile(filePath, config, registry, cwd = process.cwd()) {
     const issues = applyRuleOverrides(rawIssues, config.rules);
     const triggeredRuleIds = Array.from(new Set(issues.map((i) => i.ruleId)));
     const compScore = compositeScore(triggeredRuleIds, loadSignalStrength());
+    const fireSet = new Set(triggeredRuleIds);
+    const factsWithFireSet = Object.assign(facts, {
+      compositeFireSet: fireSet
+    });
+    for (const { rule } of rules) {
+      if (!rule.id.startsWith("composite/")) continue;
+      try {
+        const compositeIssues = rule.analyze(
+          rule.create({
+            config,
+            filePath,
+            cwd,
+            framework: config.framework,
+            uiLibraries: config.uiLibraries,
+            hasTailwind: config.hasTailwind,
+            supportsRsc: config.supportsRsc,
+            hotspotIssues: []
+          }),
+          factsWithFireSet
+        );
+        if (compositeIssues.length > 0) {
+          issues.push(...compositeIssues);
+          for (const ci of compositeIssues) fireSet.add(ci.ruleId);
+        }
+      } catch (err) {
+        if (process.env.SLOP_AUDIT_DEBUG === "1") {
+          console.error(`[worker] composite ${rule.id} threw: ${err.message}`);
+        }
+      }
+    }
     const gapValues = collectGapValues(facts);
     const styleSources = collectStyleSources(facts);
     const elementTags = facts.v2.jsx.elements.map((e) => e.tag);

package/dist/engine/worker.js

@@ -41095,23 +41095,16 @@ var builtinRules = [
 // src/rules/registry.ts
 var RuleRegistry = class {
   rules = /* @__PURE__ */ new Map();
-  register(ruleOrId, factory) {
-    if (typeof ruleOrId === "string") {
-      if (!factory) {
-        throw new Error("Factory is required when registering by id");
-      }
-      const rule = {
-        id: ruleOrId,
-        category: factory.category,
-        severity: factory.severity,
-        aiSpecific: factory.aiSpecific,
-        create: factory.create,
-        analyze: factory.analyze
-      };
-      this.rules.set(ruleOrId, rule);
-    } else {
-      this.rules.set(ruleOrId.id, ruleOrId);
-    }
+  /**
+   * Register a rule. The 2-arg overload `register(id, factory)` was
+   * deleted in v0.42.0 (architecture review F6): no callers used it,
+   * every `Rule` carries its own `id`, and the per-id registration
+   * was ergonomically redundant with the 1-arg form (`register(rule)`).
+   * External consumers that needed `register-by-id` semantics can
+   * construct a `Rule` directly and pass it in.
+   */
+  register(rule) {
+    this.rules.set(rule.id, rule);
   }
   loadBuiltins(onlyRuleId) {
     for (const rule of builtinRules) {
@@ -41140,6 +41133,15 @@ var RuleRegistry = class {
   all() {
     return Array.from(this.rules.values());
   }
+  /** v0.42.0 (§3b.5): look up a registered rule by id.
+   *  Returns undefined if not registered. */
+  get(id) {
+    return this.rules.get(id);
+  }
+  /** v0.42.0 (§3b.5): check whether a rule id is registered. */
+  has(id) {
+    return this.rules.has(id);
+  }
   createContexts(config, filePath, cwd, hotspotIssues = []) {
     const context = {
       config,
@@ -44834,6 +44836,36 @@ async function scanFile(filePath, config, registry, cwd = process.cwd()) {
     const issues = applyRuleOverrides(rawIssues, config.rules);
     const triggeredRuleIds = Array.from(new Set(issues.map((i) => i.ruleId)));
     const compScore = compositeScore(triggeredRuleIds, loadSignalStrength());
+    const fireSet = new Set(triggeredRuleIds);
+    const factsWithFireSet = Object.assign(facts, {
+      compositeFireSet: fireSet
+    });
+    for (const { rule } of rules) {
+      if (!rule.id.startsWith("composite/")) continue;
+      try {
+        const compositeIssues = rule.analyze(
+          rule.create({
+            config,
+            filePath,
+            cwd,
+            framework: config.framework,
+            uiLibraries: config.uiLibraries,
+            hasTailwind: config.hasTailwind,
+            supportsRsc: config.supportsRsc,
+            hotspotIssues: []
+          }),
+          factsWithFireSet
+        );
+        if (compositeIssues.length > 0) {
+          issues.push(...compositeIssues);
+          for (const ci of compositeIssues) fireSet.add(ci.ruleId);
+        }
+      } catch (err) {
+        if (process.env.SLOP_AUDIT_DEBUG === "1") {
+          console.error(`[worker] composite ${rule.id} threw: ${err.message}`);
+        }
+      }
+    }
     const gapValues = collectGapValues(facts);
     const styleSources = collectStyleSources(facts);
     const elementTags = facts.v2.jsx.elements.map((e) => e.tag);