johno/beads
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor(mol): remove redundant guard in showCompoundBondingInfo

Browse Source 
The caller already checks IsCompound() before calling, so the internal
guard was unnecessary. Added doc comment clarifying the precondition.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
beads/crew/grip
2026-01-02 14:08:13 -08:00
committed by Steve Yegge
parent 53790557de
commit 73ed489c9e
1 changed files with 2 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
cmd/bd/mol_show.go
View File

@@ -104,12 +104,9 @@ func showMolecule(subgraph *MoleculeSubgraph) {
fmt.Println() fmt.Println()
} }
// showCompoundBondingInfo displays the bonding lineage for compound molecules // showCompoundBondingInfo displays the bonding lineage for compound molecules.
// Caller must ensure root.IsCompound() is true.
func showCompoundBondingInfo(root *types.Issue) { func showCompoundBondingInfo(root *types.Issue) {
if !root.IsCompound() {
return
}
constituents := root.GetConstituents() constituents := root.GetConstituents()
fmt.Printf("\n%s Bonded from:\n", ui.RenderAccent("🔗")) fmt.Printf("\n%s Bonded from:\n", ui.RenderAccent("🔗"))