UNPKG

aios-core

Version:

Synkra AIOS: AI-Orchestrated System for Full Stack Development - Core Framework

109 lines (72 loc) 3.35 kB
# 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.