Skip to content

fix(messages): reject blank load conversation ids - #171

Merged
ralyodio merged 1 commit into
profullstack:masterfrom
aiirvizionz:clientkit/load-messages-conversation-id-trim
Jul 30, 2026
Merged

fix(messages): reject blank load conversation ids#171
ralyodio merged 1 commit into
profullstack:masterfrom
aiirvizionz:clientkit/load-messages-conversation-id-trim

Conversation

@aiirvizionz

Copy link
Copy Markdown
Contributor

Summary

  • Trim message-load conversationId values before validation.
  • Reject whitespace-only ids before user lookup, participant lookup, message query, or SSE room join.
  • Add regression coverage for blank ids, invalid limits, and trimmed valid ids.

Validation

  • git diff --check
  • node node_modules\vitest\vitest.mjs run --config %TEMP%\qryptchat-vitest-minimal.config.mjs "src/app/api/messages/load/route.test.js" (3 tests passed)

I used the same temporary minimal Vitest config needed locally because the repo's default Vitest config currently fails to load through the installed Vite/plugin combination; no dependency files were modified.

@ralyodio
ralyodio merged commit 8918626 into profullstack:master Jul 30, 2026
8 checks passed
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.

2 participants