Skip to content

Conversation

Roasbeef
Copy link
Member

In this PR, we add a README for a new area of documentation intended
for active developers of the project. The goal of this new set of docs
is to capture all the hard earned wisdom that devs gain while debugging
a non-trivial issue. It's meant to serve as a run book to allow for
rapid pattern matching to identify a routine issue, along with a guided
path to aide in debugging.

This should also be great for agents as well, since they can use them to
locate issues that they would otherwise need some prodding to figure
out.

The first one added is intended to devs find PSBT signing related issues more quickly. I ran into this during the dev of #1675 to get the itests working properly. It relies on some deeper knowledge of: PSBTs, the way lnd signs them, where the code lives, the logs to watch for, etc. So I figured I'd document it.

I also made a sub-agent that can be useful for such tasks.

In this commit, we add a README for a new area of documentation intended
for active developers of the project. The goal of this new set of docs
is to capture all the hard earned wisdom that devs gain while debugging
a non-trivial issue. It's meant to serve as a run book to allow for
rapid pattern matching to identify a routine issue, along with a guided
path to aide in debugging.

This should also be great for agents as well, since they can use them to
locate issues that they would otherwise need some prodding to figure
out.
@Roasbeef Roasbeef added documentation Improvements or additions to documentation debugging labels Aug 15, 2025
@coveralls
Copy link

Pull Request Test Coverage Report for Build 16980805330

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 8890 unchanged lines in 124 files lost coverage.
  • Overall coverage decreased (-8.1%) to 48.556%

Files with Coverage Reduction New Missed Lines %
authmailbox/client.go 2 69.84%
commitment/proof.go 2 87.29%
fn/iter.go 2 62.07%
fn/retry.go 2 92.5%
rpcserver.go 2 61.34%
tapdb/interfaces.go 2 78.33%
tapsend/proof.go 2 85.99%
universe/archive.go 2 79.29%
universe_rpc_diff.go 2 76.0%
universe/syncer.go 2 82.73%
Totals Coverage Status
Change from base Build 16968929223: -8.1%
Covered Lines: 51720
Relevant Lines: 106517

💛 - Coveralls

@Roasbeef
Copy link
Member Author

Roasbeef commented Sep 3, 2025

Landing this as it's purely additive docs.

@Roasbeef Roasbeef merged commit 92aec10 into lightninglabs:main Sep 3, 2025
17 of 19 checks passed
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in Taproot-Assets Project Board Sep 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
debugging documentation Improvements or additions to documentation
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants