PR Custody

chore: reorganize GitHub Actions into subdirectories with shared composite action

@acsrujandraftchecks n/achecks…srujan/gh-workflow-cleanup → master36 files · +122 −328review: @abir-taheerreview: @anshugarg15updated 2w ago

GitHub

▸Description

Summary

Moves 27 flat workflow files into app-specific subdirectories (apollo/, thermos/, smoke/, infra/, triggers/, github/, publish/)
Extracts a setup-pnpm composite action (.github/actions/setup-pnpm/) to replace an identical 4-step pnpm install/env/cache block copy-pasted across 6 workflows
Adds smoke/_reusable.yml as a shared base — prod.yml and staging.yml are now ~15-line callers instead of ~50-line duplicates
Updates all dorny/paths-filter workflow: paths and reusable uses: refs to match new locations

Structure

.github/
├── actions/setup-pnpm/        ← new composite action
└── workflows/
    ├── apollo/                 ← integration tests, MCP, SDK regression, rube
    ├── thermos/                ← unit + service isolation tests
    ├── smoke/                  ← _reusable.yml + prod/staging/self-hosted callers
    ├── infra/                  ← Depot docker image builds
    ├── triggers/               ← cross-repo dispatches (docs, zen)
    ├── github/                 ← PR management, labeling, claude, secrets detection
    └── publish/                ← npm publish

Notes

No logic changes — all transformations are mechanical moves + deduplication
Depot egress rule to block webhook.site is configured via org settings (not code)

Test plan

Verify a PR triggers apollo/integration-core-platform.yml and apollo/integration-tool-router.yml
Verify thermos/unit-tests.yml triggers on apps/thermos/** changes
Verify smoke/prod.yml and smoke/staging.yml run via the shared reusable
Check that the setup-pnpm composite action step shows correctly in workflow run logs

🤖 Generated with Claude Code

loading diff…