feat/pin-message #66

Merged

longjacksonle merged 7 commits from feat/pin-message into main

2026-05-10 23:56:29 +02:00

longjacksonle commented

2026-05-10 23:56:25 +02:00

Owner

No description provided.

longjacksonle added 7 commits

2026-05-10 23:56:25 +02:00

docs(pinned): add Phase 19 pinned messages implementation plan 09cf4bb145

feat(pinned): add DB layer for pinned messages 74b0426cbe

Add the `pinned_messages` SQLite table (migration 0016) and a new `db::pinned` module with idempotent pin/unpin operations, per-room cap enforcement (50 pins), bulk-fetch helpers, and a soft-delete-aware count query. Also add `db::auth::display_names_for_ids` for bulk cross-pool user resolution to avoid N+1 lookups when rendering the pin strip and full-list page.

feat(pinned): add MessagePinned and MessageUnpinned WebSocket events e17ff863e3

feat(pinned): add pin/unpin routes, strip UI, and pins list page 293e6961dd

Implements the HTTP layer for Phase 19 pinned messages:

- POST /messages/{id}/pin and DELETE /messages/{id}/pin write to the DB, broadcast MessagePinned/MessageUnpinned WS events, and return an OOB-swapped pinned strip fragment so the requesting tab updates immediately.
- GET /room/{id}/pins and GET /dm/{peer_id}/pins render a standalone page listing all pins for the room newest-first, with per-row Unpin buttons that remove themselves from the list via hx-swap="outerHTML".
- A pinned strip appears below the room/DM header, pre-rendered server-side at page load and updated live via WS broadcasts. Caps display at 3 entries with a "See all (N) pinned" link to the full list.
- MessageView gains is_pinned so the hover menu shows Pin vs Unpin without an extra per-message query; page renders bulk-load pinned ids in a single query, while per-message broadcast sites set it to false (the next full page load corrects the state).
- AppError::Conflict(String) maps to HTTP 409 and is returned when the per-room pin cap is exceeded.

test(pinned): add route integration tests for pin/unpin feature f53671c911

chore(pinned): apply rustfmt and add migration 0016 to test fixtures 53ed844508

chore(settings): add allowed commands for check, verify, server-down, and gh auth bda082d89f