feat(workspace): name tonight's first harmonic function on the map

[seonghobae]

Buyer next action

The mounted rehearsal workspace now names tonight's first harmonic function so the room can lock the owned part function before rehearsal starts.

Protected base: develop@acdbea6344fe1231c39535b575f4de35e4c607c9.

What the player sees

• The callout names the owning part when an active graph node corroborates it, the owned harmony.functionLabel copy, the labeled section, and the time.
• Open scrolls the renderer-owned song-structure section and arms the next action: lock that function before the room starts.
• If no part owns rehearsal-facing function copy, the card stays guidance-only. Stay on tonight's map.

Authority boundary

• Do not invent that copy from groove, cue, simplification, overlap, range, chord labels, setup notes, confirmed overrides, harmonicExplanation, or confidence notes.
• Inherited or accessor-backed runtime metadata remains guidance-only instead of becoming navigation authority.
• Analysis section.id is never DOM-ID authority.
• Distinct from first-harmonic-explanation (feat(workspace): name tonight's first harmonic explanation on the map #1003), first-confirmed-harmony (feat(workspace): name tonight's first confirmed chord on the map #1002), first-setup-note (feat(workspace): name tonight's first setup note on the map #1004), and first-transpose (feat(workspace): name tonight's tempo, starting chord, and transpose setup #987) work.

Docs

AGENTS.md, CLAUDE.md, ARCHITECTURE.md, CHANGELOG.md, and docs/design-system/component-contract.md name this next action. Reduced-motion Open uses behavior: "auto".

Dependency / merge gate

• fix(security): establish canonical npm, PDF.js, Nanoid, and Undici baseline #783 remains the canonical protected JavaScript dependency-security owner. Inherited pdfjs-dist / nanoid / undici HIGH findings are not copied or suppressed here.
• Keep unmerged until the unchanged then-current head has every applicable repository/central CI, cross-platform build, exact owned statement+branch coverage, docstring, SAST/security/SBOM/supply-chain/package/release gate terminal-success; zero valid unresolved findings; a qualifying independent non-author last-push approval; and ordinary branch-protection acceptance.

---

[coderabbitai]

Warning

Review limit reached

Next included review available in 28 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: 0b277646-2cec-436e-af4f-1daeb269e888

📥 Commits

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

📒 Files selected for processing (26)

• .Jules/palette.md
• .jules/palette.md
• AGENTS.md
• ARCHITECTURE.md
• CHANGELOG.md
• CLAUDE.md
• apps/desktop/src/features/workspace/FirstHarmonicFunctionCallout.accessibility.test.tsx
• apps/desktop/src/features/workspace/FirstHarmonicFunctionCallout.particle.test.tsx
• apps/desktop/src/features/workspace/FirstHarmonicFunctionCallout.reduced-motion.test.tsx
• apps/desktop/src/features/workspace/FirstHarmonicFunctionCallout.sections-accessor.test.tsx
• apps/desktop/src/features/workspace/FirstHarmonicFunctionCallout.test.tsx
• apps/desktop/src/features/workspace/FirstHarmonicFunctionCallout.tsx
• apps/desktop/src/features/workspace/FirstHarmonicFunctionCallout.workspace-scope.test.tsx
• apps/desktop/src/features/workspace/Workspace.test.tsx
• apps/desktop/src/features/workspace/Workspace.tsx
• apps/desktop/src/features/workspace/firstHarmonicFunction.inherited-metadata.test.ts
• apps/desktop/src/features/workspace/firstHarmonicFunction.proxy-authority.test.ts
• apps/desktop/src/features/workspace/firstHarmonicFunction.test.ts
• apps/desktop/src/features/workspace/firstHarmonicFunction.time-authority.test.ts
• apps/desktop/src/features/workspace/firstHarmonicFunction.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
• docs/design-system/component-contract.md
• docs/doctoring/reduced-motion-first-harmonic-function-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 37fe99cc9786995e5fdc56315fe6d5ba3408b41c: 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 37fe99cc9786995e5fdc56315fe6d5ba3408b41c 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 (FirstHarmonicFunctionCallout + firstRangeSqueeze), unioned en/ko locale tails (JSON valid, en/ko symmetric). Merged legacy .Jules/palette.md into canonical .jules/palette.md, dropped alias. No conflict markers remain. Merge head: 7ae4e45.