Repository navigation
Add generic collaborative visual decision cards - #417
Draft
MaggieAppleton wants to merge 8 commits into
Draft
MaggieAppleton wants to merge 8 commits into
MaggieAppleton wants to merge 8 commits into
Conversation
|
The preview deployment for chopin is ready. 🟢 Open app | Open Build Logs | Open Application Logs Last updated at: 2026-10-09 14:25:32 CET |
MaggieAppleton
marked this pull request as ready for review
October 9, 2026 11:22
MaggieAppleton
marked this pull request as draft
October 9, 2026 16:48
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
visual-decision@1contract with bounded numeric and colour controls, immutable definition and artifact revisions, shared edits, and durable version-bound Save.Actual UI
Verification
At
c7a7f905383b3638ed9a355729954a8831890d2d, currentmain(dce9a4fa3d9a0e264c162528746271b31d6ae5db) is merged without rewriting #417. The only direct conflicts were adjacent route additions: this PR's authenticated visual preview route and main's GitHub reference route. Both remain registered. Localbun run ci,bun run types,bun run build, and 79 focused server/editor tests passed. Exact-head CI validation, PostgreSQL tests, browser E2E, and container build passed. The earlier head also passed four PostgreSQL-backed visual-decision browser scenarios.Integration boundary
VisualDecisions.create()is an internal server function that requires a trusted artifact verifier. This PR adds no hosted Planner creation tool, durable pending-request API, publication pickup, production artifact resolver, or MCP saved-result readback. The two bundle resolvers are test-only; without a trusted production resolver, existing cards show Preview unavailable and cannot be saved.