Skip to content

Clarify unreleased-feature changes are not breaking in provider-doc skill#70255

Merged
shahar1 merged 1 commit into
apache:mainfrom
shahar1:update-provider-doc-skill-unreleased-breaking
Jul 22, 2026
Merged

Clarify unreleased-feature changes are not breaking in provider-doc skill#70255
shahar1 merged 1 commit into
apache:mainfrom
shahar1:update-provider-doc-skill-unreleased-breaking

Conversation

@shahar1

@shahar1 shahar1 commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

The prepare-providers-documentation skill drives AI classification of provider PRs into version bumps. Its breaking-change checklist flagged any public-symbol removal/rename as breaking, without checking whether the affected symbol had ever been released.

During a real release wave this produced a false positive: the Kafka provider added KafkaListenerPlugin (#68082) and renamed it to the Kafka Event Producer (#70014) within the same unreleased wave. The rename tripped the "public class renamed, no shim" rule and proposed a spurious major bump (1.x → 2.0), even though users never received the old name.

This change anchors the checklist on a released-only precondition: before classifying a removal/rename as breaking, confirm the symbol existed at the provider's last release tag. A within-wave rename of a brand-new operator/plugin is feature-shaped, not a major bump.

Docs/skill-only change — no runtime or user-facing behavior is affected.


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

Generated-by: Claude Code (Opus 4.8, 1M context) following the guidelines

…kill

Renaming or removing a symbol that was introduced within the same unreleased provider-release wave never reached users, so the AI-driven classification must not treat it as a breaking change and force a spurious major version bump. Anchor the breaking-change checklist on whether the affected symbol shipped in a released version.
@shahar1
shahar1 merged commit 7ef372a into apache:main Jul 22, 2026
64 checks passed
stephen-bracken pushed a commit to stephen-bracken/airflow that referenced this pull request Jul 22, 2026
…kill (apache#70255)

Renaming or removing a symbol that was introduced within the same unreleased provider-release wave never reached users, so the AI-driven classification must not treat it as a breaking change and force a spurious major version bump. Anchor the breaking-change checklist on whether the affected symbol shipped in a released version.
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.

2 participants