@lesliechan721/aw 0.1.0-beta.23 → 0.1.0-beta.25

package/dist/src/index.js

@@ -3,7 +3,7 @@ import fs from 'node:fs';
 import path from 'node:path';
 import { spawn } from 'node:child_process';
 import { fileURLToPath } from 'node:url';
-import { Command, CommanderError, Option } from 'commander';
+import { Command, CommanderError, InvalidArgumentError, Option } from 'commander';
 import { doctorEvalProviders } from '@lesliechan721/agent-skill-eval';
 import { initCommand } from './commands/init.js';
 import { resolveCatalogWorkspaceRoot } from './commands/catalog-workspace-resolution.js';
@@ -36,7 +36,8 @@ import { withAwHomeMutationLock, } from './utils/aw-home-mutation-lock.js';
 import { installPortableSkills, materializePortableSkillsToDirectory, PortableCommandError, renderPortableReview, } from './portable/commands.js';
 import { SkillsCliError } from './portable/skills-cli.js';
 import { computeReviewComponentDigest, listReviewRecords, parseReviewAttachment, publishReviewRecord, showReviewRecord, verifyReviewRecord, } from './commands/review-publication.js';
-import { createReviewInspectionFailure, inspectReviewTarget, } from './commands/review-inspection.js';
+import { createReviewInspectionFailure, getReviewInspectionExitCode, inspectReviewTarget, } from './commands/review-inspection.js';
+import { prepareReviewTarget, } from './commands/review-preparation.js';
 import { resolveRepoRoot } from './utils/state-paths.js';
 import { runCatalogBuildCommand, runCatalogInitCommand, runCatalogValidateCommand, runCatalogVersionCommand, runPluginBuildCommand, runPluginTestCommand, } from './authoring/workspace-commands.js';
 import { runCatalogPublishApplyCommand, runCatalogPublishPlanCommand, } from './commands/catalog-publish.js';
@@ -55,6 +56,62 @@ function createDefaultIo() {
 function writeLine(write, line) {
     write(`${line}\n`);
 }
+function strictValueOption(flags, description, choices) {
+    let seen = false;
+    const option = new Option(flags, description);
+    return option.argParser((value) => {
+        if (seen)
+            throw new InvalidArgumentError(`${option.long} can be specified only once.`);
+        if (choices && !choices.includes(value)) {
+            throw new InvalidArgumentError(`${option.long} must be one of: ${choices.join(', ')}.`);
+        }
+        seen = true;
+        return value;
+    });
+}
+function collectUniqueOption(value, previous) {
+    return previous.includes(value) ? previous : [...previous, value];
+}
+function renderReviewSkills(write, skills, issues) {
+    for (const skill of skills) {
+        writeLine(write, `${skill.skill}\t${skill.profile.platform}/${skill.profile.scope}\t${skill.state}\t${skill.finalPath ?? '-'}`);
+    }
+    for (const issue of issues)
+        writeLine(write, `${issue.code}: ${issue.message}`);
+}
+function renderReviewInspection(write, receipt) {
+    writeLine(write, `Review inspection (${receipt.view}): ${receipt.ok ? 'ok' : 'failed'}`);
+    if (receipt.view === 'review') {
+        renderReviewSkills(write, receipt.skills, receipt.issues);
+        return;
+    }
+    for (const issue of receipt.issues)
+        writeLine(write, `${issue.code}: ${issue.message}`);
+}
+function renderReviewPreparation(write, receipt) {
+    writeLine(write, `Review preparation: ${receipt.ok ? 'complete' : 'failed'}`);
+    if (receipt.sourceBindingPath)
+        writeLine(write, `Source binding: ${receipt.sourceBindingPath}`);
+    renderReviewSkills(write, receipt.skills, receipt.issues);
+}
+function requestedReviewView(argv) {
+    const inline = argv.find((token) => token.startsWith('--view='));
+    if (inline === '--view=release')
+        return 'release';
+    const index = argv.indexOf('--view');
+    return index >= 0 && argv[index + 1] === 'release' ? 'release' : 'review';
+}
+function createReviewPreparationFailure(message, code = 'invalidArguments') {
+    return {
+        schemaVersion: 1,
+        command: 'review:prepare',
+        ok: false,
+        target: null,
+        sourceBindingPath: null,
+        skills: [],
+        issues: [{ code, path: null, message }],
+    };
+}
 function writeWarnings(write, warnings, options = {}) {
     for (const warning of formatWarnings(warnings, options)) {
         writeLine(write, warning);
@@ -1192,10 +1249,15 @@ export function buildProgram(options = {}) {
     review
         .command('inspect')
         .description('Inspect current AW Review source and materialization facts')
-        .argument('<target>', 'canonical Catalog Workspace or Plugin path')
-        .option('--workspace-root <root>', 'explicit Catalog Workspace root')
-        .option('--materialization <receipt>', 'validate one aw-source-binding receipt', (value, previous) => [...previous, value], [])
-        .requiredOption('--json', 'emit one Review inspection JSON object')
+        .argument('<target>', 'Catalog Workspace, Plugin, or source Skill path')
+        .addOption(strictValueOption('--workspace-root <root>', 'explicit Catalog Workspace root'))
+        .option('--skill <name>', 'select one declared Skill', collectUniqueOption, [])
+        .addOption(strictValueOption('--platform <platform>', 'select codex, claude, or agent-skills', ['codex', 'claude', 'agent-skills']))
+        .addOption(strictValueOption('--scope <scope>', 'select user or project scope', ['user', 'project']))
+        .addOption(strictValueOption('--config <file>', 'review config JSON file'))
+        .option('--materialization <receipt>', 'validate one aw-source-binding receipt', collectUniqueOption, [])
+        .addOption(strictValueOption('--view <view>', 'select review or release view', ['review', 'release']).default('review'))
+        .option('--json', 'emit one Review inspection JSON object', false)
         .action(async (target, rawOptions) => {
         try {
             const receipt = await inspectReviewTarget({
@@ -1204,17 +1266,61 @@ export function buildProgram(options = {}) {
                 ...(rawOptions.workspaceRoot === undefined
                     ? {}
                     : { workspaceRoot: rawOptions.workspaceRoot }),
-                materializations: rawOptions.materialization.map((entry) => path.resolve(cwd, entry)),
+                skills: rawOptions.skill,
+                ...(rawOptions.platform === undefined ? {} : { platform: rawOptions.platform }),
+                ...(rawOptions.scope === undefined ? {} : { scope: rawOptions.scope }),
+                ...(rawOptions.config === undefined ? {} : { configFile: rawOptions.config }),
+                materializations: [...new Set(rawOptions.materialization
+                        .map((entry) => path.resolve(cwd, entry)))],
+                view: rawOptions.view,
                 env,
             });
-            writeJsonReceipt(io.stdout, receipt);
-            commandExitState.exitCode = receipt.ok ? 0 : 1;
+            if (rawOptions.json)
+                writeJsonReceipt(io.stdout, receipt);
+            else
+                renderReviewInspection(io.stdout, receipt);
+            commandExitState.exitCode = 0;
         }
         catch (error) {
-            writeJsonReceipt(io.stdout, createReviewInspectionFailure(error));
-            commandExitState.exitCode = 1;
+            const receipt = createReviewInspectionFailure(error, rawOptions.view);
+            if (rawOptions.json)
+                writeJsonReceipt(io.stdout, receipt);
+            else
+                renderReviewInspection(io.stdout, receipt);
+            commandExitState.exitCode = getReviewInspectionExitCode(error);
         }
     });
+    review
+        .command('prepare')
+        .description('Prepare deterministic Skill finals for AW Review')
+        .argument('<target>', 'Plugin or source Skill path')
+        .addOption(strictValueOption('--workspace-root <root>', 'explicit Catalog Workspace root'))
+        .option('--skill <name>', 'select one declared Skill', collectUniqueOption, [])
+        .addOption(strictValueOption('--platform <platform>', 'select codex, claude, or agent-skills', ['codex', 'claude', 'agent-skills']).makeOptionMandatory())
+        .addOption(strictValueOption('--scope <scope>', 'select user or project scope', ['user', 'project']).makeOptionMandatory())
+        .addOption(strictValueOption('--config <file>', 'review config JSON file'))
+        .addOption(strictValueOption('--out <new-dir>', 'new output directory').makeOptionMandatory())
+        .option('--json', 'emit one Review preparation JSON object', false)
+        .action(async (target, rawOptions) => {
+        const result = await prepareReviewTarget({
+            cwd,
+            target,
+            ...(rawOptions.workspaceRoot === undefined
+                ? {}
+                : { workspaceRoot: rawOptions.workspaceRoot }),
+            skills: rawOptions.skill,
+            platform: rawOptions.platform,
+            scope: rawOptions.scope,
+            ...(rawOptions.config === undefined ? {} : { configFile: rawOptions.config }),
+            out: rawOptions.out,
+            env,
+        });
+        if (rawOptions.json)
+            writeJsonReceipt(io.stdout, result.receipt);
+        else
+            renderReviewPreparation(io.stdout, result.receipt);
+        commandExitState.exitCode = result.exitCode;
+    });
     review
         .command('digest')
         .description('Compute one canonical workflow component digest')
@@ -3226,10 +3332,14 @@ export async function runCli(argv, options = {}) {
         ...createDefaultIo(),
         ...options.io,
     };
+    const reviewCommand = argv[0] === 'review' && ['inspect', 'prepare'].includes(argv[1] ?? '')
+        ? argv[1]
+        : null;
+    const jsonReviewCommand = reviewCommand !== null && argv.includes('--json');
     const jsonAuthoringCommand = argv.includes('--json')
         && ((argv[0] === 'catalog' && ['init', 'context', 'version', 'build', 'validate', 'status', 'publish'].includes(argv[1] ?? ''))
             || (argv[0] === 'plugin' && ['build', 'test', 'validate', 'status', 'remove'].includes(argv[1] ?? ''))
-            || (argv[0] === 'review' && argv[1] === 'inspect'))
+            || reviewCommand !== null)
         ? `${argv[0]}:${argv[1]}`
         : null;
     const jsonCatalogDevCommand = argv.includes('--json')
@@ -3363,7 +3473,7 @@ export async function runCli(argv, options = {}) {
         }
         let catalogPublicationActionStarted = false;
         let configMutationActionStarted = false;
-        const programIo = jsonCatalogDevCommand || jsonWriteRequest
+        const programIo = jsonCatalogDevCommand || jsonWriteRequest || jsonReviewCommand
             ? { ...io, stderr: () => undefined }
             : jsonConfigMutation
                 ? {
@@ -3407,6 +3517,17 @@ export async function runCli(argv, options = {}) {
         return program.__awCommandExitState?.exitCode ?? 0;
     }
     catch (error) {
+        if (reviewCommand && error instanceof CommanderError) {
+            if (error.exitCode === 0)
+                return 0;
+            if (jsonReviewCommand) {
+                const receipt = reviewCommand === 'inspect'
+                    ? createReviewInspectionFailure(new Error(error.message), requestedReviewView(argv), 'invalidArguments')
+                    : createReviewPreparationFailure(error.message);
+                writeJsonReceipt(io.stdout, receipt);
+            }
+            return 2;
+        }
         if (jsonCatalogDevCommand) {
             const workspaceRootIndex = argv.findIndex((token) => token === '--workspace-root');
             const workspaceRootOption = argv.find((token) => token.startsWith('--workspace-root='));
@@ -3491,7 +3612,10 @@ export async function runCli(argv, options = {}) {
                 });
             }
             else if (jsonAuthoringCommand === 'review:inspect') {
-                writeJsonReceipt(io.stdout, createReviewInspectionFailure(error));
+                writeJsonReceipt(io.stdout, createReviewInspectionFailure(error, requestedReviewView(argv)));
+            }
+            else if (jsonAuthoringCommand === 'review:prepare') {
+                writeJsonReceipt(io.stdout, createReviewPreparationFailure(message, 'reviewPreparationFailed'));
             }
             else {
                 writeJsonReceipt(io.stdout, {
@@ -3504,12 +3628,14 @@ export async function runCli(argv, options = {}) {
         }
         if (jsonAuthoringCommand && error instanceof CommanderError) {
             writeJsonReceipt(io.stdout, jsonAuthoringCommand === 'review:inspect'
-                ? createReviewInspectionFailure(error, 'invalidArguments')
-                : {
-                    command: jsonAuthoringCommand,
-                    ok: false,
-                    errors: [{ code: 'invalidArguments', message: error.message }],
-                });
+                ? createReviewInspectionFailure(new Error(error.message), requestedReviewView(argv), 'invalidArguments')
+                : jsonAuthoringCommand === 'review:prepare'
+                    ? createReviewPreparationFailure(error.message)
+                    : {
+                        command: jsonAuthoringCommand,
+                        ok: false,
+                        errors: [{ code: 'invalidArguments', message: error.message }],
+                    });
         }
         return handleCliError(error, io.stderr);
     }