Skip to content

fix(webhook): fetch output from DB as safety net in dispatcher#343

Merged
vincentgrobler merged 1 commit intomainfrom
fix/webhook-dispatcher-db-fallback
Apr 9, 2026
Merged

fix(webhook): fetch output from DB as safety net in dispatcher#343
vincentgrobler merged 1 commit intomainfrom
fix/webhook-dispatcher-db-fallback

Conversation

@vincentgrobler
Copy link
Copy Markdown
Collaborator

@vincentgrobler vincentgrobler commented Apr 9, 2026

Description

The dispatcher now fetches the team run output directly from the team_runs table if the caller didn't pass it inline. This guarantees the Trello multi-card output works regardless of how the executor passes the output.

Also added logging at every critical point:

  • dispatchTeamRunWebhooks: logs resolved output length
  • deliverTrello: logs result_full length and parsed section count

This makes the webhook dispatcher self-sufficient rather than depending on every executor to perfectly pass the output.

Related Issue

Type of Change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to change)
  • 📝 Documentation update
  • ♻️ Refactor (no functional changes)
  • 🧪 Test update

How Has This Been Tested?

  • Unit tests
  • Manual testing
  • Browser testing (specify browsers)

Screenshots (if applicable)

Checklist

  • My code follows the project's coding standards
  • I have run npm run lint and there are no errors
  • I have run npm run test and all tests pass
  • I have added tests for my changes (if applicable)
  • I have updated the documentation (if applicable)
  • My source files have AGPL licence headers
  • My changes generate no new TypeScript errors

The dispatcher now fetches the team run output directly from the
team_runs table if the caller didn't pass it inline. This guarantees
the Trello multi-card output works regardless of how the executor
passes the output.

Also added logging at every critical point:
- dispatchTeamRunWebhooks: logs resolved output length
- deliverTrello: logs result_full length and parsed section count

This makes the webhook dispatcher self-sufficient rather than
depending on every executor to perfectly pass the output.
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 9, 2026

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

Project Deployment Actions Updated (UTC)
crewform Ready Ready Preview, Comment Apr 9, 2026 0:35am

@vincentgrobler vincentgrobler merged commit 1b6bb6b into main Apr 9, 2026
8 checks passed
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