Skip to content

fix(cache): add Package-scoped negative globs to prevent cache miss in monorepos - #1198

Merged
graphite-app[bot] merged 1 commit into
mainfrom
fix-cache-missing
Mar 30, 2026
Merged

fix(cache): add Package-scoped negative globs to prevent cache miss in monorepos#1198
graphite-app[bot] merged 1 commit into
mainfrom
fix-cache-missing

Conversation

@fengmk2

Copy link
Copy Markdown
Member

In monorepos, each package has its own node_modules/. When vp build/test/pack
run, Vite creates .vite-temp/ config files and dist/ outputs that are both read
and written during execution. The existing !node_modules/.vite-temp/** glob at
Workspace scope only matched the workspace root, not package-level paths,
causing perpetual cache misses due to read-write overlap detection.

  • Build: add Package-scoped !node_modules/.vite-temp/** and !dist/**
  • Test: add Package-scoped !node_modules/.vite-temp/** and !node_modules/.vite/**/results.json
  • Pack: add Package-scoped !node_modules/.vite-temp/** and !dist/**

@fengmk2Graphite App

fengmk2 commented Mar 29, 2026

Copy link
Copy Markdown
MemberAuthor

How to use the Graphite Merge Queue

Add the label auto-merge to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

Comment threadpackages/cli/binding/src/cli.rs Outdated
Comment threadpackages/cli/binding/src/cli.rs Outdated
Comment threadpackages/cli/binding/src/cli.rs
@fengmk2
fengmk2 marked this pull request as ready for review March 30, 2026 02:43
@fengmk2
fengmk2 requested a review from wan9chiMarch 30, 2026 02:43
@fengmk2
fengmk2force-pushed the fix-issue-1176 branch 2 times, most recently from 538ff45 to ffbfc0cCompareMarch 30, 2026 07:24
@fengmk2
fengmk2force-pushed the fix-cache-missing branch 2 times, most recently from ea53d50 to 854d4c2CompareMarch 30, 2026 09:50
@graphite-app
graphite-appBot changed the base branch from fix-issue-1176 to graphite-base/1198March 30, 2026 13:15
@fengmk2
fengmk2force-pushed the graphite-base/1198 branch from e60358a to cf7593eCompareMarch 30, 2026 13:16
@fengmk2
fengmk2 changed the base branch from graphite-base/1198 to fix-issue-1176March 30, 2026 13:17
@graphite-app
graphite-appBot changed the base branch from fix-issue-1176 to mainMarch 30, 2026 13:28
@netlify

netlifyBot commented Mar 30, 2026

Copy link
Copy Markdown

Deploy Preview for viteplus-preview canceled.

NameLink
🔨 Latest commit3fc8005
🔍 Latest deploy loghttps://app.netlify.com/projects/viteplus-preview/deploys/69ca90ca83babc00080de9b6

@graphite-app

graphite-appBot commented Mar 30, 2026

Copy link
Copy Markdown

Merge activity

…n monorepos (#1198)
In monorepos, each package has its own node_modules/. When vp build/test/pack
run, Vite creates .vite-temp/ config files and dist/ outputs that are both read
and written during execution. The existing !node_modules/.vite-temp/** glob at
Workspace scope only matched the workspace root, not package-level paths,
causing perpetual cache misses due to read-write overlap detection.
- Build: add Package-scoped !node_modules/.vite-temp/** and !dist/**
- Test: add Package-scoped !node_modules/.vite-temp/** and !node_modules/.vite/**/results.json
- Pack: add Package-scoped !node_modules/.vite-temp/** and !dist/**
@graphite-app
graphite-appBot merged commit 3fc8005 into mainMar 30, 2026
33 checks passed
@graphite-app
graphite-appBot deleted the fix-cache-missing branch March 30, 2026 15:12
Sign up for freeto 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.

2 participants

@fengmk2@wan9chi