chore: update Makefile

[mbevc1]

• Add help and print by default if no/wrong target is selected
• Add missing help comments
• Clean up unnecessary empty spaces

[claude]

Claude finished @mbevc1's task in 1m 42s —— View job

---

Review of PR #738

• Run static analysis checks
• Read and review Makefile changes
• Post review feedback

Static Analysis

go build ./... and go vet ./... both pass cleanly. No issues (Makefile-only change, as expected).

Review

Overall: Nice cleanup — adding help as default target, ## help comments on targets, the BIN variable, and clean/clean-cache targets are all good additions. Trailing whitespace cleanup is welcome too.

Remaining issues (1 inline comment posted):

1. Tab vs space alignment (lines 9-10): LDFLAGS and BIN use tabs before := while the rest of the variable assignments (lines 12-15) use spaces. Minor consistency nit — see inline suggestion.

Acknowledged from author's earlier feedback: .DEFAULT: help is intentional for catching misspelled/nonexistent targets (runs help instead of erroring) — this is distinct from .DEFAULT_GOAL which only sets what runs with bare make. Both serve a purpose here. Previous review comments about this being redundant were incorrect.

What looks good:

• Comprehensive .PHONY declaration covering all targets
• BIN variable used consistently in build, clean, and cli-docs
• Help target with awk-based ## comment parsing is a well-established Makefile pattern
• Trailing whitespace cleanup throughout

[mbevc1]

1. is not true and is relevant
2. sorted
3. sorted