Self review checklist
Skill theinterneti/TTA.dev/.github/skills/self-review-checklist
AI devops for vibe coders
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.
What its author says it does
Copied from the file, not written here
Use this skill before submitting code for review or merging a pull request. Covers a pre-merge checklist of common mistakes to catch across code quality, testing, security, and documentation. Invoke when the user says "review my code", "pre-merge check", "self-review", "am I ready to submit", or before opening a PR.
SKILL.md
1.9 KB, 396 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 ttadev/— 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