feat(workspace): guide tonight's first entrance on map and player

AGENTS.md

@@ -83,6 +83,7 @@ This section applies to any agent (Claude, Codex, Cursor, opencode, ...) working
 - Keep UI and analysis engine decoupled through shared contracts.
 - Prefer minimal, test-first changes for production code.
 - Prefer practical, friendly, rehearsal-first wording over academic or authority-heavy language.
+- Name tonight's first entrance with the part, section, and start time so the next hearable action is obvious.
 - Do not reduce the product to a chord analyzer when form, timing, player coordination, playable ranges, simplification, and setup cues are the real rehearsal blockers.
 - Do not frame usability as a reason to accept weak analysis quality; BandScope should aim for both easy use and high accuracy.
 

ARCHITECTURE.md

@@ -6,6 +6,7 @@ Last updated: 2026-03-11
 
 - Product identity, UX tone, copy rules, and prioritization tie-breakers live in `docs/brand-story.md`.
 - Future PRDs, TRDs, onboarding copy, empty states, error messages, and marketing copy should use that document as the single brand source of truth.
+- Workspace and player copy for tonight's first entrance must name the part, section, and start time so the next hearable action is obvious.
 
 ## Security source
 

CHANGELOG.md

@@ -4,6 +4,7 @@
 
 ### Added
 
+- Name tonight's first entrance on the workspace and player with part, section, and start time; the workspace action arms the entrance and opens the matching map section, honors the user's reduced-motion preference for that navigation, and the player exposes a Hear action only when its owning playback surface supplies a seek callback.
 - Name tonight's first playable range on the ready rehearsal map and tell the player to check that span on their instrument before the section.
 - Display the analyzed song tempo (BPM) as a badge in the rehearsal workspace.
 - 각 합주 역할(Role)별 개인 연습 진행도를 0~100% 범위로 기록 및 시각화할 수 있는 연습 진척도(`practiceProgress`) 트래커 기능 추가. UI 컨트롤(슬라이더 및 +/- 버튼)과 한/영 다국어 지원 포함.
@@ -74,4 +75,4 @@
 
 - `ChordsFeature` (코드 분석) 화면에서 각 파트(Role)의 `transpositionPlan`(이조/조옮김 계획)을 표시하는 기능을 추가했습니다.
 - `RangesFeature` (음역대 분석) 화면에서 겹침 경고(Overlap warning) 외에 해당 파트의 채보(Transcription) 가능 노드 수를 요약하여 보여주는 기능을 추가했습니다.
-- 신규 UI 요소에 대한 단위 테스트를 추가했습니다 (`apps/desktop/src/features/chords/index.test.tsx`, `apps/desktop/src/features/ranges/index.test.tsx`).
+- 신규 UI 요소에 대한 단위 테스트를 추가했습니다 (`apps/desktop/src/features/chords/index.test.tsx`, `apps/desktop/src/features/ranges/index.test.tsx`).

CLAUDE.md

@@ -51,7 +51,7 @@ BandScope is a local-first desktop app for rehearsal prep: it turns a song into
 
 Three layers, decoupled through shared contracts:
 
-- `apps/desktop` — Tauri 2 + Vite + React 19 shell (Tailwind 4, Base UI, Storybook). Feature screens live in `src/features/` (home, workspace, chords, ranges, player, settings). The ready workspace names tonight's first playable range and the next instrument check. `src/lib/analysis.ts` and `src/lib/job_runner.ts` call typed Tauri IPC commands, with a browser fallback that serves demo data when not running inside Tauri.
+- `apps/desktop` — Tauri 2 + Vite + React 19 shell (Tailwind 4, Base UI, Storybook). Feature screens live in `src/features/` (home, workspace, chords, ranges, player, settings). Workspace and player name tonight's first entrance with its part, section, and start time; the workspace action arms the entrance and opens the mapped section, while the player renders the Hear action only when its owning playback surface supplies a seek callback. The ready workspace also names tonight's first playable range and the next instrument check. `src/lib/analysis.ts` and `src/lib/job_runner.ts` call typed Tauri IPC commands, with a browser fallback that serves demo data when not running inside Tauri.
 - `apps/desktop/src-tauri/src/main.rs` — the Rust orchestration boundary. Tauri commands (`start_analysis_job`, `get_analysis_job_status`, `select_local_audio_source`, `import_youtube_url`) validate untrusted input (project IDs, file paths, URLs) and spawn the Python engine as a subprocess. There is no loopback HTTP listener and no network path for local analysis.
 - `services/analysis-engine` — Python package `bandscope_analysis` (librosa/numpy). Entry point `cli.py` reads a JSON job request on stdin and prints a structured job-status JSON envelope on stdout (`--progress-jsonl` streams progress lines). `api.py` orchestrates the pipeline across the `separation`, `sections`, `roles`, `chords`, `ranges`, `temporal`, `transcription`, and `youtube` modules.
 
@@ -72,4 +72,4 @@ Supporting packages:
 - Documents under `docs/plans/` must include `Security Notes`; `scripts/checks/verify_security_notes.py` enforces this mechanically.
 - Lockfiles (`package-lock.json`, `uv.lock`, `Cargo.lock`) are committed and must stay in sync; GitHub Actions are SHA-pinned. Adding a direct dependency requires the admission rationale defined in `AGENTS.md` and `docs/security/dependency-policy.md`.
 - CI beyond quickcheck: `gate / ci / rust-check` (Tauri cargo check on macOS) and `build-baseline` Windows/macOS amd64+arm64 native builds are merge gates, alongside CodeQL, dependency-review, sbom, bandit, trivy, secret-scan, and security-audit workflows. Do not weaken or skip them.
-- Version metadata lives in `VERSION`, the root `package.json`, and `CHANGELOG.md`; release flow is tag-driven (see `docs/operations/deploy-runbook.md`).
+- Version metadata lives in `VERSION`, the root `package.json`, and `CHANGELOG.md`; release flow is tag-driven (see `docs/operations/deploy-runbook.md`).

apps/desktop/src/App.player-reachability.test.tsx

@@ -0,0 +1,40 @@
+import { fireEvent, render, screen, waitFor } from "@testing-library/react";
+import { createDemoRehearsalSong } from "@bandscope/shared-types";
+import { describe, expect, it, vi } from "vitest";
+import { App } from "./App";
+
+vi.mock("./features/score/pdfjs", () => ({
+  configureScorePdfWorker: vi.fn(),
+  loadScorePdf: vi.fn(() => ({
+    promise: Promise.resolve({ numPages: 1, getPage: vi.fn() }),
+    destroy: vi.fn(() => Promise.resolve())
+  }))
+}));
+
+vi.mock("./lib/analysis", async (importActual) => {
+  const actual = await importActual<typeof import("./lib/analysis")>();
+
+  return {
+    ...actual,
+    loadProject: vi.fn(async () => createDemoRehearsalSong())
+  };
+});
+
+describe("App player reachability", () => {
+  it("exposes the analyzed song's first-entrance guidance through the shipped Player view without a dead playback action", async () => {
+    render(<App />);
+
+    fireEvent.click(screen.getByRole("button", { name: /open project/i }));
+
+    const playerButton = await screen.findByRole("button", { name: /^Player$/i });
+    fireEvent.click(playerButton);
+
+    await waitFor(() => {
+      expect(screen.getByRole("heading", { name: /^Player$/i })).toBeTruthy();
+    });
+    expect(screen.getByText(/^Bass Guitar enters the verse at 0:10\./)).toBeTruthy();
+    expect(
+      screen.queryByRole("button", { name: "Hear Bass Guitar enter the verse at 0:10" })
+    ).toBeNull();
+  });
+});

apps/desktop/src/features/player/index.test.tsx

@@ -0,0 +1,35 @@
+import { fireEvent, render, screen } from "@testing-library/react";
+import { createDemoRehearsalSong } from "@bandscope/shared-types";
+import { describe, expect, it, vi } from "vitest";
+import { PlayerFeature } from "./index";
+
+describe("PlayerFeature", () => {
+  it("asks the room to analyze first when no song is loaded", () => {
+    render(<PlayerFeature title="Player" />);
+    expect(
+      screen.getByText("Analyze tonight's song first, then hear the first entrance from this player.")
+    ).toBeTruthy();
+  });
+
+  it("names tonight's first entrance without a dead action when playback is unavailable", () => {
+    render(<PlayerFeature title="Player" song={createDemoRehearsalSong()} />);
+
+    expect(screen.getByText(/^Bass Guitar enters the verse at 0:10\./)).toBeTruthy();
+    expect(screen.queryByRole("button", { name: "Hear Bass Guitar enter the verse at 0:10" })).toBeNull();
+  });
+
+  it("uses the player playback callback for the first entrance action", () => {
+    const onPlayFromSeconds = vi.fn();
+    render(
+      <PlayerFeature
+        title="Player"
+        song={createDemoRehearsalSong()}
+        onPlayFromSeconds={onPlayFromSeconds}
+      />
+    );
+
+    fireEvent.click(screen.getByRole("button", { name: "Hear Bass Guitar enter the verse at 0:10" }));
+    expect(onPlayFromSeconds).toHaveBeenCalledTimes(1);
+    expect(onPlayFromSeconds).toHaveBeenCalledWith(10);
+  });
+});

apps/desktop/src/features/player/index.tsx

@@ -1,56 +1,34 @@
 import type { RehearsalSong } from "@bandscope/shared-types";
+import { FirstEntranceCallout } from "../workspace/FirstEntranceCallout";
+import { createTranslator, detectPreferredLocale } from "../../i18n";
 
-/** Documented. */
-export function PlayerFeature(props: { title: string; song?: RehearsalSong | null }) {
-  const { title, song } = props;
+type PlayerFeatureProps = {
+  title: string;
+  song?: RehearsalSong | null;
+  onPlayFromSeconds?: (startSeconds: number) => void;
+};
+
+/** Player surface that names tonight's first entrance and delegates playback to the owning player. */
+export function PlayerFeature({ title, song, onPlayFromSeconds }: PlayerFeatureProps) {
+  const t = createTranslator(detectPreferredLocale());
 
   if (!song) {
     return (
-      <section style={{ padding: "24px" }}>
-        <h2>{title}</h2>
-        <p style={{ color: "#999" }}>No song loaded. Start an analysis to use the player.</p>
+      <section className="p-6">
+        <h2 className="text-xl font-black text-white">{title}</h2>
+        <p className="mt-2 text-sm leading-6 text-slate-400">{t("firstEntranceNeedsSong")}</p>
       </section>
     );
   }
 
   return (
-    <section style={{ padding: "24px" }}>
-      <h2>{title}</h2>
-      <div
-        style={{
-          padding: "16px",
-          backgroundColor: "#fafafa",
-          borderRadius: "8px",
-          border: "1px solid #e8e8e8",
-        }}
-      >
-        <div style={{ marginBottom: "12px" }}>
-          <strong>{song.title}</strong>
-          <span style={{ color: "#666", marginLeft: "8px" }}>
-            {song.sections.length} {song.sections.length === 1 ? "section" : "sections"}
-          </span>
-        </div>
-        <div style={{ display: "flex", gap: "8px", flexWrap: "wrap" }}>
-          {song.sections.map((section) => (
-            <span
-              key={section.id}
-              style={{
-                padding: "4px 12px",
-                borderRadius: "16px",
-                backgroundColor: "#fff",
-                border: "1px solid #d9d9d9",
-                fontSize: "0.85em",
-                textTransform: "capitalize",
-              }}
-            >
-              {section.label}
-            </span>
-          ))}
-        </div>
-        <div style={{ marginTop: "16px", color: "#999", fontSize: "0.85em" }}>
-          Audio playback requires the desktop app with a local audio source.
-        </div>
-      </div>
+    <section className="space-y-4 p-6">
+      <h2 className="text-xl font-black text-white">{title}</h2>
+      <FirstEntranceCallout
+        song={song}
+        actionMode="callback-only"
+        onHearEntrance={onPlayFromSeconds}
+      />
     </section>
   );
 }

apps/desktop/src/features/workspace/FirstEntranceCallout.accessible-name.test.tsx

@@ -0,0 +1,20 @@
+import { render, screen } from "@testing-library/react";
+import { createDemoRehearsalSong } from "@bandscope/shared-types";
+import { describe, expect, it } from "vitest";
+import { FirstEntranceCallout } from "./FirstEntranceCallout";
+
+describe("FirstEntranceCallout accessible name", () => {
+  it("keeps the concise first-entrance region name when no entrance exists", () => {
+    const song = createDemoRehearsalSong();
+    song.sections = [];
+
+    render(<FirstEntranceCallout song={song} />);
+
+    expect(
+      screen.getByRole("complementary", { name: "Tonight's first entrance" })
+    ).toBeTruthy();
+    expect(
+      screen.getByText("No first entrance yet. Stay on tonight's map until a section has a part.")
+    ).toBeTruthy();
+  });
+});

apps/desktop/src/features/workspace/FirstEntranceCallout.reduced-motion.test.tsx

@@ -0,0 +1,53 @@
+import { fireEvent, render, screen } from "@testing-library/react";
+import { createDemoRehearsalSong } from "@bandscope/shared-types";
+import { afterEach, describe, expect, it, vi } from "vitest";
+import { FirstEntranceCallout } from "./FirstEntranceCallout";
+
+afterEach(() => {
+  vi.unstubAllGlobals();
+  document.querySelector("#workspace-song-structure-grid")?.remove();
+});
+
+/** Mount the renderer-owned song-structure target used by workspace navigation. */
+function appendSongStructureTarget() {
+  const grid = document.createElement("div");
+  grid.id = "workspace-song-structure-grid";
+  const target = document.createElement("div");
+  const scrollIntoView = vi.fn();
+  Object.defineProperty(target, "scrollIntoView", {
+    configurable: true,
+    value: scrollIntoView
+  });
+  grid.appendChild(target);
+  document.body.appendChild(grid);
+  return scrollIntoView;
+}
+
+describe("FirstEntranceCallout reduced-motion navigation", () => {
+  it("avoids smooth scrolling when the user requests reduced motion", () => {
+    const scrollIntoView = appendSongStructureTarget();
+    vi.stubGlobal(
+      "matchMedia",
+      vi.fn((query: string) => ({
+        matches: query === "(prefers-reduced-motion: reduce)",
+        media: query,
+        onchange: null,
+        addListener: vi.fn(),
+        removeListener: vi.fn(),
+        addEventListener: vi.fn(),
+        removeEventListener: vi.fn(),
+        dispatchEvent: vi.fn()
+      }))
+    );
+
+    render(<FirstEntranceCallout song={createDemoRehearsalSong()} />);
+    fireEvent.click(
+      screen.getByRole("button", { name: "Open Bass Guitar entrance in the verse at 0:10" })
+    );
+
+    expect(scrollIntoView).toHaveBeenCalledWith({
+      block: "nearest",
+      behavior: "auto"
+    });
+  });
+});