Steve Yegge 1d4eb6d94c refactor: Consolidate duplicated step collection functions in cook.go (bd-9btu) ...

Replaced three near-identical functions with a single unified collectSteps()
that uses a callback strategy for label handling:

- Removed collectStepsRecursive() (DB version)
- Removed collectStepsToSubgraph() (in-memory version)
- Removed collectStepIDMappings() (now handled by unified function)

The new collectSteps() function:
- Takes optional labelHandler callback for DB path (extracts labels separately)
- Takes optional issueMap for in-memory path
- Always builds idMapping for dependency resolution
- Eliminates ~60 lines of duplicated code

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

2025-12-28 16:15:17 -08:00

28 KiB

Raw Blame History

View Raw