fix: Skip contract tests in template context

[gmelli]

Problem

CI failing on main branch with 7 test failures:

• test_identity_contract.py (3 tests)
• test_wake_contract.py (4 tests)

Root cause: Contract tests expect .aget/version.json to exist, but template repositories don't have this file until aget init runs. Tests designed for post-initialization validation were running in pre-initialization (template) context.

Solution

Added @pytest.mark.skipif decorators to all 7 contract tests:

@pytest.mark.skipif(not Path(".aget/version.json").exists(),
                    reason="Post-init contract test, skip for template context")

Behavior:

• ✅ Tests SKIP when version.json missing (template context) → CI passes
• ✅ Tests RUN when version.json exists (initialized agents) → Validation works

Changes

Test Fixes

1. test_identity_contract.py - Added skipif to 3 tests
2. test_wake_contract.py - Added skipif to 4 tests

Linting Improvements

Applied ruff auto-fixes:

• Removed unused imports (F401)
• Fixed f-strings without placeholders (F541)
• Removed unused variables (F841)
• Fixed boolean comparisons (E712)

Result: 144 violations auto-corrected, 75 remain (non-blocking)

Verification

Tested locally:

# With version.json present
pytest tests/test_*_contract.py -v
# Result: 7 passed ✅

# With version.json absent (template mode)
pytest tests/test_*_contract.py -v
# Result: 7 skipped ✅

Impact

• ✅ CI will pass on template repository
• ✅ Contract tests still validate initialized agents
• ✅ No behavioral changes to production code
• ✅ Improves code quality with linting fixes

---

Type: Bug fix + Code quality
Priority: High (blocks template releases)
Breaking: No