Skip to content

Consolidate binary relation arity checks#68

Merged
jwalin-shah merged 1 commit into
mainfrom
codex/WP-116-duplicate-logic-consolidation
May 14, 2026
Merged

Consolidate binary relation arity checks#68
jwalin-shah merged 1 commit into
mainfrom
codex/WP-116-duplicate-logic-consolidation

Conversation

@jwalin-shah

Copy link
Copy Markdown
Owner

Summary

  • route query/prove binary-arity enforcement through a named shared execution helper
  • remove the unused duplicate HTTP command wrapper
  • cover HTTP prove arity errors alongside query arity errors
  • preserve Program.query arity error compatibility for existing callers/tests

Validation

  • python3 -m pytest -q
  • git diff --check

Residual risk

  • Low: this keeps outward error text stable, but adapters still intentionally expose binary-only query/prove paths while Program relations remain variadic internally.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@jwalin-shah
jwalin-shah merged commit efb9633 into main May 14, 2026
1 check 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.

2 participants