Skip to content

[codex] Replace wiptype with bamltype and vendor facet - #60

Merged
darinkishore merged 1 commit into
mainfrom
codex/facet-bamltype-parity
Feb 7, 2026
Merged

[codex] Replace wiptype with bamltype and vendor facet#60
darinkishore merged 1 commit into
mainfrom
codex/facet-bamltype-parity

Conversation

@darinkishore

Copy link
Copy Markdown
Collaborator

Summary

This PR migrates the in-progress typed schema integration from the temporary wiptype path to a first-class bamltype/bamltype-derive implementation, vendors the Facet crates required for reflection, and wires the DSRs runtime + macros to the new typed contracts.

User impact

Typed prompt/schema generation and parsing now run through the new bamltype bridge and Facet-backed conversion path, with parity tests and UI diagnostics in place. This improves correctness and keeps macro/runtime contracts aligned for downstream users.

Root cause

The prior implementation lived in a temporary wiptype track and relied on intermediate compatibility behavior while schema/rendering/parsing semantics were still shifting. That left the integration split across temporary crates and ad-hoc fixes.

What changed

  • Added new crates: crates/bamltype, crates/bamltype-derive.
  • Updated crates/dspy-rs and crates/dsrs-macros to consume the new typed bridge and attributes.
  • Added/updated contract, integration, and UI compile-fail tests around schema generation, parsing, and macro behavior.
  • Vendored required Facet crates under vendor/facet* and integrated them as path dependencies.
  • Updated vendored BAML parsing/rendering behavior for real/rendered name handling.
  • Added workspace ignore hygiene for profiling outputs (*.profraw) and workspace dirs.

Validation

  • cargo test -p dspy-rs --tests
  • cargo test -p bamltype --tests
  • cargo test -p dsrs_macros --tests

Notes

  • Local-only handoff/planning artifacts and generated .profraw files were removed from the PR scope.

@darinkishore
darinkishore force-pushed the codex/facet-bamltype-parity branch 6 times, most recently from 344cb00 to 86fea56 Compare February 7, 2026 21:28
@darinkishore
darinkishore force-pushed the codex/facet-bamltype-parity branch from 86fea56 to e9073fa Compare February 7, 2026 21:30
@darinkishore
darinkishore marked this pull request as ready for review February 7, 2026 21:34
@darinkishore
darinkishore merged commit 1ffa920 into main Feb 7, 2026
8 of 10 checks passed
@darinkishore
darinkishore deleted the codex/facet-bamltype-parity branch February 7, 2026 21:34
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