chore: add missing .env.sample files to voice-agents examples - #80
Open
ayushsingh82 wants to merge 1 commit into
Open
chore: add missing .env.sample files to voice-agents examples#80ayushsingh82 wants to merge 1 commit into
ayushsingh82 wants to merge 1 commit into
Conversation
The root README and CONTRIBUTING state every example ships a .env.sample, but these 11 voice-agent examples had none despite their code calling load_dotenv() / os.getenv(). Add a sample for each listing exactly the variables that example reads: - agent_with_tools, call_control, getting_started, inbound_ivr, interrupt_control, language_switching: SMALLEST_API_KEY, OPENAI_API_KEY - appointment_scheduler: + CAL_API_KEY, CAL_EVENT_TYPE_ID, CAL_TIMEZONE - bank_csr: + TRANSFER_NUMBER - campaigns: SMALLEST_API_KEY, AGENT_ID, AUDIENCE_ID, CAMPAIGN_ID - form_filler: + JOTFORM_API_KEY, JOTFORM_FORM_ID - observability: + LANGFUSE_PUBLIC_KEY, LANGFUSE_SECRET_KEY, LANGFUSE_HOST Style matches the existing telephony/ and pipecat-voice-agent/ samples.
|
crim doesn't review pull requests automatically here. Comment |
|
@ayushsingh82 is attempting to deploy a commit to the developer-2074's projects Team on Vercel. A member of the Team first needs to authorize it. |
Author
|
@abhishekmishragithub mind taking a look? Config-only — adds the per-example |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The root
README.md("a.env.sample[is] included in every example directory") andCONTRIBUTING.md("Create a.env.samplelisting all required variables") both treat a per-example.env.sampleas standard, but these 11 voice-agent examples shipped without one — even though their code callsload_dotenv()/os.getenv(...). Following the README as written leaves you with a missing-credentials error and no hint which vars to set.Each new file lists exactly the variables that example reads, styled to match the existing
telephony/andpipecat-voice-agent/samples:agent_with_tools,call_control,getting_started,inbound_ivr,interrupt_control,language_switchingSMALLEST_API_KEY,OPENAI_API_KEYappointment_schedulerCAL_API_KEY,CAL_EVENT_TYPE_ID,CAL_TIMEZONEbank_csrTRANSFER_NUMBERcampaignsSMALLEST_API_KEY,AGENT_ID,AUDIENCE_ID,CAMPAIGN_IDform_fillerJOTFORM_API_KEY,JOTFORM_FORM_IDobservabilityLANGFUSE_PUBLIC_KEY,LANGFUSE_SECRET_KEY,LANGFUSE_HOSTDocs-only / config-only — no code changes.
Scope note: examples under
integrations/langchain/,blog-code-samples/,speech-to-text/benchmarks,speech-to-text/youtube-summarizerandtext-to-speech/quickstartare also missing a sample, and a few dirs use.env.exampleinstead of the documented.env.samplename. Happy to follow up on those separately if you'd like.