fix(perf): 修复缓存与卡顿诊断

Author: coderDJing

.github/workflows/release.yml

@@ -917,3 +917,21 @@ jobs:
               gh release create "$TAG_NAME" "${assets[@]}" --title "$TAG_NAME" --latest --verify-tag --generate-notes
             fi
           fi
+
+      - name: Remove superseded RC releases
+        if: contains(github.ref_name, '-rc.')
+        env:
+          GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          GH_REPO: ${{ github.repository }}
+          TAG_NAME: ${{ github.ref_name }}
+        run: |
+          set -euo pipefail
+          base_version="${TAG_NAME%%-rc.*}"
+          mapfile -t stale_tags < <(
+            gh release list --limit 100 --json tagName,isPrerelease \
+              --jq ".[] | select(.isPrerelease and .tagName != \"${TAG_NAME}\" and (.tagName | startswith(\"${base_version}-rc.\"))) | .tagName"
+          )
+          for stale_tag in "${stale_tags[@]}"; do
+            echo "Removing superseded RC release and tag: ${stale_tag}"
+            gh release delete "${stale_tag}" --yes --cleanup-tag
+          done

AGENTS.md

@@ -69,7 +69,7 @@
 - PRs should include a clear summary, testing notes, and screenshots for UI changes; link related issues and call out packaging or native-module impacts.
 
 ## Release Rules
-- 如果我说“发布预发布版本”，默认执行：更新 `package.json` 版本号为 `X.Y.Z-rc.<当前时间>`（时间格式 `YYYYMMDDHHmm`），提交，并**先 push main**，再打同名 tag 并 push tag 触发发布流程。
+- 如果我说“发布预发布版本”，默认执行：更新 `package.json` 版本号为 `X.Y.Z-rc.<当前时间>`（时间格式 `YYYYMMDDHHmm`），提交，并**先 push main**，再打同名 tag 并 push tag 触发发布流程。新 RC 的 GitHub Release 资产发布成功后，工作流自动删除同一 `X.Y.Z` 的旧 RC Release 与 tag；禁止在新 RC 发布成功前手工清理旧 RC。
 - 如果当前版本是正式版（不含 `-rc`）且我要求预发布版本，自动把版本号 **补丁位 +1**（`Z+1`，逢 10 进 1，例如 `1.1.9 -> 1.2.0`），再加 `-rc.<当前时间>` 并发布。
 - 如果我说“发布正式版”，就把 `-rc.` 以及后面的时间后缀去掉，仅保留 `X.Y.Z`，提交，并**先 push main**，再打同名 tag 并 push tag 触发发布流程。
 - 发布顺序固定：提交 → push main → 打 tag → push tag。

package.json

@@ -1,6 +1,6 @@
 {
   "name": "frkb",
-  "version": "1.2.4-rc.202609171805",
+  "version": "1.2.4-rc.202609181220",
   "description": "Track Studio — Rapid Audio Organization Tool",
   "license": "PolyForm-Noncommercial-1.0.0",
   "main": "./out/main/index.js",

src/main/index.ts

@@ -775,7 +775,11 @@ app.on('window-all-closed', async () => {
   app.quit()
 })
 
-const RC_DIAGNOSTIC_OUTPUT_LOG_SCOPES = new Set(['playlist-open-perf', 'delete-all-above-perf'])
+const RC_DIAGNOSTIC_OUTPUT_LOG_SCOPES = new Set([
+  'playlist-open-perf',
+  'delete-all-above-perf',
+  'renderer-stall-diagnostic'
+])
 
 ipcMain.on('outputLog', (_event, logMsg) => {
   if (

src/main/services/scanSongs.spec.ts

@@ -1,7 +1,7 @@
 import { describe, expect, it } from 'vitest'
 import type { ISongInfo } from '../../types/globals'
 import { createSongBeatGridMapV2FromFixedGrid } from '../../shared/songBeatGridMapV2'
-import { preserveCachedGridAnalysisFields } from './scanSongs'
+import { prepareFullMetadataCacheInfo, preserveCachedGridAnalysisFields } from './scanSongs'
 
 const createGrid = () => {
   const grid = createSongBeatGridMapV2FromFixedGrid({
@@ -88,3 +88,23 @@ describe('preserveCachedGridAnalysisFields', () => {
     expect(target.timeBasisOffsetAlgorithmVersion).toBeUndefined()
   })
 })
+
+describe('prepareFullMetadataCacheInfo', () => {
+  it('turns an analysis-only cache entry into a regular cache hit after metadata parsing', () => {
+    const cached: ISongInfo = {
+      ...createSong(),
+      analysisOnly: true,
+      key: '8A',
+      beatGridMap: createGrid(),
+      energyScore: 74,
+      energyAlgorithmVersion: 1
+    }
+
+    const result = prepareFullMetadataCacheInfo(createSong(), cached, true)
+
+    expect(result.analysisOnly).toBe(false)
+    expect(result.key).toBe('8A')
+    expect(result.beatGridMap).toEqual(cached.beatGridMap)
+    expect(result.energyScore).toBe(74)
+  })
+})

src/main/services/scanSongs.ts

@@ -279,6 +279,27 @@ const preserveCachedAnalysisFields = (target: ISongInfo, cachedInfo?: ISongInfo
   preserveBestAvailableSongStructure(target, cachedInfo)
 }
 
+/**
+ * A metadata scan has read the complete track record from disk. Even when its previous cache
+ * row only contained analysis results, the replacement row must become a normal cache hit on
+ * the next scan.
+ */
+export const prepareFullMetadataCacheInfo = (
+  info: ISongInfo,
+  cachedInfo?: ISongInfo | null,
+  cachedStatMatches = false
+): ISongInfo => {
+  const nextInfo: ISongInfo = { ...info, analysisOnly: false }
+  if (!cachedInfo) return nextInfo
+  preserveCachedKeyAndBpm(nextInfo, cachedInfo)
+  if (cachedStatMatches) {
+    preserveCachedGridAnalysisFields(nextInfo, cachedInfo)
+    preserveCachedEnergyAnalysisFields(nextInfo, cachedInfo)
+  }
+  preserveCachedUserListFields(nextInfo, cachedInfo)
+  return nextInfo
+}
+
 const preserveCachedUserListFields = (target: ISongInfo, cachedInfo?: ISongInfo | null) => {
   const cachedPlaylistTrackNumber = normalizePlaylistTrackNumber(cachedInfo?.playlistTrackNumber)
   if (
@@ -567,21 +588,10 @@ export async function scanSongList(
         for (const st of filesStatList) {
           const info = infoMap.get(st.key)
           if (!info) continue
-          const nextInfo = { ...info }
           const cached = cacheMap.get(st.key)
-          if (cached?.info) {
-            preserveCachedKeyAndBpm(nextInfo, cached.info)
-            const cachedStatMatches =
-              cached.size === st.size && Math.abs(cached.mtimeMs - st.mtimeMs) < 1
-            if (cachedStatMatches) {
-              preserveCachedGridAnalysisFields(nextInfo, cached.info)
-              preserveCachedEnergyAnalysisFields(nextInfo, cached.info)
-            }
-            if (nextInfo.analysisOnly === undefined && cached.info.analysisOnly) {
-              nextInfo.analysisOnly = true
-            }
-            preserveCachedUserListFields(nextInfo, cached.info)
-          }
+          const cachedStatMatches =
+            !!cached && cached.size === st.size && Math.abs(cached.mtimeMs - st.mtimeMs) < 1
+          const nextInfo = prepareFullMetadataCacheInfo(info, cached?.info, cachedStatMatches)
           newEntriesMap.set(st.file, {
             size: st.size,
             mtimeMs: st.mtimeMs,

src/main/window/mainWindow/responsivenessDiagnostics.ts

@@ -9,6 +9,7 @@ import { isPackagedRcBuild } from '../../services/rcDiagnostics'
 const MAIN_PROCESS_STALL_THRESHOLD_MS = 3_000
 const MAIN_PROCESS_HEARTBEAT_INTERVAL_MS = 1_000
 const MAIN_PROCESS_STALL_INCIDENT_GRACE_MS = 30_000
+const MIN_MEANINGFUL_TRACED_ACTIVITY_DURATION_MS = 50
 
 type MainProcessStallIncident = {
   startedAtMs: number
@@ -226,10 +227,16 @@ export const attachMainWindowResponsivenessDiagnostics = (
           return
         }
         const overlappingActivity = snapshot.activity.longest
-        const stallClassification = overlappingActivity
+        const meaningfulOverlappingActivity =
+          overlappingActivity &&
+          (overlappingActivity.pending ||
+            overlappingActivity.durationMs >= MIN_MEANINGFUL_TRACED_ACTIVITY_DURATION_MS)
+            ? overlappingActivity
+            : undefined
+        const stallClassification = meaningfulOverlappingActivity
           ? {
               kind: 'tracked-main-thread-activity-overlap',
-              activity: overlappingActivity
+              activity: meaningfulOverlappingActivity
             }
           : processCpuRatio >= 0.7
             ? {

src/renderer/src/composables/horizontalBrowse/horizontalBrowseUserTiming.spec.ts

@@ -0,0 +1,21 @@
+import { afterEach, describe, expect, it, vi } from 'vitest'
+import { startHorizontalBrowseUserTiming } from './horizontalBrowseUserTiming'
+
+afterEach(() => {
+  vi.restoreAllMocks()
+})
+
+describe('startHorizontalBrowseUserTiming', () => {
+  it('clears its measure after recording it so repeated render work cannot retain entries', () => {
+    const mark = vi.spyOn(performance, 'mark')
+    const measure = vi.spyOn(performance, 'measure')
+    const clearMeasures = vi.spyOn(performance, 'clearMeasures')
+
+    const finish = startHorizontalBrowseUserTiming('frkb:hb:test')
+    finish()
+
+    expect(mark).toHaveBeenCalledTimes(2)
+    expect(measure).toHaveBeenCalledWith('frkb:hb:test', expect.any(String), expect.any(String))
+    expect(clearMeasures).toHaveBeenCalledWith('frkb:hb:test')
+  })
+})

src/renderer/src/composables/horizontalBrowse/horizontalBrowseUserTiming.ts

@@ -19,5 +19,6 @@ export const startHorizontalBrowseUserTiming = (name: string) => {
     performance.measure(name, startMark, endMark)
     performance.clearMarks(startMark)
     performance.clearMarks(endMark)
+    performance.clearMeasures(name)
   }
 }

src/renderer/src/main.ts

@@ -14,6 +14,7 @@ import {
   watchUiSettings
 } from '@renderer/utils/uiSettingsStorage'
 import { installConsoleLogBridge } from '@renderer/utils/installConsoleLogBridge'
+import { installRendererStallDiagnostics } from '@renderer/utils/rendererStallDiagnostics'
 import type { ICuratedArtistFavorite } from 'src/types/globals'
 
 declare global {
@@ -26,6 +27,7 @@ const pinia = createPinia()
 const app = createApp(App)
 
 installConsoleLogBridge('main-window')
+installRendererStallDiagnostics()
 
 app.directive('dialog-drag', dialogDrag)