Skip to content

fix: restore CI workflow and smoke tests deleted by bulk revert#297

Merged
lbartoszcze merged 1 commit intomainfrom
wisent/restore-ci
Feb 9, 2026
Merged

fix: restore CI workflow and smoke tests deleted by bulk revert#297
lbartoszcze merged 1 commit intomainfrom
wisent/restore-ci

Conversation

@lbartoszcze
Copy link
Copy Markdown
Contributor

Summary

  • Restores CI workflow (.github/workflows/ci.yml) and smoke test suite (tests/test_smoke.py) that were accidentally deleted by the bulk revert in Revert all ecosystem improver agent changes #294
  • The revert removed 366 agent-created files including the CI infrastructure, leaving the repo with zero quality gates
  • All 17 smoke tests pass locally in <1s — they test imports and dataclass instantiation, no API keys needed

Changes

  • .github/workflows/ci.yml — lint (ruff) + test (Python 3.10/3.11/3.12) + build, with timeout-minutes: 10 to prevent CI hangs
  • tests/test_smoke.py — 17 smoke tests covering imports, skill base classes, cognition dataclasses, filesystem skill
  • tests/__init__.py — empty init for test package
  • pyproject.toml — restored [tool.pytest.ini_options], added pytest-timeout>=2.0.0 to dev deps with 30s default timeout

Test plan

  • All 17 smoke tests pass locally (pytest tests/ -v --tb=short --timeout=30)
  • Lint passes (ruff check singularity/ --select E,F --ignore E501,E402,E721,E731,E741,F401,F841,F541)
  • CI should pass on this PR (first CI run since the revert)

🤖 Generated with Claude Code

The revert in #294 removed 366 agent-created files, but also deleted the
CI workflow and smoke test suite that were needed for quality gates.

Restores:
- .github/workflows/ci.yml (lint + test + build)
- tests/test_smoke.py (17 smoke tests, no API keys needed)
- [tool.pytest.ini_options] in pyproject.toml

Improvements over original:
- Added timeout-minutes: 10 to test jobs to prevent CI hangs
- Added pytest-timeout>=2.0.0 to dev deps with 30s per-test default
- Added --timeout=30 flag to pytest invocation in CI

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@lbartoszcze lbartoszcze merged commit bda4b16 into main Feb 9, 2026
5 checks passed
lbartoszcze added a commit that referenced this pull request Feb 9, 2026
lbartoszcze added a commit that referenced this pull request Feb 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant