Feature/pitch shifter plugin #4043

TheSakyo · 2025-10-26T16:08:49Z

✨ Add Pitch Shifter plugin

🧩 Overview

This pull request introduces a brand new Pitch Shifter plugin for Pear Music.
It allows users to adjust the pitch (key) of songs in real time, using Tone.js for audio processing.

🎛 Features

🎚️ Real-time pitch shifting from -12 to +12 semitones
⚙️ Built with Tone.js for precise, smooth pitch control
🧱 UI implemented with Solid.js, integrated naturally above the “Next up / Lyrics / Similar” tabs
🔁 Supports instant enable/disable from the plugin manager
🧹 Cleans up automatically on disable (no duplicated sliders)
🌐 Includes basic i18n support (currently English 🇬🇧 and French 🇫🇷)

🧠 Implementation Details

Uses captureStream() from YouTube Music’s <video> element to process live audio.
Utilizes the Tone.PitchShift node to change pitch without altering tempo.
Injects a minimal and adaptive UI directly inside the YouTube Music DOM.
Built with Solid.js for declarative reactivity and cleaner lifecycle handling.

// Example of Solid component rendering inside renderer.tsx
render(() => <PitchUI />, mount);

// Configuration
{
  enabled: false,
  semitones: 0
}

🌍 Localization

The plugin includes translations for:

en.json 🇬🇧 (English)
fr.json 🇫🇷 (French)

Other languages can easily be added later to match Pear’s i18n system.

🧪 Testing

✅ Successfully tested on:

Windows 10 & 11
YouTube Music (latest version)
Electron development environment

Pitch shifting updates instantly in real time, with no noticeable latency or distortion.

🙌 Notes

Plugin structure follows the same pattern as existing ones:

src/plugins/pitch-shifter/
├── index.ts
├── renderer.tsx
└── style.css

All style rules are scoped via .pitch-* classes for isolation.
Maintainers can adapt i18n keys for full Pear integration.

🧑‍💻 Author

Developed and tested by TheSakyo

(Tone.js + Solid.js integration and functional testing)

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

eslint

🚫 [eslint] <stylistic/no-tabs> _{reported by reviewdog 🐶}
Unexpected tab character.

pear-desktop/src/plugins/pitch-shifter/index.ts

Line 36 in 67d6b05

    
           // 🎨 ─────────────── Plugin Stylesheet ───────────────

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace ↹ with ··

pear-desktop/src/plugins/pitch-shifter/index.ts

Line 37 in 67d6b05

/** Inline CSS loaded into the YT Music renderer for consistent styling. */

🚫 [eslint] <stylistic/no-tabs> _{reported by reviewdog 🐶}
Unexpected tab character.

pear-desktop/src/plugins/pitch-shifter/index.ts

Line 37 in 67d6b05

/** Inline CSS loaded into the YT Music renderer for consistent styling. */

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace ↹ with ··

pear-desktop/src/plugins/pitch-shifter/index.ts

Line 38 in 67d6b05

stylesheets: [style],

🚫 [eslint] <stylistic/no-tabs> _{reported by reviewdog 🐶}
Unexpected tab character.

pear-desktop/src/plugins/pitch-shifter/index.ts

Line 38 in 67d6b05

stylesheets: [style],

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace ↹ with ··

pear-desktop/src/plugins/pitch-shifter/index.ts

Line 40 in 67d6b05

    
           // 🎧 ─────────────── Renderer Logic ───────────────

🚫 [eslint] <stylistic/no-tabs> _{reported by reviewdog 🐶}
Unexpected tab character.

pear-desktop/src/plugins/pitch-shifter/index.ts

Line 40 in 67d6b05

    
           // 🎧 ─────────────── Renderer Logic ───────────────

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace ↹ with ··

pear-desktop/src/plugins/pitch-shifter/index.ts

Line 41 in 67d6b05

/**

🚫 [eslint] <stylistic/no-tabs> _{reported by reviewdog 🐶}
Unexpected tab character.

pear-desktop/src/plugins/pitch-shifter/index.ts

Line 41 in 67d6b05

/**

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace ↹ with ··

pear-desktop/src/plugins/pitch-shifter/index.ts

Line 42 in 67d6b05

* The renderer is triggered once the YouTube Music player API is available.

🚫 [eslint] <stylistic/no-tabs> _{reported by reviewdog 🐶}
Unexpected tab character.

pear-desktop/src/plugins/pitch-shifter/index.ts

Line 42 in 67d6b05

* The renderer is triggered once the YouTube Music player API is available.

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace ↹ with ··

pear-desktop/src/plugins/pitch-shifter/index.ts

Line 43 in 67d6b05

* It handles all DOM interactions, UI injection, and audio processing.

🚫 [eslint] <stylistic/no-tabs> _{reported by reviewdog 🐶}
Unexpected tab character.

pear-desktop/src/plugins/pitch-shifter/index.ts

Line 43 in 67d6b05

* It handles all DOM interactions, UI injection, and audio processing.

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace ↹ with ··

pear-desktop/src/plugins/pitch-shifter/index.ts

Line 44 in 67d6b05

*/

🚫 [eslint] <stylistic/no-tabs> _{reported by reviewdog 🐶}
Unexpected tab character.

pear-desktop/src/plugins/pitch-shifter/index.ts

Line 44 in 67d6b05

*/

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace ↹ with ··

pear-desktop/src/plugins/pitch-shifter/index.ts

Line 45 in 67d6b05

renderer: {

🚫 [eslint] <stylistic/no-tabs> _{reported by reviewdog 🐶}
Unexpected tab character.

pear-desktop/src/plugins/pitch-shifter/index.ts

Line 45 in 67d6b05

renderer: {

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace ↹↹ with ····

pear-desktop/src/plugins/pitch-shifter/index.ts

Line 46 in 67d6b05

onPlayerApiReady,

🚫 [eslint] <stylistic/no-tabs> _{reported by reviewdog 🐶}
Unexpected tab character.

pear-desktop/src/plugins/pitch-shifter/index.ts

Line 46 in 67d6b05

onPlayerApiReady,

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace ↹ with ··

pear-desktop/src/plugins/pitch-shifter/index.ts

Line 47 in 67d6b05

},

🚫 [eslint] <stylistic/no-tabs> _{reported by reviewdog 🐶}
Unexpected tab character.

pear-desktop/src/plugins/pitch-shifter/index.ts

Line 47 in 67d6b05

},

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace "solid-js" with 'solid-js'

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 1 in 67d6b05

import { createSignal, onCleanup, createEffect } from "solid-js";

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 1 in 67d6b05

import { createSignal, onCleanup, createEffect } from "solid-js";

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace "solid-js/web" with 'solid-js/web'

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 2 in 67d6b05

import { render } from "solid-js/web";

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 2 in 67d6b05

import { render } from "solid-js/web";

🚫 [eslint] <importPlugin/order> _{reported by reviewdog 🐶}
There should be at least one empty line between import groups

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 3 in 67d6b05

import * as Tone from "tone";

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace "tone" with 'tone'

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 3 in 67d6b05

import * as Tone from "tone";

🚫 [eslint] <importPlugin/no-unresolved> _{reported by reviewdog 🐶}
Unable to resolve path to module 'tone'.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 3 in 67d6b05

import * as Tone from "tone";

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 3 in 67d6b05

import * as Tone from "tone";

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace "@/types/contexts" with '@/types/contexts'

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 4 in 67d6b05

import type { RendererContext } from "@/types/contexts";

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 4 in 67d6b05

import type { RendererContext } from "@/types/contexts";

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace "./index" with './index'

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 5 in 67d6b05

import type { PitchShifterPluginConfig } from "./index";

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 5 in 67d6b05

import type { PitchShifterPluginConfig } from "./index";

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Insert ,

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 18 in 67d6b05

{ getConfig, setConfig }: RendererContext<PitchShifterPluginConfig>

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace "[pitch-shifter]·Renderer·(Solid)·initialized·✅" with '[pitch-shifter]·Renderer·(Solid)·initialized·✅'

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 20 in 67d6b05

console.log("[pitch-shifter] Renderer (Solid) initialized ✅");

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 20 in 67d6b05

console.log("[pitch-shifter] Renderer (Solid) initialized ✅");

🚫 [eslint] <@typescript-eslint/no-redundant-type-constituents> _{reported by reviewdog 🐶}
'PitchShift' is an 'error' type that acts as 'any' and overrides all other types in this union type.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 27 in 67d6b05

let pitchShift: Tone.PitchShift | null = null;

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace "video" with 'video'

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 36 in 67d6b05

document.querySelector("video") ||

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 36 in 67d6b05

document.querySelector("video") ||

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace "audio" with 'audio'

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 37 in 67d6b05

document.querySelector("audio") ||

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 37 in 67d6b05

document.querySelector("audio") ||

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace "ytmusic-player·video" with 'ytmusic-player·video'

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 38 in 67d6b05

document.querySelector("ytmusic-player video");

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 38 in 67d6b05

document.querySelector("ytmusic-player video");

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace "[pitch-shifter]·Media·found·🎧" with '[pitch-shifter]·Media·found·🎧'

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 46 in 67d6b05

console.log("[pitch-shifter] Media found 🎧", media);

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 46 in 67d6b05

console.log("[pitch-shifter] Media found 🎧", media);

🚫 [eslint] <@typescript-eslint/no-unsafe-call> _{reported by reviewdog 🐶}
Unsafe call of a(n) error type typed value.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 48 in 67d6b05

await Tone.start();

🚫 [eslint] <@typescript-eslint/no-unsafe-member-access> _{reported by reviewdog 🐶}
Unsafe member access .start on an error typed value.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 48 in 67d6b05

await Tone.start();

🚫 [eslint] <@typescript-eslint/no-unsafe-assignment> _{reported by reviewdog 🐶}
Unsafe assignment of an error typed value.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 49 in 67d6b05

const toneCtx = Tone.getContext();

🚫 [eslint] <@typescript-eslint/no-unsafe-call> _{reported by reviewdog 🐶}
Unsafe call of a(n) error type typed value.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 49 in 67d6b05

const toneCtx = Tone.getContext();

🚫 [eslint] <@typescript-eslint/no-unsafe-member-access> _{reported by reviewdog 🐶}
Unsafe member access .getContext on an error typed value.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 49 in 67d6b05

const toneCtx = Tone.getContext();

🚫 [eslint] <@typescript-eslint/no-unsafe-assignment> _{reported by reviewdog 🐶}
Unsafe assignment of an any value.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Lines 50 to 51 in 67d6b05

    
           const stream = 
        
             (media as any).captureStream?.() || (media as any).mozCaptureStream?.();

🚫 [eslint] <@typescript-eslint/no-unsafe-call> _{reported by reviewdog 🐶}
Unsafe call of a(n) any typed value.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 51 in 67d6b05

(media as any).captureStream?.() || (media as any).mozCaptureStream?.();

🚫 [eslint] <@typescript-eslint/no-explicit-any> _{reported by reviewdog 🐶}
Unexpected any. Specify a different type.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 51 in 67d6b05

(media as any).captureStream?.() || (media as any).mozCaptureStream?.();

🚫 [eslint] <@typescript-eslint/no-unsafe-member-access> _{reported by reviewdog 🐶}
Unsafe member access .captureStream on an any value.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 51 in 67d6b05

(media as any).captureStream?.() || (media as any).mozCaptureStream?.();

🚫 [eslint] <@typescript-eslint/no-unsafe-call> _{reported by reviewdog 🐶}
Unsafe call of a(n) any typed value.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 51 in 67d6b05

(media as any).captureStream?.() || (media as any).mozCaptureStream?.();

🚫 [eslint] <@typescript-eslint/no-explicit-any> _{reported by reviewdog 🐶}
Unexpected any. Specify a different type.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 51 in 67d6b05

(media as any).captureStream?.() || (media as any).mozCaptureStream?.();

🚫 [eslint] <@typescript-eslint/no-unsafe-member-access> _{reported by reviewdog 🐶}
Unsafe member access .mozCaptureStream on an any value.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 51 in 67d6b05

(media as any).captureStream?.() || (media as any).mozCaptureStream?.();

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace "[pitch-shifter]·❌·captureStream()·unavailable" with '[pitch-shifter]·❌·captureStream()·unavailable'

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 53 in 67d6b05

console.error("[pitch-shifter] ❌ captureStream() unavailable");

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 53 in 67d6b05

console.error("[pitch-shifter] ❌ captureStream() unavailable");

🚫 [eslint] <@typescript-eslint/no-unsafe-assignment> _{reported by reviewdog 🐶}
Unsafe assignment of an error typed value.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Lines 60 to 63 in 67d6b05

    
           pitchShift = new Tone.PitchShift({ 
        
             pitch: semitones(), 
        
             windowSize: 0.1, 
        
           }).toDestination();

🚫 [eslint] <@typescript-eslint/no-unsafe-call> _{reported by reviewdog 🐶}
Unsafe call of a(n) error type typed value.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Lines 60 to 63 in 67d6b05

    
           pitchShift = new Tone.PitchShift({ 
        
             pitch: semitones(), 
        
             windowSize: 0.1, 
        
           }).toDestination();

🚫 [eslint] <@typescript-eslint/no-unsafe-call> _{reported by reviewdog 🐶}
Unsafe construction of a(n) error type typed value.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Lines 60 to 63 in 67d6b05

    
           pitchShift = new Tone.PitchShift({ 
        
             pitch: semitones(), 
        
             windowSize: 0.1, 
        
           }).toDestination();

🚫 [eslint] <@typescript-eslint/no-unsafe-member-access> _{reported by reviewdog 🐶}
Unsafe member access .PitchShift on an error typed value.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 60 in 67d6b05

pitchShift = new Tone.PitchShift({

🚫 [eslint] <@typescript-eslint/no-unsafe-member-access> _{reported by reviewdog 🐶}
Unsafe member access .toDestination on an error typed value.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 63 in 67d6b05

}).toDestination();

🚫 [eslint] <@typescript-eslint/no-unsafe-assignment> _{reported by reviewdog 🐶}
Unsafe assignment of an error typed value.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 64 in 67d6b05

nativeSource = toneCtx.createMediaStreamSource(stream);

🚫 [eslint] <@typescript-eslint/no-unsafe-call> _{reported by reviewdog 🐶}
Unsafe call of a(n) error type typed value.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 64 in 67d6b05

nativeSource = toneCtx.createMediaStreamSource(stream);

🚫 [eslint] <@typescript-eslint/no-unsafe-member-access> _{reported by reviewdog 🐶}
Unsafe member access .createMediaStreamSource on an error typed value.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 64 in 67d6b05

nativeSource = toneCtx.createMediaStreamSource(stream);

🚫 [eslint] <@typescript-eslint/no-unsafe-call> _{reported by reviewdog 🐶}
Unsafe call of a(n) error type typed value.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 65 in 67d6b05

Tone.connect(nativeSource, pitchShift);

🚫 [eslint] <@typescript-eslint/no-unsafe-member-access> _{reported by reviewdog 🐶}
Unsafe member access .connect on an error typed value.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 65 in 67d6b05

Tone.connect(nativeSource, pitchShift);

🚫 [eslint] <@typescript-eslint/no-unnecessary-type-assertion> _{reported by reviewdog 🐶}
This assertion is unnecessary since it does not change the type of the expression.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 66 in 67d6b05

media!.muted = true;

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace "[pitch-shifter]·Pitch·processor·active·🎶" with '[pitch-shifter]·Pitch·processor·active·🎶'

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 67 in 67d6b05

console.log("[pitch-shifter] Pitch processor active 🎶");

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 67 in 67d6b05

console.log("[pitch-shifter] Pitch processor active 🎶");

🚫 [eslint] <@typescript-eslint/no-unsafe-call> _{reported by reviewdog 🐶}
Unsafe call of a(n) error type typed value.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 72 in 67d6b05

pitchShift?.dispose();

🚫 [eslint] <@typescript-eslint/no-unsafe-member-access> _{reported by reviewdog 🐶}
Unsafe member access .dispose on an error typed value.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 72 in 67d6b05

pitchShift?.dispose();

🚫 [eslint] <@typescript-eslint/no-unnecessary-type-assertion> _{reported by reviewdog 🐶}
This assertion is unnecessary since it does not change the type of the expression.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 76 in 67d6b05

media!.muted = false;

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace "[pitch-shifter]·Pitch·processor·stopped·📴" with '[pitch-shifter]·Pitch·processor·stopped·📴'

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 77 in 67d6b05

console.log("[pitch-shifter] Pitch processor stopped 📴");

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 77 in 67d6b05

console.log("[pitch-shifter] Pitch processor stopped 📴");

⚠️ [eslint] <stylistic/no-mixed-operators> _{reported by reviewdog 🐶}
Unexpected mix of '+' and '*'. Use parentheses to clarify the intended order of operations.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 94 in 67d6b05

color = cold.map((c, i) => Math.round(c + (neutral[i] - c) * t));

⚠️ [eslint] <stylistic/no-mixed-operators> _{reported by reviewdog 🐶}
Unexpected mix of '+' and '*'. Use parentheses to clarify the intended order of operations.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 94 in 67d6b05

color = cold.map((c, i) => Math.round(c + (neutral[i] - c) * t));

⚠️ [eslint] <stylistic/no-mixed-operators> _{reported by reviewdog 🐶}
Unexpected mix of '+' and '*'. Use parentheses to clarify the intended order of operations.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 98 in 67d6b05

color = neutral.map((c, i) => Math.round(c + (warm[i] - c) * t));

⚠️ [eslint] <stylistic/no-mixed-operators> _{reported by reviewdog 🐶}
Unexpected mix of '+' and '*'. Use parentheses to clarify the intended order of operations.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 98 in 67d6b05

color = neutral.map((c, i) => Math.round(c + (warm[i] - c) * t));

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace "," with ','

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 100 in 67d6b05

return `linear-gradient(90deg, rgb(${color.join(",")}) 0%, #fff 100%)`;

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 100 in 67d6b05

return `linear-gradient(90deg, rgb(${color.join(",")}) 0%, #fff 100%)`;

🚫 [eslint] <stylistic/jsx-sort-props> _{reported by reviewdog 🐶}
Props should be sorted alphabetically

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 112 in 67d6b05

min="-12"

🚫 [eslint] <stylistic/jsx-sort-props> _{reported by reviewdog 🐶}
Props should be sorted alphabetically

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 113 in 67d6b05

max="12"

🚫 [eslint] <stylistic/jsx-sort-props> _{reported by reviewdog 🐶}
Props should be sorted alphabetically

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 114 in 67d6b05

step="1"

🚫 [eslint] <stylistic/jsx-sort-props> _{reported by reviewdog 🐶}
Props should be sorted alphabetically

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 116 in 67d6b05

class="pitch-slider"

🚫 [eslint] <stylistic/jsx-sort-props> _{reported by reviewdog 🐶}
Props should be sorted alphabetically

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 117 in 67d6b05

onInput={(e) => {

🚫 [eslint] <@typescript-eslint/no-unsafe-member-access> _{reported by reviewdog 🐶}
Unsafe member access .pitch on an error typed value.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 122 in 67d6b05

if (pitchShift) pitchShift.pitch = v;

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace ".pitch-label" with '.pitch-label'

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 125 in 67d6b05

const labelEl = document.querySelector(".pitch-label");

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 125 in 67d6b05

const labelEl = document.querySelector(".pitch-label");

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace "active" with 'active'

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 127 in 67d6b05

labelEl.classList.add("active");

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 127 in 67d6b05

labelEl.classList.add("active");

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace "active" with 'active'

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 128 in 67d6b05

setTimeout(() => labelEl.classList.remove("active"), 200);

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 128 in 67d6b05

setTimeout(() => labelEl.classList.remove("active"), 200);

🚫 [eslint] <stylistic/jsx-sort-props> _{reported by reviewdog 🐶}
Props should be sorted alphabetically

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 131 in 67d6b05

ref={(el) => updateSliderColor(el, semitones())}

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace "+"·:·"" with '+'·:·''

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 134 in 67d6b05

{semitones() >= 0 ? "+" : ""}

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 134 in 67d6b05

{semitones() >= 0 ? "+" : ""}

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 134 in 67d6b05

{semitones() >= 0 ? "+" : ""}

🚫 [eslint] <stylistic/jsx-sort-props> _{reported by reviewdog 🐶}
Props should be sorted alphabetically

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 140 in 67d6b05

onClick={() => {

🚫 [eslint] <@typescript-eslint/no-unsafe-member-access> _{reported by reviewdog 🐶}
Unsafe member access .pitch on an error typed value.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 143 in 67d6b05

if (pitchShift) pitchShift.pitch = 0;

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace ".pitch-slider" with '.pitch-slider',

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 145 in 67d6b05

".pitch-slider"

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 145 in 67d6b05

".pitch-slider"

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace ".pitch-label" with '.pitch-label'

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 149 in 67d6b05

const labelEl = document.querySelector(".pitch-label");

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 149 in 67d6b05

const labelEl = document.querySelector(".pitch-label");

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace "active" with 'active'

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 151 in 67d6b05

labelEl.classList.add("active");

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 151 in 67d6b05

labelEl.classList.add("active");

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace "active" with 'active'

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 152 in 67d6b05

setTimeout(() => labelEl.classList.remove("active"), 200);

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 152 in 67d6b05

setTimeout(() => labelEl.classList.remove("active"), 200);

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace "tp-yt-paper-tabs.tab-header-container" with ⏎······'tp-yt-paper-tabs.tab-header-container',⏎····

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 164 in 67d6b05

const tabs = document.querySelector("tp-yt-paper-tabs.tab-header-container");

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 164 in 67d6b05

const tabs = document.querySelector("tp-yt-paper-tabs.tab-header-container");

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace tabs·&&·tabs.parentElement·&&·!document.querySelector(".pitch-wrapper") with ⏎······tabs·&&⏎······tabs.parentElement·&&⏎······!document.querySelector('.pitch-wrapper')⏎····

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 165 in 67d6b05

if (tabs && tabs.parentElement && !document.querySelector(".pitch-wrapper")) {

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 165 in 67d6b05

if (tabs && tabs.parentElement && !document.querySelector(".pitch-wrapper")) {

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace "div" with 'div'

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 166 in 67d6b05

mount = document.createElement("div");

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 166 in 67d6b05

mount = document.createElement("div");

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace "[pitch-shifter]·UI·injected·via·Solid·✅" with '[pitch-shifter]·UI·injected·via·Solid·✅'

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 169 in 67d6b05

console.log("[pitch-shifter] UI injected via Solid ✅");

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 169 in 67d6b05

console.log("[pitch-shifter] UI injected via Solid ✅");

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace ".pitch-wrapper" with '.pitch-wrapper'

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 175 in 67d6b05

const existing = document.querySelector(".pitch-wrapper");

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 175 in 67d6b05

const existing = document.querySelector(".pitch-wrapper");

🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace "[pitch-shifter]·UI·removed·❌" with '[pitch-shifter]·UI·removed·❌'

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 179 in 67d6b05

console.log("[pitch-shifter] UI removed ❌");

🚫 [eslint] <stylistic/quotes> _{reported by reviewdog 🐶}
Strings must use singlequote.

pear-desktop/src/plugins/pitch-shifter/renderer.tsx

Line 179 in 67d6b05

console.log("[pitch-shifter] UI removed ❌");

github-actions · 2025-10-26T16:09:50Z

src/plugins/pitch-shifter/index.ts

+
+/**


🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Delete ⏎

Suggested change

/**

/**

github-actions · 2025-10-26T16:09:51Z

src/plugins/pitch-shifter/index.ts

+ * allowing users to raise or lower the key of a song dynamically.
+ */
+export type PitchShifterPluginConfig = {
+	/** Whether the plugin is enabled (active in the player). */


🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace ↹ with ··

Suggested change

/** Whether the plugin is enabled (active in the player). */

/** Whether the plugin is enabled (active in the player). */

github-actions · 2025-10-26T16:09:51Z

src/plugins/pitch-shifter/index.ts

+ * allowing users to raise or lower the key of a song dynamically.
+ */
+export type PitchShifterPluginConfig = {
+	/** Whether the plugin is enabled (active in the player). */


🚫 [eslint] <stylistic/no-tabs> _{reported by reviewdog 🐶}
Unexpected tab character.

github-actions · 2025-10-26T16:09:51Z

src/plugins/pitch-shifter/index.ts

+ */
+export type PitchShifterPluginConfig = {
+	/** Whether the plugin is enabled (active in the player). */
+	enabled: boolean;


🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace ↹ with ··

Suggested change

enabled: boolean;

enabled: boolean;

github-actions · 2025-10-26T16:09:51Z

src/plugins/pitch-shifter/index.ts

+ */
+export type PitchShifterPluginConfig = {
+	/** Whether the plugin is enabled (active in the player). */
+	enabled: boolean;


🚫 [eslint] <stylistic/no-tabs> _{reported by reviewdog 🐶}
Unexpected tab character.

github-actions · 2025-10-26T16:09:53Z

src/plugins/pitch-shifter/index.ts

+	// ⚙️ ─────────────── Default Configuration ───────────────
+	config: {
+		enabled: false, // Plugin starts disabled by default
+		semitones: 0,   // Neutral pitch (no shift)


🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace ↹↹semitones:·0,·· with ····semitones:·0,

Suggested change

semitones: 0, // Neutral pitch (no shift)

semitones: 0, // Neutral pitch (no shift)

github-actions · 2025-10-26T16:09:53Z

src/plugins/pitch-shifter/index.ts

+	// ⚙️ ─────────────── Default Configuration ───────────────
+	config: {
+		enabled: false, // Plugin starts disabled by default
+		semitones: 0,   // Neutral pitch (no shift)


🚫 [eslint] <stylistic/no-tabs> _{reported by reviewdog 🐶}
Unexpected tab character.

github-actions · 2025-10-26T16:09:53Z

src/plugins/pitch-shifter/index.ts

+	config: {
+		enabled: false, // Plugin starts disabled by default
+		semitones: 0,   // Neutral pitch (no shift)
+	} as PitchShifterPluginConfig,


🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace ↹ with ··

Suggested change

} as PitchShifterPluginConfig,

} as PitchShifterPluginConfig,

github-actions · 2025-10-26T16:09:54Z

src/plugins/pitch-shifter/index.ts

+	config: {
+		enabled: false, // Plugin starts disabled by default
+		semitones: 0,   // Neutral pitch (no shift)
+	} as PitchShifterPluginConfig,


🚫 [eslint] <stylistic/no-tabs> _{reported by reviewdog 🐶}
Unexpected tab character.

github-actions · 2025-10-26T16:09:54Z

src/plugins/pitch-shifter/index.ts

+		semitones: 0,   // Neutral pitch (no shift)
+	} as PitchShifterPluginConfig,
+
+	// 🎨 ─────────────── Plugin Stylesheet ───────────────


🚫 [eslint] <prettier/prettier> _{reported by reviewdog 🐶}
Replace ↹ with ··

Suggested change

// 🎨 ─────────────── Plugin Stylesheet ───────────────

// 🎨 ─────────────── Plugin Stylesheet ───────────────

TheSakyo added 2 commits October 26, 2025 16:48

✨ Add Pitch Shifter plugin (Tone.js )

358f3b7

✨ Add Translation for Pitch Shifter

67d6b05

github-actions bot reviewed Oct 26, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Feature/pitch shifter plugin #4043

Feature/pitch shifter plugin #4043

Uh oh!

TheSakyo commented Oct 26, 2025 •

edited

Loading

Uh oh!

github-actions bot left a comment

Uh oh!

github-actions bot Oct 26, 2025

Uh oh!

github-actions bot Oct 26, 2025

Uh oh!

github-actions bot Oct 26, 2025

Uh oh!

github-actions bot Oct 26, 2025

Uh oh!

github-actions bot Oct 26, 2025

Uh oh!

github-actions bot Oct 26, 2025

Uh oh!

github-actions bot Oct 26, 2025

Uh oh!

github-actions bot Oct 26, 2025

Uh oh!

github-actions bot Oct 26, 2025

Uh oh!

github-actions bot Oct 26, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

	const stream =
	(media as any).captureStream?.() \|\| (media as any).mozCaptureStream?.();

	pitchShift = new Tone.PitchShift({
	pitch: semitones(),
	windowSize: 0.1,
	}).toDestination();

	/** Whether the plugin is enabled (active in the player). */
	/** Whether the plugin is enabled (active in the player). */

	semitones: 0, // Neutral pitch (no shift)
	semitones: 0, // Neutral pitch (no shift)

	} as PitchShifterPluginConfig,
	} as PitchShifterPluginConfig,

	// 🎨 ─────────────── Plugin Stylesheet ───────────────
	// 🎨 ─────────────── Plugin Stylesheet ───────────────

Feature/pitch shifter plugin #4043

Are you sure you want to change the base?

Feature/pitch shifter plugin #4043

Uh oh!

Conversation

TheSakyo commented Oct 26, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✨ Add Pitch Shifter plugin

🧩 Overview

🎛 Features

🧠 Implementation Details

🌍 Localization

🧪 Testing

🙌 Notes

🧑‍💻 Author

Uh oh!

github-actions bot left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions bot Oct 26, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Oct 26, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Oct 26, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Oct 26, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Oct 26, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Oct 26, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Oct 26, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Oct 26, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Oct 26, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Oct 26, 2025

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

TheSakyo commented Oct 26, 2025 •

edited

Loading