BC
← All projectsBitcoin Core

Bitcoin Core

The Bitcoin network's reference node and wallet implementation.

BitcoinSupply chainNormal
Repository coverage

2924 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.

258security candidates613second-pass queue2891AI analyses
183commits · 30 days
470commits · 60 days
1487commits · 180 days
2863commits · 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
960Strong · 80–100
1188Adequate · 60–79
688Thin · 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 Chow16351161065
MarcoFalke41021406074
Lőrinc17721177081
fanquake22819226057
Hennadii Stepanov21214208063
rkrux57957074
Sjors Provoost89889074
merge-script37718088
Sebastian Falbesoner33733073
David Gumberg55655072
Pieter Wuille95595066
Analysis record

Published AI watches

Last scanned 23 minutes ago

Low 28 AI analysisMessage 81 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

Merge bitcoin/bitcoin#35959: Update secp256k1 subtree to latest master

This commit updates Bitcoin Core's embedded copy of the secp256k1 cryptographic library to a newer version. The update adds a new Silent Payments module, lets advanced users swap in a custom SHA256 compression function, removes some old de…

Dependency update of a core cryptographic library (libsecp256k1)New runtime SHA256 compression callback with self-test/smoke test to catch mis-implementations earlyBehavior change in ellswift_xdh to reject secret keys >= curve order (vendor states not a security issue)
625f951bby Ava Chow+297−11626 files
No security note in commit
Informational 23 AI analysisMessage 91 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

Merge bitcoin/bitcoin#35605: wallet: rpc: Deprecate `removeprunedfunds` RPC

This commit deprecates a Bitcoin Core wallet RPC command called removeprunedfunds. The command lets users delete transactions from their own wallet, which can alter displayed balances. The change does not fix a software bug; it is a cleanu…

RPC allows deletion of arbitrary wallet transactions, affecting balancesDeprecation framed by authors as removing a dangerous and maintenance-burden featureNo authentication bypass, memory corruption, or consensus change present in diff
e9ed5e83by Ava Chow+64−695 files
No security note in commit
Informational 18 AI analysisMessage 91 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

Merge bitcoin/bitcoin#35930: wallet: post-#35501 cleanups in CWalletTx

This is a follow-up cleanup and hardening patch for the Bitcoin Core wallet's handling of transactions that have multiple possible witness versions (same transaction ID but different witness data). It renames a variable, simplifies a helpe…

Added input-validation during wallet transaction deserializationNew unit test specifically exercises rejection of mismatched variant txidFollow-up to PR #35501 which introduced witness-variant tracking in the wallet
512dc9afby Ava Chow+68−315 files
No security note in commit
Informational 15 AI analysisMessage 91 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

Merge bitcoin/bitcoin#35729: refactor: test: Unroll `&&` conditions in macros

This commit only changes Bitcoin Core's internal test code. It splits combined conditions inside test-check macros (like BOOST_CHECK(a && b)) into separate checks so that when a test fails, the test output points to the exact failing condi…

aa0e0f79by merge-script+196−9519 files
No security note in commit
Low 26 AI analysisMessage 100 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

Merge bitcoin/bitcoin#35493: wallet, descriptor: Fix MuSig private key completeness checks on `importdescriptors`

This commit fixes a bug in Bitcoin Core's wallet import logic for a new type of multi-signature setup called MuSig. Previously, when a user imported a MuSig descriptor that already contained all needed private keys, the wallet incorrectly …

False-positive warning in wallet import for fully-private MuSig descriptorsIncorrect private-key completeness check on synthetic aggregate public keyDescriptor logic now delegates completeness to `Descriptor::HavePrivateKeys()`
2c01832fby merge-script+76−374 files
No security note in commit
Informational 15 AI analysisMessage 81 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

Merge bitcoin/bitcoin#35943: doc: fix dead link in txrequest.h

This commit only fixes a broken web link in a code comment. It replaces an outdated URL to a research paper with a working URL to the same paper. There is no change to program logic, no security fix, and no vulnerability.

5d051c05by 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#33186: wallet, test: Ancient Wallet Migration from v0.14.3 (no-HD and Single Chain)

This commit only adds new automated tests for migrating very old Bitcoin Core wallets (version 0.14.3 from 2017) to the modern descriptor wallet format. It does not change any wallet production code, consensus rules, or network behavior. T…

757aa573by merge-script+246−124 files
No security note in commit
Low 28 AI analysisMessage 91 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

Merge bitcoin/bitcoin#35925: wallet, rpc: Exclude non-owned addresses from listreceivedby*

This update fixes a wallet reporting bug. When a user asked their Bitcoin wallet to list every address that had received money, including empty ones, the result incorrectly included foreign addresses that the wallet merely knew about (for …

Information disclosure: wallet RPCs leaked existence/labels of foreign addressesIncorrect access-control boundary: non-owned addresses exposed in 'received' reportFix uses IsMine() rather than address purpose metadata, avoiding inconsistent purpose handling
e8cc21c5by Ava Chow+19−32 files
No security note in commit
Informational 15 AI analysisMessage 91 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

Merge bitcoin/bitcoin#35937: test: Append print_suppressions=0 to LSAN_OPTIONS, and suppress bitcoin-qt

This commit only changes test configuration. It tells the LeakSanitizer tool used during automated testing to stop printing long lists of suppressed memory leaks, and it narrows the list of ignored leaks to the bitcoin-qt GUI test binary. …

5973e075by merge-script+3−42 files
No security note in commit
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
Repository ledger

Explore captured commits

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

Lower-priorityrefactor: Use std::span in HasValidProofOfWorkby Daniela Brozzoni · 45686522 · Nov 3, 2025 · 2 filesMessage 72 · AdequateInformational 15Details
Commit message · Daniela Brozzoni

refactor: Use std::span in HasValidProofOfWork

Co-Authored-By: Pieter Wuille <pieter@wuille.net>

72/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
AI analysis · Informational 15/100

This is a minor code cleanup that changes how a function accepts a list of block headers. It switches from requiring a specific container type (std::vector) to accepting any contiguous view (std::span), and updates the loop style. There is no change to security logic, no bug fix, and no behavior change visible to users or attackers.

Lower-priorityrefactor: Compute work from headers without CBlockIndexby Daniela Brozzoni · 4066bfe5 · Nov 3, 2025 · 5 filesMessage 85 · StrongInformational 20Details
Commit message · Daniela Brozzoni

refactor: Compute work from headers without CBlockIndex

Avoid the need to construct a CBlockIndex object just to compute work for a header,
when its nBits value suffices for that.

Co-Authored-By: Pieter Wuille <pieter@wuille.net>

85/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
AI analysis · Informational 20/100

This is a code cleanup change. It makes the way Bitcoin Core calculates 'proof of work' from block headers simpler and slightly more efficient, without changing the actual math or security rules. There is no indication this fixes a vulnerability.

Lower-priorityp2p: Avoid an IsAncestorOfBestHeaderOrTip callby Pieter Wuille · 0bf6139e · Nov 3, 2025 · 1 fileMessage 68 · AdequateInformational 14Details
Commit message · Pieter Wuille

p2p: Avoid an IsAncestorOfBestHeaderOrTip call

Just don't call this function when it won't have any effect.

Note that we can't remove the LookupBlockIndex call, since `last_received_header`
is needed to check if new headers were received (`received_new_header`).

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 14/100

This is a tiny code cleanup in Bitcoin Core's peer-to-peer message handling. It changes how one internal flag is set so that an existing true value is preserved, rather than rechecking a condition that cannot change a false value. There is no security issue visible in the diff.

Lower-priorityvalidation: don't reallocate cache for short-lived CCoinsViewCacheby Lőrinc · 0ac969cd · Nov 2, 2025 · 5 filesMessage 81 · StrongInformational 19Details
Commit message · Lőrinc

validation: don't reallocate cache for short-lived CCoinsViewCache

A few temporary `CCoinsViewCache`'s are destructed right after the `Flush()`, therefore it is not necessary to call `ReallocateCache` to recreate them right before they're killed anyway.

* `Flush()` - retains existing functionality;
* `Flush(/*will_reuse_cache=*/false)` - skips destruction and reallocation of the parent cache since it will soon go out of scope anyway;

For the `will_reuse_cache` parameter we want to see exactly which ones will reallocate memory and which won't - since both can be valid usages.

This change was based on a subset of https://github.com/bitcoin/bitcoin/pull/28945.

Co-authored-by: Martin Ankerl <martin.ankerl@gmail.com>

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 19/100

This commit is a small performance optimization for Bitcoin Core. It changes how temporary in-memory data caches are cleaned up after use. Previously, these short-lived caches were always emptied and then immediately re-created, only to be destroyed moments later. The patch adds an option to skip that pointless re-creation step. It is not a security fix and does not change what data is stored or how it is validated.

Security candidatecoins: reduce lookups in dbcache layer propagationby Lőrinc · c8f5e446 · Nov 1, 2025 · 1 fileMessage 91 · StrongInformational 18Details
Commit message · Lőrinc

coins: reduce lookups in dbcache layer propagation

Previously, when the parent coins cache had no entry and the child did, `BatchWrite` performed a find followed by `try_emplace`, which resulted in multiple `SipHash` computations and bucket traversals on the common insert path.

This change uses a single leading `try_emplace` and branches on the returned `inserted` flag.
In the `FRESH && SPENT` case (only exercised by tests), we erase the just-inserted placeholder (which is constant time with no rehash anyway).
Semantics are unchanged for all valid parent/child state combinations.

This change is a minimal version of https://github.com/bitcoin/bitcoin/pull/32128/commits/723c49b63bb10da843fbb6efc6928dca415cc47f and draws simplification ideas https://github.com/bitcoin/bitcoin/pull/30673/commits/ae76ec7bcff0a08a61f294882a71e46d177b009f.

Added TODO versions for related pre-existing issues that should be fixed in follow-ups.

Co-authored-by: Martin Ankerl <martin.ankerl@gmail.com>
Co-authored-by: Andrew Toth <andrewstoth@gmail.com>
Co-authored-by: optout <13562139+optout21@users.noreply.github.com>

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
constant-time or timing behavior
AI analysis · Informational 18/100

This is a small internal performance cleanup in Bitcoin Core's coin cache code. It replaces two hash-map operations with one, reducing CPU work when copying coin data from a child cache into a parent cache. The commit explicitly says behavior is unchanged; no security vulnerability is described or evident in the diff.

Lower-priorityAdd eclipse, partitioning, and fingerprinting note to i2p.mdby da1sychain · e346ecae · Oct 31, 2025 · 2 filesMessage 65 · AdequateInformational 15Details
Commit message · da1sychain

Add eclipse, partitioning, and fingerprinting note to i2p.md

Also introduced a Privacy Recommendations section to docs.

65/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit only changes documentation. It adds a new 'Privacy recommendations' section to the I2P guide and slightly rewords an existing note in the Tor guide. The text warns users that running a node on multiple networks at once can help protect the Bitcoin network against certain attacks, but may also allow an adversary to link those identities. There is no code change and no vulnerability being fixed.

Lower-priorityinit: Require explicit -asmap filenameby Ryan Ofsky · f6ec3519 · Oct 31, 2025 · 4 filesMessage 76 · AdequateInformational 20Details
Commit message · Ryan Ofsky

init: Require explicit -asmap filename

Currently, if `-asmap` is specified without a filename, bitcoind tries to load
`ip_asn.map` data file.

This change now requires `-asmap=ip_asn.map` or another filename to be
specified explicitly.

The change is intended to make behavior of the option explicit avoid confusion
reported https://github.com/bitcoin/bitcoin/issues/33386 where documentation
specifies a default file which is not actually loaded by default. It was
originally implemented in
https://github.com/bitcoin/bitcoin/pull/33631#issuecomment-3410302383 and
various alternatives are discussed there.

Co-authored-by: Fabian Jahr <fjahr@protonmail.com>

76/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
AI analysis · Informational 20/100

This change tightens how Bitcoin Core handles the -asmap command-line option. Previously, starting the node with just -asmap (no filename) would silently try to load a file named ip_asn.map from the data directory. Now the user must explicitly provide a filename such as -asmap=ip_asn.map, otherwise the node refuses to start with a clear error. This is a behavior-hardening and documentation-clarity fix, not a patch for an active security vulnerability.

Lower-prioritytest: Format strings in `*.rs`by rustaceanrob · 78d4d367 · Oct 31, 2025 · 3 filesMessage 90 · StrongInformational 15Details
Commit message · rustaceanrob

test: Format strings in `*.rs`

`format!` strings may contain variables within the string
representation. This is a lint as of a recent `rustc` nightly version.

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

This commit only reformats Rust 'format!' strings to use a newer, shorter style supported by recent Rust compiler versions. It changes how variables are inserted into strings, not what the strings do. There is no security impact.

Lower-priorityci: gha: Set debug_pull_request_number_str annotationby MarcoFalke · fa9d0f99 · Oct 31, 2025 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · MarcoFalke

ci: gha: Set debug_pull_request_number_str annotation

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

This commit adds a harmless debug annotation to Bitcoin Core's GitHub Actions CI workflow. It prints the pull request number as a machine-readable notice during CI runs, which helps link check runs back to their pull requests. There is no security issue here.

Lower-prioritytest: clarify submitBlock() mutates the templateby Sjors Provoost · 6eaa00fe · Oct 31, 2025 · 1 fileMessage 98 · StrongInformational 15Details
Commit message · Sjors Provoost

test: clarify submitBlock() mutates the template

PR #33374 proposed a new Mining IPC method applySolution() which
could be used by clients to obtain the reconstructed block for
inspection, especially in the case of a rejected block.

However it was pointed out during review that submitBlock() modified
the template CBlock in place, so the client can just call getBlock()
and no new method is needed.

This commit adds a test to document that (now intentional) behavior.

98/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Mentions testing or verification✓ Links an issue, advisory, or supporting reference
AI analysis · Informational 15/100

This is a harmless test-only change. It adds a new test case to make sure developers know that a mining interface function called submitBlock() changes the block template even when the submitted block is rejected. There is no security fix or vulnerability here.

Lower-prioritymining: ensure witness commitment check in submitBlockby Sjors Provoost · 862bd432 · Oct 31, 2025 · 3 filesMessage 83 · StrongModerate 63Details
Commit message · Sjors Provoost

mining: ensure witness commitment check in submitBlock

When an IPC client requests a new block template via the Mining interface,
we hold on to its CBlock. That way when they call submitSolution() we can
modify it in place, rather than having to reconstruct the full block like
the submitblock RPC does.

Before this commit however we forgot to invalidate
m_checked_witness_commitment, which we should since the client brings a
new coinbase.

This would cause us to accept an invalid chaintip.

Fix this and add a test to confirm that we now reject such a block.
As a sanity check, we add a second node to the test and confirm that will
accept our mined block.

Note that the IPC code takes the coinbase as provided, unlike the
submitblock RPC which calls UpdateUncommittedBlockStructures() and adds
witness commitment to the coinbase if it was missing.

Although that could have been an alternative fix, we instead document that
IPC clients are expected to provide the full coinbase including witness
commitment.

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
defensive validation
AI analysis · Moderate 63/100

Bitcoin Core's new mining IPC interface had a bug where it could accept an invalid block as the new chain tip. When an external mining client submitted a block solution with a new coinbase transaction, the node failed to re-check the block's witness commitment. This cached check could remain marked valid from the original template, allowing a block with a missing or wrong witness commitment to be accepted. The fix resets those cached checks whenever the coinbase is swapped in, and adds a test proving such invalid blocks are now rejected.

AI review queueddoc: clarify UpdateUncommittedBlockStructuresby Sjors Provoost · 00d1b6ef · Oct 31, 2025 · 1 fileMessage 35 · OpaqueInformational 15Details
Commit message · Sjors Provoost

doc: clarify UpdateUncommittedBlockStructures

35/100 · OpaqueMessage clarity
✓ Descriptive subject! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Informational 15/100

This commit only changes a code comment in a header file to clarify when a particular block-building helper function is safe to use. No executable code was modified, so it cannot introduce or fix a security vulnerability on its own.

AI review queuedqt: Remove HD seed reference from blank wallet tooltipby John Moffett · 929f69d0 · Oct 30, 2025 · 1 fileMessage 65 · AdequateInformational 15Details
Commit message · John Moffett

qt: Remove HD seed reference from blank wallet tooltip

Blank descriptor wallets currently do not have HD seeds and
none can be added, so remove the reference in the tooltip.

65/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit only changes a user-interface tooltip in the Bitcoin Core wallet creation dialog. It removes an outdated sentence that incorrectly said an HD seed could be added later to a blank wallet, which is not true for descriptor wallets. There is no code behavior change and no security impact.

Lower-priorityAdd eclipse, partitioning, and fingerprinting note in tor.mdby da1sychain · 19a6a3e7 · Oct 30, 2025 · 1 fileMessage 65 · AdequateInformational 15Details
Commit message · da1sychain

Add eclipse, partitioning, and fingerprinting note in tor.md

Minor spelling correction in privacy recommendations section

65/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit only adds a documentation note to the Tor setup guide. It warns users that running a Bitcoin node on multiple networks (like regular internet and Tor) can make it easier for attackers to correlate the node's identities, which may reduce privacy. No code was changed and no vulnerability was fixed.

Lower-prioritytest: resolve symlinks in which result for capnpby David Gumberg · 51093d6a · Oct 30, 2025 · 1 fileMessage 67 · AdequateInformational 15Details
Commit message · David Gumberg

test: resolve symlinks in which result for capnp

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 is a one-line change in a test file that resolves filesystem symlinks before looking up the Cap'n Proto include directory. It only affects the test suite's ability to find header files when the `capnp` binary is installed via a symbolic link. There is no security issue here.

Security candidaterefactor: Return uint64_t from GetSerializeSizeby MarcoFalke · fa6c0bed · Oct 30, 2025 · 1 fileMessage 93 · StrongLow 45Details
Commit message · MarcoFalke

refactor: Return uint64_t from GetSerializeSize

Normally, the values are small enough to fit in size_t, but the risk
that it may not fit sometimes is a reason to use uint64_t consistently
for all architectures.

On 64-bit systems, this refactor is a no-op. On 32-bit systems, it could
avoid bugs in the theoretical and unexpected case where a 32-bit size_t
is too small and overflows.

For example, 32-bit Bitcoin Core versions with CVE-2025-46597 unfixed
may overflow while checking for the bad-blk-length violation when
receiving a malformed and bloated compact block.

93/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference✓ Names security-relevant behavior explicitly
Why it was queued
explicit security languagememory safety
AI analysis · Low 45/100

This is a small defensive code change in Bitcoin Core that changes an internal size-measuring helper from using the system's native size type (size_t) to always using a 64-bit unsigned integer (uint64_t). On 64-bit computers nothing effectively changes. On 32-bit computers, the change prevents a theoretical integer overflow when calculating the serialized size of very large or malformed data. The commit message mentions this could matter for a specific malformed compact-block handling bug tracked as CVE-2025-46597 on 32-bit builds, but the patch itself only changes the return type and internal variable; it does not fix the compact-block logic directly.

Security candidaterefactor: Use uint64_t over size_t for serialized-size valuesby MarcoFalke · fad0c868 · Oct 30, 2025 · 9 filesMessage 97 · StrongInformational 18Details
Commit message · MarcoFalke

refactor: Use uint64_t over size_t for serialized-size values

The values are small enough to fit in size_t, but to avoid having to
think about it, just use uint64_t consistently for all architectures.

On 64-bit systems, this refactor is a no-op. On 32-bit systems, it could
avoid bugs in the theoretical and unexpected case where a 32-bit size_t
is too small and overflows.

97/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Explains rationale or failure mode
Why it was queued
memory safetysigning or wallet path
AI analysis · Informational 18/100

This is a code cleanup change that swaps the variable type `size_t` for `uint64_t` in a handful of places that store serialized data sizes. On 64-bit computers the two types are the same size, so nothing changes. On 32-bit computers, `uint64_t` is larger and could in theory prevent a rare overflow if a serialized object ever exceeded about 4 GB. The commit message explicitly calls this a defensive refactor and says the practical risk is theoretical and unexpected. There is no reported bug, exploit, or security incident tied to this change.

Lower-priorityrefactor: Use fixed size ints over (un)signed ints for serialized valuesby MarcoFalke · fa4f388f · Oct 30, 2025 · 3 filesMessage 85 · StrongInformational 15Details
Commit message · MarcoFalke

refactor: Use fixed size ints over (un)signed ints for serialized values

Bitcoin Core already assumes that 'unsigned int' means uint32_t and
'signed int' means int32_t. See src/compat/assumptions.h. Also, any
serialized integral value must be of a fixed size.

So make the fixed size explicit in this documenting refactor, which does
not change the behavior on any platform.

85/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
AI analysis · Informational 15/100

This commit is a pure code cleanup: it replaces plain 'int' and 'unsigned int' type names with explicitly fixed-width 'int32_t' and 'uint32_t' in a few data structures. The project already assumed these sizes elsewhere, and the serialization format does not change, so there is no security or behavior impact.

Security candidateci: Add missing python3-dev package for riscv64by MarcoFalke · facf8b77 · Oct 30, 2025 · 2 filesMessage 57 · ThinInformational 15Details
Commit message · MarcoFalke

ci: Add missing python3-dev package for riscv64

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
authentication path
AI analysis · Informational 15/100

This commit updates Bitcoin Core's continuous integration (CI) build scripts to install the python3-dev package instead of (or alongside) the base python3 package on two CI environments, including riscv64. It is a build-infrastructure fix, not a change to the Bitcoin node software that users run. There is no indication it fixes a security vulnerability in the Bitcoin protocol or wallet code.

Lower-prioritydoc: update Guix INSTALL.mdby fanquake · b4d0288c · Oct 30, 2025 · 1 fileMessage 45 · ThinInformational 15Details
Commit message · fanquake

doc: update Guix INSTALL.md

45/100 · ThinMessage clarity
✓ 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 commit only updates a documentation file (INSTALL.md) describing how to install the Guix build environment. It removes an outdated note about the packaging situation and tweaks wording. There are no code changes and no security relevance.

Lower-priorityci: fix lint docker cachingby will · 0b3b8a3b · Oct 30, 2025 · 1 fileMessage 88 · StrongInformational 15Details
Commit message · will

ci: fix lint docker caching

Fixes: 33735

Correct runner type selection for the lint job.

This was erroneously left-out during refactor of the runner selection
mechanism in #33302 causing the lint job to run on GH hosts (and
therefore not be able to acces local cirrus caches).

88/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This is a tiny GitHub Actions CI configuration fix. A previous code reorganization accidentally made the lint job run on GitHub's own virtual machines instead of the project's dedicated Cirrus runner machines. The fix changes the condition used to pick the runner so the lint job again uses the Cirrus runners and can access their local caches. It has no effect on the Bitcoin Core software users run, on wallets, transactions, consensus, or network security.

Lower-priorityfuzz: refactor memcpy to std::ranges::copy to work around ubsan warnby MarcoFalke · fa4b52bd · Oct 30, 2025 · 1 fileMessage 91 · StrongInformational 19Details
Commit message · MarcoFalke

fuzz: refactor memcpy to std::ranges::copy to work around ubsan warn

Using std::ranges::copy from the C++ standard library has a few benefits
here:

* It has the additional benefit of being a bit more type safe and
document the byte cast explicitly.
* The compiler will likely optimize it to the same asm, but performance
doesn't really matter here anyway.
* It works around an UB-Sanitizer bug, when the source range is empty.

Fixes https://github.com/bitcoin/bitcoin/issues/33643

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
defensive validationfuzzing or regression evidence
AI analysis · Informational 19/100

This is a minor cleanup change in Bitcoin Core's fuzz testing code (test-only, not production). It replaces a raw memory copy with a safer C++ standard library copy to silence a warning from the Undefined Behavior Sanitizer when the source data is empty. It does not fix a security vulnerability in live Bitcoin software.

Lower-prioritydepends: sqlite 3.50.4by fanquake · 1db74914 · Oct 29, 2025 · 1 fileMessage 69 · AdequateInformational 11Details
Commit message · fanquake

depends: sqlite 3.50.4

Migrate to the new Autosetup build system (introduced in 3.48.0).
See https://msteveb.github.io/autosetup/.
Also: https://sqlite.org/releaselog/3_50_0.html.

69/100 · AdequateMessage clarity
✓ Subject identifies a change✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
AI analysis · Informational 11/100

This commit updates the SQLite library used by Bitcoin Core's build system from version 3.46.1 to 3.50.4, and switches to SQLite's newer 'Autosetup' build system. It also tightens some compile-time options, such as disabling dynamic extensions and adding a debug flag in debug builds. There is no direct evidence in the commit that this fixes a specific security vulnerability, but routine dependency updates can include upstream security fixes.

AI review queuedguix: sqlite wants tclby fanquake · 286f3e49 · Oct 29, 2025 · 1 fileMessage 28 · OpaqueInformational 15Details
Commit message · fanquake

guix: sqlite wants tcl

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

This commit adds the Tcl programming language package to the Guix build environment used to create reproducible Bitcoin Core releases. It is a build-system dependency fix, not a change to Bitcoin's wallet, networking, consensus, or node logic. There is no indication this introduces a security vulnerability or fixes one.

AI review queuedrpc: add decoded tx details to gettransaction with extra wallet fieldsby Matthew Zipkin · 060bb555 · Oct 29, 2025 · 4 filesMessage 50 · ThinInformational 15Details
Commit message · Matthew Zipkin

rpc: add decoded tx details to gettransaction with extra wallet fields

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit is a documentation-only cleanup for Bitcoin Core's RPC help text. It makes the help description for the wallet's gettransaction RPC match the actual fields returned, and lets the shared DecodeTxDoc helper show an extra 'ischange' field only when used in wallet contexts. There is no code behavior change, no bug fix, and no security impact.