fix(worker): remove dead break in nats consumer select

[mesutoezdil]

staticcheck flags a dead break in the nats consumer select loop.

Removed two break statements that had no effect.

´Testing
go build, go vet, go test all pass.

Closes #360

[coderabbitai]

📝 Walkthrough

Walkthrough

Changes

NATS consumer fetch loop

Layer / File(s)	Summary
Fetch loop control flow src/libraries/go/worker/consumer/nats_consumer.go	Removes two ineffective break statements from message receipt and forwarding selects while retaining closed-message and context-cancellation handling.

Estimated code review effort: 2 (Simple) | ~5 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The title is well-formed, but 'fix' overstates this no-behavior-change cleanup; it is a refactor/chore to remove dead breaks and silence SA4011.	Retitle it as 'refactor(worker): remove dead break in nats consumer select' or 'chore(worker): remove dead break in nats consumer select'.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	The change removes the two dead break statements in nats_consumer.go and matches issue #360's SA4011 fix.
Out of Scope Changes check	✅ Passed	No unrelated changes are present; the diff is limited to the dead-break cleanup in the targeted consumer file.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Comment @coderabbitai help to get the list of available commands.}