[codex] Add Builder skill registry canvas run wedge#4550
Draft
1GMedia wants to merge 8 commits into
Draft
Conversation
Contributor
|
Hey @1GMedia — thanks for opening this draft PR. The Builder wedge is clearly scoped here, and the Since this is still in draft, I'll stay out of the review lane for now. Before you flip it to ready, please attach the Canvas screenshots referenced in the PR body so product/design can review the user-facing flow without recreating it locally. 💡 To drive this PR to merge hands-free, paste this to your AI coding agent (Claude Code / Codex / opencode / Cursor …): |
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.
Why
This PR packages the first Builder v1 wedge: imported marketing/playbook skills can be registered, attached to Open Design routines, started from the canvas, projected into a Builder Run Ledger, and controlled through approval placeholders.
The pain being addressed is that the Ploy-style skill library and canvas planning work was previously only product documentation and scratch context. This makes it a reviewable Open Design implementation slice without introducing the later SaaS, Mastra, Astro, Daytona, Orgo, billing, publishing, or outreach scopes.
What users will see
Projects now have a Canvas Run panel showing compact Skill, Process, and Run cards for skill-backed automation. Users can click Run Skill to start a registered skill-backed routine, watch the Run Card/timeline refresh from the Builder Run Ledger, and approve or reject requested placeholder approvals when present.
The CLI also gains embeddable surfaces for the same wedge:
od skills ...for Skill Registry inspection/validation andod run-ledger ...for Builder process/run/event/approval inspection plusod run-ledger run-skill.Surface area
apps/weborapps/desktop(including Electron menu bar)odsubcommand or flag, newtools-dev/tools-pack/tools-prflag, or newOD_*env var/api/*endpoint, new SSE event, or changed shape inpackages/contractsskills/,design-systems/,design-templates/, orcraft/, or change to the skills protocolTRANSLATIONS.mdfor the locale workflow)package.json(dependenciesordevDependencies); workspace-packagepackage.jsonfiles are out of scope. Include a paragraph on what we get vs. what bytes we ship (seeCONTRIBUTING.md→ Code style)Screenshots
Focused browser smoke screenshots were captured locally and should be attached before moving this out of draft:
/Users/gavinbasuel/conductor/agent-bus/Builder_v1/artifacts/1-hardening-project-loaded.png/Users/gavinbasuel/conductor/agent-bus/Builder_v1/artifacts/1-hardening-canvas-before-run.png/Users/gavinbasuel/conductor/agent-bus/Builder_v1/artifacts/1-hardening-canvas-after-run.pngBug fix verification
Validation
fnm exec --using 24 pnpm --filter @open-design/contracts typecheckfnm exec --using 24 pnpm --filter @open-design/contracts test— 23 files / 164 testsfnm exec --using 24 pnpm --filter @open-design/daemon typecheckfnm exec --using 24 pnpm --filter @open-design/web typecheckfnm exec --using 24 pnpm exec vitest run -c vitest.config.ts tests/skills.test.ts tests/routine-routes.test.ts tests/builder-run-ledger-routes.test.ts— 3 files / 45 testsfnm exec --using 24 pnpm guardgit diff --check c87934d..HEADgit diff --check3068/ daemon7468with mock-backedcodex: Canvas Run opened; Skill/Process/Run cards rendered; Run Skill startedroutine-run-e69bd306-1a27-4e4f-8001-9fa5d2442606; Run Ledger projected completed process/run plus 4 normalized events.Notes
This deliberately keeps Astro, Daytona, Orgo, SaaS auth, billing, publishing, outreach sends, paid compute, and Mastra execution out of scope. A non-blocking
/api/community/discord502 was observed during browser smoke; it is unrelated to the Builder wedge routes.