Skip to content

feat(doc-approval): Spec 001/002 event-UI parity across shells - #261

Closed
enricopiovesan wants to merge 1 commit into
mainfrom
cursor/ticket-event-ui-parity-doc-approval-apply
Closed

feat(doc-approval): Spec 001/002 event-UI parity across shells#261
enricopiovesan wants to merge 1 commit into
mainfrom
cursor/ticket-event-ui-parity-doc-approval-apply

Conversation

@enricopiovesan

Copy link
Copy Markdown
Contributor

Summary

  • Apply Spec 001 presentation-state and Spec 002 capability-progress mapping across doc-approval shells (web via event-ui-conformance, Rust core for CLI/GTK, Swift, Android, Windows).
  • Surface presentation / active-capability fields on host run results and light UI chrome without inventing business fields in the UI layer.

Project Item

ticket-id: event-ui-parity-doc-approval

Definition of Done

  • Language-equivalent Spec 001/002 mappers for web, Rust, Swift, Kotlin, and C#
  • Host run results expose presentation state / progress / active capability
  • Mapper unit tests cover happy / blocked / idle paths
  • UI remains a rendering layer

Validation

  • npm run test -w packages/event-ui-conformance
  • npm run test -w apps/doc-approval/web-react
  • npm run typecheck -w apps/doc-approval/web-react
  • cargo test -p doc-approval-core-rs (from apps/doc-approval)

Architecture boundary

UI shells only map and display public embedder event streams; doc-approval business fields continue to come from the Traverse runtime.

Made with Cursor

@enricopiovesan
enricopiovesan enabled auto-merge (squash) August 23, 2026 02:18
Use @unchecked Sendable for Any-backed mapper payloads and stop assigning
private(set) view-model fields from tests so PresentationMapperTests compile.

Co-authored-by: Cursor <cursoragent@cursor.com>
@enricopiovesan
enricopiovesan force-pushed the cursor/ticket-event-ui-parity-doc-approval-apply branch from d3fa9e2 to 2dd62f5 Compare August 23, 2026 02:18
@enricopiovesan

Copy link
Copy Markdown
Contributor Author

Superseded by #260 (already merged on main).

auto-merge was automatically disabled August 23, 2026 02:19

Pull request was closed

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