@compilr-dev/sdk 0.35.0 → 0.37.0

package/dist/index.d.ts

@@ -64,7 +64,7 @@ export { detectProject, suggestProjectType, detectCommon } from './detection/ind
 export type { DetectProjectOptions, DetectionResult, ContentSummary } from './detection/index.js';
 export { createGuideTool, SHARED_GUIDE_ENTRIES, searchGuideEntries, topicToGuideEntry, } from './guide/index.js';
 export type { GuideEntry, ContentTopic, ContentSection, GuideToolConfig } from './guide/index.js';
-export { createPlatformTools, createProjectTools, createWorkItemTools, createDocumentTools, createPlanTools, createBacklogTools, createAnchorTools, createArtifactTools, createEpisodeTools, createCanvasTools, createSceneTools, SCENE_TOOL_NAMES, createSceneWriter, SCENE_JOURNAL_SIZE, createImageTools, ProjectAnchorStore, FileArtifactService, } from './platform/index.js';
+export { createPlatformTools, createProjectTools, createWorkItemTools, createDocumentTools, createPlanTools, createBacklogTools, createAnchorTools, createArtifactTools, createEpisodeTools, createCanvasTools, createSceneTools, SCENE_TOOL_NAMES, resolveSelfAlias, createSceneWriter, SCENE_JOURNAL_SIZE, createImageTools, ProjectAnchorStore, FileArtifactService, } from './platform/index.js';
 export type { ProjectAnchorStoreConfig, FileArtifactServiceConfig, ImageToolsConfig, ImageResizer, } from './platform/index.js';
 export { STEP_ORDER, GUIDED_STEP_CRITERIA, getNextStep, isValidTransition, getStepCriteria, formatStepDisplay, getStepNumber, } from './platform/index.js';
 export type { CustomSkill, CompilrSkillExtension, ForkedFromMarker, InstalledFromMarker, SkillEligibilityContext, SkillCollision, SkillDiffLine, SkillValidationIssue, ScopeConfig, SkillResolution, } from './skills/index.js';

package/dist/index.js

@@ -147,7 +147,7 @@ export { createGuideTool, SHARED_GUIDE_ENTRIES, searchGuideEntries, topicToGuide
 // =============================================================================
 // Platform Tools (runtime — createPlatformTools factory + individual factories)
 // =============================================================================
-export { createPlatformTools, createProjectTools, createWorkItemTools, createDocumentTools, createPlanTools, createBacklogTools, createAnchorTools, createArtifactTools, createEpisodeTools, createCanvasTools, createSceneTools, SCENE_TOOL_NAMES, createSceneWriter, SCENE_JOURNAL_SIZE, createImageTools, ProjectAnchorStore, FileArtifactService, } from './platform/index.js';
+export { createPlatformTools, createProjectTools, createWorkItemTools, createDocumentTools, createPlanTools, createBacklogTools, createAnchorTools, createArtifactTools, createEpisodeTools, createCanvasTools, createSceneTools, SCENE_TOOL_NAMES, resolveSelfAlias, createSceneWriter, SCENE_JOURNAL_SIZE, createImageTools, ProjectAnchorStore, FileArtifactService, } from './platform/index.js';
 // =============================================================================
 // Platform Workflow (pure step-criteria logic)
 // =============================================================================

package/dist/platform/context.d.ts

@@ -47,9 +47,29 @@ export interface PlatformHooks {
         id: number;
         name: string;
     }) => void;
-    /** Resolve owner aliases (e.g., 'self' → active agent ID) */
+    /**
+     * Resolve owner aliases (e.g., 'self' → active agent ID).
+     *
+     * ⚠️ MUST read the agent executing RIGHT NOW, not the one that was active when the tools were
+     * built — platform tools are built once and memoised by every host, so a captured value freezes
+     * for the session. Same trap as `cwd`, below. Implement it with {@link resolveSelfAlias}.
+     */
     resolveOwner?: (alias: 'self') => string | undefined;
 }
+/**
+ * Turn a host's notion of "the agent running now" into an owner alias.
+ *
+ * `'default'` is the coordinator slot every single-agent session uses, not an identity, so it
+ * resolves to `undefined` — which callers then render as their own fallback (`'agent'` for a work
+ * item's author) rather than writing the literal string `default` into stored data.
+ *
+ * ⚠️ This lives here because BOTH hosts need the same answer and only one of them had it. The CLI
+ * implemented the rule inline; Desktop passed no `resolveOwner` at all, so every scene edit
+ * recorded no agent and `consumeUserEdits` keyed every agent on the empty string — one shared
+ * "already told" mark, so only the FIRST agent to touch a canvas was ever told about the user's
+ * edits to it. The rule being in one host and not the other is a bug in both.
+ */
+export declare function resolveSelfAlias(agentId: string | null | undefined): string | undefined;
 export interface PlatformToolsConfig {
     /** Data access layer */
     context: PlatformContext;

package/dist/platform/context.js

@@ -4,6 +4,24 @@
  * Provides a single entry point for data access, abstracting over the
  * underlying storage backend (SQLite in CLI, PostgreSQL in web/API).
  */
+/**
+ * Turn a host's notion of "the agent running now" into an owner alias.
+ *
+ * `'default'` is the coordinator slot every single-agent session uses, not an identity, so it
+ * resolves to `undefined` — which callers then render as their own fallback (`'agent'` for a work
+ * item's author) rather than writing the literal string `default` into stored data.
+ *
+ * ⚠️ This lives here because BOTH hosts need the same answer and only one of them had it. The CLI
+ * implemented the rule inline; Desktop passed no `resolveOwner` at all, so every scene edit
+ * recorded no agent and `consumeUserEdits` keyed every agent on the empty string — one shared
+ * "already told" mark, so only the FIRST agent to touch a canvas was ever told about the user's
+ * edits to it. The rule being in one host and not the other is a bug in both.
+ */
+export function resolveSelfAlias(agentId) {
+    if (!agentId || agentId === 'default')
+        return undefined;
+    return agentId;
+}
 /**
  * Resolve a `cwd` that may be a value or a getter.
  *

package/dist/platform/index.d.ts

@@ -5,6 +5,8 @@ export type { ProjectType, ProjectStatus, RepoPattern, WorkflowMode, LifecycleSt
 export type { IProjectRepository, IWorkItemRepository, IDocumentRepository, IPlanRepository, ICommentRepository, ICanvasRepository, } from './repositories.js';
 export type { IAnchorService, IArtifactService, IEpisodeService, ICanvasRenderer, RenderedCanvasImage, AnchorData, ArtifactType, ArtifactData, ArtifactSummaryData, WorkEpisode, ProjectWorkSummary, } from './services.js';
 export type { PlatformContext, PlatformToolsConfig, PlatformHooks } from './context.js';
+/** Owner-alias rule shared by every host — see its doc comment for why it lives here. */
+export { resolveSelfAlias } from './context.js';
 export { createPlatformTools, createProjectTools, createWorkItemTools, createDocumentTools, createPlanTools, createBacklogTools, createAnchorTools, createArtifactTools, createEpisodeTools, createCanvasTools, createSceneTools, SCENE_TOOL_NAMES, sceneOutline, createImageTools, } from './tools/index.js';
 export { createSceneWriter, SCENE_JOURNAL_SIZE } from './scene-writer.js';
 export type { ISceneWriter, SceneReadResult, SceneApplyResult, SceneCreateResult, SceneLastEdit, } from './scene-writer.js';

package/dist/platform/index.js

@@ -1,6 +1,8 @@
 /**
  * Platform — Repository interfaces, data models, tools, and workflow.
  */
+/** Owner-alias rule shared by every host — see its doc comment for why it lives here. */
+export { resolveSelfAlias } from './context.js';
 // Platform tools (runtime)
 export { createPlatformTools, createProjectTools, createWorkItemTools, createDocumentTools, createPlanTools, createBacklogTools, createAnchorTools, createArtifactTools, createEpisodeTools, createCanvasTools, createSceneTools, SCENE_TOOL_NAMES, sceneOutline, createImageTools, } from './tools/index.js';
 // The scene writer — one per host process, shared by the scene tools and the host's own

package/dist/team/team-agent.d.ts

@@ -8,7 +8,7 @@
  * - Optional tool filtering
  */
 import type { Agent, AgentConfig, ContextManager } from '@compilr-dev/agents';
-import type { TeamAgentConfig, SerializedTeamAgent, MascotExpression, AgentRole, ToolProfile } from './types.js';
+import type { TeamAgentConfig, SerializedTeamAgent, AgentRole, ToolProfile } from './types.js';
 import type { ModelTier } from '../models/index.js';
 import type { ProviderType } from '../config.js';
 import type { SharedContextManager } from './shared-context.js';
@@ -60,7 +60,8 @@ export interface TeamAgentUpdate {
      * model change), so `applyUpdate` delegates rather than reimplementing it.
      */
     modelTier?: ModelTier;
-    mascot?: MascotExpression;
+    /** See `TeamAgentConfig.mascot`: custom mascots are not in `MascotExpression`. */
+    mascot?: string;
     description?: string;
     systemPromptAddition?: string;
     /** Tone only. Composed into the prompt at build time — see `TeamAgent.personality`. */
@@ -150,10 +151,8 @@ export declare class TeamAgent {
      * Display name shown in UI
      */
     displayName: string;
-    /**
-     * Mascot expression for visual differentiation
-     */
-    mascot: MascotExpression;
+    /** See `TeamAgentConfig.mascot`: custom mascots are not in `MascotExpression`. */
+    mascot: string;
     /**
      * Role type
      */

package/dist/team/team-agent.js

@@ -165,9 +165,7 @@ export class TeamAgent {
      * Display name shown in UI
      */
     displayName;
-    /**
-     * Mascot expression for visual differentiation
-     */
+    /** See `TeamAgentConfig.mascot`: custom mascots are not in `MascotExpression`. */
     mascot;
     /**
      * Role type

package/dist/team/types.d.ts

@@ -96,9 +96,17 @@ export interface TeamAgentConfig {
      */
     displayName: string;
     /**
-     * Mascot expression for visual differentiation
+     * The agent's mascot face, e.g. "[◈_◈]".
+     *
+     * ⚠️ `string`, NOT `MascotExpression`. That union lists only the BUILT-IN role faces
+     * (`ROLE_METADATA`), while a custom agent's mascot comes from `CUSTOM_MASCOTS` — a disjoint
+     * pool (`[⊡_⊡]`, `[⊞_⊞]`, …) whose `assignMascot()` falls back to a random pick, and which
+     * the CLI's editor lets the user type freely. So every custom agent carried a mascot that was
+     * not a legal value of its own declared type; the mismatch was papered over by two `as`
+     * casts and only surfaced when the tests were first typechecked. `MascotExpression` keeps its
+     * real job on `RoleMetadata`, where the set genuinely is closed.
      */
-    mascot: MascotExpression;
+    mascot: string;
     /**
      * Role type (for presets)
      */
@@ -179,7 +187,8 @@ export interface TeamAgentConfig {
 export interface SerializedTeamAgent {
     id: string;
     displayName: string;
-    mascot: MascotExpression;
+    /** See `TeamAgentConfig.mascot` — a custom agent's face is not in `MascotExpression`. */
+    mascot: string;
     role: AgentRole;
     description?: string;
     systemPromptAddition?: string;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@compilr-dev/sdk",
-  "version": "0.35.0",
+  "version": "0.37.0",
   "description": "Universal agent runtime for building AI-powered applications",
   "type": "module",
   "main": "dist/index.js",
@@ -45,15 +45,16 @@
   "scripts": {
     "build": "tsc",
     "clean": "rm -rf dist",
-    "lint": "eslint src/",
-    "lint:fix": "eslint src/ --fix",
+    "lint": "eslint src/ tests/",
+    "lint:fix": "eslint src/ tests/ --fix",
     "format": "prettier --write \"src/**/*.ts\"",
     "format:check": "prettier --check \"src/**/*.ts\"",
     "test": "vitest run",
     "test:watch": "vitest",
     "test:coverage": "vitest run --coverage",
     "prepublishOnly": "npm run clean && npm run lint && npm run test && npm run build",
-    "typecheck": "tsc --noEmit"
+    "typecheck": "tsc --noEmit && npm run typecheck:tests",
+    "typecheck:tests": "tsc --noEmit -p tsconfig.test.json"
   },
   "repository": {
     "type": "git",