Repository navigation
refactor(runtime-tags): reset every pass memo with its solve pass - #4523
Conversation
A memo over the reasons of the moment is now declared as one (`createPassMemo`, `createPassPathMemo`), and `solveReasons` starts all of them afresh with one `resetPassMemos()` instead of a list of resets kept by hand next to the loop. Main's two such memos, `extraReaders` and `bindingReaders`, reset as before, so no output changes. A memo added later can no longer miss its reset: the declaration registers it. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #4523 +/- ##
=======================================
Coverage 94.86% 94.86%
=======================================
Files 419 419
Lines 20906 20910 +4
Branches 4176 4176
=======================================
+ Hits 19832 19836 +4
Misses 617 617
Partials 457 457 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
The reasons solve resets the memos whose answers depend on the reasons of the moment at the start of every pass, and that list of resets was kept by hand next to the loop, so a memo added later could silently miss its reset. Being per-pass is now a property of the memo's declaration:
createPassMemoandcreatePassPathMemoregister their reset when declared, andsolveReasonscallsresetPassMemos()once per pass.extraReadersandbindingReadersmove to it and reset exactly as before, so no output changes.🤖 Generated with Claude Code