Skip to content

Support recursive owned JSON graphs - #854

Merged
sanohiro merged 2 commits into
mainfrom
agent/request13-recursive-owned-json-impl
Aug 18, 2026
Merged

Support recursive owned JSON graphs#854
sanohiro merged 2 commits into
mainfrom
agent/request13-recursive-owned-json-impl

Conversation

@sanohiro

@sanohiro sanohiro commented Aug 17, 2026

Copy link
Copy Markdown
Owner

Summary

  • extend direct owned JSON support from flat records to recursive record graphs
  • carry target-bound V2 owned JSON graph descriptors through HIR, MIR, interfaces, LLVM lowering, and the runtime
  • support owned strings and arrays of owned strings inside recursive records
  • add exact cycle, depth, malformed-input, cross-unit, cache, and C6 closure coverage

Review fixes

  • resolve imported qualified and generic record graphs against complete HIR definitions
  • include reachable dependency definition changes in interface identity
  • enforce the 128-record path limit for shared DAGs reached through deeper paths
  • update the unit-cache schema golden and boundary benchmark to format 8 terminology

Verification

  • scripts/pre-pr.sh --reviewer codex --review-log .git/align-review-0e1152925938b987867ff350050e97192dbf47a4.log --findings-fixed -- scripts/cargo.sh test -p align_driver --test m5_owned_json
  • focused interface, sema, per-unit, unit-cache, owned JSON, benchmark-check, and lint-ratchet targets

Independent review found four actionable issues; all four were fixed and closed by their owner regressions before the final-SHA preflight.

@sanohiro
sanohiro marked this pull request as ready for review August 17, 2026 13:52
@sanohiro
sanohiro merged commit 340a330 into main Aug 18, 2026
9 of 10 checks passed
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