@aibridge/cli 0.8.0 → 0.10.0

package/src/commands/skill/impl.ts

@@ -0,0 +1,60 @@
+import { existsSync, readFileSync } from 'node:fs';
+import type { LocalContext } from '../../context.ts';
+import { PACKAGE_VERSION } from '../../package.ts';
+
+const TOPICS = {
+  plan: 'reference/plan.md',
+  implement: 'reference/implement.md',
+  review: 'reference/review.md',
+  subagent: 'reference/subagent.md',
+  'image-gen': 'reference/image-gen.md',
+  why: 'reference/why.md',
+} as const;
+
+export type SkillTopic = keyof typeof TOPICS;
+
+function instructionPath(relativePath: string): URL {
+  const candidates = [
+    // Built package: dist/cli.mjs -> instructions/
+    new URL(`../instructions/${relativePath}`, import.meta.url),
+    // Source tree: src/commands/skill/impl.ts -> instructions/
+    new URL(`../../../instructions/${relativePath}`, import.meta.url),
+  ];
+  const found = candidates.find(candidate => existsSync(candidate));
+  if (!found) {
+    throw new Error(`bundled instruction file is missing: ${relativePath}`);
+  }
+  return found;
+}
+
+function readInstruction(relativePath: string): string {
+  return readFileSync(instructionPath(relativePath), 'utf8').trimEnd();
+}
+
+export default function skillImpl(this: LocalContext, topic?: string): void {
+  if (topic !== undefined && !(topic in TOPICS)) {
+    this.process.stderr.write(
+      `aibridge skill: unknown topic ${JSON.stringify(topic)}; expected one of: ${Object.keys(TOPICS).join(', ')}\n`,
+    );
+    this.process.exitCode = 2;
+    return;
+  }
+
+  try {
+    const runner = `npx -y @aibridge/cli@${PACKAGE_VERSION}`;
+    const sections = [
+      `Command runner for these instructions: \`${runner}\`\nUse that exact prefix for every aibridge command below; do not substitute a global binary.`,
+      readInstruction('SKILL.md'),
+    ];
+
+    if (topic !== undefined) {
+      sections.push(readInstruction(TOPICS[topic as SkillTopic]));
+    }
+
+    this.process.stdout.write(`${sections.join('\n\n---\n\n')}\n`);
+  } catch (error) {
+    const message = error instanceof Error ? error.message : String(error);
+    this.process.stderr.write(`aibridge skill: ${message}\n`);
+    this.process.exitCode = 1;
+  }
+}

package/src/package.ts

@@ -0,0 +1,5 @@
+import { createRequire } from 'node:module';
+
+const require = createRequire(import.meta.url);
+
+export const PACKAGE_VERSION = (require('../package.json') as { version: string }).version;

package/src/quotaPreflight.test.ts

@@ -234,10 +234,22 @@ test('renderPreflightRefusal: auth kind uses unauthenticated wording', () => {
   });
   assert.strictEqual(
     msg,
-    'aibridge plan: refusing — grok session expired (401) — run `grok login`, then retry. Running with --no-preflight would only send the delegate in unauthenticated. Or use a different --model.',
+    'aibridge plan: refusing — grok session expired (401) — run `grok login`, then retry. Running with --no-preflight would only fail unauthenticated later. Or use a different --model.',
   );
 });
 
+test('renderPreflightRefusal: image-gen quota refusal points at other image seats', () => {
+  const msg = renderPreflightRefusal('image-gen', {
+    kind: 'quota',
+    message: 'grok credit quota exhausted',
+    resetAt: undefined,
+  });
+  // No claude seat renders images, so the delegation fallback would be dead advice.
+  assert.ok(!msg.includes('claude-backend fallback'));
+  assert.ok(msg.includes('another image seat'));
+  assert.ok(msg.includes('openai-codex/gpt-5.6-sol'));
+});
+
 test('renderPreflightRefusal: quota kind keeps override wording', () => {
   const msg = renderPreflightRefusal('subagent', {
     kind: 'quota',

package/src/quotaPreflight.ts

@@ -141,8 +141,16 @@ export function renderPreflightRefusal(
   verdict: { kind: 'auth' | 'quota'; message: string; resetAt: string | undefined },
 ): string {
   if (verdict.kind === 'auth') {
-    return `aibridge ${cmd}: refusing — ${verdict.message}. Running with --no-preflight would only send the delegate in unauthenticated. Or use a different --model.`;
+    // "the delegate" was wrong for image-gen, which has no delegate — the grok
+    // seat is a direct API call. Verified: --no-preflight there just fails at
+    // the render with exit 1.
+    return `aibridge ${cmd}: refusing — ${verdict.message}. Running with --no-preflight would only fail unauthenticated later. Or use a different --model.`;
   }
   const resetClause = verdict.resetAt ? ` Resets ${formatReset(verdict.resetAt)}.` : '';
-  return `aibridge ${cmd}: refusing — ${verdict.message}.${resetClause} Use --no-preflight to override, or a claude-backend fallback (subagent --model sonnet|opus — bills the Claude subscription).`;
+  // The claude fallback is delegation-only advice: no claude seat renders images.
+  const fallback =
+    cmd === 'image-gen'
+      ? 'Use --no-preflight to override, or another image seat (--model openai-codex/gpt-5.6-sol | google-antigravity/gemini-3.7-flash | xai-grok/grok-4.6).'
+      : 'Use --no-preflight to override, or a claude-backend fallback (subagent --model sonnet|opus — bills the Claude subscription).';
+  return `aibridge ${cmd}: refusing — ${verdict.message}.${resetClause} ${fallback}`;
 }