Skip to content

docs(crew): correct manager_llm field description#5906

Open
adityasingh2400 wants to merge 2 commits into
crewAIInc:mainfrom
adityasingh2400:fix/manager-llm-description
Open

docs(crew): correct manager_llm field description#5906
adityasingh2400 wants to merge 2 commits into
crewAIInc:mainfrom
adityasingh2400:fix/manager-llm-description

Conversation

@adityasingh2400
Copy link
Copy Markdown

@adityasingh2400 adityasingh2400 commented May 22, 2026

The Field description for manager_llm on Crew reads Language model that will run the agent., which is the description copied from the sibling llm field rather than the description for manager_llm. The class docstring already says manager_llm: The language model that will run manager agent., so the schema metadata is out of sync with the docstring.

This PR updates the Field description to Language model that will run the manager agent in hierarchical process. so the schema, the docstring, and the runtime use of the field all agree. It also surfaces in the description that manager_llm is only consulted when process=Process.hierarchical, which is what check_manager_llm enforces in crew.py.

The fix is documentation only and does not change runtime behavior. pytest lib/crewai/tests/test_crew.py -k manager_llm continues to pass against the updated description.

Summary by CodeRabbit

  • Documentation
    • Clarified the description for the manager language model to explicitly indicate its role in running the manager agent within hierarchical processes.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 22, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: bd0b581b-8eca-4ab3-8707-a16385afe28c

📥 Commits

Reviewing files that changed from the base of the PR and between 8b2d24e and 0d5cb4e.

📒 Files selected for processing (1)
  • lib/crewai/src/crewai/crew.py

📝 Walkthrough

Walkthrough

The Crew.manager_llm Pydantic field description was made explicit: it is the language model that will run the manager agent in hierarchical process. No defaults, types, or runtime logic were changed.

Changes

Manager LLM Documentation Update

Layer / File(s) Summary
Manager LLM field description clarification
lib/crewai/src/crewai/crew.py
The manager_llm field description is updated to clarify it is the language model that runs the manager agent in hierarchical process; default remains None.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 I nudged a sentence into place with care,
Now the manager LLM is clear and fair,
Hierarchies hum with proper name,
No logic changed—just clearer aim,
Tiny fix, big readability flair.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: correcting/updating the manager_llm field description in the Crew class to be more specific and accurate.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

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

The Field description for manager_llm on Crew read 'Language model that
will run the agent.', which is the description copied from the llm
field rather than the description for manager_llm. The class docstring
already describes manager_llm as 'The language model that will run
manager agent.' Update the Field description to match: 'Language model
that will run the manager agent in hierarchical process.' This makes
the schema metadata consistent with the docstring and clarifies that
this field is only used when process=Process.hierarchical.
@adityasingh2400 adityasingh2400 force-pushed the fix/manager-llm-description branch from 46183e1 to 8b2d24e Compare May 23, 2026 13:02
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