rubrkit 0.1.0 → 0.3.0

package/src/api.js

@@ -1,101 +1,108 @@
-import { exitCodeForApiFailure, RubrkitCliError } from './errors.js';
-
-export class RubrkitApiClient {
-    /**
-     * @param {{ apiUrl: string, apiKey: string, fetchImpl?: typeof fetch }} options
-     */
-    constructor({ apiUrl, apiKey, fetchImpl = globalThis.fetch }) {
-        this.apiUrl = apiUrl.replace(/\/+$/, '');
-        this.apiKey = apiKey;
-        this.fetchImpl = fetchImpl;
-    }
-
-    async listArtifactBundles() {
-        const data = await this.request('/artifact-bundles?status=active&limit=100');
-        return Array.isArray(data.artifactBundles) ? data.artifactBundles : [];
-    }
-
-    /**
-     * @param {string} artifactBundleId
-     */
-    async getArtifactBundle(artifactBundleId) {
-        const data = await this.request(`/artifact-bundles/${encodeURIComponent(artifactBundleId)}`);
-        return data;
-    }
-
-    /**
-     * @param {string} artifactBundleId
-     */
-    async listFiles(artifactBundleId) {
-        const data = await this.request(`/artifact-bundles/${encodeURIComponent(artifactBundleId)}/files?limit=200`);
-        return Array.isArray(data.files) ? data.files : [];
-    }
-
-    /**
-     * @param {string} artifactBundleId
-     * @param {string} fileId
-     */
-    async getFile(artifactBundleId, fileId) {
-        return this.request(`/artifact-bundles/${encodeURIComponent(artifactBundleId)}/files/${encodeURIComponent(fileId)}`);
-    }
-
-    /**
-     * @param {string} path
-     */
-    /**
-     * @param {string} path
-     * @param {{ method?: string, body?: unknown }} [options]
-     */
-    async request(path, { method = 'GET', body: requestBody = undefined } = {}) {
-        /** @type {Record<string, string>} */
-        const headers = {
-            accept: 'application/json',
-            authorization: `Bearer ${this.apiKey}`,
-            'user-agent': 'rubrkit-cli/0.0.0-local',
-        };
-
-        if (requestBody !== undefined) {
-            headers['content-type'] = 'application/json';
-        }
-
-        const response = await this.fetchImpl(`${this.apiUrl}${path}`, {
-            method,
-            headers,
-            body: requestBody === undefined ? undefined : JSON.stringify(requestBody),
-        });
-
-        const text = await response.text();
-        const body = text ? parseJson(text) : {};
-
-        if (!response.ok) {
-            const error = body && typeof body === 'object' && 'error' in body ? body.error : {};
-            const message =
-                error && typeof error === 'object' && typeof error.message === 'string'
-                    ? error.message
-                    : `Rubrkit API request failed with HTTP ${response.status}.`;
-            const code = error && typeof error === 'object' && typeof error.code === 'string' ? error.code : 'api_request_failed';
-
-            throw new RubrkitCliError(message, {
-                code,
-                exitCode: exitCodeForApiFailure(code, response.status),
-            });
-        }
-
-        return body && typeof body === 'object' && 'data' in body ? body.data : body;
-    }
-}
-
-/**
- * @param {string} text
- */
-function parseJson(text) {
-    try {
-        return JSON.parse(text);
-    } catch (error) {
-        throw new RubrkitCliError('Rubrkit API returned invalid JSON.', {
-            code: 'invalid_api_response',
-            exitCode: 1,
-            details: error,
-        });
-    }
-}
+import { exitCodeForApiFailure, RubrkitCliError } from './errors.js';
+
+export class RubrkitApiClient {
+    /**
+     * @param {{ apiUrl: string, apiKey: string, fetchImpl?: typeof fetch }} options
+     */
+    constructor({ apiUrl, apiKey, fetchImpl = globalThis.fetch }) {
+        this.apiUrl = apiUrl.replace(/\/+$/, '');
+        this.apiKey = apiKey;
+        this.fetchImpl = fetchImpl;
+    }
+
+    /**
+     * @param {{ labels?: string[] }} [options]
+     */
+    async listArtifactBundles({ labels = [] } = {}) {
+        const params = new URLSearchParams({ status: 'active', limit: '100' });
+        for (const label of labels) {
+            params.append('label', label);
+        }
+        const data = await this.request(`/artifact-bundles?${params.toString()}`);
+        return Array.isArray(data.artifactBundles) ? data.artifactBundles : [];
+    }
+
+    /**
+     * @param {string} artifactBundleId
+     */
+    async getArtifactBundle(artifactBundleId) {
+        const data = await this.request(`/artifact-bundles/${encodeURIComponent(artifactBundleId)}`);
+        return data;
+    }
+
+    /**
+     * @param {string} artifactBundleId
+     */
+    async listFiles(artifactBundleId) {
+        const data = await this.request(`/artifact-bundles/${encodeURIComponent(artifactBundleId)}/files?limit=200`);
+        return Array.isArray(data.files) ? data.files : [];
+    }
+
+    /**
+     * @param {string} artifactBundleId
+     * @param {string} fileId
+     */
+    async getFile(artifactBundleId, fileId) {
+        return this.request(`/artifact-bundles/${encodeURIComponent(artifactBundleId)}/files/${encodeURIComponent(fileId)}`);
+    }
+
+    /**
+     * @param {string} path
+     */
+    /**
+     * @param {string} path
+     * @param {{ method?: string, body?: unknown }} [options]
+     */
+    async request(path, { method = 'GET', body: requestBody = undefined } = {}) {
+        /** @type {Record<string, string>} */
+        const headers = {
+            accept: 'application/json',
+            authorization: `Bearer ${this.apiKey}`,
+            'user-agent': 'rubrkit-cli/0.0.0-local',
+        };
+
+        if (requestBody !== undefined) {
+            headers['content-type'] = 'application/json';
+        }
+
+        const response = await this.fetchImpl(`${this.apiUrl}${path}`, {
+            method,
+            headers,
+            body: requestBody === undefined ? undefined : JSON.stringify(requestBody),
+        });
+
+        const text = await response.text();
+        const body = text ? parseJson(text) : {};
+
+        if (!response.ok) {
+            const error = body && typeof body === 'object' && 'error' in body ? body.error : {};
+            const message =
+                error && typeof error === 'object' && typeof error.message === 'string'
+                    ? error.message
+                    : `Rubrkit API request failed with HTTP ${response.status}.`;
+            const code = error && typeof error === 'object' && typeof error.code === 'string' ? error.code : 'api_request_failed';
+
+            throw new RubrkitCliError(message, {
+                code,
+                exitCode: exitCodeForApiFailure(code, response.status),
+            });
+        }
+
+        return body && typeof body === 'object' && 'data' in body ? body.data : body;
+    }
+}
+
+/**
+ * @param {string} text
+ */
+function parseJson(text) {
+    try {
+        return JSON.parse(text);
+    } catch (error) {
+        throw new RubrkitCliError('Rubrkit API returned invalid JSON.', {
+            code: 'invalid_api_response',
+            exitCode: 1,
+            details: error,
+        });
+    }
+}

package/src/args.js

@@ -1,175 +1,209 @@
-import { usageError } from './errors.js';
-
-const VALUE_FLAGS = new Set([
-    'destination',
-    'agent',
-    'artifact-bundle',
-    'artifact',
-    'rubric',
-    'format',
-    'output',
-    'fail-under',
-    'fail-on',
-    'config',
-    'api-url',
-    'api-key',
-]);
-
-const BOOLEAN_FLAGS = new Set([
-    'all',
-    'yes',
-    'dry-run',
-    'force',
-    'prune',
-    'update-only',
-    'ci',
-    'local',
-    'remote',
-    'no-ai',
-    'watch',
-    'changed',
-    'help',
-    'version',
-]);
-
-const AGENTS = new Set(['auto', 'codex', 'claude', 'generic']);
-const COMMANDS = new Set(['pull', 'validate', 'test', 'audit', 'eval', 'report']);
-const FORMATS = new Set(['text', 'json', 'junit']);
-const FAIL_ON_LEVELS = new Set(['critical', 'high', 'medium', 'low']);
-
-/**
- * @param {string[]} argv
- */
-export function parseArgs(argv) {
-    const [command, ...rest] = argv;
-
-    if (!command || command === '--help' || command === '-h') {
-        return { command: command ? 'help' : 'help', selector: null, options: {} };
-    }
-
-    if (command === '--version' || command === '-v') {
-        return { command: 'version', selector: null, options: {} };
-    }
-
-    if (!COMMANDS.has(command)) {
-        throw usageError(`Unknown command "${command}". Run rubrkit --help for usage.`);
-    }
-
-    /** @type {Record<string, string | boolean>} */
-    const options = {};
-    /** @type {string[]} */
-    const positionals = [];
-
-    for (let index = 0; index < rest.length; index += 1) {
-        const arg = rest[index];
-
-        if (arg === '--') {
-            positionals.push(...rest.slice(index + 1));
-            break;
-        }
-
-        if (!arg.startsWith('-') || arg === '-') {
-            positionals.push(arg);
-            continue;
-        }
-
-        const normalized = normalizeFlag(arg);
-
-        if (normalized.alias) {
-            options[normalized.name] = true;
-            continue;
-        }
-
-        if (BOOLEAN_FLAGS.has(normalized.name)) {
-            if (normalized.value !== null) {
-                throw usageError(`Flag --${normalized.name} does not accept a value.`);
-            }
-
-            options[normalized.name] = true;
-            continue;
-        }
-
-        if (VALUE_FLAGS.has(normalized.name)) {
-            const value = normalized.value ?? rest[index + 1];
-
-            if (!value || value.startsWith('--')) {
-                throw usageError(`Flag --${normalized.name} requires a value.`);
-            }
-
-            options[normalized.name] = value;
-
-            if (normalized.value === null) {
-                index += 1;
-            }
-
-            continue;
-        }
-
-        throw usageError(`Unknown flag --${normalized.name}.`);
-    }
-
-    if (positionals.length > 1) {
-        throw usageError(`The ${command} command accepts at most one target or selector.`);
-    }
-
-    const selector = positionals[0] ?? null;
-
-    if (selector === 'all') {
-        options.all = true;
-    }
-
-    if (typeof options.agent === 'string' && !AGENTS.has(options.agent)) {
-        throw usageError('--agent must be one of auto, codex, claude, or generic.');
-    }
-
-    if (typeof options.format === 'string' && !FORMATS.has(options.format)) {
-        throw usageError('--format must be one of text, json, or junit.');
-    }
-
-    if (typeof options['fail-on'] === 'string' && !FAIL_ON_LEVELS.has(options['fail-on'])) {
-        throw usageError('--fail-on must be one of critical, high, medium, or low.');
-    }
-
-    if (options['fail-under'] !== undefined) {
-        const value = Number(options['fail-under']);
-        if (!Number.isFinite(value) || value < 0 || value > 100) {
-            throw usageError('--fail-under must be a number from 0 to 100.');
-        }
-    }
-
-    if (options.local && options.remote) {
-        throw usageError('Use either --local or --remote, not both.');
-    }
-
-    return { command, selector, options };
-}
-
-/**
- * @param {string} arg
- */
-function normalizeFlag(arg) {
-    if (arg === '-y') {
-        return { name: 'yes', value: null, alias: true };
-    }
-
-    if (arg === '-h') {
-        return { name: 'help', value: null, alias: true };
-    }
-
-    if (!arg.startsWith('--')) {
-        throw usageError(`Unknown short flag "${arg}".`);
-    }
-
-    const raw = arg.slice(2);
-    const equalsIndex = raw.indexOf('=');
-
-    if (equalsIndex === -1) {
-        return { name: raw, value: null, alias: false };
-    }
-
-    return {
-        name: raw.slice(0, equalsIndex),
-        value: raw.slice(equalsIndex + 1),
-        alias: false,
-    };
-}
+import { usageError } from './errors.js';
+
+const VALUE_FLAGS = new Set([
+    'destination',
+    'agent',
+    'artifact-bundle',
+    'audit-run-id',
+    'artifact',
+    'rubric',
+    'format',
+    'output',
+    'fail-under',
+    'fail-on',
+    'config',
+    'api-url',
+    'api-key',
+]);
+
+const BOOLEAN_FLAGS = new Set([
+    'all',
+    'yes',
+    'dry-run',
+    'force',
+    'prune',
+    'update-only',
+    'ci',
+    'local',
+    'remote',
+    'no-ai',
+    'no-cache',
+    'watch',
+    'changed',
+    'help',
+    'version',
+]);
+
+const MULTI_VALUE_FLAGS = new Set(['label']);
+
+const AGENTS = new Set(['auto', 'codex', 'claude', 'generic']);
+const COMMANDS = new Set(['pull', 'validate', 'test', 'audit', 'eval', 'audit-apply', 'report']);
+const FORMATS = new Set(['text', 'json', 'junit']);
+const FAIL_ON_LEVELS = new Set(['critical', 'high', 'medium', 'low']);
+
+/**
+ * @param {string[]} argv
+ */
+export function parseArgs(argv) {
+    const [command, ...rest] = argv;
+
+    if (!command || command === '--help' || command === '-h') {
+        return { command: command ? 'help' : 'help', selector: null, options: {} };
+    }
+
+    if (command === '--version' || command === '-v') {
+        return { command: 'version', selector: null, options: {} };
+    }
+
+    if (!COMMANDS.has(command)) {
+        throw usageError(`Unknown command "${command}". Run rubrkit --help for usage.`);
+    }
+
+    /** @type {Record<string, string | boolean | string[]>} */
+    const options = {};
+    /** @type {string[]} */
+    const positionals = [];
+
+    for (let index = 0; index < rest.length; index += 1) {
+        const arg = rest[index];
+
+        if (arg === '--') {
+            positionals.push(...rest.slice(index + 1));
+            break;
+        }
+
+        if (!arg.startsWith('-') || arg === '-') {
+            positionals.push(arg);
+            continue;
+        }
+
+        const normalized = normalizeFlag(arg);
+
+        if (normalized.alias) {
+            options[normalized.name] = true;
+            continue;
+        }
+
+        if (BOOLEAN_FLAGS.has(normalized.name)) {
+            if (normalized.value !== null) {
+                throw usageError(`Flag --${normalized.name} does not accept a value.`);
+            }
+
+            options[normalized.name] = true;
+            continue;
+        }
+
+        if (MULTI_VALUE_FLAGS.has(normalized.name)) {
+            const value = normalized.value ?? rest[index + 1];
+
+            if (!value || value.startsWith('--')) {
+                throw usageError(`Flag --${normalized.name} requires a value.`);
+            }
+
+            const parts = value
+                .split(',')
+                .map((part) => part.trim())
+                .filter(Boolean);
+            const existing = options[normalized.name];
+            const current = Array.isArray(existing) ? existing : [];
+            options[normalized.name] = [...current, ...parts];
+
+            if (normalized.value === null) {
+                index += 1;
+            }
+
+            continue;
+        }
+
+        if (VALUE_FLAGS.has(normalized.name)) {
+            const value = normalized.value ?? rest[index + 1];
+
+            if (!value || value.startsWith('--')) {
+                throw usageError(`Flag --${normalized.name} requires a value.`);
+            }
+
+            options[normalized.name] = value;
+
+            if (normalized.value === null) {
+                index += 1;
+            }
+
+            continue;
+        }
+
+        throw usageError(`Unknown flag --${normalized.name}.`);
+    }
+
+    if (positionals.length > 1) {
+        throw usageError(`The ${command} command accepts at most one target or selector.`);
+    }
+
+    const selector = positionals[0] ?? null;
+
+    if (selector === 'all') {
+        options.all = true;
+    }
+
+    if (Array.isArray(options.label)) {
+        options.label = [...new Set(options.label)];
+
+        if (options.label.length > 0 && (selector || options['artifact-bundle'] || options.artifact)) {
+            throw usageError('--label cannot be combined with a positional selector, --artifact-bundle, or --artifact.');
+        }
+    }
+
+    if (typeof options.agent === 'string' && !AGENTS.has(options.agent)) {
+        throw usageError('--agent must be one of auto, codex, claude, or generic.');
+    }
+
+    if (typeof options.format === 'string' && !FORMATS.has(options.format)) {
+        throw usageError('--format must be one of text, json, or junit.');
+    }
+
+    if (typeof options['fail-on'] === 'string' && !FAIL_ON_LEVELS.has(options['fail-on'])) {
+        throw usageError('--fail-on must be one of critical, high, medium, or low.');
+    }
+
+    if (options['fail-under'] !== undefined) {
+        const value = Number(options['fail-under']);
+        if (!Number.isFinite(value) || value < 0 || value > 100) {
+            throw usageError('--fail-under must be a number from 0 to 100.');
+        }
+    }
+
+    if (options.local && options.remote) {
+        throw usageError('Use either --local or --remote, not both.');
+    }
+
+    return { command, selector, options };
+}
+
+/**
+ * @param {string} arg
+ */
+function normalizeFlag(arg) {
+    if (arg === '-y') {
+        return { name: 'yes', value: null, alias: true };
+    }
+
+    if (arg === '-h') {
+        return { name: 'help', value: null, alias: true };
+    }
+
+    if (!arg.startsWith('--')) {
+        throw usageError(`Unknown short flag "${arg}".`);
+    }
+
+    const raw = arg.slice(2);
+    const equalsIndex = raw.indexOf('=');
+
+    if (equalsIndex === -1) {
+        return { name: raw, value: null, alias: false };
+    }
+
+    return {
+        name: raw.slice(0, equalsIndex),
+        value: raw.slice(equalsIndex + 1),
+        alias: false,
+    };
+}