@vodmal/vdx-cli 0.24.0 → 0.26.0

package/src/facts.ts

@@ -2,21 +2,47 @@ 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;
   stack: string;
   cache: Map<string, unknown>;
+  /**
+   * Meta repo: the other half, where a file missing here is looked for — the repo root
+   * for an axis evaluated in the primary subpackage (`.github/` lives there), the
+   * subpackage for an axis evaluated in a root without a stack manifest of its own.
+   */
+  fallbackRoot?: string;
+}
+
+/** Absolute path of `rel`: here, else in `fallbackRoot` when it is there. */
+function locate(ctx: Ctx, rel: string): string {
+  const here = path.join(ctx.projectRoot, rel);
+  if (ctx.fallbackRoot === undefined || fs.existsSync(here)) return here;
+  const there = path.join(ctx.fallbackRoot, rel);
+  return fs.existsSync(there) ? there : here;
+}
+
+/** The same directory without the fallback: a project's tasks are the ones its own runner sees. */
+function ownCtx(ctx: Ctx): Ctx {
+  if (ctx.fallbackRoot === undefined) return ctx;
+  let own = ctx.cache.get('ctx:own') as Ctx | undefined;
+  if (!own) {
+    own = { projectRoot: ctx.projectRoot, stack: ctx.stack, cache: new Map() };
+    ctx.cache.set('ctx:own', own);
+  }
+  return own;
 }
 
 export function fileExists(ctx: Ctx, rel: string): boolean {
-  return fs.existsSync(path.join(ctx.projectRoot, rel));
+  return fs.existsSync(locate(ctx, rel));
 }
 
 export function readText(ctx: Ctx, rel: string): string | null {
   const key = `file:${rel}`;
   if (ctx.cache.has(key)) return ctx.cache.get(key) as string | null;
-  const abs = path.join(ctx.projectRoot, rel);
+  const abs = locate(ctx, rel);
   try {
     const t = fs.readFileSync(abs, 'utf8');
     ctx.cache.set(key, t);
@@ -27,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);
@@ -36,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 {
@@ -45,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);
@@ -63,7 +89,7 @@ export function readToml(ctx: Ctx, rel: string): any {
   }
 }
 
-export function readYaml(ctx: Ctx, rel: string): any {
+export 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);
@@ -82,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);
@@ -97,44 +123,28 @@ 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('.'));
 }
 
-/** Aggregated task names from mise.toml, composer scripts, npm scripts, Makefile. */
+/**
+ * Aggregated task names from mise.toml, composer scripts, npm scripts, Makefile.
+ * Without the fallback: `vdx test` here runs this directory's tasks, not the other half's.
+ */
 export function listAllTasks(ctx: Ctx): Set<string> {
   const key = 'tasks:all';
   if (ctx.cache.has(key)) return ctx.cache.get(key) as Set<string>;
   const tasks = new Set<string>();
+  const own = ownCtx(ctx);
 
-  const miseT = readToml(ctx, '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(ctx, 'composer.json');
-  if (composer?.scripts && typeof composer.scripts === 'object') {
-    for (const k of Object.keys(composer.scripts)) tasks.add(k);
-  }
-
-  const pkg = readJson(ctx, '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(ctx, '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(ctx, 'Makefile');
+  const make = readText(own, 'Makefile');
   if (make) {
     const lines = make.split('\n');
     for (const line of lines) {
@@ -151,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;
@@ -176,7 +182,7 @@ export function expandPaths(paths: string[], ctx: Ctx): string[] {
       if (m && m[1] && m[2]) {
         const dir = m[1];
         const ext = m[2];
-        const abs = path.join(ctx.projectRoot, dir);
+        const abs = locate(ctx, dir);
         try {
           const files = fs.readdirSync(abs);
           for (const f of files) {
@@ -193,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]> = [
@@ -224,8 +228,19 @@ export function stackForDir(dir: string): string | null {
 }
 
 const SUBPACKAGE_SCAN_IGNORE = new Set([
-  'node_modules', 'vendor', 'dist', 'build', 'out', 'target',
-  '.git', '.next', '__pycache__', '.venv', 'venv', '.cache', 'coverage',
+  'node_modules',
+  'vendor',
+  'dist',
+  'build',
+  'out',
+  'target',
+  '.git',
+  '.next',
+  '__pycache__',
+  '.venv',
+  'venv',
+  '.cache',
+  'coverage',
 ]);
 
 export interface SubPackage {

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;
   });
 }
@@ -83,34 +84,41 @@ function truthy(v: unknown): boolean {
 }
 
 export function parseWorkflow(file: string, text: string): GhaWorkflow | null {
-  let data: any;
+  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,9 +126,12 @@ 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).filter((f) => /\.ya?ml$/.test(f)).sort();
+    files = fs
+      .readdirSync(dir)
+      .filter((f) => /\.ya?ml$/.test(f))
+      .sort();
   } catch {
     files = [];
   }

package/src/hooks.ts

@@ -16,6 +16,7 @@ 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';
 
@@ -36,11 +37,29 @@ export interface HookFramework {
 }
 
 export const GIT_HOOK_EVENTS = new Set([
-  'applypatch-msg', 'pre-applypatch', 'post-applypatch', 'pre-commit', 'pre-merge-commit',
-  'prepare-commit-msg', 'commit-msg', 'post-commit', 'pre-rebase', 'post-checkout', 'post-merge',
-  'pre-push', 'pre-auto-gc', 'post-rewrite', 'reference-transaction', 'push-to-checkout',
-  'post-index-change', 'sendemail-validate', 'fsmonitor-watchman', 'p4-changelist',
-  'p4-prepare-changelist', 'p4-post-changelist', 'p4-pre-submit',
+  'applypatch-msg',
+  'pre-applypatch',
+  'post-applypatch',
+  'pre-commit',
+  'pre-merge-commit',
+  'prepare-commit-msg',
+  'commit-msg',
+  'post-commit',
+  'pre-rebase',
+  'post-checkout',
+  'post-merge',
+  'pre-push',
+  'pre-auto-gc',
+  'post-rewrite',
+  'reference-transaction',
+  'push-to-checkout',
+  'post-index-change',
+  'sendemail-validate',
+  'fsmonitor-watchman',
+  'p4-changelist',
+  'p4-prepare-changelist',
+  'p4-post-changelist',
+  'p4-pre-submit',
 ]);
 
 /** Install steps a clone runs anyway: npm-family lifecycle scripts, composer's, the vdx `up` task. */
@@ -51,7 +70,7 @@ 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;
+  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;
@@ -61,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;
@@ -117,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 {
@@ -144,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;
 }
 
@@ -173,7 +193,10 @@ export function detectHookFrameworks(ctx: Ctx): HookFramework[] {
       id: 'husky',
       declaredIn: '.husky/',
       hooks: husky,
-      installedBy: findStep(steps.filter((s) => s.where.startsWith('package.json') || s.where === 'mise task up'), /(^|[\s;&|(])husky(\s+install)?(?=$|[\s;&|)])/m),
+      installedBy: findStep(
+        steps.filter((s) => s.where.startsWith('package.json') || s.where === 'mise task up'),
+        /(^|[\s;&|(])husky(\s+install)?(?=$|[\s;&|)])/m,
+      ),
       manualInstall: 'npx husky',
       hooksPath: '.husky/_',
       copies: false,
@@ -183,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;
@@ -210,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]);
@@ -222,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)'
@@ -234,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);
@@ -247,7 +268,10 @@ export function detectHookFrameworks(ctx: Ctx): HookFramework[] {
         id: 'cghooks',
         declaredIn: 'composer.json extra.hooks',
         hooks,
-        installedBy: findStep(steps.filter((s) => s.where.startsWith('composer') || s.where === 'mise task up'), /\bcghooks\s+(add|update)\b/),
+        installedBy: findStep(
+          steps.filter((s) => s.where.startsWith('composer') || s.where === 'mise task up'),
+          /\bcghooks\s+(add|update)\b/,
+        ),
         manualInstall: 'vendor/bin/cghooks update',
         hooksPath: null,
         copies: true,
@@ -277,7 +301,7 @@ export 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';
@@ -353,7 +377,13 @@ export function cloneHookStates(root: string, ctx: Ctx, frameworks: HookFramewor
     }
     if (fw.id === 'lefthook') {
       const foreign = events.filter((e) => !/lefthook/.test(readFile(path.join(dir, e))));
-      if (foreign.length > 0) return { framework: fw, enabled: false, problem: `${foreign.join(', ')} in ${path.relative(root, dir)} is not lefthook's`, remedy };
+      if (foreign.length > 0)
+        return {
+          framework: fw,
+          enabled: false,
+          problem: `${foreign.join(', ')} in ${path.relative(root, dir)} is not lefthook's`,
+          remedy,
+        };
     }
     if (fw.copies) {
       const stale = events.filter((e) => {
@@ -361,7 +391,12 @@ export function cloneHookStates(root: string, ctx: Ctx, frameworks: HookFramewor
         return fw.hooks.get(e)!.some((cmd) => !body.includes(cmd.trim()));
       });
       if (stale.length > 0) {
-        return { framework: fw, enabled: false, problem: `installed ${stale.join(', ')} older than ${fw.declaredIn}`, remedy };
+        return {
+          framework: fw,
+          enabled: false,
+          problem: `installed ${stale.join(', ')} older than ${fw.declaredIn}`,
+          remedy,
+        };
       }
     }
     return { framework: fw, enabled: true };
@@ -386,8 +421,7 @@ function readFile(file: string): string {
  * anywhere in a script says nothing about whether the hook still does its job,
  * while an unset variable inside the guard means the guarded block never runs.
  */
-const GATING_VAR_RE =
-  /\[\s+-[nxfse]\s+"\$\{([A-Z][A-Z0-9_]*)(?::-[^}]*)?\}"|\[\s+-[nxfse]\s+"\$([A-Z][A-Z0-9_]*)"/g;
+const GATING_VAR_RE = /\[\s+-[nxfse]\s+"\$\{([A-Z][A-Z0-9_]*)(?::-[^}]*)?\}"|\[\s+-[nxfse]\s+"\$([A-Z][A-Z0-9_]*)"/g;
 
 export function unsetGatingVars(commands: string[], env: NodeJS.ProcessEnv = process.env): string[] {
   const out = new Set<string>();
@@ -406,7 +440,8 @@ export function unsetGatingVars(commands: string[], env: NodeJS.ProcessEnv = pro
  * Only what is run: a path in an argument or an `echo` says nothing about
  * whether the hook works here.
  */
-const MACHINE_PROGRAM_RE = /^(?:"((?:\$HOME|\$\{HOME\}|~|\/Users|\/home|\/Volumes)\/[^"]+)"|((?:\$HOME|\$\{HOME\}|~|\/Users|\/home|\/Volumes)\/[^\s"';|&)]+))/;
+const MACHINE_PROGRAM_RE =
+  /^(?:"((?:\$HOME|\$\{HOME\}|~|\/Users|\/home|\/Volumes)\/[^"]+)"|((?:\$HOME|\$\{HOME\}|~|\/Users|\/home|\/Volumes)\/[^\s"';|&)]+))/;
 
 export function missingMachinePaths(commands: string[], home: string = os.homedir()): string[] {
   const out = new Set<string>();