Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: clean up imports and enhance error handling in WhatsApp client feat: summarize for all groups #11

Merged
merged 2 commits into from
Apr 4, 2025

Conversation

AlmogBaku
Copy link
Collaborator

refactor: clean up imports and enhance error handling in WhatsApp client
feat: summarize for all groups

Copy link

github-actions bot commented Apr 4, 2025

Test Results

1 tests   - 11   0 ✅  - 9   3s ⏱️ ±0s
1 suites ± 0   0 💤  - 1 
1 files   ± 0   0 ❌  - 2   1 🔥 +1 

For more details on these errors, see this check.

Results for commit 1a1f842. ± Comparison against base commit fdd64ec.

This pull request removes 12 and adds 1 tests. Note that renamed tests count towards both.
src.handler.test_router ‑ test_router_ask_question_route
src.handler.test_router ‑ test_router_other_route
src.handler.test_router ‑ test_router_summarize_route
src.handler.test_router ‑ test_send_message
src.models.test_jid ‑ test_ad_jid_creation
src.models.test_jid ‑ test_invalid_jid
src.models.test_jid ‑ test_jid_creation
src.models.test_jid ‑ test_normalize_jid
src.models.test_jid ‑ test_parse_jid
src.models.test_message ‑ test_message_mentions
…
src.handler.test_router

♻️ This comment has been updated with latest results.

@@ -58,7 +58,7 @@ async def __call__(self, message: Message):

async def _route(self, message: str) -> IntentEnum:
agent = Agent(
model="anthropic:claude-3-5-haiku-latest",
Copy link
Owner

Choose a reason for hiding this comment

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

Why migrate to a pricier model here?

@ilanbenb ilanbenb merged commit 8d1e8ee into main Apr 4, 2025
0 of 2 checks passed
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.

2 participants