Skip to content

Use C4 as root note for chord lesson#2

Draft
Copilot wants to merge 2 commits into
claude/add-basic-chords-lessonfrom
copilot/always-use-c4-as-root-note
Draft

Use C4 as root note for chord lesson#2
Copilot wants to merge 2 commits into
claude/add-basic-chords-lessonfrom
copilot/always-use-c4-as-root-note

Conversation

Copy link
Copy Markdown

Copilot AI commented Feb 5, 2026

Makes chord lesson consistent with intervals lesson by using C4 (MIDI 60) as the only root note.

Changes

  • app/types/chords.ts: generateChordPool() now generates 4 challenges (C4 × 4 chord types) instead of 40 (10 root notes × 4 chord types)
  • app/hooks/useChordLesson.ts: Reduced CHALLENGES_PER_LESSON from 5 to 4 to match pool size
  • Tests: Updated assertions to expect 4 challenges and C4-only root note

Before/After

// Before: 10 root notes
const rootNotes: MidiNote[] = [48, 50, 52, 53, 55, 57, 59, 60, 62, 64];

// After: Single root note
const rootMidi: MidiNote = 60; // C4
Original prompt

Pull Request: #1

Let's always use c4 as the root note


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: witalewski <783419+witalewski@users.noreply.github.com>
Copilot AI changed the title [WIP] Update root note to C4 in project Use C4 as root note for chord lesson Feb 5, 2026
Copilot AI requested a review from witalewski February 5, 2026 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants