rust-bitcoin

This commit is a routine code reorganization. It moves a small helper macro that implements conversions for script push bytes from one internal file to a shared include directory, then pulls it back in with no functional changes. There is no security issue here.

This commit is a routine internal code reorganization. It moves the PushBytes family of types and helpers from the main bitcoin crate into a lower-level primitives crate, and adjusts re-exports and trait implementations accordingly. There is no indication of a security bug being fixed or introduced.

This commit only updates two generated API snapshot files that list what functions and types are publicly exposed by the library. No actual source code was changed, and the commit message simply says 'Update API files.' There is no indication of a security fix or vulnerability.

This commit only adds new test code for existing PushBytes types. It does not change any production code, fix a bug, or alter behavior. There is no security issue here.

This is a routine code cleanup commit that fixes compiler lint warnings. It removes an unused import, simplifies how a borrowed reference is obtained, makes a large numeric constant more readable with underscores, and adds a missing semicolon. None of these changes affect program behavior or fix any security issue.

This commit only changes build configuration files (Cargo.toml and lock files) to add a new internal crate dependency. It does not modify any executable code, fix a bug, or address a security issue. It is a routine refactoring step to resolve a dependency direction problem during library development.

This commit adds a standard Rust trait implementation that lets two internal error types be created from Rust's Infallible type. Infallible is a type that can never actually exist, so this is a purely mechanical, safe change that makes the library's error types consistent with the project's conventions. It does not fix a vulnerability or change any behavior an attacker could exploit.

This commit is a routine internal code reorganization. It moves two existing helper methods (read_scriptint and read_cltv_scriptint) out of the PushBytes type into a Rust 'extension trait', and turns their shared internal helper into a plain private function. No behavior changes, bug fixes, or security-sensitive logic changes are visible in the diff.

This commit is purely a code-formatting cleanup. It re-indents lines, breaks long statements across multiple lines, and reorders imports to satisfy Rust style rules. No program logic, behavior, or security properties were changed.

This commit is a straightforward internal refactoring of the rust-bitcoin library's decoding machinery. It replaces a plain true/false return value from decoder methods with a new named DecoderStatus enum (NeedsMore / Ready). The change is purely about making the code easier to read and maintain; it does not alter what data is accepted, rejected, or how it is parsed. There is no indication this fixes a security bug or introduces a vulnerability.

This commit only updates documentation comments in a Rust Bitcoin encoding/decoding library. It rewrites explanations to match a recent API change where methods now return named enum values (like EncoderStatus::Finished) instead of plain true/false booleans. No actual code logic was changed, so there is no security impact.

This commit only updates generated API snapshot files (text listings of public functions/types). It adds two new public enum types, DecoderStatus and EncoderStatus, and changes the return type of several Decoder/Encoder methods from bool to those new enums. There is no source code change shown, no bug fix, and no security-relevant behavior described.

This commit is a routine code cleanup that fixes compiler lint warnings. It rewrites some pattern-matching to use shorter syntax, adds missing semicolons, converts a manual panic into an assertion, and reorganizes imports and test code behind feature flags. There is no functional change that affects security.

This commit only adds new unit tests and one test-tooling exclusion. It does not change any production code, so it cannot introduce or fix a security vulnerability on its own.

This commit is a straightforward code cleanup: it replaces a true/false return value from an encoder's 'advance' method with a clearly-named enum (HasMore/Finished). The behavior of the code is unchanged; it only becomes easier to read and maintain. There is no security fix or vulnerability here.

This commit is a test-only refactor. It changes how encoding is verified in unit tests: instead of checking the exact byte chunks an encoder produces one at a time, tests now only check the final concatenated byte output. It also deletes tests that poked at internal encoder details and updates documentation to say chunk sizes are not guaranteed. There is no change to production code behavior.

This commit simply removes duplicate unit tests from a Rust Bitcoin encoding library. The deleted tests covered the same compact-size encoding behavior that is already tested elsewhere, so no code logic changed and there is no security impact.

This commit adds new test-only helper functions that let developers check whether an encoder produces the expected bytes without needing to allocate memory. It also updates existing compact-size tests to use these helpers, making the tests simpler and usable in no-allocation environments. There is no change to production behavior or security-sensitive logic.

This is a small code cleanup in test files only. It replaces hand-written loops with existing helper functions that do the same thing. There is no security issue and no change to production code.

This is a small internal cleanup in the Rust Bitcoin library. It removes an unnecessary use of Option (a type that can hold either a value or nothing) from a compact-size integer encoder. The change simplifies the code but does not fix a security bug or change observable behavior.

This commit refactors how BIP-32 child key indexes are represented in the rust-bitcoin library. Previously, the index was a public enum with separate fields for normal and hardened variants, and the commit message notes a FIXME: because those fields were public, callers could directly create an out-of-range index (for example, a 'normal' index with the high bit set). The change turns the type into a private newtype around a raw u32 and forces construction through checked constructors that validate the range. It is a defensive hardening change that closes a potential invariant violation, but it does not by itself fix a demonstrated exploit.

This commit adds a scheduled GitHub Actions workflow that runs once a week to check for new releases of an internal maintainer tool (cargo-rbmt). If a newer release exists, it automatically opens a pull request updating a version-tracking file. There is no user-facing code change, no bug fix, and no security-sensitive behavior introduced.

This commit only updates generated API listing files (text snapshots of what functions and types are publicly exposed). It adds documentation of a new error type and a new 'try_push' method for an internal array vector, but it does not change any actual source code or behavior. There is no security issue here.

This commit fixes a deserialization bug in an internal array-like data structure. Previously, if untrusted serialized data claimed to contain more items than the structure could hold, the code would panic (crash) instead of returning a proper error. The fix makes deserialization gracefully reject oversized input.

This commit adds a new, safer way to insert items into a fixed-size array-like container called ArrayVec. The existing method panics when full; the new method returns a normal error instead. It does not change any existing behavior and does not fix a known security bug.