Skip to content

feat: mothership-backed sdk-review workflow, merge queue, and docs#1485

Open
vaibhavatlan wants to merge 4 commits intomainfrom
sdk-mothership-migration
Open

feat: mothership-backed sdk-review workflow, merge queue, and docs#1485
vaibhavatlan wants to merge 4 commits intomainfrom
sdk-mothership-migration

Conversation

@vaibhavatlan
Copy link
Copy Markdown
Collaborator

Summary

  • Adds .github/workflows/sdk-review.yml — thin GHA workflow that parses @sdk-review comments and dispatches to mothership's rover-native sandbox (replaces the current claude.yml-based review)
  • Adds .github/workflows/merge-queue.yml — auto-updates and auto-merges approved PRs in FIFO order (replaces branch-keeper)
  • Adds .mothership/review.yaml — repo config for the SDK reviewer snapshot
  • Updates docs/agents/review.md — documents the new mothership-backed review flow
  • Updates docs/agents/testing.md — adds reviewer test expectations
  • Updates docs/agents/coding-standards.md — adds Temporal determinism and contract evolution sections

Context

Companion to atlanhq/mothership#549 which adds the sdk-reviewer and sdk-evolution rover snapshots.

Invocation

Command What it does
@sdk-review Review only
@sdk-review auto-complete Review + auto-fix loop (max 3 iter)
@sdk-review challenge: <reason> Re-evaluate disputed findings
@sdk-review override: <reason> Admin force-pass
@sdk-review stop Cancel in-progress review

Test plan

  • Verify GHA workflow triggers on @sdk-review comment
  • Verify dispatch reaches mothership (requires atlanhq/mothership#549 deployed)
  • Verify merge-queue updates behind branches after a merge to main
  • Verify merge-queue labels needs-rebase on conflicts

🤖 Generated with Claude Code

@snykgituser
Copy link
Copy Markdown

snykgituser commented Apr 21, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@Aryamanz29
Copy link
Copy Markdown
Member

v3 relevance check: ✅ Active — mothership-backed sdk-review workflow. @vaibhavatlan is this still on track for merge?

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.

3 participants