Skip to content

Commit 5f20469

Browse files
authored
Merge branch 'main' into ci/add-codiff-ecosystem-test
2 parents 4718442 + 5f5ce6e commit 5f20469

1 file changed

Lines changed: 15 additions & 1 deletion

File tree

.github/workflows/test-vp-create.yml

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -322,10 +322,24 @@ jobs:
322322
if: matrix.template.name == 'monorepo'
323323
working-directory: ${{ runner.temp }}/test-project
324324
run: |
325+
# Under npm, `vp run ready` reaches 100% cache hit only on the
326+
# third invocation (#1638): vite-task's directory-listing
327+
# fingerprint and fspy read/write tracking surface false-positive
328+
# misses on run #2 because `packages/utils/node_modules/` is born
329+
# during run #1. pnpm/yarn/bun pre-create per-package
330+
# `node_modules/` at install time and reach 100% on run #2.
331+
# The preceding `Verify project builds` step already invoked
332+
# `vp run ready` once (verify-command for monorepo), so one
333+
# extra warm-up here is enough under npm.
334+
if [ "${{ matrix.package-manager }}" = "npm" ]; then
335+
vp run ready >/dev/null 2>&1
336+
fi
325337
output=$(vp run ready 2>&1)
326338
echo "$output"
327339
if ! echo "$output" | grep -q 'cache hit (100%)'; then
328-
echo "✗ Expected 100% cache hit on second run"
340+
echo "✗ Expected 100% cache hit"
341+
echo "--- vp run --last-details (cache-miss diagnostics) ---"
342+
vp run --last-details || true
329343
exit 1
330344
fi
331345
echo "✓ 100% cache hit verified"

0 commit comments

Comments
 (0)