@truefoundry/assistant-ui-runtime 0.1.7 → 0.1.9

package/src/server/types.ts

@@ -21,6 +21,7 @@ import type {
 export interface ModelSelectorEntry {
     name: string;
     provider: string;
+    reasoningEfforts?: string[];
 }
 
 /** Skill selector row. Host extends for fqn, preload, etc. */
@@ -249,7 +250,7 @@ export interface AgentChatServer<
     getSession(req: { sessionId: string }): Promise<TSession>;
     updateSession(req: TUpdate): Promise<TSession>;
 
-    prepareAndExecuteTurn(req: {
+    createTurn(req: {
         sessionId: string;
         input?: TurnInputItem[];
         previousTurnId?: PreviousTurnIdInput;
@@ -405,33 +406,36 @@ export interface ModelCatalogServer<
 export interface ToolBase {
     id: string;
     name: string;
+    description: string;
 }
 
-/**
- * Auth type id. Reserved literals: `"None"`, `"OAuth"`, `"API Key"`.
- * Stays `string` so hosts can widen (same pattern as `ProviderType`).
- */
-export type ConnectorAuthType = string;
+/** Strict auth type id. Hosts widen branches via intersection + re-union. */
+export type ConnectorAuthType = "dcr" | "header" | "none";
 
-/**
- * Write-time connector auth. Host extends / narrows via `TType`.
- * For `"API Key"`, pass `apiKey` (and optional `headerName`).
- */
-export interface ConnectorAuth<TType extends ConnectorAuthType = ConnectorAuthType> {
-    type: TType;
+// Write (create/update) — export branches so hosts can intersect extras
+export type ConnectorAuthOAuth = { type: "dcr"; authUrl?: string };
+export type ConnectorAuthApiKey = {
+    type: "header";
     apiKey?: string;
     headerName?: string;
-}
-
-/**
- * Catalog / list auth — no secrets. Host extends / narrows via `TType`.
- */
-export interface ConnectorAuthPublic<
-    TType extends ConnectorAuthType = ConnectorAuthType,
-> {
-    type: TType;
+};
+export type ConnectorAuthNone = { type: "none" };
+export type ConnectorAuth =
+    | ConnectorAuthOAuth
+    | ConnectorAuthApiKey
+    | ConnectorAuthNone;
+
+// Public (list/detail) — no secrets; dcr requires authUrl
+export type ConnectorAuthPublicOAuth = { type: "dcr"; authUrl: string };
+export type ConnectorAuthPublicApiKey = {
+    type: "header";
     headerName?: string;
-}
+};
+export type ConnectorAuthPublicNone = { type: "none" };
+export type ConnectorAuthPublic =
+    | ConnectorAuthPublicOAuth
+    | ConnectorAuthPublicApiKey
+    | ConnectorAuthPublicNone;
 
 /**
  * MCP / connector create-edit config. Host extends for extra fields, etc.
@@ -457,6 +461,8 @@ export interface ConnectorBase<
     description: string;
     url: string;
     auth: TAuth;
+    /** When true, UI should not show Disconnect. */
+    requiresAuth: boolean;
     authenticated: boolean;
     tools: TTool[];
 }
@@ -500,7 +506,10 @@ export interface ConnectorCatalogServer<
     createConnector(req: TCreate): Promise<TConnector>;
     /** Full replace update keyed by connector `id`. */
     updateConnector(req: TUpdate): Promise<TConnector>;
-    /** Start connector auth (e.g. OAuth). Host may widen return with `authUrl`. */
+    /**
+     * Start connector auth (e.g. OAuth).
+     * For oauth, the returned connector's `auth.authUrl` is the authorize URL.
+     */
     authenticateConnector(req: { id: string }): Promise<TConnector>;
     /** Clear connector auth. */
     disconnectConnector(req: { id: string }): Promise<TConnector>;
@@ -518,34 +527,122 @@ export interface SkillBase {
     description: string;
 }
 
-/** Create-skill request. Host extends for branch, auth, etc. */
-export interface CreateSkillRequest {
-    repo: string;
-    directory: string;
+export interface RegistrySkill extends SkillBase {
+    /** `SkillCatalogEntry.id` this skill was created from. */
+    catalogId: string;
 }
 
+export interface GithubSkill extends SkillBase {}
+
+export type DefinedSkill = RegistrySkill | GithubSkill;
+
+/** Git source fields shared by catalog entries and create requests. */
+export interface SkillConfigBase {
+    name: string;
+    description: string;
+    repoURL: string;
+    path: string;
+    ref: string;
+}
+
+export interface SkillCatalogEntry extends SkillConfigBase {
+    id: string;
+}
+
+/** Create-skill base. Hosts may intersect extra fields and re-union. */
+export interface CreateSkillRequestBase extends SkillConfigBase {}
+
+export interface SelectRegistrySkillRequest extends CreateSkillRequestBase {
+    /** `SkillCatalogEntry.id`, persisted so the created skill links back to it. */
+    catalogId: string;
+}
+
+export interface ImportGithubSkillRequest extends CreateSkillRequestBase {}
+
+export type CreateSkillRequest =
+    | SelectRegistrySkillRequest
+    | ImportGithubSkillRequest;
+
 export interface SkillCatalogServer<
     TSkill extends SkillBase = SkillBase,
+    TCatalogEntry extends SkillCatalogEntry = SkillCatalogEntry,
     TCreate extends CreateSkillRequest = CreateSkillRequest,
 > {
+    getSkillCatalog(): Promise<TCatalogEntry[]>;
     listSkills(req?: { query?: string }): Promise<TSkill[]>;
     createSkill(req: TCreate): Promise<TSkill>;
     deleteSkill?(req: { id: string }): Promise<void>;
 }
 
+// ---------------------------------------------------------------------------
+// Sandboxes catalog — public rows omit credentials; writes accept them
+// ---------------------------------------------------------------------------
+
+/** Mutable sandbox settings shared by catalog rows, create, and update. */
+export interface SandboxConfig {
+    snapshotName: string;
+    execTimeoutMs: number;
+    autoStopIntervalInMinutes: number;
+    autoArchiveIntervalInMinutes: number;
+    autoDeleteIntervalInMinutes: number;
+}
+
+export interface SandboxCatalogEntry extends SandboxConfig {
+    id: string;
+    name: string;
+    type: string;
+}
+
+export interface SandboxBase {
+    id: string;
+    name: string;
+    catalogId: string;
+    isConnected: boolean;
+}
+
+export interface CreateSandboxRequest extends SandboxConfig {
+    /** `SandboxCatalogEntry.id` used to create this sandbox. */
+    catalogId: string;
+    name: string;
+    type: string;
+    apiKey: string;
+}
+
+export interface UpdateSandboxRequest extends SandboxConfig {
+    id: string;
+    apiKey: string;
+}
+
+export interface SandboxCatalogServer<
+    TSandbox extends SandboxBase = SandboxBase,
+    TCatalogEntry extends SandboxCatalogEntry = SandboxCatalogEntry,
+    TCreate extends CreateSandboxRequest = CreateSandboxRequest,
+    TUpdate extends UpdateSandboxRequest = UpdateSandboxRequest,
+> {
+    getSandboxCatalog(): Promise<TCatalogEntry[]>;
+    listSandboxes(req?: { query?: string }): Promise<TSandbox[]>;
+    createSandbox(req: TCreate): Promise<TSandbox>;
+    updateSandbox(req: TUpdate): Promise<TSandbox>;
+    deleteSandbox(req: { id: string }): Promise<void>;
+}
+
 /**
- * Settings management aggregate — modelCatalog + connectorCatalog + optional skillCatalog.
+ * Settings management aggregate — modelCatalog + connectorCatalog + optional
+ * skill and sandbox catalogs.
  * Hosts may pass the whole object to an app shell, or a focused sub-port to a page.
  */
 export interface CatalogServer<
     TModelCatalog extends ModelCatalogServer = ModelCatalogServer,
     TConnectorCatalog extends ConnectorCatalogServer = ConnectorCatalogServer,
     TSkillCatalog extends SkillCatalogServer = SkillCatalogServer,
+    TSandboxCatalog extends SandboxCatalogServer = SandboxCatalogServer,
 > {
     modelCatalog: TModelCatalog;
     connectorCatalog: TConnectorCatalog;
     /** Optional — omit when the host has no skills settings surface. */
     skillCatalog?: TSkillCatalog;
+    /** Optional — omit when the host has no sandboxes settings surface. */
+    sandboxCatalog?: TSandboxCatalog;
 }
 
 /**

package/src/streamTurn.test.ts

@@ -33,7 +33,7 @@ describe("streamTurn", () => {
     describe("streamTurnContent", () => {
         it("prepares a user turn and yields folded stream updates", async () => {
             const foldState = new PeerThreadFoldState();
-            const prepareAndExecuteTurn = vi.fn(async function* () {
+            const createTurn = vi.fn(async function* () {
                 yield streamData(1, {
                     type: "model.message",
                     createdAt,
@@ -43,7 +43,7 @@ describe("streamTurn", () => {
                 });
             });
             const server = mockServer({
-                prepareAndExecuteTurn,
+                createTurn,
                 cancelSession: vi.fn().mockResolvedValue(undefined),
             });
 
@@ -57,7 +57,7 @@ describe("streamTurn", () => {
                 ),
             );
 
-            expect(prepareAndExecuteTurn).toHaveBeenCalledWith({
+            expect(createTurn).toHaveBeenCalledWith({
                 sessionId: SESSION_ID,
                 input: [{ type: "user.message", content: "hello" }],
                 previousTurnId: "auto",
@@ -68,7 +68,7 @@ describe("streamTurn", () => {
             ]);
         });
 
-        it("passes required-action inputs through prepareAndExecuteTurn", async () => {
+        it("passes required-action inputs through createTurn", async () => {
             const inputs = [
                 {
                     type: "user.tool_approval" as const,
@@ -83,9 +83,9 @@ describe("streamTurn", () => {
                     content: "A",
                 },
             ];
-            const prepareAndExecuteTurn = vi.fn(async function* () {});
+            const createTurn = vi.fn(async function* () {});
             const server = mockServer({
-                prepareAndExecuteTurn,
+                createTurn,
                 cancelSession: vi.fn().mockResolvedValue(undefined),
             });
 
@@ -99,7 +99,7 @@ describe("streamTurn", () => {
                 ),
             );
 
-            expect(prepareAndExecuteTurn).toHaveBeenCalledWith({
+            expect(createTurn).toHaveBeenCalledWith({
                 sessionId: SESSION_ID,
                 input: inputs,
                 previousTurnId: "auto",
@@ -108,9 +108,9 @@ describe("streamTurn", () => {
         });
 
         it("uses empty input when resuming after MCP auth", async () => {
-            const prepareAndExecuteTurn = vi.fn(async function* () {});
+            const createTurn = vi.fn(async function* () {});
             const server = mockServer({
-                prepareAndExecuteTurn,
+                createTurn,
                 cancelSession: vi.fn().mockResolvedValue(undefined),
             });
 
@@ -124,7 +124,7 @@ describe("streamTurn", () => {
                 ),
             );
 
-            expect(prepareAndExecuteTurn).toHaveBeenCalledWith({
+            expect(createTurn).toHaveBeenCalledWith({
                 sessionId: SESSION_ID,
                 input: [],
                 previousTurnId: "auto",
@@ -133,9 +133,9 @@ describe("streamTurn", () => {
         });
 
         it("forwards an explicit previousTurnId when branching", async () => {
-            const prepareAndExecuteTurn = vi.fn(async function* () {});
+            const createTurn = vi.fn(async function* () {});
             const server = mockServer({
-                prepareAndExecuteTurn,
+                createTurn,
                 cancelSession: vi.fn().mockResolvedValue(undefined),
             });
 
@@ -149,7 +149,7 @@ describe("streamTurn", () => {
                 ),
             );
 
-            expect(prepareAndExecuteTurn).toHaveBeenCalledWith({
+            expect(createTurn).toHaveBeenCalledWith({
                 sessionId: SESSION_ID,
                 input: [{ type: "user.message", content: "edited" }],
                 previousTurnId: "turn-a",
@@ -158,9 +158,9 @@ describe("streamTurn", () => {
         });
 
         it("forwards previousTurnId \"none\" when branching from root", async () => {
-            const prepareAndExecuteTurn = vi.fn(async function* () {});
+            const createTurn = vi.fn(async function* () {});
             const server = mockServer({
-                prepareAndExecuteTurn,
+                createTurn,
                 cancelSession: vi.fn().mockResolvedValue(undefined),
             });
 
@@ -174,7 +174,7 @@ describe("streamTurn", () => {
                 ),
             );
 
-            expect(prepareAndExecuteTurn).toHaveBeenCalledWith({
+            expect(createTurn).toHaveBeenCalledWith({
                 sessionId: SESSION_ID,
                 input: [{ type: "user.message", content: "first" }],
                 previousTurnId: "none",
@@ -183,9 +183,9 @@ describe("streamTurn", () => {
         });
 
         it("returns early and cancels the session when already aborted", async () => {
-            const prepareAndExecuteTurn = vi.fn(async function* () {});
+            const createTurn = vi.fn(async function* () {});
             const cancelSession = vi.fn().mockResolvedValue(undefined);
-            const server = mockServer({ prepareAndExecuteTurn, cancelSession });
+            const server = mockServer({ createTurn, cancelSession });
             const abortController = new AbortController();
             abortController.abort();
 
@@ -200,14 +200,14 @@ describe("streamTurn", () => {
             );
 
             expect(cancelSession).toHaveBeenCalledWith({ sessionId: SESSION_ID });
-            expect(prepareAndExecuteTurn).not.toHaveBeenCalled();
+            expect(createTurn).not.toHaveBeenCalled();
             expect(updates).toEqual([]);
         });
 
-        it("forwards headers to prepareAndExecuteTurn", async () => {
-            const prepareAndExecuteTurn = vi.fn(async function* () {});
+        it("forwards headers to createTurn", async () => {
+            const createTurn = vi.fn(async function* () {});
             const server = mockServer({
-                prepareAndExecuteTurn,
+                createTurn,
                 cancelSession: vi.fn().mockResolvedValue(undefined),
             });
             const abortSignal = new AbortController().signal;
@@ -227,7 +227,7 @@ describe("streamTurn", () => {
                 ),
             );
 
-            expect(prepareAndExecuteTurn).toHaveBeenCalledWith({
+            expect(createTurn).toHaveBeenCalledWith({
                 sessionId: SESSION_ID,
                 input: [{ type: "user.message", content: "hello" }],
                 previousTurnId: "auto",
@@ -240,7 +240,7 @@ describe("streamTurn", () => {
 
         it("notifies gateway turn id when turn.done errors with no content yields", async () => {
             const gatewayTurnId = "01ky6mqzmczwt6ssyd5r02gjjc";
-            const prepareAndExecuteTurn = vi.fn(async function* () {
+            const createTurn = vi.fn(async function* () {
                 yield streamData(1, {
                     type: "turn.created",
                     createdAt,
@@ -261,7 +261,7 @@ describe("streamTurn", () => {
                 });
             });
             const server = mockServer({
-                prepareAndExecuteTurn,
+                createTurn,
                 cancelSession: vi.fn().mockResolvedValue(undefined),
             });
             const onTurnIdAvailable = vi.fn();
@@ -285,7 +285,7 @@ describe("streamTurn", () => {
         });
 
         it("does not notify when an error stream never emits turn.created", async () => {
-            const prepareAndExecuteTurn = vi.fn(async function* () {
+            const createTurn = vi.fn(async function* () {
                 yield streamData(1, {
                     type: "turn.done",
                     createdAt,
@@ -298,7 +298,7 @@ describe("streamTurn", () => {
                 });
             });
             const server = mockServer({
-                prepareAndExecuteTurn,
+                createTurn,
                 cancelSession: vi.fn().mockResolvedValue(undefined),
             });
             const onTurnIdAvailable = vi.fn();

package/src/streamTurn.ts

@@ -18,7 +18,7 @@ export type StreamTurnOptions = {
     resumeMcpAuth?: boolean;
     inputs?: RequiredActionInput[];
     /**
-     * Branch anchor for prepareAndExecuteTurn. Omit for `"auto"`. Pass `"none"` for a fresh
+     * Branch anchor for createTurn. Omit for `"auto"`. Pass `"none"` for a fresh
      * root turn.
      */
     previousTurnId?: PreviousTurnIdInput;
@@ -79,7 +79,7 @@ export async function* streamTurnContent(
         }
     };
 
-    const stream: AsyncIterable<TurnStreamData> = server.prepareAndExecuteTurn({
+    const stream: AsyncIterable<TurnStreamData> = server.createTurn({
         sessionId,
         input: buildTurnInput(options),
         previousTurnId: options.previousTurnId ?? "auto",

package/src/truefoundryOwnedSessionsThreadListAdapter.ts

@@ -63,7 +63,10 @@ export function createTrueFoundryOwnedSessionsThreadListAdapter(options: {
         async rename() {},
         async archive() {},
         async unarchive() {},
-        async delete() {},
+        async delete(remoteId) {
+            if (typeof server.deleteSession !== "function") return;
+            await server.deleteSession({ sessionId: remoteId });
+        },
 
         async generateTitle() {
             return new ReadableStream();

package/src/truefoundryThreadListAdapter.test.ts

@@ -98,4 +98,26 @@ describe("createTrueFoundryThreadListAdapter", () => {
 
         expect(result.nextCursor).toBeUndefined();
     });
+
+    it("delete calls server.deleteSession when implemented", async () => {
+        const deleteSession = vi.fn().mockResolvedValue(undefined);
+        const server = mockServer({ deleteSession });
+        const adapter = createTrueFoundryThreadListAdapter({
+            server,
+            agentName: "my-agent",
+        });
+
+        await adapter.delete("s1");
+
+        expect(deleteSession).toHaveBeenCalledWith({ sessionId: "s1" });
+    });
+
+    it("delete is a no-op when server.deleteSession is missing", async () => {
+        const adapter = createTrueFoundryThreadListAdapter({
+            server: mockServer({}),
+            agentName: "my-agent",
+        });
+
+        await expect(adapter.delete("s1")).resolves.toBeUndefined();
+    });
 });

package/src/truefoundryThreadListAdapter.ts

@@ -50,7 +50,10 @@ export function createTrueFoundryThreadListAdapter(options: {
         async rename() {},
         async archive() {},
         async unarchive() {},
-        async delete() {},
+        async delete(remoteId) {
+            if (typeof server.deleteSession !== "function") return;
+            await server.deleteSession({ sessionId: remoteId });
+        },
 
         async generateTitle() {
             return new ReadableStream();

package/src/useTrueFoundryAgentMessages.test.tsx

@@ -994,7 +994,7 @@ describe("useTrueFoundryAgentMessages", () => {
     });
 
     describe("batched resume invariant", () => {
-        it("issues exactly one prepareAndExecuteTurn input batch across root and sub-agent threads", async () => {
+        it("issues exactly one createTurn input batch across root and sub-agent threads", async () => {
             vi.mocked(loadSessionSnapshot).mockResolvedValue(
                 snapshotWithAssistantMessage(assistantMessageWithMultiThreadPendingActions()),
             );