[Rex57]

Summary

Closes #1487.

GET /agentmemory/status reported the LLM provider only as "llm" / "noop", and the status page printed that word as the LLM row. The provider is picked by key-detection order (OpenAI → MiniMax → Anthropic → Gemini → OpenRouter), so with several keys set an operator could not tell which client, model and endpoint is billed.

Status now carries provider.route, built from the loaded provider config:

"provider": { "llm": "llm", "route": "openai · deepseek/deepseek-v4.1-flash:floor · openrouter.ai", ... }

• The endpoint part is only the base URL's host; an unparsable base URL shows as custom base URL, so nothing from the setting is echoed.
• route is null without an LLM.
• provider.llm keeps its "llm" / "noop" values, because the viewer and agentmemory status compare against them.
• The status page's LLM row shows the route.

This touches the same status block as #1497 but at different lines; either can merge first.

Checks

• test/llm-route.test.ts: provider + model + host, no base URL, unparsable base URL, noop
• test/status.test.ts: route in the report and on the page, null for noop
• npm run build, npm run skills:gen, npm run docs:sync, npm run skills:check
• npm test: 2600 passed, 4 skipped

Summary by CodeRabbit

• New Features
  • Status reports now display the configured LLM provider route, including its host when available. Providers with no configured route are shown as having none.
• Bug Fixes
  • Invalid base URLs are represented as “custom base URL” rather than exposing the URL.

[vercel]

@Rex57 is attempting to deploy a commit to the rohitg00's projects Team on Vercel.

A member of the Team first needs to authorize it.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: defaults
• Review profile: CHILL
• Plan: Advanced
• Run ID: 461164fa-d3eb-4e0c-a4ed-51cb7cb19cc8

---

📥 Commits

Reviewing files that changed from the base of the PR and between c0548b3 and 596587e.

---

📒 Files selected for processing (3)

• src/config.ts
• src/functions/status.ts
• src/triggers/api.ts

---

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.

---

---

📝 Walkthrough

Walkthrough

The status report now includes a description of the active LLM route, with the provider, model, and optional base-URL host. The status page displays this description when available.

Changes

LLM status route

Layer / File(s)	Summary
Format LLM route descriptions src/config.ts, test/llm-route.test.ts	Adds describeLlmRoute to format the provider and model with the base-URL host when available. Tests cover configured, missing, and invalid base URLs, and the noop provider.
Pass and display the LLM route src/functions/status.ts, src/triggers/api.ts, test/status.test.ts	Adds the route to status inputs and reports, derives it from provider configuration, and displays it in HTML. Tests cover configured routes and noop providers.

Priority: ➖ Normal

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Feature · Severity of issue fixed: Medium

Sequence Diagram(s)

sequenceDiagram
  participant ApiTrigger
  participant describeLlmRoute
  participant evaluateStatus
  participant StatusHtml
  ApiTrigger->>describeLlmRoute: format route from provider configuration
  ApiTrigger->>evaluateStatus: pass llmRoute
  evaluateStatus->>StatusHtml: provide provider route
  StatusHtml->>StatusHtml: render route when present

Loading

Suggested reviewers: rohitg00

---

Merge Risk: ⚪ Minimal · up to 59658

The status report and page now identify the configured LLM route when available. No concrete merge-blocking issue is evident; the change is ready for normal checks.

Pre-merge checks | 4 | 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage		Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 6 functions across 5 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title clearly identifies the main change: status now names the active LLM provider, model, and endpoint. It matches the implementation and PR objectives.
Linked Issues check		Issue #1487 requires the active LLM provider, model, and base-URL host in status output. describeLlmRoute builds these fields, uses custom base URL for an invalid URL, and returns null for `noop…
Out of Scope Changes check		The current change summary shows changes only in LLM configuration, status reporting, API wiring, and related tests. These changes implement or validate issue #1487. The current AGENTS.md content al…

---

• Fix all pre-merge checks with AI

✨ Finishing Touches 💡 1

🛠️ Fix failing CI checks 💡

• Commit to this branch
• Create a new PR

---

🧪 Generate unit tests (beta)

• Create a new PR

---

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

---

_{Comment @coderabbitai help to get the list of available commands.}

[rohitg00]

@Rex57 Please resolve conflicts

[Rex57]

Resolved: merged main into the branch. Only the generated stats files conflicted; I took main's copies and re-ran npm run docs:sync. Build, skills:check and the full suite (2,788 tests) pass locally, and CI is green.