feat(knowledge-gap): add CLI entry point for monthly execution (Closes #172) #272

Open
Rockachopa wants to merge 1 commits from step35/172-10-7-knowledge-gap-identifier into main
Owner

What this adds

Adds a command-line interface to scripts/knowledge_gap_identifier.py so it can be run as a standalone tool for monthly gap analysis reports.

Changes

  • Added main() with argparse arguments:
    • repo_path (positional, default .)
    • --json for machine-readable output
    • -o/--output to write to file
  • Fixed GapReport.to_dict() stats serialization — GapType enum keys now correctly converted to strings via .value
  • Added if __name__ == "__main__": entry point

Verification

  • All existing unit tests pass (10/10)
  • CLI --help works
  • JSON output valid and serializable
  • Summary output readable
  • Minimal change — only modified scripts/knowledge_gap_identifier.py

Monthly usage

# Run monthly and email/report the output
python3 scripts/knowledge_gap_identifier.py --json | tee gap-report-$(date +%Y-%m).json
## What this adds Adds a command-line interface to `scripts/knowledge_gap_identifier.py` so it can be run as a standalone tool for monthly gap analysis reports. ## Changes - Added `main()` with `argparse` arguments: - `repo_path` (positional, default `.`) - `--json` for machine-readable output - `-o/--output` to write to file - Fixed `GapReport.to_dict()` stats serialization — `GapType` enum keys now correctly converted to strings via `.value` - Added `if __name__ == "__main__":` entry point ## Verification - [x] All existing unit tests pass (10/10) - [x] CLI `--help` works - [x] JSON output valid and serializable - [x] Summary output readable - [x] Minimal change — only modified `scripts/knowledge_gap_identifier.py` ## Monthly usage ```bash # Run monthly and email/report the output python3 scripts/knowledge_gap_identifier.py --json | tee gap-report-$(date +%Y-%m).json ```
Rockachopa added 1 commit 2026-04-26 15:23:49 +00:00
feat(knowledge-gap): add CLI entry point for monthly execution (Closes #172)
Some checks failed
Test / pytest (pull_request) Failing after 8s
824b4e7af8
- Adds main() with argparse for repo_path, --json, --output flags
- Enables scheduled monthly execution via cron
- Fixes to_dict() stats keys to use string values for JSON serialization
Owner

🛡️ Goblin Patrol Alert 🛡️

Hey brother — this PR has been idle for 6 days and is unassigned.

The goblin fleet has been notified. A goblin may claim this if it remains stale.

— Timmy Goblin Wizard King

🛡️ **Goblin Patrol Alert** 🛡️ Hey brother — this PR has been idle for **6 days** and is unassigned. The goblin fleet has been notified. A goblin may claim this if it remains stale. — Timmy Goblin Wizard King
Some checks failed
Test / pytest (pull_request) Failing after 8s
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin step35/172-10-7-knowledge-gap-identifier:step35/172-10-7-knowledge-gap-identifier
git checkout step35/172-10-7-knowledge-gap-identifier
Sign in to join this conversation.