Skip to content

fix: do not hit session endpoint in ssr #807

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
danielroe merged 8 commits into from
Feb 3, 2026
+9 −2
Merged

fix: do not hit session endpoint in ssr #807

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
d20034e
fix: do not hit session endpoint in ssr
danielroe Feb 3, 2026
6bc7c57
fix: just set `server: false`
danielroe Feb 3, 2026
1d0f285
Merge remote-tracking branch 'origin/main' into fix/atproto
danielroe Feb 3, 2026
240eccb
chore: refresh session after mounting components
danielroe Feb 3, 2026
cf8f804
refactor: make mobile menu client-only
danielroe Feb 3, 2026
842f52e
fix: do not hit oauth session endpoint when testing
danielroe Feb 3, 2026
9b800b5
Merge remote-tracking branch 'origin/main' into fix/atproto
danielroe Feb 3, 2026
461459c
test: update
danielroe Feb 3, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
0 app/components/Header/MobileMenu.vue → app/components/Header/MobileMenu.client.vue
View file
Open in desktop
File renamed without changes.
9 changes: 8 additions & 1 deletion app/composables/useAtproto.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,14 @@
import type { UserSession } from '#shared/schemas/userSession'

export function useAtproto() {
const { data: user, pending, clear } = useFetch<UserSession | null>('/api/auth/session')
const {
data: user,
pending,
clear,
} = useFetch<UserSession | null>('/api/auth/session', {
server: false,
immediate: !import.meta.test,
})

async function logout() {
await $fetch('/api/auth/session', {
Expand Down
2 changes: 1 addition & 1 deletion test/unit/a11y-component-coverage.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ const SKIPPED_COMPONENTS: Record<string, string> = {
// Complex components requiring full app context or specific runtime conditions
'Header/OrgsDropdown.vue': 'Requires connector context and API calls',
'Header/PackagesDropdown.vue': 'Requires connector context and API calls',
'Header/MobileMenu.vue': 'Requires Teleport and full navigation context',
'Header/MobileMenu.client.vue': 'Requires Teleport and full navigation context',
'Modal.client.vue':
'Base modal component - tested via specific modals like ChartModal, ConnectorModal',
'Package/SkillsModal.vue': 'Complex modal with tabs - requires modal context and state',
Expand Down
Loading