feat: add dependency-aware bundle ordering - #48
Conversation
vycdev
left a comment
There was a problem hiding this comment.
The implementation and test coverage look coherent, but the documentation status conflicts with this repository's taxonomy. This PR changes docs/open_design/bundle_ordering_contract.md to Status: implemented and calls it a completed contract, while AGENTS.md reserves docs/open_design/ for unresolved design and docs/completed/ for historical handoff material. Please move this record to docs/completed/, add it to the Completed Records inventory in docs/README.md, and update the references in docs/design.md (and any other backlinks) before merging.
|
In addition to the requested documentation-taxonomy change in my review, this branch now conflicts with the updated |
1fac60f to
89444a6
Compare
|
Rebased this branch onto current
During re-verification I also found and fixed a file-boundary case where concatenation could leak a preceding namespace into later root declarations. The bundler now rejects that unrepresentable order with a structured diagnostic and preserves existing output; two regression tests cover rejected and safe boundary orders.
This was generated by an AI agent (vycdev2). Please verify any changes before merging or applying. |
Summary
docs/completed/, add it to the completed-record inventory, and update all backlinksmainsince this branch was openedVerification
cargo test -q -p jett_driver bundle_project_— passed: 11 bundle testscargo fmt --check— passedcargo build— passedcargo test -q— passed: full Rust workspace suitegit diff --check— passedRisk
Closes #13
This was generated by an AI agent (vycdev2). Please verify any changes before merging or applying.