Skip to content

fix(release): advance dev to 2.36.1 - #3006

Closed
Ingwannu wants to merge 1 commit into
devfrom
ingw/fix-dev-version-2.36.1
Closed

fix(release): advance dev to 2.36.1#3006
Ingwannu wants to merge 1 commit into
devfrom
ingw/fix-dev-version-2.36.1

Conversation

@Ingwannu

@Ingwannu Ingwannu commented Aug 30, 2026

Copy link
Copy Markdown
Owner

Summary

  • Advance the dev package version from the already-published 2.36.0 line to 2.36.1.
  • Restore the repository-wide release-version invariant without contaminating unrelated feature PRs.
  • Keep the repair to the single authoritative package.json version field; bun.lock carries no workspace package version.

2.36.1 is currently unused: npm returns 404 for @bitkyc08/opencodex@2.36.1, refs/tags/v2.36.1 is absent, and GitHub has no v2.36.1 release.

Verification

  • bun test tests/release-version-line.test.ts tests/compatibility-version.test.ts — 4 pass, 0 fail.
  • bun run typecheck — passed.
  • git diff --check — passed.
  • All checks ran with isolated temporary HOME, OPENCODEX_HOME, and CODEX_HOME and a two-CPU/nice limit.

Checklist

  • Scope stays focused and avoids unrelated cleanup.
  • Docs or release notes were updated when needed. No docs change is needed for an internal development version-line repair.
  • Security-sensitive changes were reviewed for secrets, auth, and unsafe defaults. This patch changes version metadata only.

Summary by CodeRabbit

  • Chores
    • Updated the application version to 2.36.1.

@Ingwannu
Ingwannu requested a review from lidge-jun as a code owner August 30, 2026 12:55
@github-actions

Copy link
Copy Markdown
Contributor

Deterministic PR hygiene checks passed.

@github-actions github-actions Bot added the bug Something isn't working label Aug 30, 2026
@coderabbitai

coderabbitai Bot commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 6bd64dc5-1e31-4752-9223-15e6c0763ea9

📥 Commits

Reviewing files that changed from the base of the PR and between c2778ca and 2231dde.

📒 Files selected for processing (1)
  • package.json

Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review.


📝 Walkthrough

Walkthrough

The package version in package.json changes from 2.36.0 to 2.36.1.

Changes

Package Version Update

Layer / File(s) Summary
Update package version
package.json
The package version changes from 2.36.0 to 2.36.1 on line 3.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 2231d

This localized version-metadata update advances the development package to 2.36.1 without changing runtime behavior, dependencies, permissions, or deployment configuration; no actionable merge-blocking risk remains after normal checks and review.

Suggested reviewers: lidge-jun

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the package version update from 2.36.0 to 2.36.1. It is concise and specific.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ingw/fix-dev-version-2.36.1

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.

@lidge-jun

Copy link
Copy Markdown
Owner

리뷰 · 우선순위 66 / 80

이 PR은 지금 dev HEAD c2778ca3a (#3005, 낡은 프록시를 갈아끼우면서 같이 붙어 있던 입장 자격증을 버린 고침) 위에 package.json 한 줄만 고친다. 지금 HEAD의 버전 칸은 아직 2.36.0 이다. 이 브랜치는 그 칸을 2.36.1 로 올린다. 소스, 테스트, 워크플로, bun.lock 은 건드리지 않는다. 제목은 fix(release): advance dev to 2.36.1 이고, 베이스는 dev, 헤드는 ingw/fix-dev-version-2.36.1 이다. 미리보기 배포는 계획에 없고, types.ts/config.ts 분할과도 안 겹친다.

왜 한 줄이 필요한가. 로컬에는 이미 v2.36.0, v2.36.0-preview.20260829, v2.36.0-preview.20260830 태그가 있다. tests/release-version-line.test.ts 는 트리 안 버전과 로컬 최고 릴리스 태그를 비교한다. 트리가 태그보다 뒤면 BEHIND로 실패한다. 숫자가 같아도, 그 태그가 가리키는 커밋이 지금 HEAD가 아니면 equal-but-not-tag-commit으로 실패한다. 지금 HEAD는 v2.36.0 이 찍힌 커밋이 아니다. 그런데 package.json 은 그 번호 그대로다. 그래서 테스트가 빨개진다. 릴리스 스크립트도 이미 공개된 번호를 다시 쓰려 하면 거절한다.

이게 왜 위험한가. 테스트가 빨개지는 것만 문제가 아니다. devmain 에 합치면 package.jsonmain 쪽으로 풀릴 수 있다. 그러면 트리가 이미 공개된 2.36.0 을 자기 번호라고 주장한다. 소리는 안 나고, 같은 번호를 한 번 더 쓰는 길이 열린다. 테스트 주석이 바로 그 침묵 중복을 막으려고 이 가드를 넣었다고 적혀 있다. 고치지 않은 dev 에 기능 PR을 리베이스하면 그 PR의 테스트도 같은 빨강을 물려받는다. 버전과 상관없는 고침이 초록이 될 수 없다.

역사는 이렇다. #2836이 dev2.36.0 으로 올렸다. 그때는 그 번호가 아직 공개되지 않았고, 이미 찍혀 있던 preview 태그보다 한 칸 앞선 다음 스테이블이었다. 지금은 v2.36.0 이 실제로 잘려 있다. 그때 맞았던 숫자가 지금은 이미 쓰인 숫자다. dev 는 다음 패치로 가야 한다.
작성자가 고른 2.36.1 은 쓰이지 않은 다음 패치다.
레지스트리와 태그, 릴리스 모두에 그 번호가 없다.
2.37.0 으로 뛰면 다음 마이너가 비고 패치 자리가 사라진다. 이번은 패치 한 칸이면 충분하다.

bun.lock 을 안 고친 것도 맞다. 워크스페이스 패키지 버전을 그 파일이 들고 있지 않다. 버전 칸의 권위는 package.json 한 줄이다.
작성자 본문은 release-version-line 과 compatibility-version 테스트 4개가 초록이고 typecheck도 통과했다고 적었다.
한 줄 PR로 따로 올리는 것도 맞다. 기능 PR에 이 숫자를 섞으면 그 PR이 되돌려질 때 버전 줄까지 같이 되돌아간다. 번호만 고치는 PR을 먼저 넣는 편이 깨끗하다.

package.json:3 - version을 2.36.0에서 2.36.1로 올리는 한 줄. 이 PR의 전부다. 쓰이지 않은 다음 패치이고, 가드를 다시 초록으로 만든다. 다른 파일을 건드리지 않는다.
tests/release-version-line.test.ts - 이 PR이 이 파일을 고치지는 않는다. HEAD가 최고 태그와 숫자는 같고 커밋은 다를 때 실패하는 바로 그 가드다. 이 한 줄이 그 equal-but-not-tag-commit 실패를 푼다.

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

  • 2.36.1 로 패치만 올릴지, 2.37.0 으로 마이너를 뛸지. 지금은 쓰이지 않은 다음 패치면 충분하다
  • 이 PR을 기능 PR보다 먼저 넣을지. 넣지 않으면 다른 PR이 같은 빨강을 물려받는다
  • bug 라벨과 fix(release) 접두가 맞는지, chore 가 맞는지. 번호 한 줄이지만, 가드가 이미 빨개진 상태다

너의 추천
지금 바로 머지하세요. package.json 한 줄이 릴리스 버전 줄 가드를 다시 세우고, 쓰이지 않은 2.36.1dev 를 앞으로 보낸다. 2.37.0 으로 뛸 이유는 없다. 기능 PR에 이 숫자를 섞지 말고, 이 PR을 먼저 넣으세요. types/config 분할로 닫을 패치도 아니고 중복도 아니다.

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

@Ingwannu

Copy link
Copy Markdown
Owner Author

Exact head 9eda7ce4ad43892d4beaadf3362ba0b1d140c18d is now fully green, including the aggregate ci job, all four Linux shards, macOS, npm-global, keyring, service-lifecycle, React Doctor, hygiene, and target enforcement.

The PR remains the single package.json version correction from 2.36.0 to 2.36.1, rebased onto current dev@df8b3882f221b3d68eddcfc34cc3a6edccbb32b3. @lidge-jun Please provide the required non-author approval when convenient. I will not self-merge before that policy gate is satisfied.

@lidge-jun

Copy link
Copy Markdown
Owner

Superseded by #3013, now merged into dev as 870a2adb6. dev reports 2.37.0.

This is not a claim that 2.36.1 was wrong - your PR correctly identified the
broken invariant and verified the target was unused. The reason #3013 landed
instead is that it also fixes the cause, and the mechanism forces the version
choice.

scripts/release.ts only runs on main/preview, bumps package.json on that
branch, and release.yml ends at the GitHub release. Nothing advances dev, so
the version line has been repaired by hand four times: 32529c2b2 (2.24.2 to
2.27.0), e4a85d134 (2.32.1-preview to 2.34.0), 076ad3036 (2.34.0 to 2.35.0),
befcac3e1 (2.35.0 to 2.36.0 off a preview publish). Two of those came after
e4a85d134 added the detector, so the missing piece was never visibility.

#3013 adds scripts/bump-dev-version.ts plus a release-triggered workflow that
opens the bump as a reviewable PR against dev. The rule has to be a pure
function of the released version, because a workflow cannot see what dev will
contain: prerelease of X.Y.Z gives dev X.Y.Z, stable X.Y.Z gives dev
X.(Y+1).0. That is the only formulation that reproduces all four historical
repairs, and it is what makes the target here 2.37.0 rather than 2.36.1.

One honest caveat, also stated on #3013: the workflow is dormant until dev is
promoted to main, since a release event only fires from the default branch.
The next release cut from the current main will strand dev one last time.

If maintainers would rather the policy be patch-first, that is a real choice and
it belongs in decideDevVersion and its table-driven tests, not in a one-off
version edit. Say so and I will open that change.

@lidge-jun lidge-jun closed this Aug 30, 2026
@lidge-jun lidge-jun added the landed-via-maintainer Original PR closed after landing via a maintainer merge train label Aug 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working landed-via-maintainer Original PR closed after landing via a maintainer merge train

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants