feat(workspace): name tonight's first dynamics plan on the map

[seonghobae]

Buyer next action

The mounted rehearsal workspace names tonight's first dynamics plan so a part can lock the owned dynamics before rehearsal starts.

Demo: Bass Guitar still has Keep the verse under the vocal so the chorus still has somewhere to lift. Open is the next action: it moves to the matching rendered song-structure section.

Protected base: develop@acdbea6344fe1231c39535b575f4de35e4c607c9.
Exact current head: 10a651c024e5f4f7862139b29661c3802b748f30 on feat/workspace-first-dynamics-plan.

What the player sees

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

Authority boundary and current-head repair

• Do not invent copy from groove, cue, simplification, overlap, range, chord labels, function labels, setup notes, transposition plans, confirmed overrides, harmonicExplanation, or confidence notes.
• Owned optional RehearsalRole.dynamicsPlan is distinct from setupNote, transpositionPlan, and tuningPlan.
• Runtime dynamics-plan authority is the snapshotted own data-property descriptor value. A Proxy get trap cannot substitute a different buyer-visible dynamics plan after the own-data check.
• Regression commit 07fa70fcdbe2df6aba9c8b198a099a7f1ecfe20f locks that Proxy get-trap case; production repair 10a651c024e5f4f7862139b29661c3802b748f30 consumes the descriptor value directly.
• Inherited/accessor-backed runtime metadata remains guidance-only instead of becoming navigation authority.
• Analysis section.id is never DOM-ID authority. Open uses renderer-owned data-section-index.
• Distinct from first-entrance Tempo/Key/Transpose cockpit metrics (feat(workspace): name tonight's tempo, starting chord, and transpose setup #987), first-setup-note (feat(workspace): name tonight's first setup note on the map #1004), first-transposition-plan (feat(workspace): name tonight's first transposition plan on the map #1006), first-part-handoff (feat(workspace): name tonight's first part handoff on the map #1007), first-capo-plan (feat(workspace): name tonight's first capo plan on the map #1008), and first-tuning-plan (feat(workspace): name tonight's first tuning plan on the map #1010).
• Do not mix with 🔒 [보안] CLI 무제한 파일 읽기 취약점 수정 #811, test(analysis): govern real YouTube known-stem benchmark #828, or feat(workspace): Storybook tokens for rehearsal components #897.

Exact-head verification state

Fresh repository workflows for 10a651c024e5f4f7862139b29661c3802b748f30 are queued/pending at the latest refetch. No non-terminal result and no predecessor-head evidence is counted as success.

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. No lockfile change.
• Keep unmerged until the unchanged 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.
• Queued, pending, skipped-required, cancelled, failed, predecessor-head, protected-base, model-only, self/author, or administrative-bypass evidence is not success.
• Never bypass protection or transfer predecessor evidence. Do not self-approve.

---

[coderabbitai]

Warning

Review limit reached

Next included review available in 19 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: bc28cd46-afff-42f6-89df-d92bbc3cdd28

📥 Commits

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

📒 Files selected for processing (30)

• .Jules/palette.md
• .jules/palette.md
• AGENTS.md
• ARCHITECTURE.md
• CHANGELOG.md
• CLAUDE.md
• apps/desktop/src/features/workspace/FirstDynamicsPlanCallout.identity.test.tsx
• apps/desktop/src/features/workspace/FirstDynamicsPlanCallout.memoization.test.tsx
• apps/desktop/src/features/workspace/FirstDynamicsPlanCallout.particle.test.tsx
• apps/desktop/src/features/workspace/FirstDynamicsPlanCallout.reduced-motion.test.tsx
• apps/desktop/src/features/workspace/FirstDynamicsPlanCallout.test.tsx
• apps/desktop/src/features/workspace/FirstDynamicsPlanCallout.tsx
• apps/desktop/src/features/workspace/FirstDynamicsPlanCallout.workspace-scope.test.tsx
• apps/desktop/src/features/workspace/Workspace.test.tsx
• apps/desktop/src/features/workspace/Workspace.tsx
• apps/desktop/src/features/workspace/dynamicsCoverageContract.test.ts
• apps/desktop/src/features/workspace/firstDynamicsPlan.inherited-metadata.test.ts
• apps/desktop/src/features/workspace/firstDynamicsPlan.proxy-authority.test.ts
• apps/desktop/src/features/workspace/firstDynamicsPlan.section-label.test.ts
• apps/desktop/src/features/workspace/firstDynamicsPlan.test.ts
• apps/desktop/src/features/workspace/firstDynamicsPlan.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-dynamics-plan-navigation.md
• packages/shared-types/src/index.ts
• packages/shared-types/test/index.test.ts

---

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]

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 (FirstDynamicsPlanCallout + 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: bb06076.

[devin-ai-integration]

Devin Review found 1 new potential issue.