rvlib-shipkit 2.0.1 → 2.0.2

package/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 2.0.2
+
+- **SK001 clears its own lint shadow.** A `lint` or `fmt` script that routes through `shipkit ci` beside the bunfig section of the same name is deleted by the repair, with its `:raw` half, so `bun lint` runs the built-in. A script whose body is a tool stays named for a human.
+
 ## 2.0.1
 
 - **`sk fleet --update --major` crosses the major again.** Each repo's guarded run now stays in its own process (`SHIPKIT_NO_HOME`), because a check the home answered never saw `SHIPKIT_ALLOW_MAJOR` and refused the breaking jump, naming the very command that was running.

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "rvlib-shipkit",
-    "version": "2.0.1",
+    "version": "2.0.2",
     "funding": "https://github.com/sponsors/rvion",
     "description": "All-in-one repo companion: ci gate (task DAG), auto-discovered health checks, release-contract fulfillness, asks queue, md-wiki tools, asset atelier, live tui",
     "license": "MIT",

package/src/checks/SK001-package-json-shape.check.ts

@@ -282,6 +282,19 @@ export const packageJsonShape = check({
         // none, so scaffolding this into a repo without any turns the gate red
         // on work the fixer itself did. a fixer must never manufacture a failure.
         if (pkg.scripts['test'] == null && hasTestFiles(testFiles(c))) { pkg.scripts['test'] = 'bun test'; changes.push(`script 'test' added`) }
+        // a script beside its declared bunfig section only shadows the built-in.
+        // A routed body and its `<id>:raw` are house wiring, so they go. A tool
+        // body is a repo's own answer and stays named
+        const bunfig = c.read('bunfig.toml')
+        for (const n of ['lint', 'fmt'] as const) {
+            const body = pkg.scripts[n]
+            if (body == null || !bunfigDeclares(bunfig, n)) continue
+            if (!routesToShipkit(body)) { notes.push(`'${n}' shadows \`bun ${n}\` — its body is a tool, left for a human`); continue }
+            delete pkg.scripts[n]
+            const gone: string[] = [n]
+            if (pkg.scripts[rawName(n)] != null) { delete pkg.scripts[rawName(n)]; gone.push(rawName(n)) }
+            changes.push(`script ${gone.map((g) => `'${g}'`).join(' and ')} deleted (bunfig.toml [${n}] runs \`bun ${n}\`)`)
+        }
         // ONE WAY IN, last so it also routes whatever this pass just added:
         // the tool body slides to `<id>:raw` and `<id>` becomes the shipkit
         // call. Both keys keep their place in the file, so the diff a human