Skip to content

docs: remove CrewAI from supported frameworks#1059

Open
jesseturner21 wants to merge 1 commit intomainfrom
docs/remove-crewai-from-frameworks
Open

docs: remove CrewAI from supported frameworks#1059
jesseturner21 wants to merge 1 commit intomainfrom
docs/remove-crewai-from-frameworks

Conversation

@jesseturner21
Copy link
Copy Markdown
Contributor

CrewAI is not implemented as a framework option in the CLI — it does not exist in SDKFrameworkSchema, has no template assets, and is not in the SDK_MODEL_PROVIDER_MATRIX. Remove it from README and docs/frameworks.md to match the actual CLI capabilities.

Constraint: Only frameworks in src/schema/constants.ts SDKFrameworkSchema are valid
Rejected: Mark CrewAI as "BYO only" | no special BYO treatment exists for it
Confidence: high
Scope-risk: narrow

Description

Related Issue

Closes #

Documentation PR

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (please describe):

Testing

How have you tested the change?

  • [] I ran npm run test:unit and npm run test:integ
  • I ran npm run typecheck
  • I ran npm run lint
  • If I modified src/assets/, I ran npm run test:update-snapshots and committed the updated snapshots

Checklist

  • I have read the CONTRIBUTING document
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature, or no new docs are needed
  • My changes generate no new warnings
  • Any dependent changes have been merged and published

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the
terms of your choice.

CrewAI is not implemented as a framework option in the CLI — it does not
exist in SDKFrameworkSchema, has no template assets, and is not in the
SDK_MODEL_PROVIDER_MATRIX. Remove it from README and docs/frameworks.md
to match the actual CLI capabilities.

Constraint: Only frameworks in src/schema/constants.ts SDKFrameworkSchema are valid
Rejected: Mark CrewAI as "BYO only" | no special BYO treatment exists for it
Confidence: high
Scope-risk: narrow
@jesseturner21 jesseturner21 requested a review from a team April 30, 2026 16:03
@github-actions github-actions Bot added size/s PR size: S agentcore-harness-reviewing AgentCore Harness review in progress labels Apr 30, 2026
@agentcore-cli-automation
Copy link
Copy Markdown

Reviewed the PR. The claim is accurate — CrewAI is not in SDKFrameworkSchema (src/schema/constants.ts), not in SDK_MODEL_PROVIDER_MATRIX, and there are no CrewAI template assets. The removed doc sections were promising functionality that doesn't exist in the CLI.

Verified that:

  • All CrewAI references in README.md and docs/frameworks.md are cleanly removed (no lingering references in either file).
  • The only remaining crewai mention in the codebase is in src/cli/operations/dev/otel/transforms.ts as an OTEL span scope hint, which is still valid for BYO users running CrewAI instrumentation — not a documentation claim.
  • The CHANGELOG.md entry about deprecating CrewAI is historical and correctly left untouched.
  • Markdown tables (Framework Comparison and Protocol Compatibility) still render correctly with the column removed.

Looks good to merge. :shipit:

@github-actions github-actions Bot removed the agentcore-harness-reviewing AgentCore Harness review in progress label Apr 30, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 43.01% 7793 / 18117
🔵 Statements 42.41% 8245 / 19439
🔵 Functions 40.4% 1347 / 3334
🔵 Branches 40.65% 5108 / 12564
Generated in workflow #2195 for commit b83ff72 by the Vitest Coverage Report Action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/s PR size: S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants