diff --git a/bun.lock b/bun.lock index a70711b8c11..e9f6919ca47 100644 --- a/bun.lock +++ b/bun.lock @@ -111,7 +111,7 @@ }, "apps/desktop": { "name": "@superset/desktop", - "version": "1.10.1", + "version": "1.10.2", "dependencies": { "@ai-sdk/anthropic": "3.0.64", "@ai-sdk/openai": "3.0.36", @@ -770,7 +770,7 @@ }, "packages/host-service": { "name": "@superset/host-service", - "version": "0.8.9", + "version": "0.8.10", "dependencies": { "@hono/node-server": "1.19.13", "@hono/node-ws": "1.3.0", @@ -926,7 +926,7 @@ }, "packages/sdk": { "name": "@superset/sdk", - "version": "0.0.1-alpha.10", + "version": "0.0.1-alpha.11", "devDependencies": { "@superset/typescript": "workspace:*", "bun-types": "1.3.11", diff --git a/packages/host-service/src/trpc/router/workspaces/workspaces.ts b/packages/host-service/src/trpc/router/workspaces/workspaces.ts index cec809e6b6d..fdd55e983ca 100644 --- a/packages/host-service/src/trpc/router/workspaces/workspaces.ts +++ b/packages/host-service/src/trpc/router/workspaces/workspaces.ts @@ -574,7 +574,6 @@ export const workspacesRouter = router({ let worktreePath: string; let alreadyExists = false; let workspaceRow: CloudWorkspace; - const warnings: string[] = []; if (input.pr !== undefined) { const releaseCreateLock = await acquireWorkspaceCreateLock( @@ -614,7 +613,6 @@ export const workspacesRouter = router({ ) => { if (materialized.warning) { console.warn(`[workspaces.create] ${materialized.warning}`); - warnings.push(materialized.warning); } }; const normalizeExistingPrBranch = async () => { @@ -1015,7 +1013,6 @@ export const workspacesRouter = router({ }); if (warning) { console.warn(`[workspaces.create] setup warning: ${warning}`); - warnings.push(warning); } if (terminal) { terminalsResult.push({ @@ -1036,7 +1033,6 @@ export const workspacesRouter = router({ terminals: terminalsResult, agents: agentsResult, alreadyExists, - warnings, txid: extractCreateTxid(workspaceRow), }; }), diff --git a/packages/host-service/test/integration/workspace-create-pr.integration.test.ts b/packages/host-service/test/integration/workspace-create-pr.integration.test.ts index 700243630c3..a4133803392 100644 --- a/packages/host-service/test/integration/workspace-create-pr.integration.test.ts +++ b/packages/host-service/test/integration/workspace-create-pr.integration.test.ts @@ -193,7 +193,6 @@ describe("workspaces.create PR checkout integration", () => { ).toBe("HEAD:refs/heads/feature/pr-lockfile"); const dryRunOutput = await worktreeGit.raw(["push", "--dry-run"]); expect(typeof dryRunOutput).toBe("string"); - expect(result.warnings).toEqual([]); const lockStatus = ( await worktreeGit.raw([ @@ -531,7 +530,6 @@ describe("workspaces.create PR checkout integration", () => { expect(worktreePath).toBeTruthy(); if (!worktreePath) throw new Error("expected worktree path"); expect(result.workspace.branch).toBe("feature/same-repo"); - expect(result.warnings).toEqual([]); expect( ( await scenario.repo.git.raw([ @@ -553,7 +551,7 @@ describe("workspaces.create PR checkout integration", () => { ).toBe(prHeadOid); }); - test("same-repo PR falls back to synthetic ref with a user-visible warning when the head branch is gone", async () => { + test("same-repo PR falls back to synthetic ref when the head branch is gone", async () => { const prNumber = 8081; let prHeadOid = ""; @@ -616,8 +614,7 @@ describe("workspaces.create PR checkout integration", () => { )?.worktreePath; expect(worktreePath).toBeTruthy(); if (!worktreePath) throw new Error("expected worktree path"); - expect(result.warnings).toHaveLength(1); - expect(result.warnings[0]).toContain("was unavailable from origin"); + expect(result.workspace.branch).toBe("feature/deleted-head"); expect( ( await scenario.repo.git.raw([ diff --git a/packages/mcp-v2/src/tools/workspaces/create.ts b/packages/mcp-v2/src/tools/workspaces/create.ts index 1a8f5df95c0..ec5d1036c29 100644 --- a/packages/mcp-v2/src/tools/workspaces/create.ts +++ b/packages/mcp-v2/src/tools/workspaces/create.ts @@ -79,7 +79,6 @@ export function register(server: McpServer): void { | { ok: false; error: string } >; alreadyExists: boolean; - warnings: string[]; }>( { relayUrl: ctx.relayUrl, diff --git a/packages/sdk/package.json b/packages/sdk/package.json index 39d0fb5b1cc..55f273abec7 100644 --- a/packages/sdk/package.json +++ b/packages/sdk/package.json @@ -1,6 +1,6 @@ { "name": "@superset/sdk", - "version": "0.0.1-alpha.10", + "version": "0.0.1-alpha.11", "description": "TypeScript SDK for the Superset API", "private": true, "type": "module", diff --git a/packages/sdk/src/resources/workspaces.ts b/packages/sdk/src/resources/workspaces.ts index bf662501a98..ea7ad389a5b 100644 --- a/packages/sdk/src/resources/workspaces.ts +++ b/packages/sdk/src/resources/workspaces.ts @@ -203,7 +203,6 @@ export interface WorkspaceCreateResult { terminals: Array<{ terminalId: string; label?: string }>; agents: WorkspaceCreateAgentResult[]; alreadyExists: boolean; - warnings: string[]; } export interface WorkspaceUpdateParams {