Skip to content

chore(release): open dev at 2.57.0 before releasing 2.56.0 - #4686

Merged
lidge-jun merged 1 commit into
devfrom
codex/dev-version-2.57.0
Sep 15, 2026
Merged

lidge-jun merged 1 commit into
devfrom
codex/dev-version-2.57.0

Conversation

@github-actions

@github-actions github-actions Bot commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Summary

2.56.0 is about to be released. Merging this first means dev already outranks the new tag when it lands, so neither dev nor any open pull request ever inherits the version-line failure. release.yml refuses to publish until this has merged.

This moves dev to 2.57.0.

Opened automatically by .github/workflows/dev-version-bump.yml. The same
version-line move was previously done by hand in 32529c2, e4a85d1, 076ad30, and
befcac3.

Verification

The workflow proved 2.56.0 has neither a Git tag nor an npm publication, and bun test tests/ci-workflows/release-version-line.test.ts proved the chosen development version is unused.

Checklist

  • Scope stays focused and avoids unrelated cleanup.
  • Docs or release notes were updated when needed.
  • Security-sensitive changes were reviewed for secrets, auth, and unsafe defaults.

Review readiness checklist

This PR stays in draft until every box below is ticked. Tick all four boxes once the requirements are met:

  • All CI tests are green on my local testing.

  • I pushed my PR to the latest dev commit.

  • I resolved all correct Codex and CodeRabbit findings.

  • My PR is ready for review.

@coderabbitai

coderabbitai Bot commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: b28e2a30-8c04-4b63-88bc-7c1a14883746

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Sep 15, 2026

Copy link
Copy Markdown
Contributor Author

⏳ DRAFT

  • hygiene: unsponsored_surface.

What to do

  • Fix unsponsored_surface — This changes an authentication, workflow, release-automation, or dependency surface. MAINTAINERS.md requires security review for these; ask a maintainer to apply maintainer-sponsored once they have reviewed it. Paths: package.json.
  • Tick all four boxes in the PR description once you're done (currently 0/4).

Review readiness checklist

  • ⬜ All CI tests are green on my local testing.
  • ⬜ I pushed my PR to the latest dev commit.
  • ⬜ I resolved all correct Codex and CodeRabbit findings.
  • ⬜ My PR is ready for review.

0/4 boxes ticked.

Automatic draft conversion failed (token cannot change draft status). Please convert this pull request to a draft manually. The required enforce-target check will keep failing until every issue above is resolved.

@github-actions
github-actions Bot marked this pull request as draft September 15, 2026 06:22
@lidge-jun

Copy link
Copy Markdown
Owner

리뷰 · 우선순위 73 / 80

이 PR은 제품 코드를 바꾸지 않는다. 지금 dev HEAD(270291170)의 package.json2.56.0인데, 이 한 줄만 2.57.0으로 올린다. .github/workflows/dev-version-bump.yml이 자동으로 연 브랜치이고, 예전에 #4618이 2.55.02.56.0으로 했던 것과 같은 release-train 단계다. 목적도 같다. 2.56.0 태그가 찍히기 전에 dev가 이미 그 버전보다 앞에 서 있게 해서, tests/ci-workflows/release-version-line.test.tsdev와 그 위에 열린 모든 PR을 한꺼번에 빨갛게 만들지 않게 하려는 것이다.

현재 checkout 기준으로 이 동작은 워크플로·스크립트와 맞다. scripts/bump-dev-version.ts는 stable X.Y.Z가 나가면 devX.(Y+1).0으로 옮긴다. 2.56.02.57.0이 그 규칙이다. .github/workflows/release.ymlRequire dev to be ready for this release 단계는 version-line.ts assert-ahead <dev version> <release version>을 돌리므로, dev가 아직 2.56.0인 채로는 2.56.0 publish가 거절된다. 그래서 이 PR은 “있으면 좋은 정리”가 아니라 publish 게이트를 통과하기 위한 필수 pre-move다. diff는 package.json 버전 필드 한 줄뿐이고 bun.lock 등 다른 메타데이터는 건드리지 않는다. #4618도 같은 한 줄 패턴이었다.

devlog/_plan/260915_2560_release_train/030_release.md가 정한 순서는 이렇다. (1) dev tip CI 확인 → (2) tip을 그대로 main으로 올리는 promote PR(#4687) 머지 → (3) main의 M1에서 Cross-platform CI 성공 → (4) 이 pre-move를 머지해 dev2.57.0으로 → (5) release.ymlversion: 2.56.0expected-sha: M1을 넣어 publish. 지금 #4686과 #4687이 같이 열려 있는 상태는 train이 굴러가고 있다는 뜻이다. 다만 #4686을 #4687보다 먼저 dev에 넣어도 assert-ahead는 통과하지만, 그 뒤에 #4687 브랜치를 새 dev 위로 rebase하면 promote 쪽에 2.57.0이 섞일 수 있다. promote 브랜치는 2.56.0을 유지한 tip(270291170)에 그대로 둬야 한다.

라인 3 package.json - 2.56.02.57.0 한 줄. 규칙·게이트와 일치한다. 다른 파일 변경이 없는 것도 정상이다.
경로 .github/workflows/dev-version-bump.yml / scripts/bump-dev-version.ts - 자동 생성·버전 결정 로직이 이 diff와 맞다. 손으로 다시 고칠 필요 없다.
경로 release.yml assert-ahead - 이 PR이 머지되기 전에는 2.56.0 publish가 막힌다. 그래서 “나중에 해도 된다”가 아니다.
상태 - 지금 draft이고 Cross-platform 테스트·gates가 아직 돌아가는 중이다. 게이트가 초록이 될 때까지 ready 체크리스트를 강제할 필요는 없지만, 머지 전에는 그 결과를 봐야 한다.

메인테이너의 판단이 필요한 지점

  • 030_release.md 순서대로 [WRONG BRANCH] release: promote the verified 2.56.0 product tree to main #4687(promote) → M1 CI → 이 PR 머지 → release.yml인지, 아니면 assert-ahead만 먼저 풀려고 이 PR을 promote보다 앞에 넣을지
  • draft/PR-gate 체크리스트를 봇이 열어 둔 상태인데, 버전 한 줄 chore를 사람이 직접 ready로 올릴지 아니면 CI 초록 후 바로 머지할지
  • v2.56.0 태그와 npm 출판이 아직 없다는 워크플로 검증을 머지 직전에도 한 번 더 믿을지(본문 Verification에 이미 적혀 있음)

너의 추천
표준 release-train pre-move로 보고, Cross-platform CI가 이 헤드에서 초록이면 머지해도 된다. 우선순위는 #4687 promote를 먼저 main에 안착시킨 뒤 이 PR을 넣는 030_release.md 순서를 따르는 편이 안전하다. 절대 #4687을 이 PR이 들어간 뒤의 dev 위로 rebase하지 말 것. types.ts/config.ts 분할이나 pre-split 모놀리스 경로와는 무관하다. 중복 PR도 아니다(#4618은 이미 머지된 이전 사이클).

이 댓글은 grok-bot이 작성했습니다

@lidge-jun
lidge-jun marked this pull request as ready for review September 15, 2026 08:19
@lidge-jun
lidge-jun merged commit 5dfea72 into dev Sep 15, 2026
28 checks passed
@lidge-jun
lidge-jun deleted the codex/dev-version-2.57.0 branch September 15, 2026 08:19
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 15, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-15T08:21:31.638868Z 36b9889 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

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.

1 participant