Skip to content

fix: 이슈 6 대화형 시연과 결과 표기 마감 - #8

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

fakeminjun7321 merged 1 commit into
mainfrom
one-way/issue-6-complete

Conversation

@fakeminjun7321

@fakeminjun7321 fakeminjun7321 commented Aug 24, 2026

Copy link
Copy Markdown
Member

배경

PR #7 병합 뒤 수용 기준 재감사에서 실제 시연의 대표성과 결과 표기에 남은 간극을 확인해 마감합니다.

변경 사항

  • install: 실제 hardware-fit 경고에서 No로 취소, 다운로드 전에 종료
  • run: 격리된 metadata-only registry의 실제 모델 선택기에서 Esc 취소, engine/launcher 진입 전 종료
  • contribute: GET-only loopback fixture에서 실제 사전 안내 후 No 취소, POST·benchmark·upload 없음
  • setup: 실제 wizard·hardware·runner 선택 화면에서 Esc 취소, installer 진입 전 종료
  • manifest에 success / cancelled / guard, 입력 이벤트, fixture, 안전 경로를 기록하고 verifier가 계약을 검사
  • 화면에 실제 명령, 종료 코드, 결과와 안전 설명을 manifest 기준으로 표시
  • 실제 영상과 별도의 장문 문서 예시를 시각·스크린리더 레이블에서 명확히 구분

검증 수준

Implemented

  • PR #7의 6개 명령 상세 문서·검색·한영 경로에 이 PR의 실제 시연 결과 표기와 안전 취소 흐름을 통합
  • squash merge commit: 394f91bb5b6ab4553c630e05efe88047af72a608

Unit-verified

  • npm run check:issue6: 검색 4/4, 문서 검증 6/6, 데모 6개 무결성 검사 통과
  • 최신 omm/main e66db6f82e83 기준 문서 드리프트 검사 통과
  • ffprobe 포함 6개 영상 검증 통과: H.264, 1280×720, yuv420p, 오디오 0, 해시 일치
  • npx tsc --noEmit, npm run lint, npm run build, git diff --check 통과
  • 병합 커밋의 GitHub Actions verify 통과

Preview-verified

  • Workers Builds deploy/version 명령을 OpenNext CLI로 수정하고 build cache를 1회 초기화
  • preview build 68bd0807-5cdb-40fe-b832-067db4dbe6f8 성공
  • npx opennextjs-cloudflare upload, OpenNext cache 27/27 업로드 확인
  • preview Worker version 681ec06a-af25-41a5-b69e-e225741ded2d
  • /ko/commands/install HTTP 200 및 실제 명령·사용자 취소·명령 사이드바 확인

Production-deployed

  • production build 3752f1f4-892b-432a-a8e4-a2f7f5a550c2 성공
  • npx opennextjs-cloudflare deploy, OpenNext cache 27/27 업로드 확인
  • production Worker version d1787fc0-4d9a-40b5-92b9-0fdbfff15ef1

Browser-verified

  • https://omm.run/commands: 6개 명령 표시, json huggingface AND 검색 결과가 omm search 1개로 축소, console error 0
  • https://omm.run/ko/commands/install: 실제 명령, 종료 코드 0, 사용자 취소, 6개 명령 사이드바 확인
  • 공개 install MP4가 1280×720, 5.375초, readyState 4로 로드되고 재생 시간이 1.15초 이상 진행, media error·console error 0
  • 이번 승인 범위의 intentional top-level/direct 요청 4회 사용(preview 2, production 2). 첫 preview 요청은 로컬 응답 파서 실패로 판독하지 못해 요청 수에 포함했고 재시도하지 않음. 429·Cloudflare 1027·quota/billing 응답 없음

Not verified / 미검증

  • 실제 모델 다운로드·실행·benchmark·telemetry upload는 안전한 취소 시연 범위 밖이라 수행하지 않음
  • 실제 모바일 기기
  • Vercel은 비정규 legacy dshs/omm-site 통합에서 실패하며 canonical Cloudflare 배포와 별도

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:21pm

@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 6d8ad7a Commit Preview URL

Branch Preview URL
Aug 24 2026, 01:27 PM

@fakeminjun7321
fakeminjun7321 merged commit 394f91b into main Aug 24, 2026
3 of 4 checks passed
@fakeminjun7321
fakeminjun7321 deleted the one-way/issue-6-complete branch August 24, 2026 13:30
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 6d8ad7af 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