[waleedlatif1]

Summary

• Set Conversation Tags no longer wipes every tag when tags is omitted: the param is required (and required in the block for that operation), and [] is the explicit way to remove every tag
• Update Custom Fields rejects non-object entries like [null] with the tool's validation error instead of a raw TypeError
• Help Scout IDs must be positive and canonical (no 0, no leading zeros); one parseIdText check now backs the tools, selector, webhook handler, and connector instead of four copies
• Trigger person outputs (assignee, createdBy, primaryCustomer) document their fields, including name
• Agent Replied trigger copy says "an agent replies"
• Help Scout app-credential guidance moves to the App ID field hint, since the connect modal only renders helpText under an orgId field; documented that on the descriptor type
• Addresses the Help Scout review threads on the v0.9.30 release PR

Type of Change

• Bug fix

Testing

• lib/webhooks/providers/helpscout.test.ts and blocks/blocks.test.ts pass
• Exercised the tags, custom-field, ID, and created-webhook-ID parsers directly against the edge cases (omitted/empty/[]/JSON tags, [null], 0, 007, unsafe integers, Resource-ID vs Location)
• lint, block-registry check, unused-exports check; regenerated tool metadata and integration docs

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]

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

1 Skipped Deployment

Project	Deployment	Actions	Updated
docs	Skipped		Oct 21, 2026 4:14am UTC

[cubic-dev-ai]

All reported issues were addressed across 14 files

<sub>Heads up: you’re close to your flex budget. Increase your flex budget so reviews don’t pause.

Turn on auto-fix | Re-trigger cubic</sub>

[waleedlatif1]

@greptile

[waleedlatif1]

@cubic-dev-ai review this PR

[cubic-dev-ai]

@cubic-dev-ai review this PR

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

[waleedlatif1]

@greptile

[waleedlatif1]

@cubic-dev-ai review this PR

[cubic-dev-ai]

@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]

All reported issues were addressed across 14 files

<sub>Heads up: you’re close to your flex budget. Increase your flex budget so reviews don’t pause.

Turn on auto-fix | Re-trigger cubic</sub>

[greptile-apps]

Confidence Score: 5/5

[Medium impact] The PR appears safe to merge; both previous tag-parsing issues are fixed.

Summary

This PR makes tag clearing explicit, shares positive-ID checks across Help Scout code, rejects invalid custom-field entries, and corrects setup and trigger descriptions.

• Set Conversation Tags needs an explicit tag list to change a conversation's tags.
• Help Scout entry points accept only positive, canonical IDs.
• Update Custom Fields checks each entry before using it.
• Help Scout triggers and credential setup show clearer details.

_{Reviews (3) · Last reviewed commit: "fix(helpscout): keep bracketed tags on t..." · Reviewed by Greptile}

[waleedlatif1]

@greptile

[waleedlatif1]

@cubic-dev-ai review this PR

[cubic-dev-ai]

@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]

No issues found across 14 files

Confidence score: 5/5

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

<sub>Heads up: you’re close to your flex budget. Increase your flex budget so reviews don’t pause.

Turn on auto-fix | Re-trigger cubic</sub>