Skip to content

Conversation

@ChengJiale150
Copy link
Contributor

@ChengJiale150 ChengJiale150 commented Oct 22, 2025

See in #127 .

What's Changed

add Cell and Notebook Model

same as #133

union list_cells and read_cells to read_notebook

  • add param response_format to control the output (brief for list_cells, detailed for read_cells)
  • add notebook_name to read all connected notebook without activate it.
  • add start_index and limit to control the token return by this tool when reading large notebook (>100 cells)
  • change the tests to adapt above feature.

insert_cell

  • use same format API to format output in insert_cell to reduce inconsistence.

overwrite_cell_source

  • better description

execute_cell

  • better code origization and description
  • set default timeout to 90s

insert_execute_code_cell

  • remove tools/insert_execute_code_cell.py and combine it with insert_cell and execute_cell

read_cell

  • union the output format as list to adapt standard MCP MutiModel ouput format.
  • add include_outputs param to control whether to return output.

delete_cell

  • return output add the cell source of deleted cell to avoid delete cell incorrectly and help LLMs to recover.
  • add description to tell LLMs to delete many cells in descending order of their index to avoid index shifting.

@ChengJiale150
Copy link
Contributor Author

ChengJiale150 commented Oct 23, 2025

merge the latest code, and move all same _get_jupyter_ydoc in each tools to utils.py.

@ChengJiale150 ChengJiale150 marked this pull request as ready for review October 23, 2025 09:41
@ChengJiale150
Copy link
Contributor Author

@echarles All changes in #127 are applied, next I will devote to update docs.

@eleonorecharles eleonorecharles merged commit 1ea92f2 into datalayer:main Oct 23, 2025
15 checks passed
@eleonorecharles
Copy link
Member

Thanks @ChengJiale150 !

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants