Skip to content

Add feedback and card learning flow - #3

Open
vortexopenclaw wants to merge 18 commits into
mainfrom
feature/feedback-card-learning
Open

vortexopenclaw wants to merge 18 commits into
mainfrom
feature/feedback-card-learning

Conversation

@vortexopenclaw

Copy link
Copy Markdown
Owner

Summary

  • add Feedback and Learn Card toolbar flows in the macOS app
  • submit sanitized bug/feature/training payloads to a Cloudflare Worker endpoint
  • include scan diagnostics for profile scoring and OSD OCR gates without sending video frames, raw OCR text, GPS traces, or device IDs
  • add copy-flow polish that this branch depends on: date presets, optional filename suffixes, camera settings/log copy, progress speed/ETA, and optional destination open/source eject
  • document the Worker deployment shape and privacy constraints

Validation

  • swift build
  • swift run DashcamOffloader --smoke-test
  • node --check workers/feedback/worker.js
  • git diff --check

@vortexopenclaw

Copy link
Copy Markdown
Owner Author

Deployment note:

  • Existing env token works when mapped from CLOUDFLARE_WORKERS_API_TOKEN to CLOUDFLARE_API_TOKEN for Wrangler.
  • R2 bucket creation failed with Cloudflare auth error 10000, so that token does not currently have R2 bucket-create permission.
  • Created a dedicated KV namespace for this project instead: dashcam_offloader_feedback.
  • Deployed dedicated Worker: https://dashcam-offloader-feedback.vortexradar.workers.dev
  • App endpoint updated to that live URL.
  • Smoke POST to /feedback returned ok with id f07f3e3e-5a5a-44d0-95ec-79185946e1c4.

@vortexopenclaw

Copy link
Copy Markdown
Owner Author

Updated this PR with identifiable-but-unsupported camera handling. BlackVue model metadata from version.bin / micom_version.bin / smart_gsensor_version.bin now creates a New Dashcam result instead of forcing the closest existing profile. Validated /Volumes/BLACKVUE 2 reports BlackVue DR970X LTE Plus as unsupported rather than selecting Elite 8. Smoke coverage added for supported and unsupported BlackVue metadata cases.

@vortexopenclaw

Copy link
Copy Markdown
Owner Author

Added generic unsupported-card classification to this PR. If no reliable supported profile is selected, the app now uses a New Dashcam generic fallback to parse common filename dates/times and infer folder/filename categories like Driving, Parking, Parking Events, Protected/SOS, Photos, GPS, plus conservative Front/Rear/Interior suffixes. Also updated the test-share branch and Cloudflare latest build. Validated with swift build, smoke test, packaged app smoke test, git diff --check, and live download hash match.

@vortexopenclaw

Copy link
Copy Markdown
Owner Author

Added protected/saved clip detection for dashcam cards. Scanner now treats file-level read-only video attributes as protected clips, including generic New Dashcam fallback cards, and docs now call out user-saved emergency clips as separate from G-sensor/event folders. Latest Cloudflare build: Dashcam-Offloader-protected-clips-ec4179d.zip, SHA-256 0f8faefb2def26bffeb72767b55f3e1da1ca2ce535e058b04807b704f48820a8.

@vortexopenclaw

Copy link
Copy Markdown
Owner Author

Updated the feedback/New Dashcam branch with two UI fixes: the Source Scan chooser now opens at the selected card path, and Copy Plan now previews selected clips even before an output directory is chosen while keeping Copy Selected disabled until a destination exists. Pushed test/share build c00dcd7 to GitHub and Cloudflare latest; verified the public download matches the local build internally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant