Repository navigation
fix: 포트폴리오·안전점검 데모 접근성과 Windows 잠금 재시도 - #1
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9b40e08976
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
- Darken status badge foreground tokens so all five pairs clear 4.5:1 contrast
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ae5e381c84
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
- README: mark npm run check as source-checkout-only; installation check stays the effective agent list - demo-safety: expose inspection grid with table/row/columnheader/cell roles
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a745832d94
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
- demo-safety: fix inspection grid final track to 120px so header and rows share column alignment
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 20ac003320
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
- Keep mobile column headers in a11y tree via visually-hidden technique - Wrap inspection-row action buttons in role=cell containers (all 6 rows)
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 927d56eb05
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
- demo-safety: darken avatar initial foreground so 12px text clears 4.5:1 contrast
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5e74073112
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
- README demo: regenerate safety-field-inspection.png from corrected HTML so screenshot carries darkened status contrast
khakisketch
changed the title
Summary
README에서 KSI 작업 방식과 선택형 설치기 범위를 나누어 설명하고, 한국어 현장 안전점검 UI를 정적 데모로 볼 수 있게 했습니다. 모바일 표 머리말은 보조기기에 유지하고 상태·아바타 색 대비를 4.5:1 기준에 맞췄으며, Windows 상태 잠금 경합 실패를 제한 시간까지 재시도합니다.
시연 화면은 설치기와 별개의 정적 데모이며, 현장 정보는 가상이고 실제 운영·납품·저장·API 연동이 없음을 명시했습니다. 잠금 재시도는 EPERM·EACCES·EBUSY를 기존 EEXIST와 같은 점유로 취급하며, POSIX 동작과 만료된 잠금 정리 방식은 그대로입니다. 스크린샷은 수정된 HTML에서 다시 생성했습니다.
Demo
현장 안전점검 UI 시연
Validation
npm run check— 222개 통과, 2개 건너뜀, 실패 0개.npm run check:package— 실제 패키지 묶음의 미리보기·설치·기존 설정 보호 검사 통과.build·check매트릭스(Ubuntu/macOS/Windows, Node 20/22)·deploy·report-build-status모두 통과.