Skip to content

fix(ci): avoid recursive Next.js cache hashing - #6060

Merged
pandemicsyn merged 1 commit into
mainfrom
fix/next-cache-hash-timeout
Sep 10, 2026
Merged

fix(ci): avoid recursive Next.js cache hashing#6060
pandemicsyn merged 1 commit into
mainfrom
fix/next-cache-hash-timeout

Conversation

@pandemicsyn

Copy link
Copy Markdown
Contributor

The Next.js build job can fail before compilation because its cache key recursively hashes JavaScript and TypeScript files after dependencies are installed. The main run at 6ccb257c9 hit the 120-second hashFiles limit.

Use the root lockfile hash and commit SHA for the cache key. Keep the lockfile-based restore prefix so builds can reuse the previous incremental cache without scanning the dependency tree.

Validation: workflow YAML parsing, changed-file formatting, and git diff --check passed. The application build was not rerun locally; the existing paths filter skips it for workflow-only changes.

@kilo-code-bot

kilo-code-bot Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

The workflow-only change safely replaces recursive source hashing in the Next.js cache key with the root lockfile hash and commit SHA, preserving incremental-cache reuse via the lockfile-scoped restore prefix; no defects found.

Files Reviewed (1 file)
  • .github/workflows/ci.yml

Reviewed by deepseek-v4.1-flash · Input: 23.8K · Output: 5K · Cached: 194.6K

Review guidance: REVIEW.md from base branch main

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.

2 participants