chore: fix ESLint issues for rules added in ESLint v10

susnux · susnux · commit 6db75dca923e · 2026-04-20T22:13:24.000+02:00
Signed-off-by: Ferdinand Thiessen &lt;opensource@fthiessen.de&gt;
diff --git a/apps/dav/src/components/AbsenceForm.vue b/apps/dav/src/components/AbsenceForm.vue
@@ -149,7 +149,7 @@ export default {
 				ShareType.User,
 			]
 
-			let request = null
+			let request
 			try {
 				request = await axios.get(generateOcsUrl('apps/files_sharing/api/v1/sharees'), {
 					params: {
diff --git a/apps/files/src/store/renaming.ts b/apps/files/src/store/renaming.ts
@@ -126,7 +126,7 @@ export const useRenamingStore = defineStore('renaming', () => {
 			if (isAxiosError(error)) {
 				// TODO: 409 means current folder does not exist, redirect ?
 				if (error?.response?.status === 404) {
-					throw new Error(t('files', 'Could not rename "{oldName}", it does not exist any more', { oldName }))
+					throw new Error(t('files', 'Could not rename "{oldName}", it does not exist any more', { oldName }), { cause: error })
 				} else if (error?.response?.status === 412) {
 					throw new Error(t(
 						'files',
@@ -135,11 +135,11 @@ export const useRenamingStore = defineStore('renaming', () => {
 							newName,
 							dir: basename(renamingNode.value!.dirname),
 						},
-					))
+					), { cause: error })
 				}
 			}
 			// Unknown error
-			throw new Error(t('files', 'Could not rename "{oldName}"', { oldName }))
+			throw new Error(t('files', 'Could not rename "{oldName}"', { oldName }), { cause: error })
 		} finally {
 			Vue.set(node, 'status', undefined)
 			isRenaming.value = false
diff --git a/apps/files_sharing/src/components/SharingInput.vue b/apps/files_sharing/src/components/SharingInput.vue
@@ -231,7 +231,7 @@ export default {
 				shareType.push(...remoteTypes)
 			}
 
-			let request = null
+			let request
 			try {
 				request = await axios.get(generateOcsUrl('apps/files_sharing/api/v1/sharees'), {
 					params: {
@@ -321,7 +321,7 @@ export default {
 		async getRecommendations() {
 			this.loading = true
 
-			let request = null
+			let request
 			try {
 				request = await axios.get(generateOcsUrl('apps/files_sharing/api/v1/sharees_recommended'), {
 					params: {
diff --git a/apps/files_sharing/src/mixins/ShareDetails.js b/apps/files_sharing/src/mixins/ShareDetails.js
@@ -11,7 +11,7 @@ import logger from '../services/logger.ts'
 export default {
 	methods: {
 		async openSharingDetails(shareRequestObject) {
-			let share = {}
+			let share
 			// handle externalResults from OCA.Sharing.ShareSearch
 			// TODO : Better name/interface for handler required
 			// For example `externalAppCreateShareHook` with proper documentation
diff --git a/apps/files_versions/src/utils/versions.ts b/apps/files_versions/src/utils/versions.ts
@@ -98,11 +98,11 @@ export async function restoreVersion(version: Version) {
  */
 function formatVersion(version: Required<FileStat>, node: INode): Version {
 	const mtime = Date.parse(version.lastmod)
-	let previewUrl = ''
 
+	let previewUrl: string
 	if (mtime === node.mtime?.getTime()) { // Version is the current one
 		previewUrl = generateUrl('/core/preview?fileId={fileId}&c={fileEtag}&x=250&y=250&forceIcon=0&a=0&forceIcon=1&mimeFallback=1', {
-			fileId: node.fileid,
+			fileId: node.id,
 			fileEtag: node.attributes.etag,
 		})
 	} else {
@@ -113,7 +113,7 @@ function formatVersion(version: Required<FileStat>, node: INode): Version {
 	}
 
 	return {
-		fileId: node.fileid!.toString(),
+		fileId: node.id!,
 		// If version-label is defined make sure it is a string (prevent issue if the label is a number an PHP returns a number then)
 		label: version.props['version-label'] ? String(version.props['version-label']) : '',
 		author: version.props['version-author'] ? String(version.props['version-author']) : null,
diff --git a/apps/settings/src/components/WebAuthn/WebAuthnAddDevice.vue b/apps/settings/src/components/WebAuthn/WebAuthnAddDevice.vue
@@ -170,7 +170,7 @@ export default {
 				this.$emit('added', device)
 			} catch (err) {
 				logger.error('Error persisting webauthn registration', { error: err })
-				throw new Error(t('settings', 'Server error while trying to complete WebAuthn device registration'))
+				throw new Error(t('settings', 'Server error while trying to complete WebAuthn device registration'), { cause: err })
 			}
 		},
 
diff --git a/apps/settings/src/service/WebAuthnRegistrationSerice.ts b/apps/settings/src/service/WebAuthnRegistrationSerice.ts
@@ -28,11 +28,11 @@ export async function startRegistration() {
 	} catch (e) {
 		logger.error(e as Error)
 		if (isAxiosError(e)) {
-			throw new Error(t('settings', 'Could not register device: Network error'))
+			throw new Error(t('settings', 'Could not register device: Network error'), { cause: e })
 		} else if ((e as Error).name === 'InvalidStateError') {
-			throw new Error(t('settings', 'Could not register device: Probably already registered'))
+			throw new Error(t('settings', 'Could not register device: Probably already registered'), { cause: e })
 		}
-		throw new Error(t('settings', 'Could not register device'))
+		throw new Error(t('settings', 'Could not register device'), { cause: e })
 	}
 }
 
diff --git a/apps/systemtags/src/services/api.ts b/apps/systemtags/src/services/api.ts
@@ -43,7 +43,7 @@ export async function fetchTags(): Promise<TagWithId[]> {
 		return parseTags(tags)
 	} catch (error) {
 		logger.error(t('systemtags', 'Failed to load tags'), { error })
-		throw new Error(t('systemtags', 'Failed to load tags'))
+		throw new Error(t('systemtags', 'Failed to load tags'), { cause: error })
 	}
 }
 
@@ -62,7 +62,7 @@ export async function fetchTag(tagId: number): Promise<TagWithId> {
 		return parseTags([tag])[0]!
 	} catch (error) {
 		logger.error(t('systemtags', 'Failed to load tag'), { error })
-		throw new Error(t('systemtags', 'Failed to load tag'))
+		throw new Error(t('systemtags', 'Failed to load tag'), { cause: error })
 	}
 }
 
@@ -76,7 +76,7 @@ export async function fetchLastUsedTagIds(): Promise<number[]> {
 		return lastUsedTagIds.map(Number)
 	} catch (error) {
 		logger.error(t('systemtags', 'Failed to load last used tags'), { error })
-		throw new Error(t('systemtags', 'Failed to load last used tags'))
+		throw new Error(t('systemtags', 'Failed to load last used tags'), { cause: error })
 	}
 }
 
@@ -103,10 +103,10 @@ export async function createTag(tag: Tag | ServerTag): Promise<number> {
 	} catch (error) {
 		if ((error as WebDAVClientError)?.response?.status === 409) {
 			logger.error(t('systemtags', 'A tag with the same name already exists'), { error })
-			throw new Error(t('systemtags', 'A tag with the same name already exists'))
+			throw new Error(t('systemtags', 'A tag with the same name already exists'), { cause: error })
 		}
 		logger.error(t('systemtags', 'Failed to create tag'), { error })
-		throw new Error(t('systemtags', 'Failed to create tag'))
+		throw new Error(t('systemtags', 'Failed to create tag'), { cause: error })
 	}
 }
 
@@ -137,7 +137,7 @@ export async function updateTag(tag: TagWithId): Promise<void> {
 		emit('systemtags:tag:updated', tag)
 	} catch (error) {
 		logger.error(t('systemtags', 'Failed to update tag'), { error })
-		throw new Error(t('systemtags', 'Failed to update tag'))
+		throw new Error(t('systemtags', 'Failed to update tag'), { cause: error })
 	}
 }
 
@@ -153,7 +153,7 @@ export async function deleteTag(tag: TagWithId): Promise<void> {
 		emit('systemtags:tag:deleted', tag)
 	} catch (error) {
 		logger.error(t('systemtags', 'Failed to delete tag'), { error })
-		throw new Error(t('systemtags', 'Failed to delete tag'))
+		throw new Error(t('systemtags', 'Failed to delete tag'), { cause: error })
 	}
 }
 
diff --git a/apps/systemtags/src/services/files.ts b/apps/systemtags/src/services/files.ts
@@ -28,7 +28,7 @@ export async function fetchTagsForFile(fileId: number): Promise<TagWithId[]> {
 		return parseTags(tags)
 	} catch (error) {
 		logger.error(t('systemtags', 'Failed to load tags for file'), { error })
-		throw new Error(t('systemtags', 'Failed to load tags for file'))
+		throw new Error(t('systemtags', 'Failed to load tags for file'), { cause: error })
 	}
 }
 
@@ -66,7 +66,7 @@ export async function setTagForFile(tag: TagWithId | ServerTagWithId, fileId: nu
 		})
 	} catch (error) {
 		logger.error(t('systemtags', 'Failed to set tag for file'), { error })
-		throw new Error(t('systemtags', 'Failed to set tag for file'))
+		throw new Error(t('systemtags', 'Failed to set tag for file'), { cause: error })
 	}
 }
 
@@ -82,6 +82,6 @@ export async function deleteTagForFile(tag: TagWithId, fileId: number): Promise<
 		await davClient.deleteFile(path)
 	} catch (error) {
 		logger.error(t('systemtags', 'Failed to delete tag for file'), { error })
-		throw new Error(t('systemtags', 'Failed to delete tag for file'))
+		throw new Error(t('systemtags', 'Failed to delete tag for file'), { cause: error })
 	}
 }
diff --git a/core/src/OC/util.js b/core/src/OC/util.js
@@ -64,8 +64,6 @@ export default {
 		}
 
 		const s = string.toLowerCase().trim()
-		let bytes = null
-
 		const bytesArray = {
 			b: 1,
 			k: 1024,
@@ -80,6 +78,7 @@ export default {
 			p: 1024 * 1024 * 1024 * 1024 * 1024,
 		}
 
+		let bytes
 		const matches = s.match(/^[\s+]?([0-9]*)(\.([0-9]+))?( +)?([kmgtp]?b?)$/i)
 		if (matches !== null) {
 			bytes = parseFloat(s)
diff --git a/cypress/support/commands.ts b/cypress/support/commands.ts
@@ -88,9 +88,9 @@ Cypress.Commands.add('setFileAsFavorite', (user: User, target: string, favorite
 					</d:propertyupdate>`,
 				})
 				cy.log(`Created directory ${target}`, response)
-			} catch (error) {
-				cy.log('error', error)
-				throw new Error('Unable to process fixture')
+			} catch (cause) {
+				cy.log('error', cause)
+				throw new Error('Unable to process fixture', { cause })
 			}
 		})
 })
@@ -111,9 +111,9 @@ Cypress.Commands.add('mkdir', (user: User, target: string) => {
 				})
 				cy.log(`Created directory ${target}`, response)
 				return response
-			} catch (error) {
-				cy.log('error', error)
-				throw new Error('Unable to create directory')
+			} catch (cause) {
+				cy.log('error', cause)
+				throw new Error('Unable to create directory', { cause })
 			}
 		})
 })
@@ -133,9 +133,9 @@ Cypress.Commands.add('rm', (user: User, target: string) => {
 					},
 				})
 				cy.log(`delete file or directory ${target}`, response)
-			} catch (error) {
-				cy.log('error', error)
-				throw new Error('Unable to delete file or directory')
+			} catch (cause) {
+				cy.log('error', cause)
+				throw new Error('Unable to delete file or directory', { cause })
 			}
 		})
 })
@@ -174,9 +174,9 @@ Cypress.Commands.add('uploadContent', (user: User, blob: Blob, mimeType: string,
 			})
 			cy.log(`Uploaded content as ${fileName}`, response)
 			return response
-		} catch (error) {
-			cy.log('error', error)
-			throw new Error('Unable to process fixture')
+		} catch (cause) {
+			cy.log('error', cause)
+			throw new Error('Unable to process fixture', { cause })
 		}
 	})
 })

Original file line number	Diff line number	Diff line change
`@@ -149,7 +149,7 @@ export default {`
`149`	`149`	`ShareType.User,`
`150`	`150`	`]`
`151`	`151`
`152`		`- let request = null`
	`152`	`+ let request`
`153`	`153`	`try {`
`154`	`154`	`request = await axios.get(generateOcsUrl('apps/files_sharing/api/v1/sharees'), {`
`155`	`155`	`params: {`
Original file line number	Diff line number	Diff line change
`@@ -170,7 +170,7 @@ export default {`
`170`	`170`	`this.$emit('added', device)`
`171`	`171`	`} catch (err) {`
`172`	`172`	`logger.error('Error persisting webauthn registration', { error: err })`
`173`		`- throw new Error(t('settings', 'Server error while trying to complete WebAuthn device registration'))`
	`173`	`+ throw new Error(t('settings', 'Server error while trying to complete WebAuthn device registration'), { cause: err })`
`174`	`174`	`}`
`175`	`175`	`},`
`176`	`176`
Original file line number	Diff line number	Diff line change
`@@ -28,11 +28,11 @@ export async function startRegistration() {`
`28`	`28`	`} catch (e) {`
`29`	`29`	`logger.error(e as Error)`
`30`	`30`	`if (isAxiosError(e)) {`
`31`		`- throw new Error(t('settings', 'Could not register device: Network error'))`
	`31`	`+ throw new Error(t('settings', 'Could not register device: Network error'), { cause: e })`
`32`	`32`	`} else if ((e as Error).name === 'InvalidStateError') {`
`33`		`- throw new Error(t('settings', 'Could not register device: Probably already registered'))`
	`33`	`+ throw new Error(t('settings', 'Could not register device: Probably already registered'), { cause: e })`
`34`	`34`	`}`
`35`		`- throw new Error(t('settings', 'Could not register device'))`
	`35`	`+ throw new Error(t('settings', 'Could not register device'), { cause: e })`
`36`	`36`	`}`
`37`	`37`	`}`
`38`	`38`
Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ export async function fetchTags(): Promise<TagWithId[]> {`
`43`	`43`	`return parseTags(tags)`
`44`	`44`	`} catch (error) {`
`45`	`45`	`logger.error(t('systemtags', 'Failed to load tags'), { error })`
`46`		`- throw new Error(t('systemtags', 'Failed to load tags'))`
	`46`	`+ throw new Error(t('systemtags', 'Failed to load tags'), { cause: error })`
`47`	`47`	`}`
`48`	`48`	`}`
`49`	`49`
`@@ -62,7 +62,7 @@ export async function fetchTag(tagId: number): Promise<TagWithId> {`
`62`	`62`	`return parseTags([tag])[0]!`
`63`	`63`	`} catch (error) {`
`64`	`64`	`logger.error(t('systemtags', 'Failed to load tag'), { error })`
`65`		`- throw new Error(t('systemtags', 'Failed to load tag'))`
	`65`	`+ throw new Error(t('systemtags', 'Failed to load tag'), { cause: error })`
`66`	`66`	`}`
`67`	`67`	`}`
`68`	`68`
`@@ -76,7 +76,7 @@ export async function fetchLastUsedTagIds(): Promise<number[]> {`
`76`	`76`	`return lastUsedTagIds.map(Number)`
`77`	`77`	`} catch (error) {`
`78`	`78`	`logger.error(t('systemtags', 'Failed to load last used tags'), { error })`
`79`		`- throw new Error(t('systemtags', 'Failed to load last used tags'))`
	`79`	`+ throw new Error(t('systemtags', 'Failed to load last used tags'), { cause: error })`
`80`	`80`	`}`
`81`	`81`	`}`
`82`	`82`
`@@ -103,10 +103,10 @@ export async function createTag(tag: Tag \| ServerTag): Promise<number> {`
`103`	`103`	`} catch (error) {`
`104`	`104`	`if ((error as WebDAVClientError)?.response?.status === 409) {`
`105`	`105`	`logger.error(t('systemtags', 'A tag with the same name already exists'), { error })`
`106`		`- throw new Error(t('systemtags', 'A tag with the same name already exists'))`
	`106`	`+ throw new Error(t('systemtags', 'A tag with the same name already exists'), { cause: error })`
`107`	`107`	`}`
`108`	`108`	`logger.error(t('systemtags', 'Failed to create tag'), { error })`
`109`		`- throw new Error(t('systemtags', 'Failed to create tag'))`
	`109`	`+ throw new Error(t('systemtags', 'Failed to create tag'), { cause: error })`
`110`	`110`	`}`
`111`	`111`	`}`
`112`	`112`
`@@ -137,7 +137,7 @@ export async function updateTag(tag: TagWithId): Promise<void> {`
`137`	`137`	`emit('systemtags:tag:updated', tag)`
`138`	`138`	`} catch (error) {`
`139`	`139`	`logger.error(t('systemtags', 'Failed to update tag'), { error })`
`140`		`- throw new Error(t('systemtags', 'Failed to update tag'))`
	`140`	`+ throw new Error(t('systemtags', 'Failed to update tag'), { cause: error })`
`141`	`141`	`}`
`142`	`142`	`}`
`143`	`143`
`@@ -153,7 +153,7 @@ export async function deleteTag(tag: TagWithId): Promise<void> {`
`153`	`153`	`emit('systemtags:tag:deleted', tag)`
`154`	`154`	`} catch (error) {`
`155`	`155`	`logger.error(t('systemtags', 'Failed to delete tag'), { error })`
`156`		`- throw new Error(t('systemtags', 'Failed to delete tag'))`
	`156`	`+ throw new Error(t('systemtags', 'Failed to delete tag'), { cause: error })`
`157`	`157`	`}`
`158`	`158`	`}`
`159`	`159`
Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,7 @@ export async function fetchTagsForFile(fileId: number): Promise<TagWithId[]> {`
`28`	`28`	`return parseTags(tags)`
`29`	`29`	`} catch (error) {`
`30`	`30`	`logger.error(t('systemtags', 'Failed to load tags for file'), { error })`
`31`		`- throw new Error(t('systemtags', 'Failed to load tags for file'))`
	`31`	`+ throw new Error(t('systemtags', 'Failed to load tags for file'), { cause: error })`
`32`	`32`	`}`
`33`	`33`	`}`
`34`	`34`
`@@ -66,7 +66,7 @@ export async function setTagForFile(tag: TagWithId \| ServerTagWithId, fileId: nu`
`66`	`66`	`})`
`67`	`67`	`} catch (error) {`
`68`	`68`	`logger.error(t('systemtags', 'Failed to set tag for file'), { error })`
`69`		`- throw new Error(t('systemtags', 'Failed to set tag for file'))`
	`69`	`+ throw new Error(t('systemtags', 'Failed to set tag for file'), { cause: error })`
`70`	`70`	`}`
`71`	`71`	`}`
`72`	`72`
`@@ -82,6 +82,6 @@ export async function deleteTagForFile(tag: TagWithId, fileId: number): Promise<`
`82`	`82`	`await davClient.deleteFile(path)`
`83`	`83`	`} catch (error) {`
`84`	`84`	`logger.error(t('systemtags', 'Failed to delete tag for file'), { error })`
`85`		`- throw new Error(t('systemtags', 'Failed to delete tag for file'))`
	`85`	`+ throw new Error(t('systemtags', 'Failed to delete tag for file'), { cause: error })`
`86`	`86`	`}`
`87`	`87`	`}`
Original file line number	Diff line number	Diff line change
`@@ -64,8 +64,6 @@ export default {`
`64`	`64`	`}`
`65`	`65`
`66`	`66`	`const s = string.toLowerCase().trim()`
`67`		`- let bytes = null`
`68`		`-`
`69`	`67`	`const bytesArray = {`
`70`	`68`	`b: 1,`
`71`	`69`	`k: 1024,`
`@@ -80,6 +78,7 @@ export default {`
`80`	`78`	`p: 1024 * 1024 * 1024 * 1024 * 1024,`
`81`	`79`	`}`
`82`	`80`
	`81`	`+ let bytes`
`83`	`82`	`const matches = s.match(/^[\s+]?([0-9]*)(\.([0-9]+))?( +)?([kmgtp]?b?)$/i)`
`84`	`83`	`if (matches !== null) {`
`85`	`84`	`bytes = parseFloat(s)`