Skip to content

Conversation

@bcherry
Copy link
Contributor

@bcherry bcherry commented Sep 15, 2025

  • Install agents 1.2.11 for inference support
  • Swap OpenAI, Cartesia, and Deepgram plugins for LiveKit Inference in the agent
  • Update the Cartesia voice
  • Swap OpenAI for LiveKit Inference in tests / CI (I already updated the repo secrets)
  • Update README.md to reflect the new changes
  • Place greater emphasis on LiveKit Cloud usage (moved self hosting to a single spot in README rather than inlined throughout)
  • Remove the default weather tool and associated tests to simplify the agent as a starter template

@bcherry bcherry marked this pull request as ready for review September 19, 2025 21:16
Copy link
Contributor

@rektdeckard rektdeckard left a comment

Choose a reason for hiding this comment

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

Nice! Have not run the agent, but looks good

Comment on lines +35 to +50
# To add tools, use the @function_tool decorator.
# Here's an example that adds a simple weather tool.
# You also have to add `from livekit.agents.llm import function_tool, RunContext` to the top of this file
# @function_tool
# async def lookup_weather(self, context: RunContext, location: str):
# """Use this tool to look up current weather information in the given location.
#
# If the location is not supported by the weather service, the tool will indicate this. You must tell the user the location's weather is unavailable.
#
# Args:
# location: The location to look up weather information for (e.g. city name)
# """
#
# logger.info(f"Looking up weather for {location}")
#
# return "sunny with a temperature of 70 degrees."
Copy link
Contributor

Choose a reason for hiding this comment

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

Might be extra clear here to link to tool call docs page and to state that the return value is passed to LLM node?

```bash
lk app env -w .env.local
lk cloud auth
lk app env -w -d .env.local
Copy link
Contributor

Choose a reason for hiding this comment

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

Just my preference, but I like to add the long form so it's more obvious what each does:

lk app env --write --destination .env.local

Copy link

@longcw longcw left a comment

Choose a reason for hiding this comment

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

lgtm!

@rektdeckard rektdeckard merged commit 423d4a2 into main Oct 1, 2025
9 checks passed
@rektdeckard rektdeckard deleted the bcherry/gateway branch October 1, 2025 18:50
divyanshgoel67 pushed a commit to divyanshgoel67/livekit-agent-python that referenced this pull request Nov 30, 2025
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.

4 participants