Skip to content

fix(update_column): sanitize status labels and normalize tool inputs - #460

Open
almogru wants to merge 2 commits into
masterfrom
fix/update-column-status-settings-sanitization
Open

fix(update_column): sanitize status labels and normalize tool inputs#460
almogru wants to merge 2 commits into
masterfrom
fix/update-column-status-settings-sanitization

Conversation

@almogru

@almogru almogru commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

Summary:

  • Fixes update_column failures for status/dropdown label updates.
  • Handles REVISION_MISMATCH by retrying once with a fresh column revision.
  • Truncates status label descriptions to 80 chars.
  • Converts dropdown label additions into MODIFY_LABELS actions.

New fixes for INVALID_INPUT:

  • Strips id from status label updates to avoid invalid/new-label id errors.
  • Coerces status settings received in UI object shape (labels_positions_v2/labels_colors/done_colors) into API-required { labels: [...] } and maps labels_colors.var_name to status color enums.

Safety:

  • update_column explicitly rejects itemId/value usage (use change_item_column_values instead).

Test plan:

  • npm test -- --testPathPattern=update-column-tool

Made with Cursor

almogru and others added 2 commits August 19, 2026 11:55
Co-authored-by: Cursor <cursoragent@cursor.com>
Per review feedback: restore required revision (no auto-fetch), preserve
status label ids for existing labels, sharpen tool descriptions, and revert
unrelated fetch-file-content-tool changes from this PR.

Co-authored-by: Cursor <cursoragent@cursor.com>
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