fix: resolve ruff lint errors in validation.py and test_validation.py

[Copilot]

CI was failing due to 6 ruff lint errors introduced with the new validation.py and its test file.

Changes

• B905 — Added strict=False to three zip() calls (lines 319, 367, 395 in validation.py) where input/output list lengths may legitimately differ
• B007 — Renamed unused loop variable i → _i in the enumerate(zip(...)) pairing loop
• W292 — Added missing trailing newline at EOF in validation.py
• F401 — Removed unused import pytest from tests/test_validation.py

[Copilot]

Pull request overview

This PR fixes CI failures by addressing Ruff lint issues introduced in the new validation tool and its unit tests.

Changes:

• Add explicit strict=False to zip() calls to satisfy B905.
• Rename an unused loop index variable and remove an unused test import to satisfy B007/F401.
• Ensure the source file ends with a trailing newline (W292).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
tests/test_validation.py	Removes an unused pytest import to satisfy Ruff (F401).
src/autocode_mcp/tools/validation.py	Updates zip() calls to be explicit about strict, renames an unused loop var, and fixes trailing newline.

---

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

[Copilot]

Here you already guard on len(actual_tokens) == len(expected_tokens), so using zip(..., strict=False) is redundant and slightly undermines the intent of being explicit about length mismatches. Consider strict=True (or rely on strict=True and drop the manual length check) to keep this float-compare path self-consistent.

Suggested change

	return all(abs(a - e) < tolerance for a, e in zip(actual_nums, expected_nums, strict=False))
	return all(abs(a - e) < tolerance for a, e in zip(actual_nums, expected_nums, strict=True))

[Copilot]

_i is unused in this loop; since the index isn’t used, dropping enumerate(...) entirely (or using _ instead of _i) would make the intent clearer and avoid carrying an unused variable.

Suggested change

	for _i, (inp, out) in enumerate(zip(input_blocks, output_blocks, strict=False)):
	for inp, out in zip(input_blocks, output_blocks, strict=False):