Skip to content

Consume typed Sentry signals through ChatKit conversion - #16

Merged
danielblignaut merged 6 commits into
mainfrom
codex/chatkit-sentry-signal-conversion
Aug 2, 2026
Merged

danielblignaut merged 6 commits into
mainfrom
codex/chatkit-sentry-signal-conversion

Conversation

@danielblignaut

@danielblignaut danielblignaut commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • consume Sentry issue-created signals through convertToAiSdkMessages with a strongly typed callback
  • allow the remediation bot to run from either a Sentry signal or an ordinary chat prompt
  • remove raw history polling and retry delays from the remediation bot
  • configure ChatKit request timeouts at the endpoint boundary
  • use the stable @trytilde/harness-sdk-vercel-ai-node@0.2.0 release
  • keep generated package archives out of source control

Validation

  • sentry bot: pnpm typecheck, pnpm lint, pnpm test
  • code review bot: pnpm typecheck, pnpm lint

Dependency

Uses the merged harness SDK releases from trytilde/harness-sdk#17, #10, and #18.

@danielblignaut
danielblignaut marked this pull request as ready for review August 2, 2026 20:14
@danielblignaut
danielblignaut merged commit 396e845 into main Aug 2, 2026
2 checks passed
@danielblignaut
danielblignaut deleted the codex/chatkit-sentry-signal-conversion branch August 2, 2026 20:14
@greptile-apps

greptile-apps Bot commented Aug 2, 2026

Copy link
Copy Markdown

Greptile Summary

The PR upgrades the ChatKit integration to consume typed Sentry signals directly and allows remediation from ordinary chat prompts.

  • Converts issue-created signals into typed AI SDK user messages.
  • Supplies session history and current messages to both agent endpoints.
  • Moves request timeout handling to chatKitEndpoint.
  • Removes remediation-side history polling and retry delays.
  • Upgrades the Vercel AI harness SDK to 0.2.0 and ignores generated package archives.

Confidence Score: 5/5

The PR appears safe to merge, with no concrete changed-code defect established.

The typed signal conversion, endpoint-level timeout handling, and expanded chat-triggered flow are internally consistent, and the investigated message-delivery and authorization concerns were either explicitly intended or lacked a reachable contradictory contract.

Important Files Changed

Filename Overview
sentry-remediation-bot/app/api/sentry-remediation/route.ts Replaces raw signal polling with typed ChatKit conversion, supports chat-triggered runs, and delegates timeout cancellation to the endpoint wrapper.
sentry-remediation-bot/lib/remediation/signal.ts Converts typed issue-created signals into AI SDK user messages with fallbacks for optional Sentry fields.
code-review-bot/app/api/code-review/route.ts Moves timeout configuration into chatKitEndpoint while preserving request cancellation and cleanup behavior.
sentry-remediation-bot/package.json Upgrades the Vercel AI harness SDK dependency to the stable 0.2.0 release.
code-review-bot/package.json Upgrades the Vercel AI harness SDK dependency to the stable 0.2.0 release.

Sequence Diagram

sequenceDiagram
    participant T as Tilde ChatKit
    participant E as Agent endpoint
    participant C as convertToAiSdkMessages
    participant M as Remediation model
    participant R as MCP and sandbox tools
    T->>E: Signed signal or chat request
    E->>T: Load session history
    E->>C: History plus current messages
    alt Sentry issue-created signal
        C->>C: Convert typed signal to user task
    end
    C-->>E: UI messages
    E->>M: Stream model messages
    M->>R: Investigate and remediate
    M-->>T: Stream response
    E->>R: Close resources
Loading

Reviews (1): Last reviewed commit: "chore: use stable harness SDK release" | Re-trigger Greptile

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