Skip to content

ref(pydantic-ai): Replace individual hooks with hooks.on.model_request - #7327

Merged
alexander-alderman-webb merged 1 commit into
masterfrom
webb/pydantic/wrap-model-request
Sep 3, 2026
Merged

ref(pydantic-ai): Replace individual hooks with hooks.on.model_request#7327
alexander-alderman-webb merged 1 commit into
masterfrom
webb/pydantic/wrap-model-request

Conversation

@alexander-alderman-webb

Copy link
Copy Markdown
Contributor

Description

Noticed that we can simplify while working on the major.

Use hooks.on.model_request to replace all of hooks.on.before_model_request, hooks.on.model_request_error and hooks.on.after_model_request.

Issues

Reminders

@alexander-alderman-webb
alexander-alderman-webb requested a review from a team as a code owner September 1, 2026 12:49
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

121930 passed | ⏭️ 6762 skipped | Total: 128692 | Pass Rate: 94.75% | Execution Time: 415m 5s

📊 Comparison with Base Branch

Metric Change
Total Tests
Passed Tests 📈 +1
Failed Tests 📉 -1
Skipped Tests

➖ Removed Tests (1)

View removed tests
  • test_continuous_profiler_auto_start_and_stop_sampled[experiment-thread]
    • File: tests.profiler.test_continuous_profiler

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2484 uncovered lines.
✅ Project coverage is 90.29%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    90.27%    90.29%    +0.02%
==========================================
  Files          193       193         —
  Lines        25649     25580       -69
  Branches      9454      9418       -36
==========================================
+ Hits         23153     23096       -57
- Misses        2496      2484       -12
- Partials      1445      1428       -17

Generated by Codecov Action

@alexander-alderman-webb alexander-alderman-webb changed the title ref(pydantic-ai): Use hooks.on.model_request to replace individual hooks ref(pydantic-ai): Replace individual hooks with hooks.on.model_request Sep 3, 2026
@alexander-alderman-webb
alexander-alderman-webb merged commit 4ae8604 into master Sep 3, 2026
269 of 272 checks passed
@alexander-alderman-webb
alexander-alderman-webb deleted the webb/pydantic/wrap-model-request branch September 3, 2026 07:28
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