Skip to content

feat(github): PR action-item context rule#162

Merged
jordanrburger merged 2 commits into
mainfrom
upstream/patterns-github
Jun 23, 2026
Merged

feat(github): PR action-item context rule#162
jordanrburger merged 2 commits into
mainfrom
upstream/patterns-github

Conversation

@jordanrburger

Copy link
Copy Markdown
Collaborator

Part of the de-personalized Patterns batch — upstreaming accumulated instance rules into the engine phases (Phase-1 work for the connector-based migration, #156). Follows the method from #157.

What

One rule added to phases/connectors/github.md inbound-scan (assembles into SKILL only — connector phase, requires: github, mode: [consolidation, briefing]):

  • PR Action-Item Context (Pattern docs: spec for scoutctl bootstrap auto #27) — when a PR becomes an action item, the line must state what the PR does, which project it serves, and what's blocked downstream — never just the title + merge status. Captures: the change in one plain-English phrase, the project/area resolved against the KB, who's waiting or what ships on merge, and how long it's waited. "Review feedback needing a response" and "approval blocking a release" are different priorities — the downstream context is what distinguishes them.

Complements the existing three-bucket review-request classification right above it (which decides whether a PR is a review item); this rule governs how a PR-derived item is written once it is one.

Verification

  • Assembly checked via _assemble(cfg, kind): lands in SKILL only, not DREAMING/RESEARCH (mode: [consolidation, briefing]).
  • Diff grepped clean of names / employer / customers / Slack IDs / repo-PR-Linear IDs / user paths.

🤖 Generated with Claude Code

Jordan Burger and others added 2 commits June 23, 2026 14:06
Upstreamed from accumulated instance experience into the github
inbound-scan (assembles into SKILL only, requires: github):

When a PR becomes an action item, the line must state what the PR does,
which project it serves, and what's blocked downstream — never just the
title and merge status. Captures the change in plain English, the
project/area resolved against the KB, who's waiting / what ships on merge,
and how long it's waited. A bare title + status is not enough to act on.

Generic rule; originating example dropped.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@jordanrburger jordanrburger merged commit 5b91133 into main Jun 23, 2026
5 checks passed
@jordanrburger jordanrburger deleted the upstream/patterns-github branch June 23, 2026 21:53
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