improvement(eslint-config-fluid): Disable eslint rule that yields false positives

[Josmithr]

Disables unicorn/no-array-callback-reference, which yields false positives for non-array methods with the same name as array methods.

[Copilot]

Pull request overview

This PR disables the unicorn/no-array-callback-reference ESLint rule globally to eliminate false positives that occur when non-array types implement methods with the same names as array methods (e.g., map, forEach, reduce, find).

• Disables unicorn/no-array-callback-reference in the base ESLint configuration
• Removes all local overrides and inline disable comments for this rule throughout the codebase
• Updates CHANGELOG to document the rule change

Reviewed changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
common/build/eslint-config-fluid/recommended.js	Adds global disable for unicorn/no-array-callback-reference with explanatory comment
common/build/eslint-config-fluid/CHANGELOG.md	Documents the rule change in the Unreleased section
common/build/eslint-config-fluid/printed-configs/*.json	Updates all printed config files to reflect the rule being "off"
packages/dds/tree/.eslintrc.cjs	Removes package-specific override that's no longer needed
packages/dds/tree/eslint.config.mts	Removes package-specific override that's no longer needed
packages/framework/tree-agent/src/test/schemaUtils.spec.ts	Removes file-level disable comments for the rule
packages/framework/tree-agent/src/test/prompt.spec.ts	Removes inline disable comment for sf.map() call
packages/drivers/odsp-driver/src/odspDocumentDeltaConnection.ts	Removes inline disable comment for SocketReference.find() call
packages/dds/map/src/map.ts	Removes inline disable comment for forEach() call
packages/dds/map/src/directory.ts	Removes inline disable comment for forEach() call
examples/benchmarks/bubblebench/ot/src/proxy/tree.ts	Removes inline disable comment for reduce() call

[github-actions]

🔗 No broken links found! ✅

Your attention to detail is admirable.

linkcheck output

> [email protected] ci:check-links /home/runner/work/FluidFramework/FluidFramework/docs
> start-server-and-test "npm run serve -- --no-open" 3000 check-links

1: starting server using command "npm run serve -- --no-open"
and when url "[ 'http://127.0.0.1:3000' ]" is responding with HTTP status code 200
running tests using command "npm run check-links"


> [email protected] serve
> docusaurus serve --no-open

[SUCCESS] Serving "build" directory at: http://localhost:3000/

> [email protected] check-links
> linkcheck http://localhost:3000 --skip-file skipped-urls.txt

Crawling...

Stats:
  245432 links
    1786 destination URLs
    2025 URLs ignored
       0 warnings
       0 errors