Skip to content

Conversation

@richard-hajek
Copy link

Addresses #239

Fair warning, this work is vibe coded. However I tested it and It Works On My Machine ™️

@richard-hajek
Copy link
Author

richard-hajek commented Nov 10, 2025

I already noticed few issues

  • random crashes on profile selection with unknown profile
  • what in this scenario is restish edit supposed to do?

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.
@richard-hajek
Copy link
Author

I believe i fixed most of the issues in this PR

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