fix: remove unused cleared variable causing lint failure

[Copilot]

The CI lint job was failing due to a declared-but-never-read variable cleared in internal/difc/sink_server_ids.go. The variable was assigned true in the empty-input early-return branch but the function returns immediately after, so the value was never consumed.

Change

• internal/difc/sink_server_ids.go: Remove cleared bool from the var block and drop its assignment — the existing logSink.Print call already communicates the clearing behaviour.

// Before
var (
    cleared       bool
    normalizedOut []string
    duplicateIDs  []string
)
// ...
sinkServerIDs = nil
cleared = true  // assigned, never read

// After
var (
    normalizedOut []string
    duplicateIDs  []string
)
// ...
sinkServerIDs = nil

Original prompt

Fix the failing GitHub Actions workflow lint
Analyze the workflow logs, identify the root cause of the failure, and implement a fix.
Job ID: 67273885685
Job URL: https://github.com/github/gh-aw-mcpg/actions/runs/23156873563/job/67273885685

---

💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

[Copilot]

Pull request overview

This PR fixes a CI lint failure caused by a declared-but-never-read variable cleared in internal/difc/sink_server_ids.go. The variable was assigned but its value was never consumed before the function returned.

Changes:

• Removed the unused cleared bool declaration and its assignment in SetSinkServerIDs.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.