@nuvin/nuvin-core 1.1.1 → 1.1.2

package/dist/VERSION

@@ -1,4 +1,4 @@
 {
-  "version": "1.1.1",
-  "commit": "6744c77"
+  "version": "1.1.2",
+  "commit": "76be190"
 }

package/dist/index.d.ts

@@ -19,6 +19,11 @@ type AgentTemplate = {
     shareContext?: boolean;
     metadata?: Record<string, unknown>;
 };
+/**
+ * CompleteAgent is an AgentTemplate with all required fields populated
+ * Used for registered agents that have gone through applyDefaults()
+ */
+type CompleteAgent = Required<Pick<AgentTemplate, 'id' | 'name' | 'description' | 'systemPrompt' | 'tools' | 'temperature' | 'maxTokens'>> & Pick<AgentTemplate, 'provider' | 'model' | 'topP' | 'timeoutMs' | 'shareContext' | 'metadata'>;
 /**
  * Specialist Agent Configuration (Internal - used by AgentManager)
  */
@@ -135,7 +140,7 @@ declare class AgentRegistry {
      */
     applyDefaults(partial: Partial<AgentTemplate> & {
         systemPrompt: string;
-    }): AgentTemplate;
+    }): CompleteAgent;
     /**
      * Load agents from memory persistence
      */
@@ -161,7 +166,7 @@ declare class AgentRegistry {
     /**
      * Save agent to file
      */
-    saveToFile(agent: AgentTemplate): Promise<void>;
+    saveToFile(agent: CompleteAgent): Promise<void>;
     /**
      * Delete agent from file
      */
@@ -177,11 +182,11 @@ declare class AgentRegistry {
     /**
      * Get an agent template by ID
      */
-    get(agentId: string): AgentTemplate | undefined;
+    get(agentId: string): CompleteAgent | undefined;
     /**
      * List all registered agent templates
      */
-    list(): AgentTemplate[];
+    list(): CompleteAgent[];
     /**
      * Check if an agent exists
      */
@@ -922,7 +927,7 @@ declare class DefaultSpecialistAgentFactory implements SpecialistAgentFactory {
         systemContextProvider?: SystemContextProvider;
         agentListProvider?: AgentListProvider;
         idGenerator?: IdGenerator;
-    });
+    } | undefined);
     create(input: SpecialistAgentFactoryInput): SpecialistAgentConfig;
 }
 
@@ -938,7 +943,7 @@ declare class AgentManager {
     private llmResolver;
     private activeAgents;
     private eventCollectors;
-    constructor(delegatingConfig: AgentConfig, delegatingTools: ToolPort, llmFactory?: LLMFactory, eventCallback?: (event: AgentEvent) => void, configResolver?: () => Partial<AgentConfig>);
+    constructor(delegatingConfig: AgentConfig, delegatingTools: ToolPort, llmFactory?: LLMFactory | undefined, eventCallback?: ((event: AgentEvent) => void) | undefined, configResolver?: (() => Partial<AgentConfig>) | undefined);
     /**
      * Create and execute a specialist agent for a specific task
      */
@@ -970,7 +975,7 @@ declare class AgentManagerCommandRunner implements AgentCommandRunner {
     private readonly delegatingTools;
     private readonly llmFactory?;
     private readonly configResolver?;
-    constructor(delegatingConfig: AgentConfig, delegatingTools: ToolPort, llmFactory?: LLMFactory, configResolver?: () => Partial<AgentConfig>);
+    constructor(delegatingConfig: AgentConfig, delegatingTools: ToolPort, llmFactory?: LLMFactory | undefined, configResolver?: (() => Partial<AgentConfig>) | undefined);
     run(config: Parameters<AgentManager['executeTask']>[0], context?: ToolExecutionContext): Promise<SpecialistAgentResult>;
 }
 
@@ -995,7 +1000,7 @@ declare class DefaultDelegationResultFormatter implements DelegationResultFormat
             status: "success" | "error" | "timeout";
             executionTimeMs: number;
             toolCallsExecuted: number;
-            tokensUsed: number;
+            tokensUsed: number | undefined;
         };
     };
     formatError(error: unknown): string;
@@ -1157,9 +1162,9 @@ declare class GithubAuthTransport implements HttpTransport {
 }
 
 declare class LLMError extends Error {
-    readonly statusCode?: number;
+    readonly statusCode?: number | undefined;
     readonly isRetryable: boolean;
-    constructor(message: string, statusCode?: number, isRetryable?: boolean, cause?: unknown);
+    constructor(message: string, statusCode?: number | undefined, isRetryable?: boolean, cause?: unknown);
 }
 declare abstract class BaseLLM implements LLMPort {
     protected transport: HttpTransport | null;
@@ -1328,6 +1333,7 @@ declare function loadMCPConfig(filePath?: string): Promise<MCPConfig | null>;
 declare class PersistingConsoleEventPort implements EventPort {
     private memory;
     private maxPerConversation;
+    private writeQueue;
     constructor(opts?: {
         memory?: MemoryPort<AgentEvent>;
         filename?: string;
@@ -1342,4 +1348,4 @@ declare function resolveBackspaces(s: string): string;
 declare function stripAnsiAndControls(s: string): string;
 declare function canonicalizeTerminalPaste(raw: string): string;
 
-export { AGENT_CREATOR_SYSTEM_PROMPT, type AgentAwareToolPort, type AgentCatalog, type AgentConfig, type AgentEvent, AgentEventTypes, AgentFilePersistence, AgentManager, AgentManagerCommandRunner, AgentOrchestrator, AgentRegistry, type AgentTemplate, AnthropicAISDKLLM, type AssignParams, BashTool, CompositeToolPort, type Conversation, ConversationContext, type ConversationMetadata, type ConversationSnapshot, ConversationStore, CoreMCPClient, DefaultDelegationPolicy, DefaultDelegationResultFormatter, DefaultDelegationService, DefaultSpecialistAgentFactory, type DelegationService, type DelegationServiceConfig, DelegationServiceFactory, ErrorReason, type FolderTreeOptions, GithubLLM, InMemoryMemory, InMemoryMetadata, JsonFileMemoryPersistence, type LLMConfig, LLMError, type LLMFactory, type LLMOptions, type LLMPort, LLMResolver, type MCPConfig, type MCPServerConfig, MCPToolPort, type MemoryPort, MemoryPortMetadataAdapter, type Message, type MessageContent, type MessageContentPart, type MetadataPort, NoopReminders, type OrchestratorAwareToolPort, PersistedMemory, PersistingConsoleEventPort, RuntimeEnv, type SendMessageOptions, SimpleContextBuilder, SimpleCost, SimpleId, type SpecialistAgentConfig, type SpecialistAgentResult, SystemClock, type ToolApprovalDecision, type ToolCall, type ToolExecutionResult, type ToolPort, ToolRegistry, type UserAttachment, type UserMessagePayload, buildAgentCreationPrompt, buildInjectedSystem, canonicalizeTerminalPaste, createLLM, generateFolderTree, getAvailableProviders, loadMCPConfig, normalizeNewlines, renderTemplate, resolveBackspaces, resolveCarriageReturns, stripAnsiAndControls, supportsGetModels };
+export { AGENT_CREATOR_SYSTEM_PROMPT, type AgentAwareToolPort, type AgentCatalog, type AgentConfig, type AgentEvent, AgentEventTypes, AgentFilePersistence, AgentManager, AgentManagerCommandRunner, AgentOrchestrator, AgentRegistry, type AgentTemplate, AnthropicAISDKLLM, type AssignParams, BashTool, type CompleteAgent, CompositeToolPort, type Conversation, ConversationContext, type ConversationMetadata, type ConversationSnapshot, ConversationStore, CoreMCPClient, DefaultDelegationPolicy, DefaultDelegationResultFormatter, DefaultDelegationService, DefaultSpecialistAgentFactory, type DelegationService, type DelegationServiceConfig, DelegationServiceFactory, ErrorReason, type FolderTreeOptions, GithubLLM, InMemoryMemory, InMemoryMetadata, JsonFileMemoryPersistence, type LLMConfig, LLMError, type LLMFactory, type LLMOptions, type LLMPort, LLMResolver, type MCPConfig, type MCPServerConfig, MCPToolPort, type MemoryPort, MemoryPortMetadataAdapter, type Message, type MessageContent, type MessageContentPart, type MetadataPort, NoopReminders, type OrchestratorAwareToolPort, PersistedMemory, PersistingConsoleEventPort, RuntimeEnv, type SendMessageOptions, SimpleContextBuilder, SimpleCost, SimpleId, type SpecialistAgentConfig, type SpecialistAgentResult, SystemClock, type ToolApprovalDecision, type ToolCall, type ToolExecutionResult, type ToolPort, ToolRegistry, type UserAttachment, type UserMessagePayload, buildAgentCreationPrompt, buildInjectedSystem, canonicalizeTerminalPaste, createLLM, generateFolderTree, getAvailableProviders, loadMCPConfig, normalizeNewlines, renderTemplate, resolveBackspaces, resolveCarriageReturns, stripAnsiAndControls, supportsGetModels };

package/dist/index.js

@@ -2384,8 +2384,12 @@ var BashTool = class {
       }
       arr.push(chunk);
     };
-    child.stdout.on("data", (d) => capPush(stdout, d));
-    child.stderr.on("data", (d) => capPush(stderr, d));
+    if (child.stdout) {
+      child.stdout.on("data", (d) => capPush(stdout, d));
+    }
+    if (child.stderr) {
+      child.stderr.on("data", (d) => capPush(stderr, d));
+    }
     const exit = new Promise((res) => {
       child.on("close", (code, signal2) => res({ code, signal: signal2 }));
     });
@@ -2643,8 +2647,10 @@ var AgentRegistry = class {
     this.filePersistence = options?.filePersistence;
     for (const agent of defaultAgents) {
       const complete = this.applyDefaults(agent);
-      this.agents.set(complete.id, complete);
-      this.defaultAgentIds.add(complete.id);
+      if (complete.id) {
+        this.agents.set(complete.id, complete);
+        this.defaultAgentIds.add(complete.id);
+      }
     }
     this.loadingPromise = this.loadAgents();
   }
@@ -2726,7 +2732,7 @@ var AgentRegistry = class {
       for (const agent of loadedAgents) {
         if (this.validateTemplate(agent)) {
           const complete = this.applyDefaults(agent);
-          if (!this.defaultAgentIds.has(complete.id)) {
+          if (complete.id && !this.defaultAgentIds.has(complete.id)) {
             this.agents.set(complete.id, complete);
           }
         }
@@ -2775,9 +2781,6 @@ var AgentRegistry = class {
     if (!this.filePersistence) {
       throw new Error("File persistence not configured");
     }
-    if (!agent.id) {
-      throw new Error("Cannot save agent without ID");
-    }
     if (this.defaultAgentIds.has(agent.id)) {
       throw new Error(`Cannot save default agent "${agent.id}" to file`);
     }
@@ -2999,8 +3002,8 @@ var AgentManager = class {
     this.llmFactory = llmFactory;
     this.eventCallback = eventCallback;
     this.configResolver = configResolver;
-    if (llmFactory) {
-      this.llmResolver = new LLMResolver(llmFactory);
+    if (this.llmFactory) {
+      this.llmResolver = new LLMResolver(this.llmFactory);
     }
   }
   llmResolver = null;
@@ -3178,7 +3181,9 @@ var AgentManager = class {
    */
   resolveLLM(config) {
     if (!this.llmResolver) {
-      throw new Error("AgentManager requires LLMFactory to create sub-agents. Please provide llmFactory in constructor.");
+      throw new Error(
+        "AgentManager requires LLMFactory to create sub-agents. Please provide llmFactory in constructor."
+      );
     }
     return this.llmResolver.resolve(config);
   }
@@ -3411,9 +3416,7 @@ var ToolRegistry = class {
     const delegationService = factory.create({
       agentRegistry: this.agentRegistry,
       commandRunner,
-      agentListProvider: () => this.agentRegistry.list().filter(
-        (agent) => typeof agent.id === "string" && typeof agent.name === "string" && typeof agent.description === "string"
-      ).map((agent) => ({
+      agentListProvider: () => this.agentRegistry.list().map((agent) => ({
         id: agent.id,
         name: agent.name,
         description: agent.description
@@ -3685,9 +3688,7 @@ var AgentFilePersistence = class {
         fs8.unlinkSync(filePath);
       }
     } catch (error) {
-      throw new Error(
-        `Failed to delete agent ${agentId}: ${error instanceof Error ? error.message : String(error)}`
-      );
+      throw new Error(`Failed to delete agent ${agentId}: ${error instanceof Error ? error.message : String(error)}`);
     }
   }
   /**
@@ -3927,7 +3928,7 @@ var BaseLLM = class {
         }
         for (const ch of choices) {
           const delta = ch.delta ?? ch.message ?? {};
-          const textDelta = delta.content || delta.reasoning;
+          const textDelta = delta.content ?? void 0;
           if (typeof textDelta === "string" && textDelta.length > 0) {
             if (content === "") {
               const trimmedDelta = textDelta.replace(/^\n+/, "");
@@ -4393,7 +4394,7 @@ var GithubAuthTransport = class {
     if (!Array.isArray(messages.messages)) return false;
     return messages.messages.some((msg) => isVisionMessage(msg));
   }
-  makeAuthHeaders(url, headers, body) {
+  makeAuthHeaders(headers, body) {
     const base = headers ? { ...headers } : {};
     if (this.apiKey) base.Authorization = `Bearer ${this.apiKey}`;
     base["editor-version"] = base["editor-version"] || "vscode/1.104.2";
@@ -4431,11 +4432,11 @@ var GithubAuthTransport = class {
       await this.exchangeToken(signal);
     }
     const fullUrl = this.buildFullUrl(url);
-    let res = await this.inner.get(fullUrl, this.makeAuthHeaders(fullUrl, headers), signal);
+    let res = await this.inner.get(fullUrl, this.makeAuthHeaders(headers), signal);
     if (res.status === 401 && this.accessToken) {
       await this.exchangeToken(signal);
       const retryUrl = this.buildFullUrl(url);
-      res = await this.inner.get(retryUrl, this.makeAuthHeaders(retryUrl, headers), signal);
+      res = await this.inner.get(retryUrl, this.makeAuthHeaders(headers), signal);
     }
     return res;
   }
@@ -4444,11 +4445,11 @@ var GithubAuthTransport = class {
       await this.exchangeToken(signal);
     }
     const fullUrl = this.buildFullUrl(url);
-    let res = await this.inner.postJson(fullUrl, body, this.makeAuthHeaders(fullUrl, headers, body), signal);
+    let res = await this.inner.postJson(fullUrl, body, this.makeAuthHeaders(headers, body), signal);
     if (res.status === 401 && this.accessToken) {
       await this.exchangeToken(signal);
       const retryUrl = this.buildFullUrl(url);
-      res = await this.inner.postJson(retryUrl, body, this.makeAuthHeaders(retryUrl, headers, body), signal);
+      res = await this.inner.postJson(retryUrl, body, this.makeAuthHeaders(headers, body), signal);
     }
     return res;
   }
@@ -4457,12 +4458,12 @@ var GithubAuthTransport = class {
       await this.exchangeToken(signal);
     }
     const fullUrl = this.buildFullUrl(url);
-    const hdrs = this.makeAuthHeaders(fullUrl, { Accept: "text/event-stream", ...headers || {} }, body);
+    const hdrs = this.makeAuthHeaders({ Accept: "text/event-stream", ...headers || {} }, body);
     let res = await this.inner.postStream(fullUrl, body, hdrs, signal);
     if (res.status === 401 && this.accessToken) {
       await this.exchangeToken(signal);
       const retryUrl = this.buildFullUrl(url);
-      res = await this.inner.postStream(retryUrl, body, this.makeAuthHeaders(retryUrl, hdrs, body), signal);
+      res = await this.inner.postStream(retryUrl, body, this.makeAuthHeaders(hdrs, body), signal);
     }
     return res;
   }
@@ -4601,9 +4602,11 @@ var AnthropicAISDKLLM = class {
   }
   updateCredentials(result) {
     if (result.type === "success" && result.access && result.refresh && result.expires) {
-      this.opts.oauth.access = result.access;
-      this.opts.oauth.refresh = result.refresh;
-      this.opts.oauth.expires = result.expires;
+      if (this.opts.oauth) {
+        this.opts.oauth.access = result.access;
+        this.opts.oauth.refresh = result.refresh;
+        this.opts.oauth.expires = result.expires;
+      }
       this.opts.onTokenUpdate?.({
         access: result.access,
         refresh: result.refresh,
@@ -5405,7 +5408,7 @@ function normalizeMCPConfig(raw) {
       return { mcpServers: servers };
     }
   }
-  if (asRecord.config && typeof asRecord.config === "object") {
+  if ("config" in asRecord && asRecord.config && typeof asRecord.config === "object") {
     const nested = asRecord.config;
     if (isValidConfig(nested)) {
       const servers = nested.mcpServers;
@@ -5434,23 +5437,26 @@ function isValidConfig(value) {
 
 // events.ts
 var PersistingConsoleEventPort = class {
-  // private console = new ConsoleEventPort();
   memory;
   maxPerConversation;
+  writeQueue = Promise.resolve();
   constructor(opts) {
     this.memory = opts?.memory ?? new PersistedMemory(new JsonFileMemoryPersistence(opts?.filename || "events.json"));
     this.maxPerConversation = opts?.maxPerConversation ?? 500;
   }
   async emit(event) {
-    try {
-      const key = event?.conversationId ?? "default";
-      const existing = await this.memory.get(key);
-      const next = [...existing, { ...event }];
-      const max = this.maxPerConversation;
-      const trimmed = max > 0 && next.length > max ? next.slice(next.length - max) : next;
-      await this.memory.set(key, trimmed);
-    } catch {
-    }
+    this.writeQueue = this.writeQueue.then(async () => {
+      try {
+        const key = event?.conversationId ?? "default";
+        const existing = await this.memory.get(key);
+        const next = [...existing, { ...event }];
+        const max = this.maxPerConversation;
+        const trimmed = max > 0 && next.length > max ? next.slice(next.length - max) : next;
+        await this.memory.set(key, trimmed);
+      } catch {
+      }
+    });
+    return this.writeQueue;
   }
 };
 export {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nuvin/nuvin-core",
-  "version": "1.1.1",
+  "version": "1.1.2",
   "description": "",
   "private": false,
   "main": "dist/index.js",