Skip to content

feat(cli): 시연용 샘플 데이터를 쌓는 sil seed-demo 추가 (SIL-101) - #46

Closed
space-angel wants to merge 1 commit into
mainfrom
feat/sil-101-demo-seed
Closed

space-angel wants to merge 1 commit into
mainfrom
feat/sil-101-demo-seed

Conversation

@space-angel

Copy link
Copy Markdown
Contributor

관련 이슈

무엇을 바꿨나요

  • pnpm sil seed-demo 명령을 더했어요. 시연 화면이 비어 보이지 않게 DB 함수(save_standard, revert_standard, save_answer)를 정해진 순서로 불러 이력을 쌓아요.
  • 시드 5단계 (apps/cli/src/demo-seed/steps.ts): 경하 PR에서 찾은 변경(notify.answer.two-questions v2) → 경하 app.screen.inbox-first v2 → 정민 v3(신지 v2는 대체됨) → 신지 v3에 답함 → 정민 v1로 되돌림(v4). app.notify.title-length는 시연 S2에서 라이브로 바꾸니 건드리지 않아요.
  • 안전장치: 기본은 미리 보기예요. 저장은 --apply와 연결된 DB 주소를 적는 --confirm-host가 함께 있어야 해요. 시드 전 상태가 아니면(결정 사항 버전이 다르거나 확인할 변경이 이미 있으면) 하나도 저장하지 않고 멈춰요.
  • 끝난 뒤 사람별 확인할 변경·확인한 변경 건수를 기대값과 비교해 보여 줘요.
  • apps/cli/README.md에 명령 설명을 더했어요.

왜 바꿨나요

  • 결정 사항과 답은 지울 수 없고 표에 직접 넣을 수도 없어서, 시연 DB에 이력을 쌓으려면 DB 함수를 순서대로 부르는 스크립트가 필요해요 (SIL-101 9-1).
  • 시드 문장과 사유는 시연에서 자연스럽게 읽히도록 정한 제안이에요.

어떻게 확인했나요

  • pnpm check 통과. 시드 테스트 6개를 더했어요 (run.test.ts: 저장 순서, 이미 돌렸을 때 멈춤, DB에 결정 사항이 없을 때, 답할 카드가 없을 때, 건수 비교).
  • 연결된 DB에서 pnpm sil seed-demo 미리 보기만 돌려 순서와 "시드 전 상태" 판정을 확인했어요 (읽기만 했어요).

직접 확인하지 못한 부분

  • --apply를 실제 DB에서 돌려 보지 않았어요. 결정 사항과 답은 지울 수 없어서, 시연용 DB가 생긴 뒤 리허설 DB에서 먼저 돌리고 건수가 기대와 같은지 확인해야 해요 (SIL-101 9-4).
  • 시드 문장·사유는 호윤님 확인 전이에요. "PR에서 찾음" 카드를 시연에 쓸지도 아직 정하지 않았어요 (SIL-101 9-3).
  • 시연 DB 교체 방법(.env 분리)은 이 PR에 없어요.

🤖 Generated with Claude Code

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
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