Skip to content

Conversation

ChengJiale150
Copy link
Contributor

#70
feat: Unify cell insertion functionality into insert_cell and insert_execute_code_cell

  • Replace 4 scattered cell insertion functions with 2 unified tools:
    • insert_cell: Unified API for inserting any cell type with -1 for append
    • insert_execute_code_cell: Insert and execute code cells with -1 for append
  • Remove deprecated functions: append_markdown_cell, insert_markdown_cell, append_execute_code_cell
  • Add surrounding cells context in insert_cell responses (up to 5 cells above/below)
  • Update tests to use new unified APIs
  • Update documentation to reflect new tool count (11 tools instead of 13)

“jiale.cheng” added 3 commits September 23, 2025 14:59
…execute_code_cell

- Replace 4 scattered cell insertion functions with 2 unified tools:
  * insert_cell: Unified API for inserting any cell type with -1 for append
  * insert_execute_code_cell: Insert and execute code cells with -1 for append
- Remove deprecated functions: append_markdown_cell, insert_markdown_cell, append_execute_code_cell
- Add surrounding cells context in insert_cell responses (up to 5 cells above/below)
- Update tests to use new unified APIs
- Update documentation to reflect new tool count (11 tools instead of 13)
Copy link
Member

@echarles echarles left a comment

Choose a reason for hiding this comment

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

LGTM Thx @ChengJiale150

@echarles echarles merged commit 104a02e into datalayer:main Sep 23, 2025
16 checks passed
@ChengJiale150 ChengJiale150 deleted the feature/unified-cell-insertion branch September 26, 2025 10:39
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