Increase MCP AI limits#17238
Conversation
…crease MCP AI limits
|
🚀 Dogfood this PR with:
curl -fsSL https://raw.githubusercontent.com/microsoft/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 17238Or
iex "& { $(irm https://raw.githubusercontent.com/microsoft/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 17238" |
There was a problem hiding this comment.
Pull request overview
This PR updates Dashboard log virtualization for the newer Blazor Virtualize requirement and raises MCP telemetry result count limits.
Changes:
- Adds a reference-based
ItemComparerforLogViewervirtualization. - Increases shared MCP trace, structured log, and console log count limits.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
src/Shared/ConsoleLogs/SharedAIHelpers.cs |
Raises shared AI telemetry count limits. |
src/Aspire.Dashboard/Components/Controls/LogViewer.razor.cs |
Adds the LogEntry comparer implementation. |
src/Aspire.Dashboard/Components/Controls/LogViewer.razor |
Supplies the comparer to Virtualize. |
|
❓ CLI E2E Tests unknown — 90 passed, 0 failed, 1 unknown (commit View all recordings
📹 Recordings uploaded automatically from CI run #26076709121 |
|
✅ No documentation update needed. docs_optional → internal_refactor No deterministic signals fired (signal_count = 0). The change increases internal telemetry result-count limits ( |
Description
Fix build error BL0011 in the Dashboard's
LogViewercomponent and increase MCP AI tool data limits.MCP AI limits: Increased
TracesLimit(200→800),StructuredLogsLimit(200→800), andConsoleLogsLimit(500→2000) inSharedAIHelpersto provide more context when MCP tools search through telemetry data. Previous limits were small to fix in context window. Size has increased a lot in the last year.Unrelated clean up: Use equality comparer in Virtualize. Required in .NET 11. Changes in this PR by accident but can merge them anyway for the future.
Fixes #14580
Checklist