ci: run CI only for trunk PRs and main pushes by kjgbot · Pull Request #40 · AgentWorkforce/trajectories

kjgbot · 2026-10-08T22:08:35Z

Org-wide CI reduction: CI runs only on the trunk → main PR and on pushes to main. Feature branches merge into trunk and get no CI. Adds trunk-guard.yml to flag PRs into main that don't come from trunk. Generated mechanically (job if: gates + push branch filters); actionlint shows no new findings.

🤖 Generated with Claude Code

Note

Medium Risk
Org-wide change to when automated tests and lint run; feature branches no longer get CI on PRs, so quality depends on the trunk merge train and correct branch targeting.

Overview
Restricts GitHub Actions so lint, typecheck, test, build, and the CI Success aggregator run only on pushes to main and on pull requests whose head branch is trunk (not on feature-branch PRs into main).

The compact-on-merge job gets the same head-branch gate so trajectory compaction runs only when a trunk → main PR is merged.

Adds trunk-guard.yml, which fails any PR targeting main unless the source branch is trunk, steering feature work to merge into trunk first.

^{Reviewed by Cursor Bugbot for commit 6a3e425. Bugbot is set up for automated code reviews on this repo. Configure here.}

Feature branches no longer run CI. Jobs triggered by pull_request(_target) run only when the head branch is `trunk`; push triggers are limited to main (release/env/tag pushes unchanged). trunk-guard.yml flags any PR into main that does not come from `trunk`.

coderabbitai · 2026-10-08T22:08:52Z

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Organization UI
Review profile: CHILL
Plan: Advanced
Run ID: 51b6caf8-6acc-479b-aa4e-a6959013afed

📥 Commits

Reviewing files that changed from the base of the PR and between f682efb and 6a3e425.

📒 Files selected for processing (3)

.github/workflows/ci.yml
.github/workflows/compact-on-merge.yml
.github/workflows/trunk-guard.yml

 ____________________________________________________________________________________________________________________________________________________________________________________________
< The object-oriented model makes it easy to build up programs by accretion. What this often means, in practice, is that it provides a structured way to write spaghetti code. - Paul Graham >
 --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
  \
   \   (\__/)
       (•ㅅ•)
       / 　 づ

✨ Finishing Touches

🧪 Generate unit tests (beta)

Commit to this branch
Create a new PR

Autopilot · 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.}

devin-ai-integration

Devin Review found 1 potential issue.

devin-ai-integration · 2026-10-08T22:10:23Z

loadTrajectories

+      ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') &&
+      (github.event.pull_request.merged == true)


🟡 Merged PR trajectories remain uncompacted

When a non-trunk PR merges into main, compact skips its trajectory compaction. Merged PRs outside trunk leave no compacted summary.

Learn more

This workflow runs when a PR targeting main closes. Its original merge condition compacted trajectories for every merged PR, independently of the CI workflow. The new head-branch condition skips merged PRs from other branches, so their summaries are never generated by this job. loadTrajectories only processes trajectories when the compact command runs.

Example: A hotfix PR from fix/crash merges into main. The close event reports merged: true, but the branch gate skips compact, leaving the hotfix trajectories without a summary.

Recommended fix: Restore the merged-only job condition. If compaction is also required for feature PRs merged into trunk, add a separate trunk close trigger and handle that target explicitly.

Suggested change

((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') &&

(github.event.pull_request.merged == true)

if: github.event.pull_request.merged == true

Was this helpful? React with 👍 or 👎 to provide feedback.

cursor

Cursor Bugbot has reviewed your changes using high effort and found 2 potential issues.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 6a3e425. Configure here.}

cursor · 2026-10-08T22:18:27Z

-    if: always()
+    if: >-
+      ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') &&
+      (always())


Skipped CI success passes required checks

Medium Severity

The ci-success aggregator, used for branch protection, now skips on non-trunk PRs. GitHub treats a skipped required check as passing, so existing required checks no longer block mis-targeted PRs. trunk-guard fails under a new check name that current protection rules would not require.

^{Reviewed by Cursor Bugbot for commit 6a3e425. Configure here.}

cursor · 2026-10-08T22:18:27Z

+# `trunk` (via the merge train); CI runs only on the trunk -> main PR.
+on:
+  pull_request:
+    branches: [main]


Trunk guard ignores base branch edits

Low Severity

trunk-guard.yml uses default pull_request types, which omit edited. Changing a PR base to main does not start a new run, so a retargeted feature PR can point at main without the failing guard check.

^{Reviewed by Cursor Bugbot for commit 6a3e425. Configure here.}

This reverts commit 8df818e, reversing changes made to f682efb.

kjgbot merged commit 8df818e into main Oct 8, 2026

devin-ai-integration Bot reviewed Oct 8, 2026

View reviewed changes

cursor Bot reviewed Oct 8, 2026

View reviewed changes

kjgbot added a commit that referenced this pull request Oct 8, 2026

Revert "Merge pull request #40 from AgentWorkforce/trunk"

00b72b1

This reverts commit 8df818e, reversing changes made to f682efb.

Navigation Menu

Repository navigation

ci: run CI only for trunk PRs and main pushes - #40

ci: run CI only for trunk PRs and main pushes#40
kjgbot merged 1 commit into
mainfrom
trunk

kjgbot commented Oct 8, 2026 •

edited by cursor Bot

Loading

Uh oh!

coderabbitai Bot commented Oct 8, 2026

Uh oh!

devin-ai-integration Bot left a comment

Uh oh!

devin-ai-integration Bot Oct 8, 2026

Uh oh!

cursor Bot left a comment

Uh oh!

cursor Bot Oct 8, 2026

Uh oh!

cursor Bot Oct 8, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

		((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') \|\| github.head_ref == 'trunk') &&
		(github.event.pull_request.merged == true)

	((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') \|\| github.head_ref == 'trunk') &&
	(github.event.pull_request.merged == true)
	if: github.event.pull_request.merged == true

Conversation

kjgbot commented Oct 8, 2026 • edited by cursor Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

coderabbitai Bot commented Oct 8, 2026

Uh oh!

devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

Uh oh!

devin-ai-integration Bot Oct 8, 2026

Choose a reason for hiding this comment

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

cursor Bot Oct 8, 2026

Choose a reason for hiding this comment

Skipped CI success passes required checks

Uh oh!

cursor Bot Oct 8, 2026

Choose a reason for hiding this comment

Trunk guard ignores base branch edits

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

kjgbot commented Oct 8, 2026 •

edited by cursor Bot

Loading