Skip to content

refactor: 레거시 JS 구현과 Node 테스트 제거 - #12

Merged
JeremyDev87 merged 1 commit into
masterfrom
codex/rust-js-retirement
Apr 19, 2026
Merged

refactor: 레거시 JS 구현과 Node 테스트 제거#12
JeremyDev87 merged 1 commit into
masterfrom
codex/rust-js-retirement

Conversation

@JeremyDev87

Copy link
Copy Markdown
Owner

배경

  • Rust cutover 이후에도 repository 안에 레거시 JS 분석 구현과 오래된 Node parity 테스트가 남아 있었습니다.
  • package_intelligence / impact parity 검증도 삭제 예정인 JS source를 직접 읽고 있어서, 마지막 retirement 단계 전에 snapshot 기반으로 옮길 필요가 있었습니다.

변경 사항

  • src/ 아래 레거시 JS business logic와 test/cli.test.js, test/analyze-project.test.js, test-support/parity-oracles.js 를 제거했습니다.
  • Rust 테스트가 tests/oracles/package-intelligence/registry.json, tests/oracles/impact/cases.json snapshot을 읽도록 바꿨습니다.
  • scripts/generate-rust-parity-fixtures.mjs 가 cargo run 반복 호출 대신 Rust CLI binary를 build 후 직접 실행하도록 정리했습니다.
  • CONTRIBUTING.md 의 현재 repo shape와 contributor 경로를 JS retirement 이후 상태에 맞게 갱신했습니다.

검증

  • cargo test --workspace
  • node --test test/launcher-platform.test.js
  • node ./scripts/generate-rust-parity-fixtures.mjs
  • npm run smoke
  • npm run pack:check
  • npm pack --dry-run --json --cache ./.npm-cache
  • node ./bin/legolas.js --version
  • node ./bin/legolas.js scan tests/fixtures/parity/basic-app

브랜치 / 워크트리

  • 대상 브랜치: master
  • 작업 브랜치: codex/rust-js-retirement
  • 작업 워크트리: /private/tmp/legolas-rust-js-retirement

이슈 연결

  • 별도 이슈 연결 없음

@JeremyDev87 JeremyDev87 added the refactor Code changes that restructure behavior without adding features label Apr 19, 2026
@JeremyDev87

Copy link
Copy Markdown
Owner Author

독립 fresh-session 리뷰 결과

  • Verdict: APPROVE
  • Findings: 없음
  • Rationale: 실제로 crates/legolas-corecrates/legolas-cli의 관련 테스트를 파일 단위로 확인했고, JS 구현 제거가 Rust 코어/CLI 테스트와 snapshot oracle로 대체된 뒤에도 merge-blocking regression이나 workflow/config 문제를 발견하지 못했습니다.

참고: 작성자 본인 PR이라 GitHub 정책상 공식 approve review 대신 PR comment로 남깁니다.

@JeremyDev87 JeremyDev87 self-assigned this Apr 19, 2026
@JeremyDev87
JeremyDev87 force-pushed the codex/rust-js-retirement branch from ffea8cf to 106ebc4 Compare April 19, 2026 00:21
@JeremyDev87
JeremyDev87 marked this pull request as ready for review April 19, 2026 00:22
@JeremyDev87
JeremyDev87 merged commit 064cdd1 into master Apr 19, 2026
7 checks passed
@JeremyDev87
JeremyDev87 deleted the codex/rust-js-retirement branch April 19, 2026 00:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor Code changes that restructure behavior without adding features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant