BC
← All projectsBitcoin Core

Bitcoin Core

The Bitcoin network's reference node and wallet implementation.

BitcoinSupply chainNormal
Repository coverage

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

260security candidates613second-pass queue2893AI analyses
182commits · 30 days
473commits · 60 days
1488commits · 180 days
2855commits · 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
969Strong · 80–100
1189Adequate · 60–79
689Thin · 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 Chow16452162065
MarcoFalke41021406074
Lőrinc18121177081
fanquake22919226057
Hennadii Stepanov21214208063
rkrux57957074
Sjors Provoost89889074
merge-script42819088
Sebastian Falbesoner33733073
David Gumberg55655072
Pieter Wuille95595066
Analysis record

Published AI watches

Last scanned 33 minutes ago

Low 26 AI analysisMessage 96 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

Merge bitcoin/bitcoin#32784: wallet: derivehdkey RPC to get xpub at arbitrary path

This commit adds a new Bitcoin Core wallet RPC called derivehdkey that lets a user derive an extended public key (xpub) — or optionally the matching extended private key (xprv) — at a chosen BIP32 path from one of the wallet's own HD keys.…

New RPC exposes xpub/xprv derivation from wallet HD keysRequires at least one hardened derivation stepRejects watch-only wallets and locked/encrypted wallets
a8b582ecby Ava Chow+633−15819 files
No security note in commit
Informational 15 AI analysisMessage 81 · Strong
BC Bitcoin CoreBitcoin Core BitcoinSupply chain

Merge bitcoin/bitcoin#35852: scripted-diff: Use inline const(expr) over static constexpr in headers

This is a large but purely mechanical code cleanup in Bitcoin Core. It changes how constants are declared in header files from older C++ styles (static const, static constexpr) to the modern C++17 inline constexpr/inline const form. The pu…

dec68f99by merge-script+440−440102 files
No security note in commit
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
Repository ledger

Explore captured commits

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

Lower-priorityci: port lintby will · bc41848d · Sep 1, 2025 · 1 fileMessage 55 · ThinInformational 15Details
Commit message · will

ci: port lint

Co-authored-by: Max Edwards <youwontforgetthis@gmail.com>

55/100 · ThinMessage clarity
✓ Subject identifies a change✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit adds a new automated linting job to Bitcoin Core's GitHub Actions CI workflow. Linting is a routine code-quality check that flags style issues and common mistakes; it does not change how the Bitcoin software itself runs or handle user funds, network traffic, or secrets.

Lower-priorityci: port msan-dependsby will · d290a8e6 · Sep 1, 2025 · 1 fileMessage 55 · ThinInformational 15Details
Commit message · will

ci: port msan-depends

Co-authored-by: Max Edwards <youwontforgetthis@gmail.com>

55/100 · ThinMessage clarity
✓ Subject identifies a change✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit only adds a new MemorySanitizer (MSan) CI job to the GitHub Actions workflow file. It does not change any Bitcoin Core source code, runtime behavior, or network protocol. There is no security vulnerability here.

Lower-priorityci: port tsan-dependsby will · 9bbae61e · Sep 1, 2025 · 1 fileMessage 55 · ThinInformational 15Details
Commit message · will

ci: port tsan-depends

Co-authored-by: Max Edwards <youwontforgetthis@gmail.com>

55/100 · ThinMessage clarity
✓ Subject identifies a change✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit adds a new continuous integration (CI) test job to Bitcoin Core's GitHub Actions workflow. It enables ThreadSanitizer (TSan) testing with the depends build system and disables the GUI. It also adds a system setting tweak (vm.mmap_rnd_bits=28) specifically for that TSan job to prevent crashes caused by high address space layout randomization (ASLR) entropy. This is a testing infrastructure change, not a change to Bitcoin Core's runtime code, consensus rules, wallet, networking, or any user-facing functionality.

Lower-priorityci: port tidyby will · bf7d5364 · Sep 1, 2025 · 1 fileMessage 55 · ThinInformational 15Details
Commit message · will

ci: port tidy

Co-authored-by: Max Edwards <youwontforgetthis@gmail.com>

55/100 · ThinMessage clarity
✓ Subject identifies a change✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit adds a new automated code-quality check called 'tidy' to Bitcoin Core's continuous integration (CI) pipeline. It does not change any Bitcoin software code, user-facing behavior, or network protocol. It only configures GitHub Actions to run an existing cleanup/linting script in a specific Ubuntu environment. There is no security vulnerability here.

Lower-priorityci: port centos-depends-guiby will · 549074bc · Sep 1, 2025 · 1 fileMessage 72 · AdequateInformational 15Details
Commit message · will

ci: port centos-depends-gui

Co-authored-by: Max Edwards <youwontforgetthis@gmail.com>

72/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit adds a new continuous integration (CI) test job for Bitcoin Core. It does not change any wallet, networking, consensus, or node code that end users run. It only tells the project's automated testing system to also run tests on CentOS with the GUI build. There is no security issue visible in this change.

Lower-priorityci: port previous-releases-depends-debugby will · 58e38c3a · Sep 1, 2025 · 1 fileMessage 72 · AdequateInformational 15Details
Commit message · will

ci: port previous-releases-depends-debug

Co-authored-by: Max Edwards <youwontforgetthis@gmail.com>

72/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit adds a new continuous integration (CI) test job to Bitcoin Core's GitHub Actions workflow. It does not change any production code, wallet logic, networking, or consensus rules. It simply tells the project's automated testing system to also run an existing test configuration ('previous releases, depends DEBUG') on GitHub-hosted runners in addition to wherever it ran before.

Lower-priorityci: port fuzzer-address-undefined-integer-nodependsby will · 341196d7 · Sep 1, 2025 · 1 fileMessage 77 · AdequateInformational 15Details
Commit message · will

ci: port fuzzer-address-undefined-integer-nodepends

Co-authored-by: Max Edwards <youwontforgetthis@gmail.com>

77/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
fuzzing or regression evidencedocumentation-only discount
AI analysis · Informational 15/100

This commit adds a new continuous-integration (CI) job to Bitcoin Core's GitHub Actions workflow. It runs the project's fuzz tests with AddressSanitizer, UndefinedBehaviorSanitizer, and IntegerSanitizer enabled, using the existing native fuzzing environment. There is no code change to Bitcoin Core itself, no bug fix, and no security patch.

Lower-priorityci: port no-IPC-i686-DEBUGby will · f2068f26 · Sep 1, 2025 · 1 fileMessage 72 · AdequateInformational 15Details
Commit message · will

ci: port no-IPC-i686-DEBUG

Co-authored-by: Max Edwards <youwontforgetthis@gmail.com>

72/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit adds a new continuous integration (CI) test job to Bitcoin Core's GitHub Actions workflow. It runs the existing test suite in a specific configuration: 32-bit i686 architecture, DEBUG build, and without inter-process communication (IPC). There is no code change to Bitcoin Core itself, only an addition to the CI configuration file.

Lower-priorityci: port nowallet-libbitcoinkernelby will · 2a00b12d · Sep 1, 2025 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · will

ci: port nowallet-libbitcoinkernel

Co-authored-by: Max Edwards <youwontforgetthis@gmail.com>

62/100 · AdequateMessage clarity
✓ Descriptive subject✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit adds a new automated testing job to Bitcoin Core's GitHub Actions CI pipeline. It does not change any wallet, kernel, or consensus code, nor does it alter how the software behaves on users' machines. It simply tells the project's continuous integration system to run an existing test configuration ('No wallet, libbitcoinkernel') on every relevant build.

Lower-priorityci: port mac-cross-gui-notestsby will · 9c2514de · Sep 1, 2025 · 1 fileMessage 72 · AdequateInformational 15Details
Commit message · will

ci: port mac-cross-gui-notests

Co-authored-by: Max Edwards <youwontforgetthis@gmail.com>

72/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit adds a new GitHub Actions CI job that cross-compiles the Bitcoin Core graphical user interface for macOS without running tests. It is purely a build-pipeline configuration change and does not modify any production code, network behavior, or wallet logic.

Lower-priorityci: force reinstall of kernel headers in asanby will · 2c990d84 · Sep 1, 2025 · 1 fileMessage 80 · StrongInformational 15Details
Commit message · will

ci: force reinstall of kernel headers in asan

When using hosted runners in combination with cached docker images,
there is the possibility that the host runner image is updated,
rendering the linux-headers package (stored in the cached docker image)
incompatible.

Fix this by doing a re-install of the headers package in
03_test_script.sh.

If the underlying runner kernel has not changed thie has no effect, but
prevents the job from failing if it has.

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 changes a Bitcoin Core continuous integration (CI) test script. It forces a re-install of Linux kernel headers when running an AddressSanitizer (ASan) test in a Docker container. The goal is to avoid CI failures when the host runner's Linux kernel is updated but a cached Docker image still contains older kernel headers. This is a build/test infrastructure fix, not a change to the Bitcoin software users run.

Lower-priorityci: update asan-lsan-ubsanby will · 88425144 · Sep 1, 2025 · 1 fileMessage 72 · AdequateInformational 15Details
Commit message · will

ci: update asan-lsan-ubsan

Co-authored-by: Max Edwards <youwontforgetthis@gmail.com>

72/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit is a routine internal change to Bitcoin Core's continuous integration (CI) setup. It removes a standalone GitHub Actions job for running sanitizers (ASan, LSan, UBSan) and merges that same job into a shared CI matrix. There is no change to the actual Bitcoin node software, wallet code, or network behavior, and nothing in the commit suggests a security fix or vulnerability.

Lower-priorityci: port arm 32-bit jobby will · f253031c · Sep 1, 2025 · 1 fileMessage 65 · AdequateInformational 15Details
Commit message · will

ci: port arm 32-bit job

Co-authored-by: Max Edwards <youwontforgetthis@gmail.com>

65/100 · AdequateMessage clarity
✓ Subject identifies a change✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit adds a new automated testing job for 32-bit ARM processors in Bitcoin Core's GitHub Actions CI workflow. It does not change any wallet, networking, consensus, or node code that end users run. It only affects internal continuous integration (CI) infrastructure.

AI review queuedfuzz: MockMempoolMinFee in wallet_feesby brunoerg · 5ded99a7 · Sep 1, 2025 · 1 fileMessage 45 · ThinInformational 15Details
Commit message · brunoerg

fuzz: MockMempoolMinFee in wallet_fees

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
fuzzing or regression evidencesigning or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit only changes a fuzz test file used for automated testing of Bitcoin Core's wallet fee logic. It adds a helper that simulates a higher minimum mempool fee during the test so the fuzzer can exercise more code paths. There is no change to production wallet or mempool code, and no security issue is present in the diff.

Lower-prioritytest: move MockMempoolMinFee to util/txmempoolby brunoerg · c9a7a198 · Sep 1, 2025 · 5 filesMessage 67 · AdequateInformational 15Details
Commit message · brunoerg

test: move MockMempoolMinFee to util/txmempool

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 harmless code cleanup in Bitcoin Core's test suite. A helper function used only by tests is being moved from one internal test file to another so it can be reused more easily. It does not change how the live Bitcoin network works and is not a security fix.

AI review queuedfuzz: create FeeEstimatorTestingSetup to set fee_estimatorby brunoerg · adf67eb2 · Sep 1, 2025 · 1 fileMessage 60 · AdequateInformational 15Details
Commit message · brunoerg

fuzz: create FeeEstimatorTestingSetup to set fee_estimator

60/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
fuzzing or regression evidencesigning or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit is a small cleanup inside a fuzz test (automated randomized testing) for Bitcoin Core's wallet fee logic. It creates a dedicated test helper class so the fuzz harness can safely set and reset the fee estimator. There is no change to production code, no user-facing behavior change, and no security fix.

AI review queuedfuzz: mock CBlockPolicyEstimator in wallet_fuzzby brunoerg · ff10a37e · Sep 1, 2025 · 1 fileMessage 55 · ThinInformational 15Details
Commit message · brunoerg

fuzz: mock CBlockPolicyEstimator in wallet_fuzz

55/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
fuzzing or regression evidencesigning or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit only changes a fuzz test file. It introduces a fake/mock version of Bitcoin Core's fee estimator so the wallet fuzz test can control its outputs. There is no change to production code, no user-facing behavior change, and no security fix or vulnerability.

Lower-priorityci: update windows-cross jobby will · 04e7bfbc · Sep 1, 2025 · 1 fileMessage 72 · AdequateInformational 15Details
Commit message · will

ci: update windows-cross job

Co-authored-by: Max Edwards <youwontforgetthis@gmail.com>

72/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit is a routine update to Bitcoin Core's automated testing configuration for building Windows software from a Linux machine. It swaps the virtual machine image used by the CI job and replaces several inline setup steps with reusable internal actions. There is no change to the Bitcoin software itself, no user-facing change, and no security fix or vulnerability.

Lower-priorityci: add job to determine runner typeby will · cc1735d7 · Sep 1, 2025 · 1 fileMessage 80 · StrongInformational 15Details
Commit message · will

ci: add job to determine runner type

To remove multiple occurances of the respository name, against which we
compare `${{ github.repository }}` to check if we should use Cirrus
Runners, introduce a helper job which can check a single environment
variable and output this as an input to subsequent jobs.

Forks can maintain a trivial patch of their repo name against the
`REPO_USE_CIRRUS_RUNNERS` variable in ci.yml if they have Cirrus Runners
of their own, which will then enable cache actions and docker build
cache to use Cirrus Cache.

It's not possible to use `${{ env.USE_CIRRUS_RUNNERS }}` in the
`runs-on:` directive as the context is not supported by GitHub.

If it was, this job would no longer be necessary.

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 discount
AI analysis · Informational 15/100

This commit is a routine cleanup of the project's GitHub Actions CI configuration. It introduces a small helper job that decides whether to use Cirrus-hosted runners or GitHub-hosted runners based on a repository name comparison. There is no security-relevant change; it only reorganizes existing logic to reduce repetition.

Lower-priorityci: add Cirrus cache hostby will · 020069e6 · Sep 1, 2025 · 1 fileMessage 80 · StrongInformational 15Details
Commit message · will

ci: add Cirrus cache host

Whilst the action cirruslabs/actions/cache will automatically set this
host, the docker `gha` build cache backend will not be aware of it.

Set the value here, which will later be used in the docker build args to
enable docker build cache on the cirrus cache.

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 discount
AI analysis · Informational 15/100

This commit adds a single environment variable to the project's GitHub Actions CI workflow. It tells Docker where to find a local Cirrus CI build cache server (127.0.0.1:12321). There is no security-relevant change: it does not introduce secrets, change permissions, modify network exposure, or alter how Bitcoin Core itself runs.

Lower-priorityci: have base install run in right dirby will · 9c2b96e0 · Sep 1, 2025 · 2 filesMessage 92 · StrongInformational 15Details
Commit message · will

ci: have base install run in right dir

This sets the build dir at build time so that Apple SDK gets installed
in the correct/expected location for the runtime to find it.

Co-authored-by: Max Edwards <youwontforgetthis@gmail.com>

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

This is a routine continuous integration (CI) build-script fix. It makes sure a build directory variable is passed into the container image build so that an Apple software development kit gets installed where the test runner expects it. There is no indication this affects Bitcoin Core software users, wallets, the network, or security.

Lower-priorityci: use docker build cache arg directlyby will · 18f6be09 · Sep 1, 2025 · 1 fileMessage 88 · StrongInformational 15Details
Commit message · will

ci: use docker build cache arg directly

Reverts: e87429a2d0f23eb59526d335844fa5ff5b50b21f

This was added in PR #31545 with the intention that self-hosted runners
might use it to save build cache.

As we are not using hosted runners with a registry build cache, the bulk
of this commit can be reverted, simply using the value of
$DOCKER_BUILD_CACHE_ARG in the script.

link: https://github.com/bitcoin/bitcoin/pull/31545

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

This commit removes a CI (Continuous Integration) shell script feature that managed a local Docker build cache directory. It is a cleanup/revert of an earlier change and does not touch any Bitcoin Core node, wallet, consensus, or network code. There is no security-relevant change to user-facing software.

Lower-priorityci: use buildx in ciby will · 94a09325 · Sep 1, 2025 · 1 fileMessage 73 · AdequateInformational 15Details
Commit message · will

ci: use buildx in ci

Using buildx is required to properly load the correct driver, for use
with registry caching. Neither build, nor BUILDKIT=1 currently do this
properly.

Use of `docker buildx build` is compatible with podman.

73/100 · AdequateMessage clarity
✓ Subject identifies a change✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
AI analysis · Informational 15/100

This commit changes the Bitcoin Core continuous integration (CI) script to use Docker's 'buildx' plugin instead of the older 'docker build' command with BuildKit enabled. It is a build-infrastructure tweak intended to make Docker image caching work correctly. There is no indication this change affects the Bitcoin software users run, handles money, keys, network messages, or introduces any security vulnerability.

Lower-priorityci: add configure-docker actionby will · fdf64e55 · Sep 1, 2025 · 1 fileMessage 80 · StrongInformational 15Details
Commit message · will

ci: add configure-docker action

Another action to reduce boilerplate in the main ci.yml file.

This action will set up a docker builder compatible with caching build
layers to a container registry using the `gha` build driver.

It will then configure the docker build cache args.

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 discount
AI analysis · Informational 15/100

This commit adds a new GitHub Actions helper that sets up Docker Buildx and configures Docker layer caching for Bitcoin Core's continuous integration (CI). It does not change any Bitcoin node code, wallet logic, or network behavior. There is no indication it fixes or introduces a security vulnerability.

Lower-priorityci: add REPO_USE_CIRRUS_RUNNERSby will · 33ba073d · Sep 1, 2025 · 1 fileMessage 70 · AdequateInformational 15Details
Commit message · will

ci: add REPO_USE_CIRRUS_RUNNERS

If set, Cirrus runners will be used on pushes to, and pull requests
against, this repository.

Forks can set this if they have their own cirrus runners.

70/100 · AdequateMessage clarity
✓ Descriptive subject✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit is a routine CI (Continuous Integration) configuration change for the Bitcoin Core repository. It adds a single environment variable that tells the project's GitHub Actions workflow to use faster Cirrus-provided runners and caching when running tests on the main bitcoin/bitcoin repository or pull requests against it. There is no code change affecting the Bitcoin software itself, wallets, transactions, networking, or cryptography.