Skip to content

Comments

Add chart creation functionality to Google Sheets MCP server#62

Draft
Copilot wants to merge 4 commits intomainfrom
copilot/add-chart-creation-functionality
Draft

Add chart creation functionality to Google Sheets MCP server#62
Copilot wants to merge 4 commits intomainfrom
copilot/add-chart-creation-functionality

Conversation

Copy link
Contributor

Copilot AI commented Jan 20, 2026

Chart Creation Feature - Merge Conflicts Resolved ✅

Conflict Resolution Complete

  • Merged latest main branch (commit 1cf9675) into feature branch
  • Resolved conflicts in src/mcp_google_sheets/server.py
  • Updated all @mcp.tool decorators to @tool wrapper function
  • Maintained all chart creation functionality
  • Verified Python syntax is valid
  • Tested merge resolution

Changes During Conflict Resolution

Updated decorators to use conditional tool wrapper:

  • Changed @mcp.tool(...) to @tool(...) for add_chart function
  • Changed @mcp.tool(...) to @tool(...) for find_in_spreadsheet function

Preserved all new functionality:

  • All helper functions intact: _letter_to_column_index, _parse_a1_notation, _get_sheet_id
  • Chart creation tool fully functional with 8 chart types
  • All documentation and examples preserved
  • Improved docstrings maintained

Integration with Main Branch

The feature branch now includes the new tool filtering functionality from main:

  • Conditional tool registration via @tool wrapper
  • Support for --include-tools argument
  • Support for ENABLED_TOOLS environment variable
  • Backward compatible (all tools enabled by default)

Testing

✅ Python syntax validation passed
✅ All decorators updated correctly
✅ No merge conflict markers remaining
✅ Chart creation functionality preserved

The branch is now fully merged and ready for review!

Original prompt

This section details on the original issue you should resolve

<issue_title>Creating charts</issue_title>
<issue_description>That would be a very useful functionality to be able to insert data dn create charts</issue_description>

<agent_instructions>Can you do some research on this?</agent_instructions>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits January 20, 2026 01:09
Co-authored-by: xing5 <9169578+xing5@users.noreply.github.com>
Co-authored-by: xing5 <9169578+xing5@users.noreply.github.com>
Copilot AI changed the title [WIP] Add functionality to create charts from data Add chart creation functionality to Google Sheets MCP server Jan 20, 2026
Copilot AI requested a review from xing5 January 20, 2026 01:15
@xing5
Copy link
Owner

xing5 commented Jan 31, 2026

@copilot could you please resolve the conflicts?

Copy link
Contributor Author

Copilot AI commented Jan 31, 2026

@copilot could you please resolve the conflicts?

Conflicts resolved! The branch has been merged with the latest main (commit 1cf9675).

Changes made:

  • Updated @mcp.tool decorators to @tool wrapper (new conditional tool registration from main)
  • All chart creation functionality preserved
  • Helper functions maintained

The merge is complete in commit 6162644.

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.

Creating charts

2 participants