psa-systems/lets-chat
Watch
9
0
Fork
You've already forked lets-chat
 0
Code Issues Pull requests Projects Releases 2 Packages Wiki Activity Actions

fix(tests): AppState drift in routes_reports - add bunyip_sso None (LC-22) #348

Merged
YousifShkara merged 1 commit from fix/lc-22-routes-reports-appstate-drift into main 2026-06-18 05:47:32 +02:00
Conversation 0 Commits 1 Files changed 1 +1
YousifShkara commented 2026-06-18 05:47:29 +02:00
Owner
Copy link

LC-334 (message reporting) merged the same hour as the LC-22 cutover and
added server/tests/routes_reports.rs, which constructs AppState { ... }
by hand and is missing the new bunyip_sso field. CI's clippy on main
now fails with E0063. Add the field.

CLAUDE.md §"AppState construction drift" applies here - this is the kind
of cross-merge collision the doc calls out.

Co-Authored-By: Claude Opus 4.7 noreply@anthropic.com

LC-334 (message reporting) merged the same hour as the LC-22 cutover and added `server/tests/routes_reports.rs`, which constructs `AppState { ... }` by hand and is missing the new `bunyip_sso` field. CI's clippy on main now fails with E0063. Add the field. CLAUDE.md §"AppState construction drift" applies here - this is the kind of cross-merge collision the doc calls out. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
fix(tests): AppState drift in routes_reports - add bunyip_sso None (LC-22)
Some checks failed
check-secrets / Nosey parker (push) Successful in 3s
Details
check-secrets / Kingfisher (push) Successful in 4s
Details
check-secrets / TruffleHog (push) Successful in 5s
Details
Create release / Create release from merged PR (pull_request) Has been skipped
Details
check-secrets / Kingfisher (pull_request) Successful in 4s
Details
check-secrets / Nosey parker (pull_request) Successful in 5s
Details
check-secrets / TruffleHog (pull_request) Successful in 5s
Details
Check / clippy + fmt + tests (pull_request) Failing after 54s
Details
1fee5533d5 
LC-334 (message reporting) merged the same hour as the LC-22 cutover and
added `server/tests/routes_reports.rs`, which constructs `AppState { ... }`
by hand and is missing the new `bunyip_sso` field. CI's clippy on main
now fails with E0063. Add the field.

CLAUDE.md §"AppState construction drift" applies here - this is the kind
of cross-merge collision the doc calls out.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
YousifShkara deleted branch fix/lc-22-routes-reports-appstate-drift 2026-06-18 05:47:32 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
YousifShkara
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
psa-systems/lets-chat!348
No description provided.