refactor: replace hand-rolled string search with strings.Contains in status.go

[jpleva91]

Problem

internal/status/status.go implements contains() and searchString() functions that manually iterate over runes to check substring containment. This is a reimplementation of strings.Contains() from the standard library — unnecessary code that adds maintenance surface.

Acceptance Criteria

• Replace contains(s, substr) calls with strings.Contains(s, substr)
• Replace containsChitin(data) to use strings.Contains(string(data), "chitin")
• Remove the contains() and searchString() functions
• Verify strings is already imported (it is not — add the import)
• All existing tests still pass
• go vet ./internal/status/... passes

Hints

• File: internal/status/status.go
• Lines 137-154: containsChitin(), contains(), searchString()
• containsChitin is called by checkClaudeHooks() and checkCopilotHooks()
• This is a safe 1:1 replacement — strings.Contains has identical semantics

---

Generated by /forge fill

[jpleva91]

🧠 Triage complete

Tier B — Needs Planning. Issue requires scoping and decomposition before implementation.

Reason: triage error (defaulting to safe): API returned 404: {"type":"error","error":{"type":"not_found_error","message":"model: claude-3-haiku-20241022"},"request_id":"req_011CZrvJEbCFh6ao2jCK9Pkt"}
Confidence: 0.00

Powered by Octi Pulpo pipeline

[jpleva91]

Simple 1:1 refactor replacing hand-rolled string search with strings.Contains; low complexity, safe to automate.

---

Generated by Claude Code