fix(search): make service.py the proper public facade (#783) #800

Open

sleepy wants to merge 0 commits from fix/search-facade-783 into main

sleepy commented

2026-06-03 15:41:19 +02:00

Owner

Closes #783. Make service.py the proper public facade for the search feature.

sleepy added 1 commit

2026-06-03 15:41:19 +02:00

fix(search): make service.py the proper public facade ef77c089e3

Issue #783.

Previously routes/search_routes.py bypassed service.py and imported
private functions (_call_provider, _get_provider_key, _get_search_instance)
directly from services.search.core and services.search.providers.

Changes:
- service.py: re-export call_provider, get_provider_key, get_search_instance
  as public API (aliased from internal private names)
- __init__.py: expose the new public functions in package-level __all__
- search_routes.py: import all search API from services.search only;
  use public names (call_provider, get_provider_key, get_search_instance)

No behavioural changes. All existing tests pass.

This branch is already included in the target branch. There is nothing to merge.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin fix/search-facade-783:fix/search-facade-783

git switch fix/search-facade-783

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main

git merge --no-ff fix/search-facade-783

git switch fix/search-facade-783

git rebase main

git switch main

git merge --ff-only fix/search-facade-783

git switch fix/search-facade-783

git rebase main

git switch main

git merge --no-ff fix/search-facade-783

git switch main

git merge --squash fix/search-facade-783