Skip to content

Config graph model#411

Merged
arkml merged 3 commits intodevfrom
config_graph_model
Feb 27, 2026
Merged

Config graph model#411
arkml merged 3 commits intodevfrom
config_graph_model

Conversation

@arkml
Copy link
Contributor

@arkml arkml commented Feb 27, 2026

Here's a summary of all changes:

  1. packages/shared/src/models.ts
  • Added knowledgeGraphModel: z.string().optional() to LlmModelConfig schema
  1. packages/core/src/agents/runtime.ts
  • Added model selection logic: if the agent is note_creation, email-draft, or meeting-prep and
    knowledgeGraphModel is set, use it instead of the assistant model
  • Added a log line via PrefixLogger that prints the model being used when any agent run starts
  1. apps/renderer/src/components/settings-dialog.tsx
  • Renamed "Model" label to "Assistant model"
  • Added "Knowledge graph model" dropdown beside it (side-by-side grid-cols-2 layout)
  • Knowledge graph dropdown has a "Same as assistant" option at the top, selected by default
  • Added knowledgeGraphModel to provider config state, loading, and save payload
  1. apps/renderer/src/components/onboarding-modal.tsx
  • Same changes as settings dialog: side-by-side "Assistant model" / "Knowledge graph model" dropdowns
  • Added knowledgeGraphModel to provider config state and save payload

Key behavior

  • Knowledge graph model defaults to the assistant model when unset
  • Both dropdowns share the same provider/API key — only the model name differs
  • The runtime log shows which model each agent run uses (e.g. [run-abc-note_creation] using model:
    gpt-4o)

@vercel
Copy link

vercel bot commented Feb 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
rowboat Ready Ready Preview, Comment Feb 27, 2026 6:34am

Request Review

@arkml arkml merged commit 9df1bb6 into dev Feb 27, 2026
5 checks passed
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