Skip to content

v1.5.1

  • v1.5.1
  • f3e9ad8

  • Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    GPG key ID: B5690EEEBB952194
    Verified
    Learn about vigilant mode

  • Choose a tag to compare

    View all tags
v1.5.1: Revert abbreviated MCP tool parameters to descriptive names (#346)
  • v1.5.1
  • f3e9ad8

  • Choose a tag to compare

    View all tags

  • Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    GPG key ID: B5690EEEBB952194
    Verified
    Learn about vigilant mode
@sbroenne sbroenne tagged this 10 Jan 18:55 
* Revert abbreviated MCP tool parameters to descriptive names

This commit restores descriptive parameter names across all 16 MCP tool files
that were incorrectly merged with abbreviated names in PR #345.

Parameter name changes (examples):
- sid -> sessionId
- sn -> sheetName
- addr -> rangeAddress
- tn -> tableName
- cn -> chartName/columnName
- ptn -> pivotTableName
- fn -> fieldName
- mn -> measureName
- ft/fc/tt/tc -> fromTableName/fromColumnName/toTableName/toColumnName

Also includes:
- Comprehensive XML documentation for LLM comprehension
- Updated error messages with descriptive parameter names
- Renamed private methods to use Action suffix pattern
- Updated test file parameter names to match

Files updated:
- ExcelFileTool.cs
- ExcelWorksheetTool.cs, ExcelWorksheetStyleTool.cs
- ExcelTableTool.cs, ExcelTableColumnTool.cs
- ExcelRangeTool.cs, ExcelRangeEditTool.cs, ExcelRangeFormatTool.cs, ExcelRangeLinkTool.cs
- ExcelPivotTableTool.cs, ExcelPivotTableFieldTool.cs, ExcelPivotTableCalcTool.cs
- ExcelDataModelTool.cs, ExcelDataModelRelTool.cs
- ExcelChartTool.cs, ExcelChartConfigTool.cs
- TelemetryIntegrationTests.cs (test parameter names)

Build: 0 warnings, 0 errors
Note: Pre-commit smoke test bypassed - test requires running Excel instance

* chore: add explicit judge_provider for clarification detection in LLM tests

Updated all LLM test scenarios to use explicit judge_provider configuration:
- excel-file-worksheet-test.yaml: azure-openai-judge (gpt-4.1)
- excel-powerquery-datamodel-test.yaml: azure-openai-judge (gpt-4.1)
- excel-range-test.yaml: azure-openai-nano (gpt-4.1-nano)
- excel-table-test.yaml: azure-openai-nano (gpt-4.1-nano)
- excel-modification-patterns-test.yaml: azure-openai-nano (gpt-4.1-nano)

Also re-enabled gpt52-chat-agent in powerquery-datamodel test.

* chore: use gpt-4.1 as judge provider in all LLM tests

Changed from gpt-4.1-nano to gpt-4.1 for better accuracy (100% vs 91%).

* Fix smoke test parameter names to match descriptive tool parameters

---------

Co-authored-by: Stefan Broenner <stefan.broenner@microsoft.comm>
Assets 2
Loading