Hi,

I noticed that agents keep adding newsfragments… So I tried to summarize
the discussion in this thread and turned it into a PR:

https://github.com/apache/airflow/pull/74589

Ash already made some suggestions (Thanks Ash!)
Would appreciate some more opinions on this to make it accurate.

Thanks,
Andrew

Jarek Potiuk <[email protected]> 於 2026年9月12日週六 下午7:18寫道:

> +1 to what Kaxil wrote, and +100 for removing ambiguity and contradicting
> statement.
>
> My 2c: I think the "notable" criteria should be clearer (also to make it
> easier for agentic workflows to decide what to do). What is considered
> "notable" might differ between people. I think we should be very
> precise—for example, specifying both when we should and when we should not
> use newsfragments.
>
> For example:
>
> "Only use .significant newsfragments. And only if there is something that
> might require separate, dedicated user action when migrating to that
> version of Airflow. Never add other types of newsfragments and never add
> them for regular bugfixes or features, especially if users installing
> Airflow do not have to do anything to use the feature or when the bug fix
> should not affect their regular workflows"
>
> Something like that—perhaps phrased better—but clarity and precision
> regarding when to use them and when not to use them are key. Also we
> should fail hard (configure towncrier in this way) to **only** allow
> .significant ones. Using another type should fail prek hook.
>
> Note: Dennis ... It's not that that Claude "likes" adding them. Simply our
> instructions - as noted by Kaxil, are contradictory and ambiguous, so when
> Agent get the notion about newsfragments, it gets confused - in the very
> same regular contributors - humans -  as we would be confused IF we
> actually read all the instructions and try to follow them. So in this case
> the fact that **Claude** or whatever agent is adding them semi-randomly, is
> **our** fault because we were not consistent in telling the agents what to
> do, and it was ambiguous.
>
> J.
>
>
> On Sat, Sep 12, 2026 at 2:27 AM Ferruzzi, Dennis <[email protected]>
> wrote:
>
> > FWIW, I agree they seem to be getting a bit more common than they need to
> > be.  I've also noticed that Claude really likes adding them.  Perhaps
> worth
> > an update to the AGENST file in parallel, once the policy is decided.
> >
> > -ferruzzi
> > ________________________________
> > From: Kaxil Naik <[email protected]>
> > Sent: Friday, September 11, 2026 6:44 AM
> > To: [email protected] <[email protected]>
> > Subject: [EXT] [DISCUSS] Newsfragments: when to add one
> >
> > CAUTION: This email originated from outside of the organization. Do not
> > click links or open attachments unless you can confirm the sender and
> know
> > the content is safe.
> >
> >
> >
> > AVERTISSEMENT: Ce courrier électronique provient d’un expéditeur externe.
> > Ne cliquez sur aucun lien et n’ouvrez aucune pièce jointe si vous ne
> pouvez
> > pas confirmer l’identité de l’expéditeur et si vous n’êtes pas certain
> que
> > le contenu ne présente aucun risque.
> >
> >
> >
> > Hi all,
> >
> > Pierre and I were discussing today whether a new feature should carry a
> > newsfragment. We both went looking for the what our contributing doc say,
> > neither of us found a consistent one, and we agreed that
> > if the two of us are unsure then most of contributors too would be. So I
> > went through what is on main
> > and what actually reaches the release notes.
> >
> > 1. Only one of the six types reliably reaches users.
> >
> > The towncrier config declares six types. Counting their section headings
> > across
> > the whole of RELEASE_NOTES.rst: "Significant Changes" appears 49 times,
> > "Improvements" 7, and "Features", "Doc only Changes" and "Misc" have
> never
> > appeared once. The sections carrying the bulk of every release, New
> > Features (10
> > appearances), Bug Fixes, Miscellaneous (35) and Doc Only Changes (34),
> are
> > generated by dev/airflow-github changelog, which groups commits by their
> > type:
> > label and does not read newsfragments at all.
> >
> > The per-fragment view is worse. Of the already-released bugfix fragments,
> > 19 of
> > 21 left no trace of their own wording in the release notes; the commit
> > subject
> > shipped instead. Same for 16 of 17 feature fragments. significant is the
> > only
> > type that lands with any consistency, 11 of 20.
> >
> > So it feels a bugfix or feature fragment pollutes the release notes
> > without adding anything. For those types an author
> > writes prose that nothing consumes, and the type: label produces the line
> > that
> > ships. That is the reverse of the original motivation, which was that a
> > release-note entry should be free to differ from the commit subject.
> >
> > 2. Our written guidance contradicts itself.
> >
> > contributing-docs/18_contribution_workflow.rst says "consider adding a
> > newsfragment", lists all six types, and says a significant fragment
> > "doesn't
> > have to be a breaking change, it can be something that is notable but not
> > breaking". AGENTS.md calls them "only for major or breaking changes" in
> one
> > bullet and asks for one on any "user-facing" change two paragraphs later,
> > with
> > all six types in its example command. The prek hook checks only the
> > filename
> > shape and the line count, never whether the type fits or whether a
> fragment
> > was
> > warranted. Contributors guess, and coding agents guess more confidently
> and
> > more
> > often, which is how a good chunk of those 134 files got there.
> >
> > What I would like us to settle
> >
> > My preference, held loosely:
> >
> > - Newsfragments are for significant only: a behaviour change, a removal,
> a
> > new
> >   default, a migration concern, anything someone must read before
> > upgrading. Not
> >   breaking is fine; notable is the bar.
> > - Everything else is classified by its type: label, which is already how
> > those
> >   sections are built. No fragment for an ordinary bugfix, feature or doc
> > change.
> > - The release process prunes main. After a release, the commit where
> > towncrier
> >   deleted the consumed fragments gets cherry-picked back, as originally
> >   intended. I am happy to sweep the 62 stale ones as a one-off.
> > - One rule, written once, with contributing-docs and AGENTS.md pointing
> at
> > it.
> >
> > The alternative is to keep the other five types and make the release
> > process
> > actually emit them, which means dropping the git-log sections and
> requiring
> > a
> > fragment on every user-facing PR. I do not think it is
> > worth it, but I would rather we decide than keep drifting.
> >
> > The pruning is worth doing whichever way the types question lands,
> > otherwise 3.4
> > ships with 52 entries for changes users already have.
> >
> > Original thread where we added Towncrier:
> > https://lists.apache.org/thread/1bch0s4tr4w3vvxwt12z3vx0cyh0qp4o
> >
> > Thanks,
> > Kaxil
> >
>

Reply via email to