Skip to content

Conversation

@Samigos
Copy link
Contributor

@Samigos Samigos commented Feb 10, 2026

Adds an Initialize Git flow for non-git workspaces.

  • Prompts for initial branch every time.
  • Initializes repo, creates an initial commit.
  • Optional: create a GitHub repo via gh, add origin, push HEAD, set default branch.
  • Makes the init modal opaque (no transparency) and fixes layering via DS z-index tokens.
  • Includes remote-mode (daemon) wiring + invoke wrapper test.

Tests:

  • npm run typecheck
  • npm test
  • npm run lint
  • cargo check (src-tauri)

@Samigos
Copy link
Contributor Author

Samigos commented Feb 10, 2026

@codex review

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e700876164

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +167 to +170
if (!ok) {
setInitGitRepoPrompt((prev) =>
prev ? { ...prev, error: prev.error ?? "Failed to initialize Git repository." } : prev,
);

Choose a reason for hiding this comment

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

P2 Badge Don't treat canceled init confirmation as failure

handleInitGitRepoPromptConfirm sets an error whenever initGitRepo(...) returns false, but useGitActions.initGitRepo also returns false for the intentional cancel path when the non-empty-folder ask(...) dialog is declined. In that scenario the UI now shows “Failed to initialize Git repository.” even though no operation failed, which is misleading and degrades the init flow UX. Please distinguish cancel vs. failure before setting this error.

Useful? React with 👍 / 👎.

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