@fluidframework/routerlicious-driver 2.91.0 → 2.92.0

package/src/documentDeltaConnection.ts

@@ -4,24 +4,20 @@
  */
 
 import { DocumentDeltaConnection } from "@fluidframework/driver-base/internal";
-import { IClient } from "@fluidframework/driver-definitions";
-import {
+import type { IClient } from "@fluidframework/driver-definitions";
+import type {
 	IDocumentDeltaConnection,
 	IAnyDriverError,
 	IConnect,
 } from "@fluidframework/driver-definitions/internal";
 import type { DriverErrorTelemetryProps } from "@fluidframework/driver-utils/internal";
-import { ITelemetryLoggerExt } from "@fluidframework/telemetry-utils/internal";
+import type { ITelemetryLoggerExt } from "@fluidframework/telemetry-utils/internal";
 import type { Socket } from "socket.io-client";
 
-import {
-	IR11sSocketError,
-	errorObjectFromSocketError,
-	getUrlForTelemetry,
-	socketIoPath,
-} from "./errorUtils.js";
+import type { IR11sSocketError } from "./errorUtils.js";
+import { errorObjectFromSocketError, getUrlForTelemetry, socketIoPath } from "./errorUtils.js";
 import { pkgVersion as driverVersion } from "./packageVersion.js";
-import { SocketIOClientStatic } from "./socketModule.js";
+import type { SocketIOClientStatic } from "./socketModule.js";
 
 const protocolVersions = ["^0.4.0", "^0.3.0", "^0.2.0", "^0.1.0"];
 

package/src/documentService.ts

@@ -5,8 +5,8 @@
 
 import { TypedEventEmitter } from "@fluid-internal/client-utils";
 import { assert } from "@fluidframework/core-utils/internal";
-import { IClient } from "@fluidframework/driver-definitions";
-import {
+import type { IClient } from "@fluidframework/driver-definitions";
+import type {
 	IDocumentServiceEvents,
 	IDocumentServicePolicies,
 	IDocumentDeltaConnection,
@@ -21,15 +21,12 @@ import {
 	RateLimiter,
 	canRetryOnError,
 } from "@fluidframework/driver-utils/internal";
-import {
-	ITelemetryLoggerExt,
-	PerformanceEvent,
-	wrapError,
-} from "@fluidframework/telemetry-utils/internal";
+import type { ITelemetryLoggerExt } from "@fluidframework/telemetry-utils/internal";
+import { PerformanceEvent, wrapError } from "@fluidframework/telemetry-utils/internal";
 
-import { ICache } from "./cache.js";
-import { INormalizedWholeSnapshot } from "./contracts.js";
-import { ISnapshotTreeVersion } from "./definitions.js";
+import type { ICache } from "./cache.js";
+import type { INormalizedWholeSnapshot } from "./contracts.js";
+import type { ISnapshotTreeVersion } from "./definitions.js";
 import { DeltaStorageService, DocumentDeltaStorageService } from "./deltaStorageService.js";
 import { R11sDocumentDeltaConnection } from "./documentDeltaConnection.js";
 import { DocumentStorageService } from "./documentStorageService.js";
@@ -38,16 +35,16 @@ import { GitManager } from "./gitManager.js";
 import { Historian } from "./historian.js";
 import { NullBlobStorageService } from "./nullBlobStorageService.js";
 import { pkgVersion as driverVersion } from "./packageVersion.js";
-import { IRouterliciousDriverPolicies } from "./policies.js";
+import type { IRouterliciousDriverPolicies } from "./policies.js";
+import type { TokenFetcher } from "./restWrapper.js";
 import {
 	RouterliciousOrdererRestWrapper,
 	RouterliciousStorageRestWrapper,
-	TokenFetcher,
 } from "./restWrapper.js";
-import { RestWrapper } from "./restWrapperBase.js";
+import type { RestWrapper } from "./restWrapperBase.js";
 import type { IGetSessionInfoResponse } from "./sessionInfoManager.js";
 import { SocketIOClientStatic } from "./socketModule.js";
-import { ITokenProvider } from "./tokens.js";
+import type { ITokenProvider } from "./tokens.js";
 
 /**
  * The DocumentService manages the Socket.IO connection and manages routing requests to connected

package/src/documentServiceFactory.ts

@@ -7,35 +7,34 @@ import type {
 	ILayerCompatDetails,
 	IProvideLayerCompatDetails,
 } from "@fluid-internal/client-utils";
-import { ITelemetryBaseLogger } from "@fluidframework/core-interfaces";
+import type { ITelemetryBaseLogger } from "@fluidframework/core-interfaces";
 import { assert } from "@fluidframework/core-utils/internal";
-import { ISummaryTree } from "@fluidframework/driver-definitions";
-import {
+import type { ISummaryTree } from "@fluidframework/driver-definitions";
+import type {
 	FiveDaysMs,
 	IDocumentService,
 	IDocumentServiceFactory,
 	IDocumentStorageServicePolicies,
 	IResolvedUrl,
-	LoaderCachingPolicy,
 } from "@fluidframework/driver-definitions/internal";
+import { LoaderCachingPolicy } from "@fluidframework/driver-definitions/internal";
 import {
 	RateLimiter,
 	getDocAttributesFromProtocolSummary,
 	getQuorumValuesFromProtocolSummary,
 	isCombinedAppAndProtocolSummary,
 } from "@fluidframework/driver-utils/internal";
-import {
-	ISession,
-	convertSummaryTreeToWholeSummaryTree,
-} from "@fluidframework/server-services-client";
+import type { ISession } from "@fluidframework/server-services-client";
+import { convertSummaryTreeToWholeSummaryTree } from "@fluidframework/server-services-client";
 import { PerformanceEvent, createChildLogger } from "@fluidframework/telemetry-utils/internal";
 
-import { ICache, InMemoryCache, NullCache } from "./cache.js";
-import { INormalizedWholeSnapshot } from "./contracts.js";
-import { ISnapshotTreeVersion } from "./definitions.js";
+import type { ICache } from "./cache.js";
+import { InMemoryCache, NullCache } from "./cache.js";
+import type { INormalizedWholeSnapshot } from "./contracts.js";
+import type { ISnapshotTreeVersion } from "./definitions.js";
 import { DocumentService } from "./documentService.js";
 import { pkgVersion as driverVersion } from "./packageVersion.js";
-import { IRouterliciousDriverPolicies } from "./policies.js";
+import type { IRouterliciousDriverPolicies } from "./policies.js";
 import { r11sDriverCompatDetailsForLoader } from "./r11sLayerCompatState.js";
 import {
 	RouterliciousOrdererRestWrapper,
@@ -45,7 +44,7 @@ import {
 } from "./restWrapper.js";
 import { isRouterliciousResolvedUrl } from "./routerliciousResolvedUrl.js";
 import { SessionInfoManager } from "./sessionInfoManager.js";
-import { ITokenProvider } from "./tokens.js";
+import type { ITokenProvider } from "./tokens.js";
 import { replaceDocumentIdInPath } from "./urlUtils.js";
 
 const maximumSnapshotCacheDurationMs: FiveDaysMs = 432_000_000; // 5 days in ms

package/src/documentStorageService.ts

@@ -3,24 +3,24 @@
  * Licensed under the MIT License.
  */
 
-import {
+import type {
 	IDocumentStorageService,
 	IDocumentStorageServicePolicies,
-	LoaderCachingPolicy,
 	ISnapshotTree,
 	IVersion,
 } from "@fluidframework/driver-definitions/internal";
+import { LoaderCachingPolicy } from "@fluidframework/driver-definitions/internal";
 import {
 	DocumentStorageServiceProxy,
 	PrefetchDocumentStorageService,
 } from "@fluidframework/driver-utils/internal";
-import { ITelemetryLoggerExt } from "@fluidframework/telemetry-utils/internal";
+import type { ITelemetryLoggerExt } from "@fluidframework/telemetry-utils/internal";
 
-import { ICache } from "./cache.js";
-import { INormalizedWholeSnapshot } from "./contracts.js";
-import { ISnapshotTreeVersion } from "./definitions.js";
-import { GitManager } from "./gitManager.js";
-import { IRouterliciousDriverPolicies } from "./policies.js";
+import type { ICache } from "./cache.js";
+import type { INormalizedWholeSnapshot } from "./contracts.js";
+import type { ISnapshotTreeVersion } from "./definitions.js";
+import type { GitManager } from "./gitManager.js";
+import type { IRouterliciousDriverPolicies } from "./policies.js";
 import { ShreddedSummaryDocumentStorageService } from "./shreddedSummaryDocumentStorageService.js";
 import { WholeSummaryDocumentStorageService } from "./wholeSummaryDocumentStorageService.js";
 

package/src/errorUtils.ts

@@ -3,11 +3,11 @@
  * Licensed under the MIT License.
  */
 
-import {
+import type {
 	DriverError,
-	DriverErrorTypes,
 	IDriverErrorBase,
 } from "@fluidframework/driver-definitions/internal";
+import { DriverErrorTypes } from "@fluidframework/driver-definitions/internal";
 import {
 	AuthorizationError,
 	GenericNetworkError,
@@ -15,7 +15,8 @@ import {
 	createGenericNetworkError,
 	type DriverErrorTelemetryProps,
 } from "@fluidframework/driver-utils/internal";
-import { IFluidErrorBase, LoggingError } from "@fluidframework/telemetry-utils/internal";
+import type { IFluidErrorBase } from "@fluidframework/telemetry-utils/internal";
+import { LoggingError } from "@fluidframework/telemetry-utils/internal";
 
 import { R11sServiceClusterDrainingErrorCode } from "./contracts.js";
 import { pkgVersion as driverVersion } from "./packageVersion.js";
@@ -113,33 +114,39 @@ export function createR11sNetworkError(
 		case 401:
 		// The first 401 is manually retried in RouterliciousRestWrapper with a refreshed token,
 		// so we treat repeat 401s the same as 403.
-		case 403:
+		case 403: {
 			error = new AuthorizationError(errorMessage, undefined, undefined, props);
 			break;
-		case 404:
+		}
+		case 404: {
 			const errorType = RouterliciousErrorTypes.fileNotFoundOrAccessDeniedError;
 			error = new NonRetryableError(errorMessage, errorType, props);
 			break;
-		case 429:
+		}
+		case 429: {
 			error = createGenericNetworkError(errorMessage, { canRetry: true, retryAfterMs }, props);
 			break;
+		}
 		case 500:
-		case 502:
+		case 502: {
 			error = new GenericNetworkError(errorMessage, true, props);
 			break;
-		case 503:
+		}
+		case 503: {
 			if (internalErrorCode === R11sServiceClusterDrainingErrorCode) {
 				error = new ClusterDrainingError(
 					errorMessage,
-					retryAfterMs !== undefined ? retryAfterMs / 1000 : 660,
+					retryAfterMs === undefined ? 660 : retryAfterMs / 1000,
 					props,
 				);
 				break;
 			}
-		default:
+		}
+		default: {
 			const retryInfo = { canRetry: retryAfterMs !== undefined, retryAfterMs };
 			error = createGenericNetworkError(errorMessage, retryInfo, props);
 			break;
+		}
 	}
 	error.addTelemetryProperties({ endpointReached: true });
 	return error;

package/src/gitManager.ts

@@ -11,14 +11,14 @@ import type {
 	IGitCreateTreeParams,
 	IGitTree,
 } from "@fluidframework/driver-definitions/internal";
-import {
+import type {
 	IWholeSummaryPayload,
 	IWriteSummaryResponse,
 } from "@fluidframework/server-services-client";
 
-import { IWholeFlatSnapshot } from "./contracts.js";
-import { IR11sResponse } from "./restWrapper.js";
-import { IGitManager, IHistorian } from "./storageContracts.js";
+import type { IWholeFlatSnapshot } from "./contracts.js";
+import type { IR11sResponse } from "./restWrapper.js";
+import type { IGitManager, IHistorian } from "./storageContracts.js";
 
 export class GitManager implements IGitManager {
 	constructor(private readonly historian: IHistorian) {}

package/src/historian.ts

@@ -12,16 +12,16 @@ import type {
 	IGitCreateTreeParams,
 	IGitTree,
 } from "@fluidframework/driver-definitions/internal";
-import {
+import type {
 	IWholeSummaryPayload,
 	IWriteSummaryResponse,
 } from "@fluidframework/server-services-client";
 
-import { IWholeFlatSnapshot } from "./contracts.js";
+import type { IWholeFlatSnapshot } from "./contracts.js";
 import type { QueryStringType } from "./queryStringUtils.js";
-import { IR11sResponse } from "./restWrapper.js";
-import { RestWrapper } from "./restWrapperBase.js";
-import { IHistorian } from "./storageContracts.js";
+import type { IR11sResponse } from "./restWrapper.js";
+import type { RestWrapper } from "./restWrapperBase.js";
+import type { IHistorian } from "./storageContracts.js";
 
 export interface ICredentials {
 	user: string;
@@ -104,7 +104,7 @@ export class Historian implements IHistorian {
 		return this.restWrapper.post<IWriteSummaryResponse>(
 			`/git/summaries`,
 			summary,
-			this.getQueryString(initial !== undefined ? { initial } : undefined),
+			this.getQueryString(initial === undefined ? undefined : { initial }),
 		);
 	}
 

package/src/index.ts

@@ -8,7 +8,7 @@ export type { IRouterliciousResolvedUrl } from "./routerliciousResolvedUrl.js";
 
 // Tokens
 export { DefaultTokenProvider } from "./defaultTokenProvider.js";
-export { ITokenProvider, ITokenResponse, ITokenService } from "./tokens.js";
+export type { ITokenProvider, ITokenResponse, ITokenService } from "./tokens.js";
 
 // Errors
 export { RouterliciousErrorTypes } from "./errorUtils.js";
@@ -21,7 +21,7 @@ export {
 } from "./documentServiceFactory.js";
 
 // Configuration
-export { IRouterliciousDriverPolicies } from "./policies.js";
+export type { IRouterliciousDriverPolicies } from "./policies.js";
 
 // Layer Compat details
 export { r11sDriverCompatDetailsForLoader } from "./r11sLayerCompatState.js";

package/src/mapWithExpiration.ts

@@ -18,7 +18,7 @@ export class MapWithExpiration<TKey = any, TValue = any> extends Map<TKey, TValu
 	}
 
 	private refresh(key: TKey): void {
-		this.lastRefreshedTimes.set(key, new Date().valueOf());
+		this.lastRefreshedTimes.set(key, Date.now());
 	}
 
 	/**
@@ -35,7 +35,7 @@ export class MapWithExpiration<TKey = any, TValue = any> extends Map<TKey, TValu
 		if (refreshTime === undefined) {
 			return undefined;
 		}
-		const expired = new Date().valueOf() - refreshTime >= this.expiryMs;
+		const expired = Date.now() - refreshTime >= this.expiryMs;
 		if (expired && cleanUp) {
 			this.delete(key);
 		}

package/src/nullBlobStorageService.ts

@@ -3,8 +3,8 @@
  * Licensed under the MIT License.
  */
 
-import { ISummaryTree, ISummaryHandle } from "@fluidframework/driver-definitions";
-import {
+import type { ISummaryTree, ISummaryHandle } from "@fluidframework/driver-definitions";
+import type {
 	IDocumentStorageService,
 	ISummaryContext,
 	IVersion,

package/src/packageVersion.ts

@@ -6,4 +6,4 @@
  */
 
 export const pkgName = "@fluidframework/routerlicious-driver";
-export const pkgVersion = "2.91.0";
+export const pkgVersion = "2.92.0";

package/src/r11sSnapshotParser.ts

@@ -4,9 +4,9 @@
  */
 
 import { stringToBuffer } from "@fluid-internal/client-utils";
-import { ISnapshotTree } from "@fluidframework/driver-definitions/internal";
+import type { ISnapshotTree } from "@fluidframework/driver-definitions/internal";
 
-import {
+import type {
 	INormalizedWholeSnapshot,
 	IWholeFlatSnapshot,
 	IWholeFlatSnapshotTree,
@@ -71,9 +71,9 @@ export function convertWholeFlatSnapshotToSnapshotTreeAndBlobs(
 ): INormalizedWholeSnapshot {
 	const blobs = new Map<string, ArrayBuffer>();
 	if (flatSnapshot.blobs) {
-		flatSnapshot.blobs.forEach((blob) => {
+		for (const blob of flatSnapshot.blobs) {
 			blobs.set(blob.id, stringToBuffer(blob.content, blob.encoding ?? "utf-8"));
-		});
+		}
 	}
 	const flatSnapshotTree = flatSnapshot.trees?.[0];
 	const sequenceNumber = flatSnapshotTree?.sequenceNumber;

package/src/restWrapper.ts

@@ -4,25 +4,18 @@
  */
 
 import { performanceNow } from "@fluid-internal/client-utils";
-import { ITelemetryBaseProperties } from "@fluidframework/core-interfaces";
+import type { ITelemetryBaseProperties } from "@fluidframework/core-interfaces";
 import { assert } from "@fluidframework/core-utils/internal";
-import {
-	GenericNetworkError,
-	NonRetryableError,
-	RateLimiter,
-} from "@fluidframework/driver-utils/internal";
+import type { RateLimiter } from "@fluidframework/driver-utils/internal";
+import { GenericNetworkError, NonRetryableError } from "@fluidframework/driver-utils/internal";
 import {
 	CorrelationIdHeaderName,
 	DriverVersionHeaderName,
 	RestLessClient,
 	getAuthorizationTokenFromCredentials,
 } from "@fluidframework/server-services-client";
-import {
-	ITelemetryLoggerExt,
-	PerformanceEvent,
-	numberFromString,
-} from "@fluidframework/telemetry-utils/internal";
-import fetch from "cross-fetch";
+import type { ITelemetryLoggerExt } from "@fluidframework/telemetry-utils/internal";
+import { PerformanceEvent, numberFromString } from "@fluidframework/telemetry-utils/internal";
 import safeStringify from "json-stringify-safe";
 
 import {
@@ -34,15 +27,15 @@ import { pkgVersion as driverVersion } from "./packageVersion.js";
 import { addOrUpdateQueryParams, type QueryStringType } from "./queryStringUtils.js";
 import type { RequestConfig, RawRequestHeaders } from "./request.cjs";
 import { RestWrapper } from "./restWrapperBase.js";
-import { ITokenProvider, ITokenResponse } from "./tokens.js";
+import type { ITokenProvider, ITokenResponse } from "./tokens.js";
 
 type AuthorizationHeaderGetter = (token: ITokenResponse) => string;
 export type TokenFetcher = (refresh?: boolean) => Promise<ITokenResponse>;
 
 const buildRequestUrl = (requestConfig: RequestConfig): string =>
-	requestConfig.baseURL !== undefined
-		? `${requestConfig.baseURL ?? ""}${requestConfig.url ?? ""}`
-		: (requestConfig.url ?? "");
+	requestConfig.baseURL === undefined
+		? (requestConfig.url ?? "")
+		: `${requestConfig.baseURL ?? ""}${requestConfig.url ?? ""}`;
 
 const buildRequestInitConfig = (requestConfig: RequestConfig): RequestInit => {
 	const requestInit: RequestInit = {
@@ -103,12 +96,12 @@ export function getPropsToLogFromResponse(headers: {
 	const additionalProps: ITelemetryBaseProperties = {
 		contentsize: numberFromString(headers.get("content-length")),
 	};
-	headersToLog.forEach((header) => {
+	for (const header of headersToLog) {
 		const headerValue = headers.get(header.headerName);
 		if (headerValue !== undefined && headerValue !== null) {
 			additionalProps[header.logName] = headerValue;
 		}
-	});
+	}
 
 	return additionalProps;
 }
@@ -276,11 +269,11 @@ class RouterliciousRestWrapper extends RestWrapper {
 		}
 
 		const responseSummary =
-			responseBody !== undefined
-				? typeof responseBody === "string"
+			responseBody === undefined
+				? response.statusText
+				: typeof responseBody === "string"
 					? responseBody
-					: safeStringify(responseBody)
-				: response.statusText;
+					: safeStringify(responseBody);
 		throwR11sNetworkError(
 			`R11s fetch error: ${responseSummary}`,
 			response.status,

package/src/restWrapperBase.ts

@@ -5,7 +5,7 @@
 
 import type { QueryStringType } from "./queryStringUtils.js";
 import type { RequestConfig, RequestHeaders } from "./request.cjs";
-import { IR11sResponse } from "./restWrapper.js";
+import type { IR11sResponse } from "./restWrapper.js";
 
 export abstract class RestWrapper {
 	constructor(

package/src/retriableGitManager.ts

@@ -11,15 +11,15 @@ import type {
 	IGitTree,
 } from "@fluidframework/driver-definitions/internal";
 import { runWithRetry } from "@fluidframework/driver-utils/internal";
-import {
+import type {
 	IWholeSummaryPayload,
 	IWriteSummaryResponse,
 } from "@fluidframework/server-services-client";
-import { ITelemetryLoggerExt } from "@fluidframework/telemetry-utils/internal";
+import type { ITelemetryLoggerExt } from "@fluidframework/telemetry-utils/internal";
 
-import { IWholeFlatSnapshot } from "./contracts.js";
-import { IR11sResponse } from "./restWrapper.js";
-import { IGitManager } from "./storageContracts.js";
+import type { IWholeFlatSnapshot } from "./contracts.js";
+import type { IR11sResponse } from "./restWrapper.js";
+import type { IGitManager } from "./storageContracts.js";
 
 export class RetriableGitManager implements IGitManager {
 	constructor(

package/src/sessionInfoManager.ts

@@ -5,14 +5,12 @@
 
 import { assert } from "@fluidframework/core-utils/internal";
 import { getW3CData } from "@fluidframework/driver-base/internal";
-import { IResolvedUrl } from "@fluidframework/driver-definitions/internal";
-import { ISession } from "@fluidframework/server-services-client";
-import {
-	PerformanceEvent,
-	ITelemetryLoggerExt,
-} from "@fluidframework/telemetry-utils/internal";
-
-import { RouterliciousOrdererRestWrapper } from "./restWrapper.js";
+import type { IResolvedUrl } from "@fluidframework/driver-definitions/internal";
+import type { ISession } from "@fluidframework/server-services-client";
+import type { ITelemetryLoggerExt } from "@fluidframework/telemetry-utils/internal";
+import { PerformanceEvent } from "@fluidframework/telemetry-utils/internal";
+
+import type { RouterliciousOrdererRestWrapper } from "./restWrapper.js";
 import { getDiscoveredFluidResolvedUrl } from "./urlUtils.js";
 
 /**

package/src/shreddedSummaryDocumentStorageService.ts

@@ -4,8 +4,8 @@
  */
 
 import { Uint8ArrayToString, stringToBuffer } from "@fluid-internal/client-utils";
-import { ISummaryHandle, ISummaryTree } from "@fluidframework/driver-definitions";
-import {
+import type { ISummaryHandle, ISummaryTree } from "@fluidframework/driver-definitions";
+import type {
 	IDocumentStorageService,
 	IDocumentStorageServicePolicies,
 	ISummaryContext,
@@ -14,19 +14,22 @@ import {
 	IVersion,
 } from "@fluidframework/driver-definitions/internal";
 import { buildGitTreeHierarchy } from "@fluidframework/driver-utils/internal";
-import {
+import type {
 	ITelemetryLoggerExt,
 	MonitoringContext,
+} from "@fluidframework/telemetry-utils/internal";
+import {
 	PerformanceEvent,
 	createChildMonitoringContext,
 } from "@fluidframework/telemetry-utils/internal";
 
-import { ICache, InMemoryCache } from "./cache.js";
-import { ISnapshotTreeVersion } from "./definitions.js";
-import { GitManager } from "./gitManager.js";
-import { IRouterliciousDriverPolicies } from "./policies.js";
+import type { ICache } from "./cache.js";
+import { InMemoryCache } from "./cache.js";
+import type { ISnapshotTreeVersion } from "./definitions.js";
+import type { GitManager } from "./gitManager.js";
+import type { IRouterliciousDriverPolicies } from "./policies.js";
 import { RetriableGitManager } from "./retriableGitManager.js";
-import { ISummaryUploadManager } from "./storageContracts.js";
+import type { ISummaryUploadManager } from "./storageContracts.js";
 import { SummaryTreeUploadManager } from "./summaryTreeUploadManager.js";
 
 const isNode = typeof window === "undefined";

package/src/socketModule.ts

@@ -5,5 +5,5 @@
 
 import { io } from "socket.io-client";
 
-// Import is required for side-effects.
+// eslint-disable-next-line unicorn/prefer-export-from -- Import is required for side-effects.
 export const SocketIOClientStatic = io;

package/src/storageContracts.ts

@@ -3,7 +3,7 @@
  * Licensed under the MIT License.
  */
 
-import { ISummaryTree } from "@fluidframework/driver-definitions";
+import type { ISummaryTree } from "@fluidframework/driver-definitions";
 import type {
 	IGitBlob,
 	IGitCommitDetails,
@@ -12,14 +12,14 @@ import type {
 	IGitCreateTreeParams,
 	IGitTree,
 } from "@fluidframework/driver-definitions/internal";
-import {
+import type {
 	IWholeSummaryPayload,
 	IWholeSummaryPayloadType,
 	IWriteSummaryResponse,
 } from "@fluidframework/server-services-client";
 
-import { IWholeFlatSnapshot } from "./contracts.js";
-import { IR11sResponse } from "./restWrapper.js";
+import type { IWholeFlatSnapshot } from "./contracts.js";
+import type { IR11sResponse } from "./restWrapper.js";
 
 /**
  * Interface to a generic Git provider

package/src/summaryTreeUploadManager.ts

@@ -5,13 +5,16 @@
 
 import { IsoBuffer, Uint8ArrayToString, gitHashFile } from "@fluid-internal/client-utils";
 import { assert, unreachableCase } from "@fluidframework/core-utils/internal";
-import { ISummaryTree, SummaryObject, SummaryType } from "@fluidframework/driver-definitions";
-import type { IGitCreateTreeEntry } from "@fluidframework/driver-definitions/internal";
-import { ISnapshotTreeEx } from "@fluidframework/driver-definitions/internal";
+import type { ISummaryTree, SummaryObject } from "@fluidframework/driver-definitions";
+import { SummaryType } from "@fluidframework/driver-definitions";
+import type {
+	IGitCreateTreeEntry,
+	ISnapshotTreeEx,
+} from "@fluidframework/driver-definitions/internal";
 import { getGitMode, getGitType } from "@fluidframework/driver-utils/internal";
-import { IWholeSummaryPayloadType } from "@fluidframework/server-services-client";
+import type { IWholeSummaryPayloadType } from "@fluidframework/server-services-client";
 
-import { IGitManager, ISummaryUploadManager } from "./storageContracts.js";
+import type { IGitManager, ISummaryUploadManager } from "./storageContracts.js";
 
 /**
  * Recursively writes summary tree as individual summary blobs.
@@ -67,7 +70,7 @@ export class SummaryTreeUploadManager implements ISummaryUploadManager {
 			}
 			case SummaryType.Handle: {
 				if (previousFullSnapshot === undefined) {
-					throw Error("Parent summary does not exist to reference by handle.");
+					throw new Error("Parent summary does not exist to reference by handle.");
 				}
 				return this.getIdFromPath(object.handleType, object.handle, previousFullSnapshot);
 			}
@@ -78,8 +81,9 @@ export class SummaryTreeUploadManager implements ISummaryUploadManager {
 				return object.id;
 			}
 
-			default:
+			default: {
 				unreachableCase(object, `Unknown type: ${(object as any).type}`);
+			}
 		}
 	}
 
@@ -142,8 +146,9 @@ export class SummaryTreeUploadManager implements ISummaryUploadManager {
 					);
 					return tryId;
 				}
-				default:
-					throw Error(`Unexpected handle summary object type: "${handleType}".`);
+				default: {
+					throw new Error(`Unexpected handle summary object type: "${handleType}".`);
+				}
 			}
 		}
 		return this.getIdFromPathCore(handleType, path.slice(1), previousSnapshot.trees[key]);

package/src/tokens.ts

@@ -3,7 +3,7 @@
  * Licensed under the MIT License.
  */
 
-import { ITokenClaims } from "@fluidframework/driver-definitions/internal";
+import type { ITokenClaims } from "@fluidframework/driver-definitions/internal";
 
 /**
  * Abstracts the discovery of claims contained within a token.