Repository navigation
Open an issue summarising Lauren's changes after each sync - #3
Merged
Merged
Conversation
When a sync brings in new Cursor commits, a digest job lists the merged cursor/plugins PRs that touched pstack in that range, asks Claude Sonnet 5.5 to summarise them, and opens an issue so GitHub emails the owner. Michael's upstream commits name the Cursor commit they mirror, so the range comes from the fork's upstream before and after the sync, with no stored state. Setting upstream now runs after the push, so a failed merge leaves the range in place for the next run. Dispatching with from and to skips the sync and summarises that range.
There was a problem hiding this comment.
🟡 Changes recommended
Range discovery, failed-publication recovery, and empty-result handling need correction for reliable scheduled digests.
3 open findings
What changed in this PR
Adds issue-based digests so the fork’s owner can follow upstream pstack changes without reading each PR.
Changes:
- Generates summaries from merged PR metadata, excluding version bumps.
- Runs digests after syncs or for manually supplied commit ranges.
- Defines summary content and attribution rules.
| File | Description |
|---|---|
.github/workflows/sync.yml |
Captures sync boundaries and publishes digest issues. |
.github/scripts/digest.py |
Collects PR metadata and generates digest output. |
.github/scripts/digest-prompt.md |
Defines summary structure, factual limits, and author attribution. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Comment on lines
+71
to
+72
| git push --force origin refs/remotes/michael/upstream:refs/heads/upstream | ||
| echo "moved=true" >> "$GITHUB_OUTPUT" |
Comment on lines
+34
to
+35
| for commit in gh(f'repos/{REPO}/commits?path=pstack&sha={new}&per_page=100'): | ||
| if commit['sha'].startswith(old): |
Comment on lines
+86
to
+87
| if not prs: | ||
| sys.exit(f'No merged pstack PRs between {old} and {new}') |
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.
garyj wants to see what Lauren changes in pstack without reading every cursor/plugins PR. This adds a digest that runs whenever a sync brings in new Cursor commits.
digestjob insync.ymllists the merged cursor/plugins PRs that touchedpstack/in the synced range, sends their titles, descriptions and changed files toclaude-sonnet-5-5with the prompt in.github/scripts/digest-prompt.md, and opens an issue in this repo so GitHub emails the owner. Version-bump PRs are left out.upstreamcommits name the Cursor commit they mirror, so the job reads the fork'supstreambefore and after the sync.upstreamnow moves after the push, so a failed merge keeps the range for the next run.fromandtoskips the sync and summarises that range.issues: write. It readsANTHROPIC_API_KEYfrom the repo secrets.Checks run:
e43c7eetoe5136d8(0.15.9 to 0.15.25) opened pstack 0.15.9 → 0.15.25: what Lauren changed #2, covering 30 PRs, in 2m24s. I read it against the PR descriptions and found no invented claims.autoreview(Codex) came back clean on the first commit. The two later commits pinsetup-uvtov10.2.0, becausev10doesn't exist as a tag, and turn its cache off.Generated by
claude-opus-5-5using theClaude Codeharness.Git author:
Gary Jarrel [LC] <lara@jarrel.io>