Update eslint to v9 (#35485)

Update eslint and all plugins. Many plugins still do not ship type
definitions so I had to add stubs. Also, I had to put a few typescript
error expectations because if some unknown error in the types.

`eslint-plugin-no-jquery` is disabled because it's not compatible with
eslint 9 flat config
(https://github.com/wikimedia/eslint-plugin-no-jquery/issues/311).

web_src/js/components/RepoBranchTagSelector.vue

@@ -153,7 +153,7 @@ export default defineComponent({
       return -1;
     },
     getActiveItem() {
-      const el = this.$refs[`listItem${this.activeItemIndex}`]; // eslint-disable-line no-jquery/variable-pattern
+      const el = this.$refs[`listItem${this.activeItemIndex}`];
       // @ts-expect-error - el is unknown type
       return (el && el.length) ? el[0] : null;
     },

web_src/js/components/ViewFileTreeStore.ts

@@ -4,7 +4,7 @@ import {pathEscapeSegments} from '../utils/url.ts';
 import {createElementFromHTML} from '../utils/dom.ts';
 import {html} from '../utils/html.ts';
 
-export function createViewFileTreeStore(props: { repoLink: string, treePath: string, currentRefNameSubURL: string}) {
+export function createViewFileTreeStore(props: {repoLink: string, treePath: string, currentRefNameSubURL: string}) {
   const store = reactive({
     rootFiles: [],
     selectedItem: props.treePath,

web_src/js/features/codeeditor.ts

@@ -19,7 +19,7 @@ type EditorConfig = {
   trim_trailing_whitespace?: boolean,
   insert_final_newline?: boolean,
   root?: boolean,
-}
+};
 
 const languagesByFilename: Record<string, string> = {};
 const languagesByExt: Record<string, string> = {};

web_src/js/features/common-button.ts

@@ -112,7 +112,7 @@ function onHidePanelClick(el: HTMLElement, e: MouseEvent) {
 export type ElementWithAssignableProperties = {
   getAttribute: (name: string) => string | null;
   setAttribute: (name: string, value: string) => void;
-} & Record<string, any>
+} & Record<string, any>;
 
 export function assignElementProperty(el: ElementWithAssignableProperties, kebabName: string, val: string) {
   const camelizedName = camelize(kebabName);

web_src/js/features/comp/ConfirmModal.ts

@@ -9,7 +9,7 @@ type ConfirmModalOptions = {
   header?: string;
   content?: string;
   confirmButtonColor?: 'primary' | 'red' | 'green' | 'blue';
-}
+};
 
 export function createConfirmModal({header = '', content = '', confirmButtonColor = 'primary'}:ConfirmModalOptions = {}): HTMLElement {
   const headerHtml = header ? html`<div class="header">${header}</div>` : '';

web_src/js/features/comp/Cropper.ts

@@ -4,7 +4,7 @@ type CropperOpts = {
   container: HTMLElement,
   imageSource: HTMLImageElement,
   fileInput: HTMLInputElement,
-}
+};
 
 async function initCompCropper({container, fileInput, imageSource}: CropperOpts) {
   const {default: Cropper} = await import(/* webpackChunkName: "cropperjs" */'cropperjs');

web_src/js/features/comp/EditorMarkdown.test.ts

@@ -169,7 +169,7 @@ test('EditorMarkdown', () => {
   type ValueWithCursor = string | {
     value: string;
     pos: number;
-  }
+  };
   const testInput = (input: ValueWithCursor, result: ValueWithCursor) => {
     const intputValue = typeof input === 'string' ? input : input.value;
     const inputPos = typeof input === 'string' ? intputValue.length : input.pos;

web_src/js/features/comp/EditorMarkdown.ts

@@ -18,7 +18,7 @@ type TextareaValueSelection = {
   value: string;
   selStart: number;
   selEnd: number;
-}
+};
 
 function handleIndentSelection(textarea: HTMLTextAreaElement, e: KeyboardEvent) {
   const selStart = textarea.selectionStart;
@@ -65,14 +65,14 @@ function handleIndentSelection(textarea: HTMLTextAreaElement, e: KeyboardEvent)
 type MarkdownHandleIndentionResult = {
   handled: boolean;
   valueSelection?: TextareaValueSelection;
-}
+};
 
 type TextLinesBuffer = {
   lines: string[];
   lengthBeforePosLine: number;
   posLineIndex: number;
   inlinePos: number
-}
+};
 
 export function textareaSplitLines(value: string, pos: number): TextLinesBuffer {
   const lines = value.split('\n');

web_src/js/features/repo-issue.ts

@@ -64,7 +64,7 @@ function initRepoIssueLabelFilter(elDropdown: HTMLElement) {
     selectedLabelIds.add(`${Math.abs(parseInt(id))}`); // "labels" contains negative ids, which are excluded
   }
 
-  const excludeLabel = (e: MouseEvent|KeyboardEvent, item: Element) => {
+  const excludeLabel = (e: MouseEvent | KeyboardEvent, item: Element) => {
     e.preventDefault();
     e.stopPropagation();
     const labelId = item.getAttribute('data-label-id');

web_src/js/markup/html2markdown.ts

@@ -4,13 +4,13 @@ type Processor = (el: HTMLElement) => string | HTMLElement | void;
 
 type Processors = {
   [tagName: string]: Processor;
-}
+};
 
 type ProcessorContext = {
   elementIsFirst: boolean;
   elementIsLast: boolean;
   listNestingLevel: number;
-}
+};
 
 function prepareProcessors(ctx:ProcessorContext): Processors {
   const processors: Processors = {

web_src/js/modules/diff-file.ts

@@ -15,7 +15,7 @@ export type DiffTreeEntry = {
   Children: DiffTreeEntry[],
   FileIcon: string,
   ParentEntry?: DiffTreeEntry,
-}
+};
 
 type DiffFileTreeData = {
   TreeRoot: DiffTreeEntry,
@@ -28,7 +28,7 @@ type DiffFileTree = {
   fullNameMap?: Record<string, DiffTreeEntry>
   fileTreeIsVisible: boolean;
   selectedItem: string;
-}
+};
 
 let diffTreeStoreReactive: Reactive<DiffFileTree>;
 export function diffTreeStore() {

web_src/js/modules/fomantic/base.ts

@@ -14,5 +14,4 @@ export function linkLabelAndInput(label: Element, input: Element) {
   }
 }
 
-// eslint-disable-next-line no-jquery/variable-pattern
 export const fomanticQuery = $;

web_src/js/modules/fomantic/dropdown.ts

@@ -348,7 +348,7 @@ export function hideScopedEmptyDividers(container: Element) {
   }
 }
 
-function onResponseKeepSelectedItem(dropdown: typeof $|HTMLElement, selectedValue: string) {
+function onResponseKeepSelectedItem(dropdown: typeof $ | HTMLElement, selectedValue: string) {
   // There is a bug in fomantic dropdown when using "apiSettings" to fetch data
   // * when there is a selected item, the dropdown insists on hiding the selected one from the list:
   // * in the "filter" function: ('[data-value="'+value+'"]').addClass(className.filtered)

web_src/js/modules/toast.ts

@@ -15,7 +15,7 @@ type ToastLevels = {
     background: string,
     duration: number,
   }
-}
+};
 
 const levels: ToastLevels = {
   info: {
@@ -40,7 +40,7 @@ type ToastOpts = {
   preventDuplicates?: boolean | string,
 } & Options;
 
-type ToastifyElement = HTMLElement & {_giteaToastifyInstance?: Toast };
+type ToastifyElement = HTMLElement & {_giteaToastifyInstance?: Toast};
 
 /** See https://github.com/apvarun/toastify-js#api for options */
 function showToast(message: string, level: Intent, {gravity, position, duration, useHtmlBody, preventDuplicates = true, ...other}: ToastOpts = {}): Toast {

web_src/js/render/plugin.ts

@@ -7,4 +7,4 @@ export type FileRenderPlugin = {
 
   // render file content
   render: (container: HTMLElement, fileUrl: string, options?: any) => Promise<void>;
-}
+};

web_src/js/svg.ts

@@ -166,7 +166,7 @@ export type SvgName = keyof typeof svgs;
 //  most of the SVG icons in assets couldn't be used directly.
 
 // retrieve an HTML string for given SVG icon name, size and additional classes
-export function svg(name: SvgName, size = 16, classNames?: string|string[]): string {
+export function svg(name: SvgName, size = 16, classNames?: string | string[]): string {
   const className = Array.isArray(classNames) ? classNames.join(' ') : classNames;
   if (!(name in svgs)) throw new Error(`Unknown SVG icon: ${name}`);
   if (size === 16 && !className) return svgs[name];

web_src/js/types.ts

@@ -4,7 +4,7 @@ export type MentionValue = {
   name: string,
   fullname: string,
   avatar: string,
-}
+};
 
 export type Config = {
   appUrl: string,
@@ -20,7 +20,7 @@ export type Config = {
   mentionValues?: MentionValue[],
   mermaidMaxSourceCharacters: number,
   i18n: Record<string, string>,
-}
+};
 
 export type IntervalId = ReturnType<typeof setInterval>;
 
@@ -35,21 +35,21 @@ export type RequestOpts = {
 export type RepoOwnerPathInfo = {
   ownerName: string,
   repoName: string,
-}
+};
 
 export type IssuePathInfo = {
   ownerName: string,
   repoName: string,
   pathType: string,
   indexString?: string,
-}
+};
 
 export type IssuePageInfo = {
   repoLink: string,
   repoId: number,
   issueNumber: number,
   issueDependencySearchType: string,
-}
+};
 
 export type Issue = {
   id: number;
@@ -65,6 +65,6 @@ export type Issue = {
 export type FomanticInitFunction = {
   settings?: Record<string, any>,
   (...args: any[]): any,
-}
+};
 
 export type GitRefType = 'branch' | 'tag';

web_src/js/utils/dom.ts

@@ -7,7 +7,7 @@ type ArrayLikeIterable<T> = ArrayLike<T> & Iterable<T>; // for NodeListOf and Ar
 type ElementArg = Element | string | ArrayLikeIterable<Element> | ReturnType<typeof $>;
 type ElementsCallback<T extends Element> = (el: T) => Promisable<any>;
 type ElementsCallbackWithArgs = (el: Element, ...args: any[]) => Promisable<any>;
-export type DOMEvent<E extends Event, T extends Element = HTMLElement> = E & { target: Partial<T>; };
+export type DOMEvent<E extends Event, T extends Element = HTMLElement> = E & {target: Partial<T>;};
 
 function elementsCall(el: ElementArg, func: ElementsCallbackWithArgs, ...args: any[]): ArrayLikeIterable<Element> {
   if (typeof el === 'string' || el instanceof String) {
@@ -322,7 +322,7 @@ export function createElementFromHTML<T extends HTMLElement>(htmlString: string)
   return div.firstChild as T;
 }
 
-export function createElementFromAttrs(tagName: string, attrs: Record<string, any>, ...children: (Node|string)[]): HTMLElement {
+export function createElementFromAttrs(tagName: string, attrs: Record<string, any>, ...children: (Node | string)[]): HTMLElement {
   const el = document.createElement(tagName);
   for (const [key, value] of Object.entries(attrs || {})) {
     if (value === undefined || value === null) continue;

web_src/js/utils/glob.test.ts

@@ -117,7 +117,7 @@ test('GlobCompiler', async () => {
   for (const c of golangCases) {
     const compiled = globCompile(c.pattern, c.separators);
     const msg = `pattern: ${c.pattern}, input: ${c.input}, separators: ${c.separators || '(none)'}, compiled: ${compiled.regexpPattern}`;
-    // eslint-disable-next-line @vitest/valid-expect -- Unlike Jest, Vitest supports a message as the second argument
+    // eslint-disable-next-line vitest/valid-expect -- Unlike Jest, Vitest supports a message as the second argument
     expect(compiled.regexp.test(c.input), msg).toBe(c.matched);
   }
 

web_src/js/utils/html.ts

@@ -23,7 +23,7 @@ export function html(tmpl: TemplateStringsArray, ...parts: Array<any>): string {
   return output;
 }
 
-export function htmlRaw(s: string|TemplateStringsArray, ...tmplParts: Array<any>): rawObject {
+export function htmlRaw(s: string | TemplateStringsArray, ...tmplParts: Array<any>): rawObject {
   if (typeof s === 'string') {
     if (tmplParts.length !== 0) throw new Error("either htmlRaw('str') or htmlRaw`tmpl`");
     return new rawObject(s);

web_src/js/utils/image.ts

@@ -1,7 +1,7 @@
 type PngChunk = {
   name: string,
   data: Uint8Array,
-}
+};
 
 export async function pngChunks(blob: Blob): Promise<PngChunk[]> {
   const uint8arr = new Uint8Array(await blob.arrayBuffer());
@@ -27,7 +27,7 @@ export async function pngChunks(blob: Blob): Promise<PngChunk[]> {
 type ImageInfo = {
   width?: number,
   dppx?: number,
-}
+};
 
 /** decode a image and try to obtain width and dppx. It will never throw but instead
  *  return default values. */

web_src/js/utils/time.ts

@@ -47,11 +47,11 @@ export type DayData = {
   additions: number,
   deletions: number,
   commits: number,
-}
+};
 
 export type DayDataObject = {
   [timestamp: string]: DayData,
-}
+};
 
 export function fillEmptyStartDaysWithZeroes(startDays: number[], data: DayDataObject): DayData[] {
   const result: Record<string, any> = {};