Skip to content

fix(agent): show duration units in debugger timeout flags - #993

Merged
bcherry merged 2 commits into
mainfrom
bcherry/focused-ptolemy-nq0ybo
Sep 24, 2026
Merged

bcherry merged 2 commits into
mainfrom
bcherry/focused-ptolemy-nq0ybo

Conversation

@bcherry

@bcherry bcherry commented Sep 23, 2026

Copy link
Copy Markdown
Contributor

lk agent debugger start --help printed --idle-timeout stop. The usage string put backticks around stop, and urfave/cli treats a backticked word as the flag's value placeholder. Neither --idle-timeout nor say --timeout said which units the value takes.

Before:

--idle-timeout stop  Stop the session after this long without any command (0 to keep it running until stop) (default: 30m0s)

After:

--idle-timeout DURATION  Stop the session after DURATION without any command, such as 30m, 2h, or 90s (0 keeps it running until stop) (default: 30m0s)
--timeout DURATION       Give up waiting for the agent's reply after DURATION, such as 2m or 90s (default: 2m0s)

Help text only, so no behavior changes. go vet passes, and the debugger unit tests pass.

Found in docs review on livekit/web#6338.

🤖 Generated with Claude Code

https://claude.ai/code/session_01JLeMSp9LH6VsBNFjm8cg6h


Generated by Claude Code

`--idle-timeout` wrapped `stop` in backticks, which urfave/cli reads as
the flag's placeholder, so help printed `--idle-timeout stop`. Use a
DURATION placeholder and give example values for it and `say --timeout`.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JLeMSp9LH6VsBNFjm8cg6h
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@bcherry
bcherry requested a review from u9g September 23, 2026 19:37
@bcherry
bcherry merged commit 3bb6e55 into main Sep 24, 2026
24 of 25 checks passed
@bcherry
bcherry deleted the bcherry/focused-ptolemy-nq0ybo branch September 24, 2026 06:11
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