@vodmal/vdx-cli 0.25.0 → 0.27.0

package/src/doctor.ts

@@ -6,6 +6,7 @@ import { execFileSync } from 'node:child_process';
 import { effectiveAuthor, isGitRepo, rankAuthors, scanPool } from './author.ts';
 import { authorPoolDirs, loadEnvironment, resolveEnvironmentPath, shellQuote, type Environment } from './ai.ts';
 import { checkRepoHooks } from './clone-checks.ts';
+import { at, dict } from './narrow.ts';
 import {
   checkPersonalHooks,
   defaultPersonalHookDeps,
@@ -98,24 +99,24 @@ function checkNode(): CheckResult {
       label: 'Node.js',
       status: 'missing',
       message: 'node binary not on PATH',
-      remedy: 'https://nodejs.org/en/download (LTS ≥ 20 recommended)',
+      remedy: 'https://nodejs.org/en/download (vdx needs 22 or newer)',
     };
   }
   const major = parseSemverMajor(v);
   if (major === null) {
     return { id: 'node', label: 'Node.js', status: 'warning', message: `unparseable version: ${v}` };
   }
-  if (major < 20) {
+  if (major < 22) {
     return {
       id: 'node',
       label: 'Node.js',
       status: 'warning',
       level: 1,
-      message: `${v} — below recommended (≥ 20 LTS)`,
-      remedy: 'mise use -g node@20 (or upgrade via nvm/brew)',
+      message: `${v} — vdx needs 22 or newer (engines in its package.json)`,
+      remedy: 'mise use -g node@24 (or upgrade via nvm/brew)',
     };
   }
-  return { id: 'node', label: 'Node.js', status: 'ok', level: major >= 22 ? 4 : 3, message: v };
+  return { id: 'node', label: 'Node.js', status: 'ok', level: major >= 24 ? 4 : 3, message: v };
 }
 
 function checkGit(): CheckResult {
@@ -354,7 +355,7 @@ export function checkClaudeCodePlugin(
   claudeDir: string = process.env['CLAUDE_CONFIG_DIR'] || path.join(os.homedir(), '.claude'),
 ): CheckResult {
   const base = { id: 'claude-plugin', label: 'Claude Code vdx plugin' };
-  const readJson = (file: string): any => {
+  const readJson = (file: string): unknown => {
     try {
       return JSON.parse(fs.readFileSync(path.join(claudeDir, file), 'utf8'));
     } catch {
@@ -370,13 +371,13 @@ export function checkClaudeCodePlugin(
       remedy: PLUGIN_INSTALL_HINT,
     };
   }
-  const key = Object.keys(installed.plugins ?? {}).find((k) => k.startsWith('vdx@'));
+  const key = Object.keys(dict(at(installed, 'plugins')) ?? {}).find((k) => k.startsWith('vdx@'));
   if (!key) {
     return { ...base, status: 'warning', message: 'not installed', remedy: PLUGIN_INSTALL_HINT };
   }
-  const version = installed.plugins[key]?.[0]?.version;
+  const version = at(installed, 'plugins', key, '0', 'version');
   const label = `${key}${typeof version === 'string' ? ` ${version}` : ''}`;
-  if (readJson('settings.json')?.enabledPlugins?.[key] === false) {
+  if (at(readJson('settings.json'), 'enabledPlugins', key) === false) {
     return { ...base, status: 'warning', message: `${label} installed but disabled`, remedy: `/plugin enable ${key}` };
   }
   return { ...base, status: 'ok', message: label };

package/src/evaluator.ts

@@ -1,8 +1,9 @@
 import type { Predicate } from './rubric.ts';
 import type { Ctx } from './facts.ts';
-import { REGISTRY, isUnknown, type Unknown, type Verdict } from './predicates.ts';
+import { REGISTRY, isUnknown, type Args, type Unknown, type Verdict } from './predicates.ts';
+import { at, isDict, list } from './narrow.ts';
 
-export { isUnknown, type Unknown, type Verdict };
+export { isUnknown, type Verdict };
 
 /**
  * Sugar shorthand: `{ has_task: "up" }` → `{ fn: "has_task", args: { task: "up" } }`
@@ -34,21 +35,21 @@ export function evalVerdict(p: Predicate, ctx: Ctx): Verdict {
   if (p === null || p === undefined) return false;
   if (typeof p === 'boolean') return p;
 
-  if (typeof p === 'object' && !Array.isArray(p)) {
-    const obj = p as Record<string, any>;
+  if (isDict(p)) {
+    const obj = p;
 
     if ('all_of' in obj) {
-      const vs = (obj.all_of as Predicate[]).map((q) => evalVerdict(q, ctx));
+      const vs = list(obj['all_of']).map((q) => evalVerdict(q, ctx));
       if (vs.some((v) => v === false)) return false;
       return firstUnknown(vs) ?? true;
     }
     if ('any_of' in obj) {
-      const vs = (obj.any_of as Predicate[]).map((q) => evalVerdict(q, ctx));
+      const vs = list(obj['any_of']).map((q) => evalVerdict(q, ctx));
       if (vs.some((v) => v === true)) return true;
       return firstUnknown(vs) ?? false;
     }
     if ('at_least_n_of' in obj) {
-      const { n, predicates } = obj.at_least_n_of as { n: number; predicates: Predicate[] };
+      const { n, predicates } = obj['at_least_n_of'] as { n: number; predicates: Predicate[] };
       const vs = predicates.map((q) => evalVerdict(q, ctx));
       const matched = vs.filter((v) => v === true).length;
       if (matched >= n) return true;
@@ -57,16 +58,18 @@ export function evalVerdict(p: Predicate, ctx: Ctx): Verdict {
       return firstUnknown(vs)!;
     }
     if ('not' in obj) {
-      const v = evalVerdict(obj.not, ctx);
+      const v = evalVerdict(obj['not'], ctx);
       return isUnknown(v) ? v : !v;
     }
     if ('fn' in obj) {
-      const fn = REGISTRY[String(obj.fn)];
+      const name = String(obj['fn']);
+      const fn = REGISTRY[name];
       if (!fn) {
-        process.stderr.write(`warning: unknown predicate "${obj.fn}"\n`);
+        process.stderr.write(`warning: unknown predicate "${name}"\n`);
         return false;
       }
-      return fn(obj.args ?? {}, ctx);
+      // `args` is the predicate's to read: package_present takes a bare string too.
+      return fn((obj['args'] ?? {}) as Args, ctx);
     }
 
     // Sugar form: single-key object where key is a registered predicate
@@ -76,9 +79,7 @@ export function evalVerdict(p: Predicate, ctx: Ctx): Verdict {
       const fn = REGISTRY[key];
       if (fn) {
         const value = obj[key];
-        if (typeof value === 'object' && value !== null && !Array.isArray(value)) {
-          return fn(value, ctx);
-        }
+        if (isDict(value)) return fn(value, ctx);
         const argKey = SUGAR_ARG_KEY[key];
         if (argKey) {
           return fn({ [argKey]: value }, ctx);
@@ -104,14 +105,14 @@ export function evalPredicate(p: Predicate, ctx: Ctx): boolean {
 export function predicateNames(p: Predicate): string[] {
   const out: string[] = [];
   const walk = (q: Predicate): void => {
-    if (q === null || q === undefined || typeof q !== 'object' || Array.isArray(q)) return;
-    const obj = q as Record<string, any>;
-    if ('all_of' in obj) return (obj.all_of as Predicate[]).forEach(walk);
-    if ('any_of' in obj) return (obj.any_of as Predicate[]).forEach(walk);
-    if ('at_least_n_of' in obj) return ((obj.at_least_n_of?.predicates as Predicate[]) ?? []).forEach(walk);
-    if ('not' in obj) return walk(obj.not);
+    if (!isDict(q)) return;
+    const obj = q;
+    if ('all_of' in obj) return list(obj['all_of']).forEach(walk);
+    if ('any_of' in obj) return list(obj['any_of']).forEach(walk);
+    if ('at_least_n_of' in obj) return list(at(obj, 'at_least_n_of', 'predicates')).forEach(walk);
+    if ('not' in obj) return walk(obj['not']);
     if ('fn' in obj) {
-      out.push(String(obj.fn));
+      out.push(String(obj['fn']));
       return;
     }
     const keys = Object.keys(obj);

package/src/facts.ts

@@ -2,6 +2,7 @@ import * as fs from 'node:fs';
 import * as path from 'node:path';
 import * as TOML from 'smol-toml';
 import * as YAML from 'js-yaml';
+import { at, dict, str } from './narrow.ts';
 
 export interface Ctx {
   projectRoot: string;
@@ -52,7 +53,7 @@ export function readText(ctx: Ctx, rel: string): string | null {
   }
 }
 
-export function readJson(ctx: Ctx, rel: string): any {
+export function readJson(ctx: Ctx, rel: string): unknown {
   const key = `json:${rel}`;
   if (ctx.cache.has(key)) return ctx.cache.get(key);
   const text = readText(ctx, rel);
@@ -61,7 +62,7 @@ export function readJson(ctx: Ctx, rel: string): any {
     return null;
   }
   try {
-    const data = JSON.parse(text);
+    const data: unknown = JSON.parse(text);
     ctx.cache.set(key, data);
     return data;
   } catch {
@@ -70,7 +71,7 @@ export function readJson(ctx: Ctx, rel: string): any {
   }
 }
 
-export function readToml(ctx: Ctx, rel: string): any {
+export function readToml(ctx: Ctx, rel: string): unknown {
   const key = `toml:${rel}`;
   if (ctx.cache.has(key)) return ctx.cache.get(key);
   const text = readText(ctx, rel);
@@ -88,7 +89,7 @@ export function readToml(ctx: Ctx, rel: string): any {
   }
 }
 
-export function readYaml(ctx: Ctx, rel: string): any {
+function readYaml(ctx: Ctx, rel: string): unknown {
   const key = `yaml:${rel}`;
   if (ctx.cache.has(key)) return ctx.cache.get(key);
   const text = readText(ctx, rel);
@@ -107,7 +108,7 @@ export function readYaml(ctx: Ctx, rel: string): any {
 }
 
 /** Parse a structured file by extension. Returns parsed object/array or null. */
-export function readStructured(ctx: Ctx, rel: string): any {
+export function readStructured(ctx: Ctx, rel: string): unknown {
   const lower = rel.toLowerCase();
   if (lower.endsWith('.json')) return readJson(ctx, rel);
   if (lower.endsWith('.toml')) return readToml(ctx, rel);
@@ -122,14 +123,7 @@ export function readStructured(ctx: Ctx, rel: string): any {
  */
 export function resolveConfigPath(root: unknown, jsonpath: string): unknown {
   if (root === null || root === undefined) return undefined;
-  const parts = jsonpath.split('.');
-  let cur: any = root;
-  for (const part of parts) {
-    if (cur === null || cur === undefined) return undefined;
-    if (typeof cur !== 'object') return undefined;
-    cur = (cur as Record<string, unknown>)[part];
-  }
-  return cur;
+  return at(root, ...jsonpath.split('.'));
 }
 
 /**
@@ -142,25 +136,12 @@ export function listAllTasks(ctx: Ctx): Set<string> {
   const tasks = new Set<string>();
   const own = ownCtx(ctx);
 
-  const miseT = readToml(own, 'mise.toml');
-  if (miseT?.tasks && typeof miseT.tasks === 'object') {
-    for (const k of Object.keys(miseT.tasks as object)) tasks.add(k);
-  }
-
-  const composer = readJson(own, 'composer.json');
-  if (composer?.scripts && typeof composer.scripts === 'object') {
-    for (const k of Object.keys(composer.scripts)) tasks.add(k);
-  }
-
-  const pkg = readJson(own, 'package.json');
-  if (pkg?.scripts && typeof pkg.scripts === 'object') {
-    for (const k of Object.keys(pkg.scripts)) tasks.add(k);
-  }
-
+  for (const k of Object.keys(dict(at(readToml(own, 'mise.toml'), 'tasks')) ?? {})) tasks.add(k);
+  for (const k of Object.keys(dict(at(readJson(own, 'composer.json'), 'scripts')) ?? {})) tasks.add(k);
+  for (const k of Object.keys(dict(at(readJson(own, 'package.json'), 'scripts')) ?? {})) tasks.add(k);
   // monorepo-ish fallback: server/package.json (см. bookmap)
-  const serverPkg = readJson(own, 'server/package.json');
-  if (serverPkg?.scripts && typeof serverPkg.scripts === 'object') {
-    for (const k of Object.keys(serverPkg.scripts)) tasks.add(`server:${k}`);
+  for (const k of Object.keys(dict(at(readJson(own, 'server/package.json'), 'scripts')) ?? {})) {
+    tasks.add(`server:${k}`);
   }
 
   const make = readText(own, 'Makefile');
@@ -180,18 +161,14 @@ export function isPackagePresent(name: string, ecosystem: string, ctx: Ctx): boo
   const eco = ecosystem ?? 'any';
   if (eco === 'any' || eco === 'composer') {
     const composer = readJson(ctx, 'composer.json');
-    if (composer) {
-      if (composer.require?.[name]) return true;
-      if (composer['require-dev']?.[name]) return true;
-    }
+    if (at(composer, 'require', name)) return true;
+    if (at(composer, 'require-dev', name)) return true;
   }
   if (eco === 'any' || eco === 'npm') {
     for (const rel of ['package.json', 'server/package.json']) {
       const pkg = readJson(ctx, rel);
-      if (pkg) {
-        if (pkg.dependencies?.[name]) return true;
-        if (pkg.devDependencies?.[name]) return true;
-      }
+      if (at(pkg, 'dependencies', name)) return true;
+      if (at(pkg, 'devDependencies', name)) return true;
     }
   }
   return false;
@@ -222,19 +199,17 @@ export function expandPaths(paths: string[], ctx: Ctx): string[] {
   return out;
 }
 
-export function readTsconfig(ctx: Ctx): any {
-  let cfg = readJson(ctx, 'tsconfig.json');
-  if (!cfg) cfg = readJson(ctx, 'server/tsconfig.json');
+export function readTsconfig(ctx: Ctx): { compilerOptions?: Record<string, unknown> } | null {
+  const cfg = dict(readJson(ctx, 'tsconfig.json')) ?? dict(readJson(ctx, 'server/tsconfig.json'));
   if (!cfg) return null;
+  const own = dict(cfg['compilerOptions']);
   // basic one-level `extends` resolution
-  if (cfg.extends && typeof cfg.extends === 'string' && cfg.extends.startsWith('.')) {
-    const baseRel = cfg.extends.endsWith('.json') ? cfg.extends : `${cfg.extends}.json`;
-    const base = readJson(ctx, baseRel);
-    if (base?.compilerOptions) {
-      cfg.compilerOptions = { ...base.compilerOptions, ...(cfg.compilerOptions ?? {}) };
-    }
+  const ext = str(cfg['extends']);
+  if (ext?.startsWith('.')) {
+    const base = dict(at(readJson(ctx, ext.endsWith('.json') ? ext : `${ext}.json`), 'compilerOptions'));
+    if (base) return { compilerOptions: { ...base, ...own } };
   }
-  return cfg;
+  return own ? { compilerOptions: own } : {};
 }
 
 const MANIFEST_TO_STACK: Array<[string, string]> = [

package/src/gha.ts

@@ -8,6 +8,7 @@ import * as fs from 'node:fs';
 import * as path from 'node:path';
 import YAML from 'js-yaml';
 import { fileExists, readText, type Ctx } from './facts.ts';
+import { at, dict, isDict, list, str } from './narrow.ts';
 import { projectTaskCalls } from './vocabulary.ts';
 
 export interface GhaStep {
@@ -48,9 +49,8 @@ function filterMatches(pattern: string, branch: string): boolean {
 }
 
 function branchSelected(filters: unknown, branch: string): boolean {
-  const list = Array.isArray(filters) ? filters : [filters];
   let selected = false;
-  for (const f of list) {
+  for (const f of Array.isArray(filters) ? (filters as unknown[]) : [filters]) {
     if (typeof f !== 'string') continue;
     if (f.startsWith('!')) {
       if (filterMatches(f.slice(1), branch)) selected = false;
@@ -65,15 +65,16 @@ function branchSelected(filters: unknown, branch: string): boolean {
 export function pushesMain(on: unknown): boolean {
   if (on === 'push') return true;
   if (Array.isArray(on)) return on.includes('push');
-  if (!on || typeof on !== 'object') return false;
-  if (!('push' in on)) return false;
-  const push = (on as Record<string, any>).push;
-  if (push === null || push === undefined || typeof push !== 'object') return true;
-  const hasBranchFilter = push.branches !== undefined || push['branches-ignore'] !== undefined;
-  if (!hasBranchFilter && (push.tags !== undefined || push['tags-ignore'] !== undefined)) return false;
+  if (!isDict(on) || !('push' in on)) return false;
+  const push = dict(on['push']);
+  if (!push) return true;
+  const { branches, tags } = push;
+  const branchesIgnore = push['branches-ignore'];
+  const hasBranchFilter = branches !== undefined || branchesIgnore !== undefined;
+  if (!hasBranchFilter && (tags !== undefined || push['tags-ignore'] !== undefined)) return false;
   return MAIN_BRANCHES.some((b) => {
-    if (push.branches !== undefined) return branchSelected(push.branches, b);
-    if (push['branches-ignore'] !== undefined) return !branchSelected(push['branches-ignore'], b);
+    if (branches !== undefined) return branchSelected(branches, b);
+    if (branchesIgnore !== undefined) return !branchSelected(branchesIgnore, b);
     return true;
   });
 }
@@ -82,35 +83,42 @@ function truthy(v: unknown): boolean {
   return v === true || v === 'true';
 }
 
-export function parseWorkflow(file: string, text: string): GhaWorkflow | null {
-  let data: any;
+function parseWorkflow(file: string, text: string): GhaWorkflow | null {
+  let data: unknown;
   try {
     data = YAML.load(text);
   } catch {
     return null;
   }
+  // An array is an object too, and the workflow it would be has no jobs.
   if (!data || typeof data !== 'object') return null;
   const jobs: GhaJob[] = [];
-  for (const [id, job] of Object.entries((data.jobs ?? {}) as Record<string, any>)) {
-    if (!job || typeof job !== 'object') continue;
-    const needs = job.needs === undefined ? [] : Array.isArray(job.needs) ? job.needs : [job.needs];
-    const steps: GhaStep[] = (Array.isArray(job.steps) ? job.steps : [])
-      .filter((s: unknown) => s && typeof s === 'object')
-      .map((s: any) => ({
-        ...(typeof s.run === 'string' ? { run: s.run } : {}),
-        ...(typeof s.uses === 'string' ? { uses: s.uses } : {}),
-        continueOnError: truthy(s['continue-on-error']),
-      }));
+  for (const [id, value] of Object.entries(dict(at(data, 'jobs')) ?? {})) {
+    const job = dict(value);
+    if (!job) continue;
+    const needs = job['needs'] === undefined ? [] : Array.isArray(job['needs']) ? list(job['needs']) : [job['needs']];
+    const steps: GhaStep[] = list(job['steps'])
+      .filter(isDict)
+      .map((s) => {
+        const run = str(s['run']);
+        const uses = str(s['uses']);
+        return {
+          ...(run !== undefined ? { run } : {}),
+          ...(uses !== undefined ? { uses } : {}),
+          continueOnError: truthy(s['continue-on-error']),
+        };
+      });
+    const name = str(job['name']);
     jobs.push({
       id,
-      ...(typeof job.name === 'string' ? { name: job.name } : {}),
-      needs: needs.filter((n: unknown): n is string => typeof n === 'string'),
+      ...(name !== undefined ? { name } : {}),
+      needs: needs.filter((n): n is string => typeof n === 'string'),
       steps,
       continueOnError: truthy(job['continue-on-error']),
-      matrix: Boolean(job.strategy?.matrix),
+      matrix: Boolean(at(job, 'strategy', 'matrix')),
     });
   }
-  return { file, pushesMain: pushesMain(data.on), jobs };
+  return { file, pushesMain: pushesMain(at(data, 'on')), jobs };
 }
 
 export function readWorkflows(ctx: Ctx): GhaWorkflow[] {
@@ -118,7 +126,7 @@ export function readWorkflows(ctx: Ctx): GhaWorkflow[] {
   if (ctx.cache.has(key)) return ctx.cache.get(key) as GhaWorkflow[];
   const dir = path.join(ctx.projectRoot, '.github', 'workflows');
   const out: GhaWorkflow[] = [];
-  let files: string[] = [];
+  let files: string[];
   try {
     files = fs
       .readdirSync(dir)
@@ -154,7 +162,7 @@ export function ciSystems(ctx: Ctx): string[] {
 /** `|| true` and its relatives: the step reports success whatever the command did. */
 const MASK_RE = /\|\|\s*(true|:|exit\s+0)\b/;
 
-export function stepMasked(step: GhaStep, job: GhaJob): boolean {
+function stepMasked(step: GhaStep, job: GhaJob): boolean {
   if (job.continueOnError || step.continueOnError) return true;
   return step.run !== undefined && MASK_RE.test(step.run);
 }

package/src/hooks.ts

@@ -16,8 +16,9 @@ import * as path from 'node:path';
 import { execFileSync } from 'node:child_process';
 import YAML from 'js-yaml';
 import { fileExists, isPackagePresent, readJson, readText, readToml, type Ctx } from './facts.ts';
+import { at, dict, isDict, list } from './narrow.ts';
 
-export type FrameworkId = 'husky' | 'lefthook' | 'simple-git-hooks' | 'cghooks' | 'githooks';
+type FrameworkId = 'husky' | 'lefthook' | 'simple-git-hooks' | 'cghooks' | 'githooks';
 
 export interface HookFramework {
   id: FrameworkId;
@@ -68,8 +69,8 @@ interface InstallStep {
 }
 
 /** Yarn 2+ does not run a project's `prepare` (husky docs, How To → Yarn). */
-export function yarnBerry(ctx: Ctx): boolean {
-  const pm = readJson(ctx, 'package.json')?.packageManager;
+function yarnBerry(ctx: Ctx): boolean {
+  const pm = at(readJson(ctx, 'package.json'), 'packageManager');
   if (typeof pm === 'string') {
     const m = pm.match(/^yarn@(\d+)/);
     if (m) return parseInt(m[1]!, 10) >= 2;
@@ -79,22 +80,24 @@ export function yarnBerry(ctx: Ctx): boolean {
 
 function installSteps(ctx: Ctx): InstallStep[] {
   const out: InstallStep[] = [];
-  const pkg = readJson(ctx, 'package.json');
-  const scripts = pkg?.scripts ?? {};
+  const scripts = dict(at(readJson(ctx, 'package.json'), 'scripts')) ?? {};
   const npmHooks = yarnBerry(ctx) ? ['postinstall'] : ['prepare', 'postinstall'];
   for (const name of npmHooks) {
-    if (typeof scripts[name] === 'string') {
-      out.push({ where: `package.json ${name}`, text: expandNpm(scripts, scripts[name]) });
+    const body = scripts[name];
+    if (typeof body === 'string') {
+      out.push({ where: `package.json ${name}`, text: expandNpm(scripts, body) });
     }
   }
-  const composer = readJson(ctx, 'composer.json');
-  const cmd = composer?.scripts?.['post-install-cmd'];
+  const composerScripts = dict(at(readJson(ctx, 'composer.json'), 'scripts')) ?? {};
+  const cmd = composerScripts['post-install-cmd'];
   if (cmd !== undefined) {
-    out.push({ where: 'composer post-install-cmd', text: expandComposer(composer.scripts, cmd) });
+    out.push({ where: 'composer post-install-cmd', text: expandComposer(composerScripts, cmd) });
   }
-  const up = readToml(ctx, 'mise.toml')?.tasks?.up;
-  if (up && typeof up === 'object') {
-    const runs = (Array.isArray(up.run) ? up.run : [up.run]).filter((r: unknown) => typeof r === 'string');
+  const up = dict(at(readToml(ctx, 'mise.toml'), 'tasks', 'up'));
+  if (up) {
+    const runs = (Array.isArray(up['run']) ? (up['run'] as unknown[]) : [up['run']]).filter(
+      (r): r is string => typeof r === 'string',
+    );
     if (runs.length > 0) out.push({ where: 'mise task up', text: runs.join('\n') });
   }
   return out;
@@ -135,7 +138,7 @@ function findStep(steps: InstallStep[], re: RegExp): string | null {
 /** Hook scripts in a directory: file name = event; the body is the command. */
 function hooksFromDir(ctx: Ctx, dir: string): Map<string, string[]> {
   const out = new Map<string, string[]>();
-  let names: string[] = [];
+  let names: string[];
   try {
     names = fs.readdirSync(path.join(ctx.projectRoot, dir));
   } catch {
@@ -162,15 +165,14 @@ function scriptBody(text: string): string {
 
 function lefthookRuns(node: unknown): string[] {
   const out: string[] = [];
-  if (!node || typeof node !== 'object') return out;
-  const o = node as Record<string, any>;
-  if (typeof o.run === 'string') out.push(o.run);
-  if (typeof o.runner === 'string') out.push(o.runner);
+  if (!isDict(node)) return out;
+  if (typeof node['run'] === 'string') out.push(node['run']);
+  if (typeof node['runner'] === 'string') out.push(node['runner']);
   for (const key of ['commands', 'scripts']) {
-    if (o[key] && typeof o[key] === 'object') for (const v of Object.values(o[key])) out.push(...lefthookRuns(v));
+    for (const v of Object.values(dict(node[key]) ?? {})) out.push(...lefthookRuns(v));
   }
-  if (Array.isArray(o.jobs)) for (const j of o.jobs) out.push(...lefthookRuns(j));
-  if (o.group && typeof o.group === 'object') out.push(...lefthookRuns(o.group));
+  for (const j of list(node['jobs'])) out.push(...lefthookRuns(j));
+  out.push(...lefthookRuns(node['group']));
   return out;
 }
 
@@ -204,14 +206,14 @@ export function detectHookFrameworks(ctx: Ctx): HookFramework[] {
   for (const file of LEFTHOOK_FILES) {
     const text = readText(ctx, file);
     if (text === null) continue;
-    let data: any;
+    let data: unknown;
     try {
       data = YAML.load(text);
     } catch {
       continue;
     }
     const hooks = new Map<string, string[]>();
-    for (const [event, def] of Object.entries((data ?? {}) as Record<string, unknown>)) {
+    for (const [event, def] of Object.entries(dict(data) ?? {})) {
       if (GIT_HOOK_EVENTS.has(event)) hooks.set(event, lefthookRuns(def));
     }
     if (hooks.size === 0) continue;
@@ -231,11 +233,9 @@ export function detectHookFrameworks(ctx: Ctx): HookFramework[] {
     break;
   }
 
-  const sgh =
-    (pkg?.['simple-git-hooks'] as Record<string, unknown> | undefined) ??
-    readJson(ctx, '.simple-git-hooks.json') ??
-    readJson(ctx, 'simple-git-hooks.json');
-  if (sgh && typeof sgh === 'object') {
+  const sghInPackage = dict(at(pkg, 'simple-git-hooks'));
+  const sgh = sghInPackage ?? dict(readJson(ctx, '.simple-git-hooks.json') ?? readJson(ctx, 'simple-git-hooks.json'));
+  if (sgh) {
     const hooks = new Map<string, string[]>();
     for (const [event, cmd] of Object.entries(sgh)) {
       if (GIT_HOOK_EVENTS.has(event) && typeof cmd === 'string') hooks.set(event, [cmd]);
@@ -243,7 +243,7 @@ export function detectHookFrameworks(ctx: Ctx): HookFramework[] {
     if (hooks.size > 0) {
       out.push({
         id: 'simple-git-hooks',
-        declaredIn: pkg?.['simple-git-hooks'] ? 'package.json simple-git-hooks' : 'simple-git-hooks.json',
+        declaredIn: sghInPackage ? 'package.json simple-git-hooks' : 'simple-git-hooks.json',
         hooks,
         installedBy: isPackagePresent('simple-git-hooks', 'npm', ctx)
           ? 'simple-git-hooks package (its postinstall)'
@@ -255,10 +255,10 @@ export function detectHookFrameworks(ctx: Ctx): HookFramework[] {
     }
   }
 
-  const extra = composer?.extra?.hooks;
-  if (extra && typeof extra === 'object') {
+  const extra = dict(at(composer, 'extra', 'hooks'));
+  if (extra) {
     const hooks = new Map<string, string[]>();
-    for (const [event, cmd] of Object.entries(extra as Record<string, unknown>)) {
+    for (const [event, cmd] of Object.entries(extra)) {
       if (!GIT_HOOK_EVENTS.has(event)) continue;
       const list = (Array.isArray(cmd) ? cmd : [cmd]).filter((c): c is string => typeof c === 'string');
       if (list.length > 0) hooks.set(event, list);
@@ -297,11 +297,11 @@ export function detectHookFrameworks(ctx: Ctx): HookFramework[] {
 }
 
 /** What a clone runs to switch hooks on: the install step of the project's package manager. */
-export function installCommand(fw: HookFramework, ctx: Ctx): string {
+function installCommand(fw: HookFramework, ctx: Ctx): string {
   if (fw.installedBy === null) return fw.manualInstall;
   if (fw.installedBy.startsWith('composer')) return 'composer install';
   if (fw.installedBy === 'mise task up') return 'vdx up';
-  const pm = readJson(ctx, 'package.json')?.packageManager;
+  const pm = at(readJson(ctx, 'package.json'), 'packageManager');
   if (typeof pm === 'string' && pm.startsWith('pnpm')) return 'pnpm install';
   if ((typeof pm === 'string' && pm.startsWith('yarn')) || fileExists(ctx, 'yarn.lock')) return 'yarn install';
   if (fileExists(ctx, 'pnpm-lock.yaml')) return 'pnpm install';