Skip to content

Add 'classify-provider-changes' breeze command for deterministic change classification#68215

Open
potiuk wants to merge 1 commit into
apache:mainfrom
potiuk:classify-provider-changes-command
Open

Add 'classify-provider-changes' breeze command for deterministic change classification#68215
potiuk wants to merge 1 commit into
apache:mainfrom
potiuk:classify-provider-changes-command

Conversation

@potiuk
Copy link
Copy Markdown
Member

@potiuk potiuk commented Jun 8, 2026

Adds breeze release-management classify-provider-changes — a deterministic, rules-first classifier for provider changelog changes, replacing the random --non-interactive run the prepare-providers-documentation skill abused purely for discovery.

What it does

For each provider with unreleased changes, it classifies every commit with hard-coded, high-confidence rules and outputs JSON (discovery + classification + a needs_llm flag), so an agent/skill takes the deterministic results as-is and only spends LLM calls on the genuinely ambiguous commits.

Rule
only docs/**/*.rst changed documentation
only tests/** / example_dags/** changed skip
subject starts Bump (dependency bump) misc
everything else needs_llm

Deliberately conservative: Fix … / Add … subjects are not auto-classified (an "Add …" can be a breaking change), so they come back as needs_llm. Rule logic lives in classify_change_deterministically (covered by unit tests).

Skill update

.agents/skills/prepare-providers-documentation/SKILL.md Phase 1 now calls this command (no more random answers to throw away), and Phase 3 only spawns sub-agents for needs_llm commits.

Includes the new command image and unit tests for the rule function.


Was generative AI tooling used to co-author this PR?
  • Yes — Claude Code (Opus 4.8)

Generated-by: Claude Code (Opus 4.8) following the guidelines

…ge classification

Rules-first classifier (doc-only/test-only/Bump decided; Fix/Add etc. -> needs_llm) that replaces the random --non-interactive discovery the prepare-providers-documentation skill abused. Outputs JSON so the skill takes deterministic results as-is and only LLM-classifies needs_llm commits.
Copy link
Copy Markdown
Contributor

@bugraoz93 bugraoz93 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Breeze docs check failing. Looks good :)

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

Labels

area:dev-tools backport-to-v3-2-test Mark PR with this label to backport to v3-2-test branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants