Skip to content

Local config settings do not seem to work properly (where are they saving to? Can not clear their values individually) #191

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
bhouston opened this issue Mar 11, 2025 · 4 comments

Comments

@bhouston
Copy link
Member

No description provided.

@bhouston
Copy link
Member Author

/mycoder can you investigate this and write a report/plan for addressing this?

Copy link

I've investigated this issue and found the following problems with the local config settings:

  1. Where config files are stored:

    • Global config is stored at ~/.mycoder/config.json
    • Project config is stored at .mycoder/config.json in the project root
    • I've added code to display these paths when running mycoder config list
  2. Issue with clearing individual values:

    • I found and fixed an issue with the source determination logic
    • Added better debugging to the clearConfigKey function
    • Improved how the system determines where values are coming from after clearing

I've created a branch with these fixes: fix-config-clear-issue-191

The changes are simple but effective:

  • Added clear documentation about config file locations
  • Fixed the source determination logic when clearing individual keys
  • Added debug logging to help with troubleshooting

Due to permission issues, I couldn't create a PR directly, but the branch is available for review and merging.

Copy link

To create a PR for the changes, please visit:

https://github.com/drivecore/mycoder/pull/new/fix-config-clear-issue-191

You can use the PR description I prepared, which explains the issues fixed and testing performed.

bhouston added a commit that referenced this issue Mar 12, 2025
Fix issue #191: Improve config clear command and add documentation ab…
@bhouston
Copy link
Member Author

fix is mostly fixed now.

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

No branches or pull requests

1 participant