Skip to content

Use canonical context type authority for provenance resolution - #24

Merged
trissim merged 1 commit into
mainfrom
perf/canonical-context-type-resolution-20261005
Oct 5, 2026
Merged

trissim merged 1 commit into
mainfrom
perf/canonical-context-type-resolution-20261005

Conversation

@trissim

@trissim trissim commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

resolve_with_provenance now queries the canonical type keys already admitted by extract_all_configs, eliminating repeated value scans and owner-type recovery for every field/MRO candidate. Selected field reads, scope/MRO precedence, None inheritance, context constructors, provenance and default factories remain at their existing epochs. There is no new cache or value snapshot; the public manually supplied-map resolver is unchanged.

Patch version 1.3.1 is separate from immutable 1.3.0. Installed dependencies remain unchanged pending review.

Validation: 27 existing controls PASS, including canonical map admission, saved restoration/MRO, constructor/factory effects, callable identity and detached mutable aliases. One authentic saved-consumer pair passed exact typed values/provenance/canonical source/ancestor-scope parity and all source/eight-dependency/four-ABI/input/helper guards: Illumination 85.65→67.62 ms; subsequent 3D 224.73→190.59 ms. Normal full Registry READY and parsing were outside the clocks. These are single saved-owner observations, not whole compiler/runtime gains or the main performance route; 3D is not an independent cold-first-job measurement. Arbitrary corruption/rebinding of admitted map type keys during an operation is outside the established law; mutable field values remain live.

Durable receiving and scope.

Closes #23

@trissim
trissim marked this pull request as ready for review October 5, 2026 17:19
Copilot AI balanced review requested due to automatic review settings October 5, 2026 17:19
@trissim
trissim merged commit 1d36774 into main Oct 5, 2026
0 of 11 checks passed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use admitted canonical type keys during provenance resolution

2 participants