fix(agents): update deprecated pydantic-ai parameter

[steveant]

Summary

Updates result_type to output_type in DocumentAgent for compatibility with pydantic-ai 1.0.x.

Problem

The Agent class in pydantic-ai renamed the result_type parameter to output_type in version 1.0. This causes the document agent to fail initialization:

ERROR:src.agents.server:Failed to initialize document agent: Unknown keyword arguments: `result_type`

Solution

Single-line fix renaming the parameter.

Summary by CodeRabbit

• Refactor
  • Updated Agent API parameter naming in DocumentAgent for consistency.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

A parameter name in the Agent instantiation within DocumentAgent is updated from result_type to output_type. The change reflects a public API adjustment with no functional or behavioral modifications.

Changes

Cohort / File(s)	Summary
Agent parameter rename python/src/agents/document_agent.py	Agent instantiation parameter updated from result_type=DocumentOperation to output_type=DocumentOperation in _create_agent method

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

• Single-file parameter rename with no logic changes
• No control flow or behavioral impact
• Mechanical update only

Poem

🐰 A parameter skipped, a name rearranged,
From result to output—the Agent's been changed!
No logic was harmed in this tidbit small,
Just nomenclature dancing through agent-y hall. ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description provides a clear summary, explains the problem, and outlines the solution, but does not follow the repository's required PR template structure with sections like Changes Made, Type of Change, Affected Services, Testing, and Checklist.	Restructure the description to follow the repository template, including explicit Type of Change selection, Affected Services, Testing evidence, and completion of the provided checklist items.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly identifies the fix as updating a deprecated pydantic-ai parameter, which matches the core change of renaming result_type to output_type.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ecaece4 and 74e2bb3.

📒 Files selected for processing (1)

• python/src/agents/document_agent.py (1 hunks)

🧰 Additional context used

📓 Path-based instructions (2)

python/src/**/*.py

📄 CodeRabbit inference engine (CLAUDE.md)

python/src/**/*.py: Complete batch processing, background tasks, and external API calls but log detailed errors for each failure instead of crashing
Include context about what was being attempted, preserve full stack traces with exc_info=True in Python logging, use specific exception types, include relevant IDs/URLs, never return None to indicate failure, and report both success count and detailed failure list for batch operations
Use Ruff for linting and MyPy for type checking with 120 character line length in Python backend
Use Python 3.12 with 120 character line length in backend code
Import GITHUB_REPO_OWNER and GITHUB_REPO_NAME constants from python/src/server/config/version.py as the single source of truth for repository configuration

python/src/**/*.py: Python backend code should use 120 character line length
Use Ruff for linting Python code to check for errors, warnings, and unused imports
Use Mypy for type checking in Python to ensure type safety

Files:

• python/src/agents/document_agent.py

python/**/*.py

📄 CodeRabbit inference engine (AGENTS.md)

Error messages must include context about what was being attempted, preserve full stack traces, use specific exception types, include relevant IDs/URLs, and report both success count and detailed failure list for batch operations

Files:

• python/src/agents/document_agent.py

🔇 Additional comments (1)

python/src/agents/document_agent.py (1)

79-79: Use output_type parameter for pydantic-ai 1.0.x compatibility.

The pydantic-ai Agent class constructor uses output_type as the parameter for specifying structured output types, replacing the deprecated result_type parameter. The result_type parameter was removed from the Agent class in v0.6.0, with output_type being the designated replacement. This change is necessary and correct.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Document agent ctor incompatible with pinned pydantic-ai

The document agent now calls Agent(..., output_type=DocumentOperation, ...), but python/uv.lock still pins pydantic-ai at 0.0.55 (python/uv.lock lines 2125-2133). In that version the Agent constructor expects result_type; output_type is not a recognized keyword, so _create_agent will raise TypeError and the document agent cannot start unless the dependency is upgraded to ≥1.0 or the old parameter is kept.

Useful? React with 👍 / 👎.

[steveant]

Superseded by #928 which consolidates this fix along with other improvements.