feat(workspace): name tonight's first open rehearsal comment on the map

[seonghobae]

Product outcome

Name tonight's first open rehearsal comment so the room can read the MD note and open the matching rendered map section.

Design authority: workspace next-action pattern plus docs/design-system/component-contract.md. Distinct from first-lyric (#913), first-count (#995), first-transition (#994), first-setup (#910), and ready-board (#901). This does not invent a parallel MIR product; #828 remains the owner of #770. This does not replace #740 (handoff import).

Current exact scope

• Protected target: develop@acdbea6344fe1231c39535b575f4de35e4c607c9.
• Branch: feat/workspace-first-open-comment.
• FirstOpenCommentCallout names the author, the holding part when an active graph node corroborates the owned roleId, the labeled section, and the time.
• Open scrolls the renderer-owned song-structure section. Missing targets stay guidance-only and do not claim navigation completed.
• Do not invent a note from resolved comments, assignments, approvals, lyric/count/transition cues, groove, setup, simplification, overlap, range copy, or empty/whitespace bodies.
• Owned comment body is rendered as a text node and is never rescanned as template syntax.
• Korean copy keeps particle-safe role phrasing (파트가 / 파트에) via {author}님이 and {role} 파트, and localizes the section form label.

Demo song: MD open comment for Keyboard 1 Right Hand at 0:10, body "Keep the keyboard color tone gentle on the first pass so the vocal cue stays forward."

Trust boundary

• Untrusted runtime song, collaboration, comment, section, time-range, role, and part-graph metadata are accepted only as own data properties.
• Inherited properties, accessors, sparse arrays, and arrays masquerading as records cannot establish an open note or a holding part.
• Analysis section.id is never DOM-ID authority. Navigation uses renderer child index under the song-structure region.
• The comment body is bounded to 180 Unicode code points without splitting a surrogate pair.
• Reduced-motion map navigation uses behavior: "auto" when the OS requests it.

Dependency ownership

Canonical #783 owns the protected-base JavaScript HIGH findings (pdfjs-dist / nanoid / undici). This branch has no JavaScript dependency, lockfile, or vulnerability-suppression delta. Inherited npm HIGH must not be suppressed here.

Merge gate

• Do not mix with 🔒 [보안] CLI 무제한 파일 읽기 취약점 수정 #811, test(analysis): govern real YouTube known-stem benchmark #828, or feat(workspace): Storybook tokens for rehearsal components #897.
• Require every exact-current-head repository and central CI/security/SAST/SBOM/coverage/review gate to reach terminal success.
• Require zero valid unresolved current-head findings.
• Require a qualifying independent non-author latest-head APPROVED review.
• Never bypass branch protection, self-approve, or transfer predecessor evidence.

Queued, pending, skipped-required, failed, predecessor-head, protected-base, author-only, model-only, status-only, or synthetic evidence is non-passing.

---

[coderabbitai]

Warning

Review limit reached

Next included review available in 42 minutes.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 8ec53122-0189-482a-90e5-c3e8e0b499fc

📥 Commits

Reviewing files that changed from the base of the PR and between 749511c and c0de9de.

📒 Files selected for processing (23)

• AGENTS.md
• ARCHITECTURE.md
• CHANGELOG.md
• CLAUDE.md
• apps/desktop/src/features/workspace/FirstOpenCommentCallout.particle.test.tsx
• apps/desktop/src/features/workspace/FirstOpenCommentCallout.proxy.test.tsx
• apps/desktop/src/features/workspace/FirstOpenCommentCallout.reduced-motion.test.tsx
• apps/desktop/src/features/workspace/FirstOpenCommentCallout.stories.tsx
• apps/desktop/src/features/workspace/FirstOpenCommentCallout.test.tsx
• apps/desktop/src/features/workspace/FirstOpenCommentCallout.tsx
• apps/desktop/src/features/workspace/FirstOpenCommentCallout.workspace-scope.test.tsx
• apps/desktop/src/features/workspace/Workspace.test.tsx
• apps/desktop/src/features/workspace/Workspace.tsx
• apps/desktop/src/features/workspace/firstOpenComment.inherited-metadata.test.ts
• apps/desktop/src/features/workspace/firstOpenComment.test.ts
• apps/desktop/src/features/workspace/firstOpenComment.ts
• apps/desktop/src/i18n/index.test.ts
• apps/desktop/src/i18n/index.ts
• apps/desktop/src/locales/en/common.json
• apps/desktop/src/locales/ko/common.json
• apps/desktop/vite.config.ts
• docs/design-system/component-contract.md
• docs/doctoring/reduced-motion-first-open-comment-navigation.md

---

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

[seonghobae]

Re-review request for exact head a23e8720a78b5f8eb5ef2d90b9e373037e201288: all required status contexts are green on this head (including coverage-evidence and security scans) and all review threads are resolved. @opencode-agent please review current head a23e8720a78b5f8eb5ef2d90b9e373037e201288 and update the review decision.

[seonghobae]

Resolved merge conflicts against origin/develop (749511c): union-merged AGENTS.md/CHANGELOG.md bullets, sentence-merged CLAUDE.md, kept both import sets in Workspace.tsx (FirstOpenCommentCallout + firstRangeSqueeze; render paths intact), unioned en/ko locale tails (JSON valid, en/ko symmetric). No conflict markers remain. Merge head: c0de9de.