Skip to content

Move frontend demo to apps/demo and finalize Graphify docs#29

Merged
pko89403 merged 1 commit into
mainfrom
codex/graphify-demo-apps
Apr 21, 2026
Merged

Move frontend demo to apps/demo and finalize Graphify docs#29
pko89403 merged 1 commit into
mainfrom
codex/graphify-demo-apps

Conversation

@pko89403

Copy link
Copy Markdown
Owner

한 줄 요약

  • 이 PR은 정적 추천 데모를 docs/demo에서 apps/demo로 이동하고, Graphify-first 문서/검증 surface를 현재 구조에 맞게 정리합니다.

배경 / 문제

  • 기존 상태:
    • 데모 UI가 docs/demo 아래에 있어 문서 자산인지 프론트엔드 앱인지 경계가 흐렸습니다.
    • Graphify-first 전환 이후에도 일부 문서/스크립트/검증 surface에 legacy 경로 설명이 남아 있었습니다.
  • 문제:
    • 저장소 구조만 보고는 UI 앱 위치를 짐작하기 어려웠습니다.
    • Graphify refresh 스크립트는 bare worktree 환경에서 루트 해석이 깨질 수 있었습니다.
  • 이번 PR의 목표:
    • 데모 앱을 apps/demo로 옮겨 역할을 명확히 합니다.
    • README / docs / graphify script / verify artifact를 현재 운영 모델에 맞게 정리합니다.

PR 대시보드

항목 내용
유형 refactor / docs
영향 범위 docs / harness / tests / demo frontend
동작 변경 있음
Breaking Change 없음
관련 이슈 없음
기준 브랜치 main

변경 묶음별 리뷰 가이드

변경 묶음 파일/디렉토리 무엇이 바뀌었나 리뷰 포인트 실행 없이 보는 법
데모 위치 정리 apps/demo/, docs/demo/ 정적 recommendation demo를 apps/demo/로 이동 링크/스크린샷/테스트 경로 일치 여부 rename 경로와 README 링크만 보기
문서/하네스 정리 README.md, README.ko.md, docs/README.md, .harness/reference/README.md, raw/..., graphify-out/... Graphify-first 운영 모델과 demo 위치를 문서에 반영 구현과 설명의 경로 일치 여부 경로/용어만 비교
Graphify refresh 안정화 scripts/graphify_*.sh, .harness/hooks/graphify-auto-refresh.sh, graphify-out/* bare worktree fallback 추가, verified full refresh marker 반영 repo root fallback과 verify artifact 반영 여부 shell diff와 BUILD_INFO/marker 보기
테스트 tests/... legacy docs path 회귀 방지 체크 추가 현재 active surface가 raw//apps/demo 기준인지 테스트 이름과 assert만 읽기

Before / After

Before

  • 데모 UI가 docs/demo에 있어 docs archive와 live frontend surface가 섞여 보였습니다.
  • 일부 active 문서와 Graphify 스크립트에 legacy surface/환경 가정이 남아 있었습니다.

After

  • 데모 UI는 apps/demo로 분리되어 프론트엔드 앱으로 더 명확해졌습니다.
  • README와 docs archive 설명이 현재 Graphify-first 구조를 기준으로 정리됐습니다.
  • Graphify refresh 스크립트가 bare worktree에서도 repo root를 안정적으로 찾습니다.
  • verified full refresh marker가 커밋되어 현재 graph trust state를 같이 보여줍니다.

검증

항목 결과 근거
테스트 uv run pytest tests/test_graphify_harness.py
테스트 node --test apps/demo/tests/*.cjs
린트 pre-commit에서 uv run ruff check --fix 통과
타입체크 pre-push에서 uv run mypy src 통과

남은 리스크 / 후속 작업

  • docs/wikidocs/sources를 legacy archive로 계속 둘지, 별도 정리 PR에서 제거할지 결정
  • 필요하면 apps/ 하위 앱 공통 규칙(예: 별도 README, 실행 스크립트) 추가
  • 주의할 점:
    • 로컬 dirty worktree에 unrelated staged 삭제가 있어 pre-push 전체 pytest는 branch 내용이 아니라 현재 워크트리 상태에 영향을 받았습니다.
  • 리뷰 시 특히 볼 부분:
    • README 링크와 apps/demo rename이 모두 일치하는지
    • Graphify refresh fallback이 bare git/worktree 환경에서도 자연스럽게 동작하는지

@pko89403 pko89403 merged commit a016d51 into main Apr 21, 2026
2 checks passed
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