Update project dependencies (#12118)#12119
Conversation
WalkthroughThis pull request updates NuGet package dependencies across multiple project files and a central package management configuration. Updates include Maui-related packages (10.0.40 → 10.0.41), Semantic Kernel packages (1.71.x/1.72.x versions), ModelContextProtocol (0.8.0-preview.1 → 0.9.0-preview.2), Scalar.AspNetCore (2.12.41 → 2.12.46), and Microsoft.Agents.AI packages. These changes address outdated dependencies referenced in issue Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Tip Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord. 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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
src/Websites/Platform/src/Bit.Websites.Platform.Server/Bit.Websites.Platform.Server.csproj (1)
1-1:⚠️ Potential issue | 🔴 CriticalCI pipeline failure must be resolved before merging.
The GitHub Actions workflow "bit platform CI - Websites" reports:
The target "InstallNodejsDependencies" does not exist in the project. This MSBuild target failure is unrelated to the NuGet version bump but blocks the build. Verify whether this target was removed from a referenced project or the workflow YAML references a stale target name.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@src/Websites/Platform/src/Bit.Websites.Platform.Server/Bit.Websites.Platform.Server.csproj` at line 1, The CI failure indicates MSBuild cannot find the target "InstallNodejsDependencies" referenced during the "bit platform CI - Websites" workflow; inspect the project Bit.Websites.Platform.Server.csproj and any imported project files for removal/renaming of the InstallNodejsDependencies target or imports that used to define it, then either restore the target (add a Target named InstallNodejsDependencies that runs the node/npm tasks or re-import the .targets file that provided it) or update the workflow YAML to call the current/still-existing target name (or remove the step if no longer needed); search for the symbol InstallNodejsDependencies and any MSBuild <Import> entries to locate where to change either the csproj or the workflow.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In
`@src/Websites/Platform/src/Bit.Websites.Platform.Server/Bit.Websites.Platform.Server.csproj`:
- Line 30: Upgrade to ModelContextProtocol 0.9.0-preview.2 introduces breaking
API/type changes; update code that depends on old patterns by: running
integration tests against the external DeepWiki MCP endpoint and exercising
McpClient.CreateAsync and HttpClientTransport usage to catch runtime mismatches;
replace any Add*Filter server registrations with the new
WithMessageFilters/WithRequestFilters fluent calls used by
AddMcpServer().WithHttpTransport().WithToolsFromAssembly(); audit handler option
wiring and adapt plumbing where handler options are passed; change binary
payload handling from string to ReadOnlyMemory<byte> wherever you read/write
binary content; update collection types from List<T> to IList<T> (and adjust any
deserialization expectations in ListToolsAsync/tool deserializers); and update
JSON usages from JsonElement? to JsonObject? ensuring deserializers and tool
wiring handle the new types—run unit and integration tests to validate tool
deserialization and runtime behavior.
---
Outside diff comments:
In
`@src/Websites/Platform/src/Bit.Websites.Platform.Server/Bit.Websites.Platform.Server.csproj`:
- Line 1: The CI failure indicates MSBuild cannot find the target
"InstallNodejsDependencies" referenced during the "bit platform CI - Websites"
workflow; inspect the project Bit.Websites.Platform.Server.csproj and any
imported project files for removal/renaming of the InstallNodejsDependencies
target or imports that used to define it, then either restore the target (add a
Target named InstallNodejsDependencies that runs the node/npm tasks or re-import
the .targets file that provided it) or update the workflow YAML to call the
current/still-existing target name (or remove the step if no longer needed);
search for the symbol InstallNodejsDependencies and any MSBuild <Import> entries
to locate where to change either the csproj or the workflow.
closes #12118
Summary by CodeRabbit