Toutsu/GmRelayBot
1
1
Fork 0
Code Issues 18 Pull Requests 1 Actions 1 Packages Projects Releases 85 Wiki Activity

fix(shared): bind platform when creating group manager (v3.9.8) #134

Closed
Toutsu wants to merge 0 commits from fix/issue-133-on-conflict-inference into main
pull from: fix/issue-133-on-conflict-inference
merge into: Toutsu:main
Conversation 0 Commits - Files Changed -
Toutsu commented 2026-06-09 15:20:20 +03:00
Owner
Copy Link
Copy Source

Summary

Fixes #133.

Production PostgreSQL logs showed that CreateSessionHandler sent this statement during new group creation:

WHERE p.platform = @Platform

Dapper.AOT had replaced the other parameters with $1/$2/$3, but @Platform stayed literal because the anonymous parameter object for the group_managers insert did not include Platform. PostgreSQL then parsed @ as an operator and returned 42883: operator does not exist: @ character varying.

Changes

  • Add the missing Platform = platform parameter for the group_managers owner insert.
  • Add a Testcontainers/PostgreSQL regression test that runs production migrations and creates a session for a new platform group.
  • Bump version to 3.9.8 across Directory.Build.props, compose.yaml, deploy workflow, and web nav version.

Test plan

  • dotnet test tests/GmRelay.Bot.Tests/GmRelay.Bot.Tests.csproj --nologo --verbosity normal --filter "FullyQualifiedName~CreateSessionHandlerIntegrationTests"
    • RED before fix: 42883: operator does not exist: @ character varying
    • GREEN after fix: 1 passed
  • dotnet test tests/GmRelay.Bot.Tests/GmRelay.Bot.Tests.csproj --nologo --verbosity quiet
    • 604 passed, 2 skipped, 606 total
  • dotnet format --verify-no-changes --verbosity quiet
  • dotnet build --nologo --verbosity quiet

Workflow

  • CI passes (#313)
  • Code review approved
  • Deployed (#314)
  • Release published (v3.9.8)

Merged via fallback local merge because Gitea MCP merge returned HTTP 405. Merge commit: 5014ca5c58cb5c43cb0d64ce90077bce780dbdf3.

## Summary Fixes #133. Production PostgreSQL logs showed that `CreateSessionHandler` sent this statement during new group creation: ```sql WHERE p.platform = @Platform ``` Dapper.AOT had replaced the other parameters with `$1/$2/$3`, but `@Platform` stayed literal because the anonymous parameter object for the `group_managers` insert did not include `Platform`. PostgreSQL then parsed `@` as an operator and returned `42883: operator does not exist: @ character varying`. ## Changes - Add the missing `Platform = platform` parameter for the `group_managers` owner insert. - Add a Testcontainers/PostgreSQL regression test that runs production migrations and creates a session for a new platform group. - Bump version to `3.9.8` across `Directory.Build.props`, `compose.yaml`, deploy workflow, and web nav version. ## Test plan - `dotnet test tests/GmRelay.Bot.Tests/GmRelay.Bot.Tests.csproj --nologo --verbosity normal --filter "FullyQualifiedName~CreateSessionHandlerIntegrationTests"` - RED before fix: `42883: operator does not exist: @ character varying` - GREEN after fix: 1 passed - `dotnet test tests/GmRelay.Bot.Tests/GmRelay.Bot.Tests.csproj --nologo --verbosity quiet` - 604 passed, 2 skipped, 606 total - `dotnet format --verify-no-changes --verbosity quiet` - `dotnet build --nologo --verbosity quiet` ## Workflow - [x] CI passes (#313) - [x] Code review approved - [x] Deployed (#314) - [x] Release published (`v3.9.8`) Merged via fallback local merge because Gitea MCP merge returned HTTP 405. Merge commit: `5014ca5c58cb5c43cb0d64ce90077bce780dbdf3`.
Toutsu added 1 commit 2026-06-09 15:20:20 +03:00
fix(shared): bind platform when creating group manager
PR Checks / test-and-build (pull_request) Successful in 12m53s
Details
efd86bca0a 
Add a PostgreSQL integration regression test for new-platform-group session creation. The production failure was a missing Platform parameter in the group_managers insert, leaving @Platform in SQL and causing PostgreSQL 42883. Bump version to 3.9.8.
Toutsu closed this pull request 2026-06-09 15:56:40 +03:00
Some checks are pending
PR Checks / test-and-build (pull_request) Successful in 12m53s
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
area:bot
Telegram bot and worker logic
 area:data
Database schema, migrations and persistence model
 area:discord
Discord bot service and Discord adapter
 area:infra
Infrastructure, runtime and deployment
 area:miniapp
Telegram Mini App dashboard
 area:platform
Platform-neutral abstractions and adapter boundaries
 area:shared
Shared domain and rendering contracts
 area:web
Blazor web dashboard
 in-progress
В работе (issue взят агентом)
 next-up
Recommended immediate backlog
 pending-approval
Ждёт согласования плана решения
 platform:discord
Discord-specific behavior or adapter work
 platform:multi
Cross-platform or platform-neutral work
 platform:telegram
Telegram-specific behavior or adapter work
 priority:p0
Critical priority
 priority:p0
Блокирующий приоритет: задача должна быть выполнена до признания решения production-ready
 priority:p1
High priority
 priority:p2
Medium priority
 priority:p3
Lower priority
 security
Security-sensitive work
 type:bug
Bug fix or corrective work
 type:chore
Maintenance and supporting work
 type:feature
New product functionality
 type:refactor
Structural change without direct feature growth
 type:test
Testing and coverage work
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
Toutsu
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Toutsu/GmRelayBot#134
Delete Branch "fix/issue-133-on-conflict-inference"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?