Skip to content

feat(rules): add OAI-112, PYD-106 execution-limit rules - #47

Merged
ivanpaghubasan merged 1 commit into
mainfrom
feat/openai-pydantic-execution-limit-discovery
Aug 20, 2026
Merged

feat(rules): add OAI-112, PYD-106 execution-limit rules#47
ivanpaghubasan merged 1 commit into
mainfrom
feat/openai-pydantic-execution-limit-discovery

Conversation

@ivanpaghubasan

Copy link
Copy Markdown
Collaborator

Closes the execution-limit coverage gap for OpenAI Agents SDK and Pydantic AI, using the new agent_run_call_max_turns_missing and agent_run_call_usage_limits_missing predicates (merged in the trustabl engine, feat/openai-pydantic-execution-limit-discovery).

  • OAI-112: agent has no explicit max_turns set at the Runner.run/run_sync/run_streamed call site
  • PYD-106: agent has no explicit usage_limits set at the agent.run/run_sync/run_stream call site — Pydantic AI's default UsageLimits() caps request count but leaves token/ cost usage unbounded, the specific risk this rule flags

Both severity low, confidence 0.6, matching the established calibration for this finding class (LC-102, CREW-110, CSDK-204). Default-value claims deliberately hedged where SDK docs and source conflicted, rather than asserting an unverified number.

Mirrored into the engine fixture per the sync obligation. 206 total rules, validated under schema version 14.

Closes the execution-limit coverage gap for OpenAI Agents SDK
and Pydantic AI, using the new agent_run_call_max_turns_missing
and agent_run_call_usage_limits_missing predicates (merged in
the trustabl engine, feat/openai-pydantic-execution-limit-discovery).

- OAI-112: agent has no explicit max_turns set at the
  Runner.run/run_sync/run_streamed call site
- PYD-106: agent has no explicit usage_limits set at the
  agent.run/run_sync/run_stream call site — Pydantic AI's
  default UsageLimits() caps request count but leaves token/
  cost usage unbounded, the specific risk this rule flags

Both severity low, confidence 0.6, matching the established
calibration for this finding class (LC-102, CREW-110, CSDK-204).
Default-value claims deliberately hedged where SDK docs and
source conflicted, rather than asserting an unverified number.

Mirrored into the engine fixture per the sync obligation.
206 total rules, validated under schema version 14.
@ivanpaghubasan
ivanpaghubasan marked this pull request as ready for review August 20, 2026 05:24
@ivanpaghubasan
ivanpaghubasan merged commit 713cf65 into main Aug 20, 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