[saddlepaddle]

Decisions from Satya's migration review of #7548.

One migration. The six incremental migrations sandbox-v2 accumulated (boot stamps added then dropped, position added then dropped, a cascade rebuilt) are regenerated as one, 0115_cloud_environments_repositories, before any reach main; a second drizzle-kit generate finds no changes. The dev Neon branch's journal was rewritten to the single entry after its six recorded hashes matched the six old files and its schema matched the new migration (migrate is a no-op there). This PR's preview Neon branch, which had run the old six, was deleted so CI recreates it from production.

Boot timing leaves the row. No stamp columns. The desktop's cloud_workspace_opened event, its create-start record, the terminal attach notifier and the measure script are removed (the P0 study is done; its numbers are in the start-time plan). The provision job is one Sentry transaction with a span per stage (claim, create, settle), sampled by transaction name so nothing else in the API is traced, flushed by the job since it runs after its request has answered. No fallback: whether it arrives is checked after the production deploy.

No repository order. No position column. Repositories read alphabetically everywhere; the workspace opens on the environment's config location, else the first by name; cloudWorkspace.repositories marks that one primary. Opening multi-repository workspaces at /workspace with a sidebar repository picker is a recorded TODO.

No environment hooks override. environments.hooks is dropped with the identity's SUPERSET_SANDBOX_HOOKS: its only value read back was the internal environment's start, which is now the monorepo's cloud-only "start": ["superset-dev-stack"] in .superset/config.json; the release still runs internal-setup.sh itself and builds its golden from SUPERSET_INTERNAL_BRANCH (default main). A redesign of config.json for cloud is a recorded TODO.

No stored checkout branch. cloud_workspace_repositories.branch is dropped: it only fed a box's first checkout and went stale after. The primary checks out the workspace's own branch and the rest their default branch, both read when the box is claimed. Verified on a new two-repository box (identity lists both on main, both checkouts on main, no hooks key).

Verified. Typecheck, lint, i18n (one changed string, 16 locales), sandbox lib and self-seed tests. On the dev stack: a single-repository create came up ready with the checkout at the root marked primary (stages claim 0.7 s, create 10.5 s, settle 1.3 s). Migrations review page: https://app.superset.sh/page/sandbox-v2-migrations-mhf14m

https://claude.ai/code/session_01FK6rLzBEUiHHXZWNmpQ2BP

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 7395d7f1-e0c5-4c50-9f89-39f9b3805098

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[github-actions]

🧹 Preview Cleanup Complete

The following preview resources have been cleaned up:

• ✅ Neon database branch

Thank you for your contribution! 🎉