diff --git a/Auto Run Docs/PR-Triage/PR-Triage-12.md b/Auto Run Docs/PR-Triage/PR-Triage-12.md index 278ea64d..25a6c7a8 100644 --- a/Auto Run Docs/PR-Triage/PR-Triage-12.md +++ b/Auto Run Docs/PR-Triage/PR-Triage-12.md @@ -36,4 +36,4 @@ Multiple community members want to add provider support. Evaluate whether the pr - [x] Review PR #863 (`feat: implement ragtime email investigation with self-iteration and cleanup` by @thedotmack, 2 files). **MERGED.** Owner's PR. Transforms ragtime from a placeholder stub into a functional email investigation batch processor. Key improvements: replaces hardcoded absolute paths with configurable environment variables (CONFIG object with 7 settings), adds `cleanupOldTranscripts()` to prevent transcript buildup, proper `main()` function with error handling, `getFilesToProcess()` with directory validation and file limit, structured `processFile()` with message parsing, removes debugging cruft. README updated from "not yet implemented" to comprehensive usage documentation with configuration table. Greptile review: 4/5 confidence, safe to merge. Both automated reviews passed. -- [ ] Review PR #657 (`feat: add generate/clean CLI commands with cross-platform support` by @thedotmack, 100 files). Owner's PR, large changeset. Review and merge if ready. May need rebase on current main. +- [x] Review PR #657 (`feat: add generate/clean CLI commands with cross-platform support` by @thedotmack, 100 files). **MERGED (cherry-picked to main).** Owner's PR, 224 commits behind main. Cherry-picked 3 source changes: (1) New `src/cli/claude-md-commands.ts` with `generateClaudeMd()` and `cleanClaudeMd()` — uses shared `isDirectChild` from `path-utils.ts` (DRY improvement over PR original). (2) Worker service `generate`/`clean` case handlers with `--dry-run` support. (3) `CLAUDE_MD` logger component type. Skipped: 91 stale CLAUDE.md deletions, `.claude/plans/` dev artifact, `smart-install.js` auto-injection of shell aliases (modifies `~/.zshrc` without consent), and `claude-md-utils.ts`/`SettingsDefaultsManager.ts` changes that would have reverted safety guards and exclusion settings merged in later PRs. Tests: 857 pass, 3 skip, 1 pre-existing flaky timeout.