@mcptoolshop/style-dataset-lab 3.0.0 → 3.1.0

package/scripts/run-generate.js

@@ -9,7 +9,7 @@
  *   sdlab run generate --brief brief_2026-04-15_001 --project star-freight --seed 42
  */
 
-import { parseArgs } from '../lib/args.js';
+import { parseArgs, getProjectName, parseNumberFlag } from '../lib/args.js';
 import { getProjectRoot, getRunsDir } from '../lib/paths.js';
 import { loadCompiledBrief } from '../lib/brief-compiler.js';
 import { executeRun } from '../lib/adapters/comfyui-runner.js';
@@ -20,7 +20,7 @@ import { join } from 'node:path';
 export async function run(argv = process.argv.slice(2)) {
   const { flags } = parseArgs(argv, {
     flags: {
-      project: { type: 'string', default: 'star-freight' },
+      project: { type: 'string' },
       brief: { type: 'string' },
       seed: { type: 'string' },
       'dry-run': { type: 'boolean' },
@@ -40,7 +40,7 @@ export async function run(argv = process.argv.slice(2)) {
     return;
   }
 
-  const projectName = flags.project;
+  const projectName = flags.project || getProjectName(argv);
   const projectRoot = getProjectRoot(projectName);
 
   console.log(`\x1b[1mstyle-dataset-lab\x1b[0m run generate`);
@@ -57,7 +57,7 @@ export async function run(argv = process.argv.slice(2)) {
     projectRoot,
     projectName,
     brief,
-    baseSeed: flags.seed ? parseInt(flags.seed, 10) : undefined,
+    baseSeed: flags.seed ? parseNumberFlag('seed', flags.seed, { int: true, min: 0 }) : undefined,
     dryRun: flags['dry-run'] || false,
   });
 

package/scripts/run-list.js

@@ -8,20 +8,20 @@
  *   sdlab run list --project star-freight --json
  */
 
-import { parseArgs } from '../lib/args.js';
+import { parseArgs, getProjectName } from '../lib/args.js';
 import { getProjectRoot } from '../lib/paths.js';
 import { listRuns } from '../lib/runtime-runs.js';
 
 export async function run(argv = process.argv.slice(2)) {
   const { flags } = parseArgs(argv, {
     flags: {
-      project: { type: 'string', default: 'star-freight' },
+      project: { type: 'string' },
       json: { type: 'boolean' },
     },
     deprecated: { game: 'project' },
   });
 
-  const projectName = flags.project;
+  const projectName = flags.project || getProjectName(argv);
   const projectRoot = getProjectRoot(projectName);
   const runs = listRuns(projectRoot);
 

package/scripts/run-show.js

@@ -9,7 +9,7 @@
  *   sdlab run show run_2026-04-16_001 --project star-freight --md
  */
 
-import { parseArgs } from '../lib/args.js';
+import { parseArgs, getProjectName } from '../lib/args.js';
 import { getProjectRoot } from '../lib/paths.js';
 import { loadRun, listRuns } from '../lib/runtime-runs.js';
 import { renderRunText, renderRunMarkdown } from '../lib/run-summary.js';
@@ -17,7 +17,7 @@ import { renderRunText, renderRunMarkdown } from '../lib/run-summary.js';
 export async function run(argv = process.argv.slice(2)) {
   const { positionals, flags } = parseArgs(argv, {
     flags: {
-      project: { type: 'string', default: 'star-freight' },
+      project: { type: 'string' },
       json: { type: 'boolean' },
       md: { type: 'boolean' },
     },
@@ -25,7 +25,7 @@ export async function run(argv = process.argv.slice(2)) {
   });
 
   const runId = positionals[0];
-  const projectName = flags.project;
+  const projectName = flags.project || getProjectName(argv);
   const projectRoot = getProjectRoot(projectName);
 
   // No run ID → list runs

package/scripts/select.js

@@ -9,8 +9,9 @@
  *   sdlab select --run run_2026-04-15_001 --approve 001.png --reason "best continuity" --tags portrait,keeper
  */
 
-import { parseArgs } from '../lib/args.js';
+import { parseArgs, getProjectName } from '../lib/args.js';
 import { getProjectRoot } from '../lib/paths.js';
+import { inputError, handleCliError } from '../lib/errors.js';
 import {
   createSelectionFromRun,
   createSelectionFromBatch,
@@ -21,7 +22,7 @@ import { info } from '../lib/log.js';
 export async function run(argv = process.argv.slice(2)) {
   const { flags } = parseArgs(argv, {
     flags: {
-      project: { type: 'string', default: 'star-freight' },
+      project: { type: 'string' },
       run: { type: 'string' },
       batch: { type: 'string' },
       approve: { type: 'string' },
@@ -46,7 +47,7 @@ export async function run(argv = process.argv.slice(2)) {
     return;
   }
 
-  const projectName = flags.project;
+  const projectName = flags.project || getProjectName(argv);
   const projectRoot = getProjectRoot(projectName);
   const reason = flags.reason || undefined;
   const tags = flags.tags ? flags.tags.split(',').map(t => t.trim()) : undefined;
@@ -71,7 +72,7 @@ export async function run(argv = process.argv.slice(2)) {
     const approvedSlots = pairs.map(pair => {
       const colonIdx = pair.indexOf(':');
       if (colonIdx < 0) {
-        throw new Error(`Invalid batch approve format: "${pair}". Use slot_id:filename`);
+        throw inputError('INPUT_BAD_FORMAT', `Invalid batch approve format: "${pair}". Use slot_id:filename`);
       }
       return {
         slotId: pair.slice(0, colonIdx),

package/scripts/selection-show.js

@@ -9,7 +9,7 @@
  *   sdlab selection show selection_2026-04-16_001 --json
  */
 
-import { parseArgs } from '../lib/args.js';
+import { parseArgs, getProjectName } from '../lib/args.js';
 import { getProjectRoot } from '../lib/paths.js';
 import { loadSelection, listSelections, getSelectionsDir } from '../lib/selections.js';
 import { join } from 'node:path';
@@ -18,14 +18,14 @@ import { existsSync, readFileSync } from 'node:fs';
 export async function run(argv = process.argv.slice(2)) {
   const { flags, positionals } = parseArgs(argv, {
     flags: {
-      project: { type: 'string', default: 'star-freight' },
+      project: { type: 'string' },
       selection: { type: 'string' },
       json: { type: 'boolean' },
     },
     deprecated: { game: 'project' },
   });
 
-  const projectName = flags.project;
+  const projectName = flags.project || getProjectName(argv);
   const projectRoot = getProjectRoot(projectName);
   const selectionId = positionals[0] || flags.selection;
 

package/scripts/snapshot.js

@@ -12,30 +12,33 @@
 
 import { readFile } from 'node:fs/promises';
 import { join } from 'node:path';
-import { existsSync } from 'node:fs';
-import { getProjectName } from '../lib/args.js';
+import { parseArgs, getProjectName } from '../lib/args.js';
 import { REPO_ROOT } from '../lib/paths.js';
 import { loadSelectionProfile } from '../lib/config.js';
+import { inputError, handleCliError } from '../lib/errors.js';
 import { createSnapshot, loadSnapshot, listSnapshots, diffSnapshots } from '../lib/snapshot.js';
-import { evaluateEligibility } from '../lib/eligibility.js';
 import { categorizeExclusions } from '../lib/eligibility.js';
 
 export async function run(argv = process.argv.slice(2)) {
-  const projectName = getProjectName(argv);
-  const projectRoot = join(REPO_ROOT, 'projects', projectName);
-  const subcommand = argv.find(a => !a.startsWith('--')) || 'list';
+  const { flags, positionals } = parseArgs(argv, {
+    flags: {
+      project: { type: 'string' },
+      profile: { type: 'string' },
+      'dry-run': { type: 'boolean', default: false },
+    },
+    deprecated: { game: 'project' },
+  });
 
-  // Strip subcommand from argv for flag parsing
-  const args = argv.filter(a => a !== subcommand);
+  const projectName = flags.project || getProjectName(argv);
+  const projectRoot = join(REPO_ROOT, 'projects', projectName);
+  const subcommand = positionals[0] || 'list';
+  const dryRun = flags['dry-run'] || flags.dryRun;
 
   if (subcommand === 'create') {
-    const profileIdx = args.indexOf('--profile');
-    const profileId = profileIdx >= 0 ? args[profileIdx + 1] : null;
-    const profile = loadSelectionProfile(projectRoot, profileId);
-    const dryRun = argv.includes('--dry-run');
+    const profile = loadSelectionProfile(projectRoot, flags.profile);
 
     console.log(`\x1b[1msdlab snapshot create\x1b[0m — ${projectName}`);
-    console.log(`  Profile: ${profileId || 'training-default (built-in)'}`);
+    console.log(`  Profile: ${flags.profile || 'training-default (built-in)'}`);
     if (dryRun) console.log('  Mode: DRY RUN (no files written)');
     console.log('');
 
@@ -46,7 +49,6 @@ export async function run(argv = process.argv.slice(2)) {
     console.log(`  Excluded: ${result.excluded}`);
 
     if (!dryRun) {
-      // Read and display summary
       const summary = JSON.parse(
         await readFile(join(projectRoot, 'snapshots', result.snapshotId, 'summary.json'), 'utf-8')
       );
@@ -59,7 +61,6 @@ export async function run(argv = process.argv.slice(2)) {
         console.log(`    ${f}: ${c}`);
       }
 
-      // Read and categorize exclusions
       const excludedRaw = await readFile(
         join(projectRoot, 'snapshots', result.snapshotId, 'excluded.jsonl'), 'utf-8'
       );
@@ -83,8 +84,8 @@ export async function run(argv = process.argv.slice(2)) {
     }
 
   } else if (subcommand === 'show') {
-    const snapshotId = args.find(a => a.startsWith('snap-'));
-    if (!snapshotId) throw new Error('Usage: sdlab snapshot show <snapshot-id>');
+    const snapshotId = positionals[1] || positionals.find(a => a.startsWith('snap-'));
+    if (!snapshotId) throw inputError('INPUT_MISSING_ARGS', 'Usage: sdlab snapshot show <snapshot-id>');
 
     const manifest = await loadSnapshot(projectRoot, snapshotId);
     console.log(`\x1b[1mSnapshot\x1b[0m — ${snapshotId}\n`);
@@ -102,8 +103,8 @@ export async function run(argv = process.argv.slice(2)) {
     }
 
   } else if (subcommand === 'diff') {
-    const snapIds = args.filter(a => a.startsWith('snap-'));
-    if (snapIds.length < 2) throw new Error('Usage: sdlab snapshot diff <id-a> <id-b>');
+    const snapIds = positionals.slice(1).filter(a => a.startsWith('snap-'));
+    if (snapIds.length < 2) throw inputError('INPUT_MISSING_ARGS', 'Usage: sdlab snapshot diff <id-a> <id-b>');
 
     const diff = await diffSnapshots(projectRoot, snapIds[0], snapIds[1]);
     console.log(`\x1b[1mSnapshot diff\x1b[0m — ${snapIds[0]} vs ${snapIds[1]}\n`);
@@ -123,14 +124,11 @@ export async function run(argv = process.argv.slice(2)) {
     }
 
   } else {
-    throw new Error(`Unknown subcommand: ${subcommand}. Use: create, list, show, diff`);
+    throw inputError('INPUT_BAD_SUBCOMMAND', `Unknown subcommand: ${subcommand}. Use: create, list, show, diff`);
   }
 }
 
 // Direct execution guard
 if (process.argv[1] && (process.argv[1].endsWith('snapshot.js') || process.argv[1].endsWith('snapshot'))) {
-  run().catch((err) => {
-    console.error(err.message || err);
-    process.exit(1);
-  });
+  run().catch(handleCliError);
 }

package/scripts/split.js

@@ -10,43 +10,45 @@
  *   sdlab split audit <split-id> [--project <name>]
  */
 
-import { readFile } from 'node:fs/promises';
 import { join } from 'node:path';
-import { getProjectName } from '../lib/args.js';
+import { parseArgs, getProjectName } from '../lib/args.js';
 import { REPO_ROOT } from '../lib/paths.js';
 import { loadSplitProfile } from '../lib/config.js';
+import { inputError, handleCliError } from '../lib/errors.js';
 import { listSnapshots } from '../lib/snapshot.js';
 import { createSplit, loadSplit, loadSplitAudit, listSplits } from '../lib/split.js';
 
 export async function run(argv = process.argv.slice(2)) {
-  const projectName = getProjectName(argv);
+  const { flags, positionals } = parseArgs(argv, {
+    flags: {
+      project: { type: 'string' },
+      snapshot: { type: 'string' },
+      profile: { type: 'string' },
+      'dry-run': { type: 'boolean', default: false },
+    },
+    deprecated: { game: 'project' },
+  });
+
+  const projectName = flags.project || getProjectName(argv);
   const projectRoot = join(REPO_ROOT, 'projects', projectName);
-  const subcommand = argv.find(a => !a.startsWith('--')) || 'list';
-  const args = argv.filter(a => a !== subcommand);
+  const subcommand = positionals[0] || 'list';
+  const dryRun = flags['dry-run'] || flags.dryRun;
 
   if (subcommand === 'build') {
-    // Find snapshot ID
-    let snapshotId;
-    const snapIdx = args.indexOf('--snapshot');
-    if (snapIdx >= 0) {
-      snapshotId = args[snapIdx + 1];
-    } else {
-      // Use latest snapshot
+    let snapshotId = flags.snapshot;
+    if (!snapshotId) {
       const snapshots = await listSnapshots(projectRoot);
       if (snapshots.length === 0) {
-        throw new Error('No snapshots found. Run: sdlab snapshot create');
+        throw inputError('INPUT_NO_SNAPSHOT', 'No snapshots found. Run: sdlab snapshot create');
       }
       snapshotId = snapshots[snapshots.length - 1].id;
     }
 
-    const profileIdx = args.indexOf('--profile');
-    const profileId = profileIdx >= 0 ? args[profileIdx + 1] : null;
-    const profile = loadSplitProfile(projectRoot, profileId);
-    const dryRun = argv.includes('--dry-run');
+    const profile = loadSplitProfile(projectRoot, flags.profile);
 
     console.log(`\x1b[1msdlab split build\x1b[0m — ${projectName}`);
     console.log(`  Snapshot: ${snapshotId}`);
-    console.log(`  Profile:  ${profileId || 'balanced-default (built-in)'}`);
+    console.log(`  Profile:  ${flags.profile || 'balanced-default (built-in)'}`);
     console.log(`  Strategy: ${profile.strategy}`);
     console.log(`  Ratios:   ${profile.train_ratio}/${profile.val_ratio}/${profile.test_ratio}`);
     console.log(`  Seed:     ${profile.seed}`);
@@ -60,7 +62,6 @@ export async function run(argv = process.argv.slice(2)) {
     console.log(`  Val:   ${result.val}`);
     console.log(`  Test:  ${result.test}`);
 
-    // Show audit summary
     const audit = await loadSplitAudit(projectRoot, result.splitId);
     console.log('');
     if (audit.leakage_check.passed) {
@@ -86,8 +87,8 @@ export async function run(argv = process.argv.slice(2)) {
     }
 
   } else if (subcommand === 'show') {
-    const splitId = args.find(a => a.startsWith('split-'));
-    if (!splitId) throw new Error('Usage: sdlab split show <split-id>');
+    const splitId = positionals[1] || positionals.find(a => a.startsWith('split-'));
+    if (!splitId) throw inputError('INPUT_MISSING_ARGS', 'Usage: sdlab split show <split-id>');
 
     const manifest = await loadSplit(projectRoot, splitId);
     console.log(`\x1b[1mSplit\x1b[0m — ${splitId}\n`);
@@ -112,13 +113,12 @@ export async function run(argv = process.argv.slice(2)) {
     }
 
   } else if (subcommand === 'audit') {
-    const splitId = args.find(a => a.startsWith('split-'));
-    if (!splitId) throw new Error('Usage: sdlab split audit <split-id>');
+    const splitId = positionals[1] || positionals.find(a => a.startsWith('split-'));
+    if (!splitId) throw inputError('INPUT_MISSING_ARGS', 'Usage: sdlab split audit <split-id>');
 
     const audit = await loadSplitAudit(projectRoot, splitId);
     console.log(`\x1b[1mSplit audit\x1b[0m — ${splitId}\n`);
 
-    // Leakage check
     if (audit.leakage_check.passed) {
       console.log(`  \x1b[32m✓\x1b[0m Leakage check: PASSED`);
     } else {
@@ -128,13 +128,11 @@ export async function run(argv = process.argv.slice(2)) {
       }
     }
 
-    // Family distribution
     console.log(`\n  Families: ${audit.family_count}`);
     console.log(`    Train: ${audit.families_per_split.train}`);
     console.log(`    Val:   ${audit.families_per_split.val}`);
     console.log(`    Test:  ${audit.families_per_split.test}`);
 
-    // Lane balance
     console.log('\n  Lane balance:');
     console.log(`    ${'Lane'.padEnd(20)} ${'Total'.padStart(6)} ${'Train%'.padStart(8)} ${'Val%'.padStart(8)} ${'Test%'.padStart(8)}`);
     console.log(`    ${'─'.repeat(52)}`);
@@ -143,14 +141,11 @@ export async function run(argv = process.argv.slice(2)) {
     }
 
   } else {
-    throw new Error(`Unknown subcommand: ${subcommand}. Use: build, list, show, audit`);
+    throw inputError('INPUT_BAD_SUBCOMMAND', `Unknown subcommand: ${subcommand}. Use: build, list, show, audit`);
   }
 }
 
 // Direct execution guard
 if (process.argv[1] && (process.argv[1].endsWith('split.js') || process.argv[1].endsWith('split'))) {
-  run().catch((err) => {
-    console.error(err.message || err);
-    process.exit(1);
-  });
+  run().catch(handleCliError);
 }

package/scripts/training-manifest.js

@@ -11,8 +11,9 @@
  */
 
 import { join } from 'node:path';
-import { getProjectName } from '../lib/args.js';
+import { parseArgs, getProjectName } from '../lib/args.js';
 import { REPO_ROOT } from '../lib/paths.js';
+import { inputError, handleCliError } from '../lib/errors.js';
 import { listExports } from '../lib/export.js';
 import {
   createTrainingManifest,
@@ -22,30 +23,34 @@ import {
 } from '../lib/training-manifests.js';
 
 export async function run(argv = process.argv.slice(2)) {
-  const projectName = getProjectName(argv);
+  const { flags, positionals } = parseArgs(argv, {
+    flags: {
+      project: { type: 'string' },
+      export: { type: 'string' },
+      profile: { type: 'string' },
+      adapter: { type: 'string' },
+      'base-model': { type: 'string' },
+    },
+    deprecated: { game: 'project' },
+  });
+
+  const projectName = flags.project || getProjectName(argv);
   const projectRoot = join(REPO_ROOT, 'projects', projectName);
-  const subcommand = argv.find(a => !a.startsWith('--') && !a.startsWith('tm-')) || 'list';
-  const args = argv.filter(a => a !== subcommand);
+  const subcommand = positionals.find(a => !a.startsWith('tm-')) || 'list';
 
   if (subcommand === 'create') {
-    // Required: --export and --profile
-    const exportIdx = args.indexOf('--export');
-    if (exportIdx < 0) {
-      // Try latest export
+    let exportId = flags.export;
+    if (!exportId) {
       const exports = await listExports(projectRoot);
-      if (exports.length === 0) throw new Error('No exports found. Run: sdlab export build');
+      if (exports.length === 0) throw inputError('INPUT_NO_EXPORT', 'No exports found. Run: sdlab export build');
+      exportId = exports[exports.length - 1].id;
     }
-    const exportId = exportIdx >= 0 ? args[exportIdx + 1] : (await listExports(projectRoot)).pop().id;
 
-    const profileIdx = args.indexOf('--profile');
-    if (profileIdx < 0) throw new Error('--profile is required. Use: sdlab training-profile list');
-    const profileId = args[profileIdx + 1];
+    if (!flags.profile) throw inputError('INPUT_MISSING_FLAG', '--profile is required. Use: sdlab training-profile list');
+    const profileId = flags.profile;
 
-    const adapterIdx = args.indexOf('--adapter');
-    const adapterTarget = adapterIdx >= 0 ? args[adapterIdx + 1] : undefined;
-
-    const baseModelIdx = args.indexOf('--base-model');
-    const baseModel = baseModelIdx >= 0 ? args[baseModelIdx + 1] : undefined;
+    const adapterTarget = flags.adapter;
+    const baseModel = flags['base-model'] || flags.baseModel;
 
     console.log(`\x1b[1msdlab training-manifest create\x1b[0m — ${projectName}`);
     console.log(`  Export:  ${exportId}`);
@@ -62,8 +67,8 @@ export async function run(argv = process.argv.slice(2)) {
     console.log(`  \x1b[32m✓\x1b[0m Manifest: ${result.manifestId}`);
 
   } else if (subcommand === 'validate') {
-    const manifestId = args.find(a => a.startsWith('tm-'));
-    if (!manifestId) throw new Error('Usage: sdlab training-manifest validate <manifest-id>');
+    const manifestId = positionals.find(a => a.startsWith('tm-'));
+    if (!manifestId) throw inputError('INPUT_MISSING_ARGS', 'Usage: sdlab training-manifest validate <manifest-id>');
 
     console.log(`\x1b[1msdlab training-manifest validate\x1b[0m — ${manifestId}\n`);
     const { valid, issues } = await validateTrainingManifest(projectRoot, manifestId);
@@ -78,8 +83,8 @@ export async function run(argv = process.argv.slice(2)) {
     }
 
   } else if (subcommand === 'show') {
-    const manifestId = args.find(a => a.startsWith('tm-'));
-    if (!manifestId) throw new Error('Usage: sdlab training-manifest show <manifest-id>');
+    const manifestId = positionals.find(a => a.startsWith('tm-'));
+    if (!manifestId) throw inputError('INPUT_MISSING_ARGS', 'Usage: sdlab training-manifest show <manifest-id>');
 
     const manifest = await loadTrainingManifest(projectRoot, manifestId);
     console.log(`\x1b[1mTraining manifest\x1b[0m — ${manifestId}\n`);
@@ -110,10 +115,10 @@ export async function run(argv = process.argv.slice(2)) {
     }
 
   } else {
-    throw new Error(`Unknown subcommand: ${subcommand}. Use: create, validate, show, list`);
+    throw inputError('INPUT_BAD_SUBCOMMAND', `Unknown subcommand: ${subcommand}. Use: create, validate, show, list`);
   }
 }
 
 if (process.argv[1] && (process.argv[1].endsWith('training-manifest.js') || process.argv[1].endsWith('training-manifest'))) {
-  run().catch((err) => { console.error(err.message || err); process.exit(1); });
+  run().catch(handleCliError);
 }

package/scripts/training-package.js

@@ -10,48 +10,52 @@
  */
 
 import { join } from 'node:path';
-import { getProjectName } from '../lib/args.js';
+import { parseArgs, getProjectName } from '../lib/args.js';
 import { REPO_ROOT } from '../lib/paths.js';
+import { inputError, handleCliError } from '../lib/errors.js';
 import { listTrainingManifests } from '../lib/training-manifests.js';
 import { buildTrainingPackage, listTrainingPackages, loadTrainingPackage } from '../lib/training-packages.js';
 
 export async function run(argv = process.argv.slice(2)) {
-  const projectName = getProjectName(argv);
+  const { flags, positionals } = parseArgs(argv, {
+    flags: {
+      project: { type: 'string' },
+      manifest: { type: 'string' },
+      adapter: { type: 'string' },
+      copy: { type: 'boolean', default: false },
+    },
+    deprecated: { game: 'project' },
+  });
+
+  const projectName = flags.project || getProjectName(argv);
   const projectRoot = join(REPO_ROOT, 'projects', projectName);
-  const subcommand = argv.find(a => !a.startsWith('--') && !a.startsWith('tp-') && !a.startsWith('tm-')) || 'list';
-  const args = argv.filter(a => a !== subcommand);
+  const subcommand = positionals.find(a => !a.startsWith('tp-') && !a.startsWith('tm-')) || 'list';
 
   if (subcommand === 'build') {
-    // Find manifest
-    let manifestId;
-    const manifestIdx = args.indexOf('--manifest');
-    if (manifestIdx >= 0) {
-      manifestId = args[manifestIdx + 1];
-    } else {
+    let manifestId = flags.manifest;
+    if (!manifestId) {
       const manifests = await listTrainingManifests(projectRoot);
-      if (manifests.length === 0) throw new Error('No training manifests found. Run: sdlab training-manifest create');
+      if (manifests.length === 0) throw inputError('INPUT_NO_MANIFEST', 'No training manifests found. Run: sdlab training-manifest create');
       manifestId = manifests[manifests.length - 1].id;
     }
 
-    const adapterIdx = args.indexOf('--adapter');
-    const adapterOverride = adapterIdx >= 0 ? args[adapterIdx + 1] : undefined;
-    const copy = args.includes('--copy');
+    const adapterOverride = flags.adapter;
 
     console.log(`\x1b[1msdlab training-package build\x1b[0m — ${projectName}`);
     console.log(`  Manifest: ${manifestId}`);
     if (adapterOverride) console.log(`  Adapter:  ${adapterOverride}`);
-    console.log(`  Images:   ${copy ? 'copy' : 'symlink'}`);
+    console.log(`  Images:   ${flags.copy ? 'copy' : 'symlink'}`);
     console.log('');
 
-    const result = await buildTrainingPackage(projectRoot, manifestId, { copy, adapterOverride });
+    const result = await buildTrainingPackage(projectRoot, manifestId, { copy: flags.copy, adapterOverride });
 
     console.log(`  \x1b[32m✓\x1b[0m Package: ${result.packageId}`);
     console.log(`  Records: ${result.records}`);
     console.log(`  Images:  ${result.images}`);
 
   } else if (subcommand === 'show') {
-    const packageId = args.find(a => a.startsWith('tp-'));
-    if (!packageId) throw new Error('Usage: sdlab training-package show <package-id>');
+    const packageId = positionals.find(a => a.startsWith('tp-'));
+    if (!packageId) throw inputError('INPUT_MISSING_ARGS', 'Usage: sdlab training-package show <package-id>');
 
     const pkg = await loadTrainingPackage(projectRoot, packageId);
     console.log(`\x1b[1mTraining package\x1b[0m — ${packageId}\n`);
@@ -75,10 +79,10 @@ export async function run(argv = process.argv.slice(2)) {
     }
 
   } else {
-    throw new Error(`Unknown subcommand: ${subcommand}. Use: build, show, list`);
+    throw inputError('INPUT_BAD_SUBCOMMAND', `Unknown subcommand: ${subcommand}. Use: build, show, list`);
   }
 }
 
 if (process.argv[1] && (process.argv[1].endsWith('training-package.js') || process.argv[1].endsWith('training-package'))) {
-  run().catch((err) => { console.error(err.message || err); process.exit(1); });
+  run().catch(handleCliError);
 }

package/scripts/training-profile.js

@@ -9,15 +9,20 @@
  */
 
 import { join } from 'node:path';
-import { getProjectName } from '../lib/args.js';
+import { parseArgs, getProjectName } from '../lib/args.js';
 import { REPO_ROOT } from '../lib/paths.js';
+import { inputError, handleCliError } from '../lib/errors.js';
 import { listTrainingProfiles, loadTrainingProfile } from '../lib/training-profiles.js';
 
 export async function run(argv = process.argv.slice(2)) {
-  const projectName = getProjectName(argv);
+  const { flags, positionals } = parseArgs(argv, {
+    flags: { project: { type: 'string' } },
+    deprecated: { game: 'project' },
+  });
+
+  const projectName = flags.project || getProjectName(argv);
   const projectRoot = join(REPO_ROOT, 'projects', projectName);
-  const subcommand = argv.find(a => !a.startsWith('--')) || 'list';
-  const args = argv.filter(a => a !== subcommand);
+  const subcommand = positionals[0] || 'list';
 
   if (subcommand === 'list') {
     const profiles = await listTrainingProfiles(projectRoot);
@@ -31,8 +36,8 @@ export async function run(argv = process.argv.slice(2)) {
     }
 
   } else if (subcommand === 'show') {
-    const profileId = args.find(a => !a.startsWith('--'));
-    if (!profileId) throw new Error('Usage: sdlab training-profile show <profile-id>');
+    const profileId = positionals[1];
+    if (!profileId) throw inputError('INPUT_MISSING_ARGS', 'Usage: sdlab training-profile show <profile-id>');
 
     const profile = await loadTrainingProfile(projectRoot, profileId);
     console.log(`\x1b[1mTraining profile\x1b[0m — ${profileId}\n`);
@@ -54,10 +59,10 @@ export async function run(argv = process.argv.slice(2)) {
     }
 
   } else {
-    throw new Error(`Unknown subcommand: ${subcommand}. Use: list, show`);
+    throw inputError('INPUT_BAD_SUBCOMMAND', `Unknown subcommand: ${subcommand}. Use: list, show`);
   }
 }
 
 if (process.argv[1] && (process.argv[1].endsWith('training-profile.js') || process.argv[1].endsWith('training-profile'))) {
-  run().catch((err) => { console.error(err.message || err); process.exit(1); });
+  run().catch(handleCliError);
 }

package/scripts/workflow-list.js

@@ -15,13 +15,13 @@ import { listWorkflowProfiles } from '../lib/workflow-profiles.js';
 export async function run(argv = process.argv.slice(2)) {
   const { flags } = parseArgs(argv, {
     flags: {
-      project: { type: 'string', default: 'star-freight' },
+      project: { type: 'string' },
       json: { type: 'boolean' },
     },
     deprecated: { game: 'project' },
   });
 
-  const projectName = flags.project;
+  const projectName = flags.project || getProjectName(argv);
   const projectRoot = getProjectRoot(projectName);
   const profiles = await listWorkflowProfiles(projectRoot);
 

package/scripts/workflow-show.js

@@ -8,14 +8,14 @@
  *   sdlab workflow show character-portrait-set --json
  */
 
-import { parseArgs } from '../lib/args.js';
+import { parseArgs, getProjectName } from '../lib/args.js';
 import { getProjectRoot } from '../lib/paths.js';
 import { getWorkflowProfile } from '../lib/workflow-profiles.js';
 
 export async function run(argv = process.argv.slice(2)) {
   const { flags, positionals } = parseArgs(argv, {
     flags: {
-      project: { type: 'string', default: 'star-freight' },
+      project: { type: 'string' },
       json: { type: 'boolean' },
     },
     positionals: ['workflowId'],
@@ -28,7 +28,8 @@ export async function run(argv = process.argv.slice(2)) {
     return;
   }
 
-  const projectRoot = getProjectRoot(flags.project);
+  const projectName = flags.project || getProjectName(argv);
+  const projectRoot = getProjectRoot(projectName);
   const profile = await getWorkflowProfile(projectRoot, workflowId);
 
   if (flags.json) {