Skip to content

test: Category A twins that do not need Core functional - #483

Merged
reardencode merged 4 commits into
masterfrom
test/cat-a-twins
Sep 13, 2026
Merged

test: Category A twins that do not need Core functional#483
reardencode merged 4 commits into
masterfrom
test/cat-a-twins

Conversation

@rearden-grok

@rearden-grok rearden-grok Bot commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

Summary

#482 merged. Honest Category A folds that do not wait on Core functional:

  • Packed create_fk layout stays on input_encode_create_fk_not_prev_txid; dropped from mature reconstruct.
  • One testmempoolaccept script-reject mapping test (DER details + CLTV paren table).
  • Interpreter 1SUB/SHA1/SIZE twins of no_unknown_opcode_for_enabled_set deleted; keep OP_SUCCESS / VERIFY-then-OP_1 / P2SH-style.
  • connect_chain_query_surface no longer repeats mature height/locator/SH. Keep hash-index rebuild, outs-only output, merkle, cancel, idempotent confirm_block.

Left for later (see local/testing-audit.md): handshake log needles, decode_rpc_subset, RBF/cluster HTTP, BIP352 hash-bind, Core-functional-required workflow YAML, guts-shaped journeys (two_node tip-only, smoke lifecycle, BlockCache library).

Test plan

  • consensus_mature_chain_spend_reconstruct_and_scripthash
  • testmempoolaccept_script_reject_maps_reason_and_details
  • no_unknown_opcode_for_enabled_set / eval_sig_then_p2sh_style
  • connect_chain_query_surface / spend_edge_and_confirm_idempotent_path
  • Required CI green (fmt, deny, clippy, ast-grep, test, windows, macos, multinode, coverage)

Made with Cursor

rbitcoin-grok and others added 4 commits September 12, 2026 23:03
input_encode_create_fk_not_prev_txid already asserts the v10
layout. Keep create_fk on the spend input; do not re-encode.

Co-authored-by: Cursor <cursoragent@cursor.com>
Keep a single dialect pin (DER details plus CLTV paren map).
Core feature_dersig/cltv cover the rest when labeled.

Co-authored-by: Cursor <cursoragent@cursor.com>
1SUB/1ADD/NEGATE/ABS, SHA1, and empty SIZE already sit in
no_unknown_opcode_for_enabled_set. Keep OP_SUCCESS wrappers,
VERIFY-then-OP_1, and P2SH-style redeem parse.

Co-authored-by: Cursor <cursoragent@cursor.com>
Mature reconstruct already pins height_of_hash, locator, and SH.
Keep hash-index rebuild, outs-only output, merkle, cancel, and
idempotent confirm_block.

Co-authored-by: Cursor <cursoragent@cursor.com>
@reardencode
reardencode merged commit d14263e into master Sep 13, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant