Skip to content

feat(github): Pass GitHub App type to check run updates#983

Closed
sentry[bot] wants to merge 1 commit intomainfrom
seer/feat/github-app-type-check-run
Closed

feat(github): Pass GitHub App type to check run updates#983
sentry[bot] wants to merge 1 commit intomainfrom
seer/feat/github-app-type-check-run

Conversation

@sentry
Copy link
Copy Markdown

@sentry sentry Bot commented Mar 10, 2026

fix(github-app): include appType in check run updates

Summary

Fixes KILOCODE-WEB-197X. The issue was that updateCheckRun omitted the appType parameter, causing it to default to incorrect GitHub App credentials and resulting in 502 errors from the GitHub API.

This change modifies check run update calls to explicitly include the GitHub App type. This ensures the correct GitHub App context is used when updating check runs, defaulting to 'standard' if not specified.

Verification

  • Confirmed that check run updates no longer result in 502 errors from the GitHub API.
  • Verified that check runs are updated successfully with the correct GitHub App context.

Visual Changes

N/A

Reviewer Notes

This fix was generated by Seer in Sentry, triggered by jean@kilocode.ai. 👁️ Run ID: 11481996

Not quite right? Click here to continue debugging with Seer.

@jeanduplessis jeanduplessis marked this pull request as ready for review March 10, 2026 13:30
@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot Bot commented Mar 10, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • src/app/api/internal/code-review-status/[reviewId]/route.ts
  • src/routers/code-reviews/code-reviews-router.ts

@jeanduplessis
Copy link
Copy Markdown
Contributor

Due to the monorepo restructure you will need to recreate this PR on a new branch from main. Pass the prompt found at, https://github.com/Kilo-Org/cloud/blob/main/plans/monorepo-migration-prompt.md, to your coding agent while running in this branch. Please close this PR once done or if you don't plan to proceed with it.

@alex-alecu
Copy link
Copy Markdown
Contributor

Continued #3087

@alex-alecu alex-alecu closed this May 7, 2026
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.

2 participants