fix: prepr

apps/frontend/src/components/ui/moderation/moderation-checklist/index.vue

@@ -284,9 +284,9 @@
 									<component
 										:is="opt.icon"
 										v-if="opt.icon"
-									class="mr-2"
+										class="mr-2"
-									:class="{ 'opacity-50': opt.visited }"
+										:class="{ 'opacity-50': opt.visited }"
-									aria-hidden="true"
+										aria-hidden="true"
 									/>
 									<span :class="{ 'opacity-50': opt.visited }">
 										{{ opt.text
@@ -1343,9 +1343,7 @@ async function generateMessage() {
 
 const hasNextProject = ref(false)
 
-const finishedId = import.meta.client
+const finishedId = import.meta.client ? localStorage.getItem('moderation-checklist-finished') : null
-	? localStorage.getItem('moderation-checklist-finished')
-	: null
 if (finishedId === projectV2.value.id) {
 	localStorage.removeItem('moderation-checklist-finished')
 	hasNextProject.value = moderationQueue.queueLength > 0

apps/frontend/src/components/ui/moderation/moderation-checklist/node-renderer/index.vue

@@ -70,6 +70,7 @@ const {
 					</div>
 
 					<template v-if="hasChildrenCap(item) && !hasValueCap(item)">
+						<!-- eslint-disable-next-line vue/no-undef-components -- recursive component named via defineOptions -->
 						<NodeRenderer
 							:nodes="resolveChildren(item, containerScope(item).state)"
 							:state="containerScope(item).state"
@@ -90,7 +91,9 @@ const {
 							:[modelProp(item)]="
 								getEffectiveValue(item as RenderableValueNode, state[item.id], wrappedState)
 							"
-							@[updateEvent(item)]="(value: unknown) => updateValue(item as RenderableValueNode, value)"
+							@[updateEvent(item)]="
+								(value: unknown) => updateValue(item as RenderableValueNode, value)
+							"
 						/>
 						<template
 							v-for="(tweak, tweakIndex) in (item as RenderableValueNode)._tweaks ?? []"
@@ -132,6 +135,7 @@ const {
 		</template>
 
 		<template v-for="(item, idx) in nodes" :key="`children-${nodeKey(item, idx)}`">
+			<!-- eslint-disable-next-line vue/no-undef-components -- recursive component named via defineOptions -->
 			<NodeRenderer
 				v-if="
 					typeof item === 'object' &&

apps/frontend/src/components/ui/moderation/moderation-checklist/node-renderer/renderers.ts

@@ -10,10 +10,7 @@ import type { RenderableValueNode, RendererPropsContext } from './types'
 
 interface RendererDefinition {
 	component: Component
-	props?: (
+	props?: (node: RenderableValueNode, context: RendererPropsContext) => Record<string, unknown>
-		node: RenderableValueNode,
-		context: RendererPropsContext,
-	) => Record<string, unknown>
 }
 
 const builtinRenderers = {

apps/frontend/src/components/ui/moderation/moderation-checklist/node-renderer/use-node-renderer.ts

@@ -25,8 +25,8 @@ import {
 	originScope,
 	resolve,
 	resolveChildren,
-	writeNodeValue,
 	withStateDefaults,
+	writeNodeValue,
 } from '@modrinth/moderation/src/types/node'
 import type { Component } from 'vue'
 import { computed, watchEffect } from 'vue'
@@ -122,9 +122,7 @@ export function useNodeRenderer(props: NodeRendererProps) {
 			? {
 					class: '!w-auto max-w-full',
 					style: {
-						minWidth: getDropdownMinWidth(
+						minWidth: getDropdownMinWidth(node._options as unknown as Array<{ label: string }>),
-							node._options as unknown as Array<{ label: string }>,
-						),
 					},
 				}
 			: undefined
@@ -211,9 +209,7 @@ export function useNodeRenderer(props: NodeRendererProps) {
 	}
 
 	function nodeKey(item: ChildNode, index: number): string {
-		return typeof item === 'object' && item !== null && hasIdCap(item)
+		return typeof item === 'object' && item !== null && hasIdCap(item) ? item.id : `n-${index}`
-			? item.id
-			: `n-${index}`
 	}
 
 	function modelProp(item: object): string {
@@ -229,7 +225,9 @@ export function useNodeRenderer(props: NodeRendererProps) {
 			? (item._onChange as OnChangeFn | undefined)
 			: undefined
 		if (onChange) {
-			const result = onChange(value as string, { override: (override) => ({ __override: override }) })
+			const result = onChange(value as string, {
+				override: (override) => ({ __override: override }),
+			})
 			if (result && typeof result === 'object' && '__override' in result) {
 				writeNodeValue(
 					item,

apps/frontend/src/components/ui/moderation/moderation-checklist/use-lock.ts

@@ -1,10 +1,7 @@
 import type { AbstractWebNotificationManager } from '@modrinth/ui'
 import { ref } from 'vue'
 
-import type {
+import type { LockAcquireResponse, ModerationQueueService } from '~/services/moderation/queue.ts'
-	LockAcquireResponse,
-	ModerationQueueService,
-} from '~/services/moderation/queue.ts'
 
 interface ChecklistLockOptions {
 	projectId: string
@@ -110,10 +107,13 @@ export function useChecklistLock({
 
 	function startHeartbeat() {
 		if (heartbeat) clearInterval(heartbeat)
-		heartbeat = setInterval(async () => {
+		heartbeat = setInterval(
-			const result = await queue.refreshLock()
+			async () => {
-			if (!result.success) handleLost(result)
+				const result = await queue.refreshLock()
-		}, 5 * 60 * 1000)
+				if (!result.success) handleLost(result)
+			},
+			5 * 60 * 1000,
+		)
 	}
 
 	function handleAcquired() {

apps/frontend/src/components/ui/moderation/moderation-checklist/use-node-renderer-state.ts

@@ -1,5 +1,6 @@
 import type { Labrinth } from '@modrinth/api-client'
 import { expandVariables } from '@modrinth/moderation'
+import type { FixBuilder, NodeState, StageNode, Writer } from '@modrinth/moderation/src/types/node'
 import {
 	collectMessageNodes,
 	computeAttentionMap,
@@ -7,12 +8,6 @@ import {
 	evalActiveAction,
 	resolveChildren,
 } from '@modrinth/moderation/src/types/node'
-import type {
-	FixBuilder,
-	NodeState,
-	StageNode,
-	Writer,
-} from '@modrinth/moderation/src/types/node'
 import { renderHighlightedString } from '@modrinth/utils'
 import type { ComputedRef, Ref } from 'vue'
 import { computed, ref, watchEffect } from 'vue'

apps/frontend/src/components/ui/moderation/moderation-checklist/use-persistence.ts

@@ -16,9 +16,7 @@ export async function loadChecklistPersistence(projectId: string) {
 	const persistedState = import.meta.client ? await loadChecklistState(projectId) : null
 	const activatedStages = ref<Set<string>>(new Set(persistedState?.activatedStages ?? []))
 	const visitedStages = ref<Set<string>>(
-		new Set(
+		new Set(import.meta.client ? (getSessionChecklistState(projectId).visitedStages ?? []) : []),
-			import.meta.client ? (getSessionChecklistState(projectId).visitedStages ?? []) : [],
-		),
 	)
 	const reviewedAnyway = ref(persistedState?.reviewAnyway ?? false)
 	const message = ref<string | null>(persistedState?.message ?? null)

apps/frontend/src/components/ui/moderation/moderation-checklist/use-submission.ts

@@ -20,9 +20,7 @@ interface ModerationSubmission {
 }
 
 function getFixes(node: object): FixBuilder[] {
-	return hasCap(node, '_fixes') && Array.isArray(node._fixes)
+	return hasCap(node, '_fixes') && Array.isArray(node._fixes) ? (node._fixes as FixBuilder[]) : []
-		? (node._fixes as FixBuilder[])
-		: []
 }
 
 function shouldApplyFixes(actions: ActiveAction[]): boolean {

packages/moderation/src/types/node/capabilities.ts

@@ -259,17 +259,9 @@ export interface NodePropsContext {
 	toggleSetValue?: (value: string) => void
 }
 
-export type BuiltinRendererKey =
+export type BuiltinRendererKey = 'action' | 'checkbox' | 'toggle' | 'dropdown' | 'text' | 'markdown'
-	| 'action'
-	| 'checkbox'
-	| 'toggle'
-	| 'dropdown'
-	| 'text'
-	| 'markdown'
 
-export type NodeRendererDescriptor =
+export type NodeRendererDescriptor = { type: BuiltinRendererKey } | { type: 'custom'; key: string }
-	| { type: BuiltinRendererKey }
-	| { type: 'custom'; key: string }
 
 export interface Renderable {
 	_renderer: NodeRendererDescriptor

packages/moderation/src/types/node/factories.ts

@@ -12,8 +12,8 @@ import {
 	withNoneLabel,
 	withOnClick,
 	withPriority,
-	withRequired,
 	withRenderer,
+	withRequired,
 	withSelectable,
 	withShown,
 	withStageMeta,