Agreed. I would make agent_provider.forward_host_credentials an explicit opt-in gate for this path. Default stays false, so a Codex bottle can still use device login interactively, but…
Re #732: Agreed. I will update the Claude-provider examples/docs to use BOT_BOTTLE_CLAUDE_OAUTH_TOKEN instead of the generic BOT_BOTTLE_OAUTH_TOKEN, and make the auth section explicit that…
Plan of attack after review feedback:
- Keep
agent_provider.templateas the manifest key. No rename totype. - Update docs/examples to recommend home-owned base bottles like
claude/codex…
Re #719: My recommendation is to keep template. In this design claude/codex selects a bundled runtime template: command, image, Dockerfile, prompt behavior, and auth placeholder. type…
Re #715: Agreed. I will rename this to capture_claude_session_state and add the FIXME/comment that the current transcript/session capture is agent-specific; longer term we should spike freezing…
Re #713: Yes, those modes are behavior-specific rather than provider-specific. I will rename them to append_file and read_prompt_file, and add explicit type hints for the accepted prompt modes…
Re #717: Agreed. I will rename provider_template variables/parameters to agent_provider_template throughout so the code matches the manifest shape and avoids the vague provider/template split.
Re #711: Agreed. I will rename the local variables and docstrings in that path from claude_* to agent_*, while leaving only explicitly Claude-only state/transcript pieces named Claude.
Re #707: The duplicate was a compatibility shortcut: _agent_command for the handle internals, agent_command for dashboard/tmux code that introspects the running handle. I agree it is…
Re #709: Correct. That surface is agent-agnostic now. I will rename it to agent_argv and follow through with exec_agent/call-site/test renames, keeping temporary compatibility aliases only if…
Re #705: Agreed. I will do the project rename as its own single commit and include the image rename to bot-bottle-claude:latest. I will also sweep docs, package/module names, env var names,…
Re #701: I agree the template cue should not implicitly add egress routes. That would hide a security boundary behind provider selection. My preference is a home-owned base bottle like claude…