-
Notifications
You must be signed in to change notification settings - Fork 22
Closed
Labels
bugSomething isn't workingSomething isn't working
Description
Priority: Critical
Related: #199
The web_search=True → tools=[WebSearch()] migration shim (text/client.py:54-64) has zero tests. No verification that:
- The deprecation warning fires
- The shim produces the same behavior as
tools=[WebSearch()] stacklevel=4points to the correct caller frame- Combining old + new params works or errors appropriately
What to fix:
- Add test asserting
DeprecationWarningis emitted forweb_search=True - Add test asserting
x_search=Trueandcode_execution=Trueshims work - Add test for combining
web_search=Truewithtools=[...] - Verify stacklevel points correctly
Files:
src/celeste/modalities/text/client.py:43-67
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't working