Skip to content

Fix lock file check regression for in-process watchers - #5505

Merged
myovchev merged 1 commit into
mainfrom
pro-9688-lock-file
Jul 9, 2026
Merged

myovchev merged 1 commit into
mainfrom
pro-9688-lock-file

Conversation

@myovchev

@myovchev myovchev commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Please indicate which branch this PR should merge into:

Check one

  • main
  • latest
  • stable
  • Check if this PR will be resubmitted against another branch

Summary

The lock file dependency check that forces a full rebuild now runs once per process. Watcher-triggered rebuilds stay scoped to the detected changes instead of rebuilding everything on every file change when the lock file changed or is absent. This bug was in effect only for projects missing a lock file in their npmRoot (e.g. npm monorepos).

What are the specific steps to test this change?

For example:

File change on projects not having a lock file in their npmRoot doesn't trigger full rebuild.

What kind of change does this PR introduce?

(Check at least one)

  • Bug fix
  • New feature
  • Refactor
  • Documentation
  • Build-related changes
  • Other

Make sure the PR fulfills these requirements:

  • It includes a) the existing issue ID being resolved, b) a convincing reason for adding this feature, or c) a clear description of the bug it resolves
  • The changelog is updated
  • Related documentation has been updated
  • Related tests have been updated

If adding a new feature without an already open issue, it's best to open a feature request issue first and wait for approval before working on it.

Other information:

@linear

linear Bot commented Jul 9, 2026

Copy link
Copy Markdown

PRO-9688

@myovchev
myovchev requested review from BoDonkey and boutell and removed request for boutell July 9, 2026 11:38
@myovchev
myovchev merged commit c8b7004 into main Jul 9, 2026
21 checks passed
@myovchev
myovchev deleted the pro-9688-lock-file branch July 9, 2026 12:48
boutell added a commit that referenced this pull request Jul 10, 2026
Co-authored-by: Miro Yovchev <2827783+myovchev@users.noreply.github.com>
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