[codex] promote dev workflow cleanup to main #77
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,97 @@ | ||
| #!/usr/bin/env node | ||
| import fs from "node:fs"; | ||
| import path from "node:path"; | ||
| import { execFileSync } from "node:child_process"; | ||
|
|
||
| const args = process.argv.slice(2); | ||
| const base = args.includes("--base") ? args[args.indexOf("--base") + 1] : process.env.GITHUB_BASE_REF; | ||
| const dryRun = args.includes("--dry-run"); | ||
|
|
||
| if (!base || !["dev", "main"].includes(base)) { | ||
| console.error("Usage: node .github/scripts/pr-version-bump.mjs --base <dev|main> [--dry-run]"); | ||
| process.exit(1); | ||
| } | ||
|
|
||
| function git(args, fallback = "") { | ||
| try { | ||
| return execFileSync("git", args, { encoding: "utf8", stdio: ["ignore", "pipe", "ignore"] }).trim(); | ||
| } catch { | ||
| return fallback; | ||
| } | ||
| } | ||
|
|
||
| function readJson(file) { | ||
| return JSON.parse(fs.readFileSync(file, "utf8")); | ||
| } | ||
|
|
||
| function writeJson(file, data) { | ||
| fs.writeFileSync(file, JSON.stringify(data, null, 2) + "\n"); | ||
| } | ||
|
|
||
| function bump(version) { | ||
| const parts = String(version || "0.0.0").split(".").map((part) => Number.parseInt(part, 10)); | ||
| const minor = Number.isFinite(parts[1]) ? parts[1] : 0; | ||
| const patch = Number.isFinite(parts[2]) ? parts[2] : 0; | ||
| if (base === "main") return `0.${minor + 1}.0`; | ||
| return `0.${minor}.${patch + 1}`; | ||
| } | ||
|
|
||
| function baseJson(file) { | ||
| const fromGit = git(["show", `origin/${base}:${file}`], ""); | ||
| if (!fromGit) return null; | ||
| try { | ||
| return JSON.parse(fromGit); | ||
| } catch { | ||
| return null; | ||
| } | ||
| } | ||
|
|
||
| function packageCandidates() { | ||
| const changed = git(["diff", "--name-only", `origin/${base}...HEAD`], "") | ||
| .split("\n") | ||
| .filter(Boolean); | ||
| const candidates = new Set(); | ||
| for (const file of changed) { | ||
| const appMatch = file.match(/^apps\/([^/]+)\//); | ||
| if (appMatch && fs.existsSync(path.join("apps", appMatch[1], "package.json"))) { | ||
| candidates.add(path.join("apps", appMatch[1], "package.json")); | ||
| } | ||
| if (file.startsWith("app/") && fs.existsSync(path.join("app", "package.json"))) { | ||
| candidates.add(path.join("app", "package.json")); | ||
| } | ||
| } | ||
| if (candidates.size === 0 && fs.existsSync("package.json")) candidates.add("package.json"); | ||
| if (candidates.size === 0 && fs.existsSync(path.join("app", "package.json"))) candidates.add(path.join("app", "package.json")); | ||
| return [...candidates]; | ||
| } | ||
|
|
||
| function updatePackage(file) { | ||
| const current = readJson(file); | ||
| const basePkg = baseJson(file) || current; | ||
| const nextVersion = bump(basePkg.version); | ||
| if (current.version === nextVersion) { | ||
| console.log(`${file} already at ${nextVersion}`); | ||
| return; | ||
| } | ||
| console.log(`${file}: ${current.version || "(none)"} -> ${nextVersion}`); | ||
| if (!dryRun) { | ||
| current.version = nextVersion; | ||
| writeJson(file, current); | ||
| } | ||
|
|
||
| const lockFile = path.join(path.dirname(file), "package-lock.json"); | ||
| if (fs.existsSync(lockFile)) { | ||
| const lock = readJson(lockFile); | ||
| if (lock.name === current.name || path.dirname(file) !== ".") lock.version = nextVersion; | ||
| if (lock.packages && lock.packages[""]) lock.packages[""].version = nextVersion; | ||
| if (!dryRun) writeJson(lockFile, lock); | ||
| } | ||
| } | ||
|
|
||
| const candidates = packageCandidates(); | ||
| if (candidates.length === 0) { | ||
| console.error("No package.json candidate found for version bump."); | ||
| process.exit(1); | ||
| } | ||
|
|
||
| for (const candidate of candidates) updatePackage(candidate); |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,52 @@ | ||
| name: PR Version Bump | ||
|
|
||
| on: | ||
| pull_request: | ||
| types: [opened, synchronize, reopened, ready_for_review] | ||
| branches: | ||
| - dev | ||
| - main | ||
|
|
||
| permissions: | ||
| contents: write | ||
| pull-requests: read | ||
|
|
||
| jobs: | ||
| version-bump: | ||
| if: ${{ github.event.pull_request.draft == false }} | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - name: Reject forked PR writeback | ||
| if: ${{ github.event.pull_request.head.repo.full_name != github.repository }} | ||
| run: | | ||
| echo "PR version bump writes back to same-repo branches only. Please apply the expected package.json version bump manually for forked PRs." | ||
| exit 1 | ||
|
|
||
| - name: Check out PR branch | ||
| uses: actions/checkout@v4 | ||
| with: | ||
| ref: ${{ github.event.pull_request.head.ref }} | ||
| fetch-depth: 0 | ||
| token: ${{ secrets.GITHUB_TOKEN }} | ||
|
|
||
| - name: Fetch base branch | ||
| run: git fetch origin ${{ github.base_ref }} --depth=1 | ||
|
|
||
| - name: Apply expected version bump | ||
| run: node .github/scripts/pr-version-bump.mjs --base ${{ github.base_ref }} | ||
|
|
||
| - name: Commit version bump | ||
| run: | | ||
| if git diff --quiet; then | ||
| echo "Version already matches expected bump." | ||
| exit 0 | ||
| fi | ||
| git config user.name "github-actions[bot]" | ||
| git config user.email "41898282+github-actions[bot]@users.noreply.github.com" | ||
| git add package.json package-lock.json app/package.json app/package-lock.json apps/*/package.json apps/*/package-lock.json 2>/dev/null || true | ||
| if git diff --cached --quiet; then | ||
| echo "No package version files were staged." | ||
| exit 0 | ||
| fi | ||
| git commit -m "chore: bump PR version" | ||
| git push | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Forked PRs to
dev/mainhit this step and fail before checkout or version verification. The message asks contributors to apply the bump manually, but even a manually corrected fork PR can never get this check green, which blocks the PR if this workflow is required; skip writeback for forks or run a read-only verification instead of exiting failure.Useful? React with 👍 / 👎.