Skip to content

docs(impl): isolate global HIR type validation - #676

Merged
sanohiro merged 3 commits into
mainfrom
agent/pkg-db-l2b-a2-am-split-plan
Jul 30, 2026
Merged

docs(impl): isolate global HIR type validation#676
sanohiro merged 3 commits into
mainfrom
agent/pkg-db-l2b-a2-am-split-plan

Conversation

@sanohiro

@sanohiro sanohiro commented Jul 30, 2026

Copy link
Copy Markdown
Owner

Summary

  • isolate the independently complete am-g-t global HIR type-domain validation slice
  • define exact concrete-root, template-reachability, reference, width/lane, and inline-cycle rules
  • block all remaining placement, nominal/link, callable namespace, declaration/header, and body work behind a new am-r public-contract ledger and fresh review
  • record why the 1,535-line combined checkpoint was split under the existing size and closure-matrix rules

Review closure

The initial post-open reviews found that the broader plan omitted per-position type admissibility and the complete logical/emitted/generated callable namespace, and that exact compiler-symbol rejection would silently change current source acceptance. The plan was reopened rather than patched: none of those surfaces is authorized for implementation by this PR.

The fresh rescope review found one root-predicate ambiguity. The final contract now makes every non-template tagged/function node a concrete root and excludes only the transitive Param-dependent abstract template graph. That finding is closed and the final review is clean.

Verification

  • git diff --check origin/main...HEAD
  • fresh independent adversarial rescope review: CLEAN
  • final scripts/pre-pr.sh HEAD/base-bound attestation

@sanohiro sanohiro changed the title docs(impl): split global HIR validation boundary docs(impl): isolate global HIR type validation Jul 30, 2026
@sanohiro
sanohiro marked this pull request as ready for review July 30, 2026 17:03
@sanohiro
sanohiro merged commit 541f175 into main Jul 30, 2026
5 of 8 checks passed
@sanohiro
sanohiro deleted the agent/pkg-db-l2b-a2-am-split-plan branch July 30, 2026 17:11
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.

1 participant