Complete typed MP mappings and major-2 server validation - #227
Merged
Merged
Conversation
davidalucas
changed the title
davidalucas
marked this pull request as ready for review
September 29, 2026 09:14
davidalucas
force-pushed
the
221-typed-runtime
branch
from
September 29, 2026 09:20
c1d1d29 to
b3823d4
Compare
davidalucas
force-pushed
the
221-small-domain-mappings
branch
2 times, most recently
from
September 29, 2026 09:26
b5135d8 to
389652e
Compare
davidalucas
force-pushed
the
221-small-domain-mappings
branch
from
September 29, 2026 09:44
389652e to
9843c48
Compare
This was referenced Sep 30, 2026
davidalucas
added a commit
that referenced
this pull request
Oct 1, 2026
Restore the Delete Cloud Points by X Y Z Range step text and omit absent range bounds instead of sending zero, matching the reviewed v0.8.0 catalog. Restore SDK argument names for the fit, report bar, chart, view control, XYZ uncertainty, and 2026 vector export operations in both targets, and pin them with regression tests (2026 against the committed inventory, 2024 against cited v0.8.0 catalog literals). Closes #236 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
davidalucas
added a commit
that referenced
this pull request
Oct 1, 2026
Restore the Delete Cloud Points by X Y Z Range step text and omit absent range bounds instead of sending zero, matching the reviewed v0.8.0 catalog. Restore SDK argument names for the fit, report bar, chart, view control, XYZ uncertainty, and 2026 vector export operations in both targets, and pin them with regression tests (2026 against the committed inventory, 2024 against cited v0.8.0 catalog literals). Closes #236 Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The two exact-SA server products now expose their reviewed MP surface through handwritten typed mappings instead of descriptor-interpreted runtime dispatch. This PR also completes the scoped repository cleanup and narrows worker execution, status, and lifecycle dependencies. A compiled-call guard rejects normal-path reflection/dynamic dispatch; gRPC mapping tests share the existing host fixture; admission reserves a bounded retained-work byte budget before mapping.
The current source checkpoint is
aa0c7ae1a2c5529033a2772ef9aad7b37321c58c. Reproducible0.9.0-dev.2server and conformance ZIPs were built from that commit. The publicbriosaprotocol descriptors and compatibility major 2/revision 0 are unchanged from dev.1, so the six unpublished0.4.0client binaries keep their earlier protocol imports. All six passed 12 packaged fake-SDK conformance scenarios against dev.2; twelve retained old-client pairs rejected major 2. Both actual server ZIPs passed packaged fake-SDK, Control Center, and installer receipt/coexistence/repair checks. The related .NET, JavaScript, Python, installer, docs, and examples reviews track coordinated changes.Validation: both Release builds have zero warnings/errors; complete portable suites passed (2026: 784 server, 206 worker, 28 protocol; 2024: 772 server, 223 worker, 28 protocol), along with protocol, interop, target-isolation, CI and Debug reflection checks. Three serial before/after fake-worker performance samples per target show no broad speedup or sustained regression claim. The retained-log sink still scans the shared directory per event to preserve strict cross-process byte/file limits. Batching that scan requires a durable shared reservation/recovery ledger. The maintainer delegated this choice; we retain strict limits and track the larger subsystem as focused follow-up #228. No logging speedup is claimed.
Licensed package validation against both matching exact SA/SDK targets passed external readiness, the maintained read-only smoke, both string-setter RPCs, and public SDK stop. The 2026 run passed after the maintainer registered the exact SDK and started SA normally; prior mismatch and hidden-startup failures remain separate diagnostic history. The earlier 2026 direct SDK and source-built RPC observations remain separate. Fixture-backed reporting/instrument/robot cases remain unexecuted. See
docs/development/r39-licensed-validation.mdand the target evidence for precise scope and hashes.This is a review candidate, not a published release. Publication, tutorial package pin changes, protected licensed CI and old-baseline migration cleanup are release follow-ups. Stacked on the typed runtime branch in draft PR #226. Refs #221.