Skip to content

Conversation

@giordano-lucas
Copy link
Member

@giordano-lucas giordano-lucas commented Jan 30, 2026

Move the default configuration directory to ~/.notte/cli/ for a cleaner home directory structure that allows other notte tools to share the ~/.notte/ parent directory.

Greptile Overview

Greptile Summary

Changed the default configuration directory from ~/.config/notte (using os.UserConfigDir()) to ~/.notte/cli (using os.UserHomeDir()), allowing other notte tools to share the ~/.notte/ parent directory.

Changes:

  • Updated internal/config/config.go to use os.UserHomeDir() instead of os.UserConfigDir() with ConfigDirName = ".notte/cli"
  • Updated integration tests in tests/integration/functions_test.go to reflect the new path structure
  • Updated documentation in README.md and skills/notte-browser/references/session-management.md

Migration consideration:
This is a breaking change for existing users - their configuration files at ~/.config/notte/config.json will not be automatically migrated to ~/.notte/cli/config.json. Users will need to manually move their config files or re-authenticate after upgrading. Consider adding a migration note in release documentation.

Confidence Score: 4/5

  • Safe to merge with minor migration consideration for existing users
  • The refactoring is clean and consistent across all files. Tests are properly updated. Only concern is the lack of automatic migration logic for existing users' config files - they'll need to manually move their credentials or re-authenticate
  • No files require special attention - all changes are straightforward path updates

Important Files Changed

Filename Overview
internal/config/config.go Changed config directory from os.UserConfigDir() to os.UserHomeDir() with path .notte/cli - clean implementation
tests/integration/functions_test.go Updated test assertions to use new config path ~/.notte/cli - properly aligned with config changes
README.md Updated documentation to reflect new config path ~/.notte/cli/config.json - single line change
skills/notte-browser/references/session-management.md Updated documentation to reflect new path for current_session file - single line change

Move the default configuration directory to ~/.notte/cli/ for a cleaner
home directory structure that allows other notte tools to share the
~/.notte/ parent directory.
Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

4 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@giordano-lucas giordano-lucas merged commit 8448489 into main Jan 30, 2026
4 checks passed
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.

2 participants