Skip to content

Allow flatMap streams without index fields to merge - #719

Open
ianmacartney wants to merge 4 commits into
mainfrom
codex/investigate-issue-690-in-convex-helpers
Open

Allow flatMap streams without index fields to merge#719
ianmacartney wants to merge 4 commits into
mainfrom
codex/investigate-issue-690-in-convex-helpers

Conversation

@ianmacartney

@ianmacartney ianmacartney commented Aug 5, 2025

Copy link
Copy Markdown
Member

Summary

  • allow flatMap to skip index field normalization when no mapped fields are provided
  • relax FlatMapStream index field checks for empty mapped fields
  • add regression test reproducing merging of flat-mapped and direct streams

Fixes #690

Testing

  • npm test

https://chatgpt.com/codex/tasks/task_b_68927274cefc832aa5721d4adfe91819

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@pkg-pr-new

pkg-pr-new Bot commented Aug 5, 2025

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/get-convex/convex-helpers@719

commit: ce0b700

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

impossible to mergeStream between flatmapped and not-flatmapped

1 participant