@geonosis/doctor 2.6.1 → 2.8.0

package/CHANGELOG.md

@@ -1,5 +1,35 @@
 # @geonosis/doctor
 
+## 2.8.0
+
+### Patch Changes
+
+- @geonosis/lint-parity@2.8.0
+
+## 2.7.0
+
+### Minor Changes
+
+- d90986b: A twelfth question: `seams`. Every glob `adoption.seams` declares has to name a file in the tree
+  (never one under node_modules); one that names nothing is a WARN per glob saying to drop it or
+  name the file that replaced it, no `adoption.seams` at all is a SKIP by name, and a full set is
+  one OK line with the count. `geonosis update` measures a floor's adoption by exactly those files,
+  and during.day carried three dead globs of eight for a month while every bump report read
+  `0 deleted, 0 added` over them. `--only seams` runs it alone.
+
+### Patch Changes
+
+- 7ee2097: `FLOOR_PACKAGES` names `@geonosis/buketi-ui`, the first package on the `kits/*` floor, so the doctor's
+  floor lines ask about it the way they ask about every other publishable floor.
+- c01ea60: The `exercised` check knows a retired rule. A plugin may keep a rule's id resolvable after retiring
+  it — `meta.retired` — so a config that still names it parses; judged as a live rule it read as "not
+  exported" or "never fires", a FAIL against a tree whose only fault is a block the kit made stale.
+  The id is now set aside before the judging, and every verdict names it with the way out: it reports
+  nothing, goes at the next major, remove it from this config. A verdict that would have been OK or
+  SKIP is a WARN while any such id is still named.
+- Updated dependencies [c13ade8]
+  - @geonosis/lint-parity@2.7.0
+
 ## 2.6.1
 
 ### Patch Changes

package/README.md

@@ -387,3 +387,16 @@ consumer wrote. It depends on `@geonosis/lint-parity` for the reach engine and o
 the kit's own `layer-walls` config holds every one of those walls.
 
 Apache-2.0.
+
+### `seams` — a declared seam that names nothing
+
+`adoption.seams` in `geonosis.json` is what `geonosis update` measures a floor's adoption by:
+lines deleted against lines added in the files a consumer says hold a floor's shape. A glob whose
+file was deleted and never undeclared measures nothing, and every bump report reads `0 deleted,
+0 added` over it — during.day carried three of eight that way for a month.
+
+| Seen                                            | Line                                                |
+| ----------------------------------------------- | --------------------------------------------------- |
+| no `adoption.seams`                             | **SKIP**, naming the key                            |
+| every glob names a file (never in node_modules) | **OK**, with the count                              |
+| a glob names no file                            | **WARN** per glob, saying to drop or replace it      |

package/bin/geonosis-doctor.mjs

@@ -34,13 +34,13 @@ if (existsSync(src)) {
   const built = newest(dist)
   if (built === 0) {
     process.stderr.write(
-      'geonosis-doctor: this tree has never been built — run pnpm build. No dist here at all, which is what a fresh clone or a linked git worktree starts with; nothing is stale.\n',
+      'geonosis-doctor: this tree has never been built — run pnpm build. No dist here at all, which is what a fresh clone or a linked git worktree starts with; nothing is stale.\n'
     )
     process.exit(2)
   }
   if (newest(src, NOT_SOURCE) > built) {
     process.stderr.write(
-      'geonosis-doctor: dist is older than src — run pnpm build before trusting this bin.\n',
+      'geonosis-doctor: dist is older than src — run pnpm build before trusting this bin.\n'
     )
     process.exit(2)
   }

package/dist/{chunk-6ERJOIZS.js → chunk-2ZVCJG5R.js}

@@ -113,7 +113,8 @@ var CHECKS = [
   "observability",
   "deployed",
   "rails",
-  "exams"
+  "exams",
+  "seams"
 ];
 var findingMaker = (check) => (subject, verdict, message) => ({
   check,
@@ -461,6 +462,12 @@ var probesOf = async (entry, plugin) => {
     })
   );
 };
+var retiredOf = async (entry, plugin) => {
+  const loaded = await import(pathToFileURL(entry).href);
+  return new Set(
+    Object.entries(loaded.default?.rules ?? {}).filter(([, rule]) => typeof rule?.meta?.retired === "object" && rule.meta.retired !== null).map(([name]) => `${plugin}/${name}`)
+  );
+};
 var grantsOf = async (entry) => {
   const loaded = await import(pathToFileURL(entry).href);
   const plugin = loaded.default?.meta?.name ?? "";
@@ -950,11 +957,7 @@ var linkedBins = (root, workspaces) => {
   );
   if (missing.length === 0) {
     return [
-      finding4(
-        "workspace bins",
-        "OK",
-        "every workspace bin a script names is linked under the root"
-      )
+      finding4("workspace bins", "OK", "every workspace bin a script names is linked under the root")
     ];
   }
   return missing.map(
@@ -1245,6 +1248,7 @@ var OBSERVABILITY = "@geonosis/observability";
 var COMPOSITION_ROOT = "src/platform.ts";
 var FLOOR_PACKAGES = [
   "@geonosis/assistant",
+  "@geonosis/buketi-ui",
   "@geonosis/conformance",
   "@geonosis/db",
   "@geonosis/document",
@@ -1629,6 +1633,7 @@ var reasonFrom = (config, oxlint) => {
   }
 };
 var named = (rules) => rules.join(", ");
+var retiredNote = (rules) => rules.length === 1 ? `${rules[0]} is retired by the plugin \u2014 it reports nothing and goes at the next major, so remove it from this config` : `${named(rules)} are retired by the plugin \u2014 they report nothing and go at the next major, so remove them from this config`;
 var countOf = (rules, verb) => rules.length === 1 ? `1 enabled rule ${verb}s` : `${rules.length} enabled rules ${verb}`;
 var ownReach = ({
   config,
@@ -1790,7 +1795,12 @@ var checkExercised = async ({
   repoCorpus,
   root
 }) => {
-  const said2 = (verdict, message) => finding5(config.relative, verdict, message);
+  let retired = [];
+  const said2 = (verdict, message) => retired.length === 0 ? finding5(config.relative, verdict, message) : finding5(
+    config.relative,
+    verdict === "OK" || verdict === "SKIP" ? "WARN" : verdict,
+    `${message}; ${retiredNote(retired)}`
+  );
   if (repoCorpus !== void 0 && !existsSync7(repoCorpus)) {
     return said2(
       "FAIL",
@@ -1809,8 +1819,13 @@ var checkExercised = async ({
   } catch (error) {
     return said2("SKIP", refusal(error));
   }
-  const enabled = enabledHere(config, manifest.plugin);
-  if (enabled.length === 0) return said2("SKIP", `no ${manifest.plugin} rule is enabled here`);
+  const configured2 = enabledHere(config, manifest.plugin);
+  const retiredIds = entry === void 0 ? /* @__PURE__ */ new Set() : await retiredOf(entry, manifest.plugin).catch(() => /* @__PURE__ */ new Set());
+  retired = configured2.filter((rule) => retiredIds.has(rule));
+  const enabled = configured2.filter((rule) => !retiredIds.has(rule));
+  if (enabled.length === 0) {
+    return retired.length === 0 ? said2("SKIP", `no ${manifest.plugin} rule is enabled here`) : said2("WARN", `nothing enabled here is a live ${manifest.plugin} rule`);
+  }
   let reach;
   try {
     reach = corpusOf({ configA: config.path, configB: config.path, corpus, oxlint }).reach;
@@ -2040,11 +2055,7 @@ var checkExams = ({
           `${name} \u2192 ${exam}`,
           "WARN",
           `this tree declares ${name} and no test file imports ${exam}. A floor bump that broke it would read exactly like a green tier: the exam is the only thing that asks.`
-        ) : finding7(
-          `${name} \u2192 ${exam}`,
-          "OK",
-          `${relativePath(root, importing.path)} runs ${exam}`
-        )
+        ) : finding7(`${name} \u2192 ${exam}`, "OK", `${relativePath(root, importing.path)} runs ${exam}`)
       );
     }
   }
@@ -2679,7 +2690,7 @@ var checkRunner = ({
 
 // src/doctor.ts
 import { homedir as homedir3 } from "os";
-import { join as join17 } from "path";
+import { join as join18 } from "path";
 import { resolveOxlint } from "@geonosis/lint-parity";
 
 // src/engine.ts
@@ -3000,6 +3011,46 @@ var checkRequiredOptions = async ({
   ];
 };
 
+// src/seams.ts
+import { existsSync as existsSync13, globSync, readFileSync as readFileSync16 } from "fs";
+import { join as join17 } from "path";
+var CONFIG = "geonosis.json";
+var finding12 = findingMaker("seams");
+var NO_SEAMS = `${CONFIG} declares no adoption.seams \u2014 nothing here says which files hold a floor's shape, so no bump can measure what it deleted; declare them when a floor is adopted`;
+var declaredSeams = (root) => {
+  const at = join17(root, CONFIG);
+  if (!existsSync13(at)) return void 0;
+  let parsed;
+  try {
+    parsed = JSON.parse(readFileSync16(at, "utf8"));
+  } catch {
+    return void 0;
+  }
+  const seams = parsed.adoption?.seams;
+  return Array.isArray(seams) ? seams.filter((one) => typeof one === "string") : void 0;
+};
+var namesAFile = (root, glob) => globSync(glob, {
+  cwd: root,
+  exclude: (name) => name === "node_modules" || name.startsWith("node_modules/")
+}).length > 0;
+var checkSeams = ({ root }) => {
+  const seams = declaredSeams(root);
+  if (seams === void 0 || seams.length === 0) return [finding12(CONFIG, "SKIP", NO_SEAMS)];
+  const dead = seams.filter((glob) => !namesAFile(root, glob)).toSorted();
+  if (dead.length === 0) {
+    return [
+      finding12(CONFIG, "OK", `adoption.seams: ${seams.length} seam glob(s), every one names a file`)
+    ];
+  }
+  return dead.map(
+    (glob) => finding12(
+      glob,
+      "WARN",
+      `names no file in this tree, so the seam measures nothing and every bump reads 0 deleted, 0 added over it. Next: take it out of adoption.seams in ${CONFIG}, or name the file that replaced it`
+    )
+  );
+};
+
 // src/doctor.ts
 var exercisedOf = ({
   configs,
@@ -3135,8 +3186,9 @@ var runDoctor = async ({
     ["observability", () => checkObservability({ now: Date.now(), root })],
     ["drift", () => checkDrift({ root, workspaces })],
     ["deployed", () => checkDeployed({ root })],
-    ["rails", () => checkRails({ root, userSettings: join17(home, USER_SETTINGS) })],
-    ["exams", () => checkExams({ root, workspaces })]
+    ["rails", () => checkRails({ root, userSettings: join18(home, USER_SETTINGS) })],
+    ["exams", () => checkExams({ root, workspaces })],
+    ["seams", () => checkSeams({ root })]
   ];
   const collected = [];
   const ran = [];
@@ -3163,6 +3215,7 @@ var ABOUT = {
   drift: "the gates that were set up and are no longer running",
   envelope: "every gate read as many things as it was handed",
   exams: "every adopted floor has a test file that runs its exam",
+  seams: "the seam globs adoption.seams declares each name a file",
   exercised: "every enabled rule fires on at least one corpus file",
   group: "the packages published on one version resolve to one version",
   loaded: "the plugin oxlint would load is the one the manifest pins",

package/dist/doctor-cli.js

@@ -4,7 +4,7 @@ import {
   formatDoctor,
   formatJson,
   runDoctor
-} from "./chunk-6ERJOIZS.js";
+} from "./chunk-2ZVCJG5R.js";
 
 // src/doctor-cli.ts
 import { fstatSync, statSync } from "fs";
@@ -135,6 +135,9 @@ var ABOUT = {
               installed floor's own declarations \u2014 every export ending in Conformance \u2014 never off a
               list held here. WARNs, because a floor adopted last week and one whose exam somebody
               deleted look the same from outside.`,
+  seams: `The seam globs adoption.seams declares each name a file. A glob whose file was deleted and
+              never undeclared measures nothing \u2014 every bump reads 0 deleted, 0 added over it \u2014 and a
+              floor's adoption is measured by exactly those files.`,
   exercised: `Every rule a config enables, against the corpus the loaded plugin ships. A rule at
               "error" that can never fire is indistinguishable from a clean tree. The line says what
               its number is evidence ABOUT \u2014 the corpus and the probes, never this tree \u2014 and a rule

package/dist/index.d.ts

@@ -11,7 +11,7 @@ type Overrides = {
  * question one layer out — whether what was reported is what happened; `rails` asks it of the
  * settings an unattended run is bounded by, where a rendered file is not a loaded setting.
  */
-declare const CHECKS: readonly ["loaded", "group", "exercised", "baseline", "runner", "envelope", "drift", "observability", "deployed", "rails", "exams"];
+declare const CHECKS: readonly ["loaded", "group", "exercised", "baseline", "runner", "envelope", "drift", "observability", "deployed", "rails", "exams", "seams"];
 type CheckName = (typeof CHECKS)[number];
 /**
  * `SKIP` is a first-class answer and is printed like any other: a check whose line is missing reads
@@ -140,7 +140,7 @@ declare const discoverConfigs: (root: string) => DiscoveredConfig[];
 declare const discoverWorkspaces: (root: string) => Workspace[];
 declare const readRatchet: (root: string) => RatchetConfig | undefined;
 
-declare const runDoctor: ({ baseline, home, only, oxlint, root, strict, }: DoctorOptions) => Promise<DoctorReport>;
+declare const runDoctor: ({ baseline, home, only, oxlint, root, strict }: DoctorOptions) => Promise<DoctorReport>;
 
 /** The file `create-geonosis --domains` writes, held against the scaffold's own name by a test. */
 declare const COMPOSITION_ROOT = "src/platform.ts";
@@ -160,7 +160,7 @@ declare const READERS: Record<string, string>;
  * block nothing reads. The other four checks ask whether a gate measures what it names; this one
  * asks whether it is still there at all.
  */
-declare const checkDrift: ({ readers, root, userSettings, workspaces, }: {
+declare const checkDrift: ({ readers, root, userSettings, workspaces }: {
     /**
      * Which package reads which block. It is a parameter so a test can name one that can NEVER be
      * installed: a fixture that turns on whether some real package happens to be present on the base
@@ -211,7 +211,7 @@ declare const enabledRulesOf: (rules: Record<string, unknown>, plugin: string) =
  * place where the difference is visible, so the question is asked there, under THIS config's
  * options, because the options are usually where the reach went.
  */
-declare const checkExercised: ({ config, corpus, entry, oxlint, repoCorpus, root, }: {
+declare const checkExercised: ({ config, corpus, entry, oxlint, repoCorpus, root }: {
     config: DiscoveredConfig;
     corpus: string;
     /** The plugin module oxlint loads from this config's directory — where the probes are read. */
@@ -261,7 +261,7 @@ declare const declaredGroupsOf: (root: string) => HeldTogether[] | undefined;
  * pnpm the store keeps every version ever installed, so a count of directories answers nonsense on
  * a correct tree.
  */
-declare const checkGroup: ({ groups, root, workspaces, }: {
+declare const checkGroup: ({ groups, root, workspaces }: {
     groups?: readonly HeldTogether[];
     root: string;
     workspaces: Workspace[];
@@ -293,7 +293,7 @@ declare const installedPluginVersions: (home: string, name: string) => string[];
  * runs the hooks of the one before, and a refusal added this release cannot fire. That is the
  * declared-versus-loaded question one surface out, and it is this check's to ask.
  */
-declare const checkClaudePlugin: ({ candidate, home, name, train, }: {
+declare const checkClaudePlugin: ({ candidate, home, name, train }: {
     /**
      * Whether the packages here are a RELEASE CANDIDATE the registry has never seen — installed from
      * tarballs or a `file:` override. `claude plugin update` reads the marketplace, so it cannot
@@ -320,7 +320,7 @@ declare const satisfies: (version: string, spec: string) => boolean | undefined;
  * config, that names the package in any dependency block — ending at the root's. A workspace that
  * declares nothing inherits the root's pin, which is what a hoisted install actually installed.
  */
-declare const declaredFor: ({ dir, specifier, workspaces, }: {
+declare const declaredFor: ({ dir, specifier, workspaces }: {
     dir: string;
     specifier: string;
     workspaces: Workspace[];
@@ -328,7 +328,7 @@ declare const declaredFor: ({ dir, specifier, workspaces, }: {
     at: string;
     spec: string;
 } | undefined;
-declare const checkLoaded: ({ configs, home, root, workspaces, }: {
+declare const checkLoaded: ({ configs, home, root, workspaces }: {
     configs: DiscoveredConfig[];
     home?: string;
     root: string;
@@ -354,7 +354,7 @@ type ObservabilityConfig = {
     /** What kind of sink this repo runs: `posthog`, `otlp`, `memory`, … */
     sink?: string;
 };
-declare const checkObservability: ({ now, root, }: {
+declare const checkObservability: ({ now, root }: {
     now: number;
     root: string;
 }) => Promise<Finding[]>;
@@ -413,7 +413,7 @@ declare const relativeToRoot: (root: string, path: string) => string;
  * somebody deleted, and a FAIL on every fresh install is the shape that teaches a reader to skim
  * the section — the lesson the `runner` check already learned about warning the wrong runner.
  */
-declare const checkExams: ({ root, workspaces, }: {
+declare const checkExams: ({ root, workspaces }: {
     root: string;
     workspaces: Workspace[];
 }) => Finding[];
@@ -429,7 +429,7 @@ declare const checkExams: ({ root, workspaces, }: {
  * a WARN that names the wrong runner's sin is a line a reader learns to skip, and the lines it
  * teaches them to skip are the true ones next to it.
  */
-declare const checkRunner: ({ ratchet, root, workspaces, }: {
+declare const checkRunner: ({ ratchet, root, workspaces }: {
     ratchet?: RatchetConfig;
     root: string;
     workspaces: Workspace[];

package/dist/index.js

@@ -48,7 +48,7 @@ import {
   resolveFrom,
   runDoctor,
   satisfies
-} from "./chunk-6ERJOIZS.js";
+} from "./chunk-2ZVCJG5R.js";
 export {
   CHECKS,
   COMPOSITION_ROOT,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@geonosis/doctor",
-  "version": "2.6.1",
+  "version": "2.8.0",
   "types": "./dist/index.d.ts",
   "description": "The adoption doctor — declared ≠ loaded, enabled ≠ exercised, a baseline that grew, a runner whose exit code is the only verdict.",
   "keywords": [
@@ -36,10 +36,10 @@
     "dist"
   ],
   "dependencies": {
-    "@geonosis/lint-parity": "2.6.1"
+    "@geonosis/lint-parity": "2.8.0"
   },
   "devDependencies": {
-    "@geonosis/ratchet": "2.6.1"
+    "@geonosis/ratchet": "2.8.0"
   },
   "peerDependencies": {
     "oxlint": ">=1.77"