Skip to content

fix: 토큰 정합성 수정, 테스트 보강 및 JSDoc 문서화#97

Merged
ohprettyhak merged 12 commits intov1from
fix/pr96-review-feedback
Mar 12, 2026
Merged

fix: 토큰 정합성 수정, 테스트 보강 및 JSDoc 문서화#97
ohprettyhak merged 12 commits intov1from
fix/pr96-review-feedback

Conversation

@ohprettyhak
Copy link
Contributor

@ohprettyhak ohprettyhak commented Mar 12, 2026

Summary

  • globals.css: body::before/::after 분리선에 pointer-events: none 추가 — 고정된 divider 가상 요소가 클릭/탭 입력을 차단하는 문제 수정
  • section.tsx: p-appp-6, text-mdtext-sm — Tailwind v4 미지원 유틸리티 제거
  • field.tsx: FieldContext 추가 및 useField() hook export — aria-describedby 지원을 위한 descriptionId/errorId context 노출
  • fix(tokens): --cocso-text-base 15px → 16px 복원 (PR feat: design system v2 — radius 토큰, 스크롤 개선, 웹 가이드라인 준수 #96 의도 수정)
  • fix(baseframe): tsconfig target ES2020 → ES2022, resolve.test.ts 정규식 최상위 이동 (Biome useTopLevelRegex)
  • test: 커버리지 낮은 파일 테스트 보강 (link/field/spinner/switch/tailwind/validate), css-vars/resolve 브랜치 커버리지 개선
  • fix(baseframe): golden snapshot token.css.expected text-base 16px로 업데이트 — CI Test 실패 수정
  • chore(website): dev 서버 호스트/포트 고정, CSS 속성 순서 정렬
  • docs: JSDoc/TSDoc 주석 추가 — React 컴포넌트(22), 토큰(4), 유틸리티(1), ecosystem/baseframe(13) 총 40개 파일 (+733 라인)
  • fix(typography): type="custom" 기본 font-size 15px → 16px — base token 변경과 정합성 맞춤

Test plan

  • CI Lint, Build 통과
  • 분리선(body::before/::after) 위 요소 클릭 정상 동작
  • Section 컴포넌트 패딩 24px 적용
  • useField()aria-describedby 참조 가능
  • golden snapshot 동기화 → Test 통과
  • --cocso-text-base: 16px 적용 확인
  • Typography type="custom" 기본 font-size 16px 정합성 확인 (321/321 테스트 통과)
  • JSDoc 추가 후 lint, type-check, build 전체 통과

🤖 Generated with Claude Code

- globals.css: body::before/::after 분리선에 pointer-events: none 추가
  → 상호작용 요소 위에 겹치는 divider가 클릭/탭을 차단하지 않도록 수정
- section.tsx: p-app → p-6 (Tailwind --spacing-* 네임스페이스 사용)
  → Tailwind v4에서 --padding-app은 p-* 유틸리티를 자동 생성하지 않음
- section.tsx: text-md → text-sm (Tailwind 표준 스케일)
  → text-md는 Tailwind 기본 타이포그래피 스케일에 없는 클래스
- field.tsx: FieldContext 추가 및 useField() hook export
  → aria-describedby 지원을 위해 descriptionId/errorId를 context로 노출

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@vercel
Copy link

vercel bot commented Mar 12, 2026

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

Project Deployment Actions Updated (UTC)
cocso-ui Ready Ready Preview, Comment Mar 12, 2026 5:47pm
cocso-ui-storybook Ready Ready Preview, Comment Mar 12, 2026 5:47pm

Request Review

@changeset-bot
Copy link

changeset-bot bot commented Mar 12, 2026

⚠️ No Changeset found

Latest commit: 4a2b3ed

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions
Copy link
Contributor

github-actions bot commented Mar 12, 2026

Test Coverage

Package Statements Branches Functions Lines
@cocso-ui/react 🟢 97.49% 🟢 94.73% 🟢 96.92% 🟢 97.22%
@cocso-ui/baseframe 🟢 92.79% 🟡 85.31% 🟢 100% 🟢 92.66%
@cocso-ui/react — File Coverage
File Stmts Branch Funcs Lines
src/badge/badge.tsx 🟡 88.63% 🟡 85.71% 🟡 85.29% 🟡 86.48%
src/input/input.tsx 🟢 93.33% 🟡 81.25% 🟢 100% 🟢 92.85%
src/select/select.tsx 🟢 95.23% 🟡 81.25% 🟢 100% 🟢 94.73%
src/typography/typography.tsx 🟢 96.96% 🟢 100% 🟢 95% 🟢 96.55%
src/button/button.tsx 🟢 98.5% 🟢 100% 🟢 98.03% 🟢 98.33%
src/cn.ts 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/accordion/accordion.tsx 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/checkbox/checkbox.tsx 🟢 100% 🟢 92.3% 🟢 100% 🟢 100%
src/day-picker/day-picker.tsx 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/dropdown/dropdown.tsx 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/field/field.tsx 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/link/link.tsx 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/modal/modal.tsx 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/month-picker/month-picker.tsx 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/one-time-password-field/one-time-password-field.tsx 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/pagination/pagination.tsx 🟢 100% 🟢 94.73% 🟢 100% 🟢 100%
src/popover/popover.tsx 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/radio-group/radio-group.tsx 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/spinner/spinner.tsx 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/stock-quantity-status/stock-quantity-status.tsx 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/switch/switch.tsx 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/tab/tab.tsx 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/token/color.ts 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/token/radius.ts 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/token/spacing.ts 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/token/typography.ts 🟢 100% 🟢 100% 🟢 100% 🟢 100%
@cocso-ui/baseframe — File Coverage
File Stmts Branch Funcs Lines
src/core/builders/utils/css.ts 🟡 86.66% 🟡 85.71% 🟢 100% 🟡 86.66%
src/core/parsers/value.ts 🟡 88.07% 🟡 85.85% 🟢 100% 🟡 87.91%
src/core/builders/css-vars.ts 🟢 95.83% 🟡 85.71% 🟢 100% 🟢 95.45%
src/core/transforms/validate.ts 🟢 95.52% 🔴 68.18% 🟢 100% 🟢 95.52%
src/core/builders/tailwind.ts 🟢 97.36% 🟡 89.47% 🟢 100% 🟢 97.36%
src/core/builders/utils/naming.ts 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/core/parsers/ast.ts 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/core/transforms/build.ts 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/core/transforms/resolve.ts 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/core/types/ast.ts 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/core/types/domain.ts 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/core/types/results.ts 🟢 100% 🟢 100% 🟢 100% 🟢 100%
src/utils/fs.ts 🟢 100% 🟢 100% 🟢 100% 🟢 100%

Updated: 2026-03-12 17:47 UTC

Array.at() 사용으로 인한 check-types 에러 수정.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@vercel vercel bot temporarily deployed to Preview – cocso-ui March 12, 2026 08:51 Inactive
@vercel vercel bot temporarily deployed to Preview – cocso-ui-storybook March 12, 2026 08:51 Inactive
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- link.tsx: ExternalIcon, variant CSS vars, indicator 명시 테스트 추가 → funcs 54%→100%, lines 66%→100%
- field.tsx: useField() hook 컨텍스트 검증 테스트 추가 → funcs 50%→100%
- spinner.tsx: 6가지 variant 색상, style 병합 테스트 추가 → funcs 60%→100%
- switch.tsx: size large, variant CSS vars, custom id 테스트 추가 → funcs 68%→100%
- tailwind.ts: banner 옵션, generateTailwindCSS 직접 호출, 빈 컬렉션 브랜치 커버 → branch 68%→89%
- validate.ts: ShadowLayer/Shadow 토큰 참조 검증 테스트 추가

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- css-vars.ts: :root 폴백, banner 옵션, generateCssVariables 직접 호출 커버 → branch 71%→개선
- resolve.ts: resolveTokenRef/resolveValue throw 경로, ShadowLayer/Shadow 재귀 해결 커버 신규 테스트 파일 추가 (13 cases)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@vercel vercel bot temporarily deployed to Preview – cocso-ui-storybook March 12, 2026 09:40 Inactive
@vercel vercel bot temporarily deployed to Preview – cocso-ui March 12, 2026 09:40 Inactive
@vercel vercel bot temporarily deployed to Preview – cocso-ui March 12, 2026 09:44 Inactive
@vercel vercel bot temporarily deployed to Preview – cocso-ui-storybook March 12, 2026 09:44 Inactive
ohprettyhak and others added 2 commits March 12, 2026 19:33
CSS 토큰, Typography/Link 컴포넌트, 테스트 일괄 반영.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- next dev에 --hostname 0.0.0.0 --port 6005 옵션 추가
- globals.css divider 스타일 CSS 속성 순서 Biome 기준으로 정렬

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
621fdbb에서 --cocso-text-base를 15px → 16px로 변경했으나
골든 스냅샷이 미반영되어 CI Test 실패 발생

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@vercel vercel bot temporarily deployed to Preview – cocso-ui March 12, 2026 17:08 Inactive
@vercel vercel bot temporarily deployed to Preview – cocso-ui-storybook March 12, 2026 17:08 Inactive
@ohprettyhak ohprettyhak changed the title fix: PR #96 Codex 리뷰 피드백 반영 fix: PR #96 리뷰 피드백 반영 및 후속 안정화 Mar 12, 2026
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a33749c828

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

…aseframe

Add comprehensive JSDoc documentation to 40 TypeScript files covering
React component props/functions, design token exports, and the baseframe
token generator pipeline. No logic changes — comments only.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ohprettyhak
Copy link
Contributor Author

JSDoc/TSDoc 주석 추가 (4fbf44c)

전체 프로젝트에 JSDoc/TSDoc 주석을 추가했습니다.

변경 범위

  • 40개 파일, +733 라인 (주석만 추가, 로직 변경 없음)
영역 파일 수 내용
React 컴포넌트 22 전체 컴포넌트 Props 인터페이스, 타입, 함수 문서화
토큰 시스템 4 color, radius, spacing, typography
유틸리티 1 cn.ts
Ecosystem/Baseframe 13 types, parsers, builders, transforms, CLI

검증

  • Lint: ✅ 전체 통과
  • Type check: ✅ 오류 없음
  • Build: ✅ 전체 통과
  • Tests: ✅ 320/321 통과 (1개 기존 OTP 테스트 실패 — JSDoc 무관)

Align the default font size for type="custom" with the base token
change from commit 621fdbb (text-base 15px → 16px).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ohprettyhak ohprettyhak changed the title fix: PR #96 리뷰 피드백 반영 및 후속 안정화 fix: PR #96 리뷰 피드백 반영, 후속 안정화 및 JSDoc 문서화 Mar 12, 2026
@ohprettyhak ohprettyhak changed the title fix: PR #96 리뷰 피드백 반영, 후속 안정화 및 JSDoc 문서화 fix: 후속 안정화 및 JSDoc 문서화 Mar 12, 2026
@ohprettyhak ohprettyhak changed the title fix: 후속 안정화 및 JSDoc 문서화 fix: 토큰 정합성 수정, 테스트 보강 및 JSDoc 문서화 Mar 12, 2026
@ohprettyhak ohprettyhak merged commit f4b73c7 into v1 Mar 12, 2026
9 checks passed
@ohprettyhak ohprettyhak deleted the fix/pr96-review-feedback branch March 12, 2026 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant