Merge pull request #1230 from getlarge/issue-1215-backend-client-tsconfigs

getlarge · web-flow · commit edc6ca4cb401 · 2026-05-24T18:31:47.000+02:00
refactor(tsconfig): split Vite runtime package configs
diff --git a/apps/agent-daemon/tsconfig.json b/apps/agent-daemon/tsconfig.json
@@ -10,25 +10,16 @@
   "include": ["src/**/*.ts", "src/**/*.d.ts"],
   "references": [
     {
-      "path": "../../libs/agent-runtime"
-    },
-    {
-      "path": "../../libs/bootstrap"
-    },
-    {
-      "path": "../../libs/crypto-service"
+      "path": "../../libs/tasks"
     },
     {
       "path": "../../libs/database"
     },
     {
-      "path": "../../libs/pi-extension"
-    },
-    {
-      "path": "../../libs/sdk"
+      "path": "../../libs/bootstrap"
     },
     {
-      "path": "../../libs/tasks"
+      "path": "../../libs/crypto-service"
     }
   ]
 }
diff --git a/docs/tsconfig.json b/docs/tsconfig.json
@@ -10,9 +10,6 @@
   "extends": "../tsconfig.json",
   "include": [".vitepress/**/*", "scripts/**/*"],
   "references": [
-    {
-      "path": "../libs/sdk"
-    },
     {
       "path": "../libs/api-client"
     }
diff --git a/libs/agent-runtime/tsconfig.json b/libs/agent-runtime/tsconfig.json
@@ -1,19 +1,13 @@
 {
-  "compilerOptions": {
-    "composite": true,
-    "outDir": "./out-tsc",
-    "rootDir": "./src",
-    "tsBuildInfoFile": "./out-tsc/tsconfig.tsbuildinfo"
-  },
-  "exclude": ["node_modules", "dist", "__tests__"],
   "extends": "../../tsconfig.json",
-  "include": ["src/**/*.ts"],
+  "files": [],
+  "include": [],
   "references": [
     {
-      "path": "../sdk"
+      "path": "./tsconfig.lib.json"
     },
     {
-      "path": "../tasks"
+      "path": "./tsconfig.spec.json"
     }
   ]
 }
diff --git a/libs/agent-runtime/tsconfig.lib.json b/libs/agent-runtime/tsconfig.lib.json
@@ -0,0 +1,19 @@
+{
+  "compilerOptions": {
+    "composite": true,
+    "outDir": "./out-tsc",
+    "rootDir": "./src",
+    "tsBuildInfoFile": "./out-tsc/tsconfig.tsbuildinfo"
+  },
+  "exclude": ["node_modules", "dist", "src/**/*.test.ts", "src/**/*.spec.ts"],
+  "extends": "./tsconfig.json",
+  "include": ["src/**/*.ts"],
+  "references": [
+    {
+      "path": "../sdk/tsconfig.lib.json"
+    },
+    {
+      "path": "../tasks"
+    }
+  ]
+}
diff --git a/libs/agent-runtime/tsconfig.spec.json b/libs/agent-runtime/tsconfig.spec.json
@@ -0,0 +1,20 @@
+{
+  "compilerOptions": {
+    "outDir": "./out-tsc/spec",
+    "rootDir": ".",
+    "tsBuildInfoFile": "./out-tsc/tsconfig.spec.tsbuildinfo",
+    "types": ["node", "vitest"]
+  },
+  "extends": "./tsconfig.json",
+  "include": [
+    "vitest.config.ts",
+    "src/**/*.test.ts",
+    "src/**/*.spec.ts",
+    "src/**/*.d.ts"
+  ],
+  "references": [
+    {
+      "path": "./tsconfig.lib.json"
+    }
+  ]
+}
diff --git a/libs/agent-runtime/vite.config.ts b/libs/agent-runtime/vite.config.ts
@@ -6,7 +6,7 @@ export default defineConfig({
     dts({
       rollupTypes: true,
       bundledPackages: ['@moltnet/tasks'],
-      tsconfigPath: './tsconfig.json',
+      tsconfigPath: './tsconfig.lib.json',
       include: ['src/**/*.ts'],
       exclude: ['src/**/*.test.ts'],
       compilerOptions: {
diff --git a/libs/pi-extension/tsconfig.json b/libs/pi-extension/tsconfig.json
@@ -1,24 +1,13 @@
 {
-  "compilerOptions": {
-    "composite": true,
-    "lib": ["ES2022"],
-    "outDir": "./out-tsc",
-    "rootDir": "./src",
-    "tsBuildInfoFile": "./out-tsc/tsconfig.tsbuildinfo",
-    "types": ["node"]
-  },
-  "exclude": ["node_modules", "dist"],
   "extends": "../../tsconfig.json",
-  "include": ["src"],
+  "files": [],
+  "include": [],
   "references": [
     {
-      "path": "../agent-runtime"
+      "path": "./tsconfig.lib.json"
     },
     {
-      "path": "../crypto-service"
-    },
-    {
-      "path": "../sdk"
+      "path": "./tsconfig.spec.json"
     }
   ]
 }
diff --git a/libs/pi-extension/tsconfig.lib.json b/libs/pi-extension/tsconfig.lib.json
@@ -0,0 +1,24 @@
+{
+  "compilerOptions": {
+    "composite": true,
+    "lib": ["ES2022"],
+    "outDir": "./out-tsc",
+    "rootDir": "./src",
+    "tsBuildInfoFile": "./out-tsc/tsconfig.tsbuildinfo",
+    "types": ["node"]
+  },
+  "exclude": ["node_modules", "dist", "src/**/*.test.ts", "src/**/*.spec.ts"],
+  "extends": "./tsconfig.json",
+  "include": ["src"],
+  "references": [
+    {
+      "path": "../sdk/tsconfig.lib.json"
+    },
+    {
+      "path": "../agent-runtime/tsconfig.lib.json"
+    },
+    {
+      "path": "../crypto-service"
+    }
+  ]
+}
diff --git a/libs/pi-extension/tsconfig.spec.json b/libs/pi-extension/tsconfig.spec.json
@@ -0,0 +1,21 @@
+{
+  "compilerOptions": {
+    "lib": ["ES2022"],
+    "outDir": "./out-tsc/spec",
+    "rootDir": ".",
+    "tsBuildInfoFile": "./out-tsc/tsconfig.spec.tsbuildinfo",
+    "types": ["node", "vitest"]
+  },
+  "extends": "./tsconfig.json",
+  "include": [
+    "vitest.config.ts",
+    "src/**/*.test.ts",
+    "src/**/*.spec.ts",
+    "src/**/*.d.ts"
+  ],
+  "references": [
+    {
+      "path": "./tsconfig.lib.json"
+    }
+  ]
+}
diff --git a/libs/pi-extension/vite.config.ts b/libs/pi-extension/vite.config.ts
@@ -5,7 +5,7 @@ export default defineConfig({
   plugins: [
     dts({
       rollupTypes: true,
-      tsconfigPath: './tsconfig.json',
+      tsconfigPath: './tsconfig.lib.json',
       include: ['src/**/*.ts'],
       exclude: ['src/**/*.test.ts'],
       compilerOptions: {
diff --git a/libs/sdk/__tests__/agent.test.ts b/libs/sdk/__tests__/agent.test.ts
@@ -988,14 +988,14 @@ describe('Agent facade', () => {
 
       const agent = makeAgent();
       await agent.diaries.compile('diary-1', {
-        query: 'auth flow',
+        taskPrompt: 'auth flow',
         tokenBudget: 1000,
       });
 
       expect(compileDiary).toHaveBeenCalledWith(
         expect.objectContaining({
           path: { id: 'diary-1' },
-          body: { query: 'auth flow', tokenBudget: 1000 },
+          body: { taskPrompt: 'auth flow', tokenBudget: 1000 },
         }),
       );
     });
diff --git a/libs/sdk/__tests__/errors.test.ts b/libs/sdk/__tests__/errors.test.ts
@@ -81,7 +81,7 @@ describe('problemToError', () => {
         type: 'urn:moltnet:problem:voucher-invalid',
         title: 'Invalid voucher',
         status: 403,
-        code: 'VOUCHER_INVALID',
+        code: 'FORBIDDEN',
         detail: 'Voucher has already been redeemed',
       },
       403,
diff --git a/libs/sdk/__tests__/human.test.ts b/libs/sdk/__tests__/human.test.ts
@@ -22,6 +22,16 @@ vi.mock('@moltnet/api-client', async (importOriginal) => {
 
 const mockClient = {} as Client;
 
+type AuthConfig = {
+  name?: string;
+  scheme?: 'basic' | 'bearer';
+  type: 'apiKey' | 'http';
+};
+
+type AuthCallback = (
+  auth: AuthConfig,
+) => string | undefined | Promise<string | undefined>;
+
 describe('Human client facade', () => {
   beforeEach(() => {
     vi.clearAllMocks();
@@ -79,11 +89,12 @@ describe('Human client facade', () => {
         auth: expect.any(Function),
       }),
     );
+    const auth = call?.auth as AuthCallback;
     await expect(
-      call?.auth?.({ type: 'apiKey', name: 'X-Moltnet-Session-Token' }),
+      auth({ type: 'apiKey', name: 'X-Moltnet-Session-Token' }),
     ).resolves.toBe('human-session-token');
     await expect(
-      call?.auth?.({ type: 'http', scheme: 'bearer' }),
+      auth({ type: 'http', scheme: 'bearer' }),
     ).resolves.toBeUndefined();
   });
 
@@ -101,11 +112,12 @@ describe('Human client facade', () => {
     await human.teams.list();
 
     const call = vi.mocked(listTeams).mock.calls[0]?.[0];
+    const auth = call?.auth as AuthCallback;
+    await expect(auth({ type: 'http', scheme: 'bearer' })).resolves.toBe(
+      'human-access-token',
+    );
     await expect(
-      call?.auth?.({ type: 'http', scheme: 'bearer' }),
-    ).resolves.toBe('human-access-token');
-    await expect(
-      call?.auth?.({ type: 'apiKey', name: 'X-Moltnet-Session-Token' }),
+      auth({ type: 'apiKey', name: 'X-Moltnet-Session-Token' }),
     ).resolves.toBeUndefined();
   });
 
diff --git a/libs/sdk/__tests__/retry.test.ts b/libs/sdk/__tests__/retry.test.ts
@@ -15,7 +15,7 @@ afterEach(() => {
   vi.restoreAllMocks();
 });
 
-function flushRetryDelay(): Promise<void> {
+function flushRetryDelay(): Promise<unknown> {
   return vi.advanceTimersByTimeAsync(35_000);
 }
 
diff --git a/libs/sdk/src/agent.ts b/libs/sdk/src/agent.ts
@@ -86,6 +86,7 @@ import type {
   RenderedPackPreview,
   RenderedPackResult,
   RenderedPackWithContent,
+  RequestRecoveryChallengeData,
   RevokeDiaryGrantData,
   RevokeDiaryGrantResponse,
   RotateSecretResponse,
@@ -107,6 +108,7 @@ import type {
   UpdateRenderedPackData,
   UpdateTeamMemberRoleData,
   UpdateTeamMemberRoleResponse,
+  VerifyRecoveryChallengeData,
   VerifyResult,
   Voucher,
 } from '@moltnet/api-client';
@@ -345,17 +347,12 @@ export interface AuthNamespace {
 }
 
 export interface RecoveryNamespace {
-  requestChallenge(body: {
-    fingerprint: string;
-    challenge: string;
-    timestamp: string;
-  }): Promise<RecoveryChallengeResponse>;
-  verifyChallenge(body: {
-    fingerprint: string;
-    challenge: string;
-    signature: string;
-    timestamp: string;
-  }): Promise<RecoveryVerifyResponse>;
+  requestChallenge(
+    body: RequestRecoveryChallengeData['body'],
+  ): Promise<RecoveryChallengeResponse>;
+  verifyChallenge(
+    body: VerifyRecoveryChallengeData['body'],
+  ): Promise<RecoveryVerifyResponse>;
 }
 
 export interface PublicNamespace {
diff --git a/libs/sdk/src/namespaces/recovery.ts b/libs/sdk/src/namespaces/recovery.ts
@@ -14,15 +14,11 @@ export function createRecoveryNamespace(
 
   return {
     async requestChallenge(body) {
-      return unwrapResult(
-        await requestRecoveryChallenge({ client, body: body as never }),
-      );
+      return unwrapResult(await requestRecoveryChallenge({ client, body }));
     },
 
     async verifyChallenge(body) {
-      return unwrapResult(
-        await verifyRecoveryChallenge({ client, body: body as never }),
-      );
+      return unwrapResult(await verifyRecoveryChallenge({ client, body }));
     },
   };
 }
diff --git a/libs/sdk/tsconfig.json b/libs/sdk/tsconfig.json
@@ -1,20 +1,13 @@
 {
-  "compilerOptions": {
-    "composite": true,
-    "declarationDir": "out-tsc",
-    "outDir": "out-tsc",
-    "rootDir": "src",
-    "tsBuildInfoFile": "./out-tsc/tsconfig.tsbuildinfo"
-  },
-  "exclude": ["node_modules", "dist", "out-tsc"],
   "extends": "../../tsconfig.json",
-  "include": ["src"],
+  "files": [],
+  "include": [],
   "references": [
     {
-      "path": "../api-client"
+      "path": "./tsconfig.lib.json"
     },
     {
-      "path": "../crypto-service"
+      "path": "./tsconfig.spec.json"
     }
   ]
 }
diff --git a/libs/sdk/tsconfig.lib.json b/libs/sdk/tsconfig.lib.json
@@ -0,0 +1,20 @@
+{
+  "compilerOptions": {
+    "composite": true,
+    "declarationDir": "out-tsc",
+    "outDir": "out-tsc",
+    "rootDir": "src",
+    "tsBuildInfoFile": "./out-tsc/tsconfig.tsbuildinfo"
+  },
+  "exclude": ["node_modules", "dist", "out-tsc", "__tests__"],
+  "extends": "./tsconfig.json",
+  "include": ["src"],
+  "references": [
+    {
+      "path": "../api-client"
+    },
+    {
+      "path": "../crypto-service"
+    }
+  ]
+}
diff --git a/libs/sdk/tsconfig.spec.json b/libs/sdk/tsconfig.spec.json
diff --git a/libs/sdk/vite.config.ts b/libs/sdk/vite.config.ts
diff --git a/packages/agent-daemon-action/tsconfig.json b/packages/agent-daemon-action/tsconfig.json
diff --git a/packages/github-agent/tsconfig.json b/packages/github-agent/tsconfig.json
diff --git a/packages/legreffier-cli/tsconfig.json b/packages/legreffier-cli/tsconfig.json
diff --git a/packages/legreffier-cli/tsconfig.test.json b/packages/legreffier-cli/tsconfig.test.json
diff --git a/tools/tsconfig.json b/tools/tsconfig.json

Original file line number	Diff line number	Diff line change
`@@ -10,25 +10,16 @@`
`10`	`10`	`"include": ["src/*/.ts", "src/*/.d.ts"],`
`11`	`11`	`"references": [`
`12`	`12`	`{`
`13`		`- "path": "../../libs/agent-runtime"`
`14`		`- },`
`15`		`- {`
`16`		`- "path": "../../libs/bootstrap"`
`17`		`- },`
`18`		`- {`
`19`		`- "path": "../../libs/crypto-service"`
	`13`	`+ "path": "../../libs/tasks"`
`20`	`14`	`},`
`21`	`15`	`{`
`22`	`16`	`"path": "../../libs/database"`
`23`	`17`	`},`
`24`	`18`	`{`
`25`		`- "path": "../../libs/pi-extension"`
`26`		`- },`
`27`		`- {`
`28`		`- "path": "../../libs/sdk"`
	`19`	`+ "path": "../../libs/bootstrap"`
`29`	`20`	`},`
`30`	`21`	`{`
`31`		`- "path": "../../libs/tasks"`
	`22`	`+ "path": "../../libs/crypto-service"`
`32`	`23`	`}`
`33`	`24`	`]`
`34`	`25`	`}`
Original file line number	Diff line number	Diff line change
`@@ -10,9 +10,6 @@`
`10`	`10`	`"extends": "../tsconfig.json",`
`11`	`11`	`"include": [".vitepress/*/", "scripts/*/"],`
`12`	`12`	`"references": [`
`13`		`- {`
`14`		`- "path": "../libs/sdk"`
`15`		`- },`
`16`	`13`	`{`
`17`	`14`	`"path": "../libs/api-client"`
`18`	`15`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,19 +1,13 @@`
`1`	`1`	`{`
`2`		`- "compilerOptions": {`
`3`		`- "composite": true,`
`4`		`- "outDir": "./out-tsc",`
`5`		`- "rootDir": "./src",`
`6`		`- "tsBuildInfoFile": "./out-tsc/tsconfig.tsbuildinfo"`
`7`		`- },`
`8`		`- "exclude": ["node_modules", "dist", "__tests__"],`
`9`	`2`	`"extends": "../../tsconfig.json",`
`10`		`- "include": ["src/*/.ts"],`
	`3`	`+ "files": [],`
	`4`	`+ "include": [],`
`11`	`5`	`"references": [`
`12`	`6`	`{`
`13`		`- "path": "../sdk"`
	`7`	`+ "path": "./tsconfig.lib.json"`
`14`	`8`	`},`
`15`	`9`	`{`
`16`		`- "path": "../tasks"`
	`10`	`+ "path": "./tsconfig.spec.json"`
`17`	`11`	`}`
`18`	`12`	`]`
`19`	`13`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,24 +1,13 @@`
`1`	`1`	`{`
`2`		`- "compilerOptions": {`
`3`		`- "composite": true,`
`4`		`- "lib": ["ES2022"],`
`5`		`- "outDir": "./out-tsc",`
`6`		`- "rootDir": "./src",`
`7`		`- "tsBuildInfoFile": "./out-tsc/tsconfig.tsbuildinfo",`
`8`		`- "types": ["node"]`
`9`		`- },`
`10`		`- "exclude": ["node_modules", "dist"],`
`11`	`2`	`"extends": "../../tsconfig.json",`
`12`		`- "include": ["src"],`
	`3`	`+ "files": [],`
	`4`	`+ "include": [],`
`13`	`5`	`"references": [`
`14`	`6`	`{`
`15`		`- "path": "../agent-runtime"`
	`7`	`+ "path": "./tsconfig.lib.json"`
`16`	`8`	`},`
`17`	`9`	`{`
`18`		`- "path": "../crypto-service"`
`19`		`- },`
`20`		`- {`
`21`		`- "path": "../sdk"`
	`10`	`+ "path": "./tsconfig.spec.json"`
`22`	`11`	`}`
`23`	`12`	`]`
`24`	`13`	`}`