test: remove stdlib mocks, strengthen integration coverage #300
Workflow file for this run
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| name: Claude Code Review | |
| on: | |
| pull_request: | |
| types: [opened, synchronize] | |
| # Optional: Only run on specific file changes | |
| # paths: | |
| # - "src/**/*.ts" | |
| # - "src/**/*.tsx" | |
| # - "src/**/*.js" | |
| # - "src/**/*.jsx" | |
| jobs: | |
| claude-review: | |
| # Only run for organization members and collaborators | |
| if: | | |
| github.event.pull_request.author_association == 'OWNER' || | |
| github.event.pull_request.author_association == 'MEMBER' || | |
| github.event.pull_request.author_association == 'COLLABORATOR' | |
| runs-on: ubuntu-latest | |
| permissions: | |
| contents: read | |
| pull-requests: write | |
| issues: read | |
| id-token: write | |
| steps: | |
| - name: Checkout repository | |
| uses: actions/checkout@v4 | |
| with: | |
| fetch-depth: 1 | |
| - name: Run Claude Code Review | |
| id: claude-review | |
| uses: anthropics/claude-code-action@v1 | |
| with: | |
| claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }} | |
| github_token: ${{ secrets.GITHUB_TOKEN }} | |
| track_progress: true # Enable visual progress tracking | |
| allowed_bots: '*' | |
| prompt: | | |
| Review this Basic Memory PR against our team checklist: | |
| ## Code Quality & Standards | |
| - [ ] Follows Basic Memory's coding conventions in CLAUDE.md | |
| - [ ] Python 3.12+ type annotations and async patterns | |
| - [ ] SQLAlchemy 2.0 best practices | |
| - [ ] FastAPI and Typer conventions followed | |
| - [ ] 100-character line length limit maintained | |
| - [ ] No commented-out code blocks | |
| ## Testing & Documentation | |
| - [ ] Unit tests for new functions/methods | |
| - [ ] Integration tests for new MCP tools | |
| - [ ] Test coverage for edge cases | |
| - [ ] Documentation updated (README, docstrings) | |
| - [ ] CLAUDE.md updated if conventions change | |
| ## Basic Memory Architecture | |
| - [ ] MCP tools follow atomic, composable design | |
| - [ ] Database changes include Alembic migrations | |
| - [ ] Preserves local-first architecture principles | |
| - [ ] Knowledge graph operations maintain consistency | |
| - [ ] Markdown file handling preserves integrity | |
| - [ ] AI-human collaboration patterns followed | |
| ## Security & Performance | |
| - [ ] No hardcoded secrets or credentials | |
| - [ ] Input validation for MCP tools | |
| - [ ] Proper error handling and logging | |
| - [ ] Performance considerations addressed | |
| - [ ] No sensitive data in logs or commits | |
| ## Compatability | |
| - [ ] File path comparisons must be windows compatible | |
| - [ ] Avoid using emojis and unicode characters in console and log output | |
| Read the CLAUDE.md file for detailed project context. For each checklist item, verify if it's satisfied and comment on any that need attention. Use inline comments for specific code issues and post a summary with checklist results. | |
| # Allow broader tool access for thorough code review | |
| claude_args: '--allowed-tools "Bash(gh pr:*),Bash(gh issue:*),Bash(gh api:*),Bash(git log:*),Bash(git show:*),Read,Grep,Glob"' |