[only21mil]

Problem

PR Size only triggers on pull_request_target, but its sync-label-definitions job only runs when the event is not pull_request_target. So every run skips it, as in run 41325286591. It has never run since the workflow_dispatch and push triggers were removed in 581d242. The repo's size:* labels still have their old descriptions, such as "0-9 changed lines (additions + deletions).", not the "test files excluded in mixed PRs" text #1105 added to this job.

Change

• Delete the sync-label-definitions job.
• Drop color and description from the label list in prepare-config. Only the deleted job read them; the label job only reads name.

The trigger and the whole label job are unchanged. prepare-config still emits labels_json, with the same label names. Labels already on GitHub keep their current colors and descriptions.

Scope and approval

This removes a job that cannot run, with no behavior change, so it should qualify as a small, focused fix without a prior issue. Restoring a trigger so the job runs again would be a different change. I left it out because the triggers were removed on purpose.

Verification

• Ran vp run check:changed 1d626d02d95f9a2343553f6498037ee82290d304: passes.
• Ran actionlint .github/workflows/pr-size.yml: no findings.
• Parsed the workflow: the jobs are prepare-config and label, and label.needs is prepare-config.
• Compared with main: everything from label: to the end of the file is byte-identical, and so is the trigger.
• Checked that nothing else in the repo names sync-label-definitions.

Not checked: a live PR Size run on this change. pull_request_target runs the workflow from the base branch, so this PR's own size label still comes from the current file.

Prior PRs

• ci: SHA-pin actions in pr-size workflow and drop dead job #4135 by @kridaydave made the same removal together with SHA pinning and a docs page. It was closed in a backlog sweep for review bandwidth, not for this change.
• ci: every workflow job gets a timeout instead of the six-hour default #28650 by @only21mil adds timeout-minutes to the jobs in this file, including the one deleted here. Whichever lands second needs a one-line conflict fix.

Generated with Claude Opus 5.5 and GPT-6.1 Terra and GPT-6.1 Sol in T3 Code Nightly

[macroscopeapp]

Approvability

Verdict: Approved at 5058ec8

Macroscope's review found this PR approvable — Removes a label-sync job that cannot run under the workflow’s only trigger, along with unused label metadata. Existing PR sizing and label assignment remain unchanged.

^{You can add or adjust custom eligibility rules. Learn more.}

[coderabbitai]

No actionable comments were generated in the recent review.

Recent review info

Run configuration

• Source mode: Auto
• Repository: pingdotgg/t3code
• Branch: main
• File: .coderabbit.config.ts
• Commit: 3fd2b85
• Layers: Repository TS config
• Review profile: CHILL
• Plan: Advanced
• Run ID: 6eda7b3f-e4e6-4486-822c-429b409aa849

Commits

Reviewing files that changed from the base of the PR and between 3fd2b85 and 5058ec8.

Files selected for processing: 1

• .github/workflows/pr-size.yml

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

Walkthrough

The PR size workflow now defines only the six managed label names. It no longer creates missing labels or updates existing label colors and descriptions.

Changes: PR size label configuration

Layer / File(s)	Summary
Managed label configuration .github/workflows/pr-size.yml	The label entries now contain only their names. The job that created or updated label definitions was removed.

Estimated code review effort: 2 (Simple) | ~5 minutes

Change: Refactor

Possibly related PRs

• pingdotgg/t3code#28650: Both PRs change .github/workflows/pr-size.yml and the sync-label-definitions job.

Suggested reviewers: macroscopeapp

Merge Risk: Minimal · up to 5058e

The workflow retains its existing label behavior. No merge-blocking risk was identified.

Pre-merge checks: 4

Check name	Status	Explanation
Title check		The title clearly and concisely identifies the main change: removing the PR size label sync job because it cannot run.
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.
Description check		The pull request description follows the required template. It explains the problem, change, scope and approval basis, verification steps, observed results, and the live-run limitation. It also identi…

Full details: Description check

Explanation

The pull request description follows the required template. It explains the problem, change, scope and approval basis, verification steps, observed results, and the live-run limitation. It also identifies generated-agent usage.

Finishing Touches

Generate unit tests (beta)

• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts