Skip to content

ci: add repo-card refresh stub (MIX-181) - #43

Merged
jordins merged 1 commit into
masterfrom
jordins/MIX-181/repo-card-stub
Jul 8, 2026
Merged

ci: add repo-card refresh stub (MIX-181)#43
jordins merged 1 commit into
masterfrom
jordins/MIX-181/repo-card-stub

Conversation

@jordins

@jordins jordins commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Adds the shared nightly repo-card refresh (MIX-181). Delegates all logic to mixmaxhq/github-workflows/.github/workflows/refresh_repo_card.yml@main; opens a human-reviewed docs: PR only when this repo's CLAUDE.md goes stale. No auto-merge; no CODEOWNERS needed (the card PR is assigned to the code's authors). Manual full re-derive available via the mode: full dispatch input.

@jordins
jordins requested a review from leonardoll-mixmax July 7, 2026 14:46
@coderabbitai

coderabbitai Bot commented Jul 7, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

Adds a new GitHub Actions workflow file, refresh-repo-card.yml, triggered on a weekday cron schedule or manual dispatch with mode and base-window-hours inputs. The workflow defines a card job that calls a shared reusable workflow from an external repository, passing the input values (with defaults for scheduled runs) and only the explicitly required secrets.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Related PRs: None mentioned.

Suggested labels: ci, github-actions

Suggested reviewers: None specified.

🐰

A card refreshed at dawn's first light,
Cron ticks softly through the night,
With modes to choose and hours to set,
This little rabbit won't forget,
To hop the workflow, clean and bright.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is a short summary and misses the required template sections for risks, testing, and release plan. Fill in the Context/Description, Risks, Testing, and Release Plan sections using the repository template.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the main change: adding a repo-card refresh workflow.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch jordins/MIX-181/repo-card-stub

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick comments (1)
.github/workflows/refresh-repo-card.yml (1)

22-22: 🔒 Security & Privacy | 🔵 Trivial | ⚡ Quick win

Pinning the reusable workflow to @main is a mutable reference.

uses: mixmaxhq/github-workflows/.github/workflows/refresh_repo_card.yml@main tracks a moving branch, so future changes to that shared workflow (including breaking ones) will silently affect this workflow's next scheduled run. Pinning to a release tag or commit SHA improves reproducibility and supply-chain hygiene, even for an internal org repo.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In @.github/workflows/refresh-repo-card.yml at line 22, The reusable workflow
reference in the refresh-repo-card workflow is pinned to a mutable branch, so
update the uses entry for
mixmaxhq/github-workflows/.github/workflows/refresh_repo_card.yml to a fixed
release tag or commit SHA instead of `@main`. Keep the same workflow target, but
choose an immutable ref so scheduled runs use a reproducible version and are not
affected by upstream branch changes.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In @.github/workflows/refresh-repo-card.yml:
- Line 22: The reusable workflow reference in the refresh-repo-card workflow is
pinned to a mutable branch, so update the uses entry for
mixmaxhq/github-workflows/.github/workflows/refresh_repo_card.yml to a fixed
release tag or commit SHA instead of `@main`. Keep the same workflow target, but
choose an immutable ref so scheduled runs use a reproducible version and are not
affected by upstream branch changes.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 70b6c495-9623-4418-a833-a4c15c4504bf

📥 Commits

Reviewing files that changed from the base of the PR and between 9539d00 and bcc46b0.

📒 Files selected for processing (1)
  • .github/workflows/refresh-repo-card.yml

@jordins
jordins merged commit 6f483c1 into master Jul 8, 2026
5 checks passed
@jordins
jordins deleted the jordins/MIX-181/repo-card-stub branch July 8, 2026 13:33
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.

2 participants