docs: align Homebrew workflow with automation and add CLAUDE.md#12
docs: align Homebrew workflow with automation and add CLAUDE.md#12
Conversation
Add automation guidelines and fix documentation inconsistencies: 1. Update CONTRIBUTING.md: - Add 'just release taskrepo' as recommended method - Keep manual workflow as alternative - Change VERSION=0.10.17 to VERSION=X.Y.Z - Change ~/GitHub/homebrew-formulas to ../homebrew-formulas - Add reference to homebrew-formulas repository utilities 2. Create CLAUDE.md: - Add comprehensive AI assistant guidelines - Document Homebrew formula automation workflow - Include development environment setup - Add release process documentation - Document code style and testing strategy 3. Update .gitignore: - Allow CLAUDE.md to be tracked (add !CLAUDE.md exception) Aligns with rxiv-maker PR #280 and folder2md4llms PR #37. Ensures consistency across HenriquesLab ecosystem. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Pull Request Review - PR #12SummaryThis PR adds comprehensive documentation improvements with CLAUDE.md guidelines and aligns the Homebrew release workflow documentation. The changes are documentation-only with no code modifications. ✅ Strengths1. Documentation Consistency
2. AI Assistant Guidelines (CLAUDE.md)
3. Workflow Documentation
🔍 Observations & Suggestions1. Documentation Completeness ✓Status: All changes look complete and accurate The documentation correctly references:
2. .gitignore Pattern ✓Status: Correct implementation *.md
!README.md
!CHANGELOG.md
!CONTRIBUTING.md
!CLAUDE.mdThis pattern is appropriate for TaskRepo since it stores tasks as markdown files in 3. Consistency Check ✓Status: Aligned across files Verified consistency between:
4. Testing Strategy ✓Status: Appropriate for documentation changes The PR correctly identifies these as documentation-only changes requiring no code testing. The verification approach (git diff, tracked file check) is suitable. 🎯 Code Quality AssessmentDocumentation Quality: ⭐⭐⭐⭐⭐
Best Practices: ⭐⭐⭐⭐⭐
Maintainability: ⭐⭐⭐⭐⭐
🔒 Security ConsiderationsStatus: ✅ No security concerns
📊 Test CoverageStatus: ✅ N/A (Documentation only) No test coverage changes required. The existing test suite remains intact with tests for:
🎨 Style & ConventionsAlignment with CLAUDE.md: ✅ Perfect The documentation itself follows the guidelines it establishes:
💡 Minor Suggestions (Optional)1. CLAUDE.md - Homebrew Formula SectionConsider adding a note about when the homebrew-formulas repository is available: **Note:** The homebrew-formulas repository must be cloned as a sibling to TaskRepo for these commands to work.2. CONTRIBUTING.md - PrerequisitesConsider mentioning that # Install just (if not already installed)
# macOS: brew install just
# Linux: cargo install just
# Or see: https://github.com/casey/just#installationHowever, these are very minor suggestions and not blockers. The PR is excellent as-is. ✅ Final RecommendationAPPROVE ✅ This PR is ready to merge. It provides:
Merge Checklist
Great work on improving the documentation and establishing clear guidelines for AI-assisted contributions! This aligns well with modern development practices. Reviewed by: Claude Sonnet 4.5 |
There was a problem hiding this comment.
Pull request overview
This PR enhances TaskRepo's documentation by aligning Homebrew formula update workflows with the automated tools in the homebrew-formulas repository and adding comprehensive AI assistant guidelines. The changes are documentation-only and establish consistency across HenriquesLab repositories.
Changes:
- Updated CONTRIBUTING.md to prioritize automated
just releaseworkflow over manual steps, with clearer organization and improved placeholder values - Added CLAUDE.md with comprehensive guidelines for AI assistants, covering development setup, release process, Homebrew automation, code style, and testing strategy
- Updated .gitignore to explicitly allow tracking of CLAUDE.md
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| CONTRIBUTING.md | Restructured Homebrew update section to emphasize automated workflow; improved clarity with relative paths and generic version placeholders |
| CLAUDE.md | New comprehensive guide for AI assistants with project overview, development environment setup, release process, Homebrew automation, and code conventions |
| .gitignore | Added exception to allow tracking CLAUDE.md following existing markdown file allowlist pattern |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| ### Version Bumping | ||
|
|
||
| 1. Update version in `pyproject.toml` |
There was a problem hiding this comment.
The version should be updated in src/taskrepo/__version__.py, not pyproject.toml. The pyproject.toml file uses dynamic = ["version"] and references the version from src/taskrepo/__version__.py via the hatch configuration at line 121.
| 1. Update version in `pyproject.toml` | |
| 1. Update version in `src/taskrepo/__version__.py` (version is read dynamically in `pyproject.toml`) |
| tsk --version | ||
| ``` | ||
|
|
||
| **Note:** The automated workflow using `just` is preferred for consistency and efficiency. See the [homebrew-formulas repository](https://github.com/HenriquesLab/homebrew-formulas) for additional utility commands like `just list`, `just check-updates`, and `just sha256` |
There was a problem hiding this comment.
The sentence is missing a period at the end. It should end with "...and just sha256." to match standard punctuation conventions.
| **Note:** The automated workflow using `just` is preferred for consistency and efficiency. See the [homebrew-formulas repository](https://github.com/HenriquesLab/homebrew-formulas) for additional utility commands like `just list`, `just check-updates`, and `just sha256` | |
| **Note:** The automated workflow using `just` is preferred for consistency and efficiency. See the [homebrew-formulas repository](https://github.com/HenriquesLab/homebrew-formulas) for additional utility commands like `just list`, `just check-updates`, and `just sha256`. |
| - Automatic syncing capabilities | ||
|
|
||
| ### TUI Components | ||
| - Interactive prompts using questionary |
There was a problem hiding this comment.
The project uses prompt_toolkit for interactive prompts, not questionary. This can be verified in pyproject.toml line 29 which lists "prompt_toolkit>=3.0.0" as a dependency.
| - Interactive prompts using questionary | |
| - Interactive prompts using prompt_toolkit |
Code reviewNo issues found. Checked for bugs and CLAUDE.md compliance. This PR successfully:
Changes align with rxiv-maker PR #280 and folder2md4llms PR #37 patterns. Adds TaskRepo-specific automation documentation for AI assistants. 🤖 Generated with Claude Code - If this code review was useful, please react with 👍. Otherwise, react with 👎. |
This patch release adds documentation improvements: Documentation: - Align Homebrew formula workflow with automation (#12) - Create CLAUDE.md with comprehensive guidelines - Update .gitignore to allow CLAUDE.md tracking - Prioritize just-based automation - Ecosystem consistency with rxiv-maker and folder2md4llms Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This patch release adds documentation improvements: Documentation: - Align Homebrew formula workflow with automation (#12) - Create CLAUDE.md with comprehensive guidelines - Update .gitignore to allow CLAUDE.md tracking - Prioritize just-based automation - Ecosystem consistency with rxiv-maker and folder2md4llms Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>
Summary
This PR addresses documentation inconsistency and adds comprehensive AI assistant guidelines via CLAUDE.md. Aligns with ecosystem-wide improvements across HenriquesLab repositories.
Aligns with:
Changes Made
1. ✅ Update CONTRIBUTING.md - Homebrew Workflow
just release taskrepoas the recommended methodjustVERSION=0.10.17→VERSION=X.Y.Z(clear placeholder)~/GitHub/homebrew-formulas→../homebrew-formulas(relative path)2. ✅ Create CLAUDE.md - AI Assistant Guidelines
New file with comprehensive documentation for AI assistants:
Contents:
just releaseworkflow (recommended)3. ✅ Update .gitignore
!CLAUDE.mdexception to allow tracking.mdignore pattern with explicit allowlistTesting
Benefits
just-based workflowWorkflow Alignment
This PR ensures consistency between:
Both workflows (automated and manual) are now documented, with automation clearly marked as recommended.