Skip to content

CI: re-sync the label workflows to the template (github.token) - #37

Merged
hyperpolymath merged 1 commit into
mainfrom
claude/happy-sagan-mwnm7h
Oct 27, 2026
Merged

hyperpolymath merged 1 commit into
mainfrom
claude/happy-sagan-mwnm7h

Conversation

@hyperpolymath

@hyperpolymath hyperpolymath commented Oct 27, 2026 •

Copy link
Copy Markdown
Owner

Summary

This re-syncs labels.yml and label-triage.yml to the hyperpolymath/rsr-template-repo versions, as hyperpolymath/echo-types#354 did in echo-types. It is part of the estate-wide RE001 / harden-runner decision: metadatastician/standards#1352 (ADR-009), hyperpolymath/hypatia#964 and the template PR.

Changes

  • GH_TOKEN: ${{ github.token }} replaces ${{ secrets.GITHUB_TOKEN }}. It is the same token.
  • The runner is pinned to ubuntu-24.04, and each job gets timeout-minutes: 10.
  • Neither workflow gains a uses: step, so actions.lock is unchanged.

Testing

  • node scripts/check-repository.mjs passes.
  • I ran RE001 locally: 3 findings before, 1 after, under both the current and the new rule.
  • The one that remains is push-email-notify.yml, which genuinely needs the SMTP secrets. Under ADR-009 it is cleared by an (egress …) entry in a repo deed. This repository has no <repo>_chora.deed yet, so clearing it means minting a deed first, which is out of scope here.

Generated by Claude Code

Summary by CodeRabbit

  • Chores
    • Updated the runtime and time limits for automated label management. No user-facing changes.

labels.yml and label-triage.yml now match hyperpolymath/rsr-template-repo:
GH_TOKEN reads github.token instead of secrets.GITHUB_TOKEN (the same token),
the runner is pinned to ubuntu-24.04, and each job has timeout-minutes: 10.
Neither workflow gains a uses: step, so actions.lock is unchanged.

This clears Hypatia RE001 on both files under the current rule, as
hyperpolymath/echo-types#354 did there. RE001 on push-email-notify.yml stays:
under standards ADR-009 it is cleared by an (egress …) entry in a repo deed,
and this repository has no <repo>_chora.deed yet.

Checks: scripts/check-repository.mjs passes; RE001 run locally, 3 findings
before (old rule), 1 after (push-email-notify only).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01J4QvDithtn5Xk5MbSPm4Px
@coderabbitai

coderabbitai Bot commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
Commits

Reviewing files that changed from the base of the PR and between 7e0d49e and a913516.

Files selected for processing: 2
  • .github/workflows/label-triage.yml
  • .github/workflows/labels.yml

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Walkthrough

Both label workflows now run on Ubuntu 24.04, set a 10-minute timeout, and use github.token for GH_TOKEN.

Changes: Label workflow settings

Layer / File(s) Summary
Runner, timeout, and token settings
.github/workflows/label-triage.yml, .github/workflows/labels.yml
The triage and sync jobs use ubuntu-24.04, set a 10-minute timeout, and use github.token instead of secrets.GITHUB_TOKEN for GH_TOKEN.

Priority: Low

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

Change: Other

Merge Risk: Minimal · up to a9135

The workflow updates are mergeable; no actionable current-head risk was identified.

Pre-merge checks: Passed 4
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly identifies the main change: re-synchronizing the label workflows and updating the token reference. It is concise and related to the changeset.
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.
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@sonarqubecloud

Copy link
Copy Markdown

@hyperpolymath
hyperpolymath marked this pull request as ready for review October 27, 2026 01:32
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-27T01:33:43.520091Z a913516 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@coderabbitai

coderabbitai Bot commented Oct 27, 2026

Copy link
Copy Markdown
Contributor

Add Carrot credits or activate Agent usage billing to use Autofix

@hyperpolymath
hyperpolymath enabled auto-merge (squash) October 27, 2026 01:40
@hyperpolymath
hyperpolymath merged commit 8751c7d into main Oct 27, 2026
15 checks passed
@hyperpolymath
hyperpolymath deleted the claude/happy-sagan-mwnm7h branch October 27, 2026 02:06
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