Skip to content

Conversation

@murchandamus
Copy link
Contributor

@murchandamus murchandamus commented Dec 10, 2025

This PR addresses feedback from the mailing list and a recent pull request to BIP 3. The following functional changes are made:

  • Revert the recently added AI guidance
  • Broaden the formats per which reference implementations may be provided
  • Move Type header responsibility to the author(s)

A Changelog section is added and backfilled, and a few editorial improvements are made. The Version header is omitted at this time, since it is not permitted under BIP 2.

@murchandamus murchandamus mentioned this pull request Dec 10, 2025
@murchandamus murchandamus force-pushed the 2025-12-bip3-address-activation-motion-feedback branch 2 times, most recently from 78873cf to 4429b13 Compare December 10, 2025 00:58
Copy link
Contributor

@real-or-random real-or-random left a comment

Choose a reason for hiding this comment

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

ACK 4429b13

@murchandamus murchandamus force-pushed the 2025-12-bip3-address-activation-motion-feedback branch from 4429b13 to 41fe83f Compare December 11, 2025 16:57
@murchandamus
Copy link
Contributor Author

Added a missing comma:

-"bip-title.[md|mediawiki]". Only BIP Editors may assign BIP numbers. Until one has done so authors should refer to their
+"bip-title.[md|mediawiki]". Only BIP Editors may assign BIP numbers. Until one has done so, authors should refer to their

Copy link
Member

@darosior darosior left a comment

Choose a reason for hiding this comment

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

ACK c637531

Looks good to me. Overall, the changes are minor with the exception of removing the explicit ban on LLM-generated content.

This is an important document which is important to get right, and i understand that we want to keep refining it. But it is also a living one, which can be updated and improved after being activated, and is already a substantial improvement over the current process. I hope we can collectively come to an agreement soon and conclude this years-long effort.

@murchandamus murchandamus force-pushed the 2025-12-bip3-address-activation-motion-feedback branch from c637531 to 897fa1b Compare December 11, 2025 21:30
Copy link
Member

@darosior darosior left a comment

Choose a reason for hiding this comment

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

reACK 897fa1b

@dni
Copy link

dni commented Dec 12, 2025

👍

Copy link
Member

@jonatack jonatack left a comment

Choose a reason for hiding this comment

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

ACK

the community's time. No content may be generated by AI/LLMs and authors must proactively disclose
up-front any use of AI/LLMs.
documenting design decisions that have gone into implementations. BIPs may be submitted by anyone, provided the
content is of high quality, e.g., does not waste the community’s time.
Copy link
Member

@jonatack jonatack Dec 15, 2025

Choose a reason for hiding this comment

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

Would something roughly like this (didn't cross-verify with the rest of the BIP) be a good idea? (to discourage skipping steps and opening a PR prematurely)

Suggested change
content is of high quality, e.g., does not waste the community’s time.
content is of high quality and the process followed, e.g., does not waste the community’s time.

Copy link
Member

Choose a reason for hiding this comment

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

The ease with which LLMs are enabling people to submit complete-looking, "thorough" techno-babbly drafts (h/t Greg Maxwell) might be counteracted somewhat byy insisting on following the process of discussing first the idea, and then maybe later a draft, on the list before opening a PR here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Since the whole document is providing guidance for participating in the process, the suggestion that the process should be followed does not strike me as controversial. I’d prefer to wrap on making changes, though, as every additional change resets prior ACKs. Would it be fine with you to relegate this to a follow-up?

Copy link
Member

Choose a reason for hiding this comment

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

SGTM (for a possible follow-up).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Great, thanks. I’ll leave this unresolved so it won’t be missed for the follow-up.

@sipa
Copy link
Member

sipa commented Dec 15, 2025

LGTM

Copy link
Contributor

@real-or-random real-or-random left a comment

Choose a reason for hiding this comment

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

ACK 897fa1b

@jonatack jonatack merged commit f427295 into bitcoin:master Dec 15, 2025
4 checks passed
@murchandamus murchandamus mentioned this pull request Dec 16, 2025
6 tasks
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.

7 participants