Skip to content

feat: 이슈 6 명령어별 시연 영상과 상세 문서 완성 - #7

Merged
fakeminjun7321 merged 2 commits into
mainfrom
one-way/issue-6-complete
Aug 24, 2026
Merged

fakeminjun7321 merged 2 commits into
mainfrom
one-way/issue-6-complete

Conversation

@fakeminjun7321

@fakeminjun7321 fakeminjun7321 commented Aug 24, 2026

Copy link
Copy Markdown
Member

변경 사항

  • search, install, run, recommend, contribute, setup 전용 문서와 실제 격리 CLI 실행 기반 MP4, poster, 원문 transcript를 추가했습니다.
  • 한영 명령어 문서 검색, / 포커스, Escape 초기화, 다중 키워드 AND 검색을 구현했습니다.
  • 각 페이지 사이드바에서 여섯 명령을 이동하고 현재 문서를 확인할 수 있게 했습니다.
  • 전체 옵션, 별칭, 예제 3~5개, 관련 명령, 실제 오류와 대처법을 최신 omm 소스에 맞췄습니다.
  • AST 기반 문서 드리프트 검사와 일일 GitHub Actions 검증을 추가했습니다.
  • locale 링크와 rewrite 문제, 모바일 검색 지우기 버튼의 44px 터치 영역, Footer의 install 문서 링크를 보완했습니다.
  • install, run, contribute, setup 영상은 실제 대화형 CLI 화면까지 진입한 뒤 다운로드·모델 실행·벤치마크·업로드 전에 사용자가 취소하는 흐름으로 다시 캡처했습니다.
  • manifest의 success / cancelled / guard 결과와 실제 입력·fixture·안전 경로를 화면의 명령·종료 코드·설명에 그대로 연결했습니다.

검증 수준

Implemented

  • 여섯 명령 페이지, 자체 호스팅 데모, 한영 검색, 명령 사이드바, 원문 transcript, 소스 동기화 검사가 포함되어 있습니다.

Unit-verified

  • npm run check:issue6: 검색 4/4, 문서 검증 6/6, 데모 6개 무결성 검사 통과
  • 최신 omm/main e66db6f82e83 기준 옵션·기능·오류 출처 드리프트 검사 통과
  • ffprobe 포함 6개 영상 검증 통과: H.264, 1280×720, yuv420p, 오디오 0, manifest 해시 일치
  • npx tsc --noEmit, npm run lint, npm run build, git diff --check 통과

Browser-verified

  • localhost OpenNext/workerd에서 여섯 영문 페이지와 한글 경로 모두 HTTP 200
  • 검색 / 포커스, AND 검색, Escape, 빈 결과, 결과 클릭 이동 확인
  • 사이드바 6개 링크와 aria-current, locale 유지 확인
  • 여섯 MP4의 실제 source·duration·1280×720 metadata 확인, search 영상 재생과 transcript 일치 확인
  • 새 install 영상의 실제 명령, cancelled 결과, 안전 설명, 문서 예시용 스크린리더 레이블 확인
  • 390×844에서 가로 넘침 없음, 검색 지우기 버튼 44px 이상
  • console/page 오류 0개

Remote checks

  • 직전 커밋 f501502의 GitHub Actions와 Cloudflare Workers preview build 통과
  • 최신 커밋 26e57f7의 원격 검사는 진행 중
  • Vercel은 접근 권한이 없는 기존 dshs/omm-site 통합에서 실패하며, canonical 배포 대상인 Cloudflare와 별도입니다.

Not verified / 미검증

  • Cloudflare branch preview의 애플리케이션 경로: 현재 Workers Builds가 OpenNext upload 대신 직접 wrangler versions upload를 실행해 정적 캐시 population을 건너뛰는 설정 문제를 확인했습니다.
  • 프로덕션 병합·배포 및 공개 omm.run 사용자 경로
  • 실제 모바일 기기
  • 실제 모델 다운로드·실행·benchmark·telemetry 업로드는 안전상 시연하지 않았습니다.

Refs #6

@vercel

vercel Bot commented Aug 24, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
omm-site Error Error Aug 24, 2026 1:08pm

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 24, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
omm f501502 Commit Preview URL

Branch Preview URL
Aug 24 2026, 01:09 PM

@fakeminjun7321
fakeminjun7321 merged commit 8478228 into main Aug 24, 2026
3 of 4 checks passed
minigu5 added a commit that referenced this pull request Aug 24, 2026
…line

Another contributor's PR (#7, #8) landed a parallel implementation of issue
#6 on main while this branch worked independently: self-hosted mp4/png
demos, a search UI, and a verification pipeline. Reviewed it and found real
quality problems — install.txt and run.txt end in "Cancelled."/"Aborted!"
instead of a completed run, and search.txt/recommend.txt use flags despite
this branch's own "flag-less, general case" fix a few commits back.

Per the user's explicit call, keeping this branch's version: text content
with the scroll-triggered typing animation (CommandCapture.tsx), no video
files, no search UI, no separate verification pipeline. Kept the other
PR's two genuinely unrelated improvements — the locale-rewrite redirect
loop fix in middleware.ts, and wiring omm install's footer link internal.
Everything specific to the abandoned approach (public/demos/commands/,
CommandDemo.tsx, CommandSearch*, the capture/verify scripts, their CI
workflow, docs/command-docs-sync.md) is removed.

This branch had an error being deployed

1 failed deployment
Preview f501502c Deployed Aug 24, 2026 by vercel[bot]
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