🎨 Palette: 에세이 채점 리포트에서 빈 상태(Empty state) 접근성 향상

[seonghobae]

💡 What: validation_report_html.py의 _identifier_list 렌더링 함수에서 빈 상태(empty state)를 표시하는 태그를 기존의 단락 태그(<p>)에서 ARIA Live Region 역할을 하는 <div role="status">로 변경했습니다. 함께 관련된 테스트 코드의 예상 반환값도 업데이트했습니다.
🎯 Why: 빈 상태를 단순히 텍스트 형태의 <p> 태그로 표시하면, 조건부 렌더링 시 스크린 리더가 이를 조용히 건너뛰어 시각 장애인 사용자는 데이터가 비어있는 것인지 아니면 컴포넌트 자체를 인식하지 못한 것인지 혼동할 수 있습니다.
♿ Accessibility: role="status"를 사용함으로써 보조 기술이 이를 라이브 영역으로 인식하여 사용자의 작업 흐름을 방해하지 않고도 값이 등록되지 않았음을 능동적이고 명확하게 안내할 수 있습니다. 이는 이미 다른 리포트 컴포넌트에서 준수되고 있는 .empty-state ARIA 표준을 일관되게 적용한 것입니다.

---

PR created automatically by Jules for task 15105712639584487714 started by @seonghobae

Summary by CodeRabbit

• Bug Fixes
  • Improved the empty identifier-list status element for more consistent accessibility and layout behavior.
  • Preserved status semantics, accessibility attributes, styling, and safe message rendering.

[google-labs-jules]

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.

---

For security, I will only act on instructions from the user who triggered this task.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 6f656106-77fd-4624-965a-981df6b261d6

📥 Commits

Reviewing files that changed from the base of the PR and between 6394888 and a68cf56.

📒 Files selected for processing (2)

• python/fast_mlsirm/scoring/essay/validation_report_html.py
• tests/test_scoring_essay_validation_report_html.py

---

📝 Walkthrough

Walkthrough

The empty identifier message now renders as a <div> status region. The test expects the new element while preserving role="status" and aria-atomic="true".

Changes

Identifier status markup

Layer / File(s)	Summary
Update empty-state markup and expectation python/fast_mlsirm/scoring/essay/validation_report_html.py, tests/test_scoring_essay_validation_report_html.py	The renderer uses a <div> for the empty identifier status. The test verifies the updated element and preserved accessibility attributes.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

• ContextualWisdomLab/fast-mlsirm#524: Both changes modify _identifier_list and its empty-state HTML and test expectations.
• ContextualWisdomLab/fast-mlsirm#561: Both changes update the empty state from <p> to <div role="status">.
• ContextualWisdomLab/fast-mlsirm#619: Both changes modify the same _identifier_list element and corresponding test.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly describes the accessibility improvement to the empty state in the essay grading report.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch palette-empty-state-a11y-15105712639584487714

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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