Skip to content

185#192

Merged
RAprogramm merged 1 commit into
mainfrom
185
Jul 4, 2026
Merged

185#192
RAprogramm merged 1 commit into
mainfrom
185

Conversation

@RAprogramm

Copy link
Copy Markdown
Owner

Closes #185.

The generated upsert existed only on the pool-backed repository; login-style flows that must share a transaction with adjacent statements (release a stale unique value, then upsert) were forced back to hand-written INSERT ... ON CONFLICT.

Behavior

  • With both transactions and upsert(...), {Entity}TransactionRepo gains upsert with identical SQL and action semantics (updateEntity, nothingOption<Entity>), executed on the transaction handle
  • Absent without either attribute; create-fields requirement matches the pool method

Testing

  • 3 adapter codegen tests (update/nothing shapes, gating) — 671+3 green under postgres,transactions
  • trybuild upsert pass case extended: adapter upsert exercised inside begin()/commit()
  • clippy -D warnings clean, all-features suite green, deny ok — checklist run before commit

Docs

  • README upsert section notes the transactional variant; wiki (Attributes ×5) updated in this PR

@codecov

codecov Bot commented Jul 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@RAprogramm RAprogramm merged commit 46d9990 into main Jul 4, 2026
15 checks passed
@RAprogramm RAprogramm deleted the 185 branch July 4, 2026 09:03
@RAprogramm RAprogramm mentioned this pull request Jul 4, 2026
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.

feat(transactions): expose upsert on the transaction adapter

1 participant