Skip to content

feat: add issue activity to weekly snippets (and fix GNU/BSD date) - #99

Merged
cgrindel merged 4 commits into
mainfrom
fix_error
Jun 9, 2026
Merged

feat: add issue activity to weekly snippets (and fix GNU/BSD date)#99
cgrindel merged 4 commits into
mainfrom
fix_error

Conversation

@cgrindel

@cgrindel cgrindel commented Jun 9, 2026

Copy link
Copy Markdown
Owner

Two changes on this branch:

  • Fixed the date: invalid option -- 'j' failure. lib/date.sh was hardcoded for BSD date, which breaks when GNU coreutils shadows the system date on PATH (and on Linux). It now detects the date flavor at load time and translates the BSD-style calls to GNU when needed — both paths verified.
  • Added GitHub issue activity to the snippets: issues you opened, closed, and commented on, rendered alongside the authored/reviewed PR sections. GitHub search has no "closed-by" qualifier, so closed issues are approximated via assignee + state:closed.

Also reworked the generator test to be hermetic: it injects a fake gh through a new GH_BIN seam and serves canned fixtures, so it no longer hits the live GitHub API (the extra issue searches pushed it past GitHub's ~30/min search rate limit). Added retry/backoff to search_issues for real weekly runs too.

Full suite passes (24/24), buildifier fmt+lint clean.

🤖 Generated with Claude Code

cgrindel and others added 2 commits June 8, 2026 08:14
Detect the date implementation at load time and translate BSD-style invocations to GNU when needed, so the snippet generator works on machines where GNU coreutils shadows the macOS system date (and on Linux).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Query issues the author opened, closed, and commented on for the week and render them alongside the authored/reviewed PR sections. GitHub search has no closed-by qualifier, so closed issues are approximated via assignee + state:closed.

Make the generator test hermetic: inject a fake gh via a new GH_BIN seam and serve canned fixtures, removing the live API calls that tripped GitHub's search rate limit. Also add retry/backoff to search_issues for real runs.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@cgrindel cgrindel self-assigned this Jun 9, 2026
cgrindel and others added 2 commits June 8, 2026 18:33
Make the four activity sections symmetric with the new issue headings: '### Authored' becomes '### PRs Authored' and '### Reviewed' becomes '### PRs Reviewed'. Also add jq unit cases for the reviewed-PR and three issue renderers.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Define the Glydways/Open-Source split in terms of any activity (PR or issue) in the org, and tell the model to draw on issues opened, closed, and commented on when writing bullets.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@cgrindel
cgrindel marked this pull request as ready for review June 9, 2026 00:49
@cgrindel
cgrindel merged commit d4d65d5 into main Jun 9, 2026
1 check passed
@cgrindel
cgrindel deleted the fix_error branch June 9, 2026 00:49
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