Repository navigation
Conversation
Wire Retry transcript, honest empty Events, skip/unknown stack checks, review_action cards, and Act on this run into the IDE shell so pBsT6v-ciO8 is no longer a dead rail. Co-authored-by: KK <kk-agent@users.noreply.github.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
🔍 PR Validation✅ Current validation passed. |
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Keep #2437 Ship this ZIP CTA and skip/unknown stack rows. Keep this branch's Events honesty, pack workbench, Act on this run, and transcript-tab retry. Deduplicate toolbar Retry and Tools stack lists. Co-authored-by: KK <kk-agent@users.noreply.github.com>
github-actions
Bot
added
documentation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b9f78b4441
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| statusLabel: stackCheckStatusLabel(status), | ||
| }; | ||
| }); | ||
| const actionCards = (workflowActions?.actions ?? []).map((action) => studioActionCard(action)); |
There was a problem hiding this comment.
When a user completes Act for pack A and then selects stored pack B—or changes packs while A is polling—workflowActions remains associated only with component state, so this mapping renders A's cards in B's Intent tab; the existing export path also consumes those stale actions and can package them with B. Clear/cancel Act state when the selection changes and discard late results whose URL/run does not match the current selection.
AGENTS.md reference: AGENTS.md:L70-L75
Useful? React with 👍 / 👎.
| const outputStackChecks = stackChecks.map((item) => { | ||
| const status = stackCheckStatus(item, completedChecks, 'anonymous'); |
There was a problem hiding this comment.
For a signed-in user whose pack names a runnable Vercel, Shopify, or GitHub check, this constant forces stackCheckStatus down the anonymous branch, so the newly rendered Tools panel incorrectly says the check was skipped because the user is signed out rather than showing its pending or completed state. Pass the authenticated viewer state instead of hard-coding anonymous.
Useful? React with 👍 / 👎.
| {showAgentWorkflowUi ? ( | ||
| <button | ||
| type="button" | ||
| onClick={() => void act()} | ||
| disabled={actBusy || busy || !getYouTubeId(url || selected?.url || '')} |
There was a problem hiding this comment.
With agentWorkflowUi enabled, merely typing a valid URL (or choosing Sample) enables this button even when selected is undefined; act() then submits a URL-only payload directly to startVideoToActions, bypassing runAnalysis's emitVideoPack and stored-pack path and producing actions without the hashed Video Pack the Studio workflow requires. Gate the CTA on a matching analyzed selection rather than a parseable URL alone.
AGENTS.md reference: AGENTS.md:L8-L14
Useful? React with 👍 / 👎.
| promotePack ? ( | ||
| <PackWorkbench | ||
| architecture={packFormation.architecture} | ||
| artifacts={packFormation.artifacts} | ||
| onExport={() => void exportPkg()} |
There was a problem hiding this comment.
For an empty-events pack that contains stack.tools but no architecture or artifacts, studioPromotePackWorkbench still promotes this panel and the empty-state copy says the stack is shown below, but only architecture and artifacts are passed, leaving an otherwise blank “From this pack” card with an Export button. Pass the tools into the workbench (or direct the copy to the Tools tab) so tool-only packs retain their advertised pack-quality content.
AGENTS.md reference: AGENTS.md:L34-L39
Useful? React with 👍 / 👎.
✅ E2E Test Results: ALL TESTS PASSED
Test Output |
This branch was successfully deployed
2 participants
Canonical issue
Fixes #2435.
Hayden / AXIOM dogfood URL: https://uvai.io/studio?video=https://www.youtube.com/watch?v=pBsT6v-ciO8
Main merge: this branch is merged with
origin/mainthrough #2437 (feat(web): restore Studio Ship this playflow), which already closed #2435. Remaining delta is Events honesty, pack workbench, flag-gated Act, and softened paste copy on top of Ship this.Outcome
On a stored pack like
pBsT6v-ciO8(transcript + architecture + artifacts + stack.tools, noeventsfield):agentWorkflowUiis on (flag still defaults false).review_actionsay Review this result / Needs review.bottom-4) stay.Scope
Risk
Verification
Head SHA:
c1ccfae77886293a70eb210ef5b0bb0acfff9899origin/main(feat(web): restore Studio Ship this playflow #2437). Conflicts were complementary (Ship this + Events/Act), not opposing intents.npm --workspace=apps/web exec vitest runon playflow + three-panel + build-live + job-strip + pipeline-status + official-templates + unification → 9 files, 62 passedProduction evidence
UI-only. Preview on Vercel
v0-uvaiafter CI. Do not treat the live pack GET as a production Studio E2E.Agent handoff
Fixes #2435) — note: Restore Studio playflow CTAs after IDE rebuild (B/E/F/G) #2435 is already closed by feat(web): restore Studio Ship this playflow #2437; the issue-closer check may fail closed on an already-closed issueQA on pBsT6v-ciO8
/studio?video=https://www.youtube.com/watch?v=pBsT6v-ciO8.agentWorkflowUi: Act on this run.