Skip to content

feat(excel): expand workbook operations and align Graph contracts - #9203

Merged
waleedlatif1 merged 6 commits into
stagingfrom
codex/excel-api-coverage
Oct 19, 2026
Merged

waleedlatif1 merged 6 commits into
stagingfrom
codex/excel-api-coverage

Conversation

@waleedlatif1

@waleedlatif1 waleedlatif1 commented Oct 18, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Expand Excel from 7 to 24 block operations with worksheet/table discovery and updates, table row management, sessions, and calculation.
  • Align existing tools with Microsoft Graph: expose formulas and range metadata, correct write requests, preserve existing outputs, and bound follow-up reads.
  • Preserve configured and model-supplied agent inputs, fix SharePoint browsing scopes, regenerate tool metadata and docs, and migrate renamed/unsupported controls.

Type of Change

  • Enhancement

Testing

Validated all Excel tools and block wiring against Microsoft Graph v1.0 documentation. Local lint, all-workspace type-check, 60 audits, and block-registry checks pass. Added provider execution regressions for canonical range/table targets and explicit false header flags. Both CI unit-test shards pass, including all 151 provider execution cases. Tests run in CI only. No live workbook mutations were performed.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing (new tests pass the test-audit authoring gate)
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
docs Ready Ready Preview Oct 19, 2026 12:11am UTC

Request Review

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 18, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 39 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

Comment thread packages/deployment-config/src/integrations.json
Comment thread apps/sim/tools/microsoft_excel/update_table_row.ts Outdated
Comment thread apps/sim/tools/microsoft_excel/shared.ts Outdated
Comment thread apps/sim/tools/microsoft_excel/utils.ts
Comment thread apps/sim/tools/microsoft_excel/format_range.ts Outdated
Comment thread apps/sim/tools/microsoft_excel/list_worksheets.ts
@greptile-apps

greptile-apps Bot commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[High impact] The PR appears safe to merge; no new blocking issue was established.

Summary

Expands Microsoft Excel from 7 to 24 block operations.

  • Excel range tools expose more Graph data and controls.

No new actionable issues were established. The supplied previous threads are unnumbered and were not reposted. Their reported fixes are present in the reviewed code. No tests or live workbook mutations were run.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[Workflow block] --> C[Map inputs for the selected operation]
  B[Agent tool call] --> D[Merge configured and model inputs]
  D --> E[Normalize Excel inputs]
  C --> F[Excel tool]
  E --> F
  F --> G[Microsoft Graph workbook request]
  G --> H[Return structured results]
Loading

Reviews (6) · Last reviewed commit: "fix(excel): default workbook sessions to..." · Reviewed by Greptile

Comment thread apps/sim/tools/microsoft_excel/shared.ts Outdated
Comment thread apps/sim/blocks/blocks/microsoft_excel.ts Outdated
Comment thread apps/sim/blocks/blocks/microsoft_excel.ts Outdated
Comment thread apps/sim/tools/microsoft_excel/shared.ts
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 18, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

Comment thread apps/sim/tools/microsoft_excel/update_table_row.ts

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 39 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/tools/microsoft_excel/format_range.ts Outdated
Comment thread apps/sim/tools/microsoft_excel/sort_range.ts Outdated
Comment thread apps/sim/blocks/blocks/microsoft_excel.ts
Comment thread apps/sim/tools/microsoft_excel/utils.ts Outdated
Comment thread apps/sim/tools/microsoft_excel/shared.ts Outdated
Comment thread apps/sim/tools/microsoft_excel/sort_range.ts Outdated
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 18, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 39 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/tools/microsoft_excel/utils.ts Outdated
Comment thread apps/sim/blocks/blocks/microsoft_excel.ts Outdated
Comment thread apps/sim/tools/microsoft_excel/sort_range.ts Outdated
Comment thread apps/sim/tools/microsoft_excel/format_range.ts Outdated
Comment thread apps/sim/blocks/blocks/microsoft_excel.ts
Comment thread apps/sim/tools/microsoft_excel/sort_range.ts Outdated
@waleedlatif1
waleedlatif1 force-pushed the codex/excel-api-coverage branch from 8779d41 to 878398d Compare October 18, 2026 23:25
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 18, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 39 files

You've manually re-run cubic several times on this PR. Each manual re-review checks the full PR again and counts toward your usage quota. To preserve your usage limits, we recommend letting cubic automatically review new commits.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/tools/microsoft_excel/shared.ts
Comment thread apps/sim/blocks/blocks/microsoft_excel.ts Outdated
Comment thread apps/sim/tools/microsoft_excel/write.ts Outdated
Comment thread apps/sim/blocks/blocks/microsoft_excel.ts
@waleedlatif1
waleedlatif1 force-pushed the codex/excel-api-coverage branch from 878398d to 4726575 Compare October 18, 2026 23:50
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 18, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 40 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

You've manually re-run cubic several times on this PR. Each manual re-review checks the full PR again and counts toward your usage quota. To preserve your usage limits, we recommend letting cubic automatically review new commits.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/blocks/blocks/microsoft_excel.ts
@waleedlatif1
waleedlatif1 force-pushed the codex/excel-api-coverage branch from 4726575 to 182aed9 Compare October 19, 2026 00:06
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 19, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 40 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

You've manually re-run cubic several times on this PR. Each manual re-review checks the full PR again and counts toward your usage quota. To preserve your usage limits, we recommend letting cubic automatically review new commits.

Turn on auto-fix | Re-trigger cubic

@waleedlatif1
waleedlatif1 merged commit 9324ef3 into staging Oct 19, 2026
55 checks passed
@waleedlatif1
waleedlatif1 deleted the codex/excel-api-coverage branch October 19, 2026 03:21

This branch was successfully deployed

1 active deployment
Preview — 182aed96 Deployed Oct 19, 2026 by vercel[bot]
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.

1 participant