Skip to content

mng/llm-agent-impl#858

Open
joshalbrecht wants to merge 4 commits intojosh/llm_agentfrom
mng/llm-agent-impl
Open

mng/llm-agent-impl#858
joshalbrecht wants to merge 4 commits intojosh/llm_agentfrom
mng/llm-agent-impl

Conversation

@joshalbrecht
Copy link
Contributor

Automated PR created by Claude Code session.

joshalbrecht and others added 4 commits March 12, 2026 16:54
- Refactor chat.sh to support --new --name <name> interface. Conversations
  are looked up by name and resumed if they already exist, or created if
  not. This makes the LlmAgent idempotent across restarts.
- Add MNG_LLM_MODEL env var for model selection (replaces reading
  minds.toml in chat.sh). Both LlmAgent and ClaudeMindAgent set this
  via set_llm_model_env_var() in modify_env_vars().
- Add template YAML generation in chat.sh: system prompt is built from
  GLOBAL.md and talking/PROMPT.md and passed to llm via -t flag.
- Add lookup-by-name subcommand to llmdb for name-based conversation
  lookup from the mind_conversations table.
- Update LlmAgent.assemble_command() to use the new chat.sh interface.
- Update mng chat CLI with --name option and interactive name prompt.
- Update skills, ttyd dispatch, and all affected tests.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Add 5 tests for conversation_db.lookup_by_name: found, not found,
  most recent match, missing db, and main() dispatch.
- Add 3 tests for plugin.set_llm_model_env_var: default model when
  no settings, custom model from settings, default when model key
  absent from settings.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…t yet

The conversations table is created by the llm tool on first use, so
"no such table" errors are expected on fresh databases. Silently return
the zero/empty fallback instead of printing a WARNING to stderr.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The conversations table may not exist on first run; the fallback to 0
already handles this correctly, so just discard the warning.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

1 participant