(compat) Removed deprecated properties from IContainerStorageService and IRuntimeStorageService #25708
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…and IRuntimeStorageService
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR removes deprecated properties from IRuntimeStorageService and IContainerStorageService interfaces, constituting a breaking change previously announced in release 2.52.0. The removal includes 9 properties from IRuntimeStorageService and 3 from IContainerStorageService.
Key Changes:
- Removed deprecated methods and properties from storage service interfaces
- Updated type validation configurations to mark breaking changes
- Removed obsolete test code that relied on deprecated functionality
- Updated import statements to remove unused types
Reviewed Changes
Copilot reviewed 25 out of 25 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| packages/runtime/runtime-definitions/src/protocol.ts | Removed 9 deprecated properties from IRuntimeStorageService interface |
| packages/common/container-definitions/src/runtime.ts | Removed 3 deprecated properties from IContainerStorageService interface |
| packages/runtime/container-runtime/src/storageServiceWithAttachBlobs.ts | Removed deprecated method implementations from proxy class |
| packages/loader/container-loader/src/containerStorageAdapter.ts | Removed downloadSummary method and cleaned up imports |
| packages/test/test-end-to-end-tests/src/test/deRehydrateContainerTests.spec.ts | Removed test case validating deprecated getSnapshotTree behavior |
| packages/runtime/container-runtime/src/test/containerRuntime.spec.ts | Changed mock type from IRuntimeStorageService to IContainerStorageService |
| packages/loader/container-loader/src/container.ts | Updated import and parameter type references |
| .changeset/heavy-bugs-thank.md | Added changeset documenting the breaking changes |
| Multiple package.json files | Added type validation entries marking interfaces as breaking backward compatibility |
| Multiple validation files | Added ts-expect-error comments for compatibility type checks |
github-actions
bot
added
area: framework
jason-ha
approved these changes
Oct 20, 2025
Contributor
There was a problem hiding this comment.
I was hoping to see code related to deprecations has lint suppressions. Makes it clear that the code removed was deprecation related.
Contributor
Author
There was a problem hiding this comment.
Moved this non-breaking change to a separate PR and merged it.
agarwal-navin
force-pushed
the
test/breaks/client/2.70/cleanupStorageService
branch
from
1ce13ac to
a81d271
Compare
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| - `disposed` | ||
| - `downloadSummary` | ||
|
|
||
| Please see [this Github issue](https://github.com/microsoft/FluidFramework/issues/25069) for more details. |
Contributor
There was a problem hiding this comment.
🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Github'?
noencke
approved these changes
Oct 23, 2025
Contributor
|
🔗 Found some broken links! 💔 Run a link check locally to find them. See linkcheck output |
Contributor
Author
|
@microsoft-github-policy-service rerun |
noencke
approved these changes
Oct 23, 2025
agarwal-navin
deleted the
test/breaks/client/2.70/cleanupStorageService
branch
This was referenced Oct 23, 2025
anthony-murphy-agent
pushed a commit
to anthony-murphy-agent/FluidFramework
that referenced
this pull request
Jan 14, 2026
…and IRuntimeStorageService (microsoft#25708) ## Description Removed deprecated properties from "IRuntimeStorageService" and "IContainerStorageService" The following deprecated properties have been removed from `IRuntimeStorageService`: - `createBlob` - `dispose` - `disposed` - `downloadSummary` - `getSnapshot` - `getSnapshotTree` - `getVersions` - `policies` - `uploadSummaryWithContext` The following deprecated properties have been removed from `IContainerStorageService`: - `dispose` - `disposed` - `downloadSummary` ## Breaking Changes This is a breaking change. The deprecations were announced in release 2.52.0 [here](https://github.com/microsoft/FluidFramework/releases/tag/client_v2.52.0).
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Removed deprecated properties from "IRuntimeStorageService" and "IContainerStorageService"
The following deprecated properties have been removed from
IRuntimeStorageService:createBlobdisposedisposeddownloadSummarygetSnapshotgetSnapshotTreegetVersionspoliciesuploadSummaryWithContextThe following deprecated properties have been removed from
IContainerStorageService:disposedisposeddownloadSummaryBreaking Changes
This is a breaking change. The deprecations were announced in release 2.52.0 here.