Skip to content

fix(slack): accept scalar checkbox values in List responses - #8236

Merged
TheodoreSpeaks merged 1 commit into
stagingfrom
codex/slack-lists-checkbox-responses
Sep 24, 2026
Merged

TheodoreSpeaks merged 1 commit into
stagingfrom
codex/slack-lists-checkbox-responses

Conversation

@TheodoreSpeaks

Copy link
Copy Markdown
Collaborator

Slack Lists can return a scalar checkbox: false after a successful row write. The existing response schema accepts only boolean arrays, causing Create List Item to report failure after creating the row and Read List Items to reject that row afterward.

Accept boolean and boolean-array checkbox responses without changing the provider value, retain strict rejection of malformed values, and update generated output metadata and docs. Regression coverage exercises create, list, and info response transforms for both valid formats.

Validation: reproduced scalar false/true failures before the fix; all 64 focused Lists, Canvas, and Slack resource-block tests pass afterward. Biome, API validation audit, generated metadata/docs checks, and diff checks pass. No local TypeScript check was run.

Live staging: updated custom-bot permissions now work; List create, rename, and empty-list schema reads pass. Row creation/read exposed this bug. Full row CRUD and test-row cleanup remain pending deployment of this fix. Targets staging; not merged or deployed by this PR creation.

@vercel

vercel Bot commented Sep 24, 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 Sep 24, 2026 6:00pm UTC

Request Review

@greptile-apps

greptile-apps Bot commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge; no actionable regression was established.

Summary

The PR accepts scalar booleans as well as boolean arrays in Slack List checkbox responses without changing the returned value.

  • Updates the shared response schema used by create, list, and info.
  • Aligns output metadata and documentation with the mixed response shape.
  • Adds tests for both valid shapes and malformed values.
Diagram
%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[Slack List row response] --> B[Shared field response schema]
  B --> C{Checkbox value}
  C -->|Boolean or boolean array| D[Preserve value in tool output]
  C -->|Malformed| E[Reject response]
Loading

Reviews (1) · Last reviewed commit: "fix(slack): accept scalar checkbox value..."

@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 5 files

Confidence score: 5/5

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

Re-trigger cubic

@TheodoreSpeaks
TheodoreSpeaks merged commit 99b1702 into staging Sep 24, 2026
34 checks passed
@TheodoreSpeaks
TheodoreSpeaks deleted the codex/slack-lists-checkbox-responses branch September 24, 2026 18:07

This branch was successfully deployed

1 active deployment
Preview — f69f9b8e Deployed Sep 24, 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