fix(deps): update dependency graphql to v17

.pre-commit-config.yaml

@@ -11,7 +11,7 @@ repos:
     - --allow-multiple-documents
   - id: check-added-large-files
 - repo: https://github.com/markdownlint/markdownlint
-  rev: v0.18.1
+  rev: v0.18.0
   hooks:
   - id: markdownlint
 - repo: https://gitlab.com/devopshq/gitlab-ci-linter
@@ -22,13 +22,13 @@ repos:
     - --project
     - unboundsoftware/schemas-app
 - repo: https://github.com/alessandrojcm/commitlint-pre-commit-hook
-  rev: v9.27.0
+  rev: v9.26.0
   hooks:
   - id: commitlint
     stages: [ commit-msg ]
     additional_dependencies: [ '@commitlint/config-conventional' ]
 - repo: https://github.com/pre-commit/mirrors-eslint
-  rev: v10.11.0
+  rev: v10.8.0
   hooks:
   - id: eslint
     types: [ file ]

Dockerfile

@@ -12,7 +12,7 @@ RUN npm run postinstall && npm run generate && npm run lint && npm run test:cove
 FROM scratch as export
 COPY --from=builder /build/coverage /
 
-FROM amd64/nginx:1.31.6@sha256:8e1fce8cb4de80385f5fc98dd9d8cf0fe75d5dbac383c048eff65871a0d1065e
+FROM amd64/nginx:1.31.3@sha256:48d15d50712293218f8e39cac166d2dc51c10d569b7223d69ca495710cda9374
 MAINTAINER Joakim Olsson <joakim@unbound.se>
 
 COPY ./nginx/nginx.conf /etc/nginx/nginx.conf

app/components/__tests__/OrganizationSwitcher.spec.ts

@@ -14,23 +14,20 @@ const mockOrganizations = [
 // For now, the component is indirectly tested through useOrganizationSelector tests
 // and manual testing shows it works correctly
 
-// vi.mock is hoisted above the imports and the factory runs once, on the first
-// import of the mocked module -- it was never re-applied per test despite sitting
-// in beforeEach. vitest 5 rejects the nested call outright rather than silently
-// hoisting it, so declare it where it actually executes.
-vi.mock('~/composables/useOrganizationSelector', () => ({
-  useOrganizationSelector: () => ({
-    organizations: ref([mockOrganizations[0]]),
-    selectedOrganization: ref(mockOrganizations[0]),
-    selectedOrgId: ref('1'),
-    selectOrganization: mockSelectOrganization,
-  }),
-}))
-
 describe('OrganizationSwitcher - Single Organization', () => {
   beforeEach(() => {
     vi.resetModules()
     mockSelectOrganization.mockClear()
+
+    // Mock with single organization
+    vi.mock('~/composables/useOrganizationSelector', () => ({
+      useOrganizationSelector: () => ({
+        organizations: ref([mockOrganizations[0]]),
+        selectedOrganization: ref(mockOrganizations[0]),
+        selectedOrgId: ref('1'),
+        selectOrganization: mockSelectOrganization,
+      }),
+    }))
   })
 
   it('should show a chip for single organization', async () => {

package.json

@@ -19,12 +19,12 @@
   },
   "dependencies": {
     "@apollo/client": "3.14.1",
-    "@auth0/auth0-vue": "2.10.0",
+    "@auth0/auth0-vue": "2.9.0",
     "@vue/apollo-composable": "4.2.2",
     "graphql": "17.0.2",
-    "graphql-ws": "6.3.0",
+    "graphql-ws": "6.2.1",
     "nuxt": "4.5.2",
-    "vue": "3.5.43",
+    "vue": "3.5.41",
     "vuetify-nuxt-module": "0.19.5"
   },
   "version": "1.0.0",
@@ -34,31 +34,31 @@
   "author": "",
   "license": "ISC",
   "devDependencies": {
-    "@commitlint/cli": "21.2.3",
-    "@commitlint/config-conventional": "21.2.3",
-    "@graphql-codegen/cli": "7.4.3",
+    "@commitlint/cli": "21.2.1",
+    "@commitlint/config-conventional": "21.2.0",
+    "@graphql-codegen/cli": "7.2.0",
     "@graphql-codegen/typed-document-node": "7.1.0",
     "@graphql-codegen/typescript": "6.1.0",
-    "@graphql-codegen/typescript-operations": "6.1.7",
+    "@graphql-codegen/typescript-operations": "6.1.5",
     "@graphql-codegen/typescript-vue-apollo": "5.0.1",
-    "@nuxt/devtools": "3.4.2",
+    "@nuxt/devtools": "3.4.1",
     "@nuxt/eslint": "1.17.0",
     "@nuxtjs/stylelint-module": "5.2.1",
     "@stylistic/eslint-plugin": "5.10.0",
-    "@vitejs/plugin-vue": "6.0.9",
-    "@vitest/coverage-v8": "5.0.1",
-    "@vue/test-utils": "2.5.1",
-    "eslint": "10.11.0",
+    "@vitejs/plugin-vue": "6.0.8",
+    "@vitest/coverage-v8": "4.1.10",
+    "@vue/test-utils": "2.4.11",
+    "eslint": "10.8.0",
     "eslint-plugin-simple-import-sort": "14.0.0",
-    "eslint-plugin-vue": "10.11.1",
-    "jsdom": "30.1.1",
+    "eslint-plugin-vue": "10.10.0",
+    "jsdom": "30.0.1",
     "postcss-html": "2.0.0",
-    "stylelint": "17.15.0",
+    "stylelint": "17.14.1",
     "stylelint-config-recommended-vue": "2.0.0",
     "stylelint-config-standard": "40.0.0",
     "typescript": "6.0.3",
-    "typescript-eslint": "8.70.1",
-    "vitest": "5.0.1",
+    "typescript-eslint": "8.66.0",
+    "vitest": "4.1.10",
     "vue-eslint-parser": "10.4.1"
   },
   "overrides": {

test/setup.ts

@@ -31,15 +31,7 @@ class LocalStorageMock {
   }
 }
 
-// jsdom defines localStorage as a getter-only accessor on the window object, so a
-// plain assignment throws ("Cannot set property localStorage ... which has only a
-// getter") under vitest 5, which no longer shadows it with a writable global.
-// defineProperty replaces the accessor outright and works on both.
-Object.defineProperty(globalThis, 'localStorage', {
-  value: new LocalStorageMock() as unknown as Storage,
-  writable: true,
-  configurable: true,
-})
+global.localStorage = new LocalStorageMock() as unknown as Storage
 
 // Make Vue composables globally available
 global.ref = Vue.ref