@fluidframework/container-runtime 2.0.0-dev-rc.2.0.0.246488 → 2.0.0-dev-rc.3.0.0.250606

package/src/dataStore.ts

@@ -3,15 +3,15 @@
  * Licensed under the MIT License.
  */
 
-import { ITelemetryLoggerExt, TelemetryDataTag, UsageError } from "@fluidframework/telemetry-utils";
-import { assert, unreachableCase } from "@fluidframework/core-utils";
 import { AttachState } from "@fluidframework/container-definitions";
 import { FluidObject, IFluidHandle } from "@fluidframework/core-interfaces";
+import { assert, unreachableCase } from "@fluidframework/core-utils";
 import {
 	AliasResult,
 	IDataStore,
 	IFluidDataStoreChannel,
 } from "@fluidframework/runtime-definitions";
+import { ITelemetryLoggerExt, TelemetryDataTag, UsageError } from "@fluidframework/telemetry-utils";
 import { ChannelCollection } from "./channelCollection.js";
 import { ContainerMessageType } from "./messageTypes.js";
 

package/src/dataStoreContext.ts

@@ -3,21 +3,21 @@
  * Licensed under the MIT License.
  */
 
+import { TypedEventEmitter } from "@fluid-internal/client-utils";
+import { AttachState, IAudience, IDeltaManager } from "@fluidframework/container-definitions";
 import {
-	IDisposable,
 	FluidObject,
+	IDisposable,
+	IFluidHandle,
 	IRequest,
 	IResponse,
-	IFluidHandle,
 	ITelemetryBaseProperties,
 } from "@fluidframework/core-interfaces";
-import { IAudience, IDeltaManager, AttachState } from "@fluidframework/container-definitions";
-import { TypedEventEmitter } from "@fluid-internal/client-utils";
+import { IEvent } from "@fluidframework/core-interfaces";
 import { assert, LazyPromise, unreachableCase } from "@fluidframework/core-utils";
 import { IDocumentStorageService } from "@fluidframework/driver-definitions";
 import { BlobTreeEntry, readAndParse } from "@fluidframework/driver-utils";
 import type { IIdCompressor } from "@fluidframework/id-compressor";
-import { IEvent } from "@fluidframework/core-interfaces";
 import {
 	IClientDetails,
 	IDocumentMessage,
@@ -27,16 +27,16 @@ import {
 	ITreeEntry,
 } from "@fluidframework/protocol-definitions";
 import {
-	channelsTreeName,
 	CreateChildSummarizerNodeFn,
 	CreateChildSummarizerNodeParam,
 	FluidDataStoreRegistryEntry,
+	IContainerRuntimeBase,
+	IDataStore,
 	IFluidDataStoreChannel,
 	IFluidDataStoreContext,
-	IFluidParentContext,
-	IContainerRuntimeBase,
 	IFluidDataStoreContextDetached,
 	IFluidDataStoreRegistry,
+	IFluidParentContext,
 	IGarbageCollectionData,
 	IGarbageCollectionDetailsBase,
 	IInboundSignalMessage,
@@ -44,35 +44,35 @@ import {
 	ISummarizeInternalResult,
 	ISummarizeResult,
 	ISummarizerNodeWithGC,
-	SummarizeInternalFn,
-	ITelemetryContext,
 	ISummaryTreeWithStats,
-	IDataStore,
+	ITelemetryContext,
+	SummarizeInternalFn,
+	channelsTreeName,
 	gcDataBlobKey,
 } from "@fluidframework/runtime-definitions";
 import { addBlobToSummary } from "@fluidframework/runtime-utils";
 import {
-	createChildMonitoringContext,
 	DataCorruptionError,
 	DataProcessingError,
-	extractSafePropertiesFromMessage,
-	generateStack,
 	LoggingError,
 	MonitoringContext,
-	tagCodeArtifacts,
 	ThresholdCounter,
+	createChildMonitoringContext,
+	extractSafePropertiesFromMessage,
+	generateStack,
+	tagCodeArtifacts,
 } from "@fluidframework/telemetry-utils";
+import { detectOutboundRoutesViaDDSKey, sendGCUnexpectedUsageEvent } from "./gc/index.js";
 import {
-	dataStoreAttributesBlobName,
-	hasIsolatedChannels,
-	wrapSummaryInChannelsTree,
 	ReadFluidDataStoreAttributes,
 	WriteFluidDataStoreAttributes,
+	dataStoreAttributesBlobName,
 	getAttributesFormatVersion,
 	getFluidDataStoreAttributes,
+	hasIsolatedChannels,
 	summarizerClientType,
+	wrapSummaryInChannelsTree,
 } from "./summary/index.js";
-import { detectOutboundRoutesViaDDSKey, sendGCUnexpectedUsageEvent } from "./gc/index.js";
 
 function createAttributes(
 	pkg: readonly string[],

package/src/dataStoreContexts.ts

@@ -3,9 +3,9 @@
  * Licensed under the MIT License.
  */
 
-import { assert, Deferred, Lazy } from "@fluidframework/core-utils";
 import { IDisposable, ITelemetryBaseLogger } from "@fluidframework/core-interfaces";
-import { createChildLogger, ITelemetryLoggerExt } from "@fluidframework/telemetry-utils";
+import { assert, Deferred, Lazy } from "@fluidframework/core-utils";
+import { ITelemetryLoggerExt, createChildLogger } from "@fluidframework/telemetry-utils";
 import { FluidDataStoreContext, LocalFluidDataStoreContext } from "./dataStoreContext.js";
 
 /** @internal */

package/src/dataStoreRegistry.ts

@@ -3,12 +3,12 @@
  * Licensed under the MIT License.
  */
 
-import { UsageError } from "@fluidframework/telemetry-utils";
 import {
 	FluidDataStoreRegistryEntry,
 	IFluidDataStoreRegistry,
 	NamedFluidDataStoreRegistryEntries,
 } from "@fluidframework/runtime-definitions";
+import { UsageError } from "@fluidframework/telemetry-utils";
 
 /**
  * @internal

package/src/deltaScheduler.ts

@@ -3,10 +3,10 @@
  * Licensed under the MIT License.
  */
 
-import { ITelemetryLoggerExt, formatTick } from "@fluidframework/telemetry-utils";
 import { performance } from "@fluid-internal/client-utils";
 import { IDeltaManager } from "@fluidframework/container-definitions";
 import { IDocumentMessage, ISequencedDocumentMessage } from "@fluidframework/protocol-definitions";
+import { ITelemetryLoggerExt, formatTick } from "@fluidframework/telemetry-utils";
 
 /**
  * DeltaScheduler is responsible for the scheduling of inbound delta queue in cases where there

package/src/gc/garbageCollection.ts

@@ -3,23 +3,23 @@
  * Licensed under the MIT License.
  */
 
-import { assert, LazyPromise, Timer } from "@fluidframework/core-utils";
 import { IRequest } from "@fluidframework/core-interfaces";
+import { assert, LazyPromise, Timer } from "@fluidframework/core-utils";
 import {
-	gcTreeKey,
 	IGarbageCollectionData,
 	IGarbageCollectionDetailsBase,
 	ISummarizeResult,
 	ITelemetryContext,
+	gcTreeKey,
 } from "@fluidframework/runtime-definitions";
 import { createResponseError, responseToException } from "@fluidframework/runtime-utils";
 import {
-	createChildLogger,
-	createChildMonitoringContext,
 	DataProcessingError,
 	ITelemetryLoggerExt,
 	MonitoringContext,
 	PerformanceEvent,
+	createChildLogger,
+	createChildMonitoringContext,
 	tagCodeArtifacts,
 } from "@fluidframework/telemetry-utils";
 import { BlobManager } from "../blobManager.js";
@@ -34,18 +34,18 @@ import { IRefreshSummaryResult } from "../summary/index.js";
 import { generateGCConfigs } from "./gcConfigs.js";
 import {
 	GCNodeType,
-	IGarbageCollector,
-	IGarbageCollectorCreateParams,
-	IGarbageCollectionRuntime,
+	GarbageCollectionMessage,
+	GarbageCollectionMessageType,
+	IGCMetadata,
 	IGCResult,
 	IGCStats,
-	UnreferencedState,
-	IGCMetadata,
+	IGarbageCollectionRuntime,
+	IGarbageCollector,
 	IGarbageCollectorConfigs,
+	IGarbageCollectorCreateParams,
 	IMarkPhaseStats,
 	ISweepPhaseStats,
-	GarbageCollectionMessage,
-	GarbageCollectionMessageType,
+	UnreferencedState,
 	disableAutoRecoveryKey,
 } from "./gcDefinitions.js";
 import {
@@ -57,11 +57,11 @@ import {
 import { runGarbageCollection } from "./gcReferenceGraphAlgorithm.js";
 import { IGarbageCollectionSnapshotData, IGarbageCollectionState } from "./gcSummaryDefinitions.js";
 import { GCSummaryStateTracker } from "./gcSummaryStateTracker.js";
+import { GCTelemetryTracker } from "./gcTelemetry.js";
 import {
 	UnreferencedStateTracker,
 	UnreferencedStateTrackerMap,
 } from "./gcUnreferencedStateTracker.js";
-import { GCTelemetryTracker } from "./gcTelemetry.js";
 
 /**
  * The garbage collector for the container runtime. It consolidates the garbage collection functionality and maintains

package/src/gc/gcConfigs.ts

@@ -10,17 +10,23 @@ import {
 } from "@fluidframework/telemetry-utils";
 import { IContainerRuntimeMetadata } from "../summary/index.js";
 import {
-	nextGCVersion,
+	GCFeatureMatrix,
+	GCVersion,
+	IGCMetadata_Deprecated,
+	IGCRuntimeOptions,
+	IGarbageCollectorConfigs,
 	defaultInactiveTimeoutMs,
 	defaultSessionExpiryDurationMs,
+	defaultSweepGracePeriodMs,
+	disableDatastoreSweepKey,
 	disableTombstoneKey,
-	GCFeatureMatrix,
+	gcDisableDataStoreSweepOptionName,
+	gcDisableThrowOnTombstoneLoadOptionName,
+	gcGenerationOptionName,
 	gcTestModeKey,
-	GCVersion,
 	gcVersionUpgradeToV4Key,
-	IGarbageCollectorConfigs,
-	IGCRuntimeOptions,
 	maxSnapshotCacheExpiryMs,
+	nextGCVersion,
 	oneDayMs,
 	runGCKey,
 	runSessionExpiryKey,
@@ -28,12 +34,6 @@ import {
 	stableGCVersion,
 	throwOnTombstoneLoadOverrideKey,
 	throwOnTombstoneUsageKey,
-	gcDisableThrowOnTombstoneLoadOptionName,
-	defaultSweepGracePeriodMs,
-	gcGenerationOptionName,
-	IGCMetadata_Deprecated,
-	disableDatastoreSweepKey,
-	gcDisableDataStoreSweepOptionName,
 } from "./gcDefinitions.js";
 import { getGCVersion, shouldAllowGcSweep } from "./gcHelpers.js";
 

package/src/gc/gcDefinitions.ts

@@ -14,13 +14,13 @@ import {
 } from "@fluidframework/runtime-definitions";
 import { ReadAndParseBlob } from "@fluidframework/runtime-utils";
 import { ITelemetryLoggerExt } from "@fluidframework/telemetry-utils";
+import { RuntimeHeaderData } from "../containerRuntime.js";
+import { ContainerRuntimeGCMessage } from "../messageTypes.js";
 import {
 	IContainerRuntimeMetadata,
 	ICreateContainerMetadata,
 	IRefreshSummaryResult,
 } from "../summary/index.js";
-import { RuntimeHeaderData } from "../containerRuntime.js";
-import { ContainerRuntimeGCMessage } from "../messageTypes.js";
 
 /**
  * @alpha

package/src/gc/gcHelpers.ts

@@ -6,11 +6,11 @@
 import { assert } from "@fluidframework/core-utils";
 import { ISnapshotTree } from "@fluidframework/protocol-definitions";
 import {
+	IGarbageCollectionData,
+	IGarbageCollectionDetailsBase,
 	gcBlobPrefix,
 	gcDeletedBlobKey,
 	gcTombstoneBlobKey,
-	IGarbageCollectionData,
-	IGarbageCollectionDetailsBase,
 } from "@fluidframework/runtime-definitions";
 import { GCFeatureMatrix, GCVersion, IGCMetadata } from "./gcDefinitions.js";
 import {

package/src/gc/gcSummaryStateTracker.ts

@@ -5,16 +5,16 @@
 
 import { SummaryType } from "@fluidframework/protocol-definitions";
 import {
+	ISummarizeResult,
+	ISummaryTreeWithStats,
 	gcBlobPrefix,
 	gcDeletedBlobKey,
 	gcTombstoneBlobKey,
 	gcTreeKey,
-	ISummarizeResult,
-	ISummaryTreeWithStats,
 } from "@fluidframework/runtime-definitions";
-import { mergeStats, SummaryTreeBuilder } from "@fluidframework/runtime-utils";
+import { SummaryTreeBuilder, mergeStats } from "@fluidframework/runtime-utils";
 import { IRefreshSummaryResult } from "../summary/index.js";
-import { GCVersion, IGarbageCollectorConfigs, IGCStats } from "./gcDefinitions.js";
+import { GCVersion, IGCStats, IGarbageCollectorConfigs } from "./gcDefinitions.js";
 import { generateSortedGCState } from "./gcHelpers.js";
 import { IGarbageCollectionSnapshotData, IGarbageCollectionState } from "./gcSummaryDefinitions.js";
 

package/src/gc/gcTelemetry.ts

@@ -5,23 +5,23 @@
 
 import { IGarbageCollectionData } from "@fluidframework/runtime-definitions";
 import {
-	generateStack,
+	type ITelemetryGenericEventExt,
 	ITelemetryLoggerExt,
 	MonitoringContext,
+	generateStack,
 	tagCodeArtifacts,
-	type ITelemetryGenericEventExt,
 } from "@fluidframework/telemetry-utils";
 import { RuntimeHeaderData } from "../containerRuntime.js";
 import { ICreateContainerMetadata } from "../summary/index.js";
 import {
+	GCFeatureMatrix,
 	GCNodeType,
-	UnreferencedState,
 	IGarbageCollectorConfigs,
+	UnreferencedState,
 	disableTombstoneKey,
-	throwOnTombstoneUsageKey,
-	throwOnTombstoneLoadOverrideKey,
 	runSweepKey,
-	GCFeatureMatrix,
+	throwOnTombstoneLoadOverrideKey,
+	throwOnTombstoneUsageKey,
 } from "./gcDefinitions.js";
 import { UnreferencedStateTracker } from "./gcUnreferencedStateTracker.js";
 

package/src/index.ts

@@ -11,7 +11,6 @@ export {
 	ISummaryConfigurationDisableHeuristics,
 	IContainerRuntimeOptions,
 	isRuntimeMessage,
-	RuntimeMessage,
 	agentSchedulerId,
 	ContainerRuntime,
 	TombstoneResponseHeaderKey,
@@ -21,6 +20,7 @@ export {
 	ICompressionRuntimeOptions,
 	CompressionAlgorithms,
 	RuntimeHeaderData,
+	disabledCompressionConfig,
 } from "./containerRuntime.js";
 export {
 	ContainerMessageType,
@@ -98,6 +98,13 @@ export {
 	IRetriableFailureResult,
 	ISummarizeEventProps,
 	IdCompressorMode,
+	IDocumentSchema,
+	DocumentSchemaValueType,
+	IDocumentSchemaCurrent,
+	currentDocumentVersionSchema,
+	DocumentsSchemaController,
+	IDocumentSchemaChangeMessage,
+	IDocumentSchemaFeatures,
 } from "./summary/index.js";
 export { IChunkedOp, unpackRuntimeMessage } from "./opLifecycle/index.js";
 export { ChannelCollection } from "./channelCollection.js";

package/src/messageTypes.ts

@@ -3,16 +3,17 @@
  * Licensed under the MIT License.
  */
 
+import type { IdCreationRange } from "@fluidframework/id-compressor";
 import { ISequencedDocumentMessage } from "@fluidframework/protocol-definitions";
 import {
+	IAttachMessage,
 	IEnvelope,
 	InboundAttachMessage,
-	IAttachMessage,
 } from "@fluidframework/runtime-definitions";
-import type { IdCreationRange } from "@fluidframework/id-compressor";
 import { IDataStoreAliasMessage } from "./dataStore.js";
 import { GarbageCollectionMessage } from "./gc/index.js";
 import { IChunkedOp } from "./opLifecycle/index.js";
+import { IDocumentSchemaChangeMessage } from "./summary/index.js";
 
 /**
  * @alpha
@@ -43,6 +44,11 @@ export enum ContainerMessageType {
 	 */
 	IdAllocation = "idAllocation",
 
+	/**
+	 * An op that changes document schema
+	 */
+	DocumentSchemaChange = "schema",
+
 	/**
 	 * Garbage collection specific op. This is sent by the summarizer client when GC runs. It's used to synchronize GC
 	 * state across all clients.
@@ -131,6 +137,10 @@ export type ContainerRuntimeGCMessage = TypedContainerRuntimeMessage<
 	ContainerMessageType.GC,
 	GarbageCollectionMessage
 >;
+export type ContainerRuntimeDocumentSchemaMessage = TypedContainerRuntimeMessage<
+	ContainerMessageType.DocumentSchemaChange,
+	IDocumentSchemaChangeMessage
+>;
 
 /**
  * Represents an unrecognized TypedContainerRuntimeMessage, e.g. a message from a future version of the container runtime.
@@ -161,7 +171,8 @@ export type InboundContainerRuntimeMessage =
 	| ContainerRuntimeIdAllocationMessage
 	| ContainerRuntimeGCMessage
 	// Inbound messages may include unknown types from other clients, so we include that as a special case here
-	| UnknownContainerRuntimeMessage;
+	| UnknownContainerRuntimeMessage
+	| ContainerRuntimeDocumentSchemaMessage;
 
 /** A {@link TypedContainerRuntimeMessage} that has been generated by the container runtime but is not yet being sent to the server. */
 export type LocalContainerRuntimeMessage =
@@ -174,7 +185,8 @@ export type LocalContainerRuntimeMessage =
 	| ContainerRuntimeIdAllocationMessage
 	| ContainerRuntimeGCMessage
 	// In rare cases (e.g. related to stashed ops) we could have a local message of an unknown type
-	| UnknownContainerRuntimeMessage;
+	| UnknownContainerRuntimeMessage
+	| ContainerRuntimeDocumentSchemaMessage;
 
 /** A {@link TypedContainerRuntimeMessage} that is being sent to the server from the container runtime. */
 export type OutboundContainerRuntimeMessage =
@@ -185,7 +197,8 @@ export type OutboundContainerRuntimeMessage =
 	| ContainerRuntimeRejoinMessage
 	| ContainerRuntimeAliasMessage
 	| ContainerRuntimeIdAllocationMessage
-	| ContainerRuntimeGCMessage;
+	| ContainerRuntimeGCMessage
+	| ContainerRuntimeDocumentSchemaMessage;
 
 /**
  * An unpacked ISequencedDocumentMessage with the inner TypedContainerRuntimeMessage type/contents/etc

package/src/opLifecycle/README.md

@@ -13,6 +13,9 @@
 -   [Note about performance and latency](#note-about-performance-and-latency)
 -   [How it works](#how-it-works)
 -   [How grouped batching works](#how-grouped-batching-works)
+-   [How the overall op flow works](#How-the-overall-op-flow-works)
+    -   [Outbound](#outbound)
+    -   [Inbound](#inbound)
 
 ## Introduction
 
@@ -317,3 +320,89 @@ Ungrouped batch:
 | ClientSeqNum: 1 | ClientSeqNum: 2 | ClientSeqNum: 3 | ClientSeqNum: 4 | ClientSeqNum: 5 |
 +-----------------+-----------------+-----------------+-----------------+-----------------+
 ```
+
+## How the overall op flow works
+
+### Outbound
+
+The outbound view is how ops are accumulated and sent by the runtime with `FlushMode.TurnBased` (default).
+
+```mermaid
+stateDiagram-v2
+	state "* End of JS turn *" as jsTurn
+    state "opGroupingManager.groupBatch" as groupBatch
+    state "opCompressor.compress" as compress
+    state "outbox.flush" as flush
+    state "outbox.flushInternal" as flushInternal
+	state "Send batch over the wire" as post
+	state "Send chunks (partial ops) over the wire" as postChunks
+	state "Store original (uncompressed, unchunked, ungrouped) batch locally" as store
+    state if_compression <<choice>>
+	[*] --> ContainerRuntime.submit
+	ContainerRuntime.submit --> outbox.submitAttach
+	ContainerRuntime.submit --> outbox.submitBlobAttach
+	ContainerRuntime.submit --> outbox.submit
+	outbox.submit --> scheduleFlush
+	outbox.submitAttach --> scheduleFlush
+	outbox.submitBlobAttach --> scheduleFlush
+	scheduleFlush --> jsTurn
+	jsTurn --> flush
+	flush --> outbox.flushInternalMain
+	flush --> outbox.flushInternalAttach
+	flush --> outbox.flushInternalBlobAttach
+	outbox.flushInternalMain --> flushInternal
+	outbox.flushInternalAttach --> flushInternal
+	outbox.flushInternalBlobAttach --> flushInternal
+	flushInternal --> ContainerRuntime.reSubmit: if batch has reentrant ops and should group
+	ContainerRuntime.reSubmit --> flushInternal
+    flushInternal --> groupBatch: if should group
+    groupBatch --> if_compression
+	flushInternal --> if_compression
+    if_compression --> post
+	if_compression --> compress: if compression is enabled
+	compress --> post
+	compress --> opSplitter.split: if the compressed payload is larger than the chunk size
+	opSplitter.split --> post
+	opSplitter.split --> postChunks
+    post --> store
+```
+
+With `FlushMode.Immediate`(deprecated) the difference is that ops are no longer accumulated in batches, but flushed as they are submitted, instead of waiting for the end of the JS turn. All the other components work in the exact same manner with the difference that they operate on batches with length 1.
+
+### Inbound
+
+There is no concept of batch in the inbound view when we receive the ops. Ops are being received and processed one-by-one and the batch is reconstructed in the runtime layer. This requires individual components to maintain their own internal state in order to keep track of the batch.
+
+```mermaid
+stateDiagram-v2
+    [*] --> ContainerRuntime.process
+	ContainerRuntime.process --> remoteMessageProcessor
+    state remoteMessageProcessor {
+        state "process chunk" as processChunk
+        state "return nothing" as returnNothing
+        state "decompress and store" as decompress
+        state if_chunk <<choice>>
+        state if_compressed <<choice>>
+        state if_unrolling <<choice>>
+        state if_grouped <<choice>>
+        [*] --> if_chunk
+        if_chunk --> if_compressed
+        if_chunk --> processChunk: is chunk
+        processChunk --> returnNothing
+        processChunk --> if_compressed: is final chunk
+        if_compressed --> if_unrolling
+        if_compressed --> decompress: is compressed
+        decompress --> if_unrolling
+        if_unrolling --> if_grouped
+        if_unrolling --> unroll: if currently unrolling
+        unroll --> if_grouped
+        if_grouped --> return
+        if_grouped --> ungroup: is grouped batch
+        ungroup --> return
+        return --> [*]
+        returnNothing --> [*]
+    }
+    remoteMessageProcessor --> ContainerRuntime.procesCore
+```
+
+Note that a "system op" originating outside the ContainerRuntime will pass through this flow entirely.

package/src/opLifecycle/definitions.ts

@@ -4,16 +4,13 @@
  */
 
 import { IBatchMessage } from "@fluidframework/container-definitions";
-import { ISequencedDocumentMessage, MessageType } from "@fluidframework/protocol-definitions";
 import { CompressionAlgorithms } from "../containerRuntime.js";
-import { ContainerMessageType } from "../messageTypes.js";
 
 /**
  * Batch message type used internally by the runtime
  */
 export type BatchMessage = IBatchMessage & {
-	localOpMetadata: unknown;
-	type: ContainerMessageType;
+	localOpMetadata?: unknown;
 	referenceSequenceNumber: number;
 	compression?: CompressionAlgorithms;
 };
@@ -59,7 +56,6 @@ export interface IChunkedOp {
 	chunkId: number;
 	totalChunks: number;
 	contents: string;
-	originalType: MessageType | ContainerMessageType;
 	originalMetadata?: Record<string, unknown>;
 	originalCompression?: string;
 }
@@ -72,18 +68,3 @@ export interface IChunkedOp {
  * will make the processor return `Processed`.
  */
 export type ProcessingState = "Processed" | "Skipped" | "Accepted";
-
-/**
- * Return type for functions which process remote messages
- */
-export interface IMessageProcessingResult {
-	/**
-	 * A shallow copy of the input message if processing happened, or
-	 * the original message otherwise
-	 */
-	readonly message: ISequencedDocumentMessage;
-	/**
-	 * Processing result of the input message.
-	 */
-	readonly state: ProcessingState;
-}

package/src/opLifecycle/index.ts

@@ -4,16 +4,10 @@
  */
 
 export { BatchManager, estimateSocketSize, BatchSequenceNumbers } from "./batchManager.js";
-export {
-	BatchMessage,
-	IBatch,
-	IBatchCheckpoint,
-	IChunkedOp,
-	IMessageProcessingResult,
-} from "./definitions.js";
+export { BatchMessage, IBatch, IBatchCheckpoint, IChunkedOp } from "./definitions.js";
 export { Outbox, getLongStack } from "./outbox.js";
 export { OpCompressor } from "./opCompressor.js";
 export { OpDecompressor } from "./opDecompressor.js";
-export { OpSplitter, splitOp } from "./opSplitter.js";
+export { OpSplitter, splitOp, isChunkedMessage } from "./opSplitter.js";
 export { RemoteMessageProcessor, unpackRuntimeMessage } from "./remoteMessageProcessor.js";
-export { OpGroupingManager } from "./opGroupingManager.js";
+export { OpGroupingManager, OpGroupingManagerConfig, isGroupedBatch } from "./opGroupingManager.js";

package/src/opLifecycle/opCompressor.ts

@@ -3,14 +3,14 @@
  * Licensed under the MIT License.
  */
 
-import { createChildLogger, UsageError } from "@fluidframework/telemetry-utils";
-import { assert } from "@fluidframework/core-utils";
 import { IsoBuffer } from "@fluid-internal/client-utils";
-import { compress } from "lz4js";
 import { ITelemetryBaseLogger } from "@fluidframework/core-interfaces";
+import { assert } from "@fluidframework/core-utils";
+import { UsageError, createChildLogger } from "@fluidframework/telemetry-utils";
+import { compress } from "lz4js";
 import { CompressionAlgorithms } from "../containerRuntime.js";
 import { estimateSocketSize } from "./batchManager.js";
-import { IBatch, BatchMessage } from "./definitions.js";
+import { BatchMessage, IBatch } from "./definitions.js";
 
 /**
  * Compresses batches of ops. It generates a single compressed op that contains
@@ -47,7 +47,6 @@ export class OpCompressor {
 		// Add empty placeholder messages to reserve the sequence numbers
 		for (const message of batch.content.slice(1)) {
 			messages.push({
-				type: message.type,
 				localOpMetadata: message.localOpMetadata,
 				metadata: message.metadata,
 				referenceSequenceNumber: message.referenceSequenceNumber,