Merge branch 'main' into loop

Author: danielroe

.github/workflows/autofix.yml

@@ -31,8 +31,8 @@ jobs:
       - name: 📦 Install dependencies
         run: pnpm install
 
-      - name: 📦 Install browsers
-        run: pnpm playwright install
+      - name: 🌐 Install browser
+        run: pnpm playwright install chromium-headless-shell
 
       - name: 🌐 Compare translations
         run: pnpm i18n:check

.github/workflows/ci.yml

@@ -56,7 +56,7 @@ jobs:
         run: pnpm install
 
       - name: 🌐 Install browser
-        run: pnpm playwright install
+        run: pnpm playwright install chromium-headless-shell
 
       - name: 💪 Type check
         run: pnpm test:types

CONTRIBUTING.md

@@ -522,6 +522,31 @@ Format: `type(scope): description`
 > [!NOTE]
 > The subject must start with a lowercase letter. Individual commit messages within your PR don't need to follow this format since they'll be squashed.
 
+### PR descriptions
+
+If your pull request directly addresses an open issue, use the following inside your PR description.
+
+```text
+Resolves | Fixes | Closes: #xxx
+```
+
+Replace `#xxx` with either a URL to the issue, or the number of the issue. For example:
+
+```text
+Fixes #123
+```
+
+or
+
+```text
+Closes https://github.com/npmx-dev/npmx.dev/issues/123
+```
+
+This provides the following benefits:
+
+- it links the pull request to the issue (the merge icon will appear in the issue), so everybody can see there is an open PR
+- when the pull request is merged, the linked issue is automatically closed
+
 ## Pre-commit hooks
 
 The project uses `lint-staged` with `simple-git-hooks` to automatically lint files on commit.

app/components/Header/MobileMenu.vue …/components/Header/MobileMenu.client.vueapp/components/Header/MobileMenu.vue renamed to app/components/Header/MobileMenu.client.vue app/components/Header/MobileMenu.vue renamed to app/components/Header/MobileMenu.client.vue

@@ -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', {

app/composables/useAtproto.ts

@@ -28,7 +28,7 @@ export default defineConfig<ConfigOptions>({
   },
   projects: [
     {
-      name: 'chromium',
+      name: 'chromium-headless-shell',
       use: { ...devices['Desktop Chrome'] },
     },
   ],

playwright.config.ts

@@ -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',