chore: Improve Nx task separation

.codesandbox/ci.json

@@ -1,5 +1,6 @@
 {
   "installCommand": "install:csb",
+  "buildCommand": "build:all",
   "sandboxes": [
     "/examples/react/basic-typescript",
     "/examples/solid/basic-typescript",

.github/workflows/pr.yml

@@ -4,7 +4,6 @@ on: [pull_request]
 
 env:
   NX_CLOUD_ACCESS_TOKEN: ${{ secrets.NX_CLOUD_ACCESS_TOKEN }}
-  BUNDLEWATCH_GITHUB_TOKEN: ${{ secrets.BUNDLEWATCH_GITHUB_TOKEN }}
 
 jobs:
   main:

package.json

@@ -8,25 +8,24 @@
     "preinstall": "node -e \"if(process.env.CI == 'true') {console.log('Skipping preinstall...'); process.exit(1)}\" || npx -y only-allow pnpm",
     "install:csb": "corepack enable && pnpm install --frozen-lockfile",
     "test": "pnpm run test:ci",
-    "test:ci": "nx affected --targets=test:format,test:lib,test:types,test:eslint",
+    "test:ci": "nx run-many --exclude=examples/** --targets=test:format,test:eslint,test:lib,test:types,build,test:build",
     "test:eslint": "nx affected --target=test:eslint",
     "test:format": "pnpm run prettier --check",
     "test:lib": "nx affected --target=test:lib",
     "test:lib:dev": "pnpm --filter \"./packages/**\" run test:lib:dev",
-    "test:build": "nx run-many --target=test:build --projects=root",
+    "test:build": "nx affected --target=test:build",
     "test:types": "nx affected --target=test:types",
-    "build": "nx run-many --target=build --projects=root",
+    "build": "nx affected --target=build",
+    "build:all": "nx run-many --exclude=examples/** --target=build",
     "watch": "concurrently --kill-others \"pnpm -r build:rollup -w\" \"pnpm -r build:types --watch\"",
     "dev": "pnpm run watch",
     "prettier": "prettier --plugin-search-dir . \"{packages,examples,scripts}/**/*.{md,js,jsx,cjs,ts,tsx,json,vue,svelte}\"",
     "prettier:write": "pnpm run prettier --write",
-    "cipublish": "node scripts/publish.js",
-    "validatePackages": "node scripts/validatePackages.js"
+    "cipublish": "node scripts/publish.js"
   },
   "nx": {
     "includedScripts": [
-      "test:format",
-      "test:build"
+      "test:format"
     ]
   },
   "namespace": "@tanstack",
@@ -57,7 +56,6 @@
     "@vitest/coverage-istanbul": "^0.27.1",
     "axios": "^1.4.0",
     "babel-preset-solid": "^1.6.10",
-    "bundlewatch": "^0.3.3",
     "chalk": "^5.2.0",
     "concurrently": "^8.0.1",
     "cpy-cli": "^4.2.0",
@@ -92,13 +90,6 @@
     "vitest": "^0.27.1",
     "vue": "^3.2.47"
   },
-  "bundlewatch": {
-    "files": [
-      {
-        "path": "packages/*/build/lib/*.js"
-      }
-    ]
-  },
   "pnpm": {
     "patchedDependencies": {
       "@types/[email protected]": "patches/@[email protected]"

packages/eslint-plugin-query/package.json

@@ -30,6 +30,7 @@
     "test:types": "tsc --noEmit",
     "test:lib": "vitest run --coverage",
     "test:lib:dev": "pnpm run test:lib --watch",
+    "test:build": "publint",
     "build": "tsup --minify --dts"
   },
   "files": [

packages/query-async-storage-persister/package.json

@@ -34,6 +34,7 @@
     "test:types": "tsc --noEmit",
     "test:lib": "vitest run --coverage",
     "test:lib:dev": "pnpm run test:lib --watch",
+    "test:build": "publint",
     "build": "pnpm build:rollup && pnpm build:types",
     "build:rollup": "rollup --config rollup.config.js",
     "build:types": "tsc --emitDeclarationOnly"

packages/query-broadcast-client-experimental/package.json

@@ -32,6 +32,7 @@
     "clean": "rimraf ./build && rimraf ./coverage",
     "test:eslint": "eslint --ext .ts,.tsx ./src",
     "test:types": "tsc --noEmit",
+    "test:build": "publint",
     "build": "pnpm build:rollup && pnpm build:types",
     "build:rollup": "rollup --config rollup.config.js",
     "build:types": "tsc --emitDeclarationOnly"

packages/query-core/package.json

@@ -34,6 +34,7 @@
     "test:types": "tsc --noEmit",
     "test:lib": "vitest run --coverage",
     "test:lib:dev": "pnpm run test:lib --watch",
+    "test:build": "publint",
     "build": "pnpm build:rollup && pnpm build:types",
     "build:rollup": "rollup --config rollup.config.js",
     "build:types": "tsc --emitDeclarationOnly"

packages/query-devtools/package.json

@@ -30,6 +30,7 @@
     "test:types": "tsc --noEmit",
     "test:lib": "vitest run --coverage",
     "test:lib:dev": "pnpm run test:lib --watch",
+    "test:build": "publint",
     "build": "pnpm build:rollup && pnpm build:types",
     "build:rollup": "rollup --config rollup.config.js",
     "build:types": "tsc --emitDeclarationOnly"

packages/query-persist-client-core/package.json

@@ -34,6 +34,7 @@
     "test:types": "tsc --noEmit",
     "test:lib": "vitest run --coverage",
     "test:lib:dev": "pnpm run test:lib --watch",
+    "test:build": "publint",
     "build": "pnpm build:rollup && pnpm build:types",
     "build:rollup": "rollup --config rollup.config.js",
     "build:types": "tsc --emitDeclarationOnly"

packages/query-sync-storage-persister/package.json

@@ -34,6 +34,7 @@
     "test:types": "tsc --noEmit",
     "test:lib": "vitest run --coverage",
     "test:lib:dev": "pnpm run test:lib --watch",
+    "test:build": "publint",
     "build": "pnpm build:rollup && pnpm build:types",
     "build:rollup": "rollup --config rollup.config.js",
     "build:types": "tsc --emitDeclarationOnly"

packages/react-query-devtools/package.json

@@ -40,6 +40,7 @@
     "test:types": "tsc --noEmit",
     "test:lib": "vitest run --coverage",
     "test:lib:dev": "pnpm run test:lib --watch",
+    "test:build": "publint",
     "build": "pnpm build:rollup && pnpm build:types",
     "build:rollup": "rollup --config rollup.config.js",
     "build:types": "tsc --emitDeclarationOnly && cpy index.d.ts index.prod.d.ts --cwd=build/lib"

packages/react-query-persist-client/package.json

@@ -34,6 +34,7 @@
     "test:types": "tsc --noEmit",
     "test:lib": "vitest run --coverage",
     "test:lib:dev": "pnpm run test:lib --watch",
+    "test:build": "publint",
     "build": "pnpm build:rollup && pnpm build:types",
     "build:rollup": "rollup --config rollup.config.js",
     "build:types": "tsc --emitDeclarationOnly"

packages/react-query/package.json

@@ -30,6 +30,7 @@
     "test:types": "tsc --noEmit",
     "test:lib": "vitest run --coverage",
     "test:lib:dev": "pnpm run test:lib --watch",
+    "test:build": "publint",
     "build": "pnpm build:rollup && pnpm build:codemods && pnpm build:types",
     "build:rollup": "rollup --config rollup.config.js",
     "build:codemods": "cpy ../codemods/src/**/* ./build/codemods",

packages/solid-query/package.json

@@ -33,6 +33,7 @@
     "test:types": "tsc --noEmit",
     "test:lib": "vitest run --coverage",
     "test:lib:dev": "pnpm run test:lib --watch",
+    "test:build": "publint",
     "build": "pnpm build:rollup && pnpm build:types",
     "build:rollup": "rollup --config rollup.config.js",
     "build:types": "tsc --emitDeclarationOnly"

packages/svelte-query-devtools/package.json

@@ -31,6 +31,7 @@
     "clean": "rimraf ./build && rimraf ./coverage",
     "test:types": "svelte-check --tsconfig ./tsconfig.json",
     "test:eslint": "eslint --ext .svelte,.ts ./src",
+    "test:build": "publint",
     "build": "svelte-package --input ./src --output ./build/lib"
   },
   "dependencies": {

packages/svelte-query/package.json

@@ -35,6 +35,7 @@
     "test:eslint": "eslint --ext .svelte,.ts ./src",
     "test:lib": "vitest run --coverage",
     "test:lib:dev": "pnpm run test:lib --watch",
+    "test:build": "publint",
     "build": "svelte-package --input ./src --output ./build/lib"
   },
   "dependencies": {

packages/vue-query/package.json

@@ -37,6 +37,7 @@
     "test:2.7": "vue-demi-switch 2.7 vue2.7 && vitest",
     "test:3": "vue-demi-switch 3 && vitest",
     "test:lib:dev": "pnpm run test:lib --watch",
+    "test:build": "publint",
     "build": "pnpm build:rollup && pnpm build:types",
     "build:rollup": "rollup --config rollup.config.js",
     "build:types": "tsc --emitDeclarationOnly"

scripts/getRollupConfig.js

@@ -95,15 +95,10 @@ export function buildConfigs(opts) {
           }),
       preserveDirectives(),
       visualizer({
-        filename: `./build/stats-html.html`,
+        filename: `./build/stats.html`,
         template: 'treemap',
         gzipSize: true,
       }),
-      visualizer({
-        filename: `./build/stats.json`,
-        template: 'raw-data',
-        gzipSize: true,
-      }),
     ],
   }
 }

scripts/publish.js

@@ -348,16 +348,6 @@ async function run() {
     return
   }
 
-  console.info('Building packages...')
-  execSync(`pnpm run build --skip-nx-cache`, {
-    encoding: 'utf8',
-    stdio: 'inherit',
-  })
-  console.info('')
-
-  console.info('Validating packages...')
-  execSync(`pnpm run validatePackages`, { encoding: 'utf8', stdio: 'inherit' })
-
   console.info(`Updating all changed packages to version ${version}...`)
   // Update each package to the new version
   for (const pkg of changedPackages) {