fix: more qa issues

Author: IMB11

apps/frontend/src/components/ui/moderation/ModerationTechRevCard.vue

@@ -132,12 +132,12 @@ function backToFileList() {
 	selectedFile.value = null
 }
 
-async function copyToClipboard(code: string) {
+async function copyToClipboard(code: string, detailId: string) {
 	try {
 		await navigator.clipboard.writeText(code)
-		showCopyFeedback.value = true
+		showCopyFeedback.value.set(detailId, true)
 		setTimeout(() => {
-			showCopyFeedback.value = false
+			showCopyFeedback.value.delete(detailId)
 		}, 2000)
 	} catch (error) {
 		console.error('Failed to copy code:', error)
@@ -157,7 +157,7 @@ async function updateIssueStatus(
 }
 
 const expandedIssues = ref<Set<string>>(new Set())
-const showCopyFeedback = ref(false)
+const showCopyFeedback = ref<Map<string, boolean>>(new Map())
 
 function toggleIssue(issueId: string) {
 	if (expandedIssues.value.has(issueId)) {
@@ -382,14 +382,13 @@ function toggleIssue(issueId: string) {
 					class="border-x border-b border-t-0 border-solid border-surface-3 bg-surface-2"
 					:class="{ 'rounded-bl-2xl rounded-br-2xl': idx === selectedFile.issues.length - 1 }"
 				>
-					<div class="flex items-center justify-between p-4">
+					<div
+						class="flex cursor-pointer items-center justify-between p-4 transition-colors hover:bg-surface-3"
+						@click="toggleIssue(issue.id)"
+					>
 						<div class="my-auto flex items-center gap-2">
 							<ButtonStyled type="transparent" circular>
-								<button
-									class="transition-transform"
-									:class="{ 'rotate-180': !expandedIssues.has(issue.id) }"
-									@click="toggleIssue(issue.id)"
-								>
+								<button class="transition-transform" :class="{ 'rotate-180': expandedIssues.has(issue.id) }">
 									<ChevronDownIcon class="h-5 w-5 text-contrast" />
 								</button>
 							</ButtonStyled>
@@ -417,7 +416,7 @@ function toggleIssue(issueId: string) {
 							</div>
 						</div>
 
-						<div class="flex items-center gap-2">
+						<div class="flex items-center gap-2" @click.stop>
 							<ButtonStyled color="brand" type="outlined">
 								<button class="!border-[1px]" @click="updateIssueStatus(issue.id, 'safe')">
 									Safe
@@ -448,9 +447,9 @@ function toggleIssue(issueId: string) {
 									<button
 										v-tooltip="`Copy code`"
 										class="absolute right-2 top-2 border-[1px]"
-										@click="copyToClipboard(detail.decompiled_source)"
+										@click="copyToClipboard(detail.decompiled_source, `${issue.id}-${detailIdx}`)"
 									>
-										<CopyIcon v-if="!showCopyFeedback" />
+										<CopyIcon v-if="!showCopyFeedback.get(`${issue.id}-${detailIdx}`)" />
 										<CheckIcon v-else />
 									</button>
 								</ButtonStyled>

apps/frontend/src/pages/moderation/technical-review.vue

@@ -327,7 +327,7 @@ watch(currentSortType, () => {
 				<SearchIcon aria-hidden="true" class="text-lg" />
 				<input
 					v-model="query"
-					class="h-[40px]"
+					class="h-4"
 					autocomplete="off"
 					spellcheck="false"
 					type="text"

packages/api-client/src/modules/labrinth/types.ts

@@ -1,4 +1,4 @@
-import type { ISO3166 } from '../iso3166/types'
+import type { ISO3166 } from '..'
 
 export namespace Labrinth {
 	export namespace Billing {
@@ -13,7 +13,7 @@ export namespace Labrinth {
 				price_id: string
 				interval: PriceDuration
 				status: SubscriptionStatus
-				created: string // ISO datetime string
+				created: string
 				metadata?: SubscriptionMetadata
 			}
 
@@ -40,8 +40,8 @@ export namespace Labrinth {
 				amount: number
 				currency_code: string
 				status: ChargeStatus
-				due: string // ISO datetime string
-				last_attempt: string | null // ISO datetime string
+				due: string
+				last_attempt: string | null
 				type: ChargeType
 				subscription_id: string | null
 				subscription_interval: PriceDuration | null
@@ -339,7 +339,7 @@ export namespace Labrinth {
 
 				/**
 				 * @deprecated Not recommended to use.
-				 */
+				 **/
 				[key: string]: unknown
 			}
 
@@ -383,7 +383,7 @@ export namespace Labrinth {
 			export interface GameVersion {
 				version: string
 				version_type: string
-				date: string // RFC 3339 DateTime
+				date: string
 				major: boolean
 			}
 
@@ -475,66 +475,40 @@ export namespace Labrinth {
 			}
 
 			export type SearchResponse = {
-				/** Flat list of all file reports */
 				reports: FileReport[]
-				/** Project data lookup by project ID */
 				projects: Record<string, Projects.v3.Project>
-				/** Thread data lookup by thread ID */
 				threads: Record<string, DBThread>
-				/** Ownership data lookup by project ID */
 				ownership: Record<string, Ownership>
 			}
 
 			export type FileReport = {
-				/** Unique ID of this report */
 				id: string
-				/** ID of the scanned file */
 				file_id: string
-				/** ID of the version this file belongs to */
 				version_id: string
-				/** ID of the project this file belongs to */
 				project_id: string
-				/** When this report was created (ISO datetime string) */
 				created: string
-				/** Why this file was flagged */
 				flag_reason: FlagReason
-				/** Overall severity of issues in this file */
 				severity: DelphiSeverity
-				/** Name of the flagged file */
 				file_name: string
-				/** Size of the file in bytes */
 				file_size: number
-				/** List of issues found in this file */
 				issues: FileIssue[]
 			}
 
 			export type FileIssue = {
-				/** Unique ID of this issue */
 				id: string
-				/** ID of the report this issue belongs to */
 				report_id: string
-				/** Type of issue (e.g., 'OBFUSCATED_NAMES', 'SUSPICIOUS_CODE') */
 				issue_type: string
-				/** Whether this issue is valid, false positive, or pending review */
 				status: DelphiReportIssueStatus
-				/** Specific details about where/why this issue was raised */
 				details: ReportIssueDetail[]
 			}
 
 			export type ReportIssueDetail = {
-				/** Unique ID of this detail entry */
 				id: string
-				/** ID of the issue this detail belongs to */
 				issue_id: string
-				/** Key identifier for this detail (e.g., class or method name) */
 				key: string
-				/** File path where the issue was found (e.g., 'com/example/Class.class') */
 				file_path: string
-				/** Decompiled source code showing the issue (may be omitted for large files) */
 				decompiled_source: string | null
-				/** Additional structured metadata about this detail */
 				data: Record<string, unknown>
-				/** Severity level of this specific detail */
 				severity: DelphiSeverity
 			}