Skip to content

Consolidate Extensions domain ownership in API - #186

Merged
admdly merged 8 commits into
mainfrom
codex/consolidate-extensions-domain-api
Aug 5, 2026
Merged

Consolidate Extensions domain ownership in API#186
admdly merged 8 commits into
mainfrom
codex/consolidate-extensions-domain-api

Conversation

@admdly

@admdly admdly commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Summary

Consolidates the Extensions domain under the API Worker, including the users projection and the complete D1 migration chain.

Changes

  • Add a self-contained API migration baseline for the legacy catalogue and users schema, plus the deleted_at tombstone migration.
  • Model the complete API-owned users table and remove the external-table placeholder.
  • Add authenticated account projection, identity synchronization, profile, deletion, owner-profile, and owned-extension operations.
  • Enforce active-user authorization for domain mutations and preserve OIDC reactivation of deleted subjects.
  • Make account deletion transactional and guarded, including pending-work rejection, PII clearing, disposable ownership cleanup, and audit-history preservation.
  • Expose the non-identifying unclaimed developer field publicly.
  • Keep existing v1/v2 HTTPS endpoints and DB_EXTENSIONS binding.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 5, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
api 62b3d4a Commit Preview URL

Branch Preview URL
Aug 05 2026, 03:37 PM

@admdly admdly self-assigned this Aug 5, 2026
@admdly
admdly marked this pull request as ready for review August 5, 2026 09:32

@cubic-dev-ai cubic-dev-ai Bot 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.

All reported issues were addressed across 26 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread src/services/extensions/v2/index.ts
Comment thread src/services/extensions/v2/db/migrations/0000_bootstrap_users.sql
Comment thread src/services/extensions/v2/account-routes.ts
Comment thread src/services/extensions/v2/index.ts
Comment thread src/services/extensions/v2/developer-profile-routes.ts
Comment thread src/services/extensions/v2/public-extensions-routes.ts Outdated
Comment thread src/services/extensions/v2/public-extensions-routes.ts Outdated
Comment thread src/services/extensions/v2/public-extensions-routes.ts Outdated
Comment thread src/services/extensions/v2/account-routes.ts
Comment thread src/services/extensions/v2/public-extensions-routes.ts Outdated

@cubic-dev-ai cubic-dev-ai Bot 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.

All reported issues were addressed across 17 files (changes from recent commits).

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread src/services/extensions/v2/index.ts

@cubic-dev-ai cubic-dev-ai Bot 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.

0 issues found across 1 file (changes from recent commits).

Requires human review: Adds D1 schema migrations (0000 bootstrap, 0019 tombstone) and moves DB ownership across repos, plus auth/authorization and API-contract changes (active-account enforcement, identity-sync endpoint, new public unclaimed field). These data and security tradeoffs need human sign-off.

Re-trigger cubic

@cubic-dev-ai cubic-dev-ai Bot 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.

All reported issues were addressed across 8 files (changes from recent commits).

Tip: Review your code locally with the cubic CLI to iterate faster.

Re-trigger cubic

Comment thread src/services/extensions/v2/interfaces.ts Outdated
Comment thread src/services/extensions/v2/index.ts Outdated

@cubic-dev-ai cubic-dev-ai Bot 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.

0 issues found across 4 files (changes from recent commits).

Requires human review: Auto-approval blocked by 1 unresolved issue from previous reviews.

Re-trigger cubic

@cubic-dev-ai cubic-dev-ai Bot 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.

0 issues found across 3 files (changes from recent commits).

Requires human review: Consolidates domain ownership with new DB migrations, schema changes, and auth middleware; human review required for architectural and operational tradeoffs.

Re-trigger cubic

@admdly
admdly merged commit 1fa96f1 into main Aug 5, 2026
9 checks passed
@admdly
admdly deleted the codex/consolidate-extensions-domain-api branch August 5, 2026 15:42
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.

1 participant