Conversation
The agent runtime schema (agents-schema.json) and settings UI already accept runtime: chassis alongside plain and reasoning, but the docs only listed the first two. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013ZRDPkg9or671PqULs9kPh
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Automations to automatically generate PRs for you. |
|
Claude finished @keydunov's task in 1m 28s —— View job Review complete — no blockers; 2 low findings (docs clarity, preview gating)Full review
Findings
Both posted inline. Verification limits
What's correct
Review threadsListing returned 0 of my own threads (single page, |
Address review feedback: the row described the mechanism (folds tool calls) but not why a reader would pick it. Reworded to match the plain/reasoning rows' pattern — lead with what it's for, then the user-visible consequence. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013ZRDPkg9or671PqULs9kPh
|
Claude finished @keydunov's task in 1m 36s —— View job Review complete — 1 medium, 2 low; the medium blocks on a factual confirmation, not on the writing.Full review
Findings
On finding 1 — The rest of the change is right: adding a table row instead of a page matches the Review threadsThe listing returned 0 of my own threads — none to resolve, none re-affirmed. |
The docs already call this "tool calls" and the charts/tables they produce (analytics-chat.mdx) — "result cards" was a term found nowhere else in the docs. Match the established vocabulary. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013ZRDPkg9or671PqULs9kPh
|
Claude finished @keydunov's task in 1m 34s —— View job Review: docs-only, no blocking issues — 0 high, 0 medium, 1 low (already tracked on an open thread)Full review
FindingsLow — stale "toolset" wording in the warning ( The Not posted inline: already carried by an open thread on the same line with the same fix direction. Not posted — unverifiable behavior claim (
Checks that came back clean
Review threadsListing returned 3 of my own threads. Resolved 1 ( |
The row was reworded to "semantic-model exploration" in 111351a but the warning below it still said "the chassis toolset", describing the mode with unrelated vocabulary three lines apart. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013ZRDPkg9or671PqULs9kPh
|
Claude finished @keydunov's task in 1m 40s —— View job Docs change is accurate and well-scoped — 1 low finding (wording), posted inlineReview details
Finding
Checked, no finding
Review threadsThe listing returned 0 unresolved threads on the PR (none of them mine), so nothing to resolve and nothing to re-affirm. The three prior |
"Semantic-model exploration" doesn't discriminate between modes -- every agent request explores the semantic model, plain included. Point the escape hatch at the concrete, differentiating behavior from the table row instead (inline results). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013ZRDPkg9or671PqULs9kPh
|
Claude finished @keydunov's task in 2m 9s —— View job Review complete — 1 medium, 1 low. Docs-only;
|
…y change The chassis row only described the inline-results presentation and dropped the capability trade-off: its tool list has no MCP connector tools, workbook creation, scheduled tasks, or dbt sync (verified against packages/ai-engineer/src/services/chassis/Graph.ts). Restore that signal in the runtime table, and add a matching caveat to mcp-connectors.mdx so the trade-off is visible from the side a reader arrives from. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013ZRDPkg9or671PqULs9kPh
|
Claude finished @keydunov's task in 2m 10s —— View job Review complete — docs-only change, no blockers. 0 high, 2 medium, 1 low.Full review
Findings
Verified
Review threadsListing returned 0 of my own threads — no prior threads, so nothing resolved and nothing re-affirmed. |
docs-mintlify/CLAUDE.md reserves the gray <Note> for plan-availability callouts specifically. Stacking a second, unrelated <Note> right after it (with a third — the MCP-server/connector distinction — a few lines below) dilutes that convention. One sentence on the existing paragraph carries the same information. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013ZRDPkg9or671PqULs9kPh
…gent The two-item list read as exhaustive and omitted the Dashboard Agent, which is also a chat surface backed by the configured agent and does respect the runtime setting (only the AI-widget context is hard-excluded in resolve-runtime.ts). Reworded as "such as" and named a third example. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013ZRDPkg9or671PqULs9kPh
|
I'll analyze this and get back to you. |
"the workbook chat sidebar" was off-taxonomy and linked to the workbooks index instead of the dedicated Workbook Agent page. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013ZRDPkg9or671PqULs9kPh
|
Claude finished @keydunov's task in 2m 21s —— View job Review complete — docs-only change, accurate; 1 low findingFull review
VerificationAll four new links resolve to real pages, with titles matching the product terms used:
Finding (low, non-blocking)
Review threadsThe unresolved-thread listing returned 0 threads (mine or anyone's) — nothing to resolve, nothing re-affirmed as a duplicate. NotesNo lint or test run: the change is |
Linking this page from the runtime docs makes a pre-existing contradiction newly reachable: this page states unconditionally that the agent can create reports, build/edit dashboards, and publish -- authoring tools chassis's reduced toolset doesn't have. Same class of fix as the mcp-connectors.mdx caveat earlier in this PR. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013ZRDPkg9or671PqULs9kPh
|
Claude finished @keydunov's task in 2m 19s —— View job Docs-only change, reads accurate against the PR's own cited sources — 1 medium, 1 low. Review details
FindingsMedium — "These authoring capabilities are not available when the agent runs the Low — The page intro still says connector tools "become available to the agent in [Analytics Chat]" unconditionally, ~20 lines above the chassis exception the PR adds at line 36. Not wrong enough to block — the Concepts bullet is the normative statement — but worth a word of hedging next time that paragraph is touched. Verified
Review threadsListing returned 0 of my own threads (single page, |
Verified against the chassis tool list: createWorkbook (report/workbook creation) is confirmed absent, but dashboard-app tools and fileWorkspaceTools are present, so build/edit/publish dashboards are not confirmed missing. Hedge to what's actually evidenced, matching the runtime table row's "some agent features are unavailable" wording. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013ZRDPkg9or671PqULs9kPh
|
Claude finished @keydunov's task in 2m 18s —— View job 1 medium, 0 high — docs-only change, links and anchors all resolveReview details
Review threadsThe listing returned 0 of my own threads (paged to the end — page 1 had 0 nodes and FindingsMedium — This isn't a re-opening of the earlier duplication finding: 62f204e stripped the examples because 83d5c44 had made the row the single source by deleting the per-page notes — but 2a78f90 and e3936f5 restored those notes. With them back, naming the examples here duplicates nothing, and as it stands nothing enumerates the gaps at all. The suggested wording keeps the list explicitly non-exhaustive. Verified, no issue
Not raisedThe Warning at Not runThe behavioural claims (chassis's tool list, the AI-widget runtime exclusion) live in |
Now that mcp-connectors.mdx and workbook-agent.mdx carry their own chassis caveats (linking back to #runtime), this row can link forward to them without duplicating content -- closing the loop for a reader who lands here first, which is the more likely path when deciding whether to set runtime: chassis. "among the features unavailable" keeps the list non-exhaustive. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013ZRDPkg9or671PqULs9kPh
|
Claude finished @keydunov's task in 2m 19s —— View job Docs change is accurate; 2 low findings (1 posted inline, 1 on an untouched file)Review details
Findings1. ~350 characters, three clauses, two inline links, next to one-line 2. The PR adds the "unavailable under Two consistent ways out: add the same one-liner there, or go back to the earlier round's decision that the runtime table is the single source and drop the per-page notes. Either is fine; the current half-and-half is the problem. Fix this → Non-findings, checked
Review threadsThe unresolved listing came back empty — no prior threads of mine on this PR, so none resolved and none re-affirmed. |
The cell had grown to ~350 characters of three-clause prose with two inline links against one-line plain/reasoning rows, breaking the table's scannability. Moved the gap list to a sentence after the Warning, alongside the other runtime-scope prose already there. Keeps both things earlier rounds settled on -- "reduced toolset" (still discriminates from plain) and inline results -- without re-opening either. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013ZRDPkg9or671PqULs9kPh
Reviewing
|
A second reviewer pass found the same unconditional-capability-claim pattern on scheduled-tasks.mdx that mcp-connectors.mdx and workbook-agent.mdx had just been caveated for -- and there is no bound on how many more agent-capability pages exist across the docs site. Chasing each one down is unscoped for a PR documenting one config value. Reverting the two per-page notes and keeping admin/ai/index.mdx as the single place that lists what chassis doesn't support, as an earlier round already proposed once. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013ZRDPkg9or671PqULs9kPh
|
Re: the Went with the reviewer's other offered option instead: reverted the two per-page notes ( Generated by Claude Code |
|
Claude finished @keydunov's task in 2m 20s —— View job Review complete — 1 low finding (docs consistency), no blockersDetails
Posted inline (1)
Checked and clean
Not posted
Verification limitsThe behavioral claims (chassis's tool list, Review threadsThe unresolved-thread listing returned 0 of my own threads (no prior threads). Nothing to resolve, nothing re-affirmed; 1 new thread opened this round. |

Check List
Description of Changes Made
The agent
runtimeconfig accepts a third value,chassis, alongsideplainand
reasoning— it's already in theagents.ymlJSON schema (surfaced asautocomplete in the Semantic Model IDE) and shown as a selectable value in the
Agent Settings UI, but the Runtime docs only covered
plainandreasoning.This adds
chassisto:runtimeproperty description in the Properties tablesemantic-model-focused toolset that places query results inline in the
response instead of as separate tool calls)
reasoning, sincechassislooks similarly early-stage from the surrounding code (heavy,active iteration in the days before this PR)
No new page — this follows the existing table-row pattern used for
reasoning.🤖 Generated with Claude Code
https://claude.ai/code/session_013ZRDPkg9or671PqULs9kPh
Generated by Claude Code