@detergent-software/conductor 1.0.0-dev.31 → 1.0.0-dev.33

package/mod/hooks/register.tsx

@@ -1,5 +1,5 @@
 import { atom, read, update } from 'claude-code';
-import type { EngineInterface, Register } from 'claude-code';
+import type { EngineInterface, Register, RenderElement } from 'claude-code';
 
 import type { ConductorSnapshot } from '../types';
 
@@ -8,6 +8,7 @@ import {
   type ApiAnswer,
   answerText,
   bandEntries,
+  CONSOLE_SKILL,
   endpointOf,
   errorOf,
   findPlan,
@@ -15,6 +16,8 @@ import {
   glyph,
   homeOf,
   inboxText,
+  NO_SKILL_CONTEXT,
+  NO_SKILL_WARNING,
   notifyArgs,
   notifyText,
   overrideArgs,
@@ -49,6 +52,7 @@ const NOT_RUNNING = 'the conductor is not running (conduct serve)';
 const snapshot = atom({ plugin: 'conductor', key: 'snapshot' } as const, null);
 const confirming = atom({ plugin: 'conductor', key: 'confirming' } as const, null);
 const expanded = atom({ plugin: 'conductor', key: 'expanded' } as const, null);
+const hasSkill = atom({ plugin: 'conductor', key: 'hasSkill' } as const, false);
 
 // The console is the session started with CONDUCTOR_CONSOLE=1 (`conduct console` sets it): only it
 // draws the band and the pane, answers the commands, follows the stream and acts on events. Both
@@ -315,6 +319,27 @@ export const register: Register = (on) => {
     return started;
   });
 
+  // A console the skill never ran in (one started by hand as `claude -n conductor`, ATK-385) is
+  // told to load it before it answers, each prompt until it has. `/conductor-console` or the Skill
+  // tool expands it before its prompt is submitted; a /clear or a resume starts a conversation
+  // without it.
+  on('skill.prompt', { skill: CONSOLE_SKILL }, async ($, e, next) => {
+    const prompt = await next(e);
+    if (isConsole) await update($, hasSkill, () => true);
+    return prompt;
+  });
+
+  on('session.end', async ($, e, next) => {
+    if (isConsole) await update($, hasSkill, () => false);
+    return next(e);
+  });
+
+  on('prompt.submit', async ($, e, next) => {
+    if (!isConsole || (await read($, hasSkill))) return next(e);
+    $.ui.log(`conductor: /${CONSOLE_SKILL} hasn't run; asking the model to load it`, { to: 'debug' });
+    return next({ ...e, context: [...(e.context ?? []), NO_SKILL_CONTEXT] });
+  });
+
   on('command.run', { command: 'conductor' }, async ($) => {
     await openPane($);
     return { text: 'Conductor pane opened.' };
@@ -369,17 +394,32 @@ export const register: Register = (on) => {
   on('command.run', { command: 'focus-project' }, async ($, e) => ({ text: await override($, 'focus', e.args) }));
 
   // The band above the prompt, in the console, while the inbox has items. A bare digit typed into
-  // an empty prompt presses a band Button, so a plan's approve takes a second press (`y`).
+  // an empty prompt presses a band Button, so a plan's approve takes a second press (`y`). Until
+  // /conductor-console runs, a warning sits above it, or above the engine's own (ATK-385).
   on('ui.render', { component: 'AbovePrompt' }, async ($, e, next) => {
     if (!isConsole || e.props.hasSurvey) return next(e);
-    const snap = await read($, snapshot);
-    if (snap === null || snap.error !== null || snap.inbox.length === 0) return next(e);
     const { Box, Button, Text } = $.ui.resolve(e);
+    const isWarned = !(await read($, hasSkill));
+    const withWarning = (row: RenderElement): RenderElement =>
+      isWarned ? (
+        <Box flexDirection="column">
+          <Box key="no-skill">
+            <Text bold color="warning">
+              {NO_SKILL_WARNING}
+            </Text>
+          </Box>
+          {row}
+        </Box>
+      ) : (
+        row
+      );
+    const snap = await read($, snapshot);
+    if (snap === null || snap.error !== null || snap.inbox.length === 0) return isWarned ? withWarning(await next(e)) : next(e);
     const entries = bandEntries(snap.inbox);
     const confirmId = await read($, confirming);
     const pending = entries.find((x) => x.act === 'approve' && x.id === confirmId);
     if (pending) {
-      return (
+      return withWarning(
         <Box columnGap={2} flexWrap="wrap">
           <Text bold>conductor</Text>
           <Text>
@@ -396,10 +436,10 @@ export const register: Register = (on) => {
             variant="primary"
           />
           <Button hotkey="n" key="cancel" label="n cancel" onPress={() => update($, confirming, () => null)} />
-        </Box>
+        </Box>,
       );
     }
-    return (
+    return withWarning(
       <Box columnGap={2} flexWrap="wrap">
         <Text bold>conductor · inbox {snap.inbox.length}</Text>
         {entries.map((x) => (
@@ -416,7 +456,7 @@ export const register: Register = (on) => {
         ))}
         {snap.inbox.length > entries.length && <Text dimColor>+{snap.inbox.length - entries.length} more</Text>}
         <Button hotkey="o" key="open-pane" label="o panel" onPress={() => openPane($)} />
-      </Box>
+      </Box>,
     );
   });
 

package/mod/types/index.d.ts

@@ -51,6 +51,9 @@ declare module 'claude-code' {
       confirming: number | null;
       // The worker whose row the pane shows expanded, by name.
       expanded: string | null;
+      // Whether /conductor-console ran in this conversation (ATK-385). Kept in $.state, so a reload
+      // of the mod doesn't forget it; a /clear or a resume (session.end) sets it back to false.
+      hasSkill: boolean;
       snapshot: ConductorSnapshot | null;
     };
   }

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@detergent-software/conductor",
-	"version": "1.0.0-dev.31",
+	"version": "1.0.0-dev.33",
 	"description": "Conductor: the local daemon that runs conducted projects' work loops (conduct serve) and its conduct CLI.",
 	"type": "module",
 	"bin": {