Skip to content

ci: name the attribution caller distinguishably - #1

Merged
jmcerda merged 1 commit into
masterfrom
chore/attribution-caller-name
Sep 19, 2026
Merged

jmcerda merged 1 commit into
masterfrom
chore/attribution-caller-name

Conversation

@jmcerda

@jmcerda jmcerda commented Sep 19, 2026 •

Copy link
Copy Markdown
Contributor

A pull request's Actions list shows two identical "Attribution" entries: this repository's thin caller, which strips with write permission, and the organization ruleset's own run from shared-ci, which enforces read-only and survives a repository deleting its caller. Both are by design, so the caller is renamed to tell them apart.

Change

.github/workflows/attribution.yml: the workflow is named "Attribution (strip + check)", with a four-line comment saying why. Nothing else changes. The file is byte-identical to the canonical caller every Wilkes & Liberty repository carries.

Why it cannot break enforcement

The required status check is attribution / No AI attribution. That name comes from the job, not the workflow, so it is unchanged. This pull request's own checks show it.

No module or package code changes, so there is no changelog entry.

A pull request's Actions list showed two identical "Attribution" entries:
this thin caller and the organization ruleset's own run from shared-ci. Both
are by design, so the caller is renamed "Attribution (strip + check)". The
file is byte-identical to the canonical caller. The required check is keyed
on the job, attribution / No AI attribution, and is unaffected.
@cursor

cursor Bot commented Sep 19, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_54f409de-dbb3-4e9a-982a-38cc7087634e)

@jmcerda

jmcerda commented Sep 19, 2026

Copy link
Copy Markdown
Contributor Author

Bugbot could not run here (usage limit), so this is the findings review in its place.

Scope: one file, .github/workflows/attribution.yml. The workflow's display name changes from Attribution to Attribution (strip + check) and a four-line comment explains why. No step, trigger, permission or pinned ref changes.

The one thing a workflow rename can break is a required status check that is keyed on the workflow name. It is not: the org ruleset requires the context attribution / No AI attribution, which is <job id> / <job name> from this caller, and the job id is unchanged. Checked on this PR: both attribution / No AI attribution (this caller) and No AI attribution (the org-required run) report pass at the current head.

Drift: the file is byte-identical to the org template (blob d96df4e), so the drift check stays clean for this repository.

No findings.

@jmcerda
jmcerda merged commit fd401da into master Sep 19, 2026
5 checks passed
@jmcerda
jmcerda deleted the chore/attribution-caller-name branch October 23, 2026 19:09
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