Skip to content

ci(preview-env): use --light seed so Apple is the default workspace#20464

Merged
FelixMalfait merged 1 commit into
mainfrom
fix/preview-env-light-seed
May 11, 2026
Merged

ci(preview-env): use --light seed so Apple is the default workspace#20464
FelixMalfait merged 1 commit into
mainfrom
fix/preview-env-light-seed

Conversation

@FelixMalfait
Copy link
Copy Markdown
Member

Summary

  • Pass --light to workspace:seed:dev in the preview env keepalive workflow so only the Apple workspace is created
  • Avoids Empty4 being picked as the default workspace at sign-in (which has no users), making the prefilled tim@apple.dev credentials land on a useful workspace

Why

workspace:seed:dev (no flag) seeds Apple + YCombinator + Empty3 + Empty4. Preview envs run in single-workspace mode (IS_MULTIWORKSPACE_ENABLED=false), so WorkspaceDomainsService.getDefaultWorkspace returns the most recently created workspace — Empty4 — which has no users. Users hitting the preview URL therefore see "Welcome, Empty4." and can't sign in. Same failure mode #19822 fixed for twenty-app-dev.

Test plan

  • Trigger the preview-app label on a PR and confirm the preview URL signs in to the Apple workspace, not Empty4
  • Confirm the seed step still passes (no Empty3/Empty4 references break it)

🤖 Generated with Claude Code

The full seed creates Empty3/Empty4 fixtures, and in single-workspace
mode `getDefaultWorkspace` picks the most recently created workspace —
Empty4 — which has no users, so the prefilled tim@apple.dev cannot
sign in to a useful workspace. Pass `--light` to seed only Apple.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

@FelixMalfait FelixMalfait merged commit 009f597 into main May 11, 2026
87 of 90 checks passed
@FelixMalfait FelixMalfait deleted the fix/preview-env-light-seed branch May 11, 2026 20:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants