feat(ci): mship companion pr check by Sg312 · Pull Request #5079 · simstudioai/sim

Sg312 · 2026-06-16T00:35:04Z

Summary

Adds companion ci check to check for drift with mothership service
Companion: https://github.com/simstudioai/copilot/pull/309

Type of Change

New feature

Testing

Manual

Checklist

Code follows project style guidelines
Self-reviewed my changes
Tests added/updated and passing
No new warnings introduced
I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Mirror of copilot's repo-agnostic companion-pr-check workflow: a soft, non-blocking warning on PRs to staging/main when a declared cross-repo Companion PR isn't merged in lockstep. Declare via a "Companion:" trailer or a "## Companion PRs" task list. Requires the CROSS_REPO_TOKEN secret.

…d releases Mirror of copilot: on staging->main release PRs, aggregate Companion declarations from every squashed feature PR in the release (deduped). dev->staging stays single-PR.

vercel · 2026-06-16T00:35:10Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
docs	Ready	Preview, Comment	Jun 16, 2026 12:44am

greptile-apps · 2026-06-16T00:37:50Z

Greptile Summary

This PR introduces a new GitHub Actions workflow that detects cross-repo "Companion:" PR declarations in PR descriptions and posts a sticky comment indicating whether those companion PRs have been merged, keeping the simstudioai/sim and a paired repo in lockstep.

New workflow (companion-pr-check.yml): triggers on pull_request events for staging/main and on a */30 * * * * schedule; parses "Companion:" trailers and task-list refs from PR bodies, fetches companion PR state via a CROSS_REPO_TOKEN secret, and upserts a sticky comment with ✅/⚠️/❌ status per companion.
Release-PR aggregation: for PRs targeting main, it also walks squash-merge commit messages to collect companion declarations from the feature PRs bundled into the release.
Label management: applies a has-companion label when companions are found, but never removes it — leaving a stale label when companions are later deleted from the description or when the scheduled run re-evaluates a PR that no longer has companions.

Confidence Score: 3/5

The workflow is safe to merge as a non-blocking informational check, but two logic gaps mean it will not behave as documented in common situations.

The has-companion label is applied but never removed, so PRs whose descriptions are later edited to drop companion declarations will carry a stale label forever. Separately, the scheduled refresh cron will silently do nothing until this workflow file also exists on the default branch — meaning the primary motivation for the schedule (catching companions that merge after this PR opens) won't work in the interim period.

companion-pr-check.yml — the label-removal path and the scheduled-run default-branch dependency both need attention before the workflow behaves as its comments describe.

Important Files Changed

Filename	Overview
.github/workflows/companion-pr-check.yml	New CI workflow that posts a sticky comment and applies a `has-companion` label when a PR declares cross-repo companion PRs. Label is never removed when companions are cleared from the description, and the scheduled refresh is a no-op until the file lands on the default branch.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A([Trigger]) --> B{Event type?}
    B -->|pull_request| C[checkPR: context.payload.pull_request]
    B -->|schedule / workflow_dispatch| D[List open PRs on staging + main]
    D --> E[checkPR: each PR]
    C --> F[collectCompanions]
    E --> F
    F --> G{PR targets main?}
    G -->|yes| H[Fetch commits → extract squash-merge PR numbers → parse each feature PR body]
    G -->|no| I[Parse PR body only]
    H --> J[Deduplicated companion list]
    I --> J
    J --> K{Any companions?}
    K -->|no| L[Delete sticky comment — label NOT removed]
    K -->|yes| M[ensureLabel + addLabels: has-companion]
    M --> N{CROSS_REPO_TOKEN set?}
    N -->|no| O[❓ could not verify]
    N -->|yes| P[cross.rest.pulls.get for each companion]
    P --> Q{cp.merged?}
    Q -->|yes + correct base| R[✅ line]
    Q -->|yes + wrong base| S[⚠️ line, warn=true]
    Q -->|no| T[❌ line, warn=true]
    P -->|error| U[❓ line, warn=true]
    R & S & T & U & O --> V[upsert sticky comment]

%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
    A([Trigger]) --> B{Event type?}
    B -->|pull_request| C[checkPR: context.payload.pull_request]
    B -->|schedule / workflow_dispatch| D[List open PRs on staging + main]
    D --> E[checkPR: each PR]
    C --> F[collectCompanions]
    E --> F
    F --> G{PR targets main?}
    G -->|yes| H[Fetch commits → extract squash-merge PR numbers → parse each feature PR body]
    G -->|no| I[Parse PR body only]
    H --> J[Deduplicated companion list]
    I --> J
    J --> K{Any companions?}
    K -->|no| L[Delete sticky comment — label NOT removed]
    K -->|yes| M[ensureLabel + addLabels: has-companion]
    M --> N{CROSS_REPO_TOKEN set?}
    N -->|no| O[❓ could not verify]
    N -->|yes| P[cross.rest.pulls.get for each companion]
    P --> Q{cp.merged?}
    Q -->|yes + correct base| R[✅ line]
    Q -->|yes + wrong base| S[⚠️ line, warn=true]
    Q -->|no| T[❌ line, warn=true]
    P -->|error| U[❓ line, warn=true]
    R & S & T & U & O --> V[upsert sticky comment]

_{Reviews (1): Last reviewed commit: "ci(companion): aggregate companions acro..." | Re-trigger Greptile}

…uire @actions/github) Mirror of copilot: companion-pr-check failed with MODULE_NOT_FOUND. Read the other repo's PR with a plain REST fetch + the PAT in the header (PAT stays read-only; commenting/labeling uses GITHUB_TOKEN).

@v7

…veat Mirror of copilot: clear() removes the has-companion label when a PR's companions are all removed; schedule default-branch caveat documented; github-script kept at @v7 to match repo convention.

github-actions · 2026-06-16T00:45:05Z

⚠️ Cross-repo companion check

One or more companion PRs aren't merged into staging yet. Merging this without them will leave copilot and sim out of sync — merge them in lockstep.

❌ simstudioai/copilot#309 — OPEN, not merged (targets staging) — ci(companion): cross-repo companion-PR warning + mship-release

Sg312 added 2 commits June 15, 2026 17:08

ci(companion): aggregate companions across bundled feature PRs on pro…

ef7b4d3

…d releases Mirror of copilot: on staging->main release PRs, aggregate Companion declarations from every squashed feature PR in the release (deduped). dev->staging stays single-PR.

vercel Bot deployed to Preview June 16, 2026 00:37 View deployment

greptile-apps Bot reviewed Jun 16, 2026

View reviewed changes

Comment thread .github/workflows/companion-pr-check.yml

Comment thread .github/workflows/companion-pr-check.yml

Comment thread .github/workflows/companion-pr-check.yml

Sg312 added 2 commits June 15, 2026 17:38

fix(companion): drop stale has-companion label + document schedule ca…

28526dd

…veat Mirror of copilot: clear() removes the has-companion label when a PR's companions are all removed; schedule default-branch caveat documented; github-script kept at @v7 to match repo convention.

vercel Bot deployed to Preview June 16, 2026 00:44 View deployment

github-actions Bot added the has-companion Has a cross-repo companion PR (see companion-pr-check) label Jun 16, 2026

Sg312 merged commit cbd3d22 into staging Jun 16, 2026
30 checks passed

waleedlatif1 mentioned this pull request Jun 16, 2026

v0.7.8: security hardening, code hygiene, MSFT oauth provider, new atlassian assets and google calendar tools #5088

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(ci): mship companion pr check#5079

feat(ci): mship companion pr check#5079
Sg312 merged 4 commits into
stagingfrom
dev

Sg312 commented Jun 16, 2026 •

edited

Loading

Uh oh!

vercel Bot commented Jun 16, 2026 •

edited

Loading

Uh oh!

greptile-apps Bot commented Jun 16, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Jun 16, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

Sg312 commented Jun 16, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Type of Change

Testing

Checklist

Uh oh!

vercel Bot commented Jun 16, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

greptile-apps Bot commented Jun 16, 2026

Greptile Summary

Confidence Score: 3/5

Important Files Changed

Flowchart

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Jun 16, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

⚠️ Cross-repo companion check

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Sg312 commented Jun 16, 2026 •

edited

Loading

vercel Bot commented Jun 16, 2026 •

edited

Loading

github-actions Bot commented Jun 16, 2026 •

edited

Loading