Make the bake-off-validated Workflow substrate (Arm B) reachable behind an opt-in --workflow flag for /trekreview. Default Phase 5-6 path stays prose to preserve the lower portability floor; --workflow raises the consumer floor to Claude Code 2.1.154+ (the Workflow tool). - arg-parser: --workflow added to trekreview boolean flags - commands/trekreview.md: flag row + Phase 5 substrate-routing gate + new section 'Phase 5-6 via the Workflow substrate' (invocation contract, S10 gotchas, bake-off citation, auto/bypass residual as Known limitation) - docs/command-modes.md: --workflow row in /trekreview table - routes to existing scripts/trekreview-armB.workflow.mjs (byte-identical to the S10 part-B POSITIVE build); integration is pure routing, no script change TDD: 8 new tests (arg-parser flag recognition + combine; command/doc prose pins for route, opt-in posture, 2.1.154+ floor, bake-off evidence). Suite 662 -> 670 (668 pass / 2 skip / 0 fail). plugin validate clean modulo known root-CLAUDE.md warning. Resolves W1-narrow-wins-plan.md S11.
127 lines
3.2 KiB
JavaScript
127 lines
3.2 KiB
JavaScript
// lib/parsers/arg-parser.mjs
|
|
// Parse $ARGUMENTS strings for the four voyage commands.
|
|
//
|
|
// Each command has its own valid-flag set; passing flags from another command
|
|
// produces an `unknown_flags` array but does not error — the caller decides.
|
|
|
|
const FLAG_SCHEMA = {
|
|
trekbrief: {
|
|
boolean: ['--quick', '--fg'],
|
|
valued: ['--profile'],
|
|
aliases: {},
|
|
},
|
|
trekresearch: {
|
|
boolean: ['--quick', '--local', '--external', '--fg'],
|
|
valued: ['--project', '--profile'],
|
|
aliases: {},
|
|
},
|
|
trekplan: {
|
|
boolean: ['--quick', '--fg'],
|
|
valued: ['--project', '--brief', '--export', '--decompose', '--profile'],
|
|
multi: ['--research'],
|
|
aliases: {},
|
|
},
|
|
trekexecute: {
|
|
boolean: ['--resume', '--dry-run', '--validate', '--fg'],
|
|
valued: ['--project', '--step', '--session', '--profile'],
|
|
aliases: {},
|
|
},
|
|
trekreview: {
|
|
boolean: ['--quick', '--fg', '--dry-run', '--validate', '--workflow'],
|
|
valued: ['--project', '--since', '--profile'],
|
|
aliases: {},
|
|
},
|
|
trekcontinue: {
|
|
boolean: ['--help', '--cleanup', '--confirm', '--dry-run'],
|
|
valued: ['--profile'],
|
|
aliases: {},
|
|
},
|
|
};
|
|
|
|
/**
|
|
* @param {string} argString Raw $ARGUMENTS as the command sees it.
|
|
* @param {keyof FLAG_SCHEMA} command
|
|
* @returns {{
|
|
* command: string,
|
|
* flags: Record<string, true | string | string[]>,
|
|
* positional: string[],
|
|
* unknown: string[],
|
|
* errors: Array<{code: string, message: string}>,
|
|
* }}
|
|
*/
|
|
export function parseArgs(argString, command) {
|
|
const schema = FLAG_SCHEMA[command];
|
|
if (!schema) {
|
|
return {
|
|
command,
|
|
flags: {},
|
|
positional: [],
|
|
unknown: [],
|
|
errors: [{ code: 'ARG_UNKNOWN_COMMAND', message: `Unknown command: ${command}` }],
|
|
};
|
|
}
|
|
|
|
const tokens = tokenize(argString);
|
|
const flags = {};
|
|
const positional = [];
|
|
const unknown = [];
|
|
const errors = [];
|
|
|
|
for (let i = 0; i < tokens.length; i++) {
|
|
const tok = tokens[i];
|
|
|
|
if (!tok.startsWith('--')) {
|
|
positional.push(tok);
|
|
continue;
|
|
}
|
|
|
|
if (schema.boolean.includes(tok)) {
|
|
flags[tok] = true;
|
|
continue;
|
|
}
|
|
|
|
if (schema.valued.includes(tok)) {
|
|
const next = tokens[i + 1];
|
|
if (next === undefined || next.startsWith('--')) {
|
|
errors.push({ code: 'ARG_MISSING_VALUE', message: `Flag ${tok} requires a value` });
|
|
} else {
|
|
flags[tok] = next;
|
|
i++;
|
|
}
|
|
continue;
|
|
}
|
|
|
|
if (schema.multi && schema.multi.includes(tok)) {
|
|
const collected = [];
|
|
while (i + 1 < tokens.length && !tokens[i + 1].startsWith('--')) {
|
|
collected.push(tokens[i + 1]);
|
|
i++;
|
|
}
|
|
if (collected.length === 0) {
|
|
errors.push({ code: 'ARG_MISSING_VALUE', message: `Flag ${tok} requires at least one value` });
|
|
} else {
|
|
flags[tok] = collected;
|
|
}
|
|
continue;
|
|
}
|
|
|
|
unknown.push(tok);
|
|
}
|
|
|
|
return { command, flags, positional, unknown, errors };
|
|
}
|
|
|
|
function tokenize(s) {
|
|
if (typeof s !== 'string') return [];
|
|
const trimmed = s.trim();
|
|
if (trimmed === '') return [];
|
|
const out = [];
|
|
const re = /"([^"]*)"|'([^']*)'|(\S+)/g;
|
|
let m;
|
|
while ((m = re.exec(trimmed)) !== null) {
|
|
out.push(m[1] !== undefined ? m[1] : m[2] !== undefined ? m[2] : m[3]);
|
|
}
|
|
return out;
|
|
}
|
|
|
|
export { FLAG_SCHEMA };
|