Skip to content

refactor(routines): extract declaration compiler crate - #88

Merged
jscott3201 merged 1 commit into
mainfrom
refactor/routine-compiler-crate
Aug 24, 2026
Merged

refactor(routines): extract declaration compiler crate#88
jscott3201 merged 1 commit into
mainfrom
refactor/routine-compiler-crate

Conversation

@jscott3201

Copy link
Copy Markdown
Owner

Summary

  • Move the existing G36 declaration checker and its 10 tests byte-for-byte into an unpublished ocl-routine-compiler package.
  • Keep cxf-verify's public re-export, CLI behavior, diagnostics, and binary path unchanged through a sibling path dependency.
  • Cache and test both standalone Cargo packages in CI without introducing a workspace or engine dependency in the compiler package.

Validation

  • ocl-routine-compiler: 10 tests
  • cxf-verify: 12 tests
  • routine lint suite: 189 tests
  • dataset harness: 32 tests plus tiny FPU replay assertions
  • exact G36 source inventory: 431 release / 693 development files
  • declaration, routine, and full fault-library gates passed

@jscott3201 jscott3201 left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Cycle 1 — exact head 3f30f176972621d8b688b0f99d8ea40a4ea59fe1

PASS: both independent reviewers found no blocker/major findings and no deferred minor observations. The declaration implementation is a byte-for-byte move, the verifier API/CLI call path and binary location are preserved, and the standalone package has no engine/path/build dependency. Local focused and full gates passed. Merge eligibility remains conditioned on the in-progress exact-head verify check.

@jscott3201
jscott3201 merged commit 9519a3a into main Aug 24, 2026
3 checks passed
@jscott3201
jscott3201 deleted the refactor/routine-compiler-crate branch August 24, 2026 18:43
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