@compilr-dev/cli 0.8.9 → 0.9.0

package/dist/ui/overlay/impl/custom-agent-form-overlay-v2.d.ts

@@ -4,7 +4,6 @@
  * Multi-step wizard for creating custom agents.
  * Step 1: Basic Info (Agent ID, Display Name, Specialty, Personality)
  * Step 2: Tool Profile (select profile or customize tool groups)
- * Step 3: Skills (select which skills to enable)
  * Step 4: Model Tier (select fast/balanced/powerful)
  */
 import { BaseOverlayV2 } from '../../base/index.js';
@@ -20,7 +19,7 @@ interface FormField {
     value: string;
     error?: string;
 }
-type WizardStep = 'template-select' | 'basic' | 'tools' | 'skills' | 'tier' | 'prompt';
+type WizardStep = 'template-select' | 'basic' | 'tools' | 'tier' | 'prompt';
 interface FormState {
     fields: FormField[];
     activeFieldIndex: number;
@@ -31,8 +30,6 @@ interface FormState {
     profileListIndex: number;
     groupListIndex: number;
     inGroupSelection: boolean;
-    enabledSkills: Set<string>;
-    skillListIndex: number;
     selectedModelTier: ModelTier;
     tierListIndex: number;
     systemPromptAddition: string;
@@ -67,7 +64,6 @@ export interface CustomAgentFormResult {
         personality?: string;
         systemPromptAddition?: string;
         toolConfig: ToolConfig;
-        enabledSkills: string[];
         modelTier: ModelTier;
     };
 }
@@ -84,12 +80,9 @@ export declare class CustomAgentFormOverlayV2 extends BaseOverlayV2<FormState, C
     private renderBasicStep;
     private renderToolsStep;
     private renderGroupItem;
-    private renderSkillsStep;
-    private getAvailableTools;
     handleKey(key: KeyEvent): OverlayAction<CustomAgentFormResult>;
     private handleBasicStepKey;
     private handleToolsStepKey;
-    private handleSkillsStepKey;
     private renderTierStep;
     private handleTierStepKey;
     private renderPromptStep;

package/dist/ui/overlay/impl/custom-agent-form-overlay-v2.js

@@ -4,11 +4,10 @@
  * Multi-step wizard for creating custom agents.
  * Step 1: Basic Info (Agent ID, Display Name, Specialty, Personality)
  * Step 2: Tool Profile (select profile or customize tool groups)
- * Step 3: Skills (select which skills to enable)
  * Step 4: Model Tier (select fast/balanced/powerful)
  */
 import { BaseOverlayV2, renderBorder, wrapText } from '../../base/index.js';
-import { validateAgentId, isAgentIdTaken, PREDEFINED_ROLE_IDS, TOOL_GROUPS, PROFILE_INFO, getGroupsByTier, createDefaultToolConfig, getToolsForProfile, getDefinedSkillNames, checkSkillCompatibility, getSkillsByCategory, } from '@compilr-dev/sdk';
+import { validateAgentId, isAgentIdTaken, PREDEFINED_ROLE_IDS, TOOL_GROUPS, PROFILE_INFO, getGroupsByTier, createDefaultToolConfig, } from '@compilr-dev/sdk';
 import { MODEL_TIERS, TIER_INFO } from '../../../models/index.js';
 // Profile list for selection — derived from SDK's PROFILE_INFO
 const PROFILE_LIST = Object.keys(PROFILE_INFO);
@@ -67,8 +66,6 @@ export class CustomAgentFormOverlayV2 extends BaseOverlayV2 {
             profileListIndex: 0,
             groupListIndex: 0,
             inGroupSelection: false,
-            enabledSkills: new Set(),
-            skillListIndex: 0,
             selectedModelTier: 'balanced',
             tierListIndex: 1, // Default to balanced (index 1)
             templateListIndex: 0,
@@ -89,8 +86,6 @@ export class CustomAgentFormOverlayV2 extends BaseOverlayV2 {
                 return this.renderBasicStep(context);
             case 'tools':
                 return this.renderToolsStep(context);
-            case 'skills':
-                return this.renderSkillsStep(context);
             case 'tier':
                 return this.renderTierStep(context);
             case 'prompt':
@@ -187,7 +182,7 @@ export class CustomAgentFormOverlayV2 extends BaseOverlayV2 {
         const border = renderBorder(context.width, s);
         // Header with step indicator
         lines.push(border);
-        lines.push(` ${s.primaryBold('Create Custom Agent')} ${s.muted('- Step 1/5: Basic Info')}`);
+        lines.push(` ${s.primaryBold('Create Custom Agent')} ${s.muted('- Step 1/4: Basic Info')}`);
         lines.push(border);
         lines.push('');
         // Render each field
@@ -239,7 +234,7 @@ export class CustomAgentFormOverlayV2 extends BaseOverlayV2 {
         const border = renderBorder(context.width, s);
         // Header
         lines.push(border);
-        lines.push(` ${s.primaryBold('Create Custom Agent')} ${s.muted('- Step 2/5: Tool Profile')}`);
+        lines.push(` ${s.primaryBold('Create Custom Agent')} ${s.muted('- Step 2/4: Tool Profile')}`);
         lines.push(border);
         lines.push('');
         if (!this.state.inGroupSelection) {
@@ -278,9 +273,15 @@ export class CustomAgentFormOverlayV2 extends BaseOverlayV2 {
                 const groupId = direct[i];
                 this.renderGroupItem(lines, groupId, i, s);
             }
-            // Meta tools section
+            /*
+              The parenthetical used to read "(via use_tool)". That named an invocation mechanism
+              the person choosing tool groups cannot act on, and `use_tool` is not something they
+              will see in the agent's own tool list — the registry routes these transparently. The
+              section name already says which tier these belong to, so the mechanism is dropped
+              rather than restated.
+            */
             lines.push('');
-            lines.push(` ${s.muted('─── Meta-Registry Tools (via use_tool) ───')}`);
+            lines.push(` ${s.muted('─── Meta-Registry Tools ───')}`);
             for (let i = 0; i < meta.length; i++) {
                 const groupId = meta[i];
                 this.renderGroupItem(lines, groupId, direct.length + i, s);
@@ -302,48 +303,6 @@ export class CustomAgentFormOverlayV2 extends BaseOverlayV2 {
         const readOnlyBadge = group.readOnly ? s.muted(' (read-only)') : '';
         lines.push(` ${marker} ${labelStyle(group.label)}${readOnlyBadge}`);
     }
-    renderSkillsStep(context) {
-        const s = context.styles;
-        const lines = [];
-        const border = renderBorder(context.width, s);
-        // Header
-        lines.push(border);
-        lines.push(` ${s.primaryBold('Create Custom Agent')} ${s.muted('- Step 3/5: Skills')}`);
-        lines.push(border);
-        lines.push('');
-        lines.push(` ${s.primary('Select skills to enable:')}`);
-        lines.push(` ${s.muted('(Leave empty for all skills, or select specific skills)')}`);
-        lines.push('');
-        // Get available tools for compatibility check
-        const availableTools = this.getAvailableTools();
-        const categories = getSkillsByCategory();
-        let globalIndex = 0;
-        for (const [category, skills] of Object.entries(categories)) {
-            lines.push(` ${s.muted(`─── ${category.charAt(0).toUpperCase() + category.slice(1)} ───`)}`);
-            for (const skillName of skills) {
-                const compatibility = checkSkillCompatibility(skillName, availableTools);
-                const isSelected = globalIndex === this.state.skillListIndex;
-                const isEnabled = this.state.enabledSkills.has(skillName);
-                const isCompatible = compatibility.compatible;
-                const marker = isEnabled ? s.success('[x]') : '[ ]';
-                const labelStyle = isSelected ? s.primaryBold : s.secondary;
-                // Soft warning for incompatible skills (not blocking)
-                const warningBadge = !isCompatible
-                    ? s.warning(` ⚠ may need: ${compatibility.missingRequired.slice(0, 2).join(', ')}${compatibility.missingRequired.length > 2 ? '...' : ''}`)
-                    : '';
-                lines.push(` ${marker} ${labelStyle(skillName)}${warningBadge}`);
-                globalIndex++;
-            }
-            lines.push('');
-        }
-        lines.push(border);
-        lines.push(s.muted(' ↑/↓ Navigate  Space Toggle  a Toggle All  Enter Next  ← Back  Esc Cancel'));
-        lines.push(border);
-        return lines;
-    }
-    getAvailableTools() {
-        return getToolsForProfile(this.state.selectedProfile, this.state.selectedProfile === 'custom' ? Array.from(this.state.customGroups) : undefined);
-    }
     handleKey(key) {
         // Escape - cancel
         if (key.name === 'escape') {
@@ -360,8 +319,6 @@ export class CustomAgentFormOverlayV2 extends BaseOverlayV2 {
                 return this.handleBasicStepKey(key);
             case 'tools':
                 return this.handleToolsStepKey(key);
-            case 'skills':
-                return this.handleSkillsStepKey(key);
             case 'tier':
                 return this.handleTierStepKey(key);
             case 'prompt':
@@ -495,8 +452,7 @@ export class CustomAgentFormOverlayV2 extends BaseOverlayV2 {
                     this.state.groupListIndex = 0;
                 }
                 else {
-                    // Go to skills step
-                    this.state.currentStep = 'skills';
+                    this.state.currentStep = 'tier';
                 }
                 return this.rerender();
             }
@@ -535,77 +491,19 @@ export class CustomAgentFormOverlayV2 extends BaseOverlayV2 {
                     // Don't proceed without any groups
                     return this.noAction();
                 }
-                // Go to skills step
-                this.state.currentStep = 'skills';
+                this.state.currentStep = 'tier';
                 return this.rerender();
             }
         }
         return this.noAction();
     }
-    handleSkillsStepKey(key) {
-        const skillNames = getDefinedSkillNames();
-        const totalSkills = skillNames.length;
-        // Left arrow - go back to tools step
-        if (key.name === 'left') {
-            this.state.currentStep = 'tools';
-            return this.rerender();
-        }
-        // Up/Down - navigate skills
-        if (key.name === 'up') {
-            if (this.state.skillListIndex > 0) {
-                this.state.skillListIndex--;
-            }
-            return this.rerender();
-        }
-        if (key.name === 'down') {
-            if (this.state.skillListIndex < totalSkills - 1) {
-                this.state.skillListIndex++;
-            }
-            return this.rerender();
-        }
-        // Space - toggle skill
-        if (key.name === 'space') {
-            const flatSkills = Object.values(getSkillsByCategory()).flat();
-            const skillName = flatSkills[this.state.skillListIndex];
-            // Allow toggling regardless of compatibility (soft warning only)
-            if (this.state.enabledSkills.has(skillName)) {
-                this.state.enabledSkills.delete(skillName);
-            }
-            else {
-                this.state.enabledSkills.add(skillName);
-            }
-            return this.rerender();
-        }
-        // 'a' - toggle all skills
-        if (key.char === 'a') {
-            const flatSkills = Object.values(getSkillsByCategory()).flat();
-            const allEnabled = flatSkills.every(skillName => this.state.enabledSkills.has(skillName));
-            if (allEnabled) {
-                // Clear all
-                this.state.enabledSkills.clear();
-            }
-            else {
-                // Enable all
-                for (const skillName of flatSkills) {
-                    this.state.enabledSkills.add(skillName);
-                }
-            }
-            return this.rerender();
-        }
-        // Enter - go to tier step
-        if (key.name === 'return') {
-            this.state.currentStep = 'tier';
-            return this.rerender();
-        }
-        return this.noAction();
-    }
     renderTierStep(context) {
         const s = context.styles;
         const lines = [];
         const border = renderBorder(context.width, s);
         // Header
         lines.push(border);
-        lines.push(` ${s.primaryBold('Create Custom Agent')} ${s.muted('- Step 4/5: Model Tier')}`);
+        lines.push(` ${s.primaryBold('Create Custom Agent')} ${s.muted('- Step 3/4: Model Tier')}`);
         lines.push(border);
         lines.push('');
         lines.push(` ${s.primary('Select the default model tier for this agent:')}`);
@@ -634,9 +532,9 @@ export class CustomAgentFormOverlayV2 extends BaseOverlayV2 {
         return lines;
     }
     handleTierStepKey(key) {
-        // Left arrow - go back to skills step
+        // Left arrow - go back to the tools step
         if (key.name === 'left') {
-            this.state.currentStep = 'skills';
+            this.state.currentStep = 'tools';
             return this.rerender();
         }
         // Up/Down - navigate tiers
@@ -818,7 +716,6 @@ export class CustomAgentFormOverlayV2 extends BaseOverlayV2 {
                 personality: getFieldValue('personality') || undefined,
                 systemPromptAddition: this.state.systemPromptAddition.trim() || undefined,
                 toolConfig: createDefaultToolConfig(),
-                enabledSkills: [],
                 modelTier: 'balanced', // Default tier
             },
         });
@@ -844,7 +741,6 @@ export class CustomAgentFormOverlayV2 extends BaseOverlayV2 {
                 personality: getFieldValue('personality') || undefined,
                 systemPromptAddition: this.state.systemPromptAddition.trim() || undefined,
                 toolConfig,
-                enabledSkills: Array.from(this.state.enabledSkills),
                 modelTier: this.state.selectedModelTier,
             },
         });

package/dist/ui/overlay/impl/project-edit-overlay-v2.d.ts

@@ -7,8 +7,9 @@
 import { BaseOverlayV2 } from '../../base/index.js';
 import type { RenderContext, OverlayAction, KeyEvent } from '../types.js';
 import { type Project } from '../../../db/repositories/index.js';
+import { type ProjectPatch } from '@compilr-dev/sdk';
 interface EditableField {
-    key: 'displayName' | 'description' | 'gitRemote' | 'gitBranch' | 'path';
+    key: keyof ProjectPatch;
     label: string;
     value: string;
     placeholder?: string;

package/dist/ui/overlay/impl/project-edit-overlay-v2.js

@@ -7,6 +7,7 @@
 import { BaseOverlayV2, renderBorder } from '../../base/index.js';
 import { projectRepository } from '../../../db/repositories/index.js';
 import * as terminal from '../../terminal.js';
+import { buildProjectUpdate } from '@compilr-dev/sdk';
 // =============================================================================
 // Status Labels
 // =============================================================================
@@ -25,10 +26,21 @@ export class ProjectEditOverlayV2 extends BaseOverlayV2 {
     constructor(project) {
         super({
             project,
+            /*
+              ⚠️ THE SAME FIELDS DESKTOP EDITS (Phase 5 parity). This carried five; Desktop's document
+              carried eleven, so a project edited in the app showed a language and a framework the
+              terminal could neither see nor change. Parity is on the DATA MODEL — the presentation stays
+              a field list, because a terminal has no document.
+            */
             fields: [
                 { key: 'displayName', label: 'Display Name', value: project.displayName, placeholder: 'Project display name' },
                 { key: 'description', label: 'Description', value: project.description || '', placeholder: 'Brief description' },
                 { key: 'path', label: 'Path', value: project.path || '', placeholder: '/path/to/project' },
+                { key: 'docsPath', label: 'Docs Path', value: project.docsPath || '', placeholder: 'defaults to the project folder' },
+                { key: 'language', label: 'Language', value: project.language || '', placeholder: 'TypeScript' },
+                { key: 'framework', label: 'Framework', value: project.framework || '', placeholder: 'React' },
+                { key: 'packageManager', label: 'Package Manager', value: project.packageManager || '', placeholder: 'npm' },
+                { key: 'runtimeVersion', label: 'Runtime', value: project.runtimeVersion || '', placeholder: '22' },
                 { key: 'gitRemote', label: 'Git Remote', value: project.gitRemote || '', placeholder: 'https://github.com/...' },
                 { key: 'gitBranch', label: 'Git Branch', value: project.gitBranch || '', placeholder: 'main' },
             ],
@@ -163,28 +175,23 @@ export class ProjectEditOverlayV2 extends BaseOverlayV2 {
         return this.noAction();
     }
     saveAndClose() {
-        // Build update object from fields
-        const updates = {};
+        /*
+          ⚠️ THE SDK'S MAPPING, NOT A SECOND ONE. This used to hand-write its own snake_case mapping,
+          and it had drifted from Desktop's: `field.value || undefined` turned an empty string into
+          "leave alone", so a git remote could be SET here but never CLEARED. An empty field now means
+          what it looks like it means.
+        */
+        const patch = {};
         for (const field of this.state.fields) {
-            if (field.key === 'displayName') {
-                updates.display_name = field.value || undefined;
-            }
-            else if (field.key === 'description') {
-                updates.description = field.value || undefined;
-            }
-            else if (field.key === 'gitRemote') {
-                updates.git_remote = field.value || undefined;
-            }
-            else if (field.key === 'gitBranch') {
-                updates.git_branch = field.value || undefined;
-            }
-            else {
-                // field.key === 'path'
-                updates.path = field.value || undefined;
-            }
+            patch[field.key] = field.value;
+        }
+        const built = buildProjectUpdate(patch);
+        if (!built.ok) {
+            this.state.error = built.error;
+            return this.rerender();
         }
         try {
-            projectRepository.update(this.state.project.id, updates);
+            projectRepository.update(this.state.project.id, built.input);
             return this.close({ saved: true, projectId: this.state.project.id });
         }
         catch (err) {

package/dist/ui/overlay/impl/skills-overlay-v2.d.ts

@@ -15,14 +15,21 @@
  * Spec: project-docs/00-requirements/compilr-dev-cli/skills-overlay-spec.md
  */
 import { TabbedListOverlayV2, BaseScreen } from '../../base/index.js';
-import { type CustomSkill } from '@compilr-dev/sdk';
+import { type SkillReachability, type CustomSkill } from '@compilr-dev/sdk';
 import { type SkillScope } from '../../../utils/skill-paths.js';
 /** Unified item for the list — custom skills have full data, built-in have name+description only */
 interface SkillItem {
     name: string;
     description: string;
     scope: 'project' | 'user' | 'sdk' | 'binding';
-    enabled: boolean;
+    /**
+     * ⚠️ THE TWO ANTHROPIC SWITCHES, NOT `enabled`. This used to be `skill.enabled !== false`, and
+     * `enabled` is not part of the format — zero of 50 Anthropic-authored skills carry it. A skill
+     * shelved in Desktop writes `user-invocable: false` and no `enabled` key at all, so the old
+     * expression read it back as ON: the CLI showed a shelved skill as live, and the resolver (which
+     * already uses skillReachability) disagreed with the list in front of the user.
+     */
+    reach: SkillReachability;
     /** Validation issues (empty = valid) */
     issues: string[];
     /** Full CustomSkill data (null for built-in) */
@@ -45,17 +52,38 @@ interface SkillItem {
 export interface SkillsOverlayV2Options {
     /** Project root path (for project-scoped skills) */
     projectPath?: string;
+    /**
+     * Extra prompt sources, supplied by the app layer because that is where they are configured.
+     * The same set passed to the resolver, so this tab and `getSkillPrompt` cannot disagree
+     * about what exists.
+     */
+    skillSources?: import('@compilr-dev/sdk').SkillSources;
     /** Callback to open the skill detail overlay (view/edit/diff). Returns true if modified. */
     onOpenDetail?: (filePath: string, skillName: string, scope: string, forkedFrom?: import('@compilr-dev/sdk').ForkedFromMarker, boundCommands?: string[]) => Promise<boolean>;
 }
 export interface SkillsOverlayV2Result {
     dismissed: boolean;
 }
+/**
+ * The shipped macros this tab offers.
+ *
+ * ⚠️ IT USED TO OFFER EIGHT IT CANNOT RUN. This walked `platformMacros` directly and added every
+ * one, including the canvas family — which `getSkillPrompt` REFUSES, because a terminal cannot
+ * draw a canvas. So the Built-in tab listed `canvas`, `infographic`, `carousel`, `board`,
+ * `canvas-styles` and the three reference ones, and selecting any of them got nothing. That is
+ * the same "offered, then answers Unknown command" defect the CLI had already fixed for
+ * disabled skills, in a second place nobody checked.
+ *
+ * It now asks the SDK, declaring what this host can render. The resolver's refusal and this
+ * list are therefore two consequences of one rule rather than two rules that can disagree.
+ */
+export declare function loadBuiltinMacros(projectPath?: string, sources?: import('@compilr-dev/sdk').SkillSources): SkillItem[];
 export declare class SkillsOverlayV2 extends TabbedListOverlayV2<SkillItem, SkillsOverlayV2Result> {
     readonly type: "inline";
     readonly id = "skills-overlay-v2";
     private allItems;
     private readonly projectPath?;
+    private readonly skillSources?;
     private readonly onOpenDetail?;
     constructor(options: SkillsOverlayV2Options);
     /**
@@ -73,6 +101,18 @@ export declare class SkillsOverlayV2 extends TabbedListOverlayV2<SkillItem, Skil
         pushScreen?: BaseScreen;
     };
     private createSkill;
+    /**
+     * Shelve a skill, or bring it back.
+     *
+     * ⚠️ WRITES THE TWO ANTHROPIC SWITCHES AND REMOVES THE LEGACY KEY (D-1), THROUGH THE SDK'S
+     * PATCHER RATHER THAN A REGEX (D-2). The old version rewrote frontmatter line by line, which
+     * destroys comments — measured on our own template: 17 lines to 7, all 9 comment lines to 0.
+     *
+     * One key for two switches is lossy by construction: a skill set to "agents only" in Desktop is
+     * shelved and then restored to BOTH. That is why the list renders the real state beside the name
+     * — the flattening is visible, not silent — and why the two-switch editor lives in the host that
+     * has a document to put it in.
+     */
     private toggleSkill;
     private deleteSkill;
     private renameSkill;