fix(client-presence): move validated State objects to internal

[jason-ha]

Some validation related type remain exposed as the *Raw types reference them.

[Copilot]

Pull Request Overview

This pull request refactors the creation of presence State objects by moving validated (raw) state implementations to an internal API.

• Refactors factory functions to use StateFactoryInternal instead of StateFactory.
• Updates test imports and usage accordingly.
• Adjusts API definitions and documentation in the public API reports.

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
packages/framework/presence/src/test/schemaValidation.spec.ts	Test updates to import and use StateFactoryInternal instead of the public API.
packages/framework/presence/src/test/latestValueManager.spec.ts	Updated test cases using StateFactoryInternal for latest state creation.
packages/framework/presence/src/test/latestMapValueManager.spec.ts	Updated test cases using StateFactoryInternal for latestMap state creation.
packages/framework/presence/src/stateFactory.ts	Refactored exports to expose internal state factory methods and updated documentation comments.
packages/framework/presence/src/latestValueManager.ts	Adjusted factory function overloads and error handling for validator usage.
packages/framework/presence/src/latestMapValueManager.ts	Updated overload signatures and helper function for argument casting.
packages/framework/presence/src/index.ts	Modified type re-exports to remove deprecated types from the public API.
packages/framework/presence/api-report/*	Synchronized API report documentation with internal state changes.
.changeset/green-poets-run.md	Removed changeset details reflecting breaking changes to the validator parameter.

Comments suppressed due to low confidence (6)

packages/framework/presence/src/latestValueManager.ts:329

• [nitpick] Consider adding a TODO or inline comment noting the planned timeline for validator support to aid future maintenance.

	}

packages/framework/presence/src/test/schemaValidation.spec.ts:14

• [nitpick] Ensure test cases fully cover the new StateFactoryInternal usage, particularly in scenarios where validator behavior might be extended in the future.

import { StateFactoryInternal } from "../stateFactory.js";

packages/framework/presence/src/stateFactory.ts:28

• Update the documentation comment to clearly indicate that StateFactory re-exports internal APIs to prevent accidental exposure of raw validated state behaviors.

export const StateFactory: {

packages/framework/presence/src/latestMapValueManager.ts:680

• [nitpick] Add an inline comment in or above the usableArgsType function to clarify its purpose for casting validator arguments, improving maintainability.

	const validator = usableArgsType(args)?.validator;

packages/framework/presence/src/index.ts:51

• Verify that deprecating the export of LatestMapArguments and LatestArguments does not adversely affect existing consumers, and update migration guides as necessary.

	// LatestMapArguments,

[jason-ha]

These comments didn't actually go anywhere. Moved to remarks in overall variable.

[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:
  225447 links
    1710 destination URLs
    1941 URLs ignored
       0 warnings
       0 errors