Skip to content

feat: separate deltin library and CLI packaging - #81

Merged
Teakowa merged 2 commits into
mainfrom
codex/issue-80-package-boundary
Aug 30, 2026
Merged

feat: separate deltin library and CLI packaging#81
Teakowa merged 2 commits into
mainfrom
codex/issue-80-package-boundary

Conversation

@e54-bot

@e54-bot e54-bot commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • split the public deltin-rs library from the deltin-rs-cli executable package
  • add public check, inspect, and compile facades while retaining explicit WIR interop
  • preserve CLI UX and add package, CI, documentation, and regression coverage

Validation

  • Rust 1.85 fmt, clippy, and workspace tests
  • support matrix check
  • library package dry-run and CLI workspace check

Fixes #80

@Teakowa Teakowa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Major — src/api.rs (compile_project/compile_path) and cli/src/main.rs (compile) implement and publish the compiler surface owned by #33, while #33 is still open and has not completed its end-to-end evidence/support-matrix gate. #80 explicitly says to coordinate with #33 and expose compile once that contract lands; this PR instead declares compile stable without any end-to-end-supported matrix evidence. Defer/remove the compile facade/CLI/docs from #80, or land the approved #33 compiler contract/evidence first and then make this PR consume it.

Remove the premature compile facade and CLI command until #33 provides the approved compiler contract and end-to-end evidence.

@Teakowa Teakowa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@Teakowa
Teakowa merged commit 7a99a11 into main Aug 30, 2026
2 checks passed
@Teakowa
Teakowa deleted the codex/issue-80-package-boundary branch August 30, 2026 11:55
@e54-bot e54-bot mentioned this pull request Aug 30, 2026
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.

Separate the DEL/OSTW Rust embedding surface from CLI packaging

2 participants