Skip to content

refactor(vocabulary): remove legacy graph contract aliases - #538

Draft
aaltshuler wants to merge 1 commit into
codex/vocabulary-openapi-guardfrom
codex/vocabulary-breaking-contracts
Draft

refactor(vocabulary): remove legacy graph contract aliases#538
aaltshuler wants to merge 1 commit into
codex/vocabulary-openapi-guardfrom
codex/vocabulary-breaking-contracts

Conversation

@aaltshuler

Copy link
Copy Markdown
Collaborator

Summary

  • cut public Rust, HTTP/OpenAPI, CLI JSON, human output, diagnostics, and docs over to one canonical graph vocabulary
  • remove the retired table/row/manifest aliases instead of dual-emitting them
  • distinguish graph-manifest coordinates from published/native Lance dataset coordinates throughout the contract
  • regenerate the exact four-surface vocabulary inventory with zero legacy or compatibility-alias entries

Compatibility

This is an intentional pre-1.0 breaking change. The v0.10 CLI, server, and Rust SDK must be upgraded together. Retired request fields and CLI flags are rejected or absent; old public Rust names and DTO fields are removed.

Persisted manifest columns, recovery grammar, Lance layout, query-result rows/columns, and exact upstream/internal identifiers are unchanged. Legacy cluster observation records are canonicalized on read so cluster status cannot re-emit a retired public key.

Stack

Draft, stacked on #534 (codex/vocabulary-openapi-guard). Merge only after its base lands.

Verification

  • cargo test --workspace --locked --features omnigraph-engine/failpoints,omnigraph-cluster/failpoints
  • cargo clippy --workspace --all-targets --locked -- -D warnings -W clippy::dbg_macro
  • cargo clippy --workspace --all-targets --locked --features omnigraph-engine/failpoints,omnigraph-cluster/failpoints -- -D warnings -W clippy::dbg_macro
  • exact vocabulary guard: 1,104 current / 1,464 base observations; zero legacy misuse and zero compatibility aliases
  • OpenAPI: 102 tests green, including typed export JSON rejection lanes (400/413/415)
  • cargo fmt --all --check
  • git diff --check
  • bash scripts/check-agents-md.sh
  • actionlint .github/workflows/ci.yml
  • shellcheck scripts/*.sh

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