Skip to content

fix(release): action smoke native runtime 설치 수정#98

Merged
JeremyDev87 merged 1 commit into
masterfrom
feature/release-action-smoke-fix
Apr 30, 2026
Merged

fix(release): action smoke native runtime 설치 수정#98
JeremyDev87 merged 1 commit into
masterfrom
feature/release-action-smoke-fix

Conversation

@JeremyDev87

Copy link
Copy Markdown
Owner

배경

v0.1.3 릴리스에서 npm publish와 published wrapper smoke는 성공했지만, 마지막 GitHub Action smoke가 checkout된 로컬 action path를 npm package로 설치하면서 native optional runtime package를 받지 못해 실패했습니다.

release-docs도 tag push에서 실행되며 README 예시 태그를 현재 package.json 버전으로 강제하려고 해 기존 문서 계약과 충돌했습니다.

변경 사항

  • root action.yml과 marketplace wrapper action이 checkout된 로컬 경로 대신 해당 action ref의 package.json 버전을 읽어 publish된 @jeremyfellaz/maximus@<version>을 설치하도록 변경했습니다.
  • release-docs workflow가 master branch push에서만 실행되도록 제한하고, package.json 단독 버전 변경으로 tag push 문서 drift 검사가 다시 실패하지 않게 했습니다.
  • release docs generator의 README 예시 태그를 정적 예시로 유지하도록 정리했습니다.
  • release docs generator 계약 테스트를 추가하고 release wiring 검증 스크립트를 새 설치 계약에 맞췄습니다.

검증

  • node ./scripts/update-release-docs.mjs --check
  • node ./scripts/validate-rust-release-wiring.mjs
  • node --test test/release-workflow-context.test.js test/github-action-wiring.test.js test/release-docs.test.js test/release-plan.test.js test/release-candidate-metadata.test.js test/release-helpers.test.js test/npm-error-classifiers.test.js test/bump-release-version.test.js
  • git diff --check
  • cargo metadata --format-version 1 --no-deps
  • cargo test -p maximus-core
  • cargo test -p maximus-cli
  • npm test
  • npm install --no-package-lock --no-audit --no-fund --cache /private/tmp/maximus-published-action-smoke-0.1.3/.npm-cache --prefix /private/tmp/maximus-published-action-smoke-0.1.3 @jeremyfellaz/maximus@0.1.3
  • node ./scripts/assert-installed-native-runtime.mjs /private/tmp/maximus-published-action-smoke-0.1.3
  • node /private/tmp/maximus-published-action-smoke-0.1.3/node_modules/@jeremyfellaz/maximus/bin/maximus.js audit ./test/fixtures/clean-project

브랜치 / 워크트리

  • base: master
  • head: feature/release-action-smoke-fix
  • worktree: /Users/pjw/workspace/maximus

@JeremyDev87

Copy link
Copy Markdown
Owner Author

독립 fresh-session 리뷰 결과

  • Verdict: APPROVE
  • Severity Summary: Critical 0 / High 0 / Medium 0 / Low 0
  • Findings: 없음
  • Scope Note: fresh-session 리뷰어가 crates/maximus-checks/src/npmignore_files.rsnpmignore-files false positive 가능성을 High로 제기했지만, 해당 파일과 crates/maximus-checks/src/registry.rs는 PR fix(release): action smoke native runtime 설치 수정 #98 diff에 포함되지 않았습니다. git diff origin/master...HEAD -- crates/maximus-checks/src/npmignore_files.rs crates/maximus-checks/src/registry.rs도 변경 없음으로 확인되어 이번 release/action smoke 수정 PR의 blocker로 반영하지 않았습니다.
  • Rationale: PR #98은 action/release-docs wiring만 수정하며, 로컬 merge-gate와 GitHub PR checks가 모두 통과했습니다. action ref에서 publish된 wrapper version을 설치하는 경로와 release-docs check 계약도 검증되었습니다.

@JeremyDev87 JeremyDev87 self-assigned this Apr 30, 2026
@JeremyDev87 JeremyDev87 added the fix Bug fix label Apr 30, 2026
@JeremyDev87 JeremyDev87 marked this pull request as ready for review April 30, 2026 16:12
@JeremyDev87 JeremyDev87 merged commit 22281fe into master Apr 30, 2026
43 checks passed
@JeremyDev87 JeremyDev87 deleted the feature/release-action-smoke-fix branch April 30, 2026 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant