rvlib-shipkit 2.1.0 → 2.1.1

package/CHANGELOG.md

@@ -1,3 +1,9 @@
+## 2.1.1
+
+- **`--only a,b` takes a comma list** in `sk ci`, `sk cache-audit` and `sk fleet`, the same as repeating the flag.
+- **`sk gp start` waits for pm to act.** The state the switch leaves (down for a start, up for a stop) counts as pm not there yet for 30 s, so a process that takes a few seconds to launch no longer reads as down. A crash still fails at once.
+- **`sk mv` rewrites relative markdown links on a folder move**, through the same pass the page move uses, and a page move no longer crashes on a repo with a git submodule.
+
 ## 2.1.0
 
 - **SK027 requires exhaustive agent/ indexes.** Every `agent/` page is listed by an index (a bullet, numbered item, table row, `→` pointer or `@` include names it or its folder), and every index is loaded from CLAUDE.md or listed by a loaded file. A page linked only from prose now fails, with the page that already names it as the fix. The template's house `[lint]` block makes `typescript/no-empty-object-type` an error.

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "rvlib-shipkit",
-    "version": "2.1.0",
+    "version": "2.1.1",
     "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/cli/cmd_CacheAudit.ts

@@ -4,6 +4,7 @@
 import { Command, Option } from 'clipanion'
 import { loadRepoConfig } from '../repo-config.ts'
 import { auditCache } from '../core/cache-audit.ts'
+import { listFlag } from './list-flag.ts'
 
 export class CacheAuditCommand extends Command {
     static override paths = [['cache-audit']]
@@ -28,9 +29,9 @@ export class CacheAuditCommand extends Command {
         const root = process.cwd()
         const n = Number(this.sample)
         if (!Number.isInteger(n) || n < 1) { this.context.stdout.write(`FAIL: --sample takes a whole number above 0, got '${this.sample}'\n`); return 1 }
-        const r = await auditCache(root, await loadRepoConfig(root), { sample: n, only: this.only })
+        const r = await auditCache(root, await loadRepoConfig(root), { sample: n, only: this.only == null ? undefined : listFlag(this.only) })
         if (this.json) { this.context.stdout.write(`${JSON.stringify(r)}\n`); return r.stale.length > 0 ? 1 : 0 }
-        if (r.rows.length === 0) { this.context.stdout.write(`skip: nothing to audit, the cache answers none of ${this.only?.join(', ') ?? 'the checks'} right now\n`); return 0 }
+        if (r.rows.length === 0) { this.context.stdout.write(`skip: nothing to audit, the cache answers none of ${this.only == null ? 'the checks' : listFlag(this.only).join(', ')} right now\n`); return 0 }
         for (const row of r.rows) {
             const same = row.fresh === row.cached || row.fresh === 'skip'
             this.context.stdout.write(same

package/src/cli/cmd_Ci.ts

@@ -7,6 +7,7 @@ import { runJob } from '../core/job-run.ts'
 import { runCi } from '../core/run-ci.ts'
 import { runViaHub } from '../home/via-hub.ts'
 import { maybeSelfUpdateAndRestart } from '../core/self-update.ts'
+import { listFlag } from './list-flag.ts'
 
 export class CiCommand extends Command {
     static override paths = [['ci']]
@@ -37,7 +38,7 @@ export class CiCommand extends Command {
     frozen = Option.Boolean('--frozen', false, { description: 'report-only mode for self-maintaining checks: SK005 never touches package.json, and generated paths are never untracked. STATUS.md is still refreshed on disk, but it is gitignored, so a run leaves the tree clean' })
     skip = Option.Array('--skip', [], { description: 'skip a check group (repeatable)' })
     withIds = Option.Array('--with', [], { description: 'include a check the quick gate leaves out (repeatable)' })
-    only = Option.Array('--only', [], { description: 'run only this check + its dep closure (repeatable). A group name runs every check declaring that group. No repair pass unless --fix is named' })
+    only = Option.Array('--only', [], { description: 'run only this check + its dep closure (repeatable, or comma separated). A group name runs every check declaring that group. No repair pass unless --fix is named' })
     fresh = Option.Boolean('--fresh', false, { description: 'run every check: no answer comes from the cache, and what this run finds is stored fresh' })
     full = Option.Boolean('--full', false, { description: `run the FULL test tier: every suite the repo declares, the slow ones it skips while you work included. What the release path always runs. Same as SHIPKIT_TEST_FULL=1` })
 
@@ -74,7 +75,7 @@ export class CiCommand extends Command {
             ...(this.frozen ? ['--frozen'] : []),
             ...this.skip.flatMap((v) => ['--skip', v]),
             ...this.withIds.flatMap((v) => ['--with', v]),
-            ...this.only.flatMap((v) => ['--only', v]),
+            ...listFlag(this.only).flatMap((v) => ['--only', v]),
             ...(this.fresh ? ['--fresh'] : []),
             ...(this.full ? ['--full'] : []),
             ...(this.verbose ? ['--verbose'] : []),
@@ -95,7 +96,7 @@ export class CiCommand extends Command {
                 frozen: this.frozen,
                 skipGroups: this.skip,
                 withIds: this.withIds,
-                onlyIds: this.only,
+                onlyIds: listFlag(this.only),
                 fresh: this.fresh,
                 ...(this.full ? { tests: 'full' as const } : {}),
                 ...(this.verbose ? { verbose: true } : {}),

package/src/cli/cmd_Fleet.ts

@@ -6,6 +6,7 @@ import { Command, Option } from 'clipanion'
 import { join } from 'path'
 import { npmLatest, resolveLatest, scanFleet, type FleetRepo } from '../fleet/scan-fleet.ts'
 import { short, updateFleet } from '../fleet/update-fleet.ts'
+import { listFlag } from './list-flag.ts'
 
 export class FleetCommand extends Command {
     static override paths = [['fleet']]
@@ -32,7 +33,8 @@ export class FleetCommand extends Command {
     async execute(): Promise<number> {
         const root = this.root ?? join(process.cwd(), '..')
         let repos = scanFleet(root)
-        if (this.only != null && this.only.length > 0) repos = repos.filter((r) => this.only!.includes(r.name))
+        const only = listFlag(this.only)
+        if (only.length > 0) repos = repos.filter((r) => only.includes(r.name))
         const w = (l: string) => this.context.stdout.write(`${l}\n`)
         if (repos.length === 0) {
             w(`skip: no repo under ${root} depends on an rvlib-* package`)

package/src/cli/cmd_GlobalProcesses.ts

@@ -172,6 +172,8 @@ export type SwitchDeps = {
 /** the longest a switch is followed: a bring-up polls its health for minutes */
 export const SWITCH_WAIT_MS = 10 * 60_000
 const SWITCH_POLL_MS = 1_000
+/** how long the state the switch left counts as pm not having acted yet */
+export const SWITCH_GRACE_MS = 30_000
 
 /** flip a global process through pm, then follow pm's state until its probe
  *  settles it: 0 once it is up (or down for stop), 1 otherwise. The launcher's
@@ -183,7 +185,8 @@ export async function switchProcess(id: string, act: SwitchAct, d: SwitchDeps, w
         return 1
     }
     const want = act === 'stop' ? 'down' : 'up'
-    const end = d.now() + waitMs
+    const start = d.now()
+    const end = start + waitMs
     let said = ''
     for (;;) {
         const v = await d.view(id)
@@ -195,7 +198,9 @@ export async function switchProcess(id: string, act: SwitchAct, d: SwitchDeps, w
             said = v.status
             d.log(`${id}: ${v.status}`)
         }
-        const moving = v.status === 'starting' || v.status === 'stopping' || v.want !== want
+        // the state it is leaving, read just after the switch, is pm not there yet
+        const leaving = v.status === (want === 'up' ? 'down' : 'up') && d.now() - start < SWITCH_GRACE_MS
+        const moving = v.status === 'starting' || v.status === 'stopping' || v.want !== want || leaving
         if (!moving) {
             if (v.status === want) return 0
             d.log(`FAIL ${id} is ${v.status}, not ${want}${v.note == null ? '' : `: ${v.note}`}`)

package/src/cli/list-flag.ts

@@ -0,0 +1,3 @@
+/** a repeatable flag's values, each also split on commas: `--only a,b` is `--only a --only b` */
+export const listFlag = (values: readonly string[] | undefined): string[] =>
+    (values ?? []).flatMap((v) => v.split(',')).map((v) => v.trim()).filter((v) => v !== '')

package/src/mv/mv-md.ts

@@ -154,7 +154,8 @@ export function movePage(root: string, oldRel: string, newRel: string, io: MvIo,
     let touched = 0
     for (const f of tracked) {
         const p = join(root, f)
-        if (!existsSync(p)) continue
+        // a submodule is one ls-files entry and a directory: nothing to read
+        if (!isFile(p)) continue
         const before = readFileSync(p, 'utf8')
         let after = retargetRelativeRefs(root, before, dirOf(f), dirOf(moves.get(f) ?? f), moves)
         for (const [oldTok, newTok] of pairs) after = after.replace(bound(oldTok), newTok)

package/src/ts/rename.ts

@@ -16,6 +16,7 @@ import { basename, dirname, extname, join, relative, resolve } from 'path'
 import { canonicalSpecifier, isSkipped, loadConfig, toPosix, type RvtConfig } from 'rvlib-shipkit/src/ts/config.ts'
 import { extractSpecs } from 'rvlib-shipkit/src/ts/source-graph.ts'
 import { gitBin } from '../core/git-bin.ts'
+import { retargetRelativeRefs } from '../mv/mv-md.ts'
 
 const CODE_RE = /\.[cm]?[jt]sx?$/
 
@@ -106,7 +107,15 @@ export function runRename(srcArg: string, dstArg: string, opts: { apply?: boolea
    const isFile = (rel: string): boolean => {
       try { return statSync(join(repoRoot, rel)).isFile() } catch { return false }
    }
+   // a markdown link is relative to its page: the md engine's pass retargets it
+   const mdMoves = new Map(allFiles.flatMap((f): [string, string][] => { const to = movedTo(f); return to == null ? [] : [[f, to]] }))
+   const dirOf = (p: string): string => { const d = dirname(p); return d === '.' ? '' : d }
    const rewriteFile = (file: string, content: string): { text: string; count: number } => {
+      if (file.endsWith('.md')) {
+         const linked = retargetRelativeRefs(repoRoot, content, dirOf(file), dirOf(movedTo(file) ?? file), mdMoves)
+         const scanned = scanPairs(linked, pairs)
+         return { text: scanned.text, count: scanned.count + (linked === content ? 0 : 1) }
+      }
       const rel = CODE_RE.test(file) ? relativeRewrites(file, content, movedTo, isFile) : new Map<string, string>()
       if (rel.size === 0) return scanPairs(content, pairs)
       // the relative specifiers this move retargets are set aside while the