Skip to content

Add repository_customizations table and routes - #5953

Merged
intentionally-left-nil merged 5 commits into
mainfrom
feat/per-repo-customization
Sep 8, 2026
Merged

Add repository_customizations table and routes#5953
intentionally-left-nil merged 5 commits into
mainfrom
feat/per-repo-customization

Conversation

@intentionally-left-nil

@intentionally-left-nil intentionally-left-nil commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

To continue with the work of allowing per-repository settings (AI model, pr reviews etc) this commit creates a table to store the data. It also adds the routes needed to update the data, and wires it into the GithubRepositoryCustomizationsPreview components.

No callsites consume this data yet, except for the admin page.

Summary

  • Fixed updateIntegrationMetadataForOwner to use proper atomic sql operations
  • Added repository_customizations table
  • Changed GitHubRepositoryCustomizationsPreview.tsx to be a stateful component, owning its data and connecting to the backend
  • Added the react code to the integrations view, under the PER_REPO_SETTINGS feature flag

Verification

Visited the /integrations page, updated the values, saw that it was persisted. The orgs page was harder since I didn't have an integration, but it should still work the same

Screen.Recording.2026-09-08.at.12.07.04.PM.mov

@intentionally-left-nil
intentionally-left-nil marked this pull request as ready for review September 8, 2026 19:28
Comment thread apps/web/src/lib/integrations/db/platform-integrations.ts Outdated
@kilo-code-bot

kilo-code-bot Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: 1 Issue Found | Recommendation: Address before merge

Overview

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

WARNING

File Line Issue
apps/web/src/lib/integrations/db/platform-integrations.ts 1033 Omitting integrationId updates every matching integration; GitLab callers still omit it
Files Reviewed (22 files)
  • ENVIRONMENT.md
  • apps/storybook/stories/GitHubRepositoryCustomizations.stories.tsx
  • apps/web/src/components/integrations/GitHubIntegrationDetails.tsx
  • apps/web/src/components/integrations/GitHubRepositoryCustomizationsPreview.test.ts
  • apps/web/src/components/integrations/GitHubRepositoryCustomizationsPreview.tsx
  • apps/web/src/components/integrations/IntegrationDetailPage.tsx
  • apps/web/src/components/integrations/OrganizationGitHubInstallations.tsx
  • apps/web/src/lib/config.server.ts
  • apps/web/src/lib/integrations/db/platform-integrations.test.ts
  • apps/web/src/lib/integrations/db/platform-integrations.ts - 1 issue
  • apps/web/src/lib/integrations/github-apps-service.test.ts
  • apps/web/src/lib/integrations/github-apps-service.ts
  • apps/web/src/lib/integrations/github-repository-settings.test.ts
  • apps/web/src/lib/integrations/github-repository-settings.ts
  • apps/web/src/routers/github-apps-router.test.ts
  • apps/web/src/routers/github-apps-router.ts
  • packages/db/src/migrations/0239_repository_customizations.sql
  • packages/db/src/migrations/meta/0239_snapshot.json
  • packages/db/src/migrations/meta/_journal.json
  • packages/db/src/schema-types.ts
  • packages/db/src/schema.test.ts
  • packages/db/src/schema.ts

Fix these issues in Kilo Cloud


Reviewed by grok-4.6 · Input: 474.6K · Output: 31.9K · Cached: 1.4M

Review guidance: REVIEW.md from base branch main

@pandemicsyn pandemicsyn 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.

lgtm, couple of minor things from the bot

Comment thread apps/web/src/routers/gitlab-router.ts Outdated
Comment thread apps/web/src/components/integrations/GitHubRepositoryCustomizationsPreview.tsx Outdated
To continue with the work of allowing per-repository settings (AI model, pr reviews etc)
this commit creates a table to store the data. It also adds the routes needed to update
the data, and wires it into the GithubRepositoryCustomizationsPreview components.

No callsites consume this data yet, except for the admin page.
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