@geonosis/lint-parity 2.6.1 → 2.8.0

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @geonosis/lint-parity
 
+## 2.8.0
+
+## 2.7.0
+
+### Patch Changes
+
+- c13ade8: `manifestOf` leaves out a rule the plugin has retired (`meta.retired`). A retired id still resolves so
+  a config written before the retirement parses, and it reports nothing by design — so a corpus can
+  never be evidence about it. Listed in the manifest a plugin's corpus build writes, it read as reach NO
+  for ever in every consumer's report; the kit's own shipped manifest carried both retired ids until
+  this, 82 rules where the corpus is evidence about 80.
+
 ## 2.6.1
 
 ## 2.6.0

package/bin/geonosis-lint-parity.mjs

@@ -34,13 +34,13 @@ if (existsSync(src)) {
   const built = newest(dist)
   if (built === 0) {
     process.stderr.write(
-      'geonosis-lint-parity: 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-lint-parity: 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-lint-parity: dist is older than src — run pnpm build before trusting this bin.\n',
+      'geonosis-lint-parity: dist is older than src — run pnpm build before trusting this bin.\n'
     )
     process.exit(2)
   }

package/dist/{chunk-JPGZG2HA.js → chunk-OQSXP7ZK.js}

@@ -278,9 +278,13 @@ var ruleIdOf = (line) => {
   return at === -1 ? token : `${token.slice(0, at)}/${token.slice(at + 1, -1)}`;
 };
 var MANIFEST_FILE = "manifest.json";
+var isRetired = (rule) => {
+  const retired = rule?.meta?.retired;
+  return typeof retired === "object" && retired !== null;
+};
 var manifestOf = (plugin) => ({
   plugin: plugin.meta.name,
-  rules: Object.keys(plugin.rules).map((rule) => `${plugin.meta.name}/${rule}`).toSorted()
+  rules: Object.entries(plugin.rules).filter(([, rule]) => !isRetired(rule)).map(([rule]) => `${plugin.meta.name}/${rule}`).toSorted()
 });
 var namesRule = (claim, rule) => {
   if (claim === rule) return true;

package/dist/index.d.ts

@@ -131,11 +131,6 @@ type CorpusManifest = {
     plugin: string;
     rules: string[];
 };
-/**
- * The manifest for a plugin, off the plugin itself. A plugin's corpus build writes what this
- * returns beside its copied fixture tree — derived, never hand-maintained, so a rule cannot be
- * added to the plugin and forgotten in the list of what the corpus is evidence about.
- */
 declare const manifestOf: (plugin: {
     meta: {
         name: string;
@@ -154,7 +149,7 @@ declare const readManifest: (corpus: string) => CorpusManifest;
  * consumer inherits every one of those as a claim it cannot check. The corpus is the evidence, and
  * `neither` is the list that has to stay empty.
  */
-declare const corpusOf: ({ configA, configB, corpus, expectChanged, oxlint, }: {
+declare const corpusOf: ({ configA, configB, corpus, expectChanged, oxlint }: {
     configA: string;
     configB: string;
     corpus: string;
@@ -202,21 +197,21 @@ declare const parseParityArgs: (argv: string[]) => ParityArgs;
  * config named relatively does. A census taken under different arguments would describe a run
  * nobody made, which is the failure this whole envelope is about.
  */
-declare const filesLintedBy: ({ config, cwd, oxlint, paths, }: {
+declare const filesLintedBy: ({ config, cwd, oxlint, paths }: {
     config: string;
     cwd: string;
     oxlint: string;
     paths: string[];
 }) => string[];
 /** Which files both configs would read, and which each would read alone. */
-declare const fileCensus: ({ configA, configB, cwd, oxlint, paths, }: {
+declare const fileCensus: ({ configA, configB, cwd, oxlint, paths }: {
     configA: string;
     configB: string;
     cwd: string;
     oxlint: string;
     paths: string[];
 }) => FileCensus;
-declare const parityOf: ({ configA, configB, cwd, out, oxlint, paths, }: {
+declare const parityOf: ({ configA, configB, cwd, out, oxlint, paths }: {
     configA: string;
     configB: string;
     cwd: string;

package/dist/index.js

@@ -17,7 +17,7 @@ import {
   ruleIdOf,
   rulesNamedBy,
   rulesTurnedOffBy
-} from "./chunk-JPGZG2HA.js";
+} from "./chunk-OQSXP7ZK.js";
 export {
   MANIFEST_FILE,
   ParityError,

package/dist/parity-cli.js

@@ -9,7 +9,7 @@ import {
   parseParityArgs,
   resolveOxlint,
   writeEnvelope
-} from "./chunk-JPGZG2HA.js";
+} from "./chunk-OQSXP7ZK.js";
 
 // src/parity-cli.ts
 import { mkdirSync, writeFileSync } from "fs";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@geonosis/lint-parity",
-  "version": "2.6.1",
+  "version": "2.8.0",
   "types": "./dist/index.d.ts",
   "description": "Findings parity and rule reach over any two oxlint configs — the diff a fork is deleted on.",
   "keywords": [
@@ -45,7 +45,7 @@
     "access": "public"
   },
   "devDependencies": {
-    "@geonosis/ratchet": "2.6.1"
+    "@geonosis/ratchet": "2.8.0"
   },
   "scripts": {
     "build": "tsup",