chore(eslint): fix linting on CI and resolve issues

REUSE.toml

@@ -400,7 +400,7 @@ SPDX-FileCopyrightText = "2019 Fabian Wiktor <https://www.pexels.com/photo/green
 SPDX-License-Identifier = "CC0-1.0"
 
 [[annotations]]
-path = ["openapi.json", ".envrc", "flake.nix", "flake.lock", "build/eslint-baseline.json"]
+path = ["openapi.json", ".envrc", "flake.nix", "flake.lock", "build/eslint-baseline.json", "build/eslint-baseline-legacy.json"]
 precedence = "aggregate"
 SPDX-FileCopyrightText = "2025 Nextcloud GmbH and Nextcloud contributors"
 SPDX-License-Identifier = "AGPL-3.0-or-later"

apps/files_versions/src/components/VirtualScrolling.vue

@@ -96,6 +96,8 @@ export default defineComponent({
 		},
 	},
 
+	emits: ['need-content'],
+
 	data() {
 		return {
 			scrollPosition: 0,
@@ -312,7 +314,7 @@ export default defineComponent({
 		this.container.addEventListener('scroll', this.updateScrollPosition, { passive: true })
 	},
 
-	beforeDestroy() {
+	beforeUnmount() {
 		if (this.useWindow) {
 			window.removeEventListener('resize', this.updateContainerSize)
 		}

apps/files_versions/src/utils/davClient.ts

@@ -12,10 +12,10 @@ const rootPath = 'dav'
 const remote = generateRemoteUrl(rootPath)
 const client = createClient(remote)
 
-// set CSRF token header
 /**
+ * set CSRF token header
  *
- * @param token
+ * @param token - CSRF token
  */
 function setHeaders(token) {
 	client.setHeaders({

build/demi.sh

@@ -30,17 +30,25 @@ run_sequentially() {
 	echo -e "\e[1;34m>> Running 'npm $COMMAND' for Vue 3 based frontend\e[0m"
 	echo
 	pushd "$FRONTEND"
+	set +e
 	npm $COMMAND
+	STATUS=$?
+	set -e
 	popd
 
 	echo -e "\e[1;34m>> Running 'npm $COMMAND' for Vue 2 based frontend\e[0m"
 	echo
 	pushd "$FRONTEND_LEGACY"
 	npm $COMMAND
 	popd
-}
 
+	if [ $STATUS -ne 0 ]; then
+		echo -e "\e[1;31m>> Vue 3 based frontend build failed\e[0m"
+		exit $STATUS
+	fi
+}
 
+set -e
 if [ "--parallel" = "$1" ]; then
 	build_command ${@:2}
 	run_parallel

build/eslint-baseline-legacy.json

@@ -0,0 +1,63 @@
+{
+  "apps/files/src/components/TransferOwnershipDialogue.vue": {
+    "@nextcloud/vue/no-deprecated-props": {
+      "count": 1
+    }
+  },
+  "apps/files_sharing/src/components/SharingEntryQuickShareSelect.vue": {
+    "vue/no-mutating-props": {
+      "count": 2
+    }
+  },
+  "apps/files_sharing/src/components/SharingInput.vue": {
+    "@nextcloud/vue/no-deprecated-props": {
+      "count": 1
+    }
+  },
+  "apps/files_sharing/src/views/SharingDetailsTab.vue": {
+    "vue/no-mutating-props": {
+      "count": 23
+    }
+  },
+  "apps/files_sharing/src/views/SharingLinkList.vue": {
+    "vue/no-mutating-props": {
+      "count": 1
+    }
+  },
+  "apps/settings/src/components/Users/NewUserDialog.vue": {
+    "@nextcloud/vue/no-deprecated-props": {
+      "count": 1
+    },
+    "vue/no-mutating-props": {
+      "count": 17
+    }
+  },
+  "apps/workflowengine/src/components/Check.vue": {
+    "vue/no-mutating-props": {
+      "count": 3
+    }
+  },
+  "apps/workflowengine/src/components/Rule.vue": {
+    "vue/no-mutating-props": {
+      "count": 1
+    }
+  },
+  "core/src/OC/dialogs.js": {
+    "camelcase": {
+      "count": 33
+    },
+    "no-undef": {
+      "count": 1
+    }
+  },
+  "core/src/views/Login.vue": {
+    "vue/multi-word-component-names": {
+      "count": 1
+    }
+  },
+  "core/src/views/Setup.vue": {
+    "vue/multi-word-component-names": {
+      "count": 1
+    }
+  }
+}

build/eslint-baseline.json

@@ -1,63 +1,7 @@
 {
-  "apps/files/src/components/TransferOwnershipDialogue.vue": {
+  "apps/dav/src/components/AbsenceForm.vue": {
     "@nextcloud/vue/no-deprecated-props": {
       "count": 1
     }
-  },
-  "apps/files_sharing/src/components/SharingEntryQuickShareSelect.vue": {
-    "vue/no-mutating-props": {
-      "count": 2
-    }
-  },
-  "apps/files_sharing/src/components/SharingInput.vue": {
-    "@nextcloud/vue/no-deprecated-props": {
-      "count": 1
-    }
-  },
-  "apps/files_sharing/src/views/SharingDetailsTab.vue": {
-    "vue/no-mutating-props": {
-      "count": 23
-    }
-  },
-  "apps/files_sharing/src/views/SharingLinkList.vue": {
-    "vue/no-mutating-props": {
-      "count": 1
-    }
-  },
-  "apps/settings/src/components/Users/NewUserDialog.vue": {
-    "@nextcloud/vue/no-deprecated-props": {
-      "count": 1
-    },
-    "vue/no-mutating-props": {
-      "count": 16
-    }
-  },
-  "apps/workflowengine/src/components/Check.vue": {
-    "vue/no-mutating-props": {
-      "count": 3
-    }
-  },
-  "apps/workflowengine/src/components/Rule.vue": {
-    "vue/no-mutating-props": {
-      "count": 1
-    }
-  },
-  "core/src/OC/dialogs.js": {
-    "camelcase": {
-      "count": 33
-    },
-    "no-undef": {
-      "count": 1
-    }
-  },
-  "core/src/views/Login.vue": {
-    "vue/multi-word-component-names": {
-      "count": 1
-    }
-  },
-  "core/src/views/Setup.vue": {
-    "vue/multi-word-component-names": {
-      "count": 1
-    }
   }
 }

build/frontend-legacy/package.json

@@ -8,8 +8,8 @@
   "scripts": {
     "build": "webpack --node-env production --progress",
     "dev": "webpack --node-env development --progress",
-    "lint": "eslint --suppressions-location ../eslint-baseline.json --no-error-on-unmatched-pattern ./apps/*/ ./core/",
-    "lint:fix": "eslint --suppressions-location ../eslint-baseline.json --fix --no-error-on-unmatched-pattern ./apps/*/ ./core/",
+    "lint": "eslint --suppressions-location ../eslint-baseline-legacy.json --no-error-on-unmatched-pattern ./apps/*/ ./core/",
+    "lint:fix": "eslint --suppressions-location ../eslint-baseline-legacy.json --fix --no-error-on-unmatched-pattern ./apps/*/ ./core/",
     "test": "vitest run",
     "test:coverage": "vitest run --coverage --reporter=default",
     "test:update-snapshots": "vitest run --update",