Skip to content

fix(chatwoot): fix nullish content field - #243

Merged
wojtekfarmery merged 3 commits into
masterfrom
wf/fix-chatwoot-content-nullable
Jul 27, 2026
Merged

fix(chatwoot): fix nullish content field#243
wojtekfarmery merged 3 commits into
masterfrom
wf/fix-chatwoot-content-nullable

Conversation

@wojtekfarmery

Copy link
Copy Markdown
Contributor

This PR fixes an issue described in https://linear.app/botpress/issue/SHK-1588/pluschatwoot-agent-side-hitl-handback-crashes-content-field-required

The issue is caused by misalignment with Chatwoot API where ther content field is not always passed. We validate it as non-optional and this causes a lot of errors

Copilot AI review requested due to automatic review settings July 27, 2026 16:55

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Your trial has ended. Reactivate Greptile to resume code reviews.

@linear-code

linear-code Bot commented Jul 27, 2026

Copy link
Copy Markdown

SHK-1588

Copilot AI 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.

Pull request overview

This PR addresses a Chatwoot webhook ingestion crash by aligning the webhook payload schema with the Chatwoot API behavior where content may be omitted.

Changes:

  • Allow content to be missing/undefined in the Chatwoot webhook Zod schema (nullish()).
  • Add a pnpm-lock.yaml for the Chatwoot integration.
  • Update the Intercom HITL pnpm-lock.yaml dependency set (now includes turndown).

Reviewed changes

Copilot reviewed 1 out of 4 changed files in this pull request and generated 1 comment.

File Description
integrations/intercom-hitl/pnpm-lock.yaml Updates locked dependencies (includes turndown entries).
integrations/chatwoot/src/misc/types.ts Makes content optional/nullable in the webhook payload schema to prevent validation failures.
integrations/chatwoot/pnpm-lock.yaml Adds a lockfile for the Chatwoot integration dependency graph.
Files not reviewed (1)
  • integrations/intercom-hitl/pnpm-lock.yaml: Generated file

Comment thread integrations/intercom-hitl/pnpm-lock.yaml Outdated

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Your trial has ended. Reactivate Greptile to resume code reviews.

@wojtekfarmery
wojtekfarmery merged commit b8a1114 into master Jul 27, 2026
4 checks passed
@wojtekfarmery
wojtekfarmery deleted the wf/fix-chatwoot-content-nullable branch July 27, 2026 17:08
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.

4 participants