fix(postgrest): build the typed transform builder with the row converter

[spydon]

Summary

Main does not compile since #1848 and #1849 merged within minutes of each other. #1849 replaced the typed transform builder's PostgrestTable field with a RowConverter<Row> (_rowFromJson), and #1848 added stripNulls, dryRun, maxAffected and csv on top of the old field. Neither PR's CI saw the other, so every check on main and on the open PRs fails at compile time in postgrest_typed_transform_builder.dart:

Error: The getter '_table' isn't defined for the type 'PostgrestTypedTransformBuilder<Row, T>'.

Changes

• The four methods from feat(postgrest): add csv, explain, head, dryRun, stripNulls, maxAffected and geojson to the typed transform builder #1848 pass _rowFromJson to the private constructor, like the rest of the class does after feat(postgrest)!: only accept the table's Insert and Update types on the typed builder #1849.
• The two tests from feat(postgrest): add csv, explain, head, dryRun, stripNulls, maxAffected and geojson to the typed transform builder #1848 that passed raw maps use BookInsert and BookUpdate, which feat(postgrest)!: only accept the table's Insert and Update types on the typed builder #1849 made the only accepted payload types on the typed builder.
• The dryRun dartdoc example uses BookInsert as well.

No public API change. dart analyze on packages and examples is clean, typed_query_test.dart passes (49), DCM is clean.

Summary by CodeRabbit

• Bug Fixes

  • Improved typed mutation operations so options such as dry runs, affected-row limits, CSV responses, and null stripping consistently preserve the expected result types.
  • Updated typed insert and update usage in mutation scenarios to ensure submitted data is handled with the correct typed representations.

• Documentation

  • Updated the dry-run example to demonstrate usage with a typed insert value.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: b1e86a07-f54c-49eb-80eb-cb66966ca470

📥 Commits

Reviewing files that changed from the base of the PR and between 333cc5c and 66757bb.

📒 Files selected for processing (2)

• packages/postgrest/lib/src/postgrest_typed_transform_builder.dart
• packages/postgrest/test/typed_query_test.dart

Included review availability: Your plan provides up to 4 included reviews per hour; 0 remain after this review.

---

📝 Walkthrough

Walkthrough

The typed transform builder now forwards _rowFromJson to wrapped builders for stripNulls, dryRun, maxAffected, and csv. Two tests now pass typed mutation instances instead of raw maps.

Changes

Typed transform builder

Layer / File(s)	Summary
Converter forwarding and typed mutation tests packages/postgrest/lib/src/postgrest_typed_transform_builder.dart, packages/postgrest/test/typed_query_test.dart	Result modifiers pass _rowFromJson to wrapped builders. The dryRun example and two mutation tests use typed BookInsert and BookUpdate instances. Existing request and response assertions remain unchanged.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Bug fix

Suggested reviewers: grdsdev

Merge Risk: ⚪ Minimal · up to 66757

This is a localized typed-builder compilation fix with updated typed mutation tests and no identified behavioral or availability risk.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and concisely describes the main change: constructing the typed transform builder with the row converter to fix compilation failures.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[QuintinWillison]

😬