Skip to content

[DOCS] timo-refactor 스킬에 공통 유틸 배치 기준 추가 #150

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
jjangminii merged 2 commits into from
Jul 12, 2026
Merged

[DOCS] timo-refactor 스킬에 공통 유틸 배치 기준 추가 #150

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
1f9826d
docs(root): timo-refactor 스킬에 공통 유틸 배치 기준을 추가한다 (#148)
jjangminii Jul 12, 2026
9184b74
docs(root): timo-refactor 스킬의 JSDoc 규칙과 예시를 다듬는다 (#148)
jjangminii Jul 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 11 additions & 3 deletions .agents/skills/quality/timo-refactor/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -70,11 +70,19 @@ After: _hooks/useLoginForm.ts 로 분리, 컴포넌트는 JSX만 반환

### 공통 유틸 추출

같은 변환·포맷 로직이 3곳 이상이면:
같은 변환·포맷 로직이 3곳 이상이면 아래 기준으로 위치를 정한다:

- **공통** (여러 도메인에서 재사용 가능) → `apps/timo-web/utils`(함수) 또는 `apps/timo-web/constants`(상수)로 추출
- **도메인 종속** (특정 기능에서만 사용) → 해당 도메인의 `_utils` 폴더 안에 유지
- 새로 만들거나 옮길 때는 JSDoc을 반드시 작성한다
- **함수(utils)**: 설명, `@param`, `@returns`, 필요 시 `@example`
- **상수(constants)**: 설명, 단위(있는 경우), 사용 범위(어디서/왜 쓰이는지)

```text
Before: 각 컴포넌트 안에 동일한 날짜 포맷 함수
After: lib/format.ts 로 추출 후 임포트
Before: apps/timo-web/app/[locale]/(main)/focus/_utils/duration.ts
(SECONDS_PER_MINUTE, convertDurationToTimeText 로컬 정의 — home/timer 등 다른 도메인에도 유사 로직 중복)
After: apps/timo-web/constants/time.ts (SECONDS_PER_MINUTE 등 공통 상수)
apps/timo-web/utils/convert-duration-to-time-text.ts (convertDurationToTimeText, JSDoc 포함 공통 함수)
```

---
Expand Down
Loading