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
@lidge-jun
lidge-jun deleted the ingw/fix-dev-version-2.36.1 branch September 5, 2026 09:03
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