Skip to content

fix: resolve CI lint and mypy failures#258

Merged
glebis merged 1 commit intomainfrom
fix/ci-lint-mypy
Feb 20, 2026
Merged

fix: resolve CI lint and mypy failures#258
glebis merged 1 commit intomainfrom
fix/ci-lint-mypy

Conversation

@glebis
Copy link
Copy Markdown
Owner

@glebis glebis commented Feb 20, 2026

Summary

  • Fix 45 mypy type errors across 7 files (pre-existing, not from recent PRs)
  • Fix Black formatting on 4 beads files
  • Remove unused imports flagged by flake8 across 8 test files
  • Fix actual bug: current_streak=streak= kwarg in accountability_app.py:208
  • Add isort check to pre-commit hook (was in CI but missing from hook)

Test plan

  • mypy src/ passes with 0 errors (was 45)
  • black --check src/ tests/ passes
  • flake8 src/ tests/ passes
  • isort --check src/ tests/ passes
  • pytest tests/ — 4493 tests pass

🤖 Generated with Claude Code

Fix pre-existing CI failures:
- value_objects.py: add type annotations for __slots__ attributes
- beads_service.py: fix returncode int|None, return type mismatches
- formatting.py: use typing.Any instead of builtin any
- sqlalchemy_message_repository.py: type-ignore rowcount attr
- message_persistence_service.py: use Any instead of object for repos
- beads_commands.py: cast priority to int for dict lookup
- accountability_app.py: fix bug — current_streak= → streak= kwarg
- Remove unused imports across test files
- Black-format 4 beads files
- Add isort check to pre-commit hook

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@glebis glebis merged commit 1082bbf into main Feb 20, 2026
4 of 5 checks passed
@glebis glebis deleted the fix/ci-lint-mypy branch February 22, 2026 23:31
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