Skip to content

feat(github): redesign organization integrations - #5487

Merged
RSO merged 7 commits into
mainfrom
multi-github/02-integrations-redesign
Aug 26, 2026
Merged

feat(github): redesign organization integrations#5487
RSO merged 7 commits into
mainfrom
multi-github/02-integrations-redesign

Conversation

@RSO

@RSO RSO commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Summary

  • replace the organization GitHub settings card with a focused installation list
  • show connection health, repository scope, selected repositories, and exact row actions
  • keep the personal GitHub identity flow unchanged
  • retain single-install behavior until the next pilot PR
Before After
CleanShot 2026-08-26 at 11 09 38@2x CleanShot 2026-08-26 at 11 08 39@2x

@RSO
RSO marked this pull request as draft August 25, 2026 14:42
Comment thread apps/web/src/components/integrations/GitHubIntegrationDetails.tsx Outdated
Comment thread apps/web/src/components/integrations/GitHubIntegrationDetails.tsx Outdated
@kilo-code-bot

kilo-code-bot Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

Incremental commits restore org callback toasts and integrationId on management actions, and split healthy primary vs status GitHub lookups; previous warnings are fixed with no new high-confidence issues.

Files Reviewed (11 files)
  • apps/web/src/components/integrations/GitHubIntegrationDetails.tsx
  • apps/web/src/components/integrations/OrganizationGitHubInstallations.tsx
  • apps/web/src/lib/cloud-agent/github-integration-helpers.test.ts
  • apps/web/src/lib/cloud-agent/github-integration-helpers.ts
  • apps/web/src/lib/integrations/db/platform-integrations.ts
  • apps/web/src/lib/integrations/github-apps-service.test.ts
  • apps/web/src/lib/security-agent/router/shared-handlers.test.ts
  • apps/web/src/lib/security-agent/router/shared-handlers.ts
  • apps/web/src/routers/admin/oss-sponsorship-router.ts
  • apps/web/src/routers/organizations/organization-auto-triage-router.ts
  • apps/web/src/routers/organizations/organization-security-agent-router.ts
Previous Review Summary (commit 46087d5)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit 46087d5)

Status: 2 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 2
SUGGESTION 0
Issue Details (click to expand)

WARNING

File Line Issue
apps/web/src/components/integrations/GitHubIntegrationDetails.tsx 121 Org branch drops callback outcomes and hijacks other surfaces
apps/web/src/components/integrations/GitHubIntegrationDetails.tsx 455 Org uninstall/refresh/cancel missing integrationId
Files Reviewed (4 files)
  • apps/web/src/components/integrations/GitHubIntegrationDetails.tsx - 2 issues
  • apps/web/src/components/integrations/OrganizationGitHubInstallations.tsx
  • apps/web/src/components/integrations/IntegrationDetailPage.tsx
  • apps/web/src/routers/github-apps-router.ts

Fix these issues in Kilo Cloud


Reviewed by grok-4.6 · Input: 202.3K · Output: 21K · Cached: 924.2K

Review guidance: REVIEW.md from base branch main

@RSO
RSO force-pushed the multi-github/02-integrations-redesign branch from 46087d5 to c9fff23 Compare August 26, 2026 08:57
Base automatically changed from multi-github/01-installation-safety to main August 26, 2026 09:05
@RSO
RSO marked this pull request as ready for review August 26, 2026 09:23
@RSO
RSO merged commit 5b56912 into main Aug 26, 2026
19 checks passed
@RSO
RSO deleted the multi-github/02-integrations-redesign branch August 26, 2026 09:33
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