Skip to content

fix(test): reject unhandled browser smoke API requests - #1604

Draft
seonghobae wants to merge 1 commit into
codex/smoke-late-error-guardfrom
codex/smoke-unhandled-api-guard
Draft

fix(test): reject unhandled browser smoke API requests#1604
seonghobae wants to merge 1 commit into
codex/smoke-late-error-guardfrom
codex/smoke-unhandled-api-guard

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

목적

#1599에 기록된 별도 결함을 같은 smoke-source stack에서 수리합니다. 미등록 API 요청의 기본 HTTP 200 성공을 HTTP 501로 바꾸고, 브라우저가 오류를 숨겨도 기존 오류 수집 경로가 실패를 기록하게 합니다. URL·query·payload는 진단 메시지에 포함하지 않습니다.

변경과 검증

  • head: 287ca44, base: #1599의 0c91e52. 두 smoke 파일만 변경합니다.
  • 페이지 종료 중 미등록 API 요청이 성공으로 끝나는 RED: 14 passed / 1 failed.
  • 수정 후 두 focused test 파일: 17 passed, 7.02초.
  • ESLint 종료 코드 0, git diff check 통과.
  • 실제 로컬 브라우저 실행은 첫 화면 컴파일 준비 단계에서 기존 90초 제한으로 실패했습니다. 스크린샷 또는 제품 검증 성공으로 처리하지 않습니다. 실패 직후 호스트 load average는 103.81/74.52/60.27이었으나 자원 경합을 확정 원인으로 단정하지 않습니다. 제한 확대나 무관 프로세스 종료는 하지 않았습니다.

남은 검증

실제 10개 화면 및 반응형 검증, 현재 source-head CI, 독립 리뷰와 보호된 prerequisite 통합이 필요합니다. 기존 route별 광범위 prefix fixture가 모든 method/resource 계약을 검증한다고 주장하지 않습니다. 이 PR은 최종 미등록 fallback 결함만 수리합니다. AGENTS와 gap 문서는 기존 #1549/#1602 담당 경로를 유지합니다. 배포·병합·보안 gate 우회는 실행하지 않았습니다.

Record unmatched API requests before replying with a failure so silent browser handling cannot produce passing evidence, including during page cleanup.

Signed-off-by: Seongho Bae <me@seonghobae.me>

Co-authored-by: Codex <noreply@openai.com>
@coderabbitai

coderabbitai Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@seonghobae

Copy link
Copy Markdown
Contributor Author

Exact head 287ca44cc98d9c0bfc1c57944f53fcd737264d59 verification:

  • Installed the pinned frontend dependencies in an isolated worktree.
  • vitest run scripts/full-product-ui-smoke.test.mjs --run: 15 passed.
  • node --check frontend/scripts/full-product-ui-smoke.mjs: passed.
  • The initial dependency-missing Node invocation was environment setup failure, not product evidence; it was not counted as a test pass.

@seonghobae

Copy link
Copy Markdown
Contributor Author

Focused exact-head verification

  • exact head: 287ca44cc98d9c0bfc1c57944f53fcd737264d59
  • isolated detached worktree; corepack pnpm install --frozen-lockfile completed
  • corepack pnpm exec vitest run scripts/full-product-ui-smoke.test.mjs → 1 file, 15 tests passed
  • git diff --check → exit 0

This proves the changed smoke-contract tests locally only; hosted current-head Checks, review, protected merge, and deployment remain unverified.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working priority: high High-priority or P1 work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant