What the developer wrote
Authored by Elle Mouton
graph: remove outdated TODO
This todo has been addressed.
What changed, and why it matters
This commit simply removes a three-line comment (a TODO note) from a Go source file. No code behavior changed, no bug was fixed, and no security issue is present in the diff.
No action needed; this is a non-functional cleanup commit.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit deletes a documentation TODO in graph/builder.go stating that funding-transaction validation should be moved from addEdge to the gossiper. The surrounding code and function signatures are unchanged. There is no functional modification.
Changed components
graph/builder.goInspect captured patch +0 / −3
diff --git a/graph/builder.go b/graph/builder.go
index 5cfb79e..08016dd 100644
--- a/graph/builder.go
+++ b/graph/builder.go
@@ -1037,9 +1037,6 @@ func (b *Builder) AddEdge(ctx context.Context, edge *models.ChannelEdgeInfo,
// Chain View is updated with the new edge if it is successfully added to the
// graph. We only persist the channel if we currently dont have it at all in
// our graph.
-//
-// TODO(elle): this currently also does funding-transaction validation. But this
-// should be moved to the gossiper instead.
func (b *Builder) addEdge(ctx context.Context, edge *models.ChannelEdgeInfo,
op ...batch.SchedulerOption) error {
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.