@compilr-dev/cli 0.8.0 → 0.8.2

package/dist/index.js

@@ -885,6 +885,14 @@ async function main() {
         // Show login overlay if not authenticated
         showLogin,
     });
+    // Always restore the hardware cursor on exit. Overlays/menus hide it while
+    // showing nav-only screens; without this, quitting (incl. Ctrl+C, which
+    // routes through process.exit → 'exit') from such a screen would leave the
+    // user's terminal cursor invisible.
+    process.on('exit', () => {
+        if (process.stdout.isTTY)
+            process.stdout.write('\x1b[?25h');
+    });
     // Handle Ctrl+C gracefully
     process.on('SIGINT', () => {
         // Flush episode recorder before exit

package/dist/repl-v2.js

@@ -31,7 +31,7 @@ import { renderMascotWithLogo } from './ui/mascot/renderer.js';
 import { getStartupHighlights } from './changelog/index.js';
 import { registerCommands, executeCommand, allCommands, getAutocompleteCommands, setSkillCommandLoader, saveCurrentSession, saveCurrentTeam, loadProjectSession, archiveCurrentSession, convertMessagesToItems, } from './commands-v2/index.js';
 import { getCustomCommandRegistry } from './commands/custom-registry.js';
-import { getPlanModePrompt, createHandoffTool, createConsultTool, createDelegateTool, createDelegateBackgroundTool, } from '@compilr-dev/sdk';
+import { getPlanModePrompt, createHandoffTool, createConsultTool, createDelegateTool, createDelegateBackgroundTool, classifyAgentError, } from '@compilr-dev/sdk';
 import { getHandoffHandler, getConsultHandler, getDelegateHandler, getDelegateBackgroundHandler, } from './shared-handlers.js';
 import { planRepository } from './db/repositories/index.js';
 import { PlanApprovalOverlayV2, } from './ui/overlay/impl/plan-approval-overlay-v2.js';
@@ -144,6 +144,26 @@ ${JSON.stringify(state.messages, null, 2)}
         log.error({ component: 'repl', err: error }, 'Failed to log agent request');
     }
 }
+/**
+ * Map an agent/provider error to a terminal-friendly message. Classification is
+ * shared across hosts via the SDK (classifyAgentError); only the remediation
+ * wording is CLI-specific. The CLI checks for a *missing* key at startup, so this
+ * mainly catches an *invalid/expired* key surfacing as a runtime 401 mid-turn.
+ */
+function friendlyTurnError(err) {
+    switch (classifyAgentError(err).category) {
+        case 'auth':
+            return 'Invalid or missing API key for this provider. Set a valid key with /keys (or export the provider env var). No key? Restart with --provider ollama for free local models.';
+        case 'rate_limit':
+            return 'Rate limited by the provider. Wait a moment and try again.';
+        case 'overloaded':
+            return 'The provider is overloaded right now. Try again in a moment.';
+        case 'connection':
+            return "Can't reach the provider. Check your internet connection and try again.";
+        default:
+            return String(err);
+    }
+}
 // =============================================================================
 // REPL V2 Class
 // =============================================================================
@@ -3035,7 +3055,7 @@ export class ReplV2 {
                 }
                 catch (err) {
                     if (!signal.aborted) {
-                        this.ui.print({ type: 'error', message: String(err) });
+                        this.ui.print({ type: 'error', message: friendlyTurnError(err) });
                     }
                     this.ui.setAgentRunning(false);
                     this.ui.setThinking(false); // Ensure thinking indicator is cleared
@@ -3287,7 +3307,7 @@ export class ReplV2 {
                 }
                 catch (err) {
                     if (!signal.aborted) {
-                        this.ui.print({ type: 'error', message: `${label} Error: ${String(err)}` });
+                        this.ui.print({ type: 'error', message: `${label} ${friendlyTurnError(err)}` });
                         // Notify other terminals about failure
                         try {
                             const sid = getActiveTerminalSessionId();
@@ -3602,7 +3622,7 @@ export class ReplV2 {
                 }
                 catch (err) {
                     if (!signal.aborted) {
-                        this.ui.print({ type: 'error', message: String(err) });
+                        this.ui.print({ type: 'error', message: friendlyTurnError(err) });
                     }
                     this.ui.setAgentRunning(false);
                     this.ui.setThinking(false); // Ensure thinking indicator is cleared
@@ -3719,6 +3739,12 @@ export class ReplV2 {
 // Only run main() when executed directly (not when imported)
 if (import.meta.url === `file://${process.argv[1]}`) {
     const repl = new ReplV2();
+    // Restore the hardware cursor on exit (overlays/menus hide it for nav-only
+    // screens; don't leave the user's terminal cursor invisible after quit).
+    process.on('exit', () => {
+        if (process.stdout.isTTY)
+            process.stdout.write('\x1b[?25h');
+    });
     // Handle SIGINT — restore raw mode before exit so the parent shell
     // doesn't inherit a broken terminal state.
     process.on('SIGINT', () => {

package/dist/tools/platform-adapter.js

@@ -5,7 +5,7 @@
  * wires CLI-specific hooks and services (anchors, artifacts, episodes),
  * and delegates all 32 platform tool definitions to the SDK.
  */
-import { createPlatformTools, createSQLiteRepositories, ProjectAnchorStore, } from '@compilr-dev/sdk';
+import { createPlatformTools, createSQLiteRepositories, ProjectAnchorStore, FileArtifactService, } from '@compilr-dev/sdk';
 import { createAllFactoryTools } from '@compilr-dev/factory';
 import { getDb } from '../db/index.js';
 import { getDataPath, getSessionsPath } from '../settings/paths.js';
@@ -16,7 +16,7 @@ import { getCurrentProject, setCurrentProject } from './project-db.js';
 import { awardFirstProject, awardWorkItemCompletion } from '../games/coins.js';
 import { getActiveSharedContext } from '@compilr-dev/sdk';
 import { setAnchorStore } from '../anchors/index.js';
-import { getTeamCheckpointer, recordTeamActivity } from '../multi-agent/index.js';
+import { recordTeamActivity, getSessionsPath as getCheckpointerSessionsPath } from '../multi-agent/index.js';
 import { getGlobalEpisodeStore } from '../episodes/index.js';
 // =============================================================================
 // Repositories — SDK's SQLite implementations, backed by CLI's existing DB
@@ -63,123 +63,24 @@ const anchorService = anchorStore.toService();
 // =============================================================================
 // Artifact Service — wraps ArtifactStore from TeamCheckpointer
 // =============================================================================
+// Artifacts — file-backed service shared with Desktop via the SDK
+// (FileArtifactService), pointed at the same sessions dir the checkpointer uses
+// so artifact_* tools and team hydration read/write the same artifacts.json.
+// We wrap save() to record team activity attributed to the ACTING agent
+// (input.agent), which differs from the artifact's original creator on updates.
+const baseArtifactService = new FileArtifactService({
+    sessionsDir: getCheckpointerSessionsPath(),
+    getProjectId: () => getCurrentProject()?.id ?? null,
+});
 const artifactService = {
-    save: (input) => {
-        const currentProject = getCurrentProject();
-        const projectId = currentProject?.id ?? null;
-        const store = getTeamCheckpointer().getArtifactStore(projectId);
-        const agentId = input.agent ?? 'default';
-        const existing = store.getByName(input.name);
-        if (existing) {
-            const updated = store.update(existing.id, {
-                content: input.content,
-                summary: input.summary,
-                type: input.type,
-            });
-            if (!updated)
-                throw new Error(`Failed to update artifact "${input.name}"`);
-            getTeamCheckpointer().saveArtifactStore(currentProject?.id ?? null, store);
-            recordTeamActivity(agentId, 'artifact_updated', `updated artifact "${updated.name}"`);
-            const artifact = {
-                id: updated.id,
-                name: updated.name,
-                agent: updated.agent,
-                type: updated.type,
-                content: updated.content,
-                summary: updated.summary,
-                version: updated.version,
-                createdAt: updated.createdAt,
-                updatedAt: updated.updatedAt,
-            };
-            return Promise.resolve({ artifact, isUpdate: true });
-        }
-        else {
-            const created = store.create({
-                name: input.name,
-                agent: agentId,
-                type: input.type,
-                content: input.content,
-                summary: input.summary,
-            });
-            getTeamCheckpointer().saveArtifactStore(currentProject?.id ?? null, store);
-            recordTeamActivity(agentId, 'artifact_created', `created artifact "${created.name}"`);
-            const artifact = {
-                id: created.id,
-                name: created.name,
-                agent: created.agent,
-                type: created.type,
-                content: created.content,
-                summary: created.summary,
-                version: created.version,
-                createdAt: created.createdAt,
-                updatedAt: created.updatedAt,
-            };
-            return Promise.resolve({ artifact, isUpdate: false });
-        }
-    },
-    getByName: (name) => {
-        const currentProject = getCurrentProject();
-        const store = getTeamCheckpointer().getArtifactStore(currentProject?.id ?? null);
-        const artifact = store.getByName(name);
-        if (!artifact)
-            return Promise.resolve(null);
-        const result = {
-            id: artifact.id,
-            name: artifact.name,
-            agent: artifact.agent,
-            type: artifact.type,
-            content: artifact.content,
-            summary: artifact.summary,
-            version: artifact.version,
-            createdAt: artifact.createdAt,
-            updatedAt: artifact.updatedAt,
-        };
-        return Promise.resolve(result);
-    },
-    list: (options) => {
-        const currentProject = getCurrentProject();
-        const store = getTeamCheckpointer().getArtifactStore(currentProject?.id ?? null);
-        let items = options?.search ? store.search(options.search) : store.list();
-        if (options?.type) {
-            items = items.filter(a => a.type === options.type);
-        }
-        if (options?.agent) {
-            items = items.filter(a => a.agent === options.agent);
-        }
-        const summaries = items.map(a => ({
-            id: a.id,
-            name: a.name,
-            agent: a.agent,
-            type: a.type,
-            summary: a.summary,
-            version: a.version,
-            updatedAt: a.updatedAt,
-        }));
-        return Promise.resolve(summaries);
-    },
-    delete: (name) => {
-        const currentProject = getCurrentProject();
-        const store = getTeamCheckpointer().getArtifactStore(currentProject?.id ?? null);
-        const artifact = store.getByName(name);
-        if (!artifact)
-            return Promise.resolve(null);
-        const deleted = store.delete(artifact.id);
-        if (!deleted)
-            return Promise.resolve(null);
-        getTeamCheckpointer().saveArtifactStore(currentProject?.id ?? null, store);
-        const result = {
-            id: artifact.id,
-            name: artifact.name,
-            agent: artifact.agent,
-            type: artifact.type,
-            content: artifact.content,
-            summary: artifact.summary,
-            version: artifact.version,
-            createdAt: artifact.createdAt,
-            updatedAt: artifact.updatedAt,
-        };
-        return Promise.resolve(result);
+    save: async (input) => {
+        const result = await baseArtifactService.save(input);
+        recordTeamActivity(input.agent ?? 'default', result.isUpdate ? 'artifact_updated' : 'artifact_created', `${result.isUpdate ? 'updated' : 'created'} artifact "${result.artifact.name}"`);
+        return result;
     },
+    getByName: (name) => baseArtifactService.getByName(name),
+    list: (options) => baseArtifactService.list(options),
+    delete: (name) => baseArtifactService.delete(name),
 };
 // =============================================================================
 // Episode Service — wraps FileEpisodeStore

package/dist/ui/overlay-manager.js

@@ -176,6 +176,9 @@ export class OverlayManager {
                 process.stdout.write('\n');
             }
         }
+        // Only show the hardware cursor for overlays that actually take text input
+        // (they provide a cursorPosition). Nav-only overlays (menus, dashboards,
+        // selection lists) hide it so it doesn't blink, parked, below the screen.
         if (content.cursorPosition) {
             const { line, column } = content.cursorPosition;
             const linesFromEnd = paddedLines.length - 1 - line;
@@ -183,8 +186,11 @@ export class OverlayManager {
                 ttyWrite(`\x1b[${String(linesFromEnd)}A`);
             }
             ttyWrite(`\x1b[${String(column)}G`);
+            terminal.showCursor();
+        }
+        else {
+            terminal.hideCursor();
         }
-        terminal.showCursor();
     }
     renderInlineOverlay(content, termWidth) {
         this.clearOverlayRender();
@@ -206,7 +212,21 @@ export class OverlayManager {
         for (const line of paddedLines) {
             process.stdout.write(line + '\n');
         }
-        terminal.showCursor();
+        // Same rule as fullscreen: cursor only for text-input overlays. Inline
+        // writes a trailing newline after every line, so the cursor sits one row
+        // below the last content line (hence no -1 in linesFromEnd).
+        if (content.cursorPosition) {
+            const { line, column } = content.cursorPosition;
+            const linesFromEnd = paddedLines.length - line;
+            if (linesFromEnd > 0) {
+                ttyWrite(`\x1b[${String(linesFromEnd)}A`);
+            }
+            ttyWrite(`\x1b[${String(column)}G`);
+            terminal.showCursor();
+        }
+        else {
+            terminal.hideCursor();
+        }
     }
     // ===========================================================================
     // Action processing

package/dist/utils/project-memory.d.ts

@@ -1,47 +1,12 @@
 /**
- * Project Memory Loader
+ * Project Memory Loader (CLI)
  *
- * Loads project-level instructions from COMPILR.md or CLAUDE.md files.
- * Provides size guards and warnings for large files.
- */
-export interface ProjectMemoryResult {
-    /** Whether a memory file was found */
-    found: boolean;
-    /** The loaded content (may be truncated) */
-    content: string;
-    /** Path to the file that was loaded */
-    filePath: string | null;
-    /** Original file size in bytes */
-    originalSize: number;
-    /** Whether the content was truncated */
-    truncated: boolean;
-    /** Estimated token count (chars / 4) */
-    estimatedTokens: number;
-}
-export interface ProjectMemoryOptions {
-    /** Maximum content size in bytes before truncation (default: 100KB) */
-    maxSize?: number;
-    /** Size threshold for warning (default: 30KB) */
-    warnSize?: number;
-    /** Custom search directory (default: process.cwd()) */
-    cwd?: string;
-}
-/**
- * Load project memory from COMPILR.md or CLAUDE.md
+ * Now a thin re-export of the SDK's shared loader (loadProjectContextFiles):
+ * priority search over COMPILR.md / .compilr/instructions.md / CLAUDE.md /
+ * .claude/instructions.md with size-guard truncation. Shared with Desktop.
  *
- * @param options - Configuration options
- * @returns Result with content and metadata
- */
-export declare function loadProjectMemory(options?: ProjectMemoryOptions): ProjectMemoryResult;
-/**
- * Check if a project memory file exists without loading it
- */
-export declare function hasProjectMemory(cwd?: string): boolean;
-/**
- * Get the size category for display purposes
- */
-export declare function getSizeCategory(bytes: number): 'small' | 'medium' | 'large';
-/**
- * Format bytes as human-readable string
+ * The previous CLI-only display helpers (hasProjectMemory / getSizeCategory /
+ * formatBytes) had no callers and were removed.
  */
-export declare function formatBytes(bytes: number): string;
+export { loadProjectContextFiles as loadProjectMemory } from '@compilr-dev/sdk';
+export type { ProjectContextResult as ProjectMemoryResult, LoadProjectContextOptions as ProjectMemoryOptions, } from '@compilr-dev/sdk';

package/dist/utils/project-memory.js

@@ -1,118 +1,11 @@
 /**
- * Project Memory Loader
+ * Project Memory Loader (CLI)
  *
- * Loads project-level instructions from COMPILR.md or CLAUDE.md files.
- * Provides size guards and warnings for large files.
- */
-import * as fs from 'fs';
-import * as path from 'path';
-import { estimateTokens } from './token-tracker.js';
-// =============================================================================
-// Constants
-// =============================================================================
-/** Files to search for, in priority order */
-const MEMORY_FILES = [
-    'COMPILR.md',
-    '.compilr/instructions.md',
-    'CLAUDE.md',
-    '.claude/instructions.md',
-];
-/** Default max size: 100KB */
-const DEFAULT_MAX_SIZE = 100 * 1024;
-/** Default warn size: 30KB */
-const DEFAULT_WARN_SIZE = 30 * 1024;
-// =============================================================================
-// Main Function
-// =============================================================================
-/**
- * Load project memory from COMPILR.md or CLAUDE.md
+ * Now a thin re-export of the SDK's shared loader (loadProjectContextFiles):
+ * priority search over COMPILR.md / .compilr/instructions.md / CLAUDE.md /
+ * .claude/instructions.md with size-guard truncation. Shared with Desktop.
  *
- * @param options - Configuration options
- * @returns Result with content and metadata
- */
-export function loadProjectMemory(options = {}) {
-    const cwd = options.cwd ?? process.cwd();
-    const maxSize = options.maxSize ?? DEFAULT_MAX_SIZE;
-    // Search for memory file
-    for (const filename of MEMORY_FILES) {
-        const filePath = path.join(cwd, filename);
-        if (fs.existsSync(filePath)) {
-            try {
-                const stat = fs.statSync(filePath);
-                const originalSize = stat.size;
-                // Read file content
-                let content = fs.readFileSync(filePath, 'utf-8');
-                let truncated = false;
-                // Truncate if too large
-                if (content.length > maxSize) {
-                    content = content.slice(0, maxSize);
-                    // Try to truncate at a line boundary
-                    const lastNewline = content.lastIndexOf('\n');
-                    if (lastNewline > maxSize * 0.8) {
-                        content = content.slice(0, lastNewline);
-                    }
-                    content += '\n\n[... truncated due to size ...]';
-                    truncated = true;
-                }
-                return {
-                    found: true,
-                    content,
-                    filePath,
-                    originalSize,
-                    truncated,
-                    estimatedTokens: estimateTokens(content),
-                };
-            }
-            catch {
-                // File exists but couldn't be read, continue to next
-                continue;
-            }
-        }
-    }
-    // No memory file found
-    return {
-        found: false,
-        content: '',
-        filePath: null,
-        originalSize: 0,
-        truncated: false,
-        estimatedTokens: 0,
-    };
-}
-/**
- * Check if a project memory file exists without loading it
- */
-export function hasProjectMemory(cwd) {
-    const dir = cwd ?? process.cwd();
-    for (const filename of MEMORY_FILES) {
-        const filePath = path.join(dir, filename);
-        if (fs.existsSync(filePath)) {
-            return true;
-        }
-    }
-    return false;
-}
-/**
- * Get the size category for display purposes
- */
-export function getSizeCategory(bytes) {
-    if (bytes < DEFAULT_WARN_SIZE) {
-        return 'small';
-    }
-    else if (bytes < DEFAULT_MAX_SIZE) {
-        return 'medium';
-    }
-    return 'large';
-}
-/**
- * Format bytes as human-readable string
+ * The previous CLI-only display helpers (hasProjectMemory / getSizeCategory /
+ * formatBytes) had no callers and were removed.
  */
-export function formatBytes(bytes) {
-    if (bytes < 1024) {
-        return `${String(bytes)} B`;
-    }
-    else if (bytes < 1024 * 1024) {
-        return `${(bytes / 1024).toFixed(1)} KB`;
-    }
-    return `${(bytes / (1024 * 1024)).toFixed(1)} MB`;
-}
+export { loadProjectContextFiles as loadProjectMemory } from '@compilr-dev/sdk';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@compilr-dev/cli",
-  "version": "0.8.0",
+  "version": "0.8.2",
   "description": "AI-powered coding assistant CLI using @compilr-dev/agents",
   "type": "module",
   "main": "dist/index.js",
@@ -55,12 +55,12 @@
   },
   "dependencies": {
     "@anthropic-ai/sdk": "^0.74.0",
-    "@compilr-dev/agents": "^0.5.9",
+    "@compilr-dev/agents": "^0.6.0",
     "@compilr-dev/agents-coding": "^1.0.4",
     "@compilr-dev/editor-core": "^0.0.2",
-    "@compilr-dev/factory": "^0.1.30",
+    "@compilr-dev/factory": "^0.1.35",
     "@compilr-dev/logger": "^0.1.0",
-    "@compilr-dev/sdk": "^0.11.0",
+    "@compilr-dev/sdk": "^0.15.0",
     "@compilr-dev/ui-core": "^0.0.1",
     "@modelcontextprotocol/sdk": "^1.23.0",
     "ansi-escapes": "^7.3.0",