@fluidframework/container-loader 2.0.0-internal.7.4.0 → 2.0.0-internal.8.0.0

package/CHANGELOG.md

@@ -1,5 +1,31 @@
 # @fluidframework/container-loader
 
+## 2.0.0-internal.8.0.0
+
+### Major Changes
+
+-   container-loader: Removed request(...) and IFluidRouter from ILoader and Loader [9a451d4946](https://github.com/microsoft/FluidFramework/commits/9a451d4946b5c51a52e4d1ab5bf51e7b285b0d74)
+
+    The `request(...)` method and `IFluidRouter` property have been removed from `ILoader` and `Loader`. Instead, after
+    calling `ILoader.resolve(...)`, call the `getEntryPoint()` method on the returned `IContainer`.
+
+    See
+    [Removing-IFluidRouter.md](https://github.com/microsoft/FluidFramework/blob/main/packages/common/core-interfaces/Removing-IFluidRouter.md)
+    for more details.
+
+-   container-definitions: Fix ISnapshotTreeWithBlobContents and mark internal [9a451d4946](https://github.com/microsoft/FluidFramework/commits/9a451d4946b5c51a52e4d1ab5bf51e7b285b0d74)
+
+    `ISnapshotTreeWithBlobContents` is an internal type that should not be used externally. Additionally, the type didn't
+    match the usage, specifically in runtime-utils where an `any` cast was used to work around undefined blobContents. The
+    type has been updated to reflect that blobContents can be undefined.
+
+-   container-loader: Removed requestResolvedObjectFromContainer [9a451d4946](https://github.com/microsoft/FluidFramework/commits/9a451d4946b5c51a52e4d1ab5bf51e7b285b0d74)
+
+    The helper function `requestResolvedObjectFromContainer` has been removed. Please remove all calls to it and instead use
+    the new `entryPoint` pattern. See
+    [Removing-IFluidRouter.md](https://github.com/microsoft/FluidFramework/blob/main/packages/common/core-interfaces/Removing-IFluidRouter.md)
+    for more details.
+
 ## 2.0.0-internal.7.4.0
 
 Dependency updates only.

package/api-report/container-loader.api.md

@@ -14,7 +14,6 @@ import { IDocumentServiceFactory } from '@fluidframework/driver-definitions';
 import { IDocumentStorageService } from '@fluidframework/driver-definitions';
 import { IFluidCodeDetails } from '@fluidframework/container-definitions';
 import { IFluidModule } from '@fluidframework/container-definitions';
-import { IFluidRouter } from '@fluidframework/core-interfaces';
 import { IHostLoader } from '@fluidframework/container-definitions';
 import { ILoaderOptions as ILoaderOptions_2 } from '@fluidframework/container-definitions';
 import { ILocationRedirectionError } from '@fluidframework/driver-definitions';
@@ -22,8 +21,6 @@ import { IProtocolHandler as IProtocolHandler_2 } from '@fluidframework/protocol
 import { IProvideFluidCodeDetailsComparer } from '@fluidframework/container-definitions';
 import { IQuorumSnapshot } from '@fluidframework/protocol-base';
 import { IRequest } from '@fluidframework/core-interfaces';
-import { IRequestHeader } from '@fluidframework/core-interfaces';
-import { IResponse } from '@fluidframework/core-interfaces';
 import { ISignalMessage } from '@fluidframework/protocol-definitions';
 import { ITelemetryBaseLogger } from '@fluidframework/core-interfaces';
 import { ITelemetryLoggerExt } from '@fluidframework/telemetry-utils';
@@ -118,15 +115,11 @@ export class Loader implements IHostLoader {
         canReconnect?: boolean;
         clientDetailsOverride?: IClientDetails;
     }): Promise<IContainer>;
-    // @deprecated (undocumented)
-    get IFluidRouter(): IFluidRouter;
     // (undocumented)
     rehydrateDetachedContainerFromSnapshot(snapshot: string, createDetachedProps?: {
         canReconnect?: boolean;
         clientDetailsOverride?: IClientDetails;
     }): Promise<IContainer>;
-    // @deprecated (undocumented)
-    request(request: IRequest): Promise<IResponse>;
     // (undocumented)
     resolve(request: IRequest, pendingLocalState?: string): Promise<IContainer>;
     // (undocumented)
@@ -136,9 +129,6 @@ export class Loader implements IHostLoader {
 // @alpha
 export type ProtocolHandlerBuilder = (attributes: IDocumentAttributes, snapshot: IQuorumSnapshot, sendProposal: (key: string, value: any) => number) => IProtocolHandler;
 
-// @internal @deprecated
-export function requestResolvedObjectFromContainer(container: IContainer, headers?: IRequestHeader): Promise<IResponse>;
-
 // @internal
 export function resolveWithLocationRedirectionHandling<T>(api: (request: IRequest) => Promise<T>, request: IRequest, urlResolver: IUrlResolver, logger?: ITelemetryBaseLogger): Promise<T>;
 

package/dist/container-loader-alpha.d.ts

@@ -8,7 +8,6 @@ import { IDocumentServiceFactory } from '@fluidframework/driver-definitions';
 import { IDocumentStorageService } from '@fluidframework/driver-definitions';
 import { IFluidCodeDetails } from '@fluidframework/container-definitions';
 import { IFluidModule } from '@fluidframework/container-definitions';
-import { IFluidRouter } from '@fluidframework/core-interfaces';
 import { IHostLoader } from '@fluidframework/container-definitions';
 import { ILoaderOptions as ILoaderOptions_2 } from '@fluidframework/container-definitions';
 import { ILocationRedirectionError } from '@fluidframework/driver-definitions';
@@ -16,8 +15,6 @@ import { IProtocolHandler as IProtocolHandler_2 } from '@fluidframework/protocol
 import { IProvideFluidCodeDetailsComparer } from '@fluidframework/container-definitions';
 import { IQuorumSnapshot } from '@fluidframework/protocol-base';
 import { IRequest } from '@fluidframework/core-interfaces';
-import { IRequestHeader } from '@fluidframework/core-interfaces';
-import { IResponse } from '@fluidframework/core-interfaces';
 import { ISignalMessage } from '@fluidframework/protocol-definitions';
 import { ITelemetryBaseLogger } from '@fluidframework/core-interfaces';
 import { ITelemetryLoggerExt } from '@fluidframework/telemetry-utils';
@@ -202,10 +199,6 @@ export declare class Loader implements IHostLoader {
     readonly services: ILoaderServices;
     private readonly mc;
     constructor(loaderProps: ILoaderProps);
-    /**
-     * @deprecated Will be removed in future major release. Migrate all usage of IFluidRouter to the Container's IFluidRouter/request.
-     */
-    get IFluidRouter(): IFluidRouter;
     createDetachedContainer(codeDetails: IFluidCodeDetails, createDetachedProps?: {
         canReconnect?: boolean;
         clientDetailsOverride?: IClientDetails;
@@ -215,10 +208,6 @@ export declare class Loader implements IHostLoader {
         clientDetailsOverride?: IClientDetails;
     }): Promise<IContainer>;
     resolve(request: IRequest, pendingLocalState?: string): Promise<IContainer>;
-    /**
-     * @deprecated Will be removed in future major release. Migrate all usage of IFluidRouter to the Container's IFluidRouter/request.
-     */
-    request(request: IRequest): Promise<IResponse>;
     private resolveCore;
     private loadContainer;
 }
@@ -229,8 +218,6 @@ export declare class Loader implements IHostLoader {
  */
 export declare type ProtocolHandlerBuilder = (attributes: IDocumentAttributes, snapshot: IQuorumSnapshot, sendProposal: (key: string, value: any) => number) => IProtocolHandler;
 
-/* Excluded from this release type: requestResolvedObjectFromContainer */
-
 /* Excluded from this release type: resolveWithLocationRedirectionHandling */
 
 /* Excluded from this release type: tryParseCompatibleResolvedUrl */

package/dist/container-loader-beta.d.ts

@@ -8,7 +8,6 @@ import { IDocumentServiceFactory } from '@fluidframework/driver-definitions';
 import { IDocumentStorageService } from '@fluidframework/driver-definitions';
 import { IFluidCodeDetails } from '@fluidframework/container-definitions';
 import { IFluidModule } from '@fluidframework/container-definitions';
-import { IFluidRouter } from '@fluidframework/core-interfaces';
 import { IHostLoader } from '@fluidframework/container-definitions';
 import { ILoaderOptions as ILoaderOptions_2 } from '@fluidframework/container-definitions';
 import { ILocationRedirectionError } from '@fluidframework/driver-definitions';
@@ -16,8 +15,6 @@ import { IProtocolHandler as IProtocolHandler_2 } from '@fluidframework/protocol
 import { IProvideFluidCodeDetailsComparer } from '@fluidframework/container-definitions';
 import { IQuorumSnapshot } from '@fluidframework/protocol-base';
 import { IRequest } from '@fluidframework/core-interfaces';
-import { IRequestHeader } from '@fluidframework/core-interfaces';
-import { IResponse } from '@fluidframework/core-interfaces';
 import { ISignalMessage } from '@fluidframework/protocol-definitions';
 import { ITelemetryBaseLogger } from '@fluidframework/core-interfaces';
 import { ITelemetryLoggerExt } from '@fluidframework/telemetry-utils';
@@ -49,8 +46,6 @@ import { IUrlResolver } from '@fluidframework/driver-definitions';
 
 /* Excluded from this release type: IFluidModuleWithDetails */
 
-/* Excluded from this release type: IFluidRouter */
-
 /* Excluded from this release type: IHostLoader */
 
 /* Excluded from this release type: ILoaderOptions */
@@ -71,8 +66,6 @@ import { IUrlResolver } from '@fluidframework/driver-definitions';
 
 /* Excluded from this release type: IRequest */
 
-/* Excluded from this release type: IResponse */
-
 /* Excluded from this release type: isLocationRedirectionError */
 
 /* Excluded from this release type: ITelemetryBaseLogger */
@@ -85,8 +78,6 @@ import { IUrlResolver } from '@fluidframework/driver-definitions';
 
 /* Excluded from this release type: ProtocolHandlerBuilder */
 
-/* Excluded from this release type: requestResolvedObjectFromContainer */
-
 /* Excluded from this release type: resolveWithLocationRedirectionHandling */
 
 /* Excluded from this release type: tryParseCompatibleResolvedUrl */

package/dist/container-loader-public.d.ts

@@ -8,7 +8,6 @@ import { IDocumentServiceFactory } from '@fluidframework/driver-definitions';
 import { IDocumentStorageService } from '@fluidframework/driver-definitions';
 import { IFluidCodeDetails } from '@fluidframework/container-definitions';
 import { IFluidModule } from '@fluidframework/container-definitions';
-import { IFluidRouter } from '@fluidframework/core-interfaces';
 import { IHostLoader } from '@fluidframework/container-definitions';
 import { ILoaderOptions as ILoaderOptions_2 } from '@fluidframework/container-definitions';
 import { ILocationRedirectionError } from '@fluidframework/driver-definitions';
@@ -16,8 +15,6 @@ import { IProtocolHandler as IProtocolHandler_2 } from '@fluidframework/protocol
 import { IProvideFluidCodeDetailsComparer } from '@fluidframework/container-definitions';
 import { IQuorumSnapshot } from '@fluidframework/protocol-base';
 import { IRequest } from '@fluidframework/core-interfaces';
-import { IRequestHeader } from '@fluidframework/core-interfaces';
-import { IResponse } from '@fluidframework/core-interfaces';
 import { ISignalMessage } from '@fluidframework/protocol-definitions';
 import { ITelemetryBaseLogger } from '@fluidframework/core-interfaces';
 import { ITelemetryLoggerExt } from '@fluidframework/telemetry-utils';
@@ -49,8 +46,6 @@ import { IUrlResolver } from '@fluidframework/driver-definitions';
 
 /* Excluded from this release type: IFluidModuleWithDetails */
 
-/* Excluded from this release type: IFluidRouter */
-
 /* Excluded from this release type: IHostLoader */
 
 /* Excluded from this release type: ILoaderOptions */
@@ -71,8 +66,6 @@ import { IUrlResolver } from '@fluidframework/driver-definitions';
 
 /* Excluded from this release type: IRequest */
 
-/* Excluded from this release type: IResponse */
-
 /* Excluded from this release type: isLocationRedirectionError */
 
 /* Excluded from this release type: ITelemetryBaseLogger */
@@ -85,8 +78,6 @@ import { IUrlResolver } from '@fluidframework/driver-definitions';
 
 /* Excluded from this release type: ProtocolHandlerBuilder */
 
-/* Excluded from this release type: requestResolvedObjectFromContainer */
-
 /* Excluded from this release type: resolveWithLocationRedirectionHandling */
 
 /* Excluded from this release type: tryParseCompatibleResolvedUrl */

package/dist/container-loader-untrimmed.d.ts

@@ -8,7 +8,6 @@ import { IDocumentServiceFactory } from '@fluidframework/driver-definitions';
 import { IDocumentStorageService } from '@fluidframework/driver-definitions';
 import { IFluidCodeDetails } from '@fluidframework/container-definitions';
 import { IFluidModule } from '@fluidframework/container-definitions';
-import { IFluidRouter } from '@fluidframework/core-interfaces';
 import { IHostLoader } from '@fluidframework/container-definitions';
 import { ILoaderOptions as ILoaderOptions_2 } from '@fluidframework/container-definitions';
 import { ILocationRedirectionError } from '@fluidframework/driver-definitions';
@@ -16,8 +15,6 @@ import { IProtocolHandler as IProtocolHandler_2 } from '@fluidframework/protocol
 import { IProvideFluidCodeDetailsComparer } from '@fluidframework/container-definitions';
 import { IQuorumSnapshot } from '@fluidframework/protocol-base';
 import { IRequest } from '@fluidframework/core-interfaces';
-import { IRequestHeader } from '@fluidframework/core-interfaces';
-import { IResponse } from '@fluidframework/core-interfaces';
 import { ISignalMessage } from '@fluidframework/protocol-definitions';
 import { ITelemetryBaseLogger } from '@fluidframework/core-interfaces';
 import { ITelemetryLoggerExt } from '@fluidframework/telemetry-utils';
@@ -271,10 +268,6 @@ export declare class Loader implements IHostLoader {
     readonly services: ILoaderServices;
     private readonly mc;
     constructor(loaderProps: ILoaderProps);
-    /**
-     * @deprecated Will be removed in future major release. Migrate all usage of IFluidRouter to the Container's IFluidRouter/request.
-     */
-    get IFluidRouter(): IFluidRouter;
     createDetachedContainer(codeDetails: IFluidCodeDetails, createDetachedProps?: {
         canReconnect?: boolean;
         clientDetailsOverride?: IClientDetails;
@@ -284,10 +277,6 @@ export declare class Loader implements IHostLoader {
         clientDetailsOverride?: IClientDetails;
     }): Promise<IContainer>;
     resolve(request: IRequest, pendingLocalState?: string): Promise<IContainer>;
-    /**
-     * @deprecated Will be removed in future major release. Migrate all usage of IFluidRouter to the Container's IFluidRouter/request.
-     */
-    request(request: IRequest): Promise<IResponse>;
     private resolveCore;
     private loadContainer;
 }
@@ -298,15 +287,6 @@ export declare class Loader implements IHostLoader {
  */
 export declare type ProtocolHandlerBuilder = (attributes: IDocumentAttributes, snapshot: IQuorumSnapshot, sendProposal: (key: string, value: any) => number) => IProtocolHandler;
 
-/**
- * With an already-resolved container, we can request a component directly, without loading the container again
- * @param container - a resolved container
- * @returns component on the container
- * @deprecated Will be removed in future major release. Migrate all usage of IFluidRouter to the "entryPoint" pattern. Refer to Removing-IFluidRouter.md
- * @internal
- */
-export declare function requestResolvedObjectFromContainer(container: IContainer, headers?: IRequestHeader): Promise<IResponse>;
-
 /**
  * Handles location redirection while fulfilling the loader request.
  * @param api - Callback in which user can wrap the loader.resolve or loader.request call.

package/dist/container.cjs

@@ -223,13 +223,6 @@ class Container extends telemetry_utils_1.EventEmitterWithErrorHandling {
     get connectionMode() {
         return this._deltaManager.connectionManager.connectionMode;
     }
-    /**
-     * @deprecated Will be removed in future major release. Migrate all usage of IFluidRouter to the "entryPoint" pattern. Refer to Removing-IFluidRouter.md
-     */
-    // eslint-disable-next-line import/no-deprecated
-    get IFluidRouter() {
-        return this;
-    }
     get resolvedUrl() {
         /**
          * All attached containers will have a document service,
@@ -803,12 +796,6 @@ class Container extends telemetry_utils_1.EventEmitterWithErrorHandling {
             }
         }, { start: true, end: true, cancel: "generic" });
     }
-    /**
-     * @deprecated Will be removed in future major release. Migrate all usage of IFluidRouter to the "entryPoint" pattern. Refer to Removing-IFluidRouter.md
-     */
-    async request(path) {
-        return telemetry_utils_1.PerformanceEvent.timedExecAsync(this.mc.logger, { eventName: "Request" }, async () => this.runtime.request(path), { end: true, cancel: "error" });
-    }
     setAutoReconnectInternal(mode, reason) {
         const currentMode = this._deltaManager.connectionManager.reconnectMode;
         if (currentMode === mode) {