Skip to content

fix(users): exclude current user from search#159

Merged
ralyodio merged 1 commit into
profullstack:masterfrom
rissrice2105-agent:fix/search-exclude-current-user
Jul 14, 2026
Merged

fix(users): exclude current user from search#159
ralyodio merged 1 commit into
profullstack:masterfrom
rissrice2105-agent:fix/search-exclude-current-user

Conversation

@rissrice2105-agent

Copy link
Copy Markdown
Contributor

Summary

  • exclude the authenticated user from search with the auth_user_id column instead of the internal profile id
  • add a focused regression test for the Supabase query filter

Fixes #158

Validation

  • focused Vitest route test: 1 passed (minimal Node config used because the repository's default @vitejs/plugin-react / Vite dependency mismatch aborts before test collection)
  • npx oxlint src/app/api/users/search/route.js src/app/api/users/search/route.test.js (passes with the existing unused params warning)
  • git diff --check

@ralyodio
ralyodio merged commit 32ae7c5 into profullstack:master Jul 14, 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.

User search can return the authenticated user

2 participants