Skip to content

Use dedent for multiline instruction strings#40

Merged
u9g merged 2 commits intomainfrom
jl/use-dedent
Apr 29, 2026
Merged

Use dedent for multiline instruction strings#40
u9g merged 2 commits intomainfrom
jl/use-dedent

Conversation

@u9g
Copy link
Copy Markdown
Contributor

@u9g u9g commented Apr 15, 2026

Summary

  • Use dedent tagged template literal for the agent instructions and commented-out tool description example
  • Prevents code indentation from leaking into runtime string values

Test plan

  • Agent instructions produce the same content without leading whitespace

Prevents code indentation from leaking into runtime string values.
@theomonnom
Copy link
Copy Markdown

Let's wait for a new version to merge this PR

@u9g
Copy link
Copy Markdown
Contributor Author

u9g commented Apr 15, 2026

blocked on release of livekit/agents-js#1259

dedent was introduced in @livekit/agents 1.3.0; the prior ^1.2.4 floor
let CI resolve 1.2.x and fail with "dedent is not a function" / TS2305.
@u9g u9g merged commit 73e9b45 into main Apr 29, 2026
6 checks passed
@u9g u9g deleted the jl/use-dedent branch April 29, 2026 20:46
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