(tree) Unify summarize and getAttachSummary methods on summarizables

[agarwal-navin]

Description

This PR makes the following changes:

1. Removed the getAttachSummary method from Summarizable in shared tree and replaces it with the existing summarize method.
   These methods are supposed to be used in two different scenarios - getAttachSummary for the summary that is synchronously generated during container attach and summarize for the summary that is asynchronously generated during regular summarization in case tree wants to support async summarization for scenarios like virtualization.
   However, currently only getAttachSummary is used in both these scenarios since DDSes do not do async summarization. Some of the summarizables implement both of these methods and some only implement one of them. This leads to some confusion and also using the method "getAttachSummary" for regular summarization is also misleading.
   So, this PR keeps the summarize method and makes in syncronous. The async version of this method can be added later when it is needed.
2. Added a fullTree param to the summarizeCore method in SharedObject. This is needed for tree (or any DDS) to support incremental summarization. If this param is true, a full tree must be generated without any incrementality.
3. In SharedTreeCore's summarizeCore method, append the sub-tree path's to the incremental summary context before passing it to the individual summarizables. This will keep the logic in one place without having each summarizable do it and also, the summarizables shouldn't know about its parent's incremental summary details.

AB#40395

[Copilot]

Pull Request Overview

This PR unifies the summarization interface by removing getAttachSummary, making summarize synchronous, and adding a fullTree flag and incremental context propagation.

• Replace all getAttachSummary methods and calls with the new synchronous summarize signature
• Add an optional fullTree parameter to summarizeCore and propagate it through SharedTreeCore, kernel, and public APIs
• Update tests and feature-libraries to use summarize and wire up incremental summary paths

Reviewed Changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
packages/dds/tree/src/test/shared-tree-core/sharedTreeCore.spec.ts	Removed summarizeAsync, updated mock to use summarize, and adjusted event interface
packages/dds/tree/src/test/feature-libraries/chunked-forest/chunkEncodingEndToEnd.spec.ts	Replaced getAttachSummary with summarize in forest end-to-end tests
packages/dds/tree/src/shared-tree-core/sharedTreeCore.ts	Added fullTree argument, and prepend each summarizable’s path in the incremental context
packages/dds/tree/src/shared-tree-core/sharedTreeCore.ts (interface)	Dropped getAttachSummary declaration and updated summarize signature
packages/dds/tree/src/shared-tree-core/editManagerSummarizer.ts	Renamed getAttachSummary to summarize, removed async stub
packages/dds/tree/src/feature-libraries/schema-index/schemaSummarizer.ts	Made summarize sync, defaulted fullTree, and updated incremental path logic
packages/dds/tree/src/feature-libraries/forest-summary/forestSummarizer.ts	Renamed to synchronous summarize and accept incremental context
packages/dds/tree/src/feature-libraries/detachedFieldIndexSummarizer.ts	Renamed to synchronous summarize and accept incremental context
packages/dds/shared-object-base/src/sharedObjectKernel.ts	Added fullTree parameter to kernel’s summarizeCore call
packages/dds/shared-object-base/src/sharedObject.ts	Propagated fullTree through public summarize into summarizeCore
packages/dds/shared-object-base/api-report/shared-object-base.legacy.alpha.api.md	Updated API report to include new fullTree parameter in summarizeCore signature
.changeset/loud-dolls-appear.md	Documented the new fullTree parameter in the release notes

[Josmithr]

Our most recent guidance for changeset titles is to not use inlineCode syntax (to prevent the generated changelogs from appearing too noisy). This still needs to be documented in a more accessible place, but here are the guidelines we're using when reviewing: https://eng.ms/docs/experiences-devices/opg/office-shared/fluid-framework/fluid-framework-internal/fluid-framework/docs/on-call/release/release-prep

Suggested change

	Added an optional boolean parameter `fullTree` to `SharedObject`'s `summarizeCore` method
	Added an optional boolean parameter "fullTree" to SharedObject's summarizeCore method

[Josmithr]

Nit (we do like inlineCode syntax in the changeset body when referring to APIs)

Suggested change

	Currently no known SharedObjects's do incremental summaries; however any that do exist or are made in the future must take this "fullTree" parameter into consideration to function correctly.
	Currently no known `SharedObjects`'s do incremental summaries; however, any that do exist or are made in the future must take this "fullTree" parameter into consideration to function correctly.

[Josmithr]

Nit: can we call out the default value explicitly here? Also, I'm pushing for us to use full sentence casing consistently when possible.

Suggested change

	* @param fullTree - flag indicating whether the attempt should generate a full
	* @param fullTree - A flag indicating whether the attempt should generate a full

[CraigMacomber]

Still some active suggestions that I think would be good, but approving for tree.

[github-actions]

🔗 No broken links found! ✅

Your attention to detail is admirable.

linkcheck output

> fluid-framework-docs-site@0.0.0 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"


> fluid-framework-docs-site@0.0.0 serve
> docusaurus serve --no-open

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

> fluid-framework-docs-site@0.0.0 check-links
> linkcheck http://localhost:3000 --skip-file skipped-urls.txt

Crawling...

Stats:
  224682 links
    1708 destination URLs
    1939 URLs ignored
       0 warnings
       0 errors

[agarwal-navin]

@Josmithr Could you please review this PR again. I addressed your comments.