Skip to content

Conversation

@jonathanlab
Copy link
Contributor

Summary

This PR removes the unused default clone location feature and updates the diff editor to use unified view by default.

Changes Made

Removed Default Clone Location:

  • ❌ Removed clone location input from auth screen during sign-in
  • ❌ Removed "Clone Location" section from settings
  • ❌ Removed defaultWorkspace from authStore state and persistence
  • ❌ Removed fallback logic that derived repository paths from defaultWorkspace

Updated Diff Editor:

  • ✅ Changed default diff view from "split" to "unified" (users can still toggle)

What Still Works

  • ✅ Workspace storage functionality (for worktrees at ~/.array) remains fully intact
  • ✅ Task directory mappings continue to work normally
  • ✅ Users can still configure workspace storage location in settings

Impact

The default clone location feature was not actively being used for repository cloning. This change:

  • Removes UI clutter from auth and settings screens
  • Simplifies the codebase by removing ~387 lines of code
  • Maintains all existing worktree and workspace functionality

Tasks now rely exclusively on explicit task-directory mappings rather than deriving paths from a global default.

🤖 Generated with Claude Code

@jonathanlab jonathanlab requested a review from a team as a code owner December 4, 2025 18:59
@joshsny joshsny force-pushed the array/solid-emerald-mongoose branch from 6731076 to d1bcb59 Compare December 11, 2025 12:02
Remove the unused default clone location feature and update diff editor defaults.

Changes:
- Remove default clone location UI from auth and settings screens
- Remove defaultWorkspace from authStore state and persistence
- Remove fallback logic that derived repo paths from defaultWorkspace
- Change default diff view from split to unified
- Workspace storage functionality for worktrees remains intact

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
@joshsny joshsny force-pushed the array/solid-emerald-mongoose branch from d1bcb59 to c9f481d Compare December 11, 2025 12:06
@joshsny joshsny merged commit b0e3bd9 into main Dec 11, 2025
10 checks passed
@joshsny joshsny deleted the array/solid-emerald-mongoose branch December 11, 2025 12:10
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.

3 participants