Repository navigation
Remind about WIP-titled PRs weekly instead of daily - #7
Merged
Merged
Conversation
A WIP-titled PR is one the author is still working on, so a daily "convert to draft, or drop WIP" nudge in #prs is noise (Slack thread, Oct 14: Daniel + Leonard agreed on weekly). Set the wip_title reminder threshold to 120 working hours (5 weekdays) in the team config. Only time spent in the wip_title state is affected: the threshold is read from the PR's current state on each sweep, and any state change resets the staleness clock, so a PR whose title drops "WIP" gets the normal 24-hour reminders from that point. A sweep test covers both. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
chiang-daniel
changed the title
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
The rule is about what a WIP title means (the PR is still being worked
on), so it belongs in the code's defaults rather than nathan.config.ts:
thresholdHoursByState now defaults to { wip_title: 120 }, and a team's
per-state overrides are merged on top so setting another state's
threshold doesn't silently drop the WIP week. The spec and setup
checklist describe the default; the sweep test runs on default config
and asserts the exact reminders, like its neighbours.
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
chiang-daniel
changed the title
leonardmq
approved these changes
Oct 13, 2026
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.
What this PR does
Bump the WIP PR reminder from 1 day to 5 days (weekly)
Agentic PR Summary
Why
Nathan nudges the owner of a
WIP:-titled PR every day (example). Kiln's PR template requires every agent-opened PR to start withWIP:, so WIP titles are routine there, not an edge case.What
thresholdHoursByStatenow defaults to{ wip_title: 120 }insrc/features/pr_management/config.ts. Thresholds count working hours with weekends excluded in the owner's time zone, so 120 = 5 weekdays = one reminder a week, escalating one level per week as before.approved: 8doesn't silently drop the WIP week. Settingwip_titleexplicitly replaces it.nathan.config.tsoverride because the rule is about what a WIP title means, like the 14-day draft nudge. The team config is unchanged.What it doesn't change
Only time spent in the
wip_titlestate is affected. The threshold is looked up from the PR's current state on each hourly sweep, and any state change resets the staleness clock, so a PR that becomes ready for review is reminded on the usual schedule from that moment. App Home's Overdue and the daily report's Needs attention share the threshold, so WIP PRs also show as overdue only after a week.Testing
WIPa normal level-1 reminder 24 working hours later with the new next step.npm run checkpasses: lint, typecheck, config validation for all three environments, import boundaries, 800 tests. No staging deploy.🤖 Generated with Claude Code