Self review checklist
Skill theinterneti/TTA.dev/.claude/skills/self-review-checklist
Use this skill before submitting code for review or merging a pull request. Covers a pre-merge checklist of common mistakes to catch.From its SKILL.md
npx -y skills add theinterneti/TTA.dev --skill self-review-checklistAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
One thing to look at
- 0 stars0 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.
SKILL.md
1.7 KB, 394 tokens by cl100k_base, as published. Nobody here has run it
Self-Review Checklist (TTA.dev)
Run through this checklist before marking any task or PR as complete.
Code Quality
- All new functions have type annotations (
str | None, notOptional[str]) - All public functions have Google-style docstrings
- No manual retry/timeout/cache loops — use primitives
- State passed via
WorkflowContext, not globals - Import order: stdlib → third-party → local
Testing
- 100% coverage on new code
- Tests follow AAA pattern (Arrange-Act-Assert)
- Uses
MockPrimitivefor mocking (not real implementations) - Uses
@pytest.mark.asyncioon async tests - Tests cover success, failure, and edge cases
- No external dependencies in tests (databases, APIs, filesystem)
Security
- No secrets or credentials in code
- No new
eval(),exec(), orsubprocess.shell=True - URL validation uses
urlparse(), not substring checks
Quality Gate
-
uv run ruff format --check .— zero violations -
uv run ruff check .— zero violations -
uvx pyright platform/— zero errors -
uv run pytest -v— all tests pass
Documentation
- README updated if public API changed
- Code examples are copy-paste runnable
- Changed files have updated docstrings
Deep Reference
- Testing details:
docs/agent-guides/testing-architecture.md - Python standards:
docs/agent-guides/python-standards.md
What ships with it
Read from the repository
Just SKILL.md. No reference files, no scripts.