Skip to content

Correct the common-ai toolset list and document the shields extra - #71819

Open
Lee-W wants to merge 1 commit into
apache:mainfrom
astronomer:docs-common-ai-toolsets-count
Open

Correct the common-ai toolset list and document the shields extra#71819
Lee-W wants to merge 1 commit into
apache:mainfrom
astronomer:docs-common-ai-toolsets-count

Conversation

@Lee-W

@Lee-W Lee-W commented Aug 19, 2026

Copy link
Copy Markdown
Member

The toolsets guide told readers that four toolsets were included, but documented seven. The three extra ones are not re-exported from the package root, so anyone importing them the way the first four are imported hits an ImportError with nothing in the guide to explain it. The extras list also never mentioned shields, leaving no way to find out which extra installs the guardrail support the agent guide describes.


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

Generated-by: [Claude] following the guidelines


  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

The toolsets guide told readers that four toolsets were included, but
documented seven. The three extra ones are not re-exported from the
package root, so anyone importing them the way the first four are
imported hits an ImportError with nothing in the guide to explain it.
The extras list also never mentioned shields, leaving no way to find out
which extra installs the guardrail support the agent guide describes.
Comment thread providers/common/ai/docs/toolsets.rst
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants