Skip to content

chore: include AI model config (LLM/STT) in support chat context#4824

Open
devin-ai-integration[bot] wants to merge 2 commits intomainfrom
chore/1774888738-add-model-info-to-support-context
Open

chore: include AI model config (LLM/STT) in support chat context#4824
devin-ai-integration[bot] wants to merge 2 commits intomainfrom
chore/1774888738-add-model-info-to-support-context

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration bot commented Mar 30, 2026

Summary

When users create issues via the in-app chat support, the GitHub issues now include the user's configured AI models (LLM provider/model and STT provider/model) alongside the existing device and account info. This helps with triaging — knowing whether a user is on a local STT model vs cloud, or which LLM provider they use, is useful context for debugging.

Changes:

  • Added ModelInfo struct to template-support crate (LLM/STT provider + model, all optional)
  • Extended SupportContext and the support_context.md.jinja template to render model info
  • useSupportMCP hook now reads model config from TinyBase settings and passes it into the context collector
  • Updated the support chat system prompt to instruct the LLM to include model info in filed issues
  • Manually updated plugins/template/js/bindings.gen.ts (codegen could not run due to missing apple-contact/Cargo.toml in workspace)
  • Fixed pre-existing formatting issue in plugins/tray/src/ext.rs (indentation + extra blank line) to pass fmt CI check

Review & Testing Checklist for Human

  • Verify the manually edited bindings.gen.ts matches what specta codegen would produce. Codegen couldn't run (apple-contact plugin missing Cargo.toml). Run pnpm -F @hypr/plugin-template codegen once the workspace issue is fixed and confirm the ModelInfo type and SupportContext update match.
  • Check the Jinja template indentation in support_context.md.jinja — the endif/if tags have inconsistent indent levels (some at 2 spaces, some at 4) which may produce unexpected whitespace in the rendered markdown context block.
  • Test end-to-end: Open the support chat in the desktop app with LLM and STT models configured, file a bug report, and verify the created GitHub issue body includes an "AI Models" section with the correct LLM/STT provider and model values.

Notes

  • The models field on SupportContext is optional (#[serde(default)]), so this is fully backward-compatible — existing clients without model info will just omit the section.
  • useConfigValues for the model keys returns undefined when not set; the code converts to null via ??.
  • Two non-required CI checks (check and check (macos-latest)) fail due to pre-existing clippy lints in shortcut-macos and cli crates, unrelated to this PR. All relevant checks (fmt, ci, check (ubuntu-latest)) pass.

Link to Devin session: https://app.devin.ai/sessions/2cb1964233d249d4b7e3aebe8a67be82
Requested by: @ComputelessComputer

Co-Authored-By: John <john@hyprnote.com>
@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 30, 2026

Deploy Preview for hyprnote canceled.

Name Link
🔨 Latest commit 2e2a5b1
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote/deploys/69caaaaf1f36640009a48d04

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 30, 2026

Deploy Preview for char-cli-web canceled.

Name Link
🔨 Latest commit 2e2a5b1
🔍 Latest deploy log https://app.netlify.com/projects/char-cli-web/deploys/69caaaaf9500a9000857c749

Co-Authored-By: John <john@hyprnote.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