Skip to content

Implement scan progress dashboard#442

Merged
jdfalk merged 1 commit intomainfrom
codex/implement-scan-progress-in-react-dashboard
Jun 10, 2025
Merged

Implement scan progress dashboard#442
jdfalk merged 1 commit intomainfrom
codex/implement-scan-progress-in-react-dashboard

Conversation

@jdfalk
Copy link
Copy Markdown
Owner

@jdfalk jdfalk commented Jun 9, 2025

Summary

  • show scan progress on React dashboard
  • track scan results via new scanStatus handler
  • add progress callback in scanner package
  • expand webserver tests for scan endpoints
  • rebuild embedded web UI
  • update proof-of-concept checklist

Testing

  • go test ./pkg/webserver -run TestScanHandlers -count=1 -v
  • go test ./...

https://chatgpt.com/codex/tasks/task_e_6847714067ec8321be4562d664c6867a

@jdfalk jdfalk added the codex Created or modified by AI/automation agents label Jun 9, 2025 — with ChatGPT Codex Connector
@jdfalk jdfalk requested a review from Copilot June 9, 2025 23:53
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR implements a scan progress dashboard by adding new REST endpoints, updating UI components, and expanding test coverage for scan functionality. Key changes include the introduction of the Dashboard component in the web UI, backend handlers for scan operations, and a progress callback in the scanner package.

Reviewed Changes

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

Show a summary per file
File Description
webui/src/Dashboard.jsx New React component to display scan status and progress.
webui/src/App.jsx Updated to include navigation between dashboard and settings.
pkg/webserver/server_test.go Added tests to verify the new scan endpoints.
pkg/webserver/server.go Registered the scan and scan status endpoints with appropriate middleware.
pkg/webserver/scan.go Implemented scan handler and status tracking with a global state and synchronization lock.
pkg/scanner/progress_test.go Added test to verify that the progress callback is invoked correctly.
pkg/scanner/progress.go Introduced progress callback functionality using a worker pool for processing video files.
POC.md Updated checklist to reflect completion of the dashboard progress display.
Files not reviewed (1)
  • webui/package-lock.json: Language not supported

@jdfalk jdfalk force-pushed the codex/implement-scan-progress-in-react-dashboard branch from cde0e3c to 867b968 Compare June 10, 2025 11:43
@jdfalk jdfalk merged commit 70ef0bc into main Jun 10, 2025
1 check failed
@jdfalk jdfalk deleted the codex/implement-scan-progress-in-react-dashboard branch June 10, 2025 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

codex Created or modified by AI/automation agents

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants