Skip to content

chore(scripts): 에이전트용 검증 커맨드 정비 - #34

Merged
Bori-github merged 1 commit into
mainfrom
chore/agent-verify-scripts
Jul 28, 2026
Merged

chore(scripts): 에이전트용 검증 커맨드 정비#34
Bori-github merged 1 commit into
mainfrom
chore/agent-verify-scripts

Conversation

@Bori-github

Copy link
Copy Markdown
Owner

요약 / 의도

pnpm test가 watch 모드라 CI 밖에서 실행하면 종료되지 않았다. CI만 --run
붙여 피해가고 있었고, 로컬·에이전트는 타임아웃까지 매달렸다. 검증 명령을
pnpm verify 하나로 모아 CI와 로컬이 같은 경로를 밟게 한다.

범위

  • tests (스크립트만)
  • CI · 툴링 · 설정

이번에 한 것

  • test·test:coveragevitest run으로, watch는 test:watch로 분리
  • 루트 test@zpl-kit/tests로 위임
  • build:libs·verify 추가 — build:libs → lint → format:check → type-check → test
  • 워크플로 3개가 같은 스크립트를 쓰도록 정리 (test --run 제거)

의도적으로 안 한 것 / 후속

  • 루트 vitest.config.ts가 이제 어떤 스크립트에서도 쓰이지 않는다 —
    죽은 설정 정리 PR에서 husky 잔재·docs/index.md와 함께 제거
  • CLAUDE.md 계층 분리, Claude Code 훅·권한 설정은 별도 PR

공개 패키지 영향

  • 공개 API 변경: 없음
  • 생성되는 ZPL 출력이 달라지는가: 아니오
  • 버전·태그 필요: 불필요 — 스크립트·워크플로만 변경

검증 (체크가 아니라 값으로)

  • pnpm verify: 통과 (exit 0, 5단계 전부 실행, 42 tests)
  • 프로세스 종료: watch 잔류 없이 정상 종료
  • pnpm test:watch: 20초 후에도 실행 중(exit 124) → watch 정상
  • pnpm --filter @zpl-kit/tests test:coverage: exit 0
  • 새 동작에 테스트 추가: 해당없음 — 스크립트·워크플로 변경
  • 자동 검증 불가: 워크플로 3개의 실제 동작은 이 PR의 CI 실행으로 확인된다

리뷰 포인트 / 위험 지점

  • 루트 test를 tests 패키지로 위임한 판단 — 루트 vitest는 dist를,
    tests/vitest.config.tsdevelopment 조건으로 src를 해석해 같은 테스트가
    두 경로로 갈라져 있었다. 둘 다 42개 통과하지만 src 수정 후 빌드를 안 하면
    결과가 달라진다
  • test:coverage도 watch 모드였다 — 계획에 없던 추가 수정

`pnpm test`가 watch 모드라 CI 밖에서 실행하면 종료되지 않았다. CI만
`--run`을 붙여 피해가고 있었고, 로컬·에이전트는 타임아웃까지 매달렸다.

- test·test:coverage를 `vitest run`으로 바꾸고 watch는 test:watch로 분리
- 루트 test를 @zpl-kit/tests로 위임. 루트 vitest는 dist를, tests 패키지는
  development 조건으로 src를 해석해 같은 테스트가 두 경로로 갈라져 있었다
- build:libs·verify 추가. verify는 build:libs → lint → format:check →
  type-check → test 순으로, exports가 dist를 가리키는 탓에 생기는
  빌드 선행 조건을 스크립트 안에 넣는다
- 워크플로 3개도 같은 스크립트를 쓰도록 정리

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@Bori-github Bori-github self-assigned this Jul 28, 2026
@vercel

vercel Bot commented Jul 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
zpl-viewer Ignored Ignored Jul 28, 2026 8:41am

@Bori-github Bori-github added the chore Maintenance, tooling, or CI — no user-facing change label Jul 28, 2026
@Bori-github
Bori-github merged commit 283a92b into main Jul 28, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Maintenance, tooling, or CI — no user-facing change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant