BC
← All projectsBitcoin Core

Bitcoin Core

The Bitcoin network's reference node and wallet implementation.

BitcoinSupply chainNormal
Repository coverage

2902 commits in the local evidence base

Every captured commit receives deterministic security triage and a separate communication-quality score. Security candidates and broader second-pass signals receive full-patch Ollama analysis.

254security candidates607second-pass queue2881AI analyses
192commits · 30 days
486commits · 60 days
1504commits · 180 days
2875commits · 365 days
Backfill bands
Aug 5 → Feb 61351 seen45 candidatesComplete
Feb 6 → Jun 61033 seen63 candidatesComplete
Jun 6 → Jul 6281 seen11 candidatesComplete
Jul 6 → Aug 5207 seen5 candidatesComplete
Commit communication

Does the history explain itself?

Message quality measures whether a commit identifies its scope, purpose, rationale, testing, and supporting references. It does not change the security-severity score.

70/100 average clarity
939Strong · 80–100
1188Adequate · 60–79
687Thin · 40–59
88Opaque · 0–39
6security candidates with opaque commit messaging
Read the scoring rubric →
Developer activity

Who is changing the project?

Public Git author strings; identities are not independently verified.

DeveloperCommitsCandidatesAnalyzedHigh riskMessage avg.
Antoine Poinsot22422170
Ava Chow15750157064
MarcoFalke41021406074
Lőrinc17721177081
fanquake22719226057
Hennadii Stepanov20914208063
rkrux57957074
Sjors Provoost89889074
Sebastian Falbesoner33733073
David Gumberg55655072
Pieter Wuille95595066
Hodlinator66566076
Analysis record

Published AI watches

Last scanned 21 minutes ago

Informational 15 AI analysisMessage 91 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

Merge bitcoin/bitcoin#35928: doc: mention -DWITH_ZMQ=ON in macOS build guide

This is a one-line documentation update to the macOS build guide. It adds a note telling users to pass a specific CMake option if they want ZeroMQ notification support. There is no code change and no security impact.

8397e09eby merge-script+1−11 file
No security note in commit
Informational 15 AI analysisMessage 100 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

Merge bitcoin/bitcoin#35482: fuzz: exercise the transaction-handling path in process_message(s)

This commit only changes Bitcoin Core's internal fuzz testing code. It makes the fuzz tests exercise more of the transaction-handling code path by toggling Initial Block Download mode and resetting the mempool between test runs. There is n…

No production code modifiedNo consensus, validation, net_processing, or wallet logic changedOnly fuzz test harnesses and test utilities affected
f11dc617by merge-script+105−798 files
No security note in commit
Informational 15 AI analysisMessage 91 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

Merge bitcoin/bitcoin#35914: test, fuzz: Remove unused variables

This commit removes unused variables from Bitcoin Core's test and fuzzing code. It is a cleanup/refactoring change with no effect on the live network software or user funds. It does not fix or introduce any security vulnerability.

d36bf709by merge-script+1−138 files
No security note in commit
Informational 15 AI analysisMessage 93 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

Merge bitcoin/bitcoin#35896: refactor: Default uint256::operator==, add operator<=>

This commit is a routine code cleanup in Bitcoin Core. It switches the uint256 equality and comparison operators to use standard C++20 defaults, removes an old custom Compare() helper, and marks an internal assertion-failure function as ne…

ed2c59abby merge-script+99−245 files
No security note in commit
Moderate 62 AI analysisMessage 96 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

Merge bitcoin/bitcoin#35872: rpc: avoid descriptor range counter overflow

This update fixes a counting bug in several Bitcoin Core RPC commands that scan descriptors. When a user requested a descriptor range ending at the maximum allowed value (2,147,483,647), the internal counter used a smaller integer type and…

Signed integer overflow in descriptor expansion loopCrash/undefined behavior on maximum-range descriptor scansAuthenticated RPC surface affected (scantxoutset, scanblocks, getdescriptoractivity, utxoupdatepsbt, descriptorprocesspsbt)
b388674aby Ava Chow+3−12 files
Vendor flagged security relevance
Informational 19 AI analysisMessage 91 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

Merge bitcoin/bitcoin#35842: rpc: Properly make RPCResult::Type::ANY non-test-only

This is a small cleanup and documentation fix for Bitcoin Core's RPC help system. It removes a comment that incorrectly said a certain output type was 'for testing only' and makes the help text properly display those outputs. There is no d…

No memory safety, cryptography, consensus, or authorization changes observedChange is confined to RPC help/schema metadata generationComment-only/type-label change from 'for testing only' to general use
c36ffd87by merge-script+42−364 files
No security note in commit
Informational 15 AI analysisMessage 93 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

Merge bitcoin/bitcoin#35895: refactor: Enable clang-tidy rule to reject anon namespace in header

This is a code cleanup change that turns on a static-analysis rule to prevent a specific C++ coding pattern (anonymous namespaces in header files) and updates two headers to comply. It does not change how Bitcoin Core behaves at runtime an…

No security-relevant code changeNo memory safety, cryptography, consensus, or network changesTooling-only refactor (clang-tidy configuration)
c4fbd3c7by merge-script+9−123 files
No security note in commit
Informational 15 AI analysisMessage 100 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

Merge bitcoin/bitcoin#35630: test: Add importdescriptors rpc error test coverage

This commit only adds new test cases to Bitcoin Core's functional test suite. It does not change any production wallet, node, or RPC code. The tests verify that the importdescriptors RPC reports errors in the right order, rejects bad times…

465196d0by merge-script+77−11 file
No security note in commit
Informational 15 AI analysisMessage 81 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

Merge bitcoin/bitcoin#35180: coins: group private cache helpers

This change is purely a code cleanup: it moves two internal helper functions of a Bitcoin Core cache class into the private section of the class and removes a duplicate 'private:' label. There is no change to what the code does, no bug fix…

c940fd75by merge-script+13−141 file
No security note in commit
Low 29 AI analysisMessage 81 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

Merge bitcoin/bitcoin#35582: rpc: reject null for optional parameters

This Bitcoin Core change tightens how three RPC commands (scantxoutset, scanblocks, deriveaddresses) handle the value null when it is passed for optional parameters. Previously, explicitly passing null could be treated differently from sim…

RPC parameter validation changeNull value handling changeAddition of explicit error checks for missing required contextual parameters
3db96eb5by merge-script+18−75 files
No security note in commit
Informational 15 AI analysisMessage 57 · Thin
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

refactor: Enable misc-definitions-in-headers

This commit is a code cleanup: it turns on a clang-tidy style check called 'misc-definitions-in-headers' and suppresses that check around a large inline implementation block in a benchmark header. There is no change to Bitcoin's runtime be…

fa93132dby MarcoFalke+3−02 files
No security note in commit
Informational 15 AI analysisMessage 85 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

refactor: Enable clang-tidy rule to reject anon namespace in header

This commit only changes a linting configuration file for the project's code style checker. It enables a rule that prevents anonymous namespaces from being used in header files, which is a code-quality and build-hygiene practice. There is …

No security-relevant signals in the diff or commit message.Change is purely a static-analysis/linting configuration update.
fa5ca877by MarcoFalke+2−11 file
No security note in commit
Informational 15 AI analysisMessage 62 · Adequate
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

refactor: Use C++20 std::identity over IntIdentity

This commit is a straightforward code cleanup: it replaces a small custom helper named IntIdentity with the standard C++20 std::identity from the <functional> header. The behavior of the ConvertBits function is unchanged; only the implemen…

fafe5042by MarcoFalke+4−111 file
No security note in commit
Informational 15 AI analysisMessage 87 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

test: test the result order of a multiple import request is correct

This commit only adds a new automated test to Bitcoin Core. It checks that when a user asks the wallet to import multiple descriptors at once, the list of results comes back in the same order as the original request, including any error me…

3ac8b806by Pol Espinasa+40−01 file
No security note in commit
Informational 15 AI analysisMessage 87 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

test: test invalid or missing timestamp throws importdescriptors

This commit only adds new automated tests for the Bitcoin Core wallet's importdescriptors RPC. It checks that the command correctly rejects requests with a missing or invalid timestamp. No production wallet code is changed, so this cannot …

No changes to consensus, networking, wallet logic, or cryptographyOnly functional test code is modifiedAdded assertions are for expected error handling paths
e4732bf0by Pol Espinasa+28−11 file
No security note in commit
Informational 15 AI analysisMessage 80 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

refactor: Remove unused #include in common/system

This is a minor code cleanup that removes one unused header file include and swaps another for a more specific one. It does not change any program behavior or fix any security issue.

fa7304f3by MarcoFalke+1−22 files
No security note in commit
Informational 15 AI analysisMessage 50 · Thin
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

iwyu: Fix warnings in `src/consensus` and treat them as errors

This commit is a routine code cleanup: it adjusts which C++ header files are included in several consensus-related source files and turns on a stricter compiler hygiene check (Include What You Use, or IWYU) for the src/consensus directory.…

13b53f8bby Hennadii Stepanov+42−79 files
No security note in commit
Informational 15 AI analysisMessage 55 · Thin
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

fuzz: don't connman.ReceiveMsgFrom oversized msg

This is a small fix to a Bitcoin Core fuzz test (an automated internal testing harness), not to the live network code. The fuzzer was sometimes creating fake P2P messages larger than the real protocol allows and passing them into a test he…

Test-only fuzz harness hardeningOversized message guard added before ReceiveMsgFrom() in fuzz targetNo change to production P2P message acceptance logic
bb19f1daby Greg Sanders+5−01 file
No security note in commit
Informational 15 AI analysisMessage 97 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

ci: Temporarily remove riscv32 config from GHA matrix

This commit simply removes one failing test configuration (RISC-V 32-bit bare metal) from the project's GitHub Actions CI matrix because it was failing. It is a routine CI maintenance change with no security implications.

fa06ea42by MarcoFalke+0−61 file
No security note in commit
Low 47 AI analysisMessage 90 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

ci: verify cross-build SDK archives

This change adds checksum verification to the build system's downloads of Apple, FreeBSD, NetBSD, and OpenBSD software development kits (SDKs). Before this patch, those SDK archives were downloaded at build time and extracted without confi…

Adds cryptographic digest verification for downloaded SDK archivesRemoves unchecked extraction of remote SDK archives in CIHardens CI supply chain for macOS/BSD cross-builds
873550beby Lőrinc+25−126 files
Vendor flagged security relevance
Repository ledger

Explore captured commits

Expand any commit for its author, full message, clarity score, changed files, triage signals, analysis, and source link.

Lower-priorityclusterlin: remove unused MergeLinearizations (cleanup)by Pieter Wuille · 91399a79 · Dec 18, 2025 · 3 filesMessage 65 · AdequateInformational 15Details
Commit message · Pieter Wuille

clusterlin: remove unused MergeLinearizations (cleanup)

This ended up never being used in txgraph.

65/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
AI analysis · Informational 15/100

This commit simply removes an unused helper function called MergeLinearizations, along with its benchmark and fuzz test. It is a routine code cleanup with no security relevance.

Security candidateclusterlin: randomize various decisions in SFL (feature)by Pieter Wuille · 13aad26b · Dec 18, 2025 · 3 filesMessage 90 · StrongInformational 19Details
Commit message · Pieter Wuille

clusterlin: randomize various decisions in SFL (feature)

This introduces a local RNG inside the SFL state, which is used to randomize
various decisions inside the algorithm, in order to make it hard to create
pathological clusters which predictably have bad performance.

The decisions being randomized are:
* When deciding what chunk to attempt to split, the queue order is
randomized.
* When deciding which dependency to split on, a uniformly random one is
chosen among those with higher top feerate than bottom feerate within
the chosen chunk.
* When deciding which chunks to merge, a uniformly random one among those
with the higher feerate difference is picked.
* When merging two chunks, a uniformly random dependency between them is
now activated.
* When making the state topological, the queue of chunks to process is
randomized.

90/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode✓ Names security-relevant behavior explicitly
Why it was queued
entropy or randomness
AI analysis · Informational 19/100

This commit adds randomness to an internal Bitcoin Core algorithm that orders groups of related transactions (clusters) for block inclusion. The goal is to prevent someone from deliberately crafting a cluster of transactions that would always trigger the worst-case performance of the algorithm. It is a hardening/performance improvement, not a fix for a vulnerability that steals funds or breaks consensus.

Lower-priorityclusterlin: add support for loading existing linearization (feature)by Pieter Wuille · 6a8fa821 · Dec 18, 2025 · 2 filesMessage 50 · ThinInformational 13Details
Commit message · Pieter Wuille

clusterlin: add support for loading existing linearization (feature)

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 13/100

This commit adds a new feature to Bitcoin Core's internal transaction-cluster linearization code. It lets the algorithm start from an existing ordering of transactions rather than building one from scratch. The change is purely an optimization/feature addition in mempool policy code and does not alter network rules, consensus, or wallet behavior. There is no indication it fixes a security bug.

AI review queuedclusterlin: ReadLinearization for non-topological (tests)by Pieter Wuille · da48ed9f · Dec 18, 2025 · 1 fileMessage 83 · StrongInformational 15Details
Commit message · Pieter Wuille

clusterlin: ReadLinearization for non-topological (tests)

Rather than using an ad-hoc no-dependency copy of the graph when a potentially
non-topological linearization is needed in the clusterlin fuzz test, add this
directly as a feature in ReadLinearization().

This is preparation for a later commit where another use for such a function
is added.

83/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification
Why it was queued
fuzzing or regression evidencesecond-pass: broader security terminology
AI analysis · Informational 15/100

This is a small internal cleanup of Bitcoin Core's fuzz testing code. It refactors how test inputs are converted into arbitrary orderings of transactions, with no change to production consensus or networking code. There is no security issue visible in the commit.

Lower-priorityclusterlin: add class implementing SFL state (preparation)by Pieter Wuille · c461259f · Dec 18, 2025 · 2 filesMessage 91 · StrongInformational 14Details
Commit message · Pieter Wuille

clusterlin: add class implementing SFL state (preparation)

This adds a data structure representing the optimization state for the spanning-forest
linearization algorithm (SFL), plus a fuzz test for its correctness.

This is preparation for switching over Linearize() to use this algorithm.

See https://delvingbitcoin.org/t/spanning-forest-cluster-linearization/1419 for
a description of the algorithm.

91/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification✓ Links an issue, advisory, or supporting reference
Why it was queued
fuzzing or regression evidence
AI analysis · Informational 14/100

This commit adds a new internal data structure and algorithm implementation for optimizing how Bitcoin Core orders groups of related transactions (clusters) when building blocks. It is purely preparatory code: it introduces the new 'spanning-forest linearization' state class and a fuzz test to verify its correctness, but it does not yet wire it into the actual block-building path. There is no indication this change fixes or introduces a security vulnerability.

Lower-prioritybuild: Update minimum required Boost versionby Hennadii Stepanov · f480c1e7 · Dec 18, 2025 · 2 filesMessage 57 · ThinInformational 17Details
Commit message · Hennadii Stepanov

build: Update minimum required Boost version

Building with Boost 1.73.0 is broken.

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 17/100

This commit simply raises the minimum required Boost library version from 1.73.0 to 1.74.0 because building with 1.73.0 is broken. It is a build-system/dependency documentation change, not a fix for a security vulnerability in Bitcoin Core itself.

Lower-priorityclusterlin: add known-correct optimal linearization tests (tests)by Pieter Wuille · 86dd550a · Dec 18, 2025 · 2 filesMessage 60 · AdequateInformational 15Details
Commit message · Pieter Wuille

clusterlin: add known-correct optimal linearization tests (tests)

60/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Mentions testing or verification! No meaningful explanatory body
AI analysis · Informational 15/100

This commit adds new unit tests for Bitcoin Core's transaction-cluster linearization code. It does not change any production (non-test) code, network behavior, or consensus rules. The tests verify that the linearization algorithm produces known-optimal results on a large set of real and synthetic mempool cluster examples. There is no security-relevant change here.

Lower-priorityclusterlin: replace benchmarks with SFL-hard ones (bench)by Pieter Wuille · 95bfe7d5 · Dec 18, 2025 · 2 filesMessage 65 · AdequateInformational 15Details
Commit message · Pieter Wuille

clusterlin: replace benchmarks with SFL-hard ones (bench)

This also adds a per-cost variant of each.

65/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
AI analysis · Informational 15/100

This commit only changes benchmark tests and adds a small helper function to count transaction dependencies. It does not alter how Bitcoin Core processes real transactions or network messages, so it has no direct security impact on running nodes.

Lower-priorityAdd regtest support to bitcoin-chainstate toolby stringintech · 5f3d6bdb · Dec 18, 2025 · 1 fileMessage 70 · AdequateInformational 15Details
Commit message · stringintech

Add regtest support to bitcoin-chainstate tool

Adds -regtest flag to enable testing with regtest chain parameters.

70/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Mentions testing or verification
AI analysis · Informational 15/100

This commit adds a command-line flag to an experimental developer/testing tool so it can use Bitcoin's local-only 'regtest' network instead of the real main network. It is a straightforward feature addition with no security relevance.

AI review queuedFix `ChainstateManager::AddChainstate()` assertion crashby stringintech · 2bc32656 · Dec 18, 2025 · 1 fileMessage 65 · AdequateLow 31Details
Commit message · stringintech

Fix `ChainstateManager::AddChainstate()` assertion crash

Check mempool exists before accessing size when prev_chainstate doesn't have initialized mempool.

65/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
Why it was queued
second-pass: broader security terminology
AI analysis · Low 31/100

This is a one-line fix in Bitcoin Core that prevents a program crash (assertion failure) when adding a new chainstate. The crash could occur if the previous chainstate did not have a memory pool (mempool) initialized. The change simply checks whether the mempool exists before checking its size, avoiding an invalid memory access. It is a defensive hardening fix rather than a user-triggerable exploit path under normal operation.

Lower-prioritytest: Add bitcoin-chainstate test for assumeutxo functionalityby stringintech · 7b5d256a · Dec 18, 2025 · 2 filesMessage 87 · StrongInformational 15Details
Commit message · stringintech

test: Add bitcoin-chainstate test for assumeutxo functionality

Adds functional test coverage for bitcoin-chainstate tool loading a datadir initialized with an assumeutxo snapshot

87/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body✓ Mentions testing or verification
AI analysis · Informational 15/100

This commit adds a new automated test for the bitcoin-chainstate tool. It checks that the tool can load a data directory initialized with an assumeutxo snapshot and continue processing new blocks. There is no change to production code that handles user funds, network messages, or consensus rules, and no security fix is present.

Lower-prioritydoc: add missing copyright headersby fanquake · aeb7ccb9 · Dec 18, 2025 · 2 filesMessage 45 · ThinInformational 15Details
Commit message · fanquake

doc: add missing copyright headers

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 15/100

This commit only adds standard copyright notice comments to the top of two test-only source files. It makes no changes to program logic, data handling, or network behavior, and therefore has no security relevance.

Security candidatecontrib: output copyright in generate-seeds.pyby fanquake · 68a7cb8f · Dec 18, 2025 · 1 fileMessage 45 · ThinInformational 15Details
Commit message · fanquake

contrib: output copyright in generate-seeds.py

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
seed or entropy path
AI analysis · Informational 15/100

This commit simply adds a standard copyright and license notice to the top of a generated source file. It is a legal/documentation change, not a security fix or code behavior change.

Lower-prioritykernel: revert accidentally removed copyright headerby Lőrinc · 85314dc0 · Dec 18, 2025 · 1 fileMessage 81 · StrongInformational 15Details
Commit message · Lőrinc

kernel: revert accidentally removed copyright header

See:
https://github.com/bitcoin/bitcoin/commit/7990463b1059ba5fc4ebe37fd1105a9e168ae20d?diff=split#diff-04e685224f1ac5bfd91d47d8d7528a2e44f94fab5535d4b6b5af79b5a13aeb93L1-L12

81/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
AI analysis · Informational 15/100

This commit simply restores a copyright and explanatory comment block at the top of a source file. It is a documentation-only fix with no effect on program behavior, security, or network safety.

Lower-prioritytest: [doc] Manually unify stale headersby MarcoFalke · fa4cb13b · Dec 18, 2025 · 2 filesMessage 67 · AdequateInformational 15Details
Commit message · MarcoFalke

test: [doc] Manually unify stale headers

67/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification! No meaningful explanatory body
AI analysis · Informational 15/100

This commit only updates copyright header wording in two test files, changing '2024-Present' and '2022-' to '2022-present' for consistency. It does not change any executable code, network behavior, or security logic.

AI review queueddepends: capnp 1.3.0by fanquake · 719158db · Dec 18, 2025 · 1 fileMessage 38 · OpaqueInformational 11Details
Commit message · fanquake

depends: capnp 1.3.0

38/100 · OpaqueMessage clarity
✓ Subject identifies a change✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Informational 11/100

This commit updates the version of the Cap'n Proto library used in Bitcoin Core's build system from 1.2.0 to 1.3.0, along with its corresponding checksum. It is a routine dependency bump with no direct code changes to Bitcoin Core itself. There is no explicit security claim in the commit, and no verified references were provided to indicate this fixes a known vulnerability.

Lower-prioritytest: address self-announcementby 0xb10c · 1841bf9c · Dec 18, 2025 · 2 filesMessage 80 · StrongInformational 15Details
Commit message · 0xb10c

test: address self-announcement

Test that a node sends a self-announcement with its external IP to
in- and outbound peers after connection open and again sometime later.

Since the code for the test is mostly the same for addr and addrv2
messages, I opted to add a new test file instead of having duplicate
code in p2p_addr_relay.py and p2p_addrv2_relay.py.

Co-Authored-By: rkrux <rkrux.connect@gmail.com>

80/100 · StrongMessage clarity
✓ Descriptive subject✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Mentions testing or verification
AI analysis · Informational 15/100

This commit only adds a new automated test file to Bitcoin Core. It checks that a node correctly tells its peers about its own external IP address after connecting and again later. There is no change to the actual Bitcoin network code, only new test code.

Security candidaterpc, doc: clarify the response of listtransactions RPCby rkrux · 1ed8e761 · Dec 18, 2025 · 2837 filesMessage 83 · StrongInformational 15Details
Commit message · rkrux

rpc, doc: clarify the response of listtransactions RPC

I noticed this behaviour while perf testing PR 27286 and it was not something
that I expected, updating the doc to make it present in the RPCHelp command.

83/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification
Why it was queued
cryptography-sensitive pathseed or entropy pathsigning or wallet pathboot or update pathauthentication pathparser or protocol path
AI analysis · Informational 15/100

This commit is a documentation-only change for the Bitcoin Core RPC command `listtransactions`. The author noticed an unexpected behavior while testing another change and updated the help text so users can see it. No code behavior was changed, and there is no security fix or vulnerability here.

Lower-prioritytest: Improve code coverage for pubkey checksby billymcbip · 6bb66fcc · Dec 18, 2025 · 1 fileMessage 67 · AdequateInformational 15Details
Commit message · billymcbip

test: Improve code coverage for pubkey checks

67/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification! No meaningful explanatory body
AI analysis · Informational 15/100

This commit only adds new test cases to a JSON test data file for Bitcoin Core's script validation tests. It does not change any production code, cryptographic checks, or network rules. The new tests verify that invalid public key formats are correctly rejected, which improves code coverage but introduces no security change by itself.

Lower-priorityp2p: Allow block downloads from peers without snapshot block after assumeutxo validationby stringintech · 0067abe1 · Dec 18, 2025 · 1 fileMessage 85 · StrongLow 37Details
Commit message · stringintech

p2p: Allow block downloads from peers without snapshot block after assumeutxo validation

After assumeutxo background validation completes, allow block downloads from peers that don't have the snapshot block in their best chain.

Previously, these peers were skipped until restart because `m_chainman.CurrentChainstate().SnapshotBase()` continued returning non-null even after validation finished. Add `m_chainman.CurrentChainstate().m_assumeutxo == Assumeutxo::UNVALIDATED` check to only apply the restriction while background validation is ongoing.

85/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode
Why it was queued
defensive validation
AI analysis · Low 37/100

This change fixes a logic bug in Bitcoin Core's peer-to-peer block downloading. After a node finishes validating an 'assumeutxo' snapshot, it was still refusing to download blocks from peers whose best chain did not include the snapshot block—until the node was restarted. The fix removes that unnecessary restriction once validation is complete, allowing normal block downloads to resume. It is a network-efficiency and robustness fix rather than a direct theft-of-funds vulnerability.

AI review queuedUpdate ci.ymlby Woolfgm · cd98caea · Dec 17, 2025 · 1 fileMessage 28 · OpaqueInformational 15Details
Commit message · Woolfgm

Update ci.yml

28/100 · OpaqueMessage clarity
✓ Subject identifies a change! No meaningful explanatory body
Why it was queued
documentation-only discountsecond-pass: opaque commit message
AI analysis · Informational 15/100

This commit simply bumps the GitHub Actions 'checkout' step from version 5 to version 6 in the project's continuous integration (CI) workflow file. It does not change any Bitcoin Core source code, consensus rules, wallet logic, or network behavior. There is no indication of a security fix or vulnerability.

Lower-priorityiwyu, clang-format: Sort includesby Hennadii Stepanov · 56750c4f · Dec 17, 2025 · 3 filesMessage 45 · ThinInformational 15Details
Commit message · Hennadii Stepanov

iwyu, clang-format: Sort includes

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 15/100

This commit is a routine tooling change for Bitcoin Core. It adds the clang-format package to a CI environment, runs clang-format after an automated include-sorting tool, and changes a formatting configuration option so that #include statements are regrouped and sorted. There is no change to the actual Bitcoin software logic, network behavior, or wallet handling, so it does not affect users' funds or node security.

Security candidateci: Add IWYU jobby Hennadii Stepanov · 2c78814e · Dec 17, 2025 · 6 filesMessage 83 · StrongInformational 15Details
Commit message · Hennadii Stepanov

ci: Add IWYU job

The change in `src/crypto/hex_base.cpp` is because GCC 14 is not
affected by an IWYU bug.
See: https://github.com/include-what-you-use/include-what-you-use/issues/1763.

83/100 · StrongMessage clarity
✓ Subject identifies a change✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Explains rationale or failure mode✓ Links an issue, advisory, or supporting reference
Why it was queued
cryptography-sensitive path
AI analysis · Informational 15/100

This commit adds a new automated CI (Continuous Integration) job that runs a tool called Include What You Use (IWYU) to check that source files include only the headers they actually need. It also makes minor supporting changes: adding a CI configuration, installing the IWYU tool, adjusting build scripts, adding a mapping file for IWYU, and removing an unused <tuple> header from one source file. There is no change to Bitcoin Core's runtime behavior, consensus rules, networking, wallet logic, or any user-facing functionality. It is purely a code-quality and tooling improvement.

Lower-prioritycmake: Fix target nameby Hennadii Stepanov · 94e4f04d · Dec 17, 2025 · 1 fileMessage 51 · ThinInformational 15Details
Commit message · Hennadii Stepanov

cmake: Fix target name

The executable target run by ctest is `mptest`.

This change removes unnecessary steps when building the `codegen`
target.

51/100 · ThinMessage clarity
✓ Subject identifies a change✓ Provides detailed explanatory context
AI analysis · Informational 15/100

This is a one-character CMake build-system fix. The developer corrected the name of an internal test target from 'mptests' to 'mptest' so that automated testing (ctest) can find and run it. It has no effect on the Bitcoin network, wallets, consensus rules, or user-facing security.

Lower-prioritycmake: Make `codegen` target dependent on `generate_build_info`by Hennadii Stepanov · 0f81e005 · Dec 17, 2025 · 1 fileMessage 50 · ThinInformational 15Details
Commit message · Hennadii Stepanov

cmake: Make `codegen` target dependent on `generate_build_info`

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This is a tiny CMake build-system change. It removes the EXCLUDE_FROM_ALL flag from a static library so that a code-generation target is built when needed. There is no indication this affects runtime security, wallet funds, network consensus, or user data.