Skip to content

Detect configs in .devproxy folder in config get - #1908

Open
waldekmastykarz wants to merge 1 commit into
dotnet:mainfrom
waldekmastykarz:waldekmastykarz-config-get-devproxy-folder
Open

waldekmastykarz wants to merge 1 commit into
dotnet:mainfrom
waldekmastykarz:waldekmastykarz-config-get-devproxy-folder

Conversation

@waldekmastykarz

Copy link
Copy Markdown
Collaborator

Closes #1906

devproxy config get now also scans the .devproxy subfolder of a downloaded preset (listed first, before the root folder), so it prints the command to start Dev Proxy for presets that use the current samples gallery layout.

Verified with devproxy config get github-rate-limiting, which now prints:

info    To start Dev Proxy with the config, run:
info      devproxy --config-file "~dataFolder/configs/github-rate-limiting/.devproxy/devproxyrc.json"

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings October 3, 2026 11:58
@waldekmastykarz
waldekmastykarz requested a review from a team as a code owner October 3, 2026 11:58

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The focused change correctly addresses the reported preset discovery issue without introducing unresolved problems.

Review effort: Balanced
Findings: None

What changed in this PR

Updates config get to recognize presets using the current .devproxy directory layout.

Changes:

  • Adds shared JSON/YAML config patterns.
  • Scans .devproxy before the preset root.
  • Documents the resulting file ordering.
File Description
DevProxy/​Commands/​ConfigCommand.cs Detects config entry points inside .devproxy.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

This branch has not been deployed

No deployments
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.

config get doesn't print the start command for presets with a .devproxy folder

2 participants