Skip to content

chore(ci): improve Nx sandbox cache declarations#7833

Draft
beaussan wants to merge 3 commits into
mainfrom
codex/ci-timing-baseline
Draft

chore(ci): improve Nx sandbox cache declarations#7833
beaussan wants to merge 3 commits into
mainfrom
codex/ci-timing-baseline

Conversation

@beaussan

@beaussan beaussan commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Summary

  • refresh the CI timing baseline and increase large-changeset distribution from 14 to 16 runners
  • declare workspace and dependency build outputs that cached build, lint, unit, type, and E2E tasks legitimately read
  • declare E2E artifacts as outputs and scope the benign LICENSE packaging read to test:build
  • centralize dependency outputs in the dependentTaskOutputs named input

Impact

  • CI performance: Produces a new timing baseline with two additional large-changeset runners and more accurate cache keys.
  • Cache correctness: Downstream tasks now hash the outputs of build tasks they depend on, preventing stale cache hits when generated artifacts change.
  • Sandboxing: Local validation moved 174 of 689 reports to clean. Another 13 tasks have only the scoped LICENSE read remaining locally; that exclusion requires CI verification, for an expected minimum of 187 clean reports.
  • Risk: Some tasks may see additional cache misses because their hashes now include legitimate dependency outputs. The only exclusion is limited to test:build reading LICENSE while creating temporary validation tarballs.
  • User-facing changes: None.

Testing

  • downloaded the latest PR reports with pnpm exec nx-cloud get sandbox-reports --branch 7833 --since 1d
  • validated focused reports for build, test:build, test:e2e, test:eslint, test:unit, test:types, and test:types:ssr
  • validated the full 689-report index after the configuration changes
  • confirmed dependentTaskOutputs expands through pnpm nx show target @tanstack/router-core:test:types --json
  • pnpm exec prettier --check nx.json .nx/workflows/sandboxing-config.yaml
  • git diff --check
  • the full local pnpm test:eslint command was stopped after it remained silent past the repository's 20-second Nx execution guard, including after one pnpm nx reset retry; CI is the authoritative full-suite and exclusion check

@coderabbitai

coderabbitai Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 3f2597e6-30c8-40d1-b23f-95229e860b64

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/ci-timing-baseline

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@nx-cloud

nx-cloud Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

View your CI Pipeline Execution ↗ for commit d996c44

Command Status Duration Result
nx affected --targets=test:eslint,test:unit,tes... ✅ Succeeded 13m 14s View ↗
nx run-many --target=build --exclude=examples/*... ✅ Succeeded 2m 7s View ↗

☁️ Nx Cloud last updated this comment at 2026-07-16 09:55:14 UTC

@github-actions

Copy link
Copy Markdown
Contributor

🚀 Changeset Version Preview

No changeset entries found. Merging this PR will not cause a version bump for any packages.

@pkg-pr-new

pkg-pr-new Bot commented Jul 16, 2026

Copy link
Copy Markdown
More templates

@tanstack/arktype-adapter

npm i https://pkg.pr.new/@tanstack/arktype-adapter@7833

@tanstack/eslint-plugin-router

npm i https://pkg.pr.new/@tanstack/eslint-plugin-router@7833

@tanstack/eslint-plugin-start

npm i https://pkg.pr.new/@tanstack/eslint-plugin-start@7833

@tanstack/history

npm i https://pkg.pr.new/@tanstack/history@7833

@tanstack/nitro-v2-vite-plugin

npm i https://pkg.pr.new/@tanstack/nitro-v2-vite-plugin@7833

@tanstack/react-router

npm i https://pkg.pr.new/@tanstack/react-router@7833

@tanstack/react-router-devtools

npm i https://pkg.pr.new/@tanstack/react-router-devtools@7833

@tanstack/react-router-ssr-query

npm i https://pkg.pr.new/@tanstack/react-router-ssr-query@7833

@tanstack/react-start

npm i https://pkg.pr.new/@tanstack/react-start@7833

@tanstack/react-start-client

npm i https://pkg.pr.new/@tanstack/react-start-client@7833

@tanstack/react-start-rsc

npm i https://pkg.pr.new/@tanstack/react-start-rsc@7833

@tanstack/react-start-server

npm i https://pkg.pr.new/@tanstack/react-start-server@7833

@tanstack/router-cli

npm i https://pkg.pr.new/@tanstack/router-cli@7833

@tanstack/router-core

npm i https://pkg.pr.new/@tanstack/router-core@7833

@tanstack/router-devtools

npm i https://pkg.pr.new/@tanstack/router-devtools@7833

@tanstack/router-devtools-core

npm i https://pkg.pr.new/@tanstack/router-devtools-core@7833

@tanstack/router-generator

npm i https://pkg.pr.new/@tanstack/router-generator@7833

@tanstack/router-plugin

npm i https://pkg.pr.new/@tanstack/router-plugin@7833

@tanstack/router-ssr-query-core

npm i https://pkg.pr.new/@tanstack/router-ssr-query-core@7833

@tanstack/router-utils

npm i https://pkg.pr.new/@tanstack/router-utils@7833

@tanstack/router-vite-plugin

npm i https://pkg.pr.new/@tanstack/router-vite-plugin@7833

@tanstack/solid-router

npm i https://pkg.pr.new/@tanstack/solid-router@7833

@tanstack/solid-router-devtools

npm i https://pkg.pr.new/@tanstack/solid-router-devtools@7833

@tanstack/solid-router-ssr-query

npm i https://pkg.pr.new/@tanstack/solid-router-ssr-query@7833

@tanstack/solid-start

npm i https://pkg.pr.new/@tanstack/solid-start@7833

@tanstack/solid-start-client

npm i https://pkg.pr.new/@tanstack/solid-start-client@7833

@tanstack/solid-start-server

npm i https://pkg.pr.new/@tanstack/solid-start-server@7833

@tanstack/start-client-core

npm i https://pkg.pr.new/@tanstack/start-client-core@7833

@tanstack/start-fn-stubs

npm i https://pkg.pr.new/@tanstack/start-fn-stubs@7833

@tanstack/start-plugin-core

npm i https://pkg.pr.new/@tanstack/start-plugin-core@7833

@tanstack/start-server-core

npm i https://pkg.pr.new/@tanstack/start-server-core@7833

@tanstack/start-static-server-functions

npm i https://pkg.pr.new/@tanstack/start-static-server-functions@7833

@tanstack/start-storage-context

npm i https://pkg.pr.new/@tanstack/start-storage-context@7833

@tanstack/valibot-adapter

npm i https://pkg.pr.new/@tanstack/valibot-adapter@7833

@tanstack/virtual-file-routes

npm i https://pkg.pr.new/@tanstack/virtual-file-routes@7833

@tanstack/vue-router

npm i https://pkg.pr.new/@tanstack/vue-router@7833

@tanstack/vue-router-devtools

npm i https://pkg.pr.new/@tanstack/vue-router-devtools@7833

@tanstack/vue-router-ssr-query

npm i https://pkg.pr.new/@tanstack/vue-router-ssr-query@7833

@tanstack/vue-start

npm i https://pkg.pr.new/@tanstack/vue-start@7833

@tanstack/vue-start-client

npm i https://pkg.pr.new/@tanstack/vue-start-client@7833

@tanstack/vue-start-server

npm i https://pkg.pr.new/@tanstack/vue-start-server@7833

@tanstack/zod-adapter

npm i https://pkg.pr.new/@tanstack/zod-adapter@7833

commit: 013d186

@beaussan beaussan changed the title chore(ci): establish CI timing baseline chore(ci): improve Nx sandbox cache declarations Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant