Skip to content

Replace Remote XML documentation placeholders - #8755

Merged
Aaronontheweb merged 21 commits into
akkadotnet:devfrom
Aaronontheweb:docs/tbd-remote
Oct 5, 2026
Merged

Aaronontheweb merged 21 commits into
akkadotnet:devfrom
Aaronontheweb:docs/tbd-remote

Conversation

@Aaronontheweb

Copy link
Copy Markdown
Member

Remote had 1,130 XML documentation placeholders in 591 blocks across 32 files. Document remote actor resolution and ownership, transport and association lifecycle, serialization, discovery, remote testing, and the small Hyperion support API. Preserve useful prose and ground each description in implementation, callers, and contracts. Runtime source, signatures, attributes, nullable directives, ordinary comments, tests, configuration, dependencies, and API approval files remain unchanged.

Luna authored the documentation. Independent Sol review covered the complete assigned batch and rechecked all corrections. This PR is independent against dev.

Validation:

  • Roslyn source comparison preserves executable tokens and all non-documentation trivia and directives; exact assigned-file scan has zero XML documentation placeholders.
  • Final affected Release builds pass with warnings as errors. Compiler XML member IDs are unchanged, and scoped DocFX metadata and rendered documentation pass with zero warnings.
  • Existing runtime coverage is carried from the source-equivalent Core documentation candidate: all 34 selected project suites have passing results (8,023 passed, 523 existing skips, zero failed tests). The original Incrementalist invocation failed on a framework propagation build issue; Serialization V2 passed all 566 tests with the corrected command. No test or project changes.
  • CI still runs the required affected-project checks. Actual IDE tooltip rendering was not exercised.

The agent will not attempt to merge campaign PRs.

@Aaronontheweb
Aaronontheweb merged commit 29095a1 into akkadotnet:dev Oct 5, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant