fix: watcher, incremental git, parse-worker, extractor gaps#115
Conversation
- Watcher: do not set watchActive after stop() during pending onPrime - getChangedFiles: use name-status diff so committed renames drop old paths - parse-worker: isolate statSync failures like read errors - Extractors: multi-declarator arrow scopes, refs, side-effects, test.each, process.env bare/bracket, unquoted CSS @import url()
🦋 Changeset detectedLatest commit: 1b5ab66 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (19)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Top-level `new Map()` in cache.ts and store.ts are now correctly flagged as module side effects.
Match arrow declarators: push scope when visiting the function body, not at VariableDeclaration enter.
Summary
Indexer and watcher fixes after #114:
watchActiveafterstop()duringonPrimestoppedguard + regression testgit diff --name-status --no-renames, parseD→deletedstatSyncin same per-file try/catch as readArrowFunctionExpression/FunctionExpressionvisitVariableDeclarator+for..of/for..inpattern handlingnew/++side effectsNewExpression/UpdateExpressionhandlerstest.eachcurried callsCallExpressioncallee + title from outer argsprocess.envVariableDeclarator+ computed literal branch@import url(...)IMPORT_URL_UNQUOTED_REAlso updates
find-side-effect-filesgolden (cache.ts/store.tstop-levelnew Map()).Test plan
bun test(watcher, get-changed-files, parse-worker-core, behavioral, css-parser)bun run test:golden(find-side-effect-filesupdated)bun run typecheck