Skip to content

fix(sdk): close conformance gaps in the agent, directory and service - #18

Merged
nkavian merged 1 commit into
offering-protocol:mainfrom
zhoodar:fix/sdk-conformance-audit
Sep 22, 2026
Merged

nkavian merged 1 commit into
offering-protocol:mainfrom
zhoodar:fix/sdk-conformance-audit

Conversation

@zhoodar

@zhoodar zhoodar commented Sep 16, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Complete the Agent, Directory, and Service conformance audit with regression coverage.
  • Pin validated network destinations, constrain supporting-document redirects and response sizes, and isolate shared-cache authentication contexts.
  • Preserve schema reference graphs and redirect bases; correct normalization, version compatibility, refinement comparisons, and depth limits.
  • Correct retrieval defaults, search limits and continuations, conditional responses, collection hierarchy validation, and Directory metadata retention.
  • Update integration documentation and exercise protocol-version vectors without changing odp-specs.

Verification

  • make verify and make conformance on stable Rust and Rust 1.85.0.
  • RUSTUP_TOOLCHAIN=1.85.0 make consumer-smoke.
  • ODP_INTEROP_PORT=41987 make interoperability against the Node example Service.
  • cargo llvm-cov --workspace --all-features --locked --summary-only.
  • git diff --check.

The shared conformance adapter still reports unsupported vector categories as skipped; these are not counted as passing. SDK-specific tests additionally cover network, cache, catalog, and failure paths.

@nkavian
nkavian force-pushed the fix/sdk-conformance-audit branch from 39c3d0a to dafb1b5 Compare September 21, 2026 01:53
@nkavian
nkavian force-pushed the fix/sdk-conformance-audit branch from dafb1b5 to 1311fa6 Compare September 22, 2026 20:21
@nkavian
nkavian merged commit df25ff4 into offering-protocol:main Sep 22, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants