@geonosis/oxlint-plugin-biological-architecture 1.2.0 → 1.3.0

package/corpus/.oxlintrc.json

@@ -150,7 +150,10 @@
         "roots": [
           "integrations/providers"
         ],
-        "registry": "integrations/registry.ts",
+        "registry": [
+          "integrations/registry.ts",
+          "integrations/registry.fulfillment.ts"
+        ],
         "manifests": [
           "index.ts"
         ]

package/corpus/emits/member-call-budget.ts

@@ -0,0 +1,8 @@
+// A budget built by a call the rule cannot name: reported, never a silent pass.
+declare const emitEventStep: (input: unknown) => unknown
+declare const retry: { background: () => unknown }
+
+export const emitted = emitEventStep({
+  eventName: 'sample.converted',
+  options: retry.background(),
+})

package/corpus/integrations/providers/warp/index.ts

@@ -0,0 +1 @@
+export const WarpProvider = { id: 'cp_warp' }

package/corpus/integrations/registry.fulfillment.ts

@@ -0,0 +1,4 @@
+// The other half of a SPLIT registry: the file beside it imports this one, and `warp` is
+// registered only here. Reading the named half alone reported two registered carrier directories
+// as dead weight on a real consumer, which is how a rule gets turned off.
+export const carriers = ['providers/warp']

package/corpus/integrations/registry.ts

@@ -1,9 +1,12 @@
 import { AttioProvider } from './providers/attio/index'
+import { carriers } from './registry.fulfillment'
 
-// Two of the three directories beside this file are named here. The third is `dead-cal`, which
-// ships a manifest, is imported by nothing, and reads as an integration to anyone counting
-// directories — Midday carries eight of these.
+// This registry is SPLIT: two of the four directories beside it are named here, `warp` is named in
+// the half this file imports, and `dead-cal` is named in neither. `dead-cal` ships a manifest, is
+// imported by nothing, and reads as an integration to anyone counting directories — Midday carries
+// eight of these.
 export const providers = {
+  carriers,
   cp_attio: AttioProvider,
   cp_hubspot: 'providers/hubspot',
 }

package/dist/index.d.ts

@@ -21,7 +21,20 @@ type Probe = {
     path: string;
     source: string;
 };
-type ProbeFn = (options: readonly unknown[]) => Probe;
+/**
+ * A probe that needs a TREE rather than a file.
+ *
+ * A rule that reads a sibling off disk — a registry, a schema's twin — cannot be exercised by one
+ * file however right its source is: `no-unregistered-plugin-dir` returns silently when the registry
+ * its options name is not there, so the single-file probe fired nothing and every consumer
+ * enforcing the rule over a real registry read as a permanent FAIL. The answer is not a widened
+ * probe, which would be the rule claiming evidence it does not have; it is a probe that may declare
+ * the sibling too, and a reader that mounts all of them.
+ */
+type ProbeTree = {
+    files: readonly Probe[];
+};
+type ProbeFn = (options: readonly unknown[]) => Probe | ProbeTree;
 
 type Rule = {
     create: (context: never) => Record<string, unknown>;

package/dist/index.js

@@ -593,10 +593,10 @@ var dialect_through_the_seam_default = dialectThroughTheSeam;
 // src/rules/lib/options.ts
 var NOTHING = "nothing";
 var isMissing = (value) => value === void 0 || value === null || value === "" || Array.isArray(value) && value.length === 0;
-var requireOption = (rule, value, option) => {
+var requireOption = (rule, value, option, accepts) => {
   if (isMissing(value)) {
     throw new Error(
-      `biological-architecture/${rule} is enabled but its \`${option}\` option is ${NOTHING}, so it can never fire. Configure \`${option}\`, or turn the rule off \u2014 an enabled rule that cannot fire reads exactly like a rule that found nothing.`
+      `biological-architecture/${rule} is enabled but its \`${option}\` option is ${NOTHING}, so it can never fire. Configure \`${option}\`, or turn the rule off \u2014 an enabled rule that cannot fire reads exactly like a rule that found nothing.${accepts === void 0 ? "" : ` It accepts ${accepts}.`}`
     );
   }
   return value;
@@ -854,9 +854,19 @@ var emitDeclaresAttempts = {
         const site = named?.type === "Literal" && typeof named.value === "string" ? named.value : emitter;
         const options = valueOf(properties, OPTIONS);
         if (options !== void 0 && objectOf(options) === null) {
-          if (budgetHelpers.size === 0) return;
+          if (budgetHelpers.size === 0) {
+            context.report({ data: { emitter, site }, messageId: "undeclaredHelpers", node });
+            return;
+          }
           const call = options;
-          if (call.type !== "CallExpression" || call.callee?.type !== "Identifier") return;
+          if (call.type !== "CallExpression" || call.callee?.type !== "Identifier") {
+            context.report({
+              data: { emitter, helper: "a call this rule cannot name", site },
+              messageId: "opaqueBudget",
+              node
+            });
+            return;
+          }
           if (budgetHelpers.has(call.callee.name ?? "")) return;
           context.report({
             data: { emitter, helper: call.callee.name ?? "a call", site },
@@ -886,8 +896,9 @@ var emitDeclaresAttempts = {
 than written as a number at the call site. \`attempts\` defaults to 1 and the worker reads only
 \`attempts > 1\` as "a retry was configured", so an emit with \`{ priority }\` and no \`attempts\` is
 droppable while looking deliberate. Pass \`options: { attempts: RETRY_BACKGROUND.attempts, priority }\`.
-A budget the rule cannot evaluate \u2014 a constant, a computed value \u2014 passes: it says nothing rather
-than pushing the number back inline. Name the repo's own budget builders in \`budgetHelpers\`
+A literal budget the rule cannot evaluate \u2014 a shared constant's property \u2014 passes; an options bag
+built by a CALL never passes silently: with no \`budgetHelpers\` declared it is reported as such, and
+with them declared any call that is not one of them is reported. Name the repo's own budget builders in \`budgetHelpers\`
 (dielime's is \`retryableEvent\`, which throws on \`attempts <= 1\`) and a call to one of them counts as
 a budget while a call to anything else is reported: without that, a repo whose every call site goes
 through a helper gets zero findings and no coverage. Configure \`emitters\` if the repo wraps the emit
@@ -898,6 +909,7 @@ under another name.`,
     },
     messages: {
       noBudget: '`{{emitter}}` emits "{{site}}" with no `options.attempts`. `attempts` defaults to 1 and the worker treats only `attempts > 1` as a configured retry, so this event is dropped on its first failure. Give it a budget from the shared retry constants.',
+      undeclaredHelpers: '`{{emitter}}` emits "{{site}}" with a budget built by a call, and this repo declares no `budgetHelpers` \u2014 nothing here can tell whether that call configures a retry. Name the repo\'s budget builders in `budgetHelpers`, or pass `attempts` where it can be read.',
       opaqueBudget: '`{{emitter}}` emits "{{site}}" with a budget built by `{{helper}}`, which this repo has not named as a budget helper. The bus reads only `attempts > 1` as a configured retry, and nothing here can tell whether that call produces one. Use one of the declared `budgetHelpers`, or pass `attempts` where it can be read.',
       tooFew: '`{{emitter}}` emits "{{site}}" with `attempts: {{attempts}}`, which is the default spelled out \u2014 one attempt is no retry. Take the budget from the shared retry constants instead.'
     },
@@ -3409,6 +3421,7 @@ var no_type_definitions_in_components_default = noTypeDefinitionsInComponents;
 // src/rules/no-unregistered-plugin-dir.ts
 import { readFileSync as readFileSync3 } from "fs";
 var RULE8 = "no-unregistered-plugin-dir";
+var ACCEPTS = "{ roots: string[], registry: string | string[], manifests: string[] }";
 var trimSlashes = (path2) => path2.replace(/^\/+/, "").replace(/\/+$/, "");
 var withoutComments = (source) => source.replaceAll(/\/\*[\s\S]*?\*\//g, " ").replaceAll(/\/\/[^\n]*/g, " ");
 var literalsOf = (source) => [...withoutComments(source).matchAll(/['"`]([^'"`\n]+)['"`]/g)].map((match) => match[1] ?? "");
@@ -3425,9 +3438,10 @@ var rootedAt = (filename, root) => {
 var noUnregisteredPluginDir = {
   create(context) {
     const options = context.options?.[0] ?? {};
-    const roots = requireOption(RULE8, options.roots, "roots").map(trimSlashes);
-    const registry = requireOption(RULE8, options.registry, "registry");
-    const manifests = new Set(requireOption(RULE8, options.manifests, "manifests"));
+    const roots = requireOption(RULE8, options.roots, "roots", ACCEPTS).map(trimSlashes);
+    const declared = requireOption(RULE8, options.registry, "registry", ACCEPTS);
+    const registries = (typeof declared === "string" ? [declared] : declared).map(trimSlashes);
+    const manifests = new Set(requireOption(RULE8, options.manifests, "manifests", ACCEPTS));
     const filename = normalise(context.filename);
     return {
       Program(node) {
@@ -3440,11 +3454,12 @@ var noUnregisteredPluginDir = {
           if (directory === void 0 || file === void 0 || !manifests.has(file)) continue;
           let source = "";
           try {
-            source = readFileSync3(`${rooted.repoRoot}/${trimSlashes(registry)}`, "utf8");
+            source = registries.map((half) => readFileSync3(`${rooted.repoRoot}/${half}`, "utf8")).join("\n");
           } catch {
             return;
           }
           if (literalsOf(source).some((literal) => names(literal, directory))) return;
+          const registry = registries.join(", ");
           context.report({ data: { directory, registry, root }, messageId: "unregistered", node });
           return;
         }
@@ -3454,13 +3469,14 @@ var noUnregisteredPluginDir = {
   fixShape: `Every directory under a plugin root is named by the registry. Add the directory to the registry file \u2014
 as an import path or as an id \u2014 or delete it. A directory that ships a manifest and assets and is
 reached by nothing reads as an integration to everyone who counts directories, and Midday carried
-eight of them. Configure \`roots\`, \`registry\` and \`manifests\` \u2014 all three are required, and \`manifests\` has no
-default on purpose: dielime's plugin directories hold \`service.ts\` and no index, so a defaulted list
+eight of them. Configure \`roots\`, \`registry\` and \`manifests\` \u2014 all three are required. \`registry\` takes a list when the
+registry is SPLIT across files (a config that imports a second config): name every half, because the
+rule reads source and does not follow imports. \`manifests\` has no default on purpose: dielime's plugin directories hold \`service.ts\` and no index, so a defaulted list
 would match nothing and report a clean tree it never looked at. A seam whose providers are flat files
 rather than directories is out of scope \u2014 nothing under a root matches, so nothing is reported.`,
   meta: {
     docs: {
-      description: "Every directory under a plugin root must be named by the registry file. Midday ships 35 app-store directories and 8 are reachable from nothing. The registry is read as source, so an id or an import path both count. `roots`, `registry` and `manifests` are all required; `manifests` has no default because a defaulted entry filename matched nothing in a real consumer and reported a clean tree it had never looked at."
+      description: "Every directory under a plugin root must be named by the registry file. Midday ships 35 app-store directories and 8 are reachable from nothing. The registry is read as source, so an id or an import path both count, and `registry` takes a LIST when the registry is split across files \u2014 reading only the named half reported two registered carrier directories as dead weight on a real consumer. `roots`, `registry` and `manifests` are all required; `manifests` has no default because a defaulted entry filename matched nothing in a real consumer and reported a clean tree it had never looked at."
     },
     messages: {
       unregistered: 'The plugin directory "{{directory}}" under `{{root}}` is named nowhere in `{{registry}}`. Nothing reaches it, so it ships as an integration and behaves as dead weight \u2014 register it, or delete it.'
@@ -3470,7 +3486,12 @@ rather than directories is out of scope \u2014 nothing under a root matches, so
         additionalProperties: false,
         properties: {
           manifests: { items: { type: "string" }, type: "array" },
-          registry: { type: "string" },
+          registry: {
+            anyOf: [
+              { type: "string" },
+              { items: { type: "string" }, type: "array" }
+            ]
+          },
           roots: { items: { type: "string" }, type: "array" }
         },
         type: "object"
@@ -3480,21 +3501,28 @@ rather than directories is out of scope \u2014 nothing under a root matches, so
   },
   /**
    * A directory under a configured root, carrying a manifest, that the registry does not name — and
-   * the one probe in this plugin a single file cannot make fire on its own.
+   * the registry itself, because the rule reads it off disk and returns silently when it is not
+   * there. One file cannot be both the directory under `roots` and the registry at the consumer's
+   * own path, so the probe declares two. It was a single file until 2026-08-30, and every consumer
+   * enforcing this rule over a real registry read as a permanent FAIL for it: the probe fired
+   * nothing, which was the truth about the probe and a lie about the repo.
    *
-   * The rule reads the registry off disk, and a registry it cannot read is a configuration answer
-   * rather than a verdict about the tree, so it returns silently. The probe sits under `roots` and
-   * the registry sits at the consumer's own path; one file cannot be both. It fires when the
-   * registry ALSO exists beside it and fires nothing when it does not, which is the truth and is
-   * left as the truth: a probe widened until it fired would be this rule claiming evidence it does
-   * not have, which is precisely the defect the rule is about.
+   * The registry it writes carries no string literal at all, so nothing in it can name the
+   * directory beside it — the probe must be a directory the registry does NOT reach, or the rule is
+   * right to stay quiet.
    */
   probe(options) {
-    const { manifests = [], roots = [] } = bagOf(options);
+    const { manifests = [], registry = "", roots = [] } = bagOf(options);
     const root = trimSlashes(roots[0] ?? "");
+    const half = typeof registry === "string" ? registry : registry[0] ?? "";
     return {
-      path: `${root}/geonosis-probe/${manifests[0] ?? ""}`,
-      source: "export const geonosisProbe = { id: 'geonosis-probe' }\n"
+      files: [
+        {
+          path: `${root}/geonosis-probe/${manifests[0] ?? ""}`,
+          source: "export const geonosisProbe = { id: 'geonosis-probe' }\n"
+        },
+        { path: trimSlashes(half), source: "export const registry = { providers: [] }\n" }
+      ]
     };
   }
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@geonosis/oxlint-plugin-biological-architecture",
-  "version": "1.2.0",
+  "version": "1.3.0",
   "types": "./dist/index.d.ts",
   "description": "Biological tier architecture as lint — the union of the dielime and during.day rule sets, shipped as presets.",
   "keywords": [
@@ -43,7 +43,7 @@
     "oxlint": ">=1.77"
   },
   "devDependencies": {
-    "@geonosis/lint-parity": "1.2.0"
+    "@geonosis/lint-parity": "1.3.0"
   },
   "engines": {
     "node": ">=22"