Skip to content

[BUG][MVP] Expose canonical Operation Admission rejection validation - #52

Merged
Noam-Alum merged 2 commits into
mvp/implementationfrom
mvp/fix-canonical-operation-admission-rejection-validation
Nov 1, 2026
Merged

Noam-Alum merged 2 commits into
mvp/implementationfrom
mvp/fix-canonical-operation-admission-rejection-validation

Conversation

@Noam-Alum

Copy link
Copy Markdown
Member

Summary

  • define the canonical Operation Admission reason, gRPC status, and retryability mapping in Protocol
  • expose validation for a structured rejection and its transport status as one contract
  • verify every rejection reason plus mismatched status and retryability failures

Why

Agent PR #33 must validate the complete rejection exchange without duplicating Protocol-owned wire semantics. Its source and installed-RPM checks currently fail because the validator is not yet present on mvp/implementation.

Verification

  • Ruff formatting and lint
  • mypy
  • 837 Protocol tests

@Noam-Alum
Noam-Alum merged commit 64514b3 into mvp/implementation Nov 1, 2026
16 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