🎨 Palette: 새로고침 버튼 비활성화 상태 및 링크 접근성 개선

[seonghobae]

💡 무엇을 변경했나요:

• viewer.js에 폴링, 에러, 완료 상태에 따른 '새로고침' 버튼 비활성화(disabled) 로직 추가
• viewer.css에 비활성화 버튼 시각적 피드백(.btn:disabled) 추가 및 hover 상태 충돌 방지
• 외부 아티팩트 링크와 JSON 부트스트랩 링크에 target="_blank"와 rel="noopener noreferrer" 속성 적용
• ViewerUiController.java 내 버튼 그룹 영역에 role="group" 접근성 속성 추가

🎯 왜 변경했나요:

• 비동기 처리(폴링) 중에 사용자가 불필요하게 버튼을 여러 번 클릭하는 것을 방지하여 시스템 상태의 명확성을 제공하기 위함입니다.
• 외부 링크 클릭 시 뷰어 탭이 이탈되는 문제를 방지하고 웹 접근성 표준을 준수하기 위함입니다.

📸 변경 전/후: 변경 후 폴링 시 버튼이 비활성화되며, 링크들은 새 창에서 엽니다. (첨부된 동영상/스크린샷 참조)

♿ 접근성(Accessibility): 버튼 그룹에 role을 명시하고, 외부 링크 열기에 대한 명확한 속성을 추가했습니다.

---

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

[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.

[Copilot]

Pull request overview

This PR improves the viewer UI by preventing repeated “Refresh” clicks during asynchronous polling/loading, adding consistent disabled-state styling, and strengthening accessibility/security for action links and button grouping.

Changes:

• Disable/enable the “Refresh” button based on loading/polling vs. error/ready states in viewer.js.
• Add .btn:disabled styling and prevent hover styles from applying to disabled buttons in viewer.css.
• Open artifact/JSON bootstrap links in a new tab with rel="noopener noreferrer", and add role="group" to the action cluster in ViewerUiController.java.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File	Description
src/main/resources/static/assets/viewer/viewer.js	Adds refresh button disabled-state handling during polling/loading and updates dynamically created artifact link to open safely in a new tab.
src/main/resources/static/assets/viewer/viewer.css	Adds disabled visual feedback and ensures hover styles don’t override disabled appearance.
src/main/java/com/clearfolio/viewer/controller/ViewerUiController.java	Improves accessibility semantics for the actions area and updates the JSON bootstrap link to open in a new tab with safe rel attributes.
.jules/palette.md	Records the palette learning/action summary for this change set.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[opencode-agent]

OpenCode Review Overview

• Head SHA: fa3ddad1dcc8ffaeea04984626b8dbb73c897bf0
• Workflow run: 28303316390
• Workflow attempt: 1
• Gate result: REQUEST_CHANGES (approval step)

Pull request overview

OpenCode reviewed the current-head evidence but found unresolved human review threads before approval.

Findings

1. HIGH .github/workflows/opencode-review.yml:1 - Unresolved human review thread blocks automated approval

• Problem: OpenCode reached an APPROVE control result, but the approval step found unresolved, non-outdated human review thread evidence on the current pull request.
• Root cause: Human review feedback can arrive after bounded model evidence is prepared, so the approval step must re-query GitHub immediately before publishing an approval.
• Fix: Address or resolve the listed human review thread(s), then re-run OpenCode on the current head.
• Regression test: Keep the approval gate querying reviewThreads(first: 100) after model output and before create_pull_review APPROVE.

Review thread evidence

Latest unresolved human review thread evidence

src/main/java/com/clearfolio/viewer/controller/ViewerUiController.java line 110

• Latest human comment: @copilot-pull-request-reviewer at 2026-06-27T21:28:16Z

• Comment URL: 🎨 Palette: 새로고침 버튼 비활성화 상태 및 링크 접근성 개선 #39 (comment)

• Comment excerpt: The new accessibility/security attributes on the actions block (role="group") and JSON bootstrap link (target/rel) aren’t covered by the existing UI HTML tests. Adding a couple of contains(...) assertions in ViewerUiControllerTest would prevent regressions (e.g., accidental removal of rel="noopener noreferrer").

• Result: REQUEST_CHANGES

• Reason: unresolved human review thread(s) were present before approval.

• Head SHA: fa3ddad1dcc8ffaeea04984626b8dbb73c897bf0

• Workflow run: 28303316390

• Workflow attempt: 1

Change Flow DAG

flowchart LR
  PR["PR changed files"] --> Evidence["OpenCode bounded evidence"]
  Evidence --> S1["Changed file (4 files)"]
  S1 --> I1["repository behavior"]
  I1 --> R1["Review risk: Changed file (4 files)"]
  R1 --> V1["required checks"]

Loading

[opencode-agent]

Pull request overview

OpenCode reviewed the current-head evidence but found unresolved human review threads before approval.

Findings

1. HIGH .github/workflows/opencode-review.yml:1 - Unresolved human review thread blocks automated approval

• Problem: OpenCode reached an APPROVE control result, but the approval step found unresolved, non-outdated human review thread evidence on the current pull request.
• Root cause: Human review feedback can arrive after bounded model evidence is prepared, so the approval step must re-query GitHub immediately before publishing an approval.
• Fix: Address or resolve the listed human review thread(s), then re-run OpenCode on the current head.
• Regression test: Keep the approval gate querying reviewThreads(first: 100) after model output and before create_pull_review APPROVE.

Review thread evidence

Latest unresolved human review thread evidence

src/main/java/com/clearfolio/viewer/controller/ViewerUiController.java line 110

• Latest human comment: @copilot-pull-request-reviewer at 2026-06-27T21:28:16Z

• Comment URL: #39 (comment)

• Comment excerpt: The new accessibility/security attributes on the actions block (role="group") and JSON bootstrap link (target/rel) aren’t covered by the existing UI HTML tests. Adding a couple of contains(...) assertions in ViewerUiControllerTest would prevent regressions (e.g., accidental removal of rel="noopener noreferrer").

• Result: REQUEST_CHANGES

• Reason: unresolved human review thread(s) were present before approval.

• Head SHA: fa3ddad1dcc8ffaeea04984626b8dbb73c897bf0

• Workflow run: 28303316390

• Workflow attempt: 1

[seonghobae]

중복 정리: 이 PR은 'async 버튼 로딩 상태' 클러스터의 중복입니다. 정본으로 #71 (APPROVED + CI green + MERGEABLE)을 유지합니다. 동일 파일을 수정해 동시 병합이 불가능하므로 정리합니다. 고유하게 필요한 변경이 있으면 #71에 반영하거나 재오픈하세요.

[google-labs-jules]

중복 정리: 이 PR은 'async 버튼 로딩 상태' 클러스터의 중복입니다. 정본으로 #71 (APPROVED + CI green + MERGEABLE)을 유지합니다. 동일 파일을 수정해 동시 병합이 불가능하므로 정리합니다. 고유하게 필요한 변경이 있으면 #71에 반영하거나 재오픈하세요.

Understood. Acknowledging that this work is now obsolete and stopping work on this task. I will discontinue work on this PR.