Skip to content

[browser] Remove rest of StartupMemoryCache #115878

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 22, 2025
Merged

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented May 22, 2025

This PR removes the remaining parts of the StartupMemoryCache feature, which is no longer needed:

  • Removed startupMemoryCache property from BootJsonData class in src/tasks/Microsoft.NET.Sdk.WebAssembly.Pack.Tasks/BootJsonData.cs
  • Removed startupMemoryCache configuration in src/mono/sample/wasm/browser-advanced/main.js
  • Removed StartupMemoryCache parameter from GenerateWasmBootJson task in src/tasks/Microsoft.NET.Sdk.WebAssembly.Pack.Tasks/GenerateWasmBootJson.cs
  • Removed _BlazorWebAssemblyStartupMemoryCache property and its usage in MSBuild file src/mono/nuget/Microsoft.NET.Sdk.WebAssembly.Pack/build/Microsoft.NET.Sdk.WebAssembly.Browser.targets

These changes clean up the codebase by removing a deprecated feature that is no longer used in the WebAssembly runtime.

Fixes #115107.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@dotnet-policy-service dotnet-policy-service bot added the linkable-framework Issues associated with delivering a linker friendly framework label May 22, 2025
Co-authored-by: pavelsavara <271576+pavelsavara@users.noreply.github.com>
@Copilot Copilot AI changed the title [WIP] [browser] remove rest of StartupMemoryCache [browser] Remove rest of StartupMemoryCache May 22, 2025
@Copilot Copilot AI requested a review from pavelsavara May 22, 2025 06:49
Copilot finished work on behalf of pavelsavara May 22, 2025 06:49
@pavelsavara pavelsavara marked this pull request as ready for review May 22, 2025 14:34
@Copilot Copilot AI review requested due to automatic review settings May 22, 2025 14:34
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Remove the remaining vestiges of the deprecated StartupMemoryCache feature across tasks, sample code, and build targets.

  • Dropped the StartupMemoryCache property and its usage from GenerateWasmBootJson
  • Removed the startupMemoryCache field from BootJsonData
  • Cleared the sample JS and MSBuild targets of related configuration

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
src/tasks/Microsoft.NET.Sdk.WebAssembly.Pack.Tasks/GenerateWasmBootJson.cs Removed StartupMemoryCache parameter and assignment
src/tasks/Microsoft.NET.Sdk.WebAssembly.Pack.Tasks/BootJsonData.cs Deleted the nullable startupMemoryCache property
src/mono/sample/wasm/browser-advanced/main.js Eliminated the startupMemoryCache entry in config
src/mono/nuget/Microsoft.NET.Sdk.WebAssembly.Pack/build/Microsoft.NET.Sdk.WebAssembly.Browser.targets Cleared MSBuild properties for startup memory cache

@pavelsavara pavelsavara requested a review from maraf May 22, 2025 14:36
@pavelsavara pavelsavara merged commit 3d23851 into main May 22, 2025
35 checks passed
@akoeplinger akoeplinger deleted the copilot/fix-115107 branch May 22, 2025 16:46
SimaTian pushed a commit that referenced this pull request May 27, 2025
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: pavelsavara <271576+pavelsavara@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
linkable-framework Issues associated with delivering a linker friendly framework
Projects
None yet
Development

Error loading sessions

Retrying...

Successfully merging this pull request may close these issues.

[browser] remove rest of StartupMemoryCache
3 participants