Skip to content

[EPAC-2222]: Decouple lobbying repository Queryer from pgx types#730

Merged
github-actions[bot] merged 1 commit into
mainfrom
symphony/epac-2222-replace-pgx-typed-queryer-interface-with-databas
Jun 3, 2026
Merged

[EPAC-2222]: Decouple lobbying repository Queryer from pgx types#730
github-actions[bot] merged 1 commit into
mainfrom
symphony/epac-2222-replace-pgx-typed-queryer-interface-with-databas

Conversation

@riddim-developer-bot

@riddim-developer-bot riddim-developer-bot Bot commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Removed pgx-specific return types from the lobbying repository Queryer contract by replacing pgconn.CommandTag, pgx.Rows, and pgx.Row dependencies with repository-local database-agnostic interfaces.
  • Updated repository scan helpers to consume interface-based row types instead of pgx.Rows.
  • Updated LoadMPLobbyingSummary to use Query + Next() row handling and avoid direct pgx.ErrNoRows checks.
  • Added a main package adapter (newLobbyingQueryer) so existing *pgx.Conn service wiring can still satisfy the new repository contract.
  • Updated repository unit/integration tests to use the new queryer contract.

Validation

  • Not run (per handoff constraints): go test ./... and go build ./....
  • Not run: full backend/lobbying go test and build in this pass.
  • Performed focused code review on changed files and a final diff sanity pass.

Notes

  • grep -r "jackc/pgx" backend/lobbying remains non-zero in this PR because pgx remains intentionally present in existing adapter/test files outside this issue scope, to be finalized in EPAC-2220.

Linear

@riddim-developer-bot riddim-developer-bot Bot added the autonomous Enrolled in prconverged daemon for automated review and merge label Jun 3, 2026
@github-actions github-actions Bot enabled auto-merge (squash) June 3, 2026 18:49
@github-actions github-actions Bot merged commit 321a6ad into main Jun 3, 2026
44 of 58 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autonomous Enrolled in prconverged daemon for automated review and merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants