BC
← All projectsBitcoin Core

Bitcoin Core

The Bitcoin network's reference node and wallet implementation.

BitcoinSupply chainNormal
Repository coverage

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

257security candidates612second-pass queue2889AI analyses
191commits · 30 days
474commits · 60 days
1492commits · 180 days
2879commits · 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
952Strong · 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 Chow16050159064
MarcoFalke41021406074
Lőrinc17721177081
fanquake22819226057
Hennadii Stepanov21114208063
rkrux57957074
Sjors Provoost89889074
merge-script33718088
Sebastian Falbesoner33733073
David Gumberg55655072
Pieter Wuille95595066
Analysis record

Published AI watches

Last scanned 9 minutes ago

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
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
Repository ledger

Explore captured commits

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

Lower-prioritydepends: Drop redundant check for downloaded fileby Hennadii Stepanov · 46135d90 · Sep 30, 2025 · 1 fileMessage 78 · AdequateInformational 15Details
Commit message · Hennadii Stepanov

depends: Drop redundant check for downloaded file

The `fetch_file` commands are invoked for the `$($(package)_fetched)`
target, so the existence of the download stamp has already been tested.

78/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification
AI analysis · Informational 15/100

This is a minor cleanup in Bitcoin Core's dependency build system. It removes an extra file-existence check that was already being handled elsewhere. There is no security issue visible in the change.

Lower-prioritydepends: Fix `$(package)_fetched` targetby Hennadii Stepanov · 77197895 · Sep 30, 2025 · 1 fileMessage 60 · AdequateInformational 24Details
Commit message · Hennadii Stepanov

depends: Fix `$(package)_fetched` target

Ensure the download timestamp is created only after a successful
download.

60/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
AI analysis · Informational 24/100

This commit fixes a build-system bug in Bitcoin Core's dependency downloader. Previously, the build system would create a 'download finished' timestamp file before the download actually completed. If the download was interrupted or failed, that timestamp file could still exist, making the build system think the dependency was already downloaded and skip retrying. The fix moves the timestamp creation to after the download and checksum steps succeed. This is primarily a reliability/build correctness issue; direct security impact is limited and indirect.

Lower-priorityMempool: Do not enforce TRUC checks on reorgby Greg Sanders · 26e71c23 · Sep 29, 2025 · 1 fileMessage 68 · AdequateLow 44Details
Commit message · Greg Sanders

Mempool: Do not enforce TRUC checks on reorg

Not enforcing TRUC topology on reorg was the intended
behavior, but the appropriate bypass argument was not
checked.

This mistake means we could potentially invalidate a long
chain of perfectly incentive-compatible transactions that
were made historically, including subsequent non-TRUC
transactions, all of which may have been very high feerate.

Lastly, it wastes CPU cycles doing topology checks since
this behavior cannot actually enforce the topology in
general for the reorg setting.

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Why it was queued
defensive validation
AI analysis · Low 44/100

This commit fixes a bug in Bitcoin Core's transaction memory pool (mempool) handling during blockchain reorganizations (reorgs). A reorg happens when the network briefly has two competing versions of recent history and one wins. The bug caused the mempool to wrongly enforce new TRUC topology rules on old transactions being restored after a reorg. That could reject a chain of historically valid, high-fee transactions and waste CPU checking rules that were never meant to apply in that situation. The fix adds a bypass flag so those checks are skipped during reorg recovery, matching the original design intent.

Lower-prioritytest: add more TRUC reorg covergeby Greg Sanders · 06df14ba · Sep 29, 2025 · 1 fileMessage 67 · AdequateInformational 15Details
Commit message · Greg Sanders

test: add more TRUC reorg coverge

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 and improves test coverage in Bitcoin Core's functional test suite. It changes one test file (mempool_truc.py) to more thoroughly exercise how TRUC (a new transaction relay policy) behaves during blockchain reorganizations. There is no change to production code, consensus rules, or network behavior, so it does not introduce or fix a security vulnerability on its own.

Lower-priorityfuzz: don't bypass_limits for most mempool harnessesby Greg Sanders · bbe8e906 · Sep 29, 2025 · 2 filesMessage 83 · StrongInformational 16Details
Commit message · Greg Sanders

fuzz: don't bypass_limits for most mempool harnesses

Using bypass_limits=true is essentially fuzzing part of a
reorg only, and results in TRUC invariants unable to be
checked. Remove most instances of bypassing limits, leaving
one harness able to do so.

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 evidence
AI analysis · Informational 16/100

This change only modifies Bitcoin Core's internal fuzz testing code, not the production mempool logic. It adjusts how test harnesses feed random data into the mempool acceptance routine so that most harnesses no longer skip standard policy limits. The patch improves test coverage for a new transaction type (TRUC) but does not fix a security bug in live software.

Lower-priorityci: use latest versions of lint depsby fanquake · d4f47f97 · Sep 29, 2025 · 1 fileMessage 80 · StrongInformational 15Details
Commit message · fanquake

ci: use latest versions of lint deps

Some of the versions used here are > 2 years old. i.e mypy. Use the
latest avilable versions, except for LIEF, which is generally changed
with Guix.

80/100 · StrongMessage clarity
✓ 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 simply updates the versions of several linting tools used in Bitcoin Core's continuous integration (CI) pipeline. Linting tools check code style and catch common mistakes but are not part of the actual Bitcoin software that users run. There is no security-relevant code change here.

Lower-prioritytest: Avoid shutdown race in NetworkThreadby MarcoFalke · fa6db793 · Sep 29, 2025 · 1 fileMessage 67 · AdequateInformational 16Details
Commit message · MarcoFalke

test: Avoid shutdown race in NetworkThread

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

This is a one-line fix in Bitcoin Core's internal Python test framework. It adds a wait to ensure a background network thread has fully started before tests continue. The change only affects test code, not the live Bitcoin node software that users run, so it cannot be used to attack real Bitcoin wallets or the network. It addresses a timing-related test flakiness issue, not a security vulnerability in production code.

Lower-prioritynet: merge AlreadyConnectedToAddress() and FindNode(CNetAddr)by Vasil Dimov · 3a4d1a25 · Sep 29, 2025 · 4 filesMessage 95 · StrongInformational 15Details
Commit message · Vasil Dimov

net: merge AlreadyConnectedToAddress() and FindNode(CNetAddr)

`CConnman::AlreadyConnectedToAddress()` is the only caller of
`CConnman::FindNode(CNetAddr)`, so merge the two in one function.

The unit test that checked whether `AlreadyConnectedToAddress()` ignores
the port is now unnecessary because now the function takes a `CNetAddr`
argument. It has no access to the port.

95/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode✓ Mentions testing or verification
AI analysis · Informational 15/100

This is a routine code cleanup in Bitcoin Core's networking layer. It merges two internal functions that check whether the node is already connected to a given internet address, and removes a now-redundant unit test. There is no security-relevant change: the function still does the same basic check, just with slightly simpler code.

Lower-priorityci: Reduce Alpine musl task to md runner sizeby MarcoFalke · 444409ff · Sep 29, 2025 · 1 fileMessage 80 · StrongInformational 15Details
Commit message · MarcoFalke

ci: Reduce Alpine musl task to md runner size

Per https://github.com/bitcoin/bitcoin/pull/33480#discussion_r2387381492

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

This commit simply changes the size of a virtual machine used for one automated testing job in Bitcoin Core's GitHub Actions setup, switching from a large to a medium-sized runner. It has no effect on the Bitcoin software users run, on wallets, transactions, consensus rules, or network security.

Lower-prioritytest: set par=2 in default config for functional test frameworkby Andrew Toth · dda5228e · Sep 27, 2025 · 1 fileMessage 95 · StrongInformational 15Details
Commit message · Andrew Toth

test: set par=2 in default config for functional test framework

Depending on the host machine, a default `par` value can spawn up to 15 script verification threads for each node.
Running the functional test suite with default `par` can exhaust file descriptors or hit other resource limits when many threads are spawned.
These threads are mostly idle and the same code paths are executed with a value of `par=2`.
Limit this to 2 for functional tests that do not override the default option.

Co-authored-by: maflcko <6399679+maflcko@users.noreply.github.com>

95/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Mentions testing or verification
Why it was queued
defensive validation
AI analysis · Informational 15/100

This change only affects Bitcoin Core's internal functional test framework. It limits the number of script verification threads used during automated tests to avoid running out of file descriptors or hitting other resource limits on the test machine. It does not change production Bitcoin node behavior and is not a security fix for live software.

Lower-priorityqa: Only complain about expected messages that were not foundby Hodlinator · a1f76230 · Sep 26, 2025 · 1 fileMessage 73 · AdequateInformational 15Details
Commit message · Hodlinator

qa: Only complain about expected messages that were not found

Only leaves messages we never found in the final assert message of the functions, which is more helpful (remaining_expected).

Avoids repeatedly searching for messages we have already found (pop()).

Stops searching for other expected messages if we already failed finding one. Still need to clean remaining_expected at the end, but *only if we fail*.

Co-authored-by: Lőrinc <pap.lorinc@gmail.com>

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 15/100

This is a small quality-of-life improvement to Bitcoin Core's internal testing tools. It changes how test error messages are printed so that only the log messages that were actually missing are reported, rather than the whole list. It also avoids rechecking messages that were already found. There is no change to the live Bitcoin network code, wallet, consensus rules, or any user-facing behavior.

Lower-priorityqa: Replace always-escaped regexps with "X in Y"by Hodlinator · a9021101 · Sep 26, 2025 · 1 fileMessage 68 · AdequateInformational 15Details
Commit message · Hodlinator

qa: Replace always-escaped regexps with "X in Y"

Always escaping the search string makes the use of regular expressions unnecessary.

Co-authored-by: Lőrinc <pap.lorinc@gmail.com>

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

This is a small code-quality cleanup in Bitcoin Core's test framework. It replaces regular-expression searches with simple string searches because the search strings were always being escaped anyway, making regex unnecessary. There is no security impact.

Lower-priorityrefactor(qa): Avoid unnecessary string operationsby Hodlinator · 1e54125e · Sep 26, 2025 · 1 fileMessage 80 · StrongInformational 15Details
Commit message · Hodlinator

refactor(qa): Avoid unnecessary string operations

print_log was recalculated every 0.05s in assert_debug_log(), even during successful circumstances - changed to only be computed upon failure.

Simplified terminology from "(does not) partially match(es)" to "(not) found in" since it seems to reference the first function having used regular expression matching, while it always escaped the search strings (see parent commit). (Simplified grammar also avoids issues with singular/plural "was/were not found").

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

This is a minor internal cleanup in Bitcoin Core's automated testing code. It changes how debug log messages are formatted only when a test fails, and tweaks the wording of error messages. There is no security issue.

Lower-prioritydoc: Remove no longer correct commentby Hodlinator · 5c16e463 · Sep 26, 2025 · 1 fileMessage 60 · AdequateInformational 15Details
Commit message · Hodlinator

doc: Remove no longer correct comment

Left over since fa7b07571f24b6def6effdd4cc1b96c7507bf959.

Co-authored-by: Lőrinc <pap.lorinc@gmail.com>

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

This commit only removes a two-line comment in a test helper file because the comment no longer matched what the function actually does. No code behavior was changed, so there is no security impact.

Security candidatedoc: rpc: fix case typo in `finalizepsbt` help (final_scriptwitness)by Sebastian Falbesoner · ff05bebc · Sep 26, 2025 · 1 fileMessage 50 · ThinInformational 15Details
Commit message · Sebastian Falbesoner

doc: rpc: fix case typo in `finalizepsbt` help (final_scriptwitness)

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

This commit fixes a capitalization typo in the help text for the `finalizepsbt` RPC command. The documentation previously referred to `final_scriptWitness` but the actual PSBT field name is `final_scriptwitness` (all lowercase). This is purely a documentation string change and has no effect on code behavior, transaction processing, or security.

Lower-prioritycontrib: fix using macdploy script without translations.by amisha · 7b5261f7 · Sep 26, 2025 · 1 fileMessage 73 · AdequateInformational 16Details
Commit message · amisha

contrib: fix using macdploy script without translations.

QT translations are optional, but the script would error when
'translations_dir' falls back to its default value NULL.

This PR fixes it by moving the set-up of QT translations under
the check for 'translations_dir' presence.

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 16/100

This is a build-script bug fix. The macOS packaging helper would crash with an error if no Qt translation directory was provided, because it tried to use a NULL/default value. The fix simply skips the translation-copying step when no directory is configured. It does not affect Bitcoin's network code, wallet security, or consensus rules.

AI review queuedci: remove 3rd party js from windows dll gha jobby Max Edwards · 7ae0497e · Sep 25, 2025 · 1 fileMessage 80 · StrongInformational 17Details
Commit message · Max Edwards

ci: remove 3rd party js from windows dll gha job

We can use vswhere.exe directly to create a vs developer
prompt and so can remove this third party dependency.

Co-authored-by: David Gumberg <davidzgumberg@gmail.com>

80/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
Why it was queued
documentation-only discountsecond-pass: broader security terminology
AI analysis · Informational 17/100

This change updates a Bitcoin Core GitHub Actions workflow so that it no longer downloads and runs a third-party JavaScript action (ilammy/msvc-dev-cmd) to set up the Microsoft Visual Studio build environment on Windows. Instead, it uses a small PowerShell script that runs a tool already installed with Visual Studio (vswhere.exe) and then calls the official Microsoft batch file (vsdevcmd.bat). The main benefit is reducing supply-chain risk: the project no longer depends on an external, pinned-but-third-party action that could be compromised or behave unexpectedly. There is no direct vulnerability being fixed in Bitcoin Core's own code.

Security candidateci: Turn centos config into alpine musl configby MarcoFalke · fa6b2e9e · Sep 25, 2025 · 5 filesMessage 57 · ThinInformational 15Details
Commit message · MarcoFalke

ci: Turn centos config into alpine musl config

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 swaps one continuous-integration (CI) test runner from CentOS to Alpine Linux (musl). It only changes build/test automation files, not the Bitcoin Core software that users run. There is no security-relevant code change here.

Lower-prioritylog: reduce excessive messages during block replayby Lőrinc · 1fc7a81f · Sep 25, 2025 · 1 fileMessage 73 · AdequateInformational 15Details
Commit message · Lőrinc

log: reduce excessive messages during block replay

After an incomplete reindex the blocks will need to be replayed.
This results in excessive `Rolling back` and `Rolling forward` messages which quickly triggers the recently introduced log rate limiter.

Change the logging strategy to:
- Add single `LogInfo` messages showing the full range being replayed for both rollback and roll forward;
- Log progress at `LogInfo` level only every 10,000 blocks to track the long operations.

Reproducer:
* Start a normal IBD, stop after some progress
* Do a reindex, stop before it finishes
* Restart the node normally without specifying the reindex parameter
It should start rolling the blocks forward.

Before this change the excessive logging would show:
```
[*] Rolling forward 000000002f4f55aecfccc911076dc3f73ac0288c83dc1d79db0a026441031d40 (46245)
[*] Rolling forward 0000000017ffcf34c8eac010c529670ba6745ea59cf1edf7b820928e3b40acf6 (46246)
```

After the change it shows:
```
Replaying blocks
Rolling forward to 00000000000000001034012d7e4facaf16ca747ea94b8ea66743086cfe298ef8 (326223 to 340991)
Rolling forward 00000000000000000faabab19f17c0178c754dbed023e6c871dcaf74159c5f02 (330000)
Rolling forward 00000000000000000d9b2508615d569e18f00c034d71474fc44a43af8d4a5003 (340000)
...
Rolled forward to 00000000000000001034012d7e4facaf16ca747ea94b8ea66743086cfe298ef8
```

(similarly to rolling back)

Co-authored-by: Anthony Towns <aj@erisian.com.au>
Co-authored-by: Vasil Dimov <vd@freebsd.org>

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 15/100

This commit only changes how many log messages Bitcoin Core prints while replaying blocks after an interrupted reindex. It reduces noisy 'Rolling back/forward' lines to avoid triggering a new log rate limiter. There is no security bug being fixed here.

Security candidateminer: fix `addPackageTxs` unsigned integer overflowby ismaelsadeeq · b807dfcd · Sep 24, 2025 · 1 fileMessage 55 · ThinLow 33Details
Commit message · ismaelsadeeq

miner: fix `addPackageTxs` unsigned integer overflow

55/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Names security-relevant behavior explicitly! No meaningful explanatory body
Why it was queued
memory safety
AI analysis · Low 33/100

This commit fixes a math bug in the Bitcoin Core block-building code. The original code subtracted a safety margin from the maximum block weight and then compared the current block weight to that lower number. Because the variables are unsigned integers, if the block weight was already larger than expected, the subtraction could underflow and wrap around to a huge number, making the comparison behave incorrectly. The fix rewrites the comparison as an addition on the other side, which cannot underflow. The practical effect is on when the miner decides to stop trying to add more transactions; the bug could make it give up too early or keep going too long in edge cases.

AI review queuedtest: Remove convert_to_json_for_cliby Ava Chow · df67bb6f · Sep 23, 2025 · 6 filesMessage 57 · ThinInformational 15Details
Commit message · Ava Chow

test: Remove convert_to_json_for_cli

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Uses a recognizable type or scope✓ Mentions testing or verification! 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 routine cleanup of Bitcoin Core's functional test code. It removes a helper method called convert_to_json_for_cli that was used to wrap arguments when tests were run against the command-line bitcoin-cli tool. The change only affects internal test scripts and has no impact on the live Bitcoin network, wallets, or node security.

Lower-prioritycli: Allow arguments to be both strings and jsonby Ava Chow · 44a493e1 · Sep 23, 2025 · 2 filesMessage 45 · ThinInformational 23Details
Commit message · Ava Chow

cli: Allow arguments to be both strings and json

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

This commit changes how Bitcoin Core's command-line tool (bitcoin-cli) handles certain arguments. Previously, some RPC parameters like 'hash_or_height' and 'rollback' were always treated as JSON, meaning plain strings would be rejected with a JSON parsing error. Now, for a small set of parameters, the tool accepts either a valid JSON value or a plain string. This is a usability fix, not a security patch, and it does not change the Bitcoin node itself—only the client-side argument parser.

Lower-prioritynet: use generic network key for addrcacheby Martin Zumsande · 94db966a · Sep 23, 2025 · 7 filesMessage 80 · StrongLow 31Details
Commit message · Martin Zumsande

net: use generic network key for addrcache

The generic key can also be used in other places
where behavior between different network identities should
be uncorrelated to avoid fingerprinting.
This also changes RANDOMIZER_ID - since it is not
being persisted to disk, there are no compatibility issues.

80/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode
AI analysis · Low 31/100

This Bitcoin Core change tweaks how the node picks a secret 'network key' used when replying to peer address requests. Previously the key was computed fresh each time from the peer's network type and the local bind address. Now it is computed once when a connection is created and stored on the CNode object. The stated goal is to make it easier to reuse the same anti-fingerprinting key in other parts of the code later. It is a refactoring/hardening patch, not a fix for an active bug or exploit.

Lower-prioritydepends: static libxcb_cursorby fanquake · eca50854 · Sep 23, 2025 · 3 filesMessage 66 · AdequateInformational 20Details
Commit message · fanquake

depends: static libxcb_cursor

Modern Ubuntu isn't shipping with this library installed by default.
Staticly link it to remove the need for end-users to install it.

Closes #33432.

66/100 · AdequateMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
AI analysis · Informational 20/100

This change makes Bitcoin Core's graphical wallet link a small X11 cursor helper library statically instead of relying on the operating system to provide it. That removes a runtime installation step for users on recent Ubuntu and other Linux distributions. It is a build/packaging convenience change, not a fix for an exploitable security bug.

Lower-priorityci: link against -lstdc++ in native fuzz with msan jobby fanquake · b77137a5 · Sep 23, 2025 · 2 filesMessage 72 · AdequateInformational 15Details
Commit message · fanquake

ci: link against -lstdc++ in native fuzz with msan job

72/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
fuzzing or regression evidence
AI analysis · Informational 15/100

This commit changes Bitcoin Core's internal continuous integration (CI) build scripts for a memory-sanitizer fuzz-testing job. It switches from building a custom Clang compiler from source to using pre-packaged Ubuntu LLVM 21 packages, and adds a linker flag (-lstdc++) to work around packaging-related link issues. There is no change to the Bitcoin Core software that users run, no change to consensus or networking code, and no security vulnerability is being fixed.