fix(copilot): declare item schemas on table tool array params

[j15z]

Summary

• updates, rows, order, columnNames, and disabledTagIds were declared in the tool catalog as arrays with no item schema. The provider-path sanitizer fills a missing items with {type: "string"}, so the model was told batch_update_rows takes a list of strings and sent [] or ["rowId", "data"], which the executor then crashed on (Object.entries(undefined)) and masked as a generic table failure
• sync the mothership catalog with item schemas on every array param, so the router's Ajv input validation now rejects a malformed call with an error the model can act on. Generated files change by items additions only; the description text Sim already carries is preserved
• validate updates / rows element shape in the table tool as a last line, with messages naming the bad index and the expected shape
• match table operation verbs as tokens in tool-call titles, so batch_update_rows renders as "Updating rows" instead of the generic "Editing rows"

Companion: simstudioai/mothership#473

Type of Change

• Bug fix

Testing

• vitest across the copilot tool suites (71 files, 1027 tests), including new tests that pin the malformed payloads at both the router and tool layers
• bun run type-check, bun run lint, bun run check:audits, bun run docs-manifest:check

Checklist

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

[vercel]

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

1 Skipped Deployment

Project	Deployment	Actions	Updated
docs	Skipped		Sep 3, 2026 9:42pm UTC

[github-actions]

⚠️ Cross-repo companion check

One or more companion PRs aren't merged into staging yet. Merging this without them will leave copilot and sim out of sync — merge them in lockstep.

• ❌ simstudioai/mothership#473 — OPEN, not merged (targets staging) — fix(tools): declare items on every catalog array parameter

[greptile-apps]

Greptile Summary

The PR strengthens Copilot table-tool contracts and executor validation so malformed array elements produce actionable input errors rather than downstream crashes.

• Adds item schemas for table, query-order, column-name, and knowledge-tag arrays in both generated schema representations.
• Validates batch insert and update elements before invoking table services.
• Classifies compound table operation IDs into more accurate tool-call titles.
• Adds router-, executor-, and display-level regression tests.

Confidence Score: 5/5

The PR appears safe to merge with no actionable correctness or security defects identified.

The added schemas agree with the documented executor contracts, defensive validation prevents malformed elements from reaching table services, and every current catalog operation is classified correctly by the title logic.

Important Files Changed

Filename	Overview
apps/sim/lib/copilot/generated/tool-catalog-v1.ts	Adds catalog item definitions matching the documented array element contracts.
apps/sim/lib/copilot/generated/tool-schemas-v1.ts	Mirrors the item constraints into runtime Ajv schemas so malformed elements are rejected at routing.
apps/sim/lib/copilot/tools/server/table/user-table.ts	Adds defensive shape checks for batch row inserts and updates before table-service calls.
apps/sim/lib/copilot/tools/tool-display.ts	Matches operation verbs as underscore-delimited tokens, correctly covering existing compound operation IDs.
apps/sim/lib/copilot/tools/server/generated-schema.test.ts	Covers runtime schema rejection of malformed update, row, and sort-order array elements.
apps/sim/lib/copilot/tools/server/table/user-table.test.ts	Covers executor-level malformed batch payload handling and both supported update formats.
apps/sim/lib/copilot/tools/tool-display.test.ts	Covers title classification for representative compound table operations.

_{Reviews (1): Last reviewed commit: "fix(copilot): declare item schemas on ta..." | Re-trigger Greptile}

[cubic-dev-ai]

No issues found across 7 files

Confidence score: 5/5

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

_{Re-trigger cubic}