feat(collector): implement panic recovery and crash-loop safety

[wazer24]

What

Addresses PR review feedback: replaces brittle string-sniffing exit-code logic with a typed sentinel error, sweeps remaining interface{} back to any, adds missing doc comments for the no-restart contract and unbounded panic logs, and fixes CI failures (govet shadow, missing import close paren, linter pre-build step).

Why

Fixes #54 review comments from @btwshivam

How

• Typed exit-code error: Introduced cli.ErrDaemonPanic sentinel; runStart assigns it via named return on panic recovery. cmd/kerno/main.go now uses errors.Is(err, cli.ErrDaemonPanic) instead of string-matching err.Error()[:6] == "daemon".
• interface{} → any: Reverted map[string]interface{} back to map[string]any in healthzHandler; changed r interface{} to r any in HandlePanic and HandleDaemonPanic.
• Doc comments: Added no-restart contract note on RunSafeCollectorGoroutine; added operator-responsible note for unbounded panic log files on HandlePanic.
• CI fixes: Renamed shadowed err to rootErr in runStart (govet); restored missing ) after import block; added go build ./... pre-build step before golangci-lint in ci.yml.

Testing

• go build ./... passes
• go test ./... passes
• go vet ./... passes
• golangci-lint run ./... passes
• Tested locally with: N/A — no behavioral changes, only error-typing and doc fixes

---

• N/A — pure docs/refactor (no BPF/collector/doctor logic changes)

Checklist

• PR title follows Conventional Commits (feat(core): implement panic recovery and crash-loop safety)
• All commits are DCO-signed (git commit -s)
• No unrelated changes pulled in
• Documentation updated where user-visible behavior changed
• Added/updated tests for new code paths
• If a new doctor rule, paired with a chaos scenario — N/A

[github-actions]

🚀 First PR — welcome aboard!

A few things to expect:

1. CI: every PR runs build + race tests + lint + (eventually) the kernel matrix. If something fails, the log will tell you exactly which gate.
2. DCO: every commit needs Signed-off-by: — git commit -s adds it automatically.
3. Conventional Commits: PR titles like feat(doctor): add new rule or fix(bpf): handle X. We squash-merge by default.
4. Review: a maintainer will review within 72 hours. Suggestions are conversations, not orders — push back if something doesn't fit your context.

If you get stuck, reply here or jump to Discussions. We want this PR to land.

[btwshivam]

duplicate of #159 (same 17 files), and lint plus conventional-title are red here. let's land #159 instead, please close this one. thanks.

[wazer24]

@btwshivam apologies for the delay but i fixed the remaining issues the solution of mine was having please review it,and tell me if anything needed to be changed thankyou