Skip to content

fix(list): load settings from primary worktree, not cwd#910

Merged
acreeger merged 1 commit intomainfrom
fix/list-settings-from-primary-worktree
Mar 14, 2026
Merged

fix(list): load settings from primary worktree, not cwd#910
acreeger merged 1 commit intomainfrom
fix/list-settings-from-primary-worktree

Conversation

@acreeger
Copy link
Copy Markdown
Collaborator

Summary

  • Bug: il list from a child worktree with settings.local.json overriding mainBranch showed only worktrees without metadata (e.g., develop and develop-react), filtering out all registered looms
  • Root cause: findMainWorktreePathWithSettings() loaded settings from process.cwd(), picking up the child worktree's local override, resolving currentProjectPath to the wrong worktree
  • Fix: Load settings from the primary git worktree (first in porcelain output) so project path resolution is consistent regardless of which worktree you're in

Test plan

  • il list from a child worktree with mainBranch override in settings.local.json now shows all looms
  • il list from the main worktree still works correctly

When running `il list` from a child worktree that has a
settings.local.json overriding mainBranch, the project path
resolved incorrectly, filtering out all looms with metadata.
@acreeger acreeger merged commit a7886e7 into main Mar 14, 2026
4 checks passed
@acreeger acreeger deleted the fix/list-settings-from-primary-worktree branch March 14, 2026 23:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant