feat: project specific config#303
Conversation
|
I already noticed few issues
I do not recommend to merge this at this stage |
…e validation - Load all .restish.json/.restish.yaml files from current directory up to root - Deep merge API configs and profiles across multiple files (headers, query params, etc.) - Replace panic() calls with graceful error handling showing available profiles - Add interactive menu to choose which config file to save when editing - Track all config sources for each API to enable proper save targeting - By default, save to the closest (most specific) config file This enables hierarchical config organization (e.g., project root + team + developer overrides) while maintaining a smooth editing experience with restish apis edit. Fixes issues with random crashes on invalid profile selection and clarifies the behavior of local config editing.
|
I believe i fixed most of the issues in this PR |
|
Hi. I was looking for this feature and tried out your PR and have a few remarks:
Generally, I would advise against doing any merging of config files. If a custom config file is specified with |
|
Thanks for the review! I don't use restish as frequently right now, so it's not a high priority for me, but I'll try to get to it at some point |
|
Thanks @richard-hajek for putting together the project-specific config prototype and for the follow-up feedback on the trust/permissions model. That helped shape the final version that just merged in #349. #349 now provides trusted |
Addresses #239
Fair warning, this work is vibe coded. However I tested it and It Works On My Machine ™️