Skip to content

chore: remove repos from integrations.settings column for github and github-nango integrations (CM-1005)#3882

Merged
themarolt merged 13 commits intomainfrom
fixes-and-repos-removal-CM-1005
Mar 3, 2026
Merged

chore: remove repos from integrations.settings column for github and github-nango integrations (CM-1005)#3882
themarolt merged 13 commits intomainfrom
fixes-and-repos-removal-CM-1005

Conversation

@themarolt
Copy link
Contributor

@themarolt themarolt commented Feb 28, 2026

Note

Medium Risk
Touches GitHub/GitHub-Nango onboarding/mapping and worker stream execution paths and includes a data migration that mutates production integration settings; regressions could break repo selection or syncing if hydration/stripping logic is incomplete.

Overview
GitHub repo configuration is moved out of integrations.settings. A new migration backs up integrations.settings then removes orgs[].repos plus top-level repos/unavailableRepos for github/github-nango integrations (excluding status='mapping').

API/workers now hydrate repos from public.repositories for compatibility. Backend IntegrationRepository._populateRelations repopulates settings.orgs[].repos (and settings.nangoMapping for github-nango) from DB rows, while DAL adds helpers like getReposForGithubIntegration, getReposGroupedByOrg, populateGithubSettingsWithRepos, and stripReposFromGithubSettings.

GitHub mapping/onboarding flow is updated to rely on repositories rows. mapGithubRepos/mapUnifiedRepositories propagate forkedFrom via a map built from the repositories table (or settings on first run), strip repos from settings after writing DB rows, and delay triggering the run-worker until after transaction commit; progress/monitoring and nango worker logic is updated to read desired repos from the repositories table instead of settings.

Small related changes. GitHub integration stream code avoids writing repos back into settings (only persists unavailableRepos), types mark repo fields as deprecated/optional, the LF integration list computes useGitHubNango based on existing integrations/team-user, and the repo CLI gains reset-and-restart plus improved help output.

Written by Cursor Bugbot for commit 6563d46. This will update automatically on new commits. Configure here.

…repositories table

Signed-off-by: Uroš Marolt <uros@marolt.me>
…and some old bugs

Signed-off-by: Uroš Marolt <uros@marolt.me>
Signed-off-by: Uroš Marolt <uros@marolt.me>
@themarolt themarolt requested a review from mbani01 February 28, 2026 18:58
Signed-off-by: Uroš Marolt <uros@marolt.me>
Signed-off-by: Uroš Marolt <uros@marolt.me>
Signed-off-by: Uroš Marolt <uros@marolt.me>
Signed-off-by: Uroš Marolt <uros@marolt.me>
Signed-off-by: Uroš Marolt <uros@marolt.me>
Signed-off-by: Uroš Marolt <uros@marolt.me>
Signed-off-by: Uroš Marolt <uros@marolt.me>
Signed-off-by: Uroš Marolt <uros@marolt.me>
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

@themarolt themarolt merged commit 9f6194f into main Mar 3, 2026
16 checks passed
@themarolt themarolt deleted the fixes-and-repos-removal-CM-1005 branch March 3, 2026 07:40
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