Skip to content

fix(chat): resolve async mark-read route params#116

Merged
ralyodio merged 1 commit into
profullstack:masterfrom
rissrice2105-agent:fix/qryptchat-valid-webhook-pr
Jun 29, 2026
Merged

fix(chat): resolve async mark-read route params#116
ralyodio merged 1 commit into
profullstack:masterfrom
rissrice2105-agent:fix/qryptchat-valid-webhook-pr

Conversation

@rissrice2105-agent

Copy link
Copy Markdown
Contributor

Summary

  • Resolve async Next.js route params before reading the conversation id in the mark-read API
  • Return a 400 response when the route id is missing instead of continuing with an undefined conversation filter
  • Add regression coverage for Promise-based params

Tests

  • corepack pnpm exec vitest run --config $env:TEMP/qryptchat-vitest-no-react.config.mjs "src/app/api/chat/conversations/[id]/mark-read/route.test.js"
  • corepack pnpm exec oxlint "src/app/api/chat/conversations/[id]/mark-read/route.js" "src/app/api/chat/conversations/[id]/mark-read/route.test.js"

Note: the default vitest.config.js currently fails to load locally because @vitejs/plugin-react imports a non-exported Vite internal path, so the focused test was run with a minimal no-React Vitest config.

@ralyodio ralyodio merged commit 34b517a into profullstack:master Jun 29, 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