Repository navigation
api: Stop exposing Bitcoin Core descriptor records #63
Copy link
Copy link
Closed as duplicate
Labels
area: apiPublic and supported Python API boundaries.Public and supported Python API boundaries.area: wallet/coreWallet integration and Bitcoin Core boundaries.Wallet integration and Bitcoin Core boundaries.gate: adversarial reviewResolve, merge, or explicitly defer before the next full adversarial review.Resolve, merge, or explicitly defer before the next full adversarial review.
Description
Activity
- addedgate: adversarial reviewResolve, merge, or explicitly defer before the next full adversarial review.Resolve, merge, or explicitly defer before the next full adversarial review.area: wallet/coreWallet integration and Bitcoin Core boundaries.Wallet integration and Bitcoin Core boundaries.area: apiPublic and supported Python API boundaries.Public and supported Python API boundaries.
on Sep 28, 2026 - added 7 commits that reference this issue
on Oct 1, 2026 Duplicate of #128: both are fixed by the same removal in #130. This issue's checklist is folded into #128.
Generated by Claude Code
- added 2 commits that reference this issue
on Oct 8, 2026
Metadata
Metadata
Assignees
Labels
area: apiPublic and supported Python API boundaries.Public and supported Python API boundaries.area: wallet/coreWallet integration and Bitcoin Core boundaries.Wallet integration and Bitcoin Core boundaries.gate: adversarial reviewResolve, merge, or explicitly defer before the next full adversarial review.Resolve, merge, or explicitly defer before the next full adversarial review.
Follow-up to the review question on #12 and the v1 API-boundary pass.
core_descriptors(...)is a legacy public adapter from before Bitcoin Core became the sole EC/HD-key integration. The required v1 cleanup is:core_descriptorsfrom package-levelcodex32.__all__and supported public documentation;wallet.py;master_xprvunchanged;Focused PR #130 supersedes #64. It removes the obsolete descriptor API plus the likewise-unused fresh Core Lightning generation exports, leaving the package-level
codex32.__all__at 22 names while preserving parsing/recovery/re-sharing of existing CL secrets. Current head2cbd279has a green Python matrix and real-Core fixture, its one Codex documentation finding is resolved, and Codex reports no major issue on that exact head.No automated/code-review gap remains. Keep this issue open through responsible-human review and integration of #130.