@sqdr2/dsh-agent-studio-fusion 1.0.0 → 1.0.1

package/lib/types/host/acp/config.d.ts

@@ -12,10 +12,19 @@ export declare const DEFAULT_CONTEXT_WINDOW = 200000;
 export declare const AcpAgentProfileSchema: z<AcpAgentProfile>;
 /** The `acp` block: `acp.agents` maps a provider route id to its profile. */
 export declare const AcpConfigSchema: z<AcpConfig>;
-/** Read the configured profiles; an absent block is an empty map, never an error. */
+/**
+ * Read the configured profiles; an absent block is an empty map, never an error.
+ *
+ * `agents` is declared volatile so the Host publishes a live-editable settings form, and
+ * a parsed Config therefore carries a `Volatile` reference there rather than the map. The
+ * reference is a `{ get() }` object whose only own key is `get`, so `Object.entries` over
+ * it yields the single route `"get"` — that route then reached the model catalog as a
+ * provider with an undefined profile, failing as `profile.models.length`. Unwrap the
+ * reference (and any plain value, for callers that hand in raw settings) before reading.
+ */
 export declare function agentsOf(config: {
     acp?: {
-        agents?: Record<string, AcpAgentProfile>;
+        agents?: unknown;
     };
 } | undefined): ReadonlyMap<string, AcpAgentProfile>;
 export declare function spawnFacts(profile: AcpAgentProfile): string;

package/lib/types/host/acp/types.d.ts

@@ -38,7 +38,15 @@ export interface AcpAgentProfile {
     startupTimeoutMs: number;
     idleTimeoutMs: number;
 }
-/** The `acp` configuration block: one profile per provider route. */
+/**
+ * The `acp` configuration block: one profile per provider route.
+ *
+ * `agents` is declared volatile so the Host publishes a live-editable settings form, so a
+ * parsed Config carries a `{ get() }` reference at that key rather than a plain map. Read it
+ * through `agentsOf()`; `Object.entries(config.acp.agents)` sees only the reference's own
+ * `get` method. The type below describes the declared (input) shape, which is what the
+ * schema validates.
+ */
 export interface AcpConfig {
     agents: Record<string, AcpAgentProfile>;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sqdr2/dsh-agent-studio-fusion",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "publishConfig": {
     "access": "public",
     "registry": "https://registry.npmjs.org/"
@@ -77,6 +77,7 @@
   },
   "dependencies": {
     "@agentclientprotocol/sdk": "1.4.0",
+    "@deepseek-ai/cosmokit": "~1.8.5",
     "qrcode": "^1.5.4",
     "undici": "^8.10.0"
   },

package/src/host/acp/agent-process.ts

@@ -536,7 +536,7 @@ export class AcpAgentProcess {
         connection.agent.request(methods.agent.initialize, {
           protocolVersion: PROTOCOL_VERSION,
           clientCapabilities: {},
-          clientInfo: { name: PLUGIN_ID, version: '1.0.0' },
+          clientInfo: { name: PLUGIN_ID, version: '1.0.1' },
         }),
         exited,
         timeout,

package/src/host/acp/config.ts

@@ -6,6 +6,7 @@
  */
 
 import z from '@deepseek-ai/schemastery'
+import { isVolatile } from '@deepseek-ai/cosmokit'
 import type { AcpAgentProfile, AcpConfig, AcpModelProfile } from './types.js'
 
 export const DEFAULT_MODEL_ID = 'default'
@@ -47,9 +48,20 @@ export const AcpConfigSchema: z<AcpConfig> = z.object({
   agents: z.dict(AcpAgentProfileSchema).default({}).volatile() as unknown as z<Record<string, AcpAgentProfile>>,
 })
 
-/** Read the configured profiles; an absent block is an empty map, never an error. */
-export function agentsOf(config: { acp?: { agents?: Record<string, AcpAgentProfile> } } | undefined): ReadonlyMap<string, AcpAgentProfile> {
-  return new Map(Object.entries(config?.acp?.agents ?? {}))
+/**
+ * Read the configured profiles; an absent block is an empty map, never an error.
+ *
+ * `agents` is declared volatile so the Host publishes a live-editable settings form, and
+ * a parsed Config therefore carries a `Volatile` reference there rather than the map. The
+ * reference is a `{ get() }` object whose only own key is `get`, so `Object.entries` over
+ * it yields the single route `"get"` — that route then reached the model catalog as a
+ * provider with an undefined profile, failing as `profile.models.length`. Unwrap the
+ * reference (and any plain value, for callers that hand in raw settings) before reading.
+ */
+export function agentsOf(config: { acp?: { agents?: unknown } } | undefined): ReadonlyMap<string, AcpAgentProfile> {
+  const agents = config?.acp?.agents
+  const plain: unknown = isVolatile(agents) ? agents.get() : agents
+  return new Map(Object.entries(plain ?? {}) as [string, AcpAgentProfile][])
 }
 
 export function spawnFacts(profile: AcpAgentProfile): string {

package/src/host/acp/types.ts

@@ -44,7 +44,15 @@ export interface AcpAgentProfile {
   idleTimeoutMs: number
 }
 
-/** The `acp` configuration block: one profile per provider route. */
+/**
+ * The `acp` configuration block: one profile per provider route.
+ *
+ * `agents` is declared volatile so the Host publishes a live-editable settings form, so a
+ * parsed Config carries a `{ get() }` reference at that key rather than a plain map. Read it
+ * through `agentsOf()`; `Object.entries(config.acp.agents)` sees only the reference's own
+ * `get` method. The type below describes the declared (input) shape, which is what the
+ * schema validates.
+ */
 export interface AcpConfig {
   agents: Record<string, AcpAgentProfile>
 }