📝 fanquake converted_to_draft a pull request: "fee estimate test: fix #31944 by handling a legitimate scenario that …"
(https://github.com/bitcoin/bitcoin/pull/32615)
Fix a legitimate scenario that feerate is not always available, due to the randomly generated fees that causes current percentage to be smaller than success break point, in TxConfirmStats::EstimateMedianVal (in src/policy/fees.cpp). The solution is to hard code a list of fees that guarantee to pass the check of fees and smart fees.

To reproduce/validate the issue, run the test with a specific randomseed: 2986529890161488286

build/test/functional/test_runner.py test/functional/feature_fee_
...

💬 fanquake commented on pull request "fee estimate test: fix #31944 by handling a legitimate scenario that …":
(https://github.com/bitcoin/bitcoin/pull/32615#issuecomment-3027935606)
@tnndbtc I've moved this to draft, as we aren't going to merge this with the current approach, using the generated `global_fee_list`.

💬 darosior commented on pull request "policy: make pathological transactions packed with legacy sigops non-standard":
(https://github.com/bitcoin/bitcoin/pull/32521#discussion_r2180101377)
Yeah it's just a typo when/where:
```suggestion
// Unlike the existing block wide sigop limit, BIP54 counts sigops where they are actually executed.
```

💬 darosior commented on pull request "policy: make pathological transactions packed with legacy sigops non-standard":
(https://github.com/bitcoin/bitcoin/pull/32521#discussion_r2180103816)
Done, thanks for spotting the typo.

💬 hodlinator commented on pull request "ci: update pwsh to use custom shell that fails-fast":
(https://github.com/bitcoin/bitcoin/pull/32672#issuecomment-3027942199)
Concept ACK (but my PowerShell skill is 0).

@fanquake you are currently registered as a N-A-C-K.

💬 instagibbs commented on pull request "policy: make pathological transactions packed with legacy sigops non-standard":
(https://github.com/bitcoin/bitcoin/pull/32521#discussion_r2180105702)
I think that's still ambiguous as it does read like a taproot sigops budget sense where the opcode itself must be executed to count

💬 luke-jr commented on pull request "policy: make pathological transactions packed with legacy sigops non-standard":
(https://github.com/bitcoin/bitcoin/pull/32521#discussion_r2180106942)
This is still wrong. If you bury a sigop in (false) OP_IF, they are still counted even though not actually executed

💬 fanquake commented on pull request "log: Mitigate disk filling attacks by rate limiting LogPrintf, LogInfo, LogWarning, LogError":
(https://github.com/bitcoin/bitcoin/pull/32604#issuecomment-3027954533)
Added this to `30.0`.

👍 brunoerg approved a pull request: "fuzz: Make process_message(s) more deterministic"
(https://github.com/bitcoin/bitcoin/pull/32822#pullrequestreview-2979255643)
light code review ACK fac673d434b4d32d8c44dcc50a3655ba4a1816de

✅ fanquake closed a pull request: "doc: Add hint about avoiding spaces in paths when building on Windows"
(https://github.com/bitcoin/bitcoin/pull/32397)

💬 fanquake commented on pull request "doc: Add hint about avoiding spaces in paths when building on Windows":
(https://github.com/bitcoin/bitcoin/pull/32397#issuecomment-3027979421)
Closing for now. @hebasto maybe you can followup with your own suggestion.

💬 bigshiny90 commented on pull request "RPC/txoutproof: Support including (and verifying) proofs of wtxid":
(https://github.com/bitcoin/bitcoin/pull/32844#issuecomment-3027980445)
Changes are in my branch: https://github.com/bigshiny90/bitcoin/tree/fix-pr-32844-witness

💬 fanquake commented on pull request "allocators: Apply manual ASan poisoning to `PoolResource`":
(https://github.com/bitcoin/bitcoin/pull/32581#issuecomment-3027988340)
Added this to 30.0.
cc @marcofleon

🚀 fanquake merged a pull request: "miniscript, refactor: Make `operator""_mst` `consteval` (re-take)"
(https://github.com/bitcoin/bitcoin/pull/32564)

💬 fanquake commented on pull request "util: C++20 `ToIntegral()` Improvement":
(https://github.com/bitcoin/bitcoin/pull/32522#discussion_r2180168906)
> Otherwise, it will become harder to diagnose bugs in the code with the compiler or with sanitizers.

@dergoegge any opinion?

📝 Sjors opened a pull request: "wallet: allow skipping script paths"
(https://github.com/bitcoin/bitcoin/pull/32857)
While testing MuSig2 integration in #29675 I ran into a Ledger bug https://github.com/LedgerHQ/app-bitcoin-new/issues/329 that was triggered because we signed for a script path in addition to the key path. The bug itself will be fixed, but it seems useful in general to be able to skip script path signing.

For example, you may have a 2-of-2 musig() descriptor with some sort of fallback after coins are N blocks old. For both on chain privacy and fee reasons, you don't want to use this path unle
...

💬 Sjors commented on pull request "wallet: Be able to receive and spend inputs involving MuSig2 aggregate keys":
(https://github.com/bitcoin/bitcoin/pull/29675#issuecomment-3028050886)
Meanwhile https://github.com/LedgerHQ/app-bitcoin-new/issues/329 has been figured out and I was able to do a keypath spend on the more complicated policy. That's a good sign for interoperability.

💬 instagibbs commented on pull request "test: Fix reorg patterns in tests to use proper fork-based approach":
(https://github.com/bitcoin/bitcoin/pull/32587#discussion_r2180187726)
Given that the coinbase maturity tests also use invalidateblock, let's just keep this particular test untouched for now. This test setup is going to have to be a bit more advanced and can be done in a follow-up

💬 fanquake commented on pull request "cmake: Improve Python robustness and test usability":
(https://github.com/bitcoin/bitcoin/pull/31233#issuecomment-3028091142)
Guix Build:
```bash
cc67459424a62c706895f382cff677c1e0c0ecd7ee7f43429d64d1f5d872f151 guix-build-67dc7523f3e1/output/aarch64-linux-gnu/SHA256SUMS.part
3c1eb6f629cc3b00d7e3803fc33383aea1f7cf6646d4de3e5c96d30923a1a795 guix-build-67dc7523f3e1/output/aarch64-linux-gnu/bitcoin-67dc7523f3e1-aarch64-linux-gnu-debug.tar.gz
494c55c6843e0d1f8aa133ee4c7685f06f04ca4c3dcb8ccee02aa11686a64350 guix-build-67dc7523f3e1/output/aarch64-linux-gnu/bitcoin-67dc7523f3e1-aarch64-linux-gnu.tar.gz
e12c0167012af21d
...

💬 maflcko commented on pull request "Refactor: Redefine CTransaction equality to include witness data":
(https://github.com/bitcoin/bitcoin/pull/32723#issuecomment-3028112892)
Just for reference and fun, not as a pull request, the `EqualsOptions` patch would look something like:

```diff
diff --git a/src/primitives/transaction.h b/src/primitives/transaction.h
index ad81727094..7eb8ded479 100644
--- a/src/primitives/transaction.h
+++ b/src/primitives/transaction.h
@@ -289,6 +289,10 @@ inline CAmount CalculateOutputValue(const TxType& tx)
}


+struct EqualsOptions {
+ bool include_script_sig{true};
+ bool include_witness_data{true};
+};
/** The
...