Skip to content

feat(Record): make fromIterableBy dual/pipeable#6111

Closed
mitre88 wants to merge 2 commits intoEffect-TS:mainfrom
mitre88:fix/record-fromIterableBy-dual
Closed

feat(Record): make fromIterableBy dual/pipeable#6111
mitre88 wants to merge 2 commits intoEffect-TS:mainfrom
mitre88:fix/record-fromIterableBy-dual

Conversation

@mitre88
Copy link
Copy Markdown
Contributor

@mitre88 mitre88 commented Mar 6, 2026

Summary

Make Record.fromIterableBy support both data-first and data-last (pipeable) calling conventions, consistent with Record.fromIterableWith and other dual APIs in the Record module.

Before

// Only data-first works
pipe(rows, (rows) => Record.fromIterableBy(rows, Struct.get("authEmail")))

After

// Both styles work
Record.fromIterableBy(rows, Struct.get("authEmail"))       // data-first
pipe(rows, Record.fromIterableBy(Struct.get("authEmail"))) // data-last

Closes #6092

Make Record.fromIterableBy support both data-first and data-last
(pipeable) calling conventions, consistent with fromIterableWith
and other dual APIs in the Record module.

Closes Effect-TS#6092
@mitre88 mitre88 requested a review from mikearnaldi as a code owner March 6, 2026 18:12
@github-project-automation github-project-automation Bot moved this to Discussion Ongoing in PR Backlog Mar 6, 2026
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Mar 6, 2026

🦋 Changeset detected

Latest commit: 43661a4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 36 packages
Name Type
effect Patch
@effect/cli Patch
@effect/cluster Patch
@effect/experimental Patch
@effect/opentelemetry Patch
@effect/platform-browser Patch
@effect/platform-bun Patch
@effect/platform-node-shared Patch
@effect/platform-node Patch
@effect/platform Patch
@effect/printer-ansi Patch
@effect/printer Patch
@effect/rpc Patch
@effect/sql-clickhouse Patch
@effect/sql-d1 Patch
@effect/sql-drizzle Patch
@effect/sql-kysely Patch
@effect/sql-libsql Patch
@effect/sql-mssql Patch
@effect/sql-mysql2 Patch
@effect/sql-pg Patch
@effect/sql-sqlite-bun Patch
@effect/sql-sqlite-do Patch
@effect/sql-sqlite-node Patch
@effect/sql-sqlite-react-native Patch
@effect/sql-sqlite-wasm Patch
@effect/sql Patch
@effect/typeclass Patch
@effect/vitest Patch
@effect/workflow Patch
@effect/ai Patch
@effect/ai-amazon-bedrock Patch
@effect/ai-anthropic Patch
@effect/ai-google Patch
@effect/ai-openai Patch
@effect/ai-openrouter Patch

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

@mitre88
Copy link
Copy Markdown
Contributor Author

mitre88 commented Apr 13, 2026

Gentle ping — this feature PR to make Record.fromIterableBy dual/pipeable has all CI checks passing. Would appreciate a review when possible. Thanks!

@mitre88
Copy link
Copy Markdown
Contributor Author

mitre88 commented Apr 18, 2026

Changeset added for this PR. Ready for review!

@tim-smart
Copy link
Copy Markdown
Contributor

Hello! Effect v3 is under a feature freeze now. Feel free to contribute this to the v4 repository.

@tim-smart tim-smart closed this Apr 22, 2026
@github-project-automation github-project-automation Bot moved this from Discussion Ongoing to Done in PR Backlog Apr 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Record.fromIterableBy is not dual/curried (awkward in pipe)

2 participants