[mac][js] ui polish by Copilot · Pull Request #328 · banshee-data/velocity.report

Copilot · 2026-02-24T02:36:16Z

This pull request standardizes the naming of the Markdown formatting target across the codebase from format-markdown to format-docs, updates documentation to reflect this change, and advances the project version. Additionally, it updates backlog tracking and marks the completion of all HINT sweep mode polish items.

Formatting target standardization:

Renamed the Markdown formatting target from format-markdown to format-docs in the Makefile, .pre-commit-config.yaml, and associated documentation, ensuring consistency across configuration, help text, and README references. [1] [2] [3] [4] [5] [6]

Version and backlog updates:

Updated the project version from 0.5.0-pre12 to 0.5.0-pre13 in the Makefile.
Moved two platform hardening items from the v0.5 backlog to the v0.5.1 backlog and added new tracking items for SWEEP/HINT platform hardening, HINT sweep polish, and ObjectClass schema alignment. [1] [2]

Documentation improvements:

Marked the HINT sweep mode as fully complete, updating its status and summary to reflect that all polish items are finished.
Checked off all previously incomplete polish tasks in the HINT sweep mode documentation, including label panel improvements, removal of the export labels button, inline continue button, new TypeScript types, and shared page subtitle. [1] [2]

…s, Continue button, carried badge, exportLabels removal, page subtitle, plan docs Co-authored-by: ddol <445312+ddol@users.noreply.github.com>

…n, remove HStack wrapper Co-authored-by: ddol <445312+ddol@users.noreply.github.com>

github-actions · 2026-02-24T03:28:16Z

🤖 Version Bump Advisory

Warnings

⚠️ Database migration detected

Files: internal/db/migrations/000012_add_track_quality_metrics.up.sql,internal/db/migrations/000013_create_site_config_periods.up.sql,internal/db/migrations/000015_add_site_map_fields.up.sql,internal/db/migrations/000016_lidar_labels.up.sql,internal/db/migrations/000018_add_quality_label.up.sql,internal/db/migrations/000019_add_lidar_labels_scene_source.up.sql,internal/db/migrations/000020_create_lidar_scenes.up.sql,internal/db/migrations/000022_create_lidar_sweeps.up.sql,internal/db/migrations/000023_add_vrlog_path.up.sql,internal/db/migrations/000024_add_sweep_metadata.up.sql,internal/db/migrations/000025_add_label_source.up.sql,internal/db/migrations/000028_create_lidar_evaluations.up.sql,internal/db/migrations/000029_expand_label_vocabulary.up.sql
Review for breaking changes (MAJOR) vs additive changes (MINOR)

Version Updates

✅ Radar: 0.5.0-pre12 → 0.5.0-pre13
✅ PDF Generator: 0.5.0-pre12 → 0.5.0-pre13
✅ Web Frontend: 0.5.0-pre12 → 0.5.0-pre13

📖 See CHANGELOG.md for detailed guidelines.

This is an automated advisory. Review the detected changes and update versions accordingly.

github-actions · 2026-02-24T03:29:01Z

Coverage report

This PR does not seem to contain any modification to coverable code.

Copilot

Pull request overview

This PR completes the final 11 polish items for HINT (Human-Involved Numerical Tuning) sweep mode, bringing the feature to 57/57 items complete. The changes add TypeScript type definitions for HINT state management, implement an inline "Continue to Sweep" button in the Svelte frontend for the awaiting_labels state, polish the macOS visualiser with a carried-over label badge and removal of the obsolete export labels functionality, and update the page subtitle to match the sweep dashboard description.

Changes:

Added TypeScript interfaces (HINTState, HINTRound, LabelProgress) matching Go backend structs for HINT sweep state management
Implemented inline "Continue to Sweep" button in Svelte sweeps page for awaiting_labels state with proper error handling and state refresh
Added "↻ carried" badge in macOS visualiser label panel for tracks with labelerId == "hint-carryover"
Removed obsolete exportLabels() method and corresponding test from macOS visualiser
Updated page subtitle to British English phrasing matching sweep dashboard

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
`web/src/lib/types/lidar.ts`	Added TypeScript interfaces for HINT state, rounds, and label progress matching Go backend structs
`web/src/routes/lidar/sweeps/+page.svelte`	Implemented `handleContinueHINT()` function, inline continue button for awaiting_labels state, and updated page subtitle to British English
`tools/visualiser-macos/VelocityVisualiser/UI/ContentView.swift`	Added carried-over label badge display when `track.labelerId == "hint-carryover"`
`tools/visualiser-macos/VelocityVisualiser/App/AppState.swift`	Removed obsolete `exportLabels()` method
`tools/visualiser-macos/VelocityVisualiserTests/CoverageBoostTests.swift`	Removed test for deleted `exportLabels()` functionality
`docs/plans/lidar-sweep-hint-mode-plan.md`	Marked all 11 remaining items complete, updated status to 57/57 ✅
`docs/lidar/operations/hint-sweep-mode.md`	Marked P1.4, P2.2, Phase 5, and Phase 6 items complete, updated summary

codecov · 2026-02-24T03:47:59Z

Codecov Report

❌ Patch coverage is 95.21912% with 12 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
internal/lidar/l6objects/classification.go	87.09%	6 Missing and 6 partials ⚠️

📢 Thoughts on this report? Let us know!

…DisplayView

…actor RunBrowserViewTests for improved readability

…ample tests

…setup in ContentViewTests and RunBrowserViewTests

…iate feedback

…labels cache tests

… new VRLOG replay

… test

…anagement - Introduced `PlaybackControlsDerivedState` to encapsulate UI state logic for playback controls in `ContentView.swift`. - Updated `PlaybackRPCClient` protocol and `VisualiserClient` class to return `VisualiserPlaybackStatus` from playback control methods. - Enhanced `AppState` tests to include scenarios for playback control interactions using a mock `FakePlaybackRPCClient`. - Added unit tests for `PlaybackControlsDerivedState` to verify UI behavior based on playback mode and connection state. - Improved coverage in existing tests by integrating playback command client mock and ensuring proper state transitions during playback operations.

… tracks - Added ClassifyFeatures method to classify objects from pre-built feature values. - Updated TrackClassifier to use ClassifyFeatures for VRLOG replay. - Introduced classifyOrConvert function to handle empty ObjectClass during replay. - Enhanced tests to verify classification consistency and reclassification behavior. - Updated visualiser.proto to reflect new ObjectClass definitions and user-assignable labels.

…nsolidation plan

…lity indicators

…kefile

… peak speed with a dashed line indicator

… configuration

…nt and priority details

…tion targets

Copilot

Pull request overview

Copilot reviewed 68 out of 75 changed files in this pull request and generated no new comments.

…tClass enum, track field parity, checklist Co-authored-by: ddol <445312+ddol@users.noreply.github.com>

Co-authored-by: ddol <445312+ddol@users.noreply.github.com>

…ction logic

…ved data handling and UI clarity

…nd improve TrackHistoryGraphView data handling

… to use persistent peak speed

…information in AboutView

Initial plan

3c21029

Copilot AI assigned Copilot and ddol Feb 24, 2026

Copilot AI requested a review from ddol February 24, 2026 02:36

Copilot started work on behalf of ddol February 24, 2026 03:10 View session

Copilot AI and others added 2 commits February 24, 2026 03:22

[ai][js][mac][docs] complete HINT sweep polish items: TypeScript type…

8ac6b9e

…s, Continue button, carried badge, exportLabels removal, page subtitle, plan docs Co-authored-by: ddol <445312+ddol@users.noreply.github.com>

[ai][js][mac] address review: extract isContinuing, simplify conditio…

35c5767

…n, remove HStack wrapper Co-authored-by: ddol <445312+ddol@users.noreply.github.com>

Copilot AI changed the title ~~[WIP] Complete tasks from BACKLOG.md for polish and hardening~~ Complete HINT sweep polish items (Phase 5–6, macOS visualiser P1/P2) Feb 24, 2026

Copilot finished work on behalf of ddol February 24, 2026 03:26

ddol marked this pull request as ready for review February 24, 2026 03:28

Copilot AI review requested due to automatic review settings February 24, 2026 03:28

Copilot started reviewing on behalf of ddol February 24, 2026 03:28 View session

Copilot AI reviewed Feb 24, 2026

View reviewed changes

ddol changed the title ~~Complete HINT sweep polish items (Phase 5–6, macOS visualiser P1/P2)~~ [mac][js] ui polish Feb 25, 2026

ddol added 12 commits February 24, 2026 16:08

[mac] simplify timestamp range checks and improve readability in Time…

9138781

…DisplayView

[mac] add unit tests for SparklineView and TrackHistoryGraphView; ref…

998878e

…actor RunBrowserViewTests for improved readability

[mac] refactor SparklineViewTests to use async/await for graph view s…

5bde323

…ample tests

[ver] bump version to 0.5.0-pre13 across all relevant files

34b491c

[docs] BACKLOG, complete SWEEP/HINT platform hardening and polish items

f8555ef

[mac] refactor tests to use a reusable host function for environment …

092896f

…setup in ContentViewTests and RunBrowserViewTests

[mac] add user labels caching and update track list display for immed…

e253b09

…iate feedback

[mac] add playback state reset for new VRLOG replay and enhance user …

d4456fb

…labels cache tests

[mac] refactor playback state management and gRPC stream handling for…

d0a8dd1

… new VRLOG replay

[go] reset pause state when entering VRLOG mode and add corresponding…

e6436db

… test

[mac] add regression test for late finish stream handling in AppState

bc0b790

ddol added 12 commits February 24, 2026 16:08

[mac] update object classification labels and enhance display logic

f039fd8

[docs] add classification maths documentation and label vocabulary co…

351fedb

…nsolidation plan

[mac] enhance track display with tag pills for classification and qua…

57670f4

…lity indicators

[sql] improve SQL migration formatting and enhance script functionality

12de9bc

[mac] TrackListView to improve tag display and reduce layout complexity

b9d8c53

[make] rename format-markdown target to format-docs for clarity in Ma…

eabd9e0

…kefile

[mac] enhance TrackHistoryGraphView and SparklineView to display live…

13a5330

… peak speed with a dashed line indicator

[git] rename format-markdown to format-docs for clarity in pre-commit…

90973c3

… configuration

[docs] update BACKLOG and AV integration plan with new schema alignme…

4d25ab2

…nt and priority details

[make] rename format-markdown to format-docs for clarity in documenta…

9116590

…tion targets

[mac] enhance track data handling with peak speed and ranking updates

2e1b2ba

ddol requested a review from Copilot February 25, 2026 00:10

Copilot started reviewing on behalf of ddol February 25, 2026 00:10 View session

Copilot AI reviewed Feb 25, 2026

View reviewed changes

Copilot started work on behalf of ddol February 25, 2026 00:49 View session

Copilot AI and others added 2 commits February 25, 2026 00:55

[ai][docs] update proto contract plan with implemented reality: Objec…

60b6ba3

…tClass enum, track field parity, checklist Co-authored-by: ddol <445312+ddol@users.noreply.github.com>

[ai][docs] clarify debug test note wording in proto contract plan

1ac05e3

Co-authored-by: ddol <445312+ddol@users.noreply.github.com>

Copilot AI changed the title ~~[mac][js] ui polish~~ Update proto contract plan with implemented reality (ObjectClass enum, track field parity) Feb 25, 2026

Copilot finished work on behalf of ddol February 25, 2026 00:56

ddol added 7 commits February 24, 2026 17:35

[mac] update TrackListView to enhance rank tracking and climbing dete…

48f8315

…ction logic

[mac] add AboutView and integrate About panel in VelocityVisualiserApp

69c6024

[mac] refactor TrackInspectorView and TrackHistoryGraphView for impro…

ea475a0

…ved data handling and UI clarity

[docs] reorder

255f6c3

[mac] enhance TimeDisplayView with fallback for frame-index display a…

91ce3d9

…nd improve TrackHistoryGraphView data handling

[mac] add trackPeakSpeed to AppState and update TrackHistoryGraphView…

5789cf8

… to use persistent peak speed

[mac] update app name references to VelocityReport and add licensing …

49cbb65

…information in AboutView

ddol merged commit fe1b34a into main Feb 25, 2026
23 checks passed

ddol deleted the copilot/complete-backlog-items-polishing branch February 25, 2026 05:04

ddol changed the title ~~Update proto contract plan with implemented reality (ObjectClass enum, track field parity)~~ [mac][js] ui polish Feb 25, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Comments

[mac][js] ui polish#328

[mac][js] ui polish#328
ddol merged 44 commits intomainfrom
copilot/complete-backlog-items-polishing

Copilot AI commented Feb 24, 2026 •

edited by ddol

Loading

Uh oh!

github-actions bot commented Feb 24, 2026 •

edited

Loading

Uh oh!

github-actions bot commented Feb 24, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

codecov bot commented Feb 24, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Comments

Conversation

Copilot AI commented Feb 24, 2026 • edited by ddol Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions bot commented Feb 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🤖 Version Bump Advisory

Warnings

Version Updates

Uh oh!

github-actions bot commented Feb 24, 2026

Coverage report

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

codecov bot commented Feb 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Copilot AI commented Feb 24, 2026 •

edited by ddol

Loading

github-actions bot commented Feb 24, 2026 •

edited

Loading

codecov bot commented Feb 24, 2026 •

edited

Loading