Game annotation review v2: durable corrections #5

Open
waylon wants to merge 24 commits from codex/game-annotation-review-v2 into main
Owner

Summary

  • Preserve sensor corrections across reprocessing by stable observation identity and source provenance.
  • Keep observation selection and correction input stable during playback and refresh.
  • Deduplicate retried correction responses in the review UI.

Verification

  • just test: 306 passed
  • just lint, just typecheck: passed
  • node --check and node --test tests/test_game_sensor_review_core.js: 8 passed
  • just build-css: completed; generated output was not included because no CSS changed
  • Disposable authenticated PostgreSQL/browser lifecycle verified for fixture media 1009 and 1010, including reload, local service restart, and v1.21 sensor reprocessing.

Production database and authenticated review route are healthy, but production currently has no review batch or sensor fixture records for media 1009 and 1010. No production data was modified and no deployment was performed.

## Summary - Preserve sensor corrections across reprocessing by stable observation identity and source provenance. - Keep observation selection and correction input stable during playback and refresh. - Deduplicate retried correction responses in the review UI. ## Verification - `just test`: 306 passed - `just lint`, `just typecheck`: passed - `node --check` and `node --test tests/test_game_sensor_review_core.js`: 8 passed - `just build-css`: completed; generated output was not included because no CSS changed - Disposable authenticated PostgreSQL/browser lifecycle verified for fixture media 1009 and 1010, including reload, local service restart, and v1.21 sensor reprocessing. Production database and authenticated review route are healthy, but production currently has no review batch or sensor fixture records for media 1009 and 1010. No production data was modified and no deployment was performed.
ci: add Forgejo checks for review workflow
Some checks are pending
Dropper checks / checks (push) Waiting to run
Dropper checks / checks (pull_request) Waiting to run
71bb95a205
Some checks are pending
Dropper checks / checks (push) Waiting to run
Dropper checks / checks (pull_request) Waiting to run
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin "codex/game-annotation-review-v2:codex/game-annotation-review-v2"
git switch "codex/game-annotation-review-v2"

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch "main"
git merge --no-ff "codex/game-annotation-review-v2"
git switch "codex/game-annotation-review-v2"
git rebase "main"
git switch "main"
git merge --ff-only "codex/game-annotation-review-v2"
git switch "codex/game-annotation-review-v2"
git rebase "main"
git switch "main"
git merge --no-ff "codex/game-annotation-review-v2"
git switch "main"
git merge --squash "codex/game-annotation-review-v2"
git switch "main"
git merge --ff-only "codex/game-annotation-review-v2"
git switch "main"
git merge "codex/game-annotation-review-v2"
git push origin "main"
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
waylon/dropper!5
No description provided.