Skip to content

[TrimmableTypeMap][Core C] JavaPeerScanner logic#10818

Closed
simonrozsival wants to merge 4 commits intodotnet:dev/simonrozsival/ttm-core-b-indexfrom
simonrozsival:dev/simonrozsival/ttm-core-c-scanner-only
Closed

[TrimmableTypeMap][Core C] JavaPeerScanner logic#10818
simonrozsival wants to merge 4 commits intodotnet:dev/simonrozsival/ttm-core-b-indexfrom
simonrozsival:dev/simonrozsival/ttm-core-c-scanner-only

Conversation

@simonrozsival
Copy link
Member

@simonrozsival simonrozsival commented Feb 14, 2026

Core slice C from #10805.

Scope

  • JavaPeerScanner execution logic over indexed metadata
  • Scanner orchestration paths for peer discovery and extraction

Notes

  • Draft slice for focused scanner-logic review.
  • Complements the foundation/index slices.

@simonrozsival simonrozsival changed the title [TrimmableTypeMap][Core C] JavaPeerScanner logic preview (<1k) [TrimmableTypeMap][Core C] JavaPeerScanner logic Feb 14, 2026
@simonrozsival simonrozsival changed the base branch from main to dev/simonrozsival/ttm-core-b-index February 14, 2026 00:19
@simonrozsival simonrozsival added copilot `copilot-cli` or other AIs were used to author this trimmable-type-map Area: CoreCLR Issues that only occur when using CoreCLR. Area: NativeAOT Issues that only occur when using NativeAOT. labels Feb 14, 2026
@simonrozsival simonrozsival force-pushed the dev/simonrozsival/ttm-core-b-index branch from 23e6975 to 9822cf0 Compare February 14, 2026 09:35
@simonrozsival
Copy link
Member Author

Applied all style feedback from downstream reviews: null patterns (is null/is not null), string interpolation, and updated scanner to use new typed TypeAttributeInfo model hierarchy. All changes verified to compile.

@simonrozsival simonrozsival force-pushed the dev/simonrozsival/ttm-core-c-scanner-only branch 2 times, most recently from 9b93b13 to 1d8dc92 Compare February 15, 2026 15:37
@simonrozsival simonrozsival force-pushed the dev/simonrozsival/ttm-core-b-index branch from 9822cf0 to 2f303ed Compare February 15, 2026 15:37
simonrozsival and others added 4 commits February 16, 2026 16:06
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Remove Phase 1b (JniNameProviderAttribute merging/reclassification)
- Use record 'with' expression for immutable ForceUnconditionalIfPresent
- Use .IsNullOrEmpty() extension method
- Use record init syntax for RegisterInfo and ExportInfo construction

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@simonrozsival simonrozsival force-pushed the dev/simonrozsival/ttm-core-c-scanner-only branch from 1d8dc92 to b6895af Compare February 16, 2026 15:08
@simonrozsival simonrozsival force-pushed the dev/simonrozsival/ttm-core-b-index branch from 2f303ed to 3fe1722 Compare February 16, 2026 15:08
@simonrozsival
Copy link
Member Author

Closing: merged into unified PR #10823. The Core A/B/C split was too granular — a single scanner-core PR is more practical for review while still being much smaller than the original #10805.

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

Labels

Area: CoreCLR Issues that only occur when using CoreCLR. Area: NativeAOT Issues that only occur when using NativeAOT. copilot `copilot-cli` or other AIs were used to author this trimmable-type-map

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments