Remove vendored throughput-forecast files; skill now installed globally - #673
Merged
troymyree-lang merged 1 commit intoAug 6, 2026
Conversation
Per review on #655/#666: rather than relocating gh_fetch.py to dq-documentation while still vendoring project-specific copies here, the whole skill is now installed at ~/.claude/skills/github-throughput-forecast/ instead - available for any GitHub Projects v2 board, nothing committed into any one repo's history. Removes .claude/scripts/{gh_fetch,forecast,forecast-html}.py, .claude/skills/throughput-forecast/, and the settings.json that existed solely for their Bash allowlist. create-ticket (#654/#656) untouched - separate concern, out of scope. No merged history reverted - this is a normal forward delete commit. PR #665 closed unmerged rather than revised (its fixes carried over into the global install already). Closes #666
6 tasks
Michael Gonzalez (MicGon7)
approved these changes
Aug 6, 2026
Michael Gonzalez (MicGon7)
left a comment
Collaborator
There was a problem hiding this comment.
Look good 👍
troymyree-lang
approved these changes
Aug 6, 2026
troymyree-lang
left a comment
Collaborator
There was a problem hiding this comment.
Approve — clean resolution of the skill-location thread. Deleting the vendored throughput-forecast files and installing the skill globally (~/.claude/skills/github-throughput-forecast/) is the right call: it works for any GitHub Projects board without committing into an app repo, create-ticket stays untouched, and #665's fixes were carried into the global install. Pure forward delete, no app code or secrets.
Merging via admin bypass since the org-required "Required Secret Scan (TruffleHog)" check is stuck pending (org-config issue — a public repo can't invoke the private-repo workflow), being routed to org admins separately.
troymyree-lang
deleted the
feature/666/remove-vendored-throughput-forecast
branch
August 6, 2026 18:22
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
.claude/scripts/{gh_fetch.py,forecast.py,forecast-html.py},.claude/skills/throughput-forecast/{SKILL.md,config.yaml}, and the.claude/settings.jsonthat existed solely for those scripts' Bash allowlist.create-ticket(create-ticket: auto-add issues to Project #50 and link sub-issue parents #654/create-ticket: auto-add issues to Project #50 and link sub-issue parents #656) is untouched — separate concern, explicitly out of scope.feature/voice-selectionsince.Why
Per review feedback on #655/#666: rather than relocating
gh_fetch.pyintodq-documentationwhile still keeping project-specific copies vendored here, the whole skill (gather script + engine + config + docs) is now installed globally at~/.claude/skills/github-throughput-forecast/— available for any GitHub Projects v2 board, without committing anything into any one app's repo. PR #665 (the parent-breakdown follow-up) was closed unmerged rather than revised — its fixes were carried over into the global install directly.Ticket
Closes #666 — same non-default-branch caveat as prior PRs in this thread: will manually close after merge since
closes #Nwon't auto-fire againstfeature/voice-selection.Type of Change
Testing
~/.claude/skills/github-throughput-forecast/works end-to-end after this removal (separate verification, not part of this repo's history)Checklist
./gradlew testDebug) — N/A, no app code (app/) touchedSee
Documentation/work-log/666-remove-vendored-throughput-forecast.mdfor the full writeup.