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.
Message quality measures whether a commit identifies its scope, purpose, rationale, testing, and supporting references. It does not change the security-severity score.
This change fixes a Bitcoin Core wallet bug where the `importprunedfunds` RPC command could only re-import transactions that sent money to the wallet, not transactions that spent money from it. After this fix, both incoming and outgoing tr…
Logic bug in wallet transaction import scopeIncorrect balance possible after removing and re-importing spending transactionFix routes import through existing involvement check (IsMine + IsFromMe)
This commit adds a new Bitcoin Core wallet startup option called -maxfeerate. It lets users set a maximum fee rate (fee per unit of transaction size) that the wallet will allow when creating or broadcasting transactions. Previously, the wa…
New wallet startup option -maxfeerate to cap transaction fee rateNew transaction error type MAX_FEE_RATE_EXCEEDEDBroadcastTransaction now checks both max absolute fee and max fee rate
This Bitcoin Core update fixes a wallet-signing quirk. When a user chose the SIGHASH_SINGLE signature mode, an input that had no matching output index would sign essentially nothing meaningful. That signature could then stay valid even if …
Funds-redirection footgun from SIGHASH_SINGLE signatures with no committed outputInconsistent guard between SignTransaction and SignPSBTInput pathsFix centralizes the guard in the low-level signature creator to cover future signing paths
This change updates Bitcoin Core's I2P (Invisible Internet Project) privacy network settings to use newer, stronger encryption for the published 'leaseset' that describes how other peers can contact a node. The old setting included ElGamal…
Cryptographic algorithm update (ElGamal to MLKEM-768)Use of I2P 'legacy' encryption type removedConfiguration-only change in network privacy layer
This change fixes a labeling bug in Bitcoin Core's first-run disk-space warning. The estimate was stored in GiB (binary gigabytes, 1024-based) but displayed as GB (decimal gigabytes, 1000-based), and for pruned nodes it showed the full-cha…
This is a wallet bug, not a theft or remote-code bug. When a Bitcoin Core user turns on the optional 'avoidpartialspends' or 'avoid_reuse' setting, an output group rejected during coin selection could be counted twice as 'discarded.' That …
Logic error causing double-counting of discarded UTXO groupsCan trigger false 'insufficient funds' failure in coin selectionAffects avoidpartialspends / avoid_reuse wallets only
This is a documentation-only fix in a tutorial file. It changes two shell examples from using '>>' (append to file) to '>' (overwrite file). If a user followed the old instructions and ran the same command twice, the file would contain two…
No security signal: change is limited to documentationNo code changes to Bitcoin Core binaries, RPC, wallet, or consensus logicNo cryptographic, network, or privilege-boundary implications
This is a large internal code reorganization (refactor) in Bitcoin Core. It creates a new BlockTemplateManager class that takes over block-template creation, block submission, and tip-waiting helpers that were previously spread across seve…
Large refactor touching mining, RPC, interfaces, and test shutdown pathsNew object lifetime dependency: BlockTemplateManager holds references to mempool, chainman, and notifications; explicit reset ordering added in Shutdown/InitAndLoadChainstate/test setupsRemoval of early-init node.mining interface; BlockTemplateManager is now created after chainstate load, with a comment that it must exist before setChainstateLoaded(true) unblocks IPC waiters
This commit adds the first implementation of BIP352 (Silent Payments) to Bitcoin Core. Silent Payments are a new type of privacy-preserving Bitcoin address that lets someone receive payments without publicly revealing a fixed address. The …
New cryptographic feature implementation (BIP352 Silent Payments)Extensive use of secp256k1 silentpayments moduleInput public key extraction from P2PKH, P2WPKH, P2SH-P2WPKH, and P2TR inputs
This update fixes a wallet database loading bug where a damaged or tampered Bitcoin wallet file could cause the program to read past the end of a stored extended public key (xpub). The patch makes the loader check the stored xpub length be…
Out-of-bounds read in wallet descriptor cache deserializationASan container-overflow triggered by malformed on-disk recordMissing length validation between record size prefix and fixed-size decoder
This commit adds a new wallet RPC called listrawtransactions to Bitcoin Core. It is a feature addition that lets users list every transaction their wallet knows about, including internal transfers and consolidations that the existing listt…
No security-relevant bug fix or vulnerability patch is present in the diff.New RPC exposes additional wallet transaction metadata, but only to callers already authorized for wallet RPCs.Code is a refactor of existing gettransaction logic into shared helpers; no new cryptographic, network, or consensus code.
This Bitcoin Core update fixes several wallet bugs where a failed database write could leave a wallet in an inconsistent state. For example, encrypting a wallet or changing its passphrase could appear to succeed in memory while the change …
Atomicity fix for encryption state and descriptor key persistenceFailure to persist master key during encryption previously reported success in memoryPassphrase change could activate new passphrase only in memory
This commit only changes Bitcoin Core's internal functional test code. It replaces hard-coded test keys and addresses with ones generated from a new test helper class, and unifies how tests tell nodes not to create a default wallet. There …
This commit only adds a new automated test to Bitcoin Core. It checks that when two partially-signed Bitcoin transactions (PSBTs) are combined, any custom 'unknown' data fields attached to them are preserved correctly. There is no change t…
This is a Bitcoin Core wallet maintenance patch. It speeds up a wallet function that checks whether a descriptor already exists by caching a hash of the descriptor's canonical text, instead of rebuilding that text every time. It also tidie…
No security-relevant signal in commit message or diffChange is described as performance improvement and code cleanupBackwards-compatibility test notes a known miniscript wallet loading incompatibility between v31.0/v31.1 and other versions, but this is a documented compatibility quirk, not a vulnerability
This is a documentation-only fix for Bitcoin Core's machine-readable RPC help data. It changes several default values from literal strings to 'hint' labels (because the real default depends on context) and corrects one boolean default from…
OpenRPC schema/default mismatch correctionRPC help metadata type correction (string 'false' to boolean false)No executable code path changes
This commit fixes documentation metadata for six Bitcoin Core RPC arguments. It changes how default values are described so that automatically generated API docs and schemas are accurate. The actual behavior of the software when running is…
No runtime code changesOnly RPC help/schema metadata modifiedVendor explicitly states runtime behavior is unchanged
This commit fixes a bug in Bitcoin Core's MuHash3072 cryptographic code where dividing a MuHash object by itself (x /= x) produced the wrong mathematical result. The fix is straightforward: the code now saves the divisor's numerator before…
Cryptographic correctness bug in MuHash3072 division operatorSelf-aliasing in operator/= produces incorrect 1/D result instead of empty setNo production code path identified that triggers self-division
This is a build-compatibility fix, not a security patch. It changes how some constant data is stored internally so that Apple's macOS linker (ld64) can build Bitcoin Core correctly. The change avoids a linker bug that caused build failures…
No security-relevant code logic changedChange is a linker bug workaround, not a vulnerability fixConstants remain read-only; no new attack surface introduced
This commit refactors Bitcoin Core's wallet descriptor import feature so the same logic can be used by both the RPC command and a new GUI-facing interface. It also tightens one input rule: negative timestamps are now rejected, and the mini…
Refactor of security-sensitive wallet import code into shared CWallet pathNew input validation: negative timestamps rejected for importdescriptorsCentralization of descriptor range bound checks in CheckDescriptorRangeBounds
Variant resetting the other fields in Clear(): ₿ hyperfine --warmup 2 "./build/bin/test_bitcoin -t coinselector_tests/knapsack_solver_test" Benchmark 1: ./build/bin/test_bitcoin -t coinselector_tests/knapsack_solver_test Time (mean ± σ): 4.289 s ± 0.008 s [User: 4.252 s, System: 0.027 s] Range (min … max): 4.279 s … 4.301 s 10 runs
This variant: ₿ hyperfine --warmup 2 "./build/bin/test_bitcoin -t coinselector_tests/knapsack_solver_test" Benchmark 1: ./build/bin/test_bitcoin -t coinselector_tests/knapsack_solver_test Time (mean ± σ): 4.279 s ± 0.005 s [User: 4.244 s, System: 0.026 s] Range (min … max): 4.271 s … 4.287 s 10 runs
66/100 · AdequateMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Low 29/100
This Bitcoin Core commit removes a custom `Clear()` method from a wallet coin-selection helper struct and replaces its uses with a full default reset (`available_coins = {}`). The stated reason is to avoid accidentally leaving fields partially reset in the future. The change is defensive code cleanup; there is no direct evidence in the commit or supplied references that it fixes an active security vulnerability or exploitable bug.
p2p: Saturate LocalServiceInfo::nScore updates at INT_MAX
Signed overflow on nScore updates is undefined behavior. Use SaturatingAdd in AddLocal() and SeenLocal() so increments saturate at INT_MAX instead of overflowing.
Add unit test coverage for saturation in both code paths.
88/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification✓ Names security-relevant behavior explicitly
Why it was queued
memory safety
AI analysis · Low 37/100
This commit fixes a subtle programming bug in Bitcoin Core's peer-to-peer networking code. A local network address score counter could, in theory, overflow when repeatedly incremented, which is 'undefined behavior' in C++ and could potentially cause the program to misbehave or crash. The fix makes the counter stop increasing once it reaches the maximum integer value, and adds tests to confirm that behavior.
Lower-priorityci: use Warp cache for Docker layersby will · 82901981 · Jun 3, 2026 · 2 filesMessage 80 · StrongInformational 15Details
Commit message · will
ci: use Warp cache for Docker layers
Speeds of 1MB/s and 15 minute cached docker image pulls during builds are not uncommon.
Warp runners provide a local GitHub Actions cache protocol proxy for Docker layer cache traffic. Point BuildKit's gha cache backend at that proxy on Warp runners so cached image layers do not have to be fetched from GitHub's slower cache service.
Add a default for provider so other users (e.g. qa-assets) don't have to update this unless they use custome runners.
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 CI infrastructure change. It tells Bitcoin Core's GitHub Actions build system to use a faster local cache service (called 'Warp') for Docker image layers, instead of GitHub's slower default cache. There is no change to Bitcoin's actual code, consensus rules, wallet handling, or network behavior. It is not a security fix and does not introduce an obvious security issue based on the supplied materials.
Lower-priorityci: add pyzmq to msan jobby fanquake · 4a6d1458 · Jun 3, 2026 · 1 fileMessage 69 · AdequateInformational 15Details
Commit message · fanquake
ci: add pyzmq to msan job
So that interface_zmq.py is run.
69/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Explains rationale or failure mode! No meaningful explanatory body
AI analysis · Informational 15/100
This commit only changes a continuous integration (CI) test script. It adds the Python package 'pyzmq' to the list of packages installed for a memory-sanitizer (MSAN) test job, so that a ZeroMQ-related functional test can run. It does not change any production code, wallet logic, networking code, or anything users run.
Lower-priorityci: use pyzmq over zmqby fanquake · c21b58e2 · Jun 3, 2026 · 1 fileMessage 81 · StrongInformational 16Details
Commit message · fanquake
ci: use pyzmq over zmq
zmq seems to be an alias for pyzmq, and the project page, https://pypi.org/project/zmq/, states "You are probably looking for pyzmq.". So switch to pyzmq, which is what we document, and use in all other jobs.
81/100 · StrongMessage clarity
✓ Subject identifies a change✓ 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 16/100
This is a one-line change to a Bitcoin Core CI (continuous integration) script for macOS. It swaps the Python package name 'zmq' for 'pyzmq' when installing test dependencies. The project page for 'zmq' on PyPI says it is effectively an alias for 'pyzmq' and recommends using 'pyzmq'. This is a routine build/test maintenance fix with no apparent security relevance.
Security candidatemigrate: Handle HD chains that have identical seeds but different IDsby Ava Chow · de92208c · Jun 3, 2026 · 2 filesMessage 78 · AdequateLow 41Details
Commit message · Ava Chow
migrate: Handle HD chains that have identical seeds but different IDs
The seed ID is calculated from a pubkey produced by treating the seed as a private key. This calculation includes a pubkey compression parameter, even thought that compression is completely irrelevant for the usage of the seed as a BIP 32 seed. Thus migration should detect if a seed has been used multiple times by checking if the computed master key was already processed.
The spkm_migration fuzzer needs to have it's added descriptors accounting to be updated for this fix.
78/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Names security-relevant behavior explicitly
Why it was queued
secret or key materialfuzzing or regression evidencesigning or wallet path
AI analysis · Low 41/100
This commit fixes a wallet migration bug in Bitcoin Core. When converting old-style wallets to modern descriptor wallets, the migration code could process the same BIP 32 seed more than once because it identified seeds by a 'seed ID' that changes depending on a technical detail (public-key compression) that does not actually affect the seed's value. The fix now detects duplicate seeds by comparing the derived master extended public key, preventing redundant or conflicting descriptors from being created during migration. There is no direct evidence this is exploitable as an attack; it appears to be a correctness and robustness fix.
AI review queueddepends: Build `qt` and `qrencode` packages for OpenBSD hostsby Hennadii Stepanov · a54ec373 · Jun 2, 2026 · 9 filesMessage 50 · ThinInformational 19Details
Commit message · Hennadii Stepanov
depends: Build `qt` and `qrencode` packages for OpenBSD hosts
50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
parser or protocol pathsecond-pass: security-sensitive path
AI analysis · Informational 19/100
This commit adds build-system support so that Bitcoin Core's graphical user interface (GUI) and QR-code library can be compiled on OpenBSD. It does not change how Bitcoin validates transactions or handles wallets; it only fixes compilation and packaging issues for a new operating-system target. There is no indication this introduces a security vulnerability.
Lower-prioritytest: run bitcoin-cli -ipcconnect check under valgrind with -datadirby Michael Dietz · 7735c134 · Jun 2, 2026 · 1 fileMessage 95 · StrongInformational 15Details
Commit message · Michael Dietz
test: run bitcoin-cli -ipcconnect check under valgrind with -datadir
This case invoked bitcoin-cli via raw subprocess.run() without the valgrind wrapper (bypassing valgrind) and without -datadir (so it read the default datadir's bitcoin.conf, e.g. ~/.bitcoin, and failed whenever that real config was unusable). Build the command like the rest of the framework: prepend binaries.valgrind_cmd and pass the node's -datadir so the check runs under valgrind and depends only on the build's IPC support, not the host environment.
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
AI analysis · Informational 15/100
This is a test-only fix. It changes one functional test so that a bitcoin-cli subprocess is launched with the project's standard valgrind wrapper and uses the test node's temporary data directory instead of the developer's real default Bitcoin directory. There is no change to production code, no vulnerability, and no security risk to users.
Lower-priorityfuzz: target concurrent leveldb readsby Andrew Toth · 8cb8653a · Jun 2, 2026 · 1 fileMessage 55 · ThinInformational 12Details
Commit message · Andrew Toth
fuzz: target concurrent leveldb reads
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 evidence
AI analysis · Informational 12/100
This commit adds a new fuzz test that exercises Bitcoin Core's database wrapper (CDBWrapper) with many simultaneous read threads while a background database compaction runs. It also makes the test's mock LevelDB environment thread-safe by adding a mutex around its task queue. The change is purely in test code and does not alter production behavior.
Lower-priorityfuzz: extract ConsumeDBParams helperby Andrew Toth · 6609088f · Jun 2, 2026 · 1 fileMessage 68 · AdequateInformational 15Details
Commit message · Andrew Toth
fuzz: extract ConsumeDBParams helper
Pull the inline DBParams construction out of TestDbWrapper into a shared ConsumeDBParams() helper. This is a pure refactor with no behavior change, preparing for an additional harness that needs to build the same params.
This commit is a simple code cleanup inside a fuzz test file. It pulls out a repeated block of code into a helper function named ConsumeDBParams() so it can be reused later. The commit message explicitly calls it a 'pure refactor with no behavior change,' and the diff only moves existing logic around without changing any values, checks, or program behavior.
Lower-priorityci: inline runner selectionby will · 726e196e · Jun 2, 2026 · 1 fileMessage 70 · AdequateInformational 15Details
Commit message · will
ci: inline runner selection
We are currently sometimes backlogged on waiting for runner selection.
Selecting Warp or GitHub-hosted runners directly from the repository context avoids serializing all CI jobs behind a metadata job. This keeps forks on public runners while allowing upstream jobs to schedule immediately on the intended runner labels.
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 GitHub Actions CI optimization. It removes a separate 'runner selection' metadata job and instead directly decides whether to use fast internal (Warp) runners or standard GitHub-hosted runners based on the repository name. There is no security-relevant change to Bitcoin Core's code, build outputs, or secrets handling.
build: exclude mptest target from compile commands
build: exclude mptest target from compile commands
77/100 · AdequateMessage clarity
✓ Specific, 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 build-system tweak. It adds one internal test target ('mptest') to a list of targets that should not be included in the compile_commands.json database used by developer tools like clang-tidy. There is no user-facing or security-relevant change.
test: Improve loopback address check in `rcp_bind.py`
A loopback address can range from `127.0.0.0` to `127.255.255.255`. This commit relaxes the loopback check in `rpc_bind.py` by checking whether an IP address (from `all_interfaces()`) starts with `'127.'` instead of strictly matching `'127.0.0.1'`.
Programs like VPNs might add an extra loopback address (e.g., 127.1.130.83), which failed under the previous state. These addresses will now pass with this update.
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
AI analysis · Informational 15/100
This commit changes a single line in a Bitcoin Core test script. It makes the test recognize any IP address beginning with '127.' as a loopback address, instead of only '127.0.0.1'. This is a test-only fix to prevent false failures when VPNs or other software add extra loopback addresses. It does not change the Bitcoin Core software that users run, and it has no security impact.
Lower-priorityversionbits: update VersionBitsCache doc comment to match current behaviourby Antoine Poinsot · 107d4178 · Jun 1, 2026 · 1 fileMessage 65 · AdequateInformational 15Details
Commit message · Antoine Poinsot
versionbits: update VersionBitsCache doc comment to match current behaviour
Co-Authored-by: Anthony Towns <aj@erisian.com.au>
65/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
AI analysis · Informational 15/100
This commit only updates a code comment in a header file to describe the current behavior of a cache used for tracking Bitcoin softfork deployment status. No executable code was changed, and there is no security impact.
Lower-prioritydoc: release notes and bips doc update for #34779by Antoine Poinsot · 94e3ac0b · Jun 1, 2026 · 2 filesMessage 53 · ThinInformational 15Details
Commit message · Antoine Poinsot
doc: release notes and bips doc update for #34779
53/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Links an issue, advisory, or supporting reference! No meaningful explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100
This commit only updates documentation. It adds a release note and a BIP implementation list entry describing a prior code change (PR #34779) that makes Bitcoin Core ignore certain BIP 9 soft-fork signal bits per BIP 323. There is no code change here and no direct security risk in the commit itself.
Lower-priorityqa: test we don't warn for ignored unknown version bits deploymentsby Antoine Poinsot · 1d524057 · Jun 1, 2026 · 1 fileMessage 75 · AdequateInformational 12Details
Commit message · Antoine Poinsot
qa: test we don't warn for ignored unknown version bits deployments
Co-Authored-by: Anthony Towns <aj@erisian.com.au>
75/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Mentions testing or verification
AI analysis · Informational 12/100
This commit only adds a new test case to Bitcoin Core's existing test suite. It checks that the software does not produce spurious warnings for version bits that belong to deployments that have been explicitly disabled/ignored. There is no change to production code, no bug fix, and no security vulnerability in the commit itself.
Security candidateversionbits: Limit live activation params and activation warnings per BIP323by Anthony Towns · f802edf5 · Jun 1, 2026 · 5 filesMessage 75 · AdequateLow 27Details
Commit message · Anthony Towns
versionbits: Limit live activation params and activation warnings per BIP323
Test bits are conserved. This only has an effect on the warnings.
Co-Authored-By: Antoine Poinsot <mail@antoinep.com>
75/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Mentions testing or verification
Why it was queued
secret or key material
AI analysis · Low 27/100
This Bitcoin Core change narrows which version bits trigger live 'unknown soft fork' warnings from 29 bits down to 5, following the new BIP323 specification. It does not change how many bits exist in block headers or how test networks behave; it only reduces false or misleading upgrade alerts for bits that the network no longer treats as active deployment candidates.
Lower-priorityci: Add dynamic cache switching to warp cacheby will · 2ce4ae7d · Jun 1, 2026 · 6 filesMessage 80 · StrongInformational 15Details
Commit message · will
ci: Add dynamic cache switching to warp cache
The GHA cache is very slow, taking on the order of minutes to save and restore from.
Use WarpBuild's cache instead as this is in the same region and much faster.
WarpBuild cache action does not auto-fallback to GHA if not being run on Warp. To allow fork runs to fallback to GHA caching, whilst minimising duplication in the action files, create new "interal" actions which perform the switching logic, and use these in the (renamed) cache|save actions.
Without this we would need the `if` logic in our prvious actions, 4 times in each of save and restore.
Plumb the provider through into the action, as a composite action can't read `env` (`GITHUB_OUTPUT`) from previous steps.
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 change to Bitcoin Core's continuous integration (CI) setup. It swaps the GitHub Actions cache for a faster third-party cache service (WarpBuild) on Warp runners, while keeping GitHub's cache as a fallback for forks. There is no change to the Bitcoin software itself, to wallets, transactions, consensus rules, or network code.
net: un-default the OpenNetworkConnection()'s proxy_override argument
This way callers will not forget to set it.
65/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
AI analysis · Informational 19/100
This change removes a default value for a network-connection setting called proxy_override. Previously, callers could leave it unset without thinking. Now every caller must explicitly pass a value, which helps prevent accidental mistakes about which proxy a connection uses. It is a defensive code-quality fix, not a patch for a known active bug or attack.
Lower-prioritytest: add a regression test for private broadcast v1 retriesby Vasil Dimov · 5a3756d1 · May 30, 2026 · 2 filesMessage 72 · AdequateInformational 15Details
Commit message · Vasil Dimov
test: add a regression test for private broadcast v1 retries
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 only adds a new regression test file and registers it in the test runner. It does not change any production code in Bitcoin Core. The test verifies that if a private broadcast connection using the newer v2 transport fails, the node falls back to the older v1 transport through the Tor proxy. There is no security patch or vulnerability fix here.
AI review queuedwallet, test: fix sendall anti-fee-sniping when locktime is not specifiedby rkrux · 9c1fcaca · May 29, 2026 · 1 fileMessage 95 · StrongInformational 12Details
Commit message · rkrux
wallet, test: fix sendall anti-fee-sniping when locktime is not specified
This particular test case only needs to ensure that locktime is not specified in the RPC request, it doesn't need to rely on the wallet optin RBF default that causes the test to pass coincidentally.
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode✓ Mentions testing or verification
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 12/100
This commit only changes a test file. It makes a wallet functional test more explicit by passing replaceable=False so the test reliably checks anti-fee-sniping locktime behavior, instead of relying on a coincidental wallet default. There is no change to production wallet or node code, so it does not fix a live security vulnerability.
AI review queuedkernel: assert invalid buffer preconditions in `btck_*_create` functionsby stringintech · 570a6276 · May 29, 2026 · 3 filesMessage 73 · AdequateLow 28Details
Commit message · stringintech
kernel: assert invalid buffer preconditions in `btck_*_create` functions
Switch buffer `ptr == nullptr && len > 0` checks from `nullptr` returns to assertions. These checks represent invalid caller preconditions, not failures encountered while deserializing or constructing the requested object. `btck_block_header_create` additionally asserts the pre-existing documented length contract (must be 80 bytes).
73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Why it was queued
second-pass: broader security terminology
AI analysis · Low 28/100
This commit changes several Bitcoin Core kernel library functions so they crash the program (via assertions) instead of returning a null pointer when callers pass a null buffer with a non-zero length. It also tightens one function to require exactly 80-byte block headers. The change treats these as programmer errors rather than runtime failures, which can help catch misuse early but may cause denial-of-service crashes if an attacker can trigger those calls with bad inputs.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Explains rationale or failure mode✓ Mentions testing or verification
AI analysis · Informational 15/100
This is a test-only code change that restores the string format of an error message shown when a Bitcoin Core functional test encounters a JSON-RPC failure. It only affects test framework output formatting and has no bearing on the live Bitcoin Core software or its users' funds or network security.
Lower-prioritytest: make reusable filling of a node's addrmanby Vasil Dimov · ab35a028 · May 29, 2026 · 2 filesMessage 90 · StrongInformational 15Details
Commit message · Vasil Dimov
test: make reusable filling of a node's addrman
Extract the part of `p2p_private_broadcast.py` that fills a given node's addrman and put it into `test_framework/test_framework.py`.
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 is a pure test-code refactoring. It moves a list of fake network addresses and a helper loop from one test file into the shared test framework so other tests can reuse it. No production Bitcoin Core code is changed, and nothing about how real nodes connect or broadcast is modified.
Lower-prioritytest: make reusable starting a standalone P2P listenerby Vasil Dimov · 2333be9c · May 29, 2026 · 2 filesMessage 95 · StrongInformational 15Details
Commit message · Vasil Dimov
test: make reusable starting a standalone P2P listener
Extract the part of `p2p_private_broadcast.py` that starts listening on a `P2PConnection` object (or its children classes) and put it into `test_framework/p2p.py`.
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
AI analysis · Informational 15/100
This commit is a simple code cleanup in Bitcoin Core's test suite. It moves a small helper that starts a temporary peer-to-peer listener from one test file into a shared test framework file so other tests can reuse it. There is no change to the actual Bitcoin network code, no change to how nodes communicate, and no security fix or vulnerability.