130 Commits

Author SHA1 Message Date
2b944cf9d1 fix: use search/people API (Core Team deployed)
- Remove duplicate search_people case
- Remove search_identities (no longer needed)
- Add search_people response handler
- SearchView already uses search_people
2026-07-25 23:16:59 +08:00
453168f326 chore: remove unused .gs-sidebar-lock CSS 2026-07-25 21:13:22 +08:00
bcffe62a15 fix: remove lock icon from sidebar toggle 2026-07-25 21:13:07 +08:00
ee9a602f7f fix: toggle shows expand chevron when locked, lock icon when unlocked 2026-07-25 20:57:58 +08:00
3aef7d115f fix: sidebar default to collapsed 2026-07-25 20:56:29 +08:00
19846a8773 revert: restore localStorage-based sidebar default 2026-07-25 20:51:48 +08:00
bb4ff9b007 fix: sidebar default to collapsed (locked) 2026-07-25 20:51:38 +08:00
fcb612ff0e revert: restore search_identities API case 2026-07-25 20:38:54 +08:00
1a0b3f2c1f fix: use search/people API (no identity concept) 2026-07-25 20:38:30 +08:00
9386a8e8ef fix: fallback to search_identities (search/people endpoint not deployed) 2026-07-25 20:35:47 +08:00
fb9a55a64c feat: use new search/people API for People search mode 2026-07-25 20:33:15 +08:00
9691e75e36 fix: use height:100% on sidebar/main (resolves to parent content height) 2026-07-25 20:29:35 +08:00
674126d687 fix: add html/body height:100% + 100% height on sidebar/main
- html,body { height:100% } to ensure proper height chain
- ms-side { height: 100% } and ms-main { height: 100% }
- Both resolve to same parent content height: 100vh - 56px
2026-07-25 20:29:26 +08:00
d37967d38e fix: explicit calc(100vh - 56px) on both sidebar and main 2026-07-25 20:28:56 +08:00
a289346fdc fix: revert to flexbox with height:100vh, fix footer visibility
- height:100vh (not min-height) so align-items:stretch actually works
- Footer shows when expanded, hides when collapsed (dynamic class)
- Remove align-self:stretch (unnecessary with flex stretch)
- Restore flex-shrink on divider
2026-07-25 20:27:09 +08:00
dd843c29ee fix: use CSS grid for equal-height sidebar and main 2026-07-25 20:24:11 +08:00
d7592c528c fix: sync sidebar height with main content area 2026-07-25 20:21:17 +08:00
f75eb44351 feat: show compact login/logout icon when sidebar collapsed 2026-07-25 20:20:03 +08:00
652ba77c77 fix: use CSS instead of v-show for sidebar collapse text
- Replace v-show with CSS .gs-nav-label + .collapsed display: none
- Use CSS class .gs-hidden instead of v-show for footer/divider
- Fix content not re-displaying after collapse/expand
2026-07-25 20:18:03 +08:00
ca94259fba feat: add sidebar collapse/expand toggle 2026-07-25 20:11:56 +08:00
660f01c6c5 fix: People search mode label 2026-07-25 19:27:56 +08:00
cb7ebfb633 feat: add Face Group search mode back to SearchView
- Add 'Face Group' mode to search modes
- Use search_identities API temporarily
- Update summary: 'Identity' → 'FS'
- Will replace with Core Team's search_tkg_node_face_trace_profile API
2026-07-25 19:24:54 +08:00
25f718c0d5 fix: remove duplicate case clauses in api/index.ts 2026-07-25 18:28:04 +08:00
0b19a0af0d refactor: QC report - 'Trace Profile' → 'FS Profile' 2026-07-25 15:53:37 +08:00
cf397b268e refactor: QC - use FS instead of Trace
- Display format: FS#123 instead of Trace #123
- Update labels: 'Traces' → 'FS'
- Update messages: 'clusters' → 'groups'
2026-07-25 15:53:17 +08:00
aa25600580 refactor: remove identity/pending concepts
- identity_uuid → group_uuid
- Remove status: 'pending'
- Remove starred: false
- Keep only: face, FS, face group
2026-07-25 15:51:38 +08:00
464ddd16a9 chore: remove debug console.log from faceGroups 2026-07-25 15:49:27 +08:00
4463ff2f96 style: change 'Faces' → 'FS' in Face Groups card 2026-07-25 15:48:13 +08:00
15647a47ac fix: replace all clusterAsPending → faceGroups 2026-07-25 15:43:28 +08:00
0a7902c2f5 refactor: rename clusterAsPending → faceGroups (no pending concept)
- Rename computed property to match current API semantics
- face_groups are definitive, not pending
- Add detailed console.log for debugging empty groups issue
2026-07-25 15:43:15 +08:00
d3cd940818 fix: revert to clusterAsPending (remove allGroupsForMove) 2026-07-25 15:42:31 +08:00
7cc4708c66 debug: add more console.log for clusterAsPending diagnosis 2026-07-25 15:42:17 +08:00
5db1b68fa0 debug: add allGroupsForMove computed for Move Faces Modal
- Explicitly define data source for Move Faces Modal
- Use clusterAsPending directly
- Add console.log for debugging

Purpose: Diagnose why only one group shows in Move Faces Modal
2026-07-25 15:40:48 +08:00
ec8d8d9d1a fix: add fallback for missing group names in Move Faces Modal
Problem:
- Some groups may not have name field
- Display would show blank or undefined

Solution:
- Add fallback: c.name || 'Unnamed Group'
- Add fallback: c.trace_count || 0
- Ensures always shows meaningful text

Impact:
- Move Faces Modal now always shows group names
- No blank entries even with incomplete data
2026-07-25 15:40:01 +08:00
f727b21e99 refactor: remove unused updateClusterAfterMerge function
- No longer needed with new merge_groups API
- merge_groups API handles group merging server-side
- Frontend only needs to refresh data after merge
2026-07-25 14:25:59 +08:00
8af2dc6128 feat: integrate new merge_groups API from Core Team
Replace merge_trace implementation with merge_groups API

Changes:
1. src/api/index.ts:
   - Add merge_groups API definition
   - POST /api/v1/file/:file_uuid/groups/merge
   - Parameters: file_uuid, source_groups[], target_group_name

2. src/views/PeopleView.vue:
   - Rewrite executeMerge() function
   - Use group names instead of trace IDs
   - Single API call for all groups
   - Remove trace-level loop

3. src/views/PeopleView.vue:
   - Delete updateClusterAfterMerge() function
   - No longer needed with new API

Benefits:
-  Atomic operation (single API call)
-  Correct merge behavior (groups, not traces)
-  Better performance (1 call vs N calls)
-  Data consistency guaranteed

API behavior:
- Merge N groups into 1 target group
- All traces from source groups merged to target
- Source groups deleted after merge
- Returns traces_merged count

Core API: Implemented by Core Team (2026-07-25)
Status: Ready for testing
2026-07-25 14:25:35 +08:00
042bb45574 fix: update Pending Faces display to FS# format
- Line 102: Pending Faces name display
- Changed from F# to FS# (Face Sequence)
- Consistent with other Face Trace displays
2026-07-25 13:55:18 +08:00
e6b9b7b4b8 refactor: add # separator to display format
Display format with # separator:
- Frame: F#{number} (e.g., F#333)
- Face Trace: FS#{number} (e.g., FS#233)

Changes:
- Line 105: Frame range display (F#2219–F#2225)
- Line 143: Face Detail Modal (file_uuid:FS#233)
- Line 262: Cluster strip fallback (FS#233)
- Line 264: Cluster strip label (FS#233)

Benefits:
- # separator makes ID clearer
- Consistent format across all displays
- Visual distinction between type and number
2026-07-25 13:55:01 +08:00
98534bebfb refactor: change Face Trace display from 'Face{id}' to 'FS{id}'
Display format clarification:
- Frame: F{number} (e.g., F333)
- Face Trace: FS{number} (e.g., FS233)

Internal data remains: trace_id = 233 (integer)
Display text: FS233 (Face Sequence)

Changes:
- Line 143: Face Detail Modal display
- Line 262: Cluster strip fallback display
- Line 264: Cluster strip label

Reason:
- FS = Face Sequence (clear meaning)
- Avoid confusion with Frame (F)
- Separation of concerns: internal ID vs display text
2026-07-25 13:54:01 +08:00
389b3c8ba3 feat: add BuildVersion component to App.vue template
- Add BuildVersion at root level of template
- Shows build version in top-right corner on all views
2026-07-25 13:36:47 +08:00
9de4910aa0 feat: add BuildVersion component to App.vue
- Import BuildVersion component
- Add to template (top-right corner fixed)
- Visible on all views

Build version format: 20260725133504-gitHash
Shows in top-right corner with monospace font
2026-07-25 13:36:14 +08:00
baa42fa625 feat: add build version display to all views
Add build version indicator to verify code updates:

1. Create BuildVersion.vue component:
   - Displays in top-right corner (fixed position)
   - Format: YYYYMMDDHHMMSS-gitHash
   - Example: 20260725133504-39ce37e

2. Update vite.config.ts:
   - Inject __BUILD_VERSION__ global variable
   - Generated from build time + git commit hash

3. Add to App.vue:
   - Shown on all views automatically
   - Z-index: 9999 (always visible)

4. Add vite-env.d.ts:
   - TypeScript declarations for __BUILD_VERSION__

Purpose:
- User can verify if browser loaded latest code
- Easy to confirm updates are deployed
- No need to check source code

Location: Top-right corner, fixed position
Format: Monospace font, small text, subtle styling
2026-07-25 13:35:41 +08:00
39ce37e69c fix: remove premature updateClusterAfterMerge call in merge loop
Critical bug fix:
- updateClusterAfterMerge() was called after EACH trace merge
- This caused source group to be deleted after first trace
- Remaining traces had no group to merge into

Problem flow:
1. Merge trace #1 from Group B → Group A
2. updateClusterAfterMerge(B, A) deletes Group B
3. Try to merge trace #2 from Group B → Group B is gone!
4. Result: All source groups deleted after first trace

Solution:
- Remove updateClusterAfterMerge() from loop
- Let all traces merge successfully
- Reload cluster data at the end (Line 1813)
- Core API handles the merge, we just refresh UI

Impact:
- Merge now works correctly for all traces in a group
- No premature deletion of source groups
- UI refreshes after all operations complete
2026-07-25 13:29:47 +08:00
7b5650d098 fix: correct updateClusterAfterMerge to trigger computed update
Problem:
- Previous commit tried to splice computed property (clusterAsPending)
- Computed properties cannot be modified directly
- Need to update underlying data (clusterResultsCache)

Solution:
- Remove incorrect clusterAsPending.value.splice()
- Trigger reactive update via clusterResultsCache reassignment
- clusterAsPending computed will auto-update

Technical details:
- Vue computed properties are read-only
- Must modify underlying reactive data
- Reassignment triggers computed recalculation
2026-07-25 13:25:41 +08:00
c235d11d03 fix: handle empty groups after merge operation
Problem:
- After merge, source groups become empty
- Empty groups stay in UI as garbage data
- User sees empty groups with 0 faces

Solution:
- updateClusterAfterMerge() now removes source groups after merge
- Added removal from clusterAsPending computed property
- Ensures clean UI after merge operation

Behavior:
- Source group is deleted after all members merged to target
- Target group receives all members
- No empty groups left in UI

Related: merge functionality debug logging added in previous commit
2026-07-25 13:25:16 +08:00
361a01a025 debug: add comprehensive logging for merge groups functionality
Add debug logs to diagnose merge not working issue:

1. batchMergeGroups():
   - Log selected groupIds
   - Log mergeCandidateGroups count

2. executeMerge():
   - Log targetGroupId and targetGroup
   - Log targetTraceId type and value
   - Log targetTrace data
   - Log sourceGroups count
   - Log totalTraces and validTraces
   - Log each merge_trace API call params
   - Log success/fail counts

Purpose: Identify why merge fails when move works
Next: Check browser console for specific errors
2026-07-25 13:24:45 +08:00
69601fe512 docs: add Face Move/Merge diagnosis report
Complete documentation for face move functionality:

1. Display text improvements (completed):
   - Changed 'F13' to 'Face13' (3 locations)
   - Preserved frame display 'F2219-2225'

2. Data flow analysis:
   - Ideal flow: Core API → clusterTracesMap → selectedFaces → merge_trace API
   - 3 key checkpoints identified

3. Browser verification commands:
   - Check selectedFaces format
   - Check clusterTracesMap data
   - Test Move functionality

4. Potential issues:
   - Data source errors
   - Display text pollution
   - Frontend transformation errors

5. Core API specifications confirmed:
   - merge_trace: POST /file/:uuid/trace/:id/merge/:target_id
   - face-groups: GET /file/:uuid/face-groups

Status: Display improved, awaiting data flow verification
2026-07-25 13:13:19 +08:00
c0832f32f9 refactor: complete face trace display migration to 'Face{id}'
Additional changes:
- Line 143: Face detail modal trace ID display
- Line 262: Cluster strip fallback display

All trace_id displays now use 'Face{id}' format:
- F13 → Face13
- F142 → Face142

Frame displays remain with 'F' prefix:
- F2219-2225 (frame range) - unchanged

Total changes: 3 locations in PeopleView.vue
2026-07-25 13:12:10 +08:00
e7f9740e72 refactor: change face trace display from 'F{id}' to 'Face{id}'
Problem:
- 'F13' display text confused with Frame numbers
- Users cannot distinguish between Trace ID and Frame Number

Solution:
- Change display format from 'F{trace_id}' to 'Face{trace_id}'
- Example: F13 → Face13, F142 → Face142

Changes:
- src/views/PeopleView.vue:264 - Cluster Detail Modal label

Benefits:
- Clear semantic meaning (Face vs Frame)
- No ambiguity for users
- Display layer only, data layer unchanged (still trace_id)
2026-07-25 13:11:47 +08:00
70889b25d1 docs: update Core Team response - deployment complete
Core Team has deployed all fixes:
- src/worker/job_worker.rs: 6 running status updates
- src/api/files.rs:866: Initialize stages at registration
- src/api/files.rs:1762: sync-status TKG + chunks check
- src/api/files.rs:16: New imports

Status:  Production (3002) running
Build: 2026-07-24T18:43:52Z

Studio team ready to test
2026-07-25 03:35:28 +08:00