ci(lint): ignore false positive, nonce is [16]byte

[wolf31o2]

Summary by CodeRabbit

• Chores
  • Internal code quality improvements to the mining module.

Note: No user-facing changes in this release.

[coderabbitai]

📝 Walkthrough

Walkthrough

A nolint:gosec directive was added to the nonce incrementation path within the calculateHash function in internal/miner/miner.go. This is a linting annotation only; no functional logic, control flow, or behavioral changes were made to the code. The byte-wise nonce increment logic remains unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

• Single file modification with only a linting directive addition
• No behavioral or logic changes to review
• Homogeneous change (consistent directive pattern)

Pre-merge checks and finishing touches

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The PR title "ci(lint): ignore false positive, nonce is [16]byte" is directly related to the main change in the changeset, which adds only a nolint:gosec directive to the nonce incrementation path in internal/miner/miner.go. The title is specific and clear, using conventional commit formatting (ci: scope for CI-related changes about linting), and it explicitly states the purpose (ignore false positive) along with relevant context (nonce is [16]byte). It avoids vague terminology and provides meaningful information about the change without unnecessary details.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch ci/lint-ignore-false-positive

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e3145c7 and 9f7c767.

📒 Files selected for processing (1)

• internal/miner/miner.go (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Analyze (go)

🔇 Additional comments (1)

internal/miner/miner.go (1)

404-404: LGTM! The nolint directive is appropriately placed.

The gosec warning is indeed a false positive. Byte overflow during nonce incrementation is expected and correct behavior in this proof-of-work mining context, where the nonce is exhaustively searched through its entire value space.

---

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