aios-core
Version:
Synkra AIOS: AI-Orchestrated System for Full Stack Development - Core Framework
109 lines (72 loc) • 3.35 kB
Markdown
# Pre-Push Quality Gate Checklist
## Instructions for DevOps Agent
Before pushing code to the remote repository, go through each item in this checklist. Report the status of each item (e.g., [x] Done, [ ] Not Done, [N/A] Not Applicable) and provide brief comments if necessary.
[[LLM: INITIALIZATION INSTRUCTIONS - PRE-PUSH VALIDATION
This checklist is for DEVOPS AGENTS validating code before pushing to remote.
IMPORTANT: Pushing broken code affects the entire team. This checklist ensures quality.
EXECUTION APPROACH:
1. Run all automated checks first
2. Mark items as [x] Done, [ ] Not Done, or [N/A] Not Applicable
3. Add comments for any failures or concerns
4. Block push if any critical items fail
5. Get user confirmation before pushing
Quality gates protect everyone.]]
## Checklist Items
1. **Code Quality:**
[[LLM: Code must pass all quality checks]]
- [ ] `npm run lint` passes with no errors
- [ ] `npm run typecheck` passes (if TypeScript)
- [ ] `npm run build` completes successfully
- [ ] No console.log or debugging statements in production code
- [ ] No commented-out code blocks
2. **Testing:**
[[LLM: Tests prove the code works]]
- [ ] `npm test` passes with all tests green
- [ ] No skipped tests without documented reason
- [ ] Test coverage meets project standards
- [ ] New features have corresponding tests
- [ ] No flaky tests introduced
3. **Security:**
[[LLM: Security vulnerabilities must be caught before push]]
- [ ] No hardcoded credentials or secrets
- [ ] No API keys or tokens in code
- [ ] `.env` files not staged for commit
- [ ] `npm audit` shows no high/critical vulnerabilities
- [ ] No sensitive data in logs or error messages
4. **CodeRabbit Review:**
[[LLM: Automated code review catches issues humans miss]]
- [ ] CodeRabbit review executed (if available)
- [ ] No CRITICAL issues identified
- [ ] HIGH issues addressed or documented
- [ ] Security concerns resolved
- [ ] Performance issues noted
5. **Git State:**
[[LLM: Clean git state prevents merge problems]]
- [ ] Working directory is clean (no uncommitted changes)
- [ ] All intended changes are committed
- [ ] Commit messages follow conventional commits format
- [ ] No merge conflicts pending
- [ ] Branch is up to date with target branch
6. **Story Compliance:**
[[LLM: Code should match story requirements]]
- [ ] All story acceptance criteria met
- [ ] Story status is "Ready for Review" or "Done"
- [ ] File List in story is complete
- [ ] No scope creep (only story-related changes)
- [ ] Story checkboxes updated
7. **Documentation:**
[[LLM: Changes should be documented]]
- [ ] README updated if needed
- [ ] API documentation updated if endpoints changed
- [ ] Breaking changes documented
- [ ] Changelog entry prepared (if release)
## Final Confirmation
[[LLM: FINAL PRE-PUSH SUMMARY
After completing the checklist:
1. Summarize what is being pushed
2. List any items marked as [ ] Not Done with explanations
3. Confirm all critical gates pass
4. Note any concerns for reviewers
5. Get user approval before executing push
Only push when confident in quality.]]
- [ ] I, the DevOps Agent, confirm that all quality gates have passed and the code is safe to push to remote.