Skip to content

Commit 6e258d1

Browse files
yanxue06claude
andcommitted
style(release): drop the verbose explanatory comments
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 1e98205 commit 6e258d1

2 files changed

Lines changed: 0 additions & 22 deletions

File tree

.github/blocks/bump-monorepo-versions/action.yaml

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -96,14 +96,8 @@ runs:
9696
CURRENT_VER=$(jq -r '.version // "0.0.0"' "$PKG_PATH/package.json")
9797
9898
COMMITS=$(git log --oneline "${LAST_SHA}..HEAD" -- "$PKG_PATH" 2>/dev/null || git log --oneline HEAD~10..HEAD -- "$PKG_PATH")
99-
# Pre-compute the diff per package and inject it — codex-action does not
100-
# auto-attach a diff, so the version must be decided on the actual code,
101-
# not just commit messages (see determine-publish-version for the why).
10299
NAME_STATUS=$(git diff --name-status "${LAST_SHA}..HEAD" -- "$PKG_PATH" 2>/dev/null || true)
103100
PKG_DIFF=$(git diff "${LAST_SHA}..HEAD" -- "$PKG_PATH" 2>/dev/null || true)
104-
# Per-package cap: the prompt output concatenates every changed
105-
# package's diff into one step output, which GitHub caps at 1 MB/job —
106-
# keep headroom for several changed packages.
107101
CAP=100000
108102
if [ "${#PKG_DIFF}" -gt "$CAP" ]; then
109103
PKG_DIFF="${PKG_DIFF:0:$CAP}
@@ -134,9 +128,6 @@ runs:
134128
with:
135129
openai-api-key: ${{ inputs.openai-api-key }}
136130
safety-strategy: read-only
137-
# Diff-based breaking-change judgment is a real reasoning task — run at
138-
# high effort. The diff is provided per package below; the agent may also
139-
# run read-only git for any truncated file.
140131
effort: high
141132
prompt: |
142133
You are a release manager for the "${{ inputs.service-name }}" monorepo.

.github/blocks/determine-publish-version/action.yaml

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -84,20 +84,10 @@ runs:
8484
HEAD_SHA: ${{ github.sha }}
8585
run: |
8686
set -uo pipefail
87-
# Pre-compute the diff and inject it into the prompt. openai/codex-action
88-
# does NOT auto-attach a diff — it is an agentic `codex exec`, so left to
89-
# itself the agent may only run `git log` (commit messages) and never look
90-
# at the code (exactly what happened on the v0.2.0-rc.11 run). Embedding
91-
# the diff here GUARANTEES the version is decided on the actual changes.
92-
# name-status + churn are always complete (they reveal removed/renamed
93-
# files — the strongest breaking signal); the full diff is capped to keep
94-
# the prompt bounded, and the agent can pull any truncated file via git.
9587
NAME_STATUS=$(git diff --name-status "$PREV_SHA" "$HEAD_SHA" 2>/dev/null || true)
9688
STAT=$(git diff --stat "$PREV_SHA" "$HEAD_SHA" 2>/dev/null || true)
9789
LOG=$(git log --no-merges --format='- %s%n%b' "$PREV_SHA..$HEAD_SHA" 2>/dev/null || true)
9890
DIFF=$(git diff "$PREV_SHA" "$HEAD_SHA" 2>/dev/null || true)
99-
# The diff is injected via a step output, which GitHub caps at 1 MB/job;
100-
# 400 KB leaves headroom for the log + stat. Don't raise past ~800 KB.
10191
CAP=400000
10292
TRUNC=""
10393
if [ "${#DIFF}" -gt "$CAP" ]; then
@@ -130,9 +120,6 @@ runs:
130120
with:
131121
openai-api-key: ${{ inputs.openai-api-key }}
132122
safety-strategy: read-only
133-
# Judging backward-compatibility from a diff is a real reasoning task —
134-
# run it at high effort. This step runs once per release and a wrong
135-
# major bump is expensive, so the budget is justified.
136123
effort: high
137124
prompt: |
138125
Determine the next semantic version after ${{ steps.last-release.outputs.version }}.

0 commit comments

Comments
 (0)