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

[seonghobae]

Product outcome

After analysis, the room can see tonight's first owned voicing plan and take one next action: open that part on the map.

Cites docs/design-system/component-contract.md (First Voicing Plan Callout).

This is a setup-cue next action after first-setup-note, first-transposition-plan, first-tuning-plan, first-dynamics-plan, and first-articulation-plan. It is not a parallel MIR product and does not invent voicing from groove, cue, simplification, overlap, range, chords, function labels, setup notes, transposition plans, tuning plans, dynamics plans, articulation plans, confirmed overrides, harmonic explanations, or confidence notes.

Exact current identity

• Protected base: develop@acdbea6344fe1231c39535b575f4de35e4c607c9.
• Branch: feat/workspace-first-voicing-plan.
• Head: b3e8922bc9507593698d86a6eb57fd73259391e1.

Buyer-facing copy

• Named: {role} still has a voicing plan in the {section} at {at} plus the owned voicingPlan text.
• Open: Open {role} voicing at {at} scrolls the renderer-owned [data-testid=song-structure-grid] [data-section-index=N].
• Armed: Lock that voicing on {role} at {at} before the room starts.
• Unavailable: stay on tonight's map until a part owns rehearsal-facing voicing copy. That unavailable sentence is the next action.
• Korean copy stays particle-safe with 파트.

Safety and current-head repair

• Fail closed on inherited metadata, own accessors, sparse arrays, duplicate role/graph identities, inactive graph nodes, and unbounded time ranges.
• Runtime voicing-plan authority is now the snapshotted own data-property descriptor value. A Proxy get trap cannot substitute different buyer-visible voicing copy after the own-data check.
• Regression commit 7f79b6d7d445da5e10bad5f5e8ec30d5873613d1 locks the Proxy get-trap case; production repair b3e8922bc9507593698d86a6eb57fd73259391e1 consumes the descriptor value directly.
• Open is renderer-owned map navigation only. Player Hear remains [Product] Add an active rehearsal player with precise looping and role controls #961.
• Reduced motion uses scroll-behavior: auto.
• Ambiguous workspace mounts do not steal another surface's section.

Exact-head verification state

Fresh repository workflows for b3e8922bc9507593698d86a6eb57fd73259391e1 have been dispatched and were queued at the latest refetch. No non-terminal result and no predecessor-head evidence is counted as success.

Out of scope / merge gate

• No parallel MIR product. test(analysis): govern real YouTube known-stem benchmark #828 still owns [Product Gap] Add real-audio MIR accuracy acceptance benchmarks #770.
• Does not change CLI job-file authority (🔒 [보안] CLI 무제한 파일 읽기 취약점 수정 #811).
• Does not land Storybook tokens (feat(workspace): Storybook tokens for rehearsal components #897).
• Does not suppress inherited npm HIGH findings owned by canonical fix(security): establish canonical npm, PDF.js, Nanoid, and Undici baseline #783.
• Keep unmerged until the unchanged current head has every applicable repository and central CI/build/release/security/SAST/SBOM/supply-chain/coverage/review 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.

---

[coderabbitai]

Warning

Review limit reached

Next included review available in 17 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: 57e3d038-426b-4138-8361-a52f04f6c4f0

📥 Commits

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

📒 Files selected for processing (32)

• .Jules/palette.md
• .jules/palette.md
• AGENTS.md
• ARCHITECTURE.md
• CHANGELOG.md
• CLAUDE.md
• apps/desktop/src/features/workspace/FirstVoicingPlanCallout.identity.test.tsx
• apps/desktop/src/features/workspace/FirstVoicingPlanCallout.particle.test.tsx
• apps/desktop/src/features/workspace/FirstVoicingPlanCallout.reduced-motion.test.tsx
• apps/desktop/src/features/workspace/FirstVoicingPlanCallout.test.tsx
• apps/desktop/src/features/workspace/FirstVoicingPlanCallout.tsx
• apps/desktop/src/features/workspace/FirstVoicingPlanCallout.unavailable-copy.test.tsx
• apps/desktop/src/features/workspace/FirstVoicingPlanCallout.workspace-scope.test.tsx
• apps/desktop/src/features/workspace/Workspace.test.tsx
• apps/desktop/src/features/workspace/Workspace.tsx
• apps/desktop/src/features/workspace/firstVoicingPlan.inherited-metadata.test.ts
• apps/desktop/src/features/workspace/firstVoicingPlan.proxy-authority.test.ts
• apps/desktop/src/features/workspace/firstVoicingPlan.role-authority.test.ts
• apps/desktop/src/features/workspace/firstVoicingPlan.section-authority.test.ts
• apps/desktop/src/features/workspace/firstVoicingPlan.section-label.test.ts
• apps/desktop/src/features/workspace/firstVoicingPlan.test.ts
• apps/desktop/src/features/workspace/firstVoicingPlan.ts
• apps/desktop/src/features/workspace/voicingCoverageContract.test.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-voicing-plan-navigation.md
• packages/shared-types/src/index.ts
• packages/shared-types/test/index.test.ts

✨ 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 feat/workspace-first-voicing-plan

---

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]

@opencode-agent

Repair only canonical BandScope PR #1013 branch feat/workspace-first-voicing-plan at exact head a0285ef59d84f27cf109b1f4b26cd92d45f13c70 if that head is still current; if it moved, refetch and adapt to the intervening delta rather than overwriting another writer.

Current-head review finding PRRT_kwDORjvEXs6brkMM is valid: FirstVoicingPlanCallout currently calls detectPreferredLocale(), createTranslator(locale), and resolveFirstVoicingPlan(song) on every render, while the resolver scans/sorts rehearsal metadata. Use test-driven development: first add a deterministic regression showing an unchanged song rerender does not recompute the voicing-plan resolution/translator work, then memoize only the derived locale/translator/resolution at the narrowest React boundary with correct dependencies. Preserve stable identity reset, renderer fail-closed behavior, copy, reduced-motion navigation, coverage thresholds, dependency files, and #783-owned security state.

Run the focused callout tests, desktop typecheck/lint/coverage, and applicable repository quickcheck. Commit only this performance repair to the same branch and report the exact successor head plus GREEN evidence. Do not create another PR, merge, force-push, self-approve, or weaken protection.

[seonghobae]

Re-review request for exact head 065a239277c78275f6613821e7e0f4cd68c64b3a: 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 065a239277c78275f6613821e7e0f4cd68c64b3a 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 (FirstVoicingPlanCallout + 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: e0fd59d.

[devin-ai-integration]

Devin Review found 1 new potential issue.