RB
← All projectsRust Bitcoin

rust-bitcoin

Rust library for Bitcoin data structures, serialization, consensus encoding, and scripts.

BitcoinCryptographic librariesNormal
Repository coverage

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

529security candidates508second-pass queue2200AI analyses
136commits · 30 days
272commits · 60 days
1122commits · 180 days
2033commits · 365 days
Backfill bands
Aug 5 → Feb 6787 seen32 candidatesComplete
Feb 6 → Jun 6878 seen53 candidatesComplete
Jun 6 → Jul 6211 seen15 candidatesComplete
Jul 6 → Aug 5184 seen2 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.

66/100 average clarity
499Strong · 80–100
1084Adequate · 60–79
567Thin · 40–59
151Opaque · 0–39
20security 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.
Andrew Poelstra22076151190
Mitchell Bagot649193645068
Tobin C. Harding41566410063
jrakibi944994068
Nick Johnson19121190060
Jamil Lambert, PhD11919116061
satsfy (Renato Britto)381527066
Fmt Bot331431045
Trevor Arjeski111111069
Shing Him Ng31731056
Martin Habovstiak30628068
Ismail Daif22622050
Analysis record

Published AI watches

Last scanned 19 minutes ago

Low 33 AI analysisMessage 100 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6946: Fix integer overflow in `get_array`

This commit fixes a small but real bug in a Rust helper that reads fixed-size chunks from a data slice. The helper was supposed to safely return 'nothing' when asked to read past the end of the data, but it accidentally added two numbers t…

Integer overflow in bounds-checking helperContract violation: method documented to return None on out-of-bounds access could panic insteadDebug-build panic (denial of service) possible
c6e80843by Andrew Poelstra+2−11 file
Vendor flagged security relevance
Moderate 62 AI analysisMessage 73 · Adequate
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Fix integer overflow in `get_array`

This commit fixes a bug in a Rust helper method called `get_array`, which is meant to safely read a fixed-size chunk from a slice and return nothing if the requested range is out of bounds. The bug was that the code added the caller's offs…

Integer overflow in bounds calculationPotential panic due to violated internal length expectationCaller-controlled arithmetic used for memory access bounds
56fb1287by Martin Habovstiak+2−11 file
Vendor flagged security relevance
High 71 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6915: primitives: Fix `Witness` handling of oversized items

This commit fixes a bug in how the Rust Bitcoin library counts and compares transaction witness data when a witness contains an oversized item. Previously, several functions relied on an iterator that silently skips oversized items, causin…

Inconsistent serialization/iterator behavior for oversized witness itemswtxid collision risk between transactions differing only in oversized witness bytesIncorrect witness equality for oversized single-item stacks
e1ed5884by Andrew Poelstra+106−273 files
Vendor flagged security relevance
Informational 19 AI analysisMessage 100 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6922: Use `try_fold` instead of `fold` in `Sum` impl

This is a code-quality and performance improvement, not a security fix. It changes how the library adds up lists of Bitcoin amounts so that it stops early once an overflow is detected, rather than continuing to process the rest of the list…

No security-relevant signal in commit message or diffRefactor preserves overflow-checking behavior (short-circuits instead of continuing)New API method `NumOpResult::from_result` is a pure inverse of existing `into_result`
86e4d5daby Andrew Poelstra+60−562 files
No security note in commit
Moderate 52 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6893: units: Reject malformed amount strings

This update fixes a bug in how the library reads Bitcoin amount strings like '1.5 BTC'. Previously, certain malformed inputs such as '.', '._', '1_', '1_.0', and '1._0' were incorrectly accepted and treated as valid amounts (often zero), i…

Input validation bypass in amount parserMalformed strings silently parsed as zero or ordinary amountsUnderscore separator placement not enforced
fcb14622by Andrew Poelstra+88−343 files
Vendor flagged security relevance
Low 48 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6921: units: fix div_by_fee_rate_ceil precision

This commit fixes a rounding bug in how the rust-bitcoin library calculates the minimum transaction weight needed to pay a given fee at a given fee rate. The old code rounded the fee rate up too early, which could produce a weight slightly…

Incorrect fee-weight calculation due to premature integer roundingPotential transaction fee shortfall when using div_by_fee_rate_ceilOverflow protection added for Amount::MAX * 4_000_000 intermediate value
b31212e0by Andrew Poelstra+38−82 files
No security note in commit
Informational 15 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6898: Release tracking PR: `consensus-encoding 1.3.0`

This is a routine release-management commit that bumps the version number of the `bitcoin-consensus-encoding` crate from 1.2.0 to 1.3.0 and updates lock files accordingly. It contains no code changes that fix or introduce a security issue.…

0cfc7908by Andrew Poelstra+37−349 files
No security note in commit
Informational 15 AI analysisMessage 96 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6909: build(deps): bump actions/labeler from 6.2.0 to 7.0.0

This commit updates a GitHub Actions automation tool (actions/labeler) used to automatically tag pull requests with labels. It is a routine dependency version bump from 6.2.0 to 7.0.0, with no indication of a security fix or vulnerability.…

4ed7c068by Andrew Poelstra+1−11 file
No security note in commit
Informational 15 AI analysisMessage 96 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6910: build(deps): bump actions/checkout from 7.0.0 to 7.0.1

This commit is a routine update to the GitHub Actions checkout tool used by the project's automated workflows. It only changes version numbers in configuration files and does not alter the actual Bitcoin library code that users run. There …

328c4ae9by Andrew Poelstra+37−3717 files
No security note in commit
Informational 15 AI analysisMessage 100 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6911: build(deps): bump astral-sh/setup-uv from 8.3.2 to 9.0.0

This commit updates a GitHub Actions helper used to install a Python tool called uv, which runs the zizmor security scanner. The change only bumps the pinned version of the helper from 8.3.2 to 9.0.0. The new version's release notes mentio…

No security-relevant signals in commit or upstream release notesDependency bump in CI only, not in library codeNo CVE or advisory referenced
67600795by Andrew Poelstra+2−22 files
No security note in commit
Informational 15 AI analysisMessage 96 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6912: build(deps): bump github/codeql-action/upload-sarif from 4.37.0 to 4.37.3

This is a routine Dependabot update that changes the pinned version of GitHub's official CodeQL upload-sarif action from 4.37.0 to 4.37.3 in a single CI workflow. The action only uploads static analysis results to GitHub; it does not touch…

b51cec63by Andrew Poelstra+1−11 file
No security note in commit
Informational 15 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6913: build(deps): bump dtolnay/rust-toolchain from 6c977a6ca4077a0ceb28ffbe03f59d46e9ac8772 to 02cb101ec7c40f2c49e1d9714d64511d8e1b74de

This is a routine update to a GitHub Actions helper used to install Rust during automated testing. It only changes the pinned version of the dtolnay/rust-toolchain action in workflow files. There is no change to the actual rust-bitcoin lib…

90330d15by Andrew Poelstra+8−84 files
No security note in commit
Informational 20 AI analysisMessage 100 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6906: consensus_encoding, primitives: expose exact encoding size for block and transaction

This commit adds a way to ask, in advance, exactly how many bytes a Bitcoin block or transaction will take when serialized. It is a feature addition for the library's encoding system, not a fix for a vulnerability. There is no indication i…

No security-relevant signals in commit message or diffFeature addition: expose exact encoded sizeNo mention of vulnerability, CVE, bug bounty, or security report
1a365d53by Andrew Poelstra+129−1068 files
No security note in commit
Informational 15 AI analysisMessage 88 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

build(deps): bump dtolnay/rust-toolchain

This is a routine update by Dependabot that changes which version of a popular GitHub Action (dtolnay/rust-toolchain) is used to install Rust in automated CI workflows. The commit only updates pinned commit hashes in workflow files; it doe…

a31e0b0eby dependabot[bot]+8−84 files
No security note in commit
Informational 15 AI analysisMessage 93 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

build(deps): bump github/codeql-action/upload-sarif

This is a routine patch-version update of a GitHub-maintained action used only to upload static-analysis results (SARIF files) from a scheduled CI job. The change does not touch any project source code, cryptographic logic, or user-facing …

2cac6e38by dependabot[bot]+1−11 file
No security note in commit
Informational 15 AI analysisMessage 93 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

build(deps): bump astral-sh/setup-uv from 8.3.2 to 9.0.0

This is a routine automated update by Dependabot that changes the pinned version of a GitHub Action used to install a Python tool called 'uv' in two workflow files. The new version is a major release of the setup-uv action itself, but the …

7d7e7269by dependabot[bot]+2−22 files
No security note in commit
Informational 21 AI analysisMessage 100 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6894: Harden `Copy` policy and apply to all pre-1.0 crates

This commit removes the automatic `Copy` trait from several public error types in the rust-bitcoin library and updates the project's written policy to discourage `Copy` on error types. `Copy` is a Rust trait that lets values be duplicated …

API hardening: removes `Copy` from public error types to preserve future flexibilityPolicy update: docs/policy.md now explicitly discourages `Copy` on error typesNo vulnerability fix: change is defensive/preventive, not reactive to a disclosed issue
457cb9b5by Andrew Poelstra+14−146 files
No security note in commit
Informational 18 AI analysisMessage 96 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6787: Move `from_script` to `Address` and drop `AddressExt`

This change is a routine code cleanup, not a security fix. It moves a method that converts Bitcoin output scripts into human-readable addresses from a temporary 'extension trait' directly onto the main Address type. The actual conversion l…

ed42dabdby Andrew Poelstra+38−414 files
No security note in commit
Low 35 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6061: Units improvements

This is a routine code-quality and API-expansion pull request for the rust-bitcoin library's 'units' crate. It widens some fee-rate constructors from 32-bit to 64-bit inputs, renames the old 32-bit versions, improves serde error messages, …

API surface change: new u64 fee-rate constructors return NumOpResult to prevent silent overflowRenaming of existing u32 constructors to *_u32 may break downstream callers at compile time (breaking API change)Removal of artificial restriction in from_per_vb allows larger valid inputs
f9460859by Andrew Poelstra+266−12211 files
No security note in commit
Low 49 AI analysisMessage 96 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6862: units: Preserve error in NumOpResult add and sub

This commit fixes a bug in how the library handles math errors. Previously, if you added or subtracted two values and one of them already had an error (for example, dividing by zero), the library would silently replace that original error …

Error-state information loss in arithmetic wrapper typeIncorrect error propagation could mask prior failures such as division by zeroDefensive correctness fix in numeric operation result handling
5324e59aby Andrew Poelstra+26−41 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-priorityUpdate the API text filesby Tobin C. Harding · 4d35ff28 · Jun 9, 2026 · 3 filesMessage 45 · ThinInformational 15Details
Commit message · Tobin C. Harding

Update the API text files

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit only updates generated text files that list the public API surface of the crate. The change records that a new standard Rust conversion trait (`From<Infallible>`) is now implemented for an existing error type. No source code, logic, or behavior was changed, so there is no security issue here.

Lower-prioritynetwork: Implement From<Infallible> for error typeby Tobin C. Harding · ed405f6d · Jun 9, 2026 · 1 fileMessage 50 · ThinInformational 15Details
Commit message · Tobin C. Harding

network: Implement From<Infallible> for error type

As is customary add the impl.

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 15/100

This is a routine Rust language addition: it lets the library's network-parsing error type be created from Rust's 'Infallible' type, which can never actually occur. It has no security relevance on its own.

Lower-prioritynetwork: Move serde code down the fileby Tobin C. Harding · c5cf458f · Jun 9, 2026 · 1 fileMessage 68 · AdequateInformational 15Details
Commit message · Tobin C. Harding

network: Move serde code down the file

This one is a bit subjective; following the idea that the least
important stuff goes further down the file move `serde` down.

This code move also leaves the `Display` and `FromStr` impls next to
each other, which is nice.

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

This commit simply rearranges code within a single file. It moves the Display, FromStr, and AsRef implementations for the Network type to appear earlier in the file, and moves the serde-related code lower down. No logic, behavior, or security properties of the code were changed.

Lower-prioritynetwork: Put impl block directly under typeby Tobin C. Harding · 21e5c690 · Jun 9, 2026 · 1 fileMessage 60 · AdequateInformational 15Details
Commit message · Tobin C. Harding

network: Put impl block directly under type

As is customary put the impl block for `Network` directly under the
type definition.

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

This commit simply moves existing code around in the same file. It places the main implementation block for the Network type directly underneath the type definition, which is a common Rust style preference. No behavior, logic, or security properties changed.

AI review queuedUpdate API filesby Mitchell Bagot · 2ccf8548 · Jun 9, 2026 · 3 filesMessage 28 · OpaqueInformational 15Details
Commit message · Mitchell Bagot

Update API files

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

This commit only updates three text files that list the project's public API surface. It removes entries for modules, macros, and types that are no longer part of the public API. There is no actual source code change, no bug fix, and no security-related content.

Lower-priorityRemove unused modules from internalsby Mitchell Bagot · b2607fd5 · Jun 9, 2026 · 3 filesMessage 68 · AdequateInformational 15Details
Commit message · Mitchell Bagot

Remove unused modules from internals

There are two modules, parse and const_tools, that are completely
unused in internals. That is, their defined macros and types are
only ever used within themselves, not even elsewhere within internals.
These modules can be trivially dropped.

Drop parse and const_tools modules from internals.

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

This commit simply deletes two internal helper code modules that were not being used anywhere in the project. It is a cleanup change with no security relevance.

Lower-priorityRemove unused types and macros from internals::serdeby Mitchell Bagot · 0b2cb459 · Jun 9, 2026 · 1 fileMessage 73 · AdequateInformational 15Details
Commit message · Mitchell Bagot

Remove unused types and macros from internals::serde

The IntoDeError and serde_struct_human_string_impl macro in
internals::serde are both entirely unused. These two are neither used
privately within internals, nor publicly by other crates.

Drop IntoDeError and serde_struct_human_string_impl from
internals::serde.

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

This commit simply deletes unused code—an internal trait and a macro for serde serialization helpers. There is no bug fix, behavior change, or security patch here. It is a routine cleanup that removes dead code.

Lower-priorityMove SerializeBytesAsHex to primitivesby Mitchell Bagot · 966d9295 · Jun 9, 2026 · 2 filesMessage 58 · ThinInformational 15Details
Commit message · Mitchell Bagot

Move SerializeBytesAsHex to primitives

The SerializeBytesAsHex type in internals is only ever used in Witness
in primitives. Rather than provide the type from internals, the type
can instead be pasted in-line in the primitives witness module (as a
private type) and used there.

Move SerializeBytesAsHex type as private type to primitives::witness.

58/100 · ThinMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context
AI analysis · Informational 15/100

This commit simply moves an existing helper type used for serializing bytes as hexadecimal from one internal module to another. The code is copied verbatim and made private to the module that uses it. There is no functional change, no bug fix, and no security relevance.

Lower-priorityDrop wrap_debug module from internalsby Mitchell Bagot · a6683bc2 · Jun 9, 2026 · 3 filesMessage 68 · AdequateInformational 15Details
Commit message · Mitchell Bagot

Drop wrap_debug module from internals

The wrap_debug module in internals is only ever used in Witness in
primitives. Rather than provide the module from internals, the type
can instead be pasted in-line in the primitives witness module (as a
private type) and used there.

Copy WrapDebug type as private type to primitives::witness and drop
wrap_debug module from internals.

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

This commit is a simple internal code cleanup. It moves a small helper type used only for formatting debug output from a shared internal utility module directly into the only place it is used, then removes the now-unused utility module. There is no change to how the code behaves or to any security-sensitive logic.

Security candidateMove serde_round_trip macro to bip32by Mitchell Bagot · 3cf7554a · Jun 9, 2026 · 5 filesMessage 78 · AdequateInformational 15Details
Commit message · Mitchell Bagot

Move serde_round_trip macro to bip32

The serde_round_trip macro from internals is only used in the bip32
module for testing. Rather than pulling it in from internals, it should
instead be defined in-line in the tests module.

Move serde_round_trip macro to bip32. Add bincode dev dependency to
key_expression.

78/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification
Why it was queued
secret or key materialcryptography-sensitive path
AI analysis · Informational 15/100

This commit is a routine code cleanup: it moves a small test-only helper macro from a shared internal utility crate into the one place it is actually used. There is no change to user-facing behavior, no bug fix, and no security relevance.

AI review queuedprimitives: Re-order transaction codeby Tobin C. Harding · 6e2bb39a · Jun 9, 2026 · 1 fileMessage 80 · StrongInformational 15Details
Commit message · Tobin C. Harding

primitives: Re-order transaction code

Put the encoding stuff in the conventional order and put other trait
impls all together.

Do `transaction` separately because its a bit more involved.

80/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit is a pure code reorganization: it moves existing implementations of string formatting, hex display, encoding, and decoding for Bitcoin transactions into a different order within the same file. No logic was changed, no bugs were fixed, and no security behavior was altered.

Lower-priorityprimitives: Re-order encoding codeby Tobin C. Harding · 031392c3 · Jun 9, 2026 · 4 filesMessage 80 · StrongInformational 15Details
Commit message · Tobin C. Harding

primitives: Re-order encoding code

I thought we scrubbed this code but the code layout is not correct
here.

We are trying to make all the encoding code be **exactly** the same
because it is boilerplate and is going to live on various branches.

Re-order the code to match our convention.

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

This commit only moves existing code around in four source files. It reorders the encoding and decoding definitions so they follow the same pattern across the project. No logic was changed, no bugs were fixed, and no security issue is present.

Security candidateRemove Verification re-exportsby Mitchell Bagot · 31c3e51b · Jun 9, 2026 · 2 filesMessage 68 · AdequateInformational 20Details
Commit message · Mitchell Bagot

Remove Verification re-exports

These re-exports exist from the days when secp required an explicit
context for cryptographic operations. With the introduction of the
global context in recent secp versions, they are no longer needed.

Remove Verification re-exports from key modules.

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context✓ Mentions testing or verification
Why it was queued
defensive validationcryptography-sensitive path
AI analysis · Informational 20/100

This commit simply stops publicly re-exporting a secp256k1 type called `Verification` from two Rust Bitcoin key modules. It is a routine API cleanup: the type is still available directly from the underlying secp256k1 library, so no cryptographic capability is removed, only a convenience shortcut. There is no security bug or vulnerability here.

Lower-prioritypolicy: update docs on ordering derivesby Nick Johnson · ee1b1416 · Jun 8, 2026 · 1 fileMessage 76 · AdequateInformational 15Details
Commit message · Nick Johnson

policy: update docs on ordering derives

`rust-ordering` is no longer recommended and was removed from the
workspace in #4065 (38c8c9).

There are types like `NetworkKind` which derive order and it is used
downstream in other workspace packages to also derive order (e.g. Xpub).
I believe this derive actually caused issues before in v0.32.4,
discussed in issue #4116. But it probably low risk going forward since
the enum is exhaustive and I don't think someone would change the
variant order just for fun.

This more general policy is discussed in #3865.

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

This commit only updates project documentation (docs/policy.md). It removes a recommendation to use a third-party 'ordered' crate and instead cautions developers about deriving standard Rust ordering traits on enums because variant order becomes part of the public API. There is no code change and no security fix or vulnerability introduced.

Lower-priorityunits: Add mention of wu to hex parsing functionsby Tobin C. Harding · db49559d · Jun 8, 2026 · 1 fileMessage 68 · AdequateInformational 15Details
Commit message · Tobin C. Harding

units: Add mention of wu to hex parsing functions

The `Weight` type has a bunch of constructors each of which mentions
the unit used. However the hex parsing constructors do not mention the
weight unit, add it.

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

This commit only adds two documentation comments to clarify that hex strings passed to certain Weight constructors are interpreted as weight units. There are no code changes, no behavior changes, and no security implications.

Lower-priorityunits: Fix typoby Tobin C. Harding · c91ce9ef · Jun 8, 2026 · 1 fileMessage 43 · ThinInformational 15Details
Commit message · Tobin C. Harding

units: Fix typo

Should be 'from amount' like the other constructors that take an
`Amount` type.

43/100 · ThinMessage clarity
✓ Subject identifies a change✓ Provides an explanatory body
AI analysis · Informational 15/100

This commit fixes a single-word typo in a documentation comment. The function takes an Amount type, but the comment incorrectly said 'satoshis'. There is no code change and no security impact.

AI review queuedunits: Use uniform whitespace in constsby Tobin C. Harding · cc9441fb · Jun 8, 2026 · 4 filesMessage 80 · StrongInformational 15Details
Commit message · Tobin C. Harding

units: Use uniform whitespace in consts

In some places we have no newlines between associated consts and in
others we have newlines. Lets pick one and be uniform.

Elect to use whitespace.

\me ducks to avoid flaming.

80/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit only adds or removes blank lines between constant definitions in four source files to make the code style consistent. It does not change any program logic, values, or behavior, so it has no security relevance.

Lower-priorityunits: Use uniform docs style for 1000by Tobin C. Harding · 35303ad9 · Jun 8, 2026 · 1 fileMessage 80 · StrongInformational 15Details
Commit message · Tobin C. Harding

units: Use uniform docs style for 1000

We have '1000' and also '1,000' - pick one and be uniform.

Favour using the comma because we already have other usages eg
1,000,000

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

This commit only changes two documentation comments in the rust-bitcoin library to use a consistent style for writing the number 1000 (with a comma as '1,000'). It does not modify any executable code, logic, or data handling.

Lower-priorityunits: Use little t in docsby Tobin C. Harding · 4b4341b9 · Jun 8, 2026 · 1 fileMessage 68 · AdequateInformational 15Details
Commit message · Tobin C. Harding

units: Use little t in docs

We use locktime 'n' (lowercase) but time 'T' (uppercase) in docs.
There does not seem to be a reason, I think lowercase reads better.
This may be subjective.

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

This commit only changes the letter case of a variable name in documentation comments, from uppercase 'T' to lowercase 't'. It is a purely cosmetic documentation edit with no effect on code behavior or security.

Lower-priorityunits: Group private and public modules separatelyby Tobin C. Harding · a4b8391d · Jun 8, 2026 · 1 fileMessage 65 · AdequateInformational 15Details
Commit message · Tobin C. Harding

units: Group private and public modules separately

As is customary group the private modules above the public ones.

65/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
AI analysis · Informational 15/100

This commit simply reorders module declarations in a Rust source file so that private modules are listed before public ones. It makes no functional changes to the code and has no security relevance.

Lower-priorityunits: no_inline errorby Tobin C. Harding · 322e49da · Jun 8, 2026 · 1 fileMessage 43 · ThinInformational 15Details
Commit message · Tobin C. Harding

units: no_inline error

As is customary here, use `no_inline` on error re-export.

43/100 · ThinMessage clarity
✓ Subject identifies a change✓ Provides an explanatory body
AI analysis · Informational 15/100

This is a one-line documentation-only change in the Rust Bitcoin library. It adds a Rust attribute that prevents a re-exported error type from being shown as inlined in generated API documentation. There is no change to executable code, behavior, or security.

Lower-priorityunits: Fix fee_rate docsby Tobin C. Harding · e0cbd282 · Jun 8, 2026 · 1 fileMessage 43 · ThinInformational 15Details
Commit message · Tobin C. Harding

units: Fix fee_rate docs

Users get unit using the `to_sat_per` family of functions not
`from_sat`.

43/100 · ThinMessage clarity
✓ Subject identifies a change✓ Provides an explanatory body
AI analysis · Informational 15/100

This is a one-word documentation fix in a Rust Bitcoin library comment. It corrects a typo in the user-facing documentation for the FeeRate type, changing 'from_sat_per*' to 'to_sat_per*' so the described function names match the actual API. There is no code change and no security impact.

Lower-priorityunits: Remove unnecessary unit testsby Tobin C. Harding · 46abb61b · Jun 8, 2026 · 1 fileMessage 70 · AdequateInformational 15Details
Commit message · Tobin C. Harding

units: Remove unnecessary unit tests

We have two unit tests that test that we call through from a
deprecated function to its replacement. Remove them.

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

This commit simply deletes two unit tests that checked deprecated functions still forwarded to their replacements. No production code was changed, so it cannot introduce or fix a security issue on its own.

AI review queuedunits: Remove all deprecated codeby Tobin C. Harding · f2618b6c · Jun 8, 2026 · 9 filesMessage 60 · AdequateInformational 19Details
Commit message · Tobin C. Harding

units: Remove all deprecated code

In preparation for releasing `v1.0.0` remove all deprecated code.

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

This commit removes old, deprecated function names and type aliases from the rust-bitcoin `units` crate in preparation for a 1.0.0 release. It is a routine API cleanup: no security bug is fixed and no new unsafe code is added. Downstream projects that still use the removed names will get compile-time errors and must switch to the replacement APIs.

Lower-priorityUpdate the API text filesby Tobin C. Harding · 05855fee · Jun 8, 2026 · 3 filesMessage 45 · ThinInformational 15Details
Commit message · Tobin C. Harding

Update the API text files

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit only updates generated text files that list the public API surface of the library. No actual source code, logic, or behavior changed. It is not a security fix and does not introduce or remove any vulnerability.