Skip to content

chore(release): open dev at 2.51.0 before releasing 2.50.0 - #4194

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

chore(release): open dev at 2.51.0 before releasing 2.50.0#4194
lidge-jun merged 1 commit into
devfrom
codex/dev-version-2.51.0

Conversation

@github-actions

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

Copy link
Copy Markdown
Contributor

Summary

2.50.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.51.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.50.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 10, 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: e1d29f4f-11e2-4138-af0d-714a5e9a598a

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 10, 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 10, 2026 09:56
@lidge-jun

Copy link
Copy Markdown
Owner

리뷰 · 우선순위 78 / 80

이 PR은 제품 기능을 고치는 게 아니라, 지금 dev에 적혀 있는 버전 숫자만 올리는 릴리스 준비 작업입니다. 현재 dev HEAD(12c248f52, #4186 Cursor ACP 실현성 문서 포함)의 package.json은 아직 2.50.0입니다. 안정판 2.50.0main에서 찍으려면 .github/workflows/release.yml의 “Require dev to be ready for this release” 단계가 origin/dev 버전이 앞으로 찍을 태그보다 엄격히 앞이어야 한다고 검사합니다. 지금처럼 dev2.50.0이면 “앞선다”를 통과하지 못해 안정판 배포 디스패치가 거절됩니다.

게다가 tests/ci-workflows/release-version-line.test.ts는 이미 찍힌 태그보다 트리 버전이 뒤처지면 빨갛게 만듭니다. 안정판 2.50.0이 한 번 나가면 dev와 그 위를 향한 모든 PR이 자기 디프와 무관한 버전 라인 실패를 물려받게 됩니다. 그래서 이 PR이 package.json"version"2.50.0에서 2.51.0으로 올립니다. 숫자는 손으로 고른 게 아니라 scripts/bump-dev-version.ts / nextDevelopmentVersion 규칙입니다. 안정판 X.Y.Z가 나가면 devX.(Y+1).0으로 갑니다. 2.50.0이면 2.51.0이 맞습니다. 로컬에서도 nextDevelopmentVersion("2.50.0")2.51.0을 돌려줍니다.

열림 경로는 .github/workflows/dev-version-bump.yml의 pre-move 모드입니다. 워크플로가 v2.50.0 태그와 npm @bitkyc08/opencodex@2.50.0이 아직 없다고 확인한 뒤 PR을 열었고, 같은 검증을 이 리뷰 시점에도 다시 보면 태그는 없고 npm 최신은 아직 2.49.0입니다. release.yml은 이 이동이 dev에 합쳐질 때까지 게시를 거절합니다. 봇이 dev에 직접 푸시하지 않고 PR만 연 것도 맞습니다. ruleset Protect dev는 승인 리뷰와 코드오너 서명이 필요합니다. 디프는 진짜로 package.json 한 줄뿐이고, types/config 대형 분리 캠페인과도 무관합니다. 이전에도 같은 패턴이 #4115(2.50.0 오픈), #4019(2.49.0 오픈), #3925(2.48.0 오픈)로 반복됐습니다. 지금은 draft이고 체크리스트가 비어 있으며, merge 상태는 REVIEW_REQUIRED / BLOCKED입니다.

라인 3 (package.json) - 2.50.02.51.0 한 줄 변경. bump-dev-version.ts의 안정판→다음 마이너 규칙과 일치하고, 제품 코드·워크플로 파일은 건드리지 않았습니다.
경로 scripts/bump-dev-version.ts / nextDevelopmentVersion - 안정판 입력에 대한 후보가 2.51.0인지 이 PR 범위에서는 서술·워크플로 검증에 맡기고, 호스티드 CI의 release-version-line 스위트가 최종 확인을 맡습니다.
경로 .github/workflows/release.yml 게이트 - 이 PR만으로는 게이트를 통과시킨 것이 아닙니다. dev에 머지된 뒤에야 “Require dev to be ready for this release”가 2.51.0 vs 찍으려는 2.50.0을 통과합니다.
호스티드 CI / draft - 헤드 codex/dev-version-2.51.0(d375d75af)은 draft이고, 이 리뷰 시점에 의미 있는 체크 롤업이 아직 안 보였습니다. 버전 한 줄이라 실패 여지는 작지만, Ready for review로 올린 뒤 초록을 확인한 다음 합치는 편이 안전합니다.

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

  • 안정판 2.50.0을 지금 바로 찍을 준비가 main/preview에 되어 있는지, 이 PR을 먼저 합친 직후 디스패치할지
  • draft 체크리스트를 봇 PR답게 건너뛰고 Ready로 올린 뒤 CI 초록만 볼지, 사람이 네 칸을 다 찍을지
  • 합친 뒤 opencodex-dev-state.jsonpackageVersion2.51.0으로 갱신하고, 안정판 태그 작업으로 바로 넘어갈지

너의 추천
지금 dev 방향(#4186 ACP 문서 직후, package.json은 아직 2.50.0 open)과 충돌하지 않는 릴리스 하이진이라서 Ready로 올리고 CI 초록 확인 뒤 바로 머지하는 게 맞습니다. 머지 직후 안정판 2.50.0 디스패치를 이어가면, dev/열린 PR이 버전 라인 빨강을 물려받는 창을 거의 만들지 않습니다. types/config 분리와 무관하니 닫지 말고 살려 두세요.

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

@lidge-jun
lidge-jun marked this pull request as ready for review September 10, 2026 10:02
@lidge-jun
lidge-jun merged commit cf44f6f into dev Sep 10, 2026
2 of 3 checks passed
@lidge-jun
lidge-jun deleted the codex/dev-version-2.51.0 branch September 10, 2026 10:02
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 10, 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-10T10:04:07.932554Z d375d75 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.

harryzhou2000 pushed a commit to harryzhou2000/opencodex that referenced this pull request Sep 10, 2026
dev pre-moved to 2.51.0 via lidge-jun#4194, and lidge-jun#4195 promoted freeze SHA 12c248f into
main as 2d4d7a2 with a tree byte-identical to the audited freeze tree.
Records why enforce-target failed by design on a promotion PR, with the 2.49.0
precedent, and why local prepush was skipped on a tree already green on three
platforms.
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