Skip to content

Commit 1ec2d87

Browse files
committed
fix(typescript-sdk): random build failures
Due to the way pnpm runs scripts in parallel, some packages were not ready when others tried to access them, leading to random build failures. Fixes applied to eliminate race conditions: - Disable clean in tsup config files - Use the same clean method across all packages Also: - Add clean script to the root package.json for consistency - Disable minify when watch is enabled for performance Fixes #59
1 parent d53b012 commit 1ec2d87

File tree

17 files changed

+26
-25
lines changed

17 files changed

+26
-25
lines changed

typescript-sdk/integrations/agno/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
"scripts": {
1313
"build": "tsup",
1414
"dev": "tsup --watch",
15-
"clean": "rm -rf dist",
15+
"clean": "rm -rf dist .turbo node_modules",
1616
"typecheck": "tsc --noEmit",
1717
"test": "jest",
1818
"link:global": "pnpm link --global",

typescript-sdk/integrations/langgraph/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
"scripts": {
1212
"build": "tsup",
1313
"dev": "tsup --watch",
14-
"clean": "rm -rf dist",
14+
"clean": "rm -rf dist .turbo node_modules",
1515
"typecheck": "tsc --noEmit",
1616
"test": "jest",
1717
"link:global": "pnpm link --global",

typescript-sdk/integrations/mastra/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
"scripts": {
1313
"build": "tsup",
1414
"dev": "tsup --watch",
15-
"clean": "rm -rf dist",
15+
"clean": "rm -rf dist .turbo node_modules",
1616
"typecheck": "tsc --noEmit",
1717
"test": "jest",
1818
"link:global": "pnpm link --global",

typescript-sdk/integrations/middleware-starter/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
"scripts": {
1313
"build": "tsup",
1414
"dev": "tsup --watch",
15-
"clean": "rm -rf dist",
15+
"clean": "rm -rf dist .turbo node_modules",
1616
"typecheck": "tsc --noEmit",
1717
"test": "jest",
1818
"link:global": "pnpm link --global",

typescript-sdk/integrations/server-starter-all-features/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
"scripts": {
1313
"build": "tsup",
1414
"dev": "tsup --watch",
15-
"clean": "rm -rf dist",
15+
"clean": "rm -rf dist .turbo node_modules",
1616
"typecheck": "tsc --noEmit",
1717
"test": "jest",
1818
"link:global": "pnpm link --global",

typescript-sdk/integrations/server-starter/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
"scripts": {
1313
"build": "tsup",
1414
"dev": "tsup --watch",
15-
"clean": "rm -rf dist",
15+
"clean": "rm -rf dist .turbo node_modules",
1616
"typecheck": "tsc --noEmit",
1717
"test": "jest",
1818
"link:global": "pnpm link --global",

typescript-sdk/integrations/vercel-ai-sdk/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
"scripts": {
1212
"build": "tsup",
1313
"dev": "tsup --watch",
14-
"clean": "rm -rf dist",
14+
"clean": "rm -rf dist .turbo node_modules",
1515
"typecheck": "tsc --noEmit",
1616
"test": "jest",
1717
"link:global": "pnpm link --global",

typescript-sdk/package.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
"private": true,
55
"scripts": {
66
"build": "turbo run build",
7+
"clean": "pnpm -r clean",
78
"build:clean": "pnpm -r clean && pnpm install && turbo run build",
89
"dev": "turbo run dev",
910
"lint": "turbo run lint",

typescript-sdk/packages/cli/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
"scripts": {
1818
"build": "tsup",
1919
"dev": "tsup --watch",
20-
"clean": "rm -rf dist",
20+
"clean": "rm -rf dist .turbo node_modules",
2121
"typecheck": "tsc --noEmit",
2222
"test": "jest",
2323
"link:global": "pnpm link --global",

typescript-sdk/packages/client/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
"scripts": {
1717
"build": "tsup",
1818
"dev": "tsup --watch",
19-
"clean": "rm -rf dist",
19+
"clean": "rm -rf dist .turbo node_modules",
2020
"typecheck": "tsc --noEmit",
2121
"test": "jest",
2222
"link:global": "pnpm link --global",
Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11
import { defineConfig } from "tsup";
22

3-
export default defineConfig({
3+
export default defineConfig((options) => ({
44
entry: ["src/index.ts"],
55
format: ["cjs", "esm"],
66
dts: true,
77
splitting: false,
88
sourcemap: true,
9-
clean: true,
10-
minify: true,
11-
});
9+
clean: !options.watch, // Don't clean in watch mode to prevent race conditions
10+
minify: !options.watch, // Don't minify in watch mode for faster builds
11+
}));

typescript-sdk/packages/core/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
"build": "tsup",
1414
"dev": "tsup --watch",
1515
"lint": "eslint \"src/**/*.ts*\"",
16-
"clean": "rm -rf .turbo && rm -rf node_modules && rm -rf dist",
16+
"clean": "rm -rf dist .turbo node_modules",
1717
"test": "jest",
1818
"link:global": "pnpm link --global",
1919
"unlink:global": "pnpm unlink --global"
Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
import { defineConfig } from "tsup";
22

3-
export default defineConfig({
3+
export default defineConfig((options) => ({
44
entry: ["src/index.ts"],
55
format: ["cjs", "esm"],
66
dts: true,
77
splitting: false,
88
sourcemap: true,
9-
clean: true,
10-
});
9+
clean: !options.watch, // Don't clean in watch mode to prevent race conditions
10+
}));

typescript-sdk/packages/encoder/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
"build": "tsup",
1414
"dev": "tsup --watch",
1515
"lint": "eslint \"src/**/*.ts*\"",
16-
"clean": "rm -rf .turbo && rm -rf node_modules && rm -rf dist",
16+
"clean": "rm -rf dist .turbo node_modules",
1717
"test": "jest",
1818
"link:global": "pnpm link --global",
1919
"unlink:global": "pnpm unlink --global"
Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
import { defineConfig } from "tsup";
22

3-
export default defineConfig({
3+
export default defineConfig((options) => ({
44
entry: ["src/index.ts"],
55
format: ["cjs", "esm"],
66
dts: true,
77
splitting: false,
88
sourcemap: true,
9-
clean: true,
10-
});
9+
clean: !options.watch, // Don't clean in watch mode to prevent race conditions
10+
}));

typescript-sdk/packages/proto/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
"build": "tsup",
1414
"dev": "tsup --watch",
1515
"lint": "eslint \"src/**/*.ts*\"",
16-
"clean": "rm -rf .turbo && rm -rf node_modules && rm -rf dist",
16+
"clean": "rm -rf dist .turbo node_modules",
1717
"test": "jest",
1818
"generate": "mkdir -p ./src/generated && protoc --plugin=./node_modules/.bin/protoc-gen-ts_proto --ts_proto_out=./src/generated --ts_proto_opt=esModuleInterop=true,outputJsonMethods=false,outputClientImpl=false -I ./src/proto ./src/proto/*.proto",
1919
"link:global": "pnpm link --global",
Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
import { defineConfig } from "tsup";
22

3-
export default defineConfig({
3+
export default defineConfig((options) => ({
44
entry: ["src/index.ts"],
55
format: ["esm", "cjs"],
66
dts: true,
77
splitting: false,
88
sourcemap: true,
9-
clean: true,
10-
});
9+
clean: !options.watch, // Don't clean in watch mode to prevent race conditions
10+
}));

0 commit comments

Comments
 (0)