Skip to content

feat: added remote required request flag#33

Merged
petruki merged 1 commit intomasterfrom
staging
Feb 21, 2026
Merged

feat: added remote required request flag#33
petruki merged 1 commit intomasterfrom
staging

Conversation

@petruki
Copy link
Member

@petruki petruki commented Feb 21, 2026

This pull request introduces support for forcing remote API resolution even when local mode is enabled, and adds comprehensive tests for this new functionality. The changes include updates to the core client logic, new methods for controlling remote behavior, and improvements to documentation and test coverage.

Remote Resolution Enhancements:

  • Added a new remote method to SwitcherData that allows forcing remote API calls when local mode is enabled, including error handling if local mode is not active. (switcher_client/switcher_data.py)
  • Updated the client execution logic to check the _force_remote flag, ensuring that forced remote requests bypass local execution. (switcher_client/switcher.py) [1] [2]
  • Introduced the _force_remote attribute to SwitcherData for tracking forced remote requests. (switcher_client/switcher_data.py)

Testing Improvements:

  • Added new tests to verify forced remote resolution behavior, including successful remote calls and error handling when local mode is disabled. (tests/test_switcher_remote.py)
  • Enhanced test helpers to support context options, making tests more flexible for local/remote scenarios. (tests/test_switcher_remote.py)

Documentation Updates:

  • Updated the README.md to remove the "Coming Soon" label from Hybrid Mode and clarify usage of forced remote resolution. (README.md)

@petruki petruki self-assigned this Feb 21, 2026
@sonarqubecloud
Copy link

@petruki petruki merged commit 49e6d87 into master Feb 21, 2026
27 of 28 checks passed
@petruki petruki deleted the staging branch February 21, 2026 20:31
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