docs: inventory SQL port source history - #1286
Draft
Roasbeef wants to merge 2 commits into
Draft
Conversation
This was referenced Jul 11, 2026
In this commit, we freeze the SQL port source universe and record a concrete disposition for every commit before transplanting code onto the port-first branch. The inventory covers all 152 PRs and all 969 commits between the pinned master and sql-wallet revisions. It preserves primary authors and co-authors, records semantic exceptions to path-based classification, and keeps live GitHub attribution separate from reproducible snapshot verification. We treat the SQL schema as extraction material, route backend implementations into Stage 2 review, and defer retained transaction-status and replacement-history semantics. This leaves no path-only pick candidates in the inventory. We also capture the btcwallet API used by the pinned lnd checkout from clean detached worktrees and document why lnd sqldb/v2 remains a post-port dependency decision.
In this commit, we explicitly defer the transaction replacement query commit from the port-first extraction set. We also pin the decision in the disposition override tests and update the inventory totals.
Roasbeef
force-pushed
the
sql-port-stage0-inventory
branch
from
July 11, 2026 01:16
83b2dd9 to
7c63c51
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
In this PR, we freeze the full SQL port source universe before moving code onto the smaller port-first branch.
This draft is the review and attribution artifact for the salvage effort. It isn't part of the final port stack, so we can leave it unmerged once the concrete Stage 1 work has carried the required code and contributor history forward.
The inventory covers all 152 PRs in the series and all 969 commits between
masterandsql-wallet. Each commit retains its original SHA, author identity, changed paths, disposition, and destination cut. GitHub still identifies a canonical feeder for 107 commits; the remaining 862 are explicitly unresolved because repeated rebases and force-pushes removed the historical SHA links. We keep that uncertainty visible instead of guessing away contributor history.The PR also records the existing btcwallet API used by lnd and pins the surviving umbrella and feeder heads. The included generators make the PR inventory, commit map, and API baseline reproducible.
Verification:
go test ./scripts