Skip to content

[codex] Harden bamltype parity tests and decouple from legacy bridge oracle - #63

Merged
darinkishore merged 1 commit into
mainfrom
codex/facet-parity-frozen-oracles
Feb 8, 2026
Merged

[codex] Harden bamltype parity tests and decouple from legacy bridge oracle#63
darinkishore merged 1 commit into
mainfrom
codex/facet-parity-frozen-oracles

Conversation

@darinkishore

Copy link
Copy Markdown
Collaborator

Summary

This PR hardens bamltype parity testing so it no longer depends on runtime comparisons against the legacy baml-bridge path.

Key changes:

  • Replaced the legacy bridge oracle test with frozen fixture contract coverage in crates/bamltype/tests/contract_frozen_oracle.rs.
  • Converted UI message parity checks to fixture digest assertions in crates/bamltype/tests/contract_bridge_ui_messages.rs.
  • Added fixture-backed parity suites for golden schemas, render options, and property behavior:
    • crates/bamltype/tests/golden_parity.rs
    • crates/bamltype/tests/render_options_parity.rs
    • crates/bamltype/tests/property_parity.rs
  • Removed baml-bridge as a bamltype dev-dependency.
  • Moved test-suite guidance to crate-local docs at crates/bamltype/AGENTS.md and removed stale testing TODO at crates/baml-bridge/TODO.md.

Legacy usage status

  • baml-bridge and baml-bridge-derive are still workspace members.
  • No other crate currently depends on them in Cargo.toml.
  • bamltype parity tests no longer import or call into the legacy crates.

Validation

  • cargo fmt --all --check
  • cargo clippy --workspace --all-targets --all-features -- -D warnings
  • cargo test --workspace

All commands passed on this branch.

@darinkishore
darinkishore force-pushed the codex/facet-parity-frozen-oracles branch from 0b93c57 to 91d415e Compare February 8, 2026 03:52
@darinkishore
darinkishore force-pushed the codex/facet-parity-frozen-oracles branch from 91d415e to 87ba2d6 Compare February 8, 2026 06:45
@darinkishore
darinkishore merged commit ed2dff3 into main Feb 8, 2026
8 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