Skip to content

Conversation

@magnetised
Copy link
Contributor

Preserve param_ref metadata in infer_unknown and explicit_cast_const so parameters like $1::text can be resolved during query rebuilding.

Fixes #3686

@codecov
Copy link

codecov bot commented Jan 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.93%. Comparing base (b11b8ea) to head (636bcbe).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3687   +/-   ##
=======================================
  Coverage   87.93%   87.93%           
=======================================
  Files          23       23           
  Lines        1898     1898           
  Branches      499      502    +3     
=======================================
  Hits         1669     1669           
  Misses        227      227           
  Partials        2        2           
Flag Coverage Δ
packages/experimental 87.73% <ø> (ø)
packages/react-hooks 86.48% <ø> (ø)
packages/start 89.58% <ø> (ø)
packages/typescript-client 93.35% <ø> (ø)
packages/y-electric 56.05% <ø> (ø)
typescript 87.93% <ø> (ø)
unit-tests 87.93% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@blacksmith-sh

This comment has been minimized.

…RE clause parameters

Preserve param_ref metadata in infer_unknown and explicit_cast_const
so parameters like $1::text can be resolved during query rebuilding.

Fixes #3686
@magnetised magnetised force-pushed the magnetised/issue-3686-keyerror branch from 11641a4 to 636bcbe Compare January 8, 2026 15:07
@magnetised magnetised merged commit 9d27e85 into main Jan 12, 2026
48 of 50 checks passed
@magnetised magnetised deleted the magnetised/issue-3686-keyerror branch January 12, 2026 10:39
@github-actions
Copy link
Contributor

This PR has been released! 🚀

The following packages include changes from this PR:

Thanks for contributing to Electric!

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.

KeyError with parameterised queries

3 participants