Skip to content

fix: correct settingSources order to local, project, user#315

Merged
rjroy merged 1 commit intomainfrom
fix/sources-order
Jan 16, 2026
Merged

fix: correct settingSources order to local, project, user#315
rjroy merged 1 commit intomainfrom
fix/sources-order

Conversation

@rjroy
Copy link
Copy Markdown
Owner

@rjroy rjroy commented Jan 16, 2026

Summary

  • Fixes the order of settingSources from ["project", "user", "local"] to ["local", "project", "user"]
  • The order matters for precedence: local settings should override project settings, which should override user settings

Test plan

  • Existing session-manager tests pass (65 tests)
  • Pre-commit hooks pass (typecheck, lint, unit tests)

🤖 Generated with Claude Code

The order of setting sources matters for precedence. Local settings
should be checked first, then project, then user.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@rjroy rjroy merged commit 0e567ec into main Jan 16, 2026
1 check passed
@rjroy rjroy deleted the fix/sources-order branch January 16, 2026 17:28
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