You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Adds org-scoped dedicated native builder request + status UI under Organization settings
Stores dedicated-builder state in dedicated_builders and preferred pool on build_requests.builder_pool
Private API for request / status / cancel / shared-fallback toggle (billing permissions)
Build request payload prefers the dedicated pool with optional shared fallback
Builds page banner + preferred-pool badges in build history
Ops activation remains service-role SQL (no platform-admin write path)
Motivation (AI generated)
Customers with high native-build volume need a dedicated worker/pool for isolation and queue priority, while still falling back to Capgo shared builders when their worker is busy.
Business Impact (AI generated)
Enables an Enterprise-ready dedicated builder offering: self-serve request flow, clear provisioning/worker status, and fallback controls without blocking day-to-day builds.
Test Plan (AI generated)
Unit tests for dedicated-builder helpers and builder payload routing
Lint / typecheck
CI: Run tests (backend, unit, Playwright, Cloudflare) green
Manual: request dedicated builder as billing admin; verify pending → active UI after ops activation
Manual: toggle shared fallback; confirm preferred-pool column on builds
Visual (AI generated)
Generated with AI
Summary by CodeRabbit
New Features
Added dedicated builder management to organization settings, including request/provisioning, cancelation, worker details, suspension messaging, and shared-pool fallback controls.
Added dedicated builder visibility in the builds dashboard, including new pill indicators for dedicated vs shared pool.
Builds can now be routed to dedicated or shared builder pools, with pool information persisted and sent to the builder.
UI Improvements
Improved build table rendering (status, last error details dialog, and error truncation).
Added dedicated builder banner on app builds pages and updated enterprise plan feature list.
Tests
Added unit coverage for dedicated routing, builder payload shaping, worker status derivation, and public view output.
Let orgs request a dedicated native builder, track provisioning status,
prefer that pool for builds with shared fallback, and surface worker
state in organization settings and build history.
Co-authored-by: Martin DONADIEU <martindonadieu@gmail.com>
Storage and type contracts supabase/migrations/..., src/types/supabase.types.ts, supabase/functions/_backend/utils/supabase.types.ts, cli/src/types/supabase.types.ts
Adds the dedicated_builders table, builder_pool metadata, lifecycle constraints, indexes, RLS policies, and synchronized database typings.
Dedicated routing and build persistence supabase/functions/_backend/utils/dedicated_builder.ts, supabase/functions/_backend/public/build/request.ts, tests/builder-payload.unit.test.ts, tests/dedicated-builder.unit.test.ts
Derives pool routing and worker status, forwards routing to builder jobs, persists the selected pool, and tests routing and view behavior.
Dedicated builder private API supabase/functions/_backend/private/dedicated_builder.ts, supabase/functions/private/index.ts, cloudflare_workers/api/index.ts
Adds authenticated GET, POST, and PATCH handlers for dedicated builder records and mounts them in private API routers.
Settings client and lifecycle UI src/services/dedicatedBuilder.ts, src/pages/settings/organization/DedicatedBuilder.vue
Adds client calls and settings UI for requesting, cancelling, loading, and configuring dedicated builders across lifecycle states.
No successful run was found on main (7e18610) during the generation of this report, so 42b818a was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩
The reason will be displayed to describe this comment to others. Learn more.
Stale comment
Risk: high. Cursor Bugbot did not complete (skipped due to usage limit), so automated review is unavailable. Human review is required for the migration, build routing, and dedicated-builder API/UI changes. Reviewers assigned.
Sent by Cursor Approval Agent: Pull Request Approver External
The reason will be displayed to describe this comment to others. Learn more.
Stale comment
Risk: medium. Cursor Bugbot did not complete successfully (check skipped due to usage limit), so automated review coverage is missing. Human review is required before merge.
Sent by Cursor Approval Agent: Pull Request Approver
The reason will be displayed to describe this comment to others. Learn more.
Stale comment
Risk: high. Cursor Bugbot did not complete successfully (skipped due to usage limit), so automated review coverage is missing. Human review is required for the migration, build routing, and dedicated-builder API/UI changes. Reviewers are already assigned.
Sent by Cursor Approval Agent: Pull Request Approver External
The reason will be displayed to describe this comment to others. Learn more.
Stale comment
Risk: high. Cursor Bugbot did not complete (skipped due to usage limit), so automated review coverage is missing. Human review is still required for the migration, build routing, and dedicated-builder API/UI changes. Reviewers are already assigned.
Sent by Cursor Approval Agent: Pull Request Approver
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@supabase/functions/_backend/private/dedicated_builder.ts`:
- Around line 44-48: Update requireUpdateBilling to throw the quickError result
when the org.update_billing permission check fails, matching the enforcement
behavior of requireReadBilling and ensuring unauthorized callers cannot reach
the POST/PATCH mutation paths.
In `@supabase/migrations/20260723144749_dedicated_native_builder.sql`:
- Line 28: Constrain the platforms column in the dedicated native builder
migration with a database-level CHECK constraint requiring every array element
to be either "ios" or "android". Apply the same constraint to the related
definitions referenced in the additional locations, while preserving the
existing NOT NULL and empty-array default.
- Around line 11-16: Update the ADD CONSTRAINT statement for
build_requests_builder_pool_check to include NOT VALID, avoiding immediate
validation and the blocking table scan while preserving the existing CHECK
expression.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
Push a commit to this branch (recommended)
Create a new PR with the fixes
ℹ️ Review info⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: bb5fdb8a-b7f1-42c1-8472-255cab40143b
📥 Commits
Reviewing files that changed from the base of the PR and between 7e18610 and 0d58455.
Make billing guards explicitly throw, add builder_pool CHECK as NOT VALID
to avoid a blocking scan, and constrain dedicated builder platforms.
Co-authored-by: Martin DONADIEU <martindonadieu@gmail.com>
The reason will be displayed to describe this comment to others. Learn more.
Stale comment
Risk: high. Cursor Bugbot did not complete successfully (skipped due to usage limit), so automated review coverage is missing. Human review is still required for the migration, build routing, and dedicated-builder API/UI changes; reviewers are already assigned.
Sent by Cursor Approval Agent: Pull Request Approver External
The reason will be displayed to describe this comment to others. Learn more.
Stale comment
Risk: high. Cursor Bugbot did not complete (skipped due to usage limit), so automated review coverage is missing. Human review is still required for the migration, build routing, and dedicated-builder API/UI changes. Reviewers are already assigned.
Sent by Cursor Approval Agent: Pull Request Approver
Guard stale org loads, resolve banner org by app, surface invoke error
codes, avoid false idle/busy from preferred-pool counts, and render pool
badges via DataTable renderFunction.
Co-authored-by: Martin DONADIEU <martindonadieu@gmail.com>
The reason will be displayed to describe this comment to others. Learn more.
Stale comment
Risk: high. Cursor Bugbot did not complete successfully (skipped due to usage limit), so automated review coverage is missing. Human review is still required for the migration, build routing, and dedicated-builder API/UI changes; reviewers are already assigned.
Sent by Cursor Approval Agent: Pull Request Approver External
The reason will be displayed to describe this comment to others. Learn more.
Stale comment
Risk: high. Cursor Bugbot did not complete successfully (skipped due to usage limit), so automated review coverage is missing. Human review is still required for the migration, build routing, and dedicated-builder API/UI changes. Reviewers are already assigned.
Sent by Cursor Approval Agent: Pull Request Approver
Preserve the resolved organization when opening settings.
Line 87 navigates without the organization resolved from appId. When this app belongs to a non-selected organization, the banner shows that app’s builder state but the CTA opens the selected organization’s dedicated-builder settings. Retain the resolved orgId and pass/select it through the settings navigation flow.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@src/components/dashboard/DedicatedBuilderBanner.vue` around lines 86 - 87,
Update goToDedicatedBuilder to preserve the organization resolved from appId
when navigating to dedicated-builder settings. Pass the resolved orgId through
the router navigation so the settings view opens for that organization instead
of the currently selected one, using the existing settings navigation parameter
or selection mechanism.
Respect the dedicated builder’s supported platforms.
An active iOS-only dedicated pool is currently selected for Android builds because routing never evaluates row.platforms.
supabase/functions/_backend/utils/dedicated_builder.ts#L49-L58: accept the requested platform and return null when the pool does not support it.
supabase/functions/_backend/public/build/request.ts#L482-L506: pass the validated platform into dedicated-pool routing.
tests/dedicated-builder.unit.test.ts#L51-L89: cover supported-platform routing and unsupported-platform fallback.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@supabase/functions/_backend/utils/dedicated_builder.ts` around lines 49 - 58,
Update toDedicatedPoolRouting to accept the validated requested platform and
return null when row.platforms does not include it, while preserving existing
active-builder checks and routing fields. In
supabase/functions/_backend/public/build/request.ts lines 482-506, pass platform
to the routing call. In tests/dedicated-builder.unit.test.ts lines 51-89, add
coverage for supported-platform routing and unsupported-platform fallback.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/components/tables/BuildTable.vue`:
- Around line 328-337: Add an accessible name to the icon-only button in the
BuildTable render block by providing a descriptive aria-label that tells
screen-reader users it opens the full error details. Keep the existing click
behavior and IconEye rendering unchanged.
In `@src/pages/settings/organization/DedicatedBuilder.vue`:
- Line 45: Update the canReadBilling permission watcher around
loadDedicatedBuilder so it invokes the loader on both true and false
transitions, rather than only when permission is granted. Reuse the existing
no-permission branch in loadDedicatedBuilder to clear dedicatedBuilder and
increment loadToken, ensuring in-flight responses fail the token check after
read access is revoked.
In `@supabase/migrations/20260723144749_dedicated_native_builder.sql`:
- Line 9: Update the migration’s column comments and the line 18 ANY expression
to satisfy SQLFluff: wrap long comment text across lines without changing its
meaning, and format the ANY(...) usage according to the configured SQL style.
Keep the migration behavior and schema unchanged.
---
Outside diff comments:
In `@src/components/dashboard/DedicatedBuilderBanner.vue`:
- Around line 86-87: Update goToDedicatedBuilder to preserve the organization
resolved from appId when navigating to dedicated-builder settings. Pass the
resolved orgId through the router navigation so the settings view opens for that
organization instead of the currently selected one, using the existing settings
navigation parameter or selection mechanism.
In `@supabase/functions/_backend/utils/dedicated_builder.ts`:
- Around line 49-58: Update toDedicatedPoolRouting to accept the validated
requested platform and return null when row.platforms does not include it, while
preserving existing active-builder checks and routing fields. In
supabase/functions/_backend/public/build/request.ts lines 482-506, pass platform
to the routing call. In tests/dedicated-builder.unit.test.ts lines 51-89, add
coverage for supported-platform routing and unsupported-platform fallback.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
Push a commit to this branch (recommended)
Create a new PR with the fixes
ℹ️ Review info⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: 0a5813ad-3aa2-4d66-9b08-64ad3870ee5f
📥 Commits
Reviewing files that changed from the base of the PR and between 0d58455 and 64b9267.
The reason will be displayed to describe this comment to others. Learn more.
Risk: high. Cursor Bugbot did not complete successfully (skipped due to usage limit), so automated review coverage is missing. Human review is still required for the migration, build routing, and dedicated-builder API/UI changes. Reviewers are already assigned.
Sent by Cursor Approval Agent: Pull Request Approver External
The reason will be displayed to describe this comment to others. Learn more.
Risk: high. Cursor Bugbot did not complete successfully (skipped due to usage limit), so automated review coverage is missing. Human review is still required for the migration, build routing, and dedicated-builder API/UI changes; reviewers are already assigned.
Sent by Cursor Approval Agent: Pull Request Approver
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
Summary (AI generated)
dedicated_buildersand preferred pool onbuild_requests.builder_poolMotivation (AI generated)
Customers with high native-build volume need a dedicated worker/pool for isolation and queue priority, while still falling back to Capgo shared builders when their worker is busy.
Business Impact (AI generated)
Enables an Enterprise-ready dedicated builder offering: self-serve request flow, clear provisioning/worker status, and fallback controls without blocking day-to-day builds.
Test Plan (AI generated)
Visual (AI generated)
Generated with AI
Summary by CodeRabbit
New Features
UI Improvements
Tests