Daniel Madrid

This commit **optimizes** the **`DuplicateBatchDetector`** within the **`container-runtime`** by refactoring its internal logic to use a bidirectional map (`seqNumByBatchId`) for O(1) duplicate detection and cleanup, replacing a less efficient Set. Concurrently, it **adds comprehensive performance benchmarks** for batch ID tracking, extending the op critical path benchmark to test with and without `Fluid.ContainerRuntime.enableBatchIdTracking`. These new tests, specifically in `duplicateBatchDetector.perf.spec.ts`, provide a baseline for batch ID overhead, confirming that enabling this feature has no significant performance impact on the **Fluid Framework**. This **refactoring and new performance testing** ensures more efficient duplicate batch processing and validates the performance characteristics of batch ID tracking.

This commit **adjusts the execution scope** for the `RefreshSerializerLifeCycle.spec.ts` end-to-end tests to run **only in local server environments**. This **maintenance** change addresses persistent flakiness and timeouts observed when these tests were executed on different server types. By modifying a conditional check within `packages/test/test-end-to-end-tests/src/test/offline/refreshSerializerLifeCycle.spec.ts`, the tests are now explicitly restricted to the 'local' driver type. This ensures continued local validation of the `SnapshotRefresher` functionality while preventing unreliable failures in broader CI/CD pipelines, thereby **improving test reliability**.

update type tests (#26494)

This commit performs a **client release version bump**, updating the internal `packageVersion.ts` constants from `2.83.0` to `2.90.0` across numerous packages. This **maintenance** task ensures that a wide range of **Distributed Data Structures (DDS)**, **drivers**, **runtime components**, and **utility packages** reflect the new release version. Specifically, it affects modules within `experimental/dds`, `packages/dds`, `packages/drivers`, `packages/framework`, `packages/loader`, `packages/runtime`, `packages/test`, `packages/tools`, and `packages/utils`. This change is crucial for aligning the codebase with the upcoming **2.90.0 client release**, signifying a new stable version for these components.

This commit **refactors** various assertion and failure messages across several core **Fluid Framework components** by introducing **numeric short codes**. Specifically, it updates messages within the **DDS Tree's snapshot compatibility checks** and **array node operations**, as well as the **Container Runtime's summarizer component**. This **maintenance refactoring** improves error diagnostics and is part of the ongoing release process. The `assertionShortCodesMap.ts` file is also updated to reflect these new codes, ensuring consistent and easily identifiable error reporting.

This commit **adds release notes** for version `2.83.0` to the project's documentation. It specifically updates the `RELEASE_NOTES/2.83.0.md` file, providing a comprehensive overview of **new features, changes, and APIs** introduced in this release for the `client` component. This **documentation update** is a crucial **maintenance** task, ensuring users and developers have clear, consolidated information about the latest version's enhancements and modifications.

This commit **re-enables** a previously skipped end-to-end test within `packages/test/test-end-to-end-tests/src/test/offline/stashedOps.spec.ts`. The test, which validates **offline stashed operations** forking behavior and **ID compressor logic**, was temporarily disabled due to an issue that is no longer reproducible. This **test re-activation** improves the robustness of the CI/CD pipeline by restoring full coverage for a critical scenario. It ensures continued validation of the `stashedOps` subsystem's stability and its interaction with the ID compressor.

This commit **fixes a regression bug** within the **`SerializedStateManager`** in the **`@fluidframework/container-loader`** package. It reintroduces the necessary logic to **restart the snapshot refresher timer** after the `updateBaseSnapshot` method is called. This addresses scenarios where the base snapshot is updated through means other than the direct snapshot refresher, preventing the timer from becoming stale and ensuring the container's state remains current. The fix is crucial for correct **snapshot refreshing** behavior and resolves a specific failing test related to snapshot updates in ODSP.

This commit **enables previously skipped snapshot refresh lifecycle tests** within the `packages/test/test-end-to-end-tests` suite, significantly **improving test coverage** for this critical feature. It **fixes associated bugs** in the test logic, addressing issues like incorrect container names and ensuring proper wait times for snapshot refreshes based on the specific driver. This **enhances the robustness of the snapshot refresh feature** by validating its behavior with comprehensive end-to-end tests, involving adjustments to test timeouts and data object references in `refreshSerializerLifeCycle.spec.ts`.

This commit **refactors** the snapshot refresh functionality within the **`container-loader`** package by extracting it from `SerializedStateManager` into a new, dedicated **`SnapshotRefresher`** class. This change significantly improves **separation of concerns** and **maintainability** by encapsulating the periodic snapshot refreshing logic. The `SerializedStateManager` now delegates these operations and includes a new handler for refreshed snapshots. Furthermore, this work **restores and enhances test coverage** for snapshot refresh scenarios by introducing comprehensive unit tests for `SnapshotRefresher`, addressing previous flakiness in end-to-end tests.

This commit performs a **refactoring** within the **`container-loader` package**, specifically in `serializedStateManager.ts`. It **removes an unnecessary getter** for the `offlineLoadEnabled` property, which was not being used elsewhere and was immutable. The `offlineLoadEnabled` property is now directly exposed as a public readonly member, eliminating the need for a private backing field and its getter within the `SerializedStateManager`. This change improves code clarity and removes dead code without altering any external behavior or functionality.

This commit **improves test stability** by integrating `waitForContainerConnection()` calls into **end-to-end tests** that load containers from a pending state. Specifically, this **maintenance fix** targets tests within `packages/test/test-end-to-end-tests/src/test/offline/stashedOps.spec.ts`, which simulate offline scenarios and container rehydration. By ensuring the container is fully connected before subsequent operations, it addresses race conditions and prevents premature synchronization attempts. This change aims to **resolve "Timeout on waiting a container to be saved" errors** and significantly reduce flakiness in **Fluid Runtime Services (FRS)** tests.

This commit implements a **bug fix** to enhance the robustness of **ODSP blob uploads**, particularly in offline or intermittently connected scenarios. It modifies the `createOdspNetworkError` function within the `odsp-doclib-utils` to classify the `fluidEpochNotProvided` error as **retryable**. This ensures that blob upload operations, which previously failed when the `fluidEpoch` was unavailable during initial offline loading, can now succeed once the client connects and obtains the necessary epoch value. Additionally, the commit includes **test enhancements** in `stashedOps.spec.ts` to enable and add new end-to-end tests that specifically cover blob upload and attach scenarios before loading, ensuring proper handling of these complex offline operations.

This commit delivers a **bug fix** for the **`SharedArray` DDS** and enhances its **fuzz testing infrastructure**. It **resolves an issue** where operations that had been rollbacked could still be incorrectly toggled, by introducing an `isRollback` flag to `SharedArrayEntry` and updating `toggle`, `toggleMove`, and `rollback` methods to prevent such invalid state transitions. Concurrently, it **enables the toggling of delete operations** within `SharedArray` fuzz tests, significantly improving test coverage for undo/redo scenarios. This ensures the `SharedArray`'s state management is robust against rollback interactions and expands the scope of automated validation.

Update type tests (#25786)

This commit performs a **client version bump** across a wide array of **Fluid Framework packages**, updating them from version `2.70.0` to `2.71.0`. This **maintenance** task affects numerous **Distributed Data Structures (DDSs)**, **drivers**, **loaders**, **runtime components**, and **testing utilities**. The update ensures **version consistency** and **compatibility** across the entire client-side ecosystem, preparing for a new release or consolidating recent changes.

This commit **updates the project documentation** by generating and adding the **release notes and changelogs for version 2.70.0**. Specifically, it creates the `RELEASE_NOTES/2.70.0.md` file, which details **breaking changes, new features, and deprecations** for the `client` component. This **maintenance** task ensures that users and developers have comprehensive information regarding the latest release. It facilitates smoother upgrades and a better understanding of new capabilities and changes introduced in version 2.70.0.

This commit performs a **refactoring** to standardize assertion messages across several core **Fluid Framework** components. It replaces descriptive string literals in assertion messages with compact numeric short codes within the **DDS Tree**, **Container Loader**, and **Container Runtime** modules. Specifically, methods like `encode`, `decode`, `connect`, and `setConnectionState` now use these codes. This **maintenance** task updates the central `assertionShortCodesMap` by adding new codes and removing deprecated ones, aiming to reduce bundle size and streamline internal error reporting for the 2.70 release.

This commit introduces a **new public API** function, `createFrozenDocumentServiceFactory`, within the **`container-loader` package**. This **new capability** provides a dedicated and exposed mechanism for creating a specialized document service factory, particularly for containers loaded with pending local state. Internally, the `FrozenDocumentServiceFactory` was enhanced to accept promise-like document service factories, and the `loadFrozenContainerFromPendingState` utility was refactored to utilize this new function. This change is crucial for the **Pages container load flow**, enabling more consistent enforcement of storage policies for containers with local state.

This commit **enhances testing** for the **frozen container** feature within the `local-server-tests` package. It introduces **new test cases** in `loadFrozenContainerFromPendingState.spec.ts` to validate loading frozen containers with various data structures (DDS and blobs) and to verify proper handling of unsupported operations. This **testing** work improves the robustness and reliability of the **frozen container** functionality by expanding its test coverage. A new `initialize` helper function is also added to streamline these tests.