🎨 Palette: [UX improvement] Update terminology from "요약" to "종합" by seonghobae · Pull Request #1154 · ContextualWisdomLab/naruon

seonghobae · 2026-07-26T14:11:48Z

💡 What

frontend/src/components/SourceDrawer.tsx 파일 내에서 하드코딩된 "증거 요약" 텍스트를 "증거 종합"으로 변경했습니다.
frontend/src/app/search/page.test.tsx 파일 내의 테스트 목업 데이터에서 "Q2 일정 요약"을 "Q2 일정 종합"으로 일치되게 업데이트했습니다.
관련된 프론트엔드 테스트를 실행하여(cd frontend && pnpm run test) 회귀(regression)가 없음을 확인했습니다.
패치 적용을 위해 임시로 생성했던 update.patch 파일은 리뷰어 피드백에 따라 삭제했습니다.

🎯 Why

Naruon 제품의 UX/UI 기획 가이드라인(docs/ui-ux/naruon-ui-ux-mapping.md)에서는 "Summary"를 "요약" 대신 "종합" 또는 "핵심 맥락"으로 사용하도록 엄격하게 규칙(Terminology Rules)을 정하고 있습니다.
이는 Naruon이 단순히 내용을 줄이는 요약 도구가 아니라, 여러 정보의 맥락을 연결하여 판단과 실행을 돕는 도구라는 핵심 철학을 반영하기 위함입니다.

📸 Before/After

Before: SourceDrawer에서 "증거 요약"으로 표시됨, 테스트 결과에서 "Q2 일정 요약" 문자열 사용.
After: SourceDrawer에서 "증거 종합"으로 표시됨, 테스트 결과에서 "Q2 일정 종합" 문자열 사용.

♿ Accessibility

N/A (시각적/구조적 접근성 관련 변경 사항은 없으며, 순수 텍스트(용어) 변경에 해당합니다.)

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

Summary by CodeRabbit

Style
- Updated the source drawer heading to use revised Korean terminology.
Tests
- Updated search answer test data to match the revised Q2 snippet wording.

💡 What - `SourceDrawer.tsx` 컴포넌트 내의 텍스트 "증거 요약"을 "증거 종합"으로 변경했습니다. - 관련된 테스트 파일 (`search/page.test.tsx`)의 목업 데이터에서도 "Q2 일정 요약"을 "Q2 일정 종합"으로 변경했습니다. 🎯 Why - Naruon의 핵심 UX 철학 및 기획 문서(`naruon-ui-ux-mapping.md`)에 정의된 용어 규칙("Summary -> 종합, 핵심 맥락")을 준수하기 위함입니다. "요약" 대신 "종합"이라는 표현을 사용하여 맥락을 단순히 줄이는 것이 아니라 모아서 판단을 돕는다는 의미를 명확히 합니다. 📸 Before/After - 변경 전: "증거 요약" - 변경 후: "증거 종합" ♿ Accessibility - 스크린 리더 등 보조 기술 사용 시에도 Naruon의 일관된 기획 의도에 맞는 용어("종합")가 전달되도록 개선되었습니다.

google-labs-jules · 2026-07-26T14:11:49Z

👋 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 · 2026-07-26T14:11:56Z

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: 99d038e3-7f8f-4523-8da7-4654fe53f541

📥 Commits

Reviewing files that changed from the base of the PR and between eae74e2 and 0dbc09f.

📒 Files selected for processing (2)

frontend/src/app/search/page.test.tsx
frontend/src/components/SourceDrawer.tsx

📝 Walkthrough

Walkthrough

The search answer test fixtures now use “Q2 일정 종합”, and the source drawer summary heading changed from “증거 요약” to “증거 종합”.

Changes

Search terminology alignment

Layer / File(s)	Summary
Terminology and test fixture updates `frontend/src/components/SourceDrawer.tsx`, `frontend/src/app/search/page.test.tsx`	The source drawer heading and grounded search answer citation snippets now use “종합” terminology.

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

Possibly related PRs

ContextualWisdomLab/naruon#340: Applies the same “요약” to “종합” terminology pattern.
ContextualWisdomLab/naruon#1145: Updates the same SourceDrawer heading terminology.

Suggested reviewers: copilot

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly matches the main change: updating Korean terminology from “요약” to “종합” in the frontend.
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 fix-ui-terminology-summary-8243381197382916517

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

github-actions · 2026-07-26T14:12:23Z

PR governance metadata gate is not ready for 0dbc09fcba37c84c50c154fa415b9d58a697c6e5:

Review decision is CHANGES_REQUESTED; address requested changes before merge.

Copilot

Pull request overview

Updates Korean UX terminology to align with the product’s terminology rules by replacing “요약” with “종합” in a user-facing label and keeping related test mock data consistent.

Changes:

Update SourceDrawer section header text from “증거 요약” to “증거 종합”.
Update Search page test mock snippets from “Q2 일정 요약” to “Q2 일정 종합”.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
frontend/src/components/SourceDrawer.tsx	Updates a hardcoded UI label to the preferred terminology (“증거 종합”).
frontend/src/app/search/page.test.tsx	Aligns test mock data strings with the updated terminology to prevent assertion/mismatch issues.

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

…16517

opencode-agent

Pull request overview

OpenCode cannot approve yet because required coverage evidence did not pass.

Review outcome

1. HIGH .github/workflows/opencode-review.yml:1 - Coverage evidence did not prove required test/docstring evidence

Problem: The required coverage-evidence job result was failure, so OpenCode cannot establish approval sufficiency for this head.
Root cause: Automated approval is only valid when the same-head coverage-evidence job proves supported repository test suites passed and configured docstring gates passed or were advisory, or reports not applicable because no supported source files or package manifests exist. Missing, failed, skipped, unavailable, or unsupported-tooling test evidence is a blocker.
Fix: Install or configure the repository test/docstring evidence tooling when source files or package manifests exist, rerun the current-head coverage-evidence job, and approve only after it reports success with required evidence or explicit no-source not-applicable evidence.
Regression test: Keep the approval branch checking needs.coverage-evidence.result == success before posting APPROVE, and publish REQUEST_CHANGES when coverage-evidence blocker states such as cancelled, skipped, failed, unsupported-tooling, or below-100 evidence are present.
Result: REQUEST_CHANGES
Reason: coverage-evidence result was failure, so required test/docstring evidence was not proven for current head 0dbc09fcba37c84c50c154fa415b9d58a697c6e5.
Head SHA: 0dbc09fcba37c84c50c154fa415b9d58a697c6e5
Workflow run: 30532461676
Workflow attempt: 1

Coverage evidence

Coverage Decision

Result: FAIL
Test evidence: not proven passing
Docstring evidence: not proven passing when configured
Failure count: 1

Changed-File Evidence Map

flowchart LR
  PR["PR changed files"] --> Evidence["OpenCode bounded evidence"]
  Evidence --> S1["Frontend (2 files)"]
  S1 --> I1["browser runtime and bundle"]
  I1 --> R1["Review risk: Frontend (2 files)"]
  R1 --> V1["frontend tests"]

opencode-agent · 2026-07-30T09:56:47Z

OpenCode Review Overview

Head SHA: 0dbc09fcba37c84c50c154fa415b9d58a697c6e5
Workflow run: 30532461676
Workflow attempt: 1
Gate result: REQUEST_CHANGES (approval step)

Pull request overview

OpenCode cannot approve yet because required coverage evidence did not pass.

Review outcome

1. HIGH .github/workflows/opencode-review.yml:1 - Coverage evidence did not prove required test/docstring evidence

Problem: The required coverage-evidence job result was failure, so OpenCode cannot establish approval sufficiency for this head.
Root cause: Automated approval is only valid when the same-head coverage-evidence job proves supported repository test suites passed and configured docstring gates passed or were advisory, or reports not applicable because no supported source files or package manifests exist. Missing, failed, skipped, unavailable, or unsupported-tooling test evidence is a blocker.
Fix: Install or configure the repository test/docstring evidence tooling when source files or package manifests exist, rerun the current-head coverage-evidence job, and approve only after it reports success with required evidence or explicit no-source not-applicable evidence.
Regression test: Keep the approval branch checking needs.coverage-evidence.result == success before posting APPROVE, and publish REQUEST_CHANGES when coverage-evidence blocker states such as cancelled, skipped, failed, unsupported-tooling, or below-100 evidence are present.
Result: REQUEST_CHANGES
Reason: coverage-evidence result was failure, so required test/docstring evidence was not proven for current head 0dbc09fcba37c84c50c154fa415b9d58a697c6e5.
Head SHA: 0dbc09fcba37c84c50c154fa415b9d58a697c6e5
Workflow run: 30532461676
Workflow attempt: 1

Coverage evidence

Coverage Decision

Result: FAIL
Test evidence: not proven passing
Docstring evidence: not proven passing when configured
Failure count: 1

Changed-File Evidence Map

flowchart LR
  PR["PR changed files"] --> Evidence["OpenCode bounded evidence"]
  Evidence --> S1["Frontend (2 files)"]
  S1 --> I1["browser runtime and bundle"]
  I1 --> R1["Review risk: Frontend (2 files)"]
  R1 --> V1["frontend tests"]

seonghobae · 2026-07-31T12:45:27Z

Superseded duplicate Palette UX/terminology/loading cohort. Close to drain PR queue; re-land single focused PR if still needed on develop.

google-labs-jules · 2026-07-31T12:46:45Z

Superseded duplicate Palette UX/terminology/loading cohort. Close to drain PR queue; re-land single focused PR if still needed on develop.

Understood. Acknowledging that this work is superseded and closing as requested. I will stop work on this task.

Copilot AI review requested due to automatic review settings July 26, 2026 14:11

Copilot started reviewing on behalf of seonghobae July 26, 2026 14:12 View session

Copilot AI reviewed Jul 26, 2026

View reviewed changes

Merge branch 'develop' into fix-ui-terminology-summary-82433811973829…

0dbc09f

…16517

coderabbitai Bot approved these changes Jul 29, 2026

View reviewed changes

opencode-agent Bot requested changes Jul 30, 2026

View reviewed changes

seonghobae closed this Jul 31, 2026

coderabbitai Bot mentioned this pull request Jul 31, 2026

🎨 Palette: [UX improvement] UX/UI 용어 가이드라인에 맞춘 텍스트 수정 #1208

Merged

Navigation Menu

🎨 Palette: [UX improvement] Update terminology from "요약" to "종합" - #1154

🎨 Palette: [UX improvement] Update terminology from "요약" to "종합"#1154
seonghobae wants to merge 2 commits into
developfrom
fix-ui-terminology-summary-8243381197382916517

seonghobae commented Jul 26, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

google-labs-jules Bot commented Jul 26, 2026

Uh oh!

coderabbitai Bot commented Jul 26, 2026 •

edited

Loading

Walkthrough

Changes

Uh oh!

github-actions Bot commented Jul 26, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

opencode-agent Bot left a comment

Uh oh!

opencode-agent Bot commented Jul 30, 2026

Uh oh!

seonghobae commented Jul 31, 2026

Uh oh!

google-labs-jules Bot commented Jul 31, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

seonghobae commented Jul 26, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Uh oh!

google-labs-jules Bot commented Jul 26, 2026

Uh oh!

coderabbitai Bot commented Jul 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Uh oh!

github-actions Bot commented Jul 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

opencode-agent Bot left a comment

Choose a reason for hiding this comment

Pull request overview

Review outcome

1. HIGH .github/workflows/opencode-review.yml:1 - Coverage evidence did not prove required test/docstring evidence

Coverage evidence

Coverage Decision

Changed-File Evidence Map

Uh oh!

opencode-agent Bot commented Jul 30, 2026

OpenCode Review Overview

Pull request overview

Review outcome

1. HIGH .github/workflows/opencode-review.yml:1 - Coverage evidence did not prove required test/docstring evidence

Coverage evidence

Coverage Decision

Changed-File Evidence Map

Uh oh!

seonghobae commented Jul 31, 2026

Uh oh!

google-labs-jules Bot commented Jul 31, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

seonghobae commented Jul 26, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented Jul 26, 2026 •

edited

Loading

github-actions Bot commented Jul 26, 2026 •

edited

Loading