mirror of
https://github.com/NousResearch/hermes-agent.git
synced 2026-07-31 19:16:29 +00:00
feat(skills): expand phone-calls into optional telephony skill
Follow up on salvaged PR #965 by moving the capability into optional-skills and broadening it from outbound AI calling to a full telephony skill. Add Twilio number provisioning, env/state persistence, SMS/MMS, inbound SMS polling, Vapi import helpers, and a provider decision tree while keeping telephony out of core runtime code.
This commit is contained in:
parent
73591c978a
commit
37ef6b4a5f
5 changed files with 1966 additions and 630 deletions
394
optional-skills/productivity/telephony/SKILL.md
Normal file
394
optional-skills/productivity/telephony/SKILL.md
Normal file
|
|
@ -0,0 +1,394 @@
|
|||
---
|
||||
name: telephony
|
||||
description: Give Hermes phone capabilities without core tool changes. Provision and persist a Twilio number, send and receive SMS/MMS, make direct calls, and place AI-driven outbound calls through Bland.ai or Vapi.
|
||||
version: 1.0.0
|
||||
author: Nous Research
|
||||
license: MIT
|
||||
metadata:
|
||||
hermes:
|
||||
tags: [telephony, phone, sms, mms, voice, twilio, bland.ai, vapi, calling, texting]
|
||||
related_skills: [find-nearby, google-workspace, agentmail]
|
||||
category: productivity
|
||||
---
|
||||
|
||||
# Telephony — Numbers, Calls, and Texts without Core Tool Changes
|
||||
|
||||
This optional skill gives Hermes practical phone capabilities while keeping telephony out of the core tool list.
|
||||
|
||||
It ships with a helper script, `scripts/telephony.py`, that can:
|
||||
- save provider credentials into `~/.hermes/.env`
|
||||
- search for and buy a Twilio phone number
|
||||
- remember that owned number for later sessions
|
||||
- send SMS / MMS from the owned number
|
||||
- poll inbound SMS for that number with no webhook server required
|
||||
- make direct Twilio calls using TwiML `<Say>` or `<Play>`
|
||||
- import the owned Twilio number into Vapi
|
||||
- place outbound AI calls through Bland.ai or Vapi
|
||||
|
||||
## What this solves
|
||||
|
||||
This skill is meant to cover the practical phone tasks users actually want:
|
||||
- outbound calls
|
||||
- texting
|
||||
- owning a reusable agent number
|
||||
- checking messages that arrive to that number later
|
||||
- preserving that number and related IDs between sessions
|
||||
- future-friendly telephony identity for inbound SMS polling and other automations
|
||||
|
||||
It does **not** turn Hermes into a real-time inbound phone gateway. Inbound SMS is handled by polling the Twilio REST API. That is enough for many workflows, including notifications and some one-time-code retrieval, without adding core webhook infrastructure.
|
||||
|
||||
## Safety rules — mandatory
|
||||
|
||||
1. Always confirm before placing a call or sending a text.
|
||||
2. Never dial emergency numbers.
|
||||
3. Never use telephony for harassment, spam, impersonation, or anything illegal.
|
||||
4. Treat third-party phone numbers as sensitive operational data:
|
||||
- do not save them to Hermes memory
|
||||
- do not include them in skill docs, summaries, or follow-up notes unless the user explicitly wants that
|
||||
5. It is fine to persist the **agent-owned Twilio number** because that is part of the user's configuration.
|
||||
6. VoIP numbers are **not guaranteed** to work for all third-party 2FA flows. Use with caution and set user expectations clearly.
|
||||
|
||||
## Decision tree — which service to use?
|
||||
|
||||
Use this logic instead of hardcoded provider routing:
|
||||
|
||||
### 1) "I want Hermes to own a real phone number"
|
||||
Use **Twilio**.
|
||||
|
||||
Why:
|
||||
- easiest path to buying and keeping a number
|
||||
- best SMS / MMS support
|
||||
- simplest inbound SMS polling story
|
||||
- cleanest future path to inbound webhooks or call handling
|
||||
|
||||
Use cases:
|
||||
- receive texts later
|
||||
- send deployment alerts / cron notifications
|
||||
- maintain a reusable phone identity for the agent
|
||||
- experiment with phone-based auth flows later
|
||||
|
||||
### 2) "I only need the easiest outbound AI phone call right now"
|
||||
Use **Bland.ai**.
|
||||
|
||||
Why:
|
||||
- quickest setup
|
||||
- one API key
|
||||
- no need to first buy/import a number yourself
|
||||
|
||||
Tradeoff:
|
||||
- less flexible
|
||||
- voice quality is decent, but not the best
|
||||
|
||||
### 3) "I want the best conversational AI voice quality"
|
||||
Use **Twilio + Vapi**.
|
||||
|
||||
Why:
|
||||
- Twilio gives you the owned number
|
||||
- Vapi gives you better conversational AI call quality and more voice/model flexibility
|
||||
|
||||
Recommended flow:
|
||||
1. Buy/save a Twilio number
|
||||
2. Import it into Vapi
|
||||
3. Save the returned `VAPI_PHONE_NUMBER_ID`
|
||||
4. Use `ai-call --provider vapi`
|
||||
|
||||
### 4) "I want to call with a custom prerecorded voice message"
|
||||
Use **Twilio direct call** with a public audio URL.
|
||||
|
||||
Why:
|
||||
- easiest way to play a custom MP3
|
||||
- pairs well with Hermes `text_to_speech` plus a public file host or tunnel
|
||||
|
||||
## Files and persistent state
|
||||
|
||||
The skill persists telephony state in two places:
|
||||
|
||||
### `~/.hermes/.env`
|
||||
Used for long-lived provider credentials and owned-number IDs, for example:
|
||||
- `TWILIO_ACCOUNT_SID`
|
||||
- `TWILIO_AUTH_TOKEN`
|
||||
- `TWILIO_PHONE_NUMBER`
|
||||
- `TWILIO_PHONE_NUMBER_SID`
|
||||
- `BLAND_API_KEY`
|
||||
- `VAPI_API_KEY`
|
||||
- `VAPI_PHONE_NUMBER_ID`
|
||||
- `PHONE_PROVIDER` (AI call provider: bland or vapi)
|
||||
|
||||
### `~/.hermes/telephony_state.json`
|
||||
Used for skill-only state that should survive across sessions, for example:
|
||||
- remembered default Twilio number / SID
|
||||
- remembered Vapi phone number ID
|
||||
- last inbound message SID/date for inbox polling checkpoints
|
||||
|
||||
This means:
|
||||
- the next time the skill is loaded, `diagnose` can tell you what number is already configured
|
||||
- `twilio-inbox --since-last --mark-seen` can continue from the previous checkpoint
|
||||
|
||||
## Locate the helper script
|
||||
|
||||
After installing this skill, locate the script like this:
|
||||
|
||||
```bash
|
||||
SCRIPT="$(find ~/.hermes/skills -path '*/telephony/scripts/telephony.py' -print -quit)"
|
||||
```
|
||||
|
||||
If `SCRIPT` is empty, the skill is not installed yet.
|
||||
|
||||
## Install
|
||||
|
||||
This is an official optional skill, so install it from the Skills Hub:
|
||||
|
||||
```bash
|
||||
hermes skills search telephony
|
||||
hermes skills install official/productivity/telephony
|
||||
```
|
||||
|
||||
## Provider setup
|
||||
|
||||
### Twilio — owned number, SMS/MMS, direct calls, inbound SMS polling
|
||||
|
||||
Sign up at:
|
||||
- https://www.twilio.com/try-twilio
|
||||
|
||||
Then save credentials into Hermes:
|
||||
|
||||
```bash
|
||||
python3 "$SCRIPT" save-twilio ACXXXXXXXXXXXXXXXXXXXXXXXXXXXX your_auth_token_here
|
||||
```
|
||||
|
||||
Search for available numbers:
|
||||
|
||||
```bash
|
||||
python3 "$SCRIPT" twilio-search --country US --area-code 702 --limit 5
|
||||
```
|
||||
|
||||
Buy and remember a number:
|
||||
|
||||
```bash
|
||||
python3 "$SCRIPT" twilio-buy "+17025551234" --save-env
|
||||
```
|
||||
|
||||
List owned numbers:
|
||||
|
||||
```bash
|
||||
python3 "$SCRIPT" twilio-owned
|
||||
```
|
||||
|
||||
Set one of them as the default later:
|
||||
|
||||
```bash
|
||||
python3 "$SCRIPT" twilio-set-default "+17025551234" --save-env
|
||||
# or
|
||||
python3 "$SCRIPT" twilio-set-default PNXXXXXXXXXXXXXXXXXXXXXXXXXXXX --save-env
|
||||
```
|
||||
|
||||
### Bland.ai — easiest outbound AI calling
|
||||
|
||||
Sign up at:
|
||||
- https://app.bland.ai
|
||||
|
||||
Save config:
|
||||
|
||||
```bash
|
||||
python3 "$SCRIPT" save-bland your_bland_api_key --voice mason
|
||||
```
|
||||
|
||||
### Vapi — better conversational voice quality
|
||||
|
||||
Sign up at:
|
||||
- https://dashboard.vapi.ai
|
||||
|
||||
Save the API key first:
|
||||
|
||||
```bash
|
||||
python3 "$SCRIPT" save-vapi your_vapi_api_key
|
||||
```
|
||||
|
||||
Import your owned Twilio number into Vapi and persist the returned phone number ID:
|
||||
|
||||
```bash
|
||||
python3 "$SCRIPT" vapi-import-twilio --save-env
|
||||
```
|
||||
|
||||
If you already know the Vapi phone number ID, save it directly:
|
||||
|
||||
```bash
|
||||
python3 "$SCRIPT" save-vapi your_vapi_api_key --phone-number-id vapi_phone_number_id_here
|
||||
```
|
||||
|
||||
## Diagnose current state
|
||||
|
||||
At any time, inspect what the skill already knows:
|
||||
|
||||
```bash
|
||||
python3 "$SCRIPT" diagnose
|
||||
```
|
||||
|
||||
Use this first when resuming work in a later session.
|
||||
|
||||
## Common workflows
|
||||
|
||||
### A. Buy an agent number and keep using it later
|
||||
|
||||
1. Save Twilio credentials:
|
||||
```bash
|
||||
python3 "$SCRIPT" save-twilio AC... auth_token_here
|
||||
```
|
||||
|
||||
2. Search for a number:
|
||||
```bash
|
||||
python3 "$SCRIPT" twilio-search --country US --area-code 702 --limit 10
|
||||
```
|
||||
|
||||
3. Buy it and save it into `~/.hermes/.env` + state:
|
||||
```bash
|
||||
python3 "$SCRIPT" twilio-buy "+17025551234" --save-env
|
||||
```
|
||||
|
||||
4. Next session, run:
|
||||
```bash
|
||||
python3 "$SCRIPT" diagnose
|
||||
```
|
||||
This shows the remembered default number and inbox checkpoint state.
|
||||
|
||||
### B. Send a text from the agent number
|
||||
|
||||
```bash
|
||||
python3 "$SCRIPT" twilio-send-sms "+15551230000" "Your deployment completed successfully."
|
||||
```
|
||||
|
||||
With media:
|
||||
|
||||
```bash
|
||||
python3 "$SCRIPT" twilio-send-sms "+15551230000" "Here is the chart." --media-url "https://example.com/chart.png"
|
||||
```
|
||||
|
||||
### C. Check inbound texts later with no webhook server
|
||||
|
||||
Poll the inbox for the default Twilio number:
|
||||
|
||||
```bash
|
||||
python3 "$SCRIPT" twilio-inbox --limit 20
|
||||
```
|
||||
|
||||
Only show messages that arrived after the last checkpoint, and advance the checkpoint when you're done reading:
|
||||
|
||||
```bash
|
||||
python3 "$SCRIPT" twilio-inbox --since-last --mark-seen
|
||||
```
|
||||
|
||||
This is the main answer to “how do I access messages the number receives next time the skill is loaded?”
|
||||
|
||||
### D. Make a direct Twilio call with built-in TTS
|
||||
|
||||
```bash
|
||||
python3 "$SCRIPT" twilio-call "+15551230000" --message "Hello! This is Hermes calling with your status update." --voice Polly.Joanna
|
||||
```
|
||||
|
||||
### E. Call with a prerecorded / custom voice message
|
||||
|
||||
Generate or host audio separately, then:
|
||||
|
||||
```bash
|
||||
python3 "$SCRIPT" twilio-call "+15551230000" --audio-url "https://example.com/briefing.mp3"
|
||||
```
|
||||
|
||||
Useful pairing:
|
||||
- use Hermes `text_to_speech` to generate the MP3
|
||||
- make it publicly reachable
|
||||
- then place the call with `--audio-url`
|
||||
|
||||
### F. Navigate a phone tree / IVR with Twilio direct calling
|
||||
|
||||
If you need to press digits after the call connects, use `--send-digits`.
|
||||
Twilio interprets `w` as a short wait.
|
||||
|
||||
```bash
|
||||
python3 "$SCRIPT" twilio-call "+18005551234" --message "Connecting to billing now." --send-digits "ww1w2w3"
|
||||
```
|
||||
|
||||
This is useful for reaching a specific menu branch before handing off to a human or delivering a short status message.
|
||||
|
||||
### G. Outbound AI phone call with Bland.ai
|
||||
|
||||
```bash
|
||||
python3 "$SCRIPT" ai-call "+15551230000" "Call the dental office, ask for a cleaning appointment on Tuesday afternoon, and if they do not have Tuesday availability, ask for Wednesday or Thursday instead." --provider bland --voice mason --max-duration 3
|
||||
```
|
||||
|
||||
Check status:
|
||||
|
||||
```bash
|
||||
python3 "$SCRIPT" ai-status <call_id> --provider bland
|
||||
```
|
||||
|
||||
Ask Bland analysis questions after completion:
|
||||
|
||||
```bash
|
||||
python3 "$SCRIPT" ai-status <call_id> --provider bland --analyze "Was the appointment confirmed?,What date and time?,Any special instructions?"
|
||||
```
|
||||
|
||||
### H. Outbound AI phone call with Vapi on your owned number
|
||||
|
||||
1. Import your Twilio number into Vapi:
|
||||
```bash
|
||||
python3 "$SCRIPT" vapi-import-twilio --save-env
|
||||
```
|
||||
|
||||
2. Place the call:
|
||||
```bash
|
||||
python3 "$SCRIPT" ai-call "+15551230000" "You are calling to make a dinner reservation for two at 7:30 PM. If that is unavailable, ask for the nearest time between 6:30 and 8:30 PM." --provider vapi --max-duration 4
|
||||
```
|
||||
|
||||
3. Check result:
|
||||
```bash
|
||||
python3 "$SCRIPT" ai-status <call_id> --provider vapi
|
||||
```
|
||||
|
||||
## Suggested agent procedure
|
||||
|
||||
When the user asks for a call or text:
|
||||
|
||||
1. Determine which path fits the request via the decision tree.
|
||||
2. Run `diagnose` if configuration state is unclear.
|
||||
3. Gather the full task details.
|
||||
4. Confirm with the user before dialing or texting.
|
||||
5. Use the correct command.
|
||||
6. Poll for results if needed.
|
||||
7. Summarize the outcome without persisting third-party numbers to Hermes memory.
|
||||
|
||||
## What this skill still does not do
|
||||
|
||||
- real-time inbound call answering
|
||||
- webhook-based live SMS push into the agent loop
|
||||
- guaranteed support for arbitrary third-party 2FA providers
|
||||
|
||||
Those would require more infrastructure than a pure optional skill.
|
||||
|
||||
## Pitfalls
|
||||
|
||||
- Twilio trial accounts and regional rules can restrict who you can call/text.
|
||||
- Some services reject VoIP numbers for 2FA.
|
||||
- `twilio-inbox` polls the REST API; it is not instant push delivery.
|
||||
- Vapi outbound calling still depends on having a valid imported number.
|
||||
- Bland is easiest, but not always the best-sounding.
|
||||
- Do not store arbitrary third-party phone numbers in Hermes memory.
|
||||
|
||||
## Verification checklist
|
||||
|
||||
After setup, you should be able to do all of the following with just this skill:
|
||||
|
||||
1. `diagnose` shows provider readiness and remembered state
|
||||
2. search and buy a Twilio number
|
||||
3. persist that number to `~/.hermes/.env`
|
||||
4. send an SMS from the owned number
|
||||
5. poll inbound texts for the owned number later
|
||||
6. place a direct Twilio call
|
||||
7. place an AI call via Bland or Vapi
|
||||
|
||||
## References
|
||||
|
||||
- Twilio phone numbers: https://www.twilio.com/docs/phone-numbers/api
|
||||
- Twilio messaging: https://www.twilio.com/docs/messaging/api/message-resource
|
||||
- Twilio voice: https://www.twilio.com/docs/voice/api/call-resource
|
||||
- Vapi docs: https://docs.vapi.ai/
|
||||
- Bland.ai: https://app.bland.ai/
|
||||
1343
optional-skills/productivity/telephony/scripts/telephony.py
Normal file
1343
optional-skills/productivity/telephony/scripts/telephony.py
Normal file
File diff suppressed because it is too large
Load diff
|
|
@ -1,212 +0,0 @@
|
|||
---
|
||||
name: phone-calls
|
||||
description: Make outbound phone calls on the user's behalf using AI voice agents (Bland.ai or Vapi). Schedule appointments, make reservations, or deliver messages via realistic voice calls. Always confirm with user before dialing.
|
||||
version: 2.0.0
|
||||
author: NousResearch
|
||||
license: MIT
|
||||
metadata:
|
||||
hermes:
|
||||
tags: [phone, calling, voice, appointments, scheduling, bland.ai, vapi, elevenlabs, twilio, telephony]
|
||||
related_skills: [google-workspace, find-nearby]
|
||||
---
|
||||
|
||||
# Phone Calls — AI Voice Agent
|
||||
|
||||
Make outbound phone calls on the user's behalf using AI voice agents. Uses the `phone_call.py` helper script (in this skill's `scripts/` directory) to call Bland.ai or Vapi APIs.
|
||||
|
||||
## When to Use
|
||||
|
||||
- User asks you to **call someone** (schedule appointment, make reservation, leave message)
|
||||
- User asks to **schedule an appointment** (dentist, doctor, haircut, etc.)
|
||||
- User asks to **make a reservation** (restaurant, hotel, etc.)
|
||||
- User says "call", "phone", "ring", "dial", or mentions making a phone call
|
||||
|
||||
## Safety Rules — MANDATORY
|
||||
|
||||
1. **ALWAYS confirm with the user before making any call.** Show them:
|
||||
- The phone number you're about to call
|
||||
- A summary of what the AI will say/do
|
||||
- The voice and max duration
|
||||
2. **Never call emergency numbers** (911, 112, 999, etc.)
|
||||
3. **Only share user info they've explicitly authorized** (check memory/user profile)
|
||||
4. **Never make calls with hostile, harassing, or offensive content**
|
||||
5. **Phone number privacy:**
|
||||
- All phone numbers (except the user's own stored number) are SENSITIVE — never save to memory, never persist in session summaries or skills
|
||||
- Always mask numbers in responses: show last 4 digits only (e.g. "Called ***-***-1234")
|
||||
- The user's own number may only be shared with businesses during appointment booking when they need a callback/contact number — never in any other context
|
||||
- When confirming a call with the user, you may show the full number in the confirmation prompt, but mask it in all subsequent messages and summaries
|
||||
|
||||
## Providers
|
||||
|
||||
### Bland.ai (default — start here)
|
||||
- All-in-one platform, simplest setup, one API key and you're calling
|
||||
- Needs only `BLAND_API_KEY` env var
|
||||
- Sign up free at https://app.bland.ai (~$2 trial credit)
|
||||
- Voices: mason, josh, ryan, matt (male); evelyn, tina, june (female)
|
||||
- ~$0.07-0.12/min
|
||||
- Downside: voice quality is decent but noticeably robotic
|
||||
|
||||
### Vapi (upgrade — better voices)
|
||||
- Flexible platform: plug in any voice (ElevenLabs, Deepgram, PlayHT, Cartesia) and any LLM
|
||||
- Much more natural-sounding than Bland
|
||||
- Requires a Twilio number for outbound calls (Vapi's free numbers are inbound-only)
|
||||
- Setup:
|
||||
1. Sign up at https://dashboard.vapi.ai ($10 free credit)
|
||||
2. Sign up at https://twilio.com ($15 free credit)
|
||||
3. Buy a Twilio number (~$1/mo)
|
||||
4. Import it into Vapi (needs Twilio Account SID + Auth Token)
|
||||
5. Set `VAPI_API_KEY` and `VAPI_PHONE_NUMBER_ID`
|
||||
- ~$0.10-0.25/min depending on voice/LLM choices
|
||||
- If the user wants to upgrade from Bland, walk them through Twilio setup
|
||||
|
||||
## Helper Script
|
||||
|
||||
The script at `scripts/phone_call.py` handles all API calls. It uses only Python stdlib (no pip dependencies). Run it via `terminal` or `execute_code`.
|
||||
|
||||
```bash
|
||||
# Locate the script
|
||||
SCRIPT="$(find ~/.hermes/skills -path '*/phone-calls/scripts/phone_call.py' -print -quit)"
|
||||
|
||||
# Make a call
|
||||
python3 "$SCRIPT" call "+15551234567" "Schedule a cleaning for Tuesday afternoon" --voice mason
|
||||
|
||||
# Check call result
|
||||
python3 "$SCRIPT" status <call_id>
|
||||
|
||||
# Check call result with analysis questions (Bland only)
|
||||
python3 "$SCRIPT" status <call_id> --analyze "Was appointment confirmed?,What time?"
|
||||
|
||||
# Check configuration
|
||||
python3 "$SCRIPT" diagnose
|
||||
```
|
||||
|
||||
## Configuration
|
||||
|
||||
Set via env vars (preferred) or `~/.hermes/config.yaml` under the `phone:` key. Env vars take priority.
|
||||
|
||||
**Bland.ai (quick start):**
|
||||
```
|
||||
BLAND_API_KEY=org_xxx
|
||||
BLAND_DEFAULT_VOICE=mason # optional
|
||||
```
|
||||
|
||||
**Vapi:**
|
||||
```
|
||||
PHONE_PROVIDER=vapi
|
||||
VAPI_API_KEY=xxx-xxx
|
||||
VAPI_PHONE_NUMBER_ID=xxx-xxx
|
||||
VAPI_VOICE_PROVIDER=11labs # optional (default: 11labs)
|
||||
VAPI_VOICE_ID=cjVigY5... # optional (default: ElevenLabs "Eric")
|
||||
VAPI_MODEL=gpt-4o # optional
|
||||
```
|
||||
|
||||
**config.yaml alternative:**
|
||||
```yaml
|
||||
phone:
|
||||
provider: bland # or "vapi"
|
||||
bland:
|
||||
api_key: org_xxx
|
||||
default_voice: mason
|
||||
vapi:
|
||||
api_key: xxx-xxx
|
||||
phone_number_id: xxx-xxx
|
||||
default_voice_provider: 11labs
|
||||
default_voice_id: cjVigY5qzO86Huf0OWal
|
||||
model: gpt-4o
|
||||
```
|
||||
|
||||
## Procedure
|
||||
|
||||
### Step 0: First-time setup (only once)
|
||||
|
||||
Run `diagnose` to check if a provider is configured:
|
||||
```bash
|
||||
python3 "$SCRIPT" diagnose
|
||||
```
|
||||
|
||||
If not configured, ask the user to choose a provider and set the env vars or config.
|
||||
|
||||
### Step 1: Gather call details
|
||||
|
||||
Collect from the user:
|
||||
- **Who to call**: Name and phone number (look up if needed)
|
||||
- **Purpose**: What should the AI say/accomplish
|
||||
- **User info to share**: Name, preferences, insurance, etc.
|
||||
- **Constraints**: Preferred times, budget, special requests
|
||||
|
||||
### Step 2: Craft the task prompt
|
||||
|
||||
Write the task like you're briefing a human assistant. Include:
|
||||
- All necessary details (names, dates, preferences)
|
||||
- Fallback options ("if Tuesday isn't available, try Wednesday")
|
||||
- Boundaries on what info to share
|
||||
- A natural first sentence
|
||||
|
||||
**Name pronunciation**: If the user's name has a non-obvious pronunciation, spell it phonetically in the task prompt (e.g., "Morganne" for Morgane).
|
||||
|
||||
### Step 3: Confirm with user
|
||||
|
||||
Present a summary and wait for explicit approval:
|
||||
```
|
||||
I'm ready to call:
|
||||
Number: +1 (555) 123-4567 (Dr. Smith's Dental Office)
|
||||
Purpose: Schedule a cleaning, Tuesday afternoon preferred
|
||||
Voice: mason (male)
|
||||
Max: 3 minutes
|
||||
|
||||
Shall I go ahead?
|
||||
```
|
||||
|
||||
### Step 4: Make the call
|
||||
|
||||
```bash
|
||||
python3 "$SCRIPT" call "+15551234567" "You are calling Dr. Smith's Dental Office on behalf of Morganne. Schedule a dental cleaning for Tuesday afternoon. If Tuesday is not available, try Wednesday or Thursday. Morganne's phone number for callbacks is +14385551234." --voice mason --max-duration 3
|
||||
```
|
||||
|
||||
### Step 5: Get results
|
||||
|
||||
Wait 60-90 seconds, then check the status:
|
||||
```bash
|
||||
python3 "$SCRIPT" status <call_id>
|
||||
```
|
||||
|
||||
If the call is still in progress, wait and try again. Once completed, present a summary:
|
||||
- Was the objective accomplished?
|
||||
- Key details (date, time, location, confirmations)
|
||||
- Any follow-up needed
|
||||
|
||||
For Bland.ai, you can also ask structured analysis questions:
|
||||
```bash
|
||||
python3 "$SCRIPT" status <call_id> --analyze "Was the appointment confirmed?,What date and time?,Any special instructions?"
|
||||
```
|
||||
|
||||
## Importing a Twilio Number into Vapi
|
||||
|
||||
For Vapi outbound calls, you need to import a Twilio number:
|
||||
|
||||
1. Sign up at https://www.twilio.com/try-twilio
|
||||
2. Buy a phone number in the Twilio console
|
||||
3. Copy your Account SID and Auth Token from Account > API keys & tokens
|
||||
4. Import into Vapi:
|
||||
```bash
|
||||
curl -X POST https://api.vapi.ai/phone-number \
|
||||
-H "Authorization: Bearer $VAPI_API_KEY" \
|
||||
-H "Content-Type: application/json" \
|
||||
-d '{
|
||||
"provider": "twilio",
|
||||
"number": "+1XXXXXXXXXX",
|
||||
"twilioAccountSid": "AC...",
|
||||
"twilioAuthToken": "..."
|
||||
}'
|
||||
```
|
||||
5. Use the returned `id` as `VAPI_PHONE_NUMBER_ID`
|
||||
|
||||
## Pitfalls
|
||||
|
||||
- **No pip dependencies needed**: The script uses only Python stdlib (`urllib`)
|
||||
- **Call goes to voicemail**: Check `answered_by` field in status results
|
||||
- **"Terrible voice"**: Switch from Bland to Vapi with ElevenLabs voices for much better quality
|
||||
- **Vapi free numbers can't make outbound calls**: You must import a Twilio number
|
||||
- **Vapi free numbers can't call international**: Canadian numbers count as international from US numbers
|
||||
- **Name pronunciation**: ElevenLabs ignores phonetic hyphens — spell names literally as they should be pronounced
|
||||
- **Transcript not ready**: Poll status a few times with 30-60s delays between attempts
|
||||
|
|
@ -1,418 +0,0 @@
|
|||
#!/usr/bin/env python3
|
||||
"""Phone Call CLI — Make and check outbound AI voice calls.
|
||||
|
||||
Usage:
|
||||
python3 phone_call.py call <phone_number> <task> [--voice NAME] [--first-sentence TEXT] [--max-duration MIN]
|
||||
python3 phone_call.py status <call_id> [--analyze "question1,question2"]
|
||||
python3 phone_call.py diagnose
|
||||
|
||||
Providers:
|
||||
Bland.ai (default): set BLAND_API_KEY env var
|
||||
Vapi: set VAPI_API_KEY + VAPI_PHONE_NUMBER_ID env vars
|
||||
and PHONE_PROVIDER=vapi
|
||||
|
||||
Configuration via env vars:
|
||||
PHONE_PROVIDER "bland" (default) or "vapi"
|
||||
BLAND_API_KEY Bland.ai organization key
|
||||
BLAND_DEFAULT_VOICE Bland voice name (default: mason)
|
||||
VAPI_API_KEY Vapi private key
|
||||
VAPI_PHONE_NUMBER_ID Vapi phone number ID (imported Twilio number)
|
||||
VAPI_VOICE_PROVIDER Voice provider for Vapi (default: 11labs)
|
||||
VAPI_VOICE_ID Voice ID for Vapi (default: ElevenLabs "Eric")
|
||||
VAPI_MODEL LLM model for Vapi assistant (default: gpt-4o)
|
||||
|
||||
Or via ~/.hermes/config.yaml under the 'phone:' key (env vars take priority).
|
||||
|
||||
Examples:
|
||||
# Make a call with Bland.ai
|
||||
BLAND_API_KEY=org_xxx python3 phone_call.py call "+15551234567" "Schedule a cleaning for Tuesday afternoon"
|
||||
|
||||
# Check call result
|
||||
python3 phone_call.py status abc-123-def
|
||||
|
||||
# Check config
|
||||
python3 phone_call.py diagnose
|
||||
"""
|
||||
|
||||
import json
|
||||
import os
|
||||
import sys
|
||||
import urllib.error
|
||||
import urllib.request
|
||||
|
||||
# ---------------------------------------------------------------------------
|
||||
# Defaults
|
||||
# ---------------------------------------------------------------------------
|
||||
BLAND_API_BASE = "https://api.bland.ai/v1"
|
||||
BLAND_DEFAULT_VOICE = "mason"
|
||||
BLAND_DEFAULT_MODEL = "enhanced"
|
||||
BLAND_VOICES = {
|
||||
"mason": "Male, natural, friendly (recommended)",
|
||||
"josh": "Male, conversational",
|
||||
"ryan": "Male, professional",
|
||||
"matt": "Male, casual",
|
||||
"evelyn": "Female, natural, warm (recommended)",
|
||||
"tina": "Female, warm, friendly",
|
||||
"june": "Female, conversational",
|
||||
}
|
||||
|
||||
VAPI_API_BASE = "https://api.vapi.ai"
|
||||
VAPI_DEFAULT_VOICE_PROVIDER = "11labs"
|
||||
VAPI_DEFAULT_VOICE_ID = "cjVigY5qzO86Huf0OWal" # ElevenLabs "Eric"
|
||||
VAPI_DEFAULT_MODEL = "gpt-4o"
|
||||
|
||||
|
||||
# ---------------------------------------------------------------------------
|
||||
# Config loading
|
||||
# ---------------------------------------------------------------------------
|
||||
def _load_config() -> dict:
|
||||
"""Load phone config from ~/.hermes/config.yaml, falling back to {}."""
|
||||
config_path = os.path.expanduser("~/.hermes/config.yaml")
|
||||
if not os.path.exists(config_path):
|
||||
return {}
|
||||
try:
|
||||
import yaml # optional dependency
|
||||
with open(config_path, "r") as f:
|
||||
cfg = yaml.safe_load(f) or {}
|
||||
return cfg.get("phone", {})
|
||||
except ImportError:
|
||||
return {}
|
||||
except Exception:
|
||||
return {}
|
||||
|
||||
|
||||
def _env_or_config(env_key: str, config_path: list, default: str = "") -> str:
|
||||
"""Get a value from env var first, then config.yaml, then default."""
|
||||
val = os.environ.get(env_key, "")
|
||||
if val:
|
||||
return val
|
||||
cfg = _load_config()
|
||||
for key in config_path:
|
||||
if isinstance(cfg, dict):
|
||||
cfg = cfg.get(key, {})
|
||||
else:
|
||||
return default
|
||||
return str(cfg) if cfg and not isinstance(cfg, dict) else default
|
||||
|
||||
|
||||
def _get_provider() -> str:
|
||||
return _env_or_config("PHONE_PROVIDER", ["provider"], "bland").lower().strip()
|
||||
|
||||
|
||||
# ---------------------------------------------------------------------------
|
||||
# HTTP helper (stdlib only — no requests dependency)
|
||||
# ---------------------------------------------------------------------------
|
||||
def _http(method: str, url: str, headers: dict, data: dict | None = None) -> dict:
|
||||
"""Make an HTTP request and return parsed JSON."""
|
||||
body = json.dumps(data).encode() if data else None
|
||||
req = urllib.request.Request(url, data=body, headers=headers, method=method)
|
||||
try:
|
||||
with urllib.request.urlopen(req, timeout=30) as resp:
|
||||
return json.loads(resp.read().decode())
|
||||
except urllib.error.HTTPError as e:
|
||||
err_body = e.read().decode() if e.fp else ""
|
||||
print(f"HTTP {e.code}: {e.reason}", file=sys.stderr)
|
||||
if err_body:
|
||||
print(err_body, file=sys.stderr)
|
||||
sys.exit(1)
|
||||
except urllib.error.URLError as e:
|
||||
print(f"Connection error: {e.reason}", file=sys.stderr)
|
||||
sys.exit(1)
|
||||
|
||||
|
||||
# ---------------------------------------------------------------------------
|
||||
# Bland.ai
|
||||
# ---------------------------------------------------------------------------
|
||||
def _bland_api_key() -> str:
|
||||
return _env_or_config("BLAND_API_KEY", ["bland", "api_key"])
|
||||
|
||||
|
||||
def bland_call(phone_number: str, task: str, voice: str | None = None,
|
||||
first_sentence: str | None = None, max_duration: int = 3) -> dict:
|
||||
api_key = _bland_api_key()
|
||||
if not api_key:
|
||||
print("Error: No Bland.ai API key. Set BLAND_API_KEY or add to ~/.hermes/config.yaml under phone.bland.api_key", file=sys.stderr)
|
||||
print("Sign up free at https://app.bland.ai", file=sys.stderr)
|
||||
sys.exit(1)
|
||||
|
||||
if voice is None:
|
||||
voice = _env_or_config("BLAND_DEFAULT_VOICE", ["bland", "default_voice"], BLAND_DEFAULT_VOICE)
|
||||
|
||||
payload = {
|
||||
"phone_number": phone_number,
|
||||
"task": task,
|
||||
"voice": voice,
|
||||
"model": BLAND_DEFAULT_MODEL,
|
||||
"max_duration": max_duration,
|
||||
"record": True,
|
||||
"wait_for_greeting": True,
|
||||
}
|
||||
if first_sentence:
|
||||
payload["first_sentence"] = first_sentence
|
||||
|
||||
result = _http("POST", f"{BLAND_API_BASE}/calls",
|
||||
{"Content-Type": "application/json", "authorization": api_key},
|
||||
payload)
|
||||
|
||||
call_id = result.get("call_id")
|
||||
if not call_id:
|
||||
print(f"Error: Bland.ai returned no call_id: {json.dumps(result)}", file=sys.stderr)
|
||||
sys.exit(1)
|
||||
|
||||
return {
|
||||
"success": True,
|
||||
"provider": "bland",
|
||||
"call_id": call_id,
|
||||
"phone_number": phone_number,
|
||||
"voice": voice,
|
||||
"max_duration": max_duration,
|
||||
"message": "Call initiated. Use 'status' command to check results.",
|
||||
}
|
||||
|
||||
|
||||
def bland_status(call_id: str, analyze: str | None = None) -> dict:
|
||||
api_key = _bland_api_key()
|
||||
if not api_key:
|
||||
print("Error: No Bland.ai API key.", file=sys.stderr)
|
||||
sys.exit(1)
|
||||
|
||||
data = _http("GET", f"{BLAND_API_BASE}/calls/{call_id}",
|
||||
{"authorization": api_key})
|
||||
|
||||
result = {
|
||||
"success": True,
|
||||
"provider": "bland",
|
||||
"status": data.get("status"),
|
||||
"duration_minutes": data.get("call_length"),
|
||||
"answered_by": data.get("answered_by"),
|
||||
"transcript": data.get("concatenated_transcript", ""),
|
||||
"recording_url": data.get("recording_url"),
|
||||
}
|
||||
|
||||
if analyze and data.get("status") == "completed":
|
||||
questions = [[q.strip(), "string"] for q in analyze.split(",") if q.strip()]
|
||||
if questions:
|
||||
try:
|
||||
analysis = _http("POST", f"{BLAND_API_BASE}/calls/{call_id}/analyze",
|
||||
{"Content-Type": "application/json", "authorization": api_key},
|
||||
{"questions": questions})
|
||||
result["analysis"] = analysis
|
||||
except Exception as e:
|
||||
result["analysis_error"] = str(e)
|
||||
|
||||
return result
|
||||
|
||||
|
||||
# ---------------------------------------------------------------------------
|
||||
# Vapi
|
||||
# ---------------------------------------------------------------------------
|
||||
def _vapi_api_key() -> str:
|
||||
return _env_or_config("VAPI_API_KEY", ["vapi", "api_key"])
|
||||
|
||||
|
||||
def _vapi_phone_number_id() -> str:
|
||||
return _env_or_config("VAPI_PHONE_NUMBER_ID", ["vapi", "phone_number_id"])
|
||||
|
||||
|
||||
def vapi_call(phone_number: str, task: str, voice_id: str | None = None,
|
||||
first_sentence: str | None = None, max_duration: int = 3) -> dict:
|
||||
api_key = _vapi_api_key()
|
||||
if not api_key:
|
||||
print("Error: No Vapi API key. Set VAPI_API_KEY or add to ~/.hermes/config.yaml under phone.vapi.api_key", file=sys.stderr)
|
||||
print("Sign up at https://dashboard.vapi.ai", file=sys.stderr)
|
||||
sys.exit(1)
|
||||
|
||||
phone_number_id = _vapi_phone_number_id()
|
||||
if not phone_number_id:
|
||||
print("Error: No Vapi phone number ID. Vapi requires a Twilio number for outbound calls.", file=sys.stderr)
|
||||
print("Setup: 1) Sign up at twilio.com 2) Buy a number 3) Import into Vapi 4) Set VAPI_PHONE_NUMBER_ID", file=sys.stderr)
|
||||
sys.exit(1)
|
||||
|
||||
voice_provider = _env_or_config("VAPI_VOICE_PROVIDER", ["vapi", "default_voice_provider"], VAPI_DEFAULT_VOICE_PROVIDER)
|
||||
if voice_id is None:
|
||||
voice_id = _env_or_config("VAPI_VOICE_ID", ["vapi", "default_voice_id"], VAPI_DEFAULT_VOICE_ID)
|
||||
model = _env_or_config("VAPI_MODEL", ["vapi", "model"], VAPI_DEFAULT_MODEL)
|
||||
|
||||
assistant = {
|
||||
"model": {
|
||||
"provider": "openai",
|
||||
"model": model,
|
||||
"messages": [{"role": "system", "content": task}],
|
||||
},
|
||||
"voice": {
|
||||
"provider": voice_provider,
|
||||
"voiceId": voice_id,
|
||||
},
|
||||
"maxDurationSeconds": max_duration * 60,
|
||||
}
|
||||
if first_sentence:
|
||||
assistant["firstMessage"] = first_sentence
|
||||
|
||||
payload = {
|
||||
"phoneNumberId": phone_number_id,
|
||||
"customer": {"number": phone_number},
|
||||
"assistant": assistant,
|
||||
}
|
||||
|
||||
result = _http("POST", f"{VAPI_API_BASE}/call",
|
||||
{"Authorization": f"Bearer {api_key}", "Content-Type": "application/json"},
|
||||
payload)
|
||||
|
||||
call_id = result.get("id")
|
||||
if not call_id:
|
||||
print(f"Error: Vapi returned no call id: {json.dumps(result)}", file=sys.stderr)
|
||||
sys.exit(1)
|
||||
|
||||
return {
|
||||
"success": True,
|
||||
"provider": "vapi",
|
||||
"call_id": call_id,
|
||||
"phone_number": phone_number,
|
||||
"voice_provider": voice_provider,
|
||||
"voice_id": voice_id,
|
||||
"max_duration": max_duration,
|
||||
"message": "Call initiated. Use 'status' command to check results.",
|
||||
}
|
||||
|
||||
|
||||
def vapi_status(call_id: str) -> dict:
|
||||
api_key = _vapi_api_key()
|
||||
if not api_key:
|
||||
print("Error: No Vapi API key.", file=sys.stderr)
|
||||
sys.exit(1)
|
||||
|
||||
data = _http("GET", f"{VAPI_API_BASE}/call/{call_id}",
|
||||
{"Authorization": f"Bearer {api_key}"})
|
||||
|
||||
return {
|
||||
"success": True,
|
||||
"provider": "vapi",
|
||||
"status": data.get("status"),
|
||||
"duration_seconds": data.get("duration"),
|
||||
"ended_reason": data.get("endedReason"),
|
||||
"transcript": data.get("transcript", ""),
|
||||
"recording_url": data.get("recordingUrl"),
|
||||
"summary": data.get("summary"),
|
||||
"cost": data.get("cost"),
|
||||
}
|
||||
|
||||
|
||||
# ---------------------------------------------------------------------------
|
||||
# Diagnose — check config
|
||||
# ---------------------------------------------------------------------------
|
||||
def diagnose():
|
||||
provider = _get_provider()
|
||||
print(f"Phone Call Tool — Diagnostics")
|
||||
print("=" * 45)
|
||||
print(f" Active provider: {provider}")
|
||||
|
||||
# Bland
|
||||
bland_key = _bland_api_key()
|
||||
bland_voice = _env_or_config("BLAND_DEFAULT_VOICE", ["bland", "default_voice"], BLAND_DEFAULT_VOICE)
|
||||
print(f"\n Bland.ai:")
|
||||
print(f" API key: {'set' if bland_key else 'NOT SET (BLAND_API_KEY)'}")
|
||||
print(f" Voice: {bland_voice}")
|
||||
|
||||
# Vapi
|
||||
vapi_key = _vapi_api_key()
|
||||
vapi_phone = _vapi_phone_number_id()
|
||||
vapi_voice_provider = _env_or_config("VAPI_VOICE_PROVIDER", ["vapi", "default_voice_provider"], VAPI_DEFAULT_VOICE_PROVIDER)
|
||||
vapi_voice_id = _env_or_config("VAPI_VOICE_ID", ["vapi", "default_voice_id"], VAPI_DEFAULT_VOICE_ID)
|
||||
vapi_model = _env_or_config("VAPI_MODEL", ["vapi", "model"], VAPI_DEFAULT_MODEL)
|
||||
print(f"\n Vapi:")
|
||||
print(f" API key: {'set' if vapi_key else 'NOT SET (VAPI_API_KEY)'}")
|
||||
print(f" Phone number: {'set' if vapi_phone else 'NOT SET (VAPI_PHONE_NUMBER_ID)'}")
|
||||
print(f" Voice: {vapi_voice_provider}:{vapi_voice_id}")
|
||||
print(f" Model: {vapi_model}")
|
||||
|
||||
print(f"\n Bland.ai voices:")
|
||||
for name, desc in BLAND_VOICES.items():
|
||||
print(f" {name:10s} — {desc}")
|
||||
|
||||
# Ready?
|
||||
ready = False
|
||||
if provider == "bland" and bland_key:
|
||||
ready = True
|
||||
elif provider == "vapi" and vapi_key and vapi_phone:
|
||||
ready = True
|
||||
print(f"\n Ready: {'YES' if ready else 'NO — configure API keys above'}")
|
||||
|
||||
|
||||
# ---------------------------------------------------------------------------
|
||||
# CLI
|
||||
# ---------------------------------------------------------------------------
|
||||
def main():
|
||||
args = sys.argv[1:]
|
||||
|
||||
if not args or args[0] in ("-h", "--help", "help"):
|
||||
print(__doc__)
|
||||
sys.exit(0)
|
||||
|
||||
command = args[0]
|
||||
|
||||
if command == "diagnose":
|
||||
diagnose()
|
||||
return
|
||||
|
||||
if command == "call":
|
||||
if len(args) < 3:
|
||||
print("Usage: phone_call.py call <phone_number> <task> [options]", file=sys.stderr)
|
||||
sys.exit(1)
|
||||
|
||||
phone_number = args[1]
|
||||
task = args[2]
|
||||
|
||||
# Parse optional flags
|
||||
voice = None
|
||||
first_sentence = None
|
||||
max_duration = 3
|
||||
i = 3
|
||||
while i < len(args):
|
||||
if args[i] == "--voice" and i + 1 < len(args):
|
||||
voice = args[i + 1]; i += 2
|
||||
elif args[i] == "--first-sentence" and i + 1 < len(args):
|
||||
first_sentence = args[i + 1]; i += 2
|
||||
elif args[i] == "--max-duration" and i + 1 < len(args):
|
||||
max_duration = int(args[i + 1]); i += 2
|
||||
else:
|
||||
print(f"Unknown option: {args[i]}", file=sys.stderr); sys.exit(1)
|
||||
|
||||
if not phone_number.startswith("+"):
|
||||
print(f"Error: Phone number must be E.164 format (e.g. +15551234567), got: {phone_number}", file=sys.stderr)
|
||||
sys.exit(1)
|
||||
|
||||
provider = _get_provider()
|
||||
if provider == "vapi":
|
||||
result = vapi_call(phone_number, task, voice_id=voice,
|
||||
first_sentence=first_sentence, max_duration=max_duration)
|
||||
else:
|
||||
result = bland_call(phone_number, task, voice=voice,
|
||||
first_sentence=first_sentence, max_duration=max_duration)
|
||||
|
||||
print(json.dumps(result, indent=2))
|
||||
|
||||
elif command == "status":
|
||||
if len(args) < 2:
|
||||
print("Usage: phone_call.py status <call_id> [--analyze 'q1,q2']", file=sys.stderr)
|
||||
sys.exit(1)
|
||||
|
||||
call_id = args[1]
|
||||
analyze = None
|
||||
if len(args) > 2 and args[2] == "--analyze" and len(args) > 3:
|
||||
analyze = args[3]
|
||||
|
||||
provider = _get_provider()
|
||||
if provider == "vapi":
|
||||
result = vapi_status(call_id)
|
||||
else:
|
||||
result = bland_status(call_id, analyze=analyze)
|
||||
|
||||
print(json.dumps(result, indent=2, ensure_ascii=False))
|
||||
|
||||
else:
|
||||
print(f"Unknown command: {command}. Use 'call', 'status', or 'diagnose'.", file=sys.stderr)
|
||||
sys.exit(1)
|
||||
|
||||
|
||||
if __name__ == "__main__":
|
||||
main()
|
||||
229
tests/skills/test_telephony_skill.py
Normal file
229
tests/skills/test_telephony_skill.py
Normal file
|
|
@ -0,0 +1,229 @@
|
|||
from __future__ import annotations
|
||||
|
||||
import importlib.util
|
||||
import json
|
||||
import os
|
||||
import sys
|
||||
from pathlib import Path
|
||||
|
||||
|
||||
SCRIPT_PATH = (
|
||||
Path(__file__).resolve().parents[2]
|
||||
/ "optional-skills"
|
||||
/ "productivity"
|
||||
/ "telephony"
|
||||
/ "scripts"
|
||||
/ "telephony.py"
|
||||
)
|
||||
|
||||
|
||||
def load_module():
|
||||
spec = importlib.util.spec_from_file_location("telephony_skill", SCRIPT_PATH)
|
||||
module = importlib.util.module_from_spec(spec)
|
||||
assert spec.loader is not None
|
||||
sys.modules[spec.name] = module
|
||||
spec.loader.exec_module(module)
|
||||
return module
|
||||
|
||||
|
||||
def test_save_twilio_writes_env_and_state(tmp_path: Path, monkeypatch):
|
||||
mod = load_module()
|
||||
monkeypatch.setenv("HERMES_HOME", str(tmp_path / ".hermes"))
|
||||
|
||||
result = mod.save_twilio(
|
||||
"AC123",
|
||||
"secret-token",
|
||||
phone_number="+1 (702) 555-1234",
|
||||
phone_sid="PN123",
|
||||
)
|
||||
|
||||
env_text = (tmp_path / ".hermes" / ".env").read_text(encoding="utf-8")
|
||||
state = json.loads((tmp_path / ".hermes" / "telephony_state.json").read_text(encoding="utf-8"))
|
||||
|
||||
assert result["success"] is True
|
||||
assert "TWILIO_ACCOUNT_SID=AC123" in env_text
|
||||
assert "TWILIO_AUTH_TOKEN=secret-token" in env_text
|
||||
assert "TWILIO_PHONE_NUMBER=+17025551234" in env_text
|
||||
assert "TWILIO_PHONE_NUMBER_SID=PN123" in env_text
|
||||
assert state["twilio"]["default_phone_number"] == "+17025551234"
|
||||
assert state["twilio"]["default_phone_sid"] == "PN123"
|
||||
|
||||
|
||||
def test_upsert_env_updates_existing_values(tmp_path: Path):
|
||||
mod = load_module()
|
||||
env_path = tmp_path / ".env"
|
||||
env_path.write_text("TWILIO_PHONE_NUMBER=+15550000000\nOTHER=keep\n", encoding="utf-8")
|
||||
|
||||
mod._upsert_env_file(
|
||||
{
|
||||
"TWILIO_PHONE_NUMBER": "+15551112222",
|
||||
"TWILIO_PHONE_NUMBER_SID": "PN999",
|
||||
},
|
||||
env_path=env_path,
|
||||
)
|
||||
|
||||
env_text = env_path.read_text(encoding="utf-8")
|
||||
assert "TWILIO_PHONE_NUMBER=+15551112222" in env_text
|
||||
assert "TWILIO_PHONE_NUMBER_SID=PN999" in env_text
|
||||
assert "OTHER=keep" in env_text
|
||||
|
||||
|
||||
def test_messages_after_checkpoint_returns_only_newer_items():
|
||||
mod = load_module()
|
||||
messages = [
|
||||
{"sid": "SM3", "body": "newest"},
|
||||
{"sid": "SM2", "body": "middle"},
|
||||
{"sid": "SM1", "body": "oldest"},
|
||||
]
|
||||
|
||||
assert mod._messages_after_checkpoint(messages, "") == messages
|
||||
assert mod._messages_after_checkpoint(messages, "SM2") == [{"sid": "SM3", "body": "newest"}]
|
||||
assert mod._messages_after_checkpoint(messages, "SM3") == []
|
||||
|
||||
|
||||
def test_twilio_buy_number_saves_env_and_state(tmp_path: Path):
|
||||
mod = load_module()
|
||||
state_path = tmp_path / "telephony_state.json"
|
||||
env_path = tmp_path / ".env"
|
||||
|
||||
mod._twilio_request = lambda method, path, params=None, form=None: {
|
||||
"sid": "PN111",
|
||||
"phone_number": "+17025550123",
|
||||
"friendly_name": "Test Number",
|
||||
"capabilities": {"voice": True, "sms": True},
|
||||
}
|
||||
|
||||
result = mod._twilio_buy_number(
|
||||
"+17025550123",
|
||||
save_env=True,
|
||||
state_path=state_path,
|
||||
env_path=env_path,
|
||||
)
|
||||
|
||||
state = json.loads(state_path.read_text(encoding="utf-8"))
|
||||
env_text = env_path.read_text(encoding="utf-8")
|
||||
|
||||
assert result["phone_sid"] == "PN111"
|
||||
assert state["twilio"]["default_phone_number"] == "+17025550123"
|
||||
assert state["twilio"]["default_phone_sid"] == "PN111"
|
||||
assert "TWILIO_PHONE_NUMBER=+17025550123" in env_text
|
||||
assert "TWILIO_PHONE_NUMBER_SID=PN111" in env_text
|
||||
|
||||
|
||||
def test_twilio_inbox_marks_seen_checkpoint(tmp_path: Path):
|
||||
mod = load_module()
|
||||
state_path = tmp_path / "telephony_state.json"
|
||||
mod._save_state(
|
||||
{
|
||||
"version": 1,
|
||||
"twilio": {
|
||||
"default_phone_number": "+17025550123",
|
||||
"default_phone_sid": "PN111",
|
||||
"last_inbound_message_sid": "SM1",
|
||||
},
|
||||
},
|
||||
state_path,
|
||||
)
|
||||
|
||||
mod._twilio_owned_numbers = lambda limit=50: [
|
||||
mod.OwnedTwilioNumber(
|
||||
sid="PN111",
|
||||
phone_number="+17025550123",
|
||||
friendly_name="Main",
|
||||
capabilities={"voice": True, "sms": True},
|
||||
)
|
||||
]
|
||||
mod._twilio_request = lambda method, path, params=None, form=None: {
|
||||
"messages": [
|
||||
{
|
||||
"sid": "SM3",
|
||||
"direction": "inbound",
|
||||
"status": "received",
|
||||
"from": "+15551230000",
|
||||
"to": "+17025550123",
|
||||
"date_sent": "Tue, 14 Mar 2026 09:00:00 +0000",
|
||||
"body": "new message",
|
||||
"num_media": "0",
|
||||
},
|
||||
{
|
||||
"sid": "SM1",
|
||||
"direction": "inbound",
|
||||
"status": "received",
|
||||
"from": "+15551110000",
|
||||
"to": "+17025550123",
|
||||
"date_sent": "Tue, 14 Mar 2026 08:00:00 +0000",
|
||||
"body": "old message",
|
||||
"num_media": "0",
|
||||
},
|
||||
]
|
||||
}
|
||||
|
||||
result = mod._twilio_inbox(limit=10, since_last=True, mark_seen=True, state_path=state_path)
|
||||
state = json.loads(state_path.read_text(encoding="utf-8"))
|
||||
|
||||
assert result["count"] == 1
|
||||
assert result["messages"][0]["sid"] == "SM3"
|
||||
assert state["twilio"]["last_inbound_message_sid"] == "SM3"
|
||||
|
||||
|
||||
def test_vapi_import_twilio_number_saves_phone_number_id(tmp_path: Path):
|
||||
mod = load_module()
|
||||
state_path = tmp_path / "telephony_state.json"
|
||||
env_path = tmp_path / ".env"
|
||||
|
||||
mod._vapi_api_key = lambda: "vapi-key"
|
||||
mod._twilio_creds = lambda: ("AC123", "token123")
|
||||
mod._resolve_twilio_number = lambda identifier=None: mod.OwnedTwilioNumber(
|
||||
sid="PN111",
|
||||
phone_number="+17025550123",
|
||||
friendly_name="Main",
|
||||
capabilities={"voice": True, "sms": True},
|
||||
)
|
||||
mod._json_request = lambda method, url, headers=None, params=None, form=None, json_body=None: {
|
||||
"id": "vapi-phone-xyz"
|
||||
}
|
||||
|
||||
result = mod._vapi_import_twilio_number(
|
||||
save_env=True,
|
||||
state_path=state_path,
|
||||
env_path=env_path,
|
||||
)
|
||||
|
||||
state = json.loads(state_path.read_text(encoding="utf-8"))
|
||||
env_text = env_path.read_text(encoding="utf-8")
|
||||
|
||||
assert result["phone_number_id"] == "vapi-phone-xyz"
|
||||
assert state["vapi"]["phone_number_id"] == "vapi-phone-xyz"
|
||||
assert "VAPI_PHONE_NUMBER_ID=vapi-phone-xyz" in env_text
|
||||
|
||||
|
||||
def test_diagnose_includes_decision_tree_and_saved_state(tmp_path: Path, monkeypatch):
|
||||
mod = load_module()
|
||||
hermes_home = tmp_path / ".hermes"
|
||||
monkeypatch.setenv("HERMES_HOME", str(hermes_home))
|
||||
mod._save_state(
|
||||
{
|
||||
"version": 1,
|
||||
"twilio": {
|
||||
"default_phone_number": "+17025550123",
|
||||
"last_inbound_message_sid": "SM123",
|
||||
},
|
||||
"vapi": {
|
||||
"phone_number_id": "vapi-abc",
|
||||
},
|
||||
},
|
||||
hermes_home / "telephony_state.json",
|
||||
)
|
||||
(hermes_home / ".env").parent.mkdir(parents=True, exist_ok=True)
|
||||
(hermes_home / ".env").write_text(
|
||||
"TWILIO_ACCOUNT_SID=AC123\nTWILIO_AUTH_TOKEN=token\nBLAND_API_KEY=bland\n",
|
||||
encoding="utf-8",
|
||||
)
|
||||
|
||||
result = mod.diagnose()
|
||||
|
||||
assert result["providers"]["twilio"]["default_phone_number"] == "+17025550123"
|
||||
assert result["providers"]["twilio"]["last_inbound_message_sid"] == "SM123"
|
||||
assert result["providers"]["bland"]["configured"] is True
|
||||
assert result["providers"]["vapi"]["phone_number_id"] == "vapi-abc"
|
||||
assert any(item["use"] == "Twilio" for item in result["decision_tree"])
|
||||
Loading…
Add table
Add a link
Reference in a new issue