Skip to content

fix(conversations): reject blank archive ids - #169

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

fix(conversations): reject blank archive ids#169
ralyodio merged 1 commit into
profullstack:masterfrom
aiirvizionz:clientkit/archive-conversation-id-trim

Conversation

@aiirvizionz

Copy link
Copy Markdown
Contributor

Summary

  • Trim archive and unarchive conversationId values before passing them to RPC calls.
  • Return the existing missing-id 400 response for whitespace-only ids.
  • Add regression coverage for archive/unarchive blank ids 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/conversations/archive/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 01e4b2a 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