Skip to content

Comments

Update docs and improve CLI#28

Merged
alex-feel merged 5 commits intomainfrom
alex-feel-dev
Aug 16, 2025
Merged

Update docs and improve CLI#28
alex-feel merged 5 commits intomainfrom
alex-feel-dev

Conversation

@alex-feel
Copy link
Owner

No description provided.

Changed PowerShell launcher to use @Args (splat operator) instead of $args
for proper argument expansion when calling external commands. This ensures
all arguments like --continue, --model, etc. are correctly passed to Claude
after the --append-system-prompt parameter.

Also added debug output to show when additional arguments are being passed.

Bash launcher already correctly uses "$@" for argument passing.
Batch wrappers correctly use %* for argument forwarding.
…developer prompt

Added comprehensive software engineering principles as critical requirements:

Core Principles Section:
- YAGNI (You Aren't Gonna Need It) - avoid premature features
- DRY (Don't Repeat Yourself) - single source of truth
- KISS (Keep It Simple, Stupid) - simplicity over complexity
- SOLID principles - all five OOP design principles

Integrated Throughout:
- TDD section emphasizes KISS in implementation and DRY in refactoring
- Modularity tied to Single Responsibility Principle
- Documentation follows DRY and KISS
- Async implementation guided by SRP and KISS
- Pythonic Code Style section for Python-specific best practices

New Sections:
- Design Patterns and Anti-Patterns with principle alignment
- Code Review Checklist for verifying all principles

These additions ensure code is maintainable, testable, and follows
industry best practices while avoiding overengineering and complexity.
Removed documentation files that were referenced but never created,
causing broken links throughout the repository. Also removed redundant
platform-specific README files in scripts subdirectories as all
documentation is maintained in the main scripts/README.md file.
- Fixed broken documentation links pointing to non-existent docs/ files
- Corrected description of subagents as general-purpose (not Python-specific)
- Updated file lists to match actual repository contents
- Added missing YAML frontmatter documentation for slash commands
- Fixed directory structures and removed references to deleted files
- Ensured all README files accurately describe their respective components
- Added comprehensive pre-commit hooks documentation
- Documented all configured linting tools (Ruff, markdownlint, PSScriptAnalyzer, shellcheck)
- Added Python development guidelines with uv package management
- Updated project structure to reflect actual repository layout
- Documented GitHub Actions workflows (lint.yml, release-please.yml)
- Added CI/CD pipeline section with detailed workflow descriptions
- Enhanced testing section with pre-commit verification steps
@alex-feel alex-feel enabled auto-merge August 16, 2025 00:06
@alex-feel alex-feel merged commit 6950546 into main Aug 16, 2025
5 checks passed
@alex-feel alex-feel deleted the alex-feel-dev branch August 16, 2025 00:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant