Alex Villarreal

This commit **reverts a previous change** by **re-introducing the `isomorphic-fetch` dependency** into the **`odsp-doclib-utils` package**. This **critical bug fix** addresses a regression that caused "`fetch is not defined`" errors during **e2e tests** within the **Loop integration pipeline**. By restoring the import in `packages/utils/odsp-doclib-utils/src/odspRequest.ts`, it ensures the proper functioning of network requests in environments lacking a global `fetch` implementation. This action **stabilizes the integration pipeline** and prevents test failures, maintaining the reliability of the **Fluid Framework's ODSP document library utilities**.

This commit introduces a **build system enhancement** by leveraging **pnpm catalogs** to centralize dependency version management for the **client release group**. It adds a new `buildTools` catalog to `pnpm-workspace.yaml`, which standardizes versions for critical **Fluid Framework build tools** like `@fluid-tools/build-cli`, `@fluid-tools/version-tools`, `@fluidframework/build-tools`, and `@fluidframework/bundle-size-tools`. This **refactoring** updates all relevant `package.json` files within the release group to reference this catalog instead of explicit version specifiers. The change significantly simplifies and ensures consistent dependency management across the affected build tools, improving maintainability and future updates.

This commit performs a **build optimization** by **dropping unnecessary transitive AWS-related dependencies** originating from `oclif` packages. It extends existing dependency cleanup efforts to **additional release groups**, applying a more robust override approach that avoids stub packages. This **maintenance** task reduces the overall **dependency footprint** and improves the efficiency of the **build system** by removing unneeded packages from the project's dependency graph.

This commit implements a **maintenance fix** to resolve pipeline failures caused by a breaking change in `pnpm` version 10.31, specifically impacting **package publishing steps** in the CI. The `tools/pipelines/templates/include-publish-npm-package-deployment.yml` file is updated to reorder `pnpm` installation, ensuring an explicit, pinned `pnpm` version is used. This **workaround** involves creating a stub `package.json` *before* `pnpm` installation, allowing the specified version to be respected and unblocking CI publishing. The change is a temporary measure to restore functionality while a permanent solution or upstream fix is investigated.

This commit **fixes** a critical issue in the **test infrastructure** by **enabling** the `--fail-zero` option within the **shared Mocha configuration** located at `packages/test/mocha-test-setup/mocharc-common.cjs`. This ensures that **Mocha test runs will now explicitly fail with an exit code of 1 if zero tests are found or executed**, preventing scenarios where misconfigured or broken test files could silently pass in CI. This **maintenance improvement** addresses a specific problem where Node.js ESM module loading quirks could lead to empty test runs being reported as successful. While generally applied to enhance **test reliability**, this behavior is specifically **disabled** for the `packages/framework/aqueduct` and `packages/framework/request-handler` packages, indicating targeted exceptions.

build: Drop unnecessary deps with a better approach (#26673)

This commit provides a **CI/CD fix** for the **`publish-api-model-artifact` Azure DevOps pipeline** by addressing an undefined variable issue. It **defines** the `consistentSourcesDirectory` and `FluidFrameworkDirectory` variables within `tools/pipelines/publish-api-model-artifact.yml`, which were previously referenced but not set. This **resolves a build failure** in the "Check Build Tools Installation" step, ensuring correct path resolution and allowing the pipeline to execute as intended. The change mirrors a similar fix applied to the `deploy-website.yml` pipeline.

This commit **fixes a critical issue** where `pnpm test:mocha` in the **`inventory-app`** was silently failing to execute any tests. The problem stemmed from **Quill**, transitively imported via **`@fluidframework/react/alpha`**, attempting to access the `document` object at module import time before JSDOM was initialized. To resolve this, a new `src/test/mochaHooks.ts` file was introduced, which calls `globalJsdom()` at the module level. This hook is now loaded proactively via Node's `--import` flag in `.mocharc.cjs`, ensuring JSDOM is set up before any test files or their dependencies are evaluated. This **test infrastructure improvement** ensures all **`inventory-app`** Mocha tests run correctly, preventing silent failures and significantly improving the reliability of the test suite.

This commit performs a **maintenance chore** by updating the **version control configuration** to ignore `CLAUDE.local.md`. This specific file was previously not covered by existing `.gitignore` rules, as it resides outside the `.claude/` directory. The change ensures that **local development artifacts** or temporary files related to Claude are no longer tracked by Git. This prevents accidental commits of user-specific or temporary data, thereby maintaining **repository cleanliness** and consistency across development environments.

This commit introduces a **new agent skill** designed to **trigger Azure DevOps (ADO) pipelines** specifically for pull requests. This **new capability** primarily benefits maintainers by allowing them to manually initiate pipeline runs on **Copilot-generated PRs**, streamlining the review and validation process for automated contributions. The skill definition is added in `.claude/skills/trigger-pipelines-for-copilot-pr/SKILL.md`, enhancing the project's automation toolkit and improving workflow for PR validation.

Update devdeps and their peer deps (#26627)

This commit introduces **support for pnpm catalog references** within the **`build-cli`** package, primarily for the **`flub release report` command** and its underlying dependency resolution logic. It adds a new utility, `pnpmCatalog.ts`, to parse `pnpm-workspace.yaml` files and resolve `catalog:` references to their actual version ranges, preventing `TypeError: Invalid comparator` errors. This **new capability** updates `getFluidDependencies` and `getPreReleaseDependencies` in `package.ts` to correctly handle these references, making `getFluidDependencies` asynchronous. The change enables the use of pnpm catalogs to **simplify version ranges** across the repository and includes comprehensive unit and integration tests to validate the resolution logic.

This commit performs a **CI configuration update** by adding `include-build-lint.yml` to the trigger and PR path filters of multiple build pipelines. This **maintenance chore** ensures that changes to the shared linting configuration or related build logic correctly trigger the affected pipelines, including those for `api-markdown-documenter`, `benchmark-tool`, `client`, and various `eslint` and `build-tools` components. The update improves the **reliability and accuracy of the CI system** by ensuring dependent pipelines are re-evaluated when their shared build/linting definitions change, preventing stale builds or missed CI runs.

This commit **improves documentation** within the **CI pipeline templates** by adding explanatory comments to `tools/pipelines/templates/build-npm-client-package.yml`. The new comments guide developers and maintainers on **how to retrieve uploaded `tinylicious.log` files** from pipeline runs. Additionally, they alert users to potential **path truncation** issues that might occur when accessing these logs. This **maintenance** update enhances the clarity and usability of the CI system, aiding in debugging and understanding Tinylicious-related issues.

This **CI configuration update** addresses an issue where `tinylicious.log` files were not consistently uploaded during **client package builds**. The `build-npm-client-package.yml` pipeline template is modified to correctly locate and upload multiple `tinylicious` log files from various paths, specifically for `realsvc` and `stress tests`. This **chore** ensures that all relevant diagnostic logs are collected and uniquely named, significantly improving the **observability and debugging capabilities** of these critical CI runs.

This commit performs a **refactoring** and **maintenance** task by marking several `@legacy` types as `@sealed` within the **`runtime-definitions`** and **`test-runtime-utils`** packages. Specifically, interfaces such as `IFluidDataStoreContext`, `IFluidDataStoreContextDetached`, `IFluidParentContext`, and the `MockFluidDataStoreRuntime` class are now annotated. This change **prevents consumers from implementing or extending these types**, clarifying their intended usage as internal or stable APIs not meant for external modification. The update improves API surface clarity and enforces design constraints, ensuring better long-term stability for these core runtime components.

This commit provides **documentation updates** within the **`container-definitions`** package, specifically for the `IFluidCodeDetails` and `IFluidCodeDetailsConfig` interfaces. It clarifies that string keys prefixed with `FluidFramework.` in `IFluidCodeDetailsConfig` are now **reserved for internal Fluid Framework use**. This **maintenance** change establishes a clear API contract, preventing potential future conflicts by guiding partners away from using these designated keys. The update primarily involves modifying JSDoc comments in `packages/common/container-definitions/src/fluidPackage.ts` to reflect this reservation.

This commit provides a **documentation improvement** by fixing a **grammatical error** and adding a missing word within a comment. Specifically, it enhances the readability of the schema compatibility explanation in the **`simple-tree` API**, located in `packages/dds/tree/src/simple-tree/api/tree.ts`. This **maintenance** task ensures that developers can more easily understand the complex concepts related to schema evolution within the **`tree` package**. The change is purely textual and has no functional impact on the runtime behavior of the system.

This commit introduces a **maintenance chore** to the **CI/CD infrastructure**, specifically modifying the **Performance Benchmarks pipeline**. It updates the `include-copy-dev-dependencies.yml` template to **skip installing** the `@fluidframework/eslint-config-fluid` package during development dependency installation. This change addresses a temporary build issue where the pipeline failed because this package, though part of the client release group, was not publishable. By preventing its installation, the commit ensures the **performance pipeline** can complete successfully, as this dependency is not required for running the benchmarks.

test-drivers: Remove unnecessary axios dep (#25690)