Skip to content

Improve marimo-pair invocationMessage#607

Open
manzt wants to merge 1 commit into
mainfrom
push-ptyymynprnuo
Open

Improve marimo-pair invocationMessage#607
manzt wants to merge 1 commit into
mainfrom
push-ptyymynprnuo

Conversation

@manzt

@manzt manzt commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

As code is executing via the marimo_executeCode tool, we can provide a high-level "invocation message" which summarizes the action in the kernel. We could maybe use a language model to summarize this in the future, but this is a simple first step.

@manzt manzt added the enhancement New feature or request label Jun 18, 2026
Copilot AI review requested due to automatic review settings June 18, 2026 18:28

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Improves the VS Code Language Model Tool invocation spinner message for marimo_executeCode by summarizing what the Python snippet appears to do (particularly when using marimo code-mode context operations), instead of always showing a generic “running code” message.

Changes:

  • Added lightweight parsing to detect marimo code-mode context variable(s) (async with cm.get_context() as ctx:) and infer common actions (create/edit/delete/move/run cells, package add/remove).
  • Added a summarizer that maps detected actions into short, human-readable phrases (e.g., “Editing and running cells”).
  • Updated the tool’s prepareInvocation to use the summarizer output as invocationMessage.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions

github-actions Bot commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Coverage Report

Area Lines Branches Functions
Python 50.15% (545 / 1030) 39.55% (106 / 268)
TypeScript 47.88% (2283 / 4768) 39.22% (905 / 2307) 45.50% (627 / 1378)

TypeScript statements: 47.56% (2376 / 4995)

@manzt manzt force-pushed the push-ptyymynprnuo branch from b06b5a6 to 22ee93c Compare June 22, 2026 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants