fix(config): keep path suffix out of project overrides#2776
Conversation
There was a problem hiding this comment.
Hmbown has reached the 50-review limit for trial accounts. To continue receiving code reviews, upgrade your plan.
There was a problem hiding this comment.
Code Review
This pull request removes the merging of path_suffix from project-level provider configurations in merge_project_provider_config. It also adds corresponding test cases to verify that the base configuration's path_suffix is not overwritten by the project configuration. No review comments were provided, so there is no feedback to address.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
Summary
path_suffixpath_suffixaligned with the existing project-config policy that ignores endpoint routing fieldspath_suffixVerification
cargo test -p codewhale-config --locked project_merge_denies_credentials_endpoints_and_provider_selection -- --nocapturecargo test -p codewhale-tui --bin codewhale-tui --locked api_url_with_suffix -- --nocapturecargo fmt --all -- --checkgit diff --checkCredit
Follow-up hardening for the provider path-suffix feature from #2558, with review coverage noted on related PRs #2506/#2508.