Skip to content

Revert "feat: use --name as repository domain in repo create and init…#186

Merged
jomifepe merged 1 commit into
mainfrom
jp/remove-name
May 12, 2026
Merged

Revert "feat: use --name as repository domain in repo create and init…#186
jomifepe merged 1 commit into
mainfrom
jp/remove-name

Conversation

@jomifepe
Copy link
Copy Markdown
Contributor

@jomifepe jomifepe commented May 12, 2026

This reverts commit 1b7e4bf.

The team agreed on keeping the random domain generation.


View in Codesmith
Need help on this PR? Tag @codesmith with what you need.

  • Let Codesmith autofix CI failures and bot reviews

Note

Medium Risk
Changes repository provisioning behavior in init/repo create (auto-creating random domains and removing CLI-side name validation), which could affect user workflows and error handling but doesn’t touch security-sensitive logic.

Overview
Restores random repository domain generation for prismic repo create and prismic init, instead of using a user-supplied --name/--repo as the domain.

init now auto-creates a repository when --repo isn’t provided, and when --repo is provided it must already be accessible in the user’s account (no implicit creation). repo create’s --name becomes an optional display name only, with the domain picked via repeated availability checks.

Removes the zod-based per-option schema validation from the command framework, updates createRepository to require a name, and adjusts tests to reflect the new CLI behavior and cleanup flow.

Reviewed by Cursor Bugbot for commit 3c51b96. Bugbot is set up for automated code reviews on this repo. Configure here.

@jomifepe jomifepe self-assigned this May 12, 2026
Copy link
Copy Markdown

@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.

Fix All in Cursor

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

Reviewed by Cursor Bugbot for commit 3c51b96. Configure here.

Comment thread test/init.test.ts
@jomifepe jomifepe merged commit 2391b3b into main May 12, 2026
13 checks passed
@jomifepe jomifepe deleted the jp/remove-name branch May 12, 2026 11:20
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