Skip to content

Conversation

@jnfisher
Copy link
Contributor

Added the following tools to the MCP server:
insert_row
insert_cols
delete_rows
delete_cols

@haris-musa
Copy link
Owner

Thanks

This is useful but I am thinking if we can make one tool to fit all 4 of these operations.

I am trying to avoid adding too many tools because it will bloat the context window

We already have 20 or 21 tools

@jnfisher
Copy link
Contributor Author

jnfisher commented Jul 29, 2025

Thanks Haris, I submitted the PR because a tool I am building for my own use needed to manipulate rows (and this worked as expected). No worries if you don't want to integrate the PR.

On the other hand, it could be a reasonable to accept these now and then refactor into a single tool.

Appreciate the feedback!

@haris-musa
Copy link
Owner

Yeah, I think it's a good idea to merge it now and refactor later.

@haris-musa haris-musa merged commit 429a1e2 into haris-musa:main Jul 29, 2025
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