feat(desktop): make 'none' the default branch prefix option

[Kitenite]

Summary

• Move "none" to the first position in the branch prefix dropdown
• Add clickable edit button on branch prefix in new workspace modal

Test plan

• Open settings > behavior and verify "No prefix" appears first in dropdown
• Create new workspace and verify the prefix edit button navigates to settings

Summary by CodeRabbit

• New Features

  • Added a pencil-button in the New Workspace modal to quickly edit branch prefix settings; it closes the modal then opens the behavior settings panel.

• Chores

  • Reordered branch prefix mode options for improved consistency and presentation.

[coderabbitai]

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Adds an inline "Edit prefix" pencil button to NewWorkspaceModal that closes the modal and navigates to /settings/behavior when clicked; also reorders "none" to the start of branch-prefix constant lists in two files (no behavioral change).

Changes

Cohort / File(s)	Summary
Modal — branch-prefix UI apps/desktop/src/renderer/components/NewWorkspaceModal/NewWorkspaceModal.tsx	Imported useNavigate and HiOutlinePencil. Added a right-aligned "Edit prefix" pencil button next to the Branch name display that calls handleClose() then navigate('/settings/behavior'). Wrapped branch label in a div.
Branch-prefix labels apps/desktop/src/renderer/routes/_authenticated/settings/utils/branch-prefix/branch-prefix.ts	Moved/inserted the "none": "No prefix" entry to the start of BRANCH_PREFIX_MODE_LABELS (ordering change only).
Branch-prefix modes packages/local-db/src/schema/zod.ts	Reordered BRANCH_PREFIX_MODES so "none" is first instead of last (ordering change only; exported type unchanged).

Sequence Diagram(s)

sequenceDiagram
    participant User as User
    participant Modal as NewWorkspaceModal
    participant Router as Router (useNavigate)
    participant Settings as Settings Page

    User->>Modal: Click "Edit prefix" (pencil)
    Modal->>Modal: handleClose() — close modal UI
    Modal->>Router: navigate('/settings/behavior')
    Router->>Settings: Render Settings (Behavior / Branch Prefix)
    Settings-->>User: Settings UI shown

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• feat(desktop): worktree bar refactor + open workspace modal base branch #438: Modifies NewWorkspaceModal branch/base-branch UI and related branch controls — strong overlap with this modal UI change.
• feat(desktop): add configurable branch name prefix #1015: Adds/configures branch prefix support and adjusts BRANCH_PREFIX_MODE_LABELS/BRANCH_PREFIX_MODES constants used by settings.
• Custom prefix written twice #1075: Changes NewWorkspaceModal branch-prefix UX and branch preview/applyPrefix behavior, related to the inline edit navigation introduced here.

Poem

🐰 I nibble a pencil, tap the modal's side,

Off to Behavior settings — a hopping glide.
"None" now leads the list, neat and spry,
A tiny change, a shortcut — hop, goodbye! ✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The PR description is incomplete; it lacks required template sections such as Related Issues, Type of Change, and Testing details beyond a checkbox test plan.	Fill out all required template sections: add Related Issues links, explicitly mark the Type of Change (likely 'New feature'), provide detailed Testing steps, and include Additional Notes if applicable.
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: reordering branch prefix options to make 'none' the default, which aligns with the primary objective of the PR.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch kitenite/edit-prefix-and-make-none-default

---

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 and usage tips.}

[github-actions]

🚀 Preview Deployment

🔗 Preview Links

Service	Status	Link
Database (Neon)	❌	Failed to create
Electric (Fly.io)	❌	Failed to deploy
API (Vercel)	❌	Failed to deploy
Web (Vercel)	❌	Failed to deploy
Marketing (Vercel)	❌	Failed to deploy
Admin (Vercel)	❌	Failed to deploy
Docs (Vercel)	❌	Failed to deploy

---

Preview updates automatically with new commits