Skip to content

perf(remote): Skip ILookup in read error detection walk#10

Merged
chickendreanso merged 7 commits intomainfrom
skip-ilookup-in-detect-walk
Apr 24, 2026
Merged

perf(remote): Skip ILookup in read error detection walk#10
chickendreanso merged 7 commits intomainfrom
skip-ilookup-in-detect-walk

Conversation

@chickendreanso
Copy link
Copy Markdown
Collaborator

@chickendreanso chickendreanso commented Apr 23, 2026

  • execute-read now detects per-path errors by walking the matcher's realized :val instead of the raw data, so ILookups are invoked once per accessed key in the pattern
  • Fold error detection, var filtering, and all-covered classification into one pass via new private classify-vars; replaces detect-read-errors and classify-result
  • Drop error-covered bindings from partial-success read responses — previously leaked as nil, or as the error map itself when a var's path landed exactly on the error
  • Match-failure branch reuses the already-computed errs list for the "covered by existing error?" check, avoiding a separate pass over the error map
  • Update remote/CLAUDE.md to describe the single-pass detection
  • Bump remote to 0.1.4 with matching changelog entry

@chickendreanso chickendreanso merged commit 975687c into main Apr 24, 2026
2 checks passed
@chickendreanso chickendreanso deleted the skip-ilookup-in-detect-walk branch April 24, 2026 02:44
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