Skip to content

style: hide marimo-pair in wasm, fix opencode prompt#9375

Merged
Light2Dark merged 1 commit into
mainfrom
ms/style-hide-marimo-pair-in-wasm-fix-opencode-prompt
Apr 27, 2026
Merged

style: hide marimo-pair in wasm, fix opencode prompt#9375
Light2Dark merged 1 commit into
mainfrom
ms/style-hide-marimo-pair-in-wasm-fix-opencode-prompt

Conversation

@mscolnick

@mscolnick mscolnick commented Apr 24, 2026

Copy link
Copy Markdown
Contributor
  • hide marimo-pair in wasm
  • fix the opencode copy command
Copilot AI review requested due to automatic review settings April 24, 2026 19:25
@vercel

vercel Bot commented Apr 24, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
marimo-docs Ready Ready Preview, Comment Apr 24, 2026 7:26pm

Request Review

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adjusts the editor’s “Pair with an agent” UX by hiding the action in WebAssembly runtimes and updating the OpenCode terminal command formatting shown in the pairing modal.

Changes:

  • Hide the “Pair with an agent” notebook action when running in WASM (isWasm()).
  • Update the OpenCode prompt command to use opencode --prompt ....

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
frontend/src/components/editor/actions/useNotebookActions.tsx Hides the pairing action in WASM by setting hidden: isWasm().
frontend/src/components/editor/actions/pair-with-agent-modal.tsx Changes the OpenCode command template to opencode --prompt "$(...)".
Comment thread frontend/src/components/editor/actions/pair-with-agent-modal.tsx

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

Architecture diagram
sequenceDiagram
    participant User
    participant Actions as useNotebookActions
    participant Env as Environment (isWasm)
    participant Modal as PairWithAgentModal

    Note over User,Modal: Action Menu Initialization
    
    User->>Actions: Open actions menu
    Actions->>Env: NEW: Check if running in WASM
    Env-->>Actions: return boolean
    
    alt isWasm() is true
        Actions-->>User: Hide "Pair with an agent" option
    else isWasm() is false
        Actions-->>User: Show "Pair with an agent" option
        
        User->>Actions: Click "Pair with an agent"
        Actions->>Modal: openModal()
        
        Note over Modal: User selects "opencode"
        
        Modal->>Modal: CHANGED: getPromptCommand("opencode")
        Note right of Modal: Generates command with --prompt flag
        
        Modal-->>User: Display command for copy/paste
    end
Loading
@mscolnick mscolnick added the bug Something isn't working label Apr 24, 2026
@mscolnick mscolnick requested a review from Light2Dark April 27, 2026 13:23
@Light2Dark Light2Dark merged commit 348d529 into main Apr 27, 2026
38 of 40 checks passed
@Light2Dark Light2Dark deleted the ms/style-hide-marimo-pair-in-wasm-fix-opencode-prompt branch April 27, 2026 13:56
@github-actions

Copy link
Copy Markdown
Contributor

🚀 Development release published. You may be able to view the changes at https://marimo.app?v=0.23.4-dev8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

3 participants