feat: add support for update report definition endpoint

[smcgowan-smartsheet]

Pull Request

Description

Adding support for update report definition endpoint

Type of Change

• Bug fix
• New feature
• Documentation update
• Code refactoring
• Other (please describe):

What Changes Were Made

• Support for PATCH /reports/{reportId}/definition endpoint
• New update_report_definition() method in Reports class to update report definitions
• New ReportDefinition model with support for filters, grouping, aggregation, and sorting criteria
• New ReportFilterExpression model with recursive structure for complex filter logic
• New ReportFilterCriterion model for individual filter conditions
• New ReportColumnIdentifier model for identifying columns in report criteria
• New ReportGroupingCriterion model for report grouping configuration
• New ReportAggregationCriterion model for report aggregation configuration
• New ReportSortingCriterion model for report sorting configuration
• New ReportAggregationType enum (SUM, AVG, MIN, MAX, COUNT, FIRST, LAST)
• New ReportBooleanOperator enum (AND, OR) for filter expressions
• New ReportFilterOperator enum with 36 operators for filter criteria
• New ReportSystemColumnType enum with report-specific system columns including SHEET_NAME
• Type hints for all new report models and methods
• WireMock integration tests for report definition update endpoint including nested filter validation

Checklist

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• I have updated the relevant files in docs-source/ (see Documentation guidelines)
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes

[roomote-v0]

Rooviewer   See task

Review completed. All previously flagged issues remain resolved in the latest commit.

• Fix alphabetical ordering in smartsheet_enums.rst - PredecessorType should appear before Report* enums
• Use SDK's standard query_params pattern instead of manual string concatenation for updateFilters parameter

Previous reviews

• e66d073: Review #1
• 5d11d01: Review #2
• 58b1032: Review #3
• d2ac9f8: Review #4
• a77fa0b: Review #5
• 5f6ace8: Review #6
• 5f6ace8: Review #7
• 8d438b4: Review #8
• 45cab93: Review #9
• b8e56d7: Review #10
• 571be30: Review #11

_{Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.}

[roomote-v0]

Rooviewer   See task

Review completed. The implementation looks solid and follows established SDK patterns. All models properly implement serialization, use appropriate type hints, and include comprehensive tests. The CHANGELOG has been updated appropriately. The previous alphabetical ordering issue in docs-source/smartsheet_enums.rst has been resolved.

• Fix alphabetical ordering in smartsheet_enums.rst - PredecessorType should appear before Report* enums

_{Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.}

[roomote-v0]

The query parameter is manually appended to the path string instead of using the SDK's standard query_params pattern. Every other method in this SDK uses _op["query_params"]["key"] = value (e.g., line 288: _op["query_params"]["sendEmail"] = send_email). This bypasses the SDK's query parameter processing in smartsheet.py and could cause URL encoding issues with more complex parameter values. The implementation should use _op["query_params"]["updateFilters"] = updateFilters instead of manual string concatenation.

_{Fix it with Roo Code or mention @roomote and request a fix.}