Skip to content

feat: add public deck geometry review workspace - #55

Merged
FHoffarth merged 1 commit into
developfrom
feature/public-deck-geometry-review-workspace-v1
Aug 23, 2026
Merged

FHoffarth merged 1 commit into
developfrom
feature/public-deck-geometry-review-workspace-v1

Conversation

@FHoffarth

Copy link
Copy Markdown
Owner

Summary

  • Human Review Workspace: Created a dedicated review interface for Decks 5, 6, and 7 to visually compare official source drawings (ART-0001 Page 3) against extracted vector geometries.
  • Fail-Closed Dual-Gate Governance: Integrated the review finalization directly with canonical Gatekeeper rules via is_canonical_statement_admitted() in timonelo.evidence.gatekeeper. Geometry acceptance approves visual bounds, but publication is strictly withheld until statement identity admission is confirmed.
  • Reviewer Identity Integrity: Eliminated phantom reviewers (UNSPECIFIED_REVIEWER, human_curator, etc.). Finalization strictly requires an explicit reviewer name.
  • Dev-Only Route & Asset Isolation: The review tool is code-split and accessible only in dev mode (import.meta.env.DEV). frontend/dist/ contains 0 bytes of ART-0001 review imagery.
  • Initial Baseline Cleanliness: All 71 candidate objects on Decks 5, 6, and 7 remain unadmitted in DRAFT / UNKNOWN / PUBLISH_BLOCKED state. Passenger exposure remains strictly limited to Deck 14.

Tests & CI Coverage

  • Frontend Vitest Suite: 91/91 tests passing (including 15 review boundary and invariant tests).
  • Frontend Build: Clean Vite production build with zero review assets bundled in dist/.
  • Backend Test Suite: 899/899 pytest tests passing across 73 test suites.
  • Diff Hygiene: git diff --check is clean (0 whitespace warnings).

@FHoffarth
FHoffarth merged commit 868325a into develop Aug 23, 2026
4 checks passed
@FHoffarth
FHoffarth deleted the feature/public-deck-geometry-review-workspace-v1 branch August 23, 2026 22:13
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