* fix(preload): expose desktop bridge as window.feedBackDesktop The core feedback app reads window.feedBackDesktop, but the desktop preload exposed the bridge as window.slopsmithDesktop. On the desktop build window.feedBackDesktop was therefore undefined: the DLC-folder Browse button stayed hidden in both the first-run wizard (#v3-ob-songdir-browse) and Settings (#btn-pick-dlc), and the rest of the bridge silently fell back to browser mode. Finish the rebrand: rename the exposed global slopsmithDesktop -> feedBackDesktop, plus the internal api object, the renderer + plugin-manager consumers, the private __feedBackDesktopAudioHooks scratch namespace, and the comments/migration doc. No compatibility alias — the ecosystem moves to the new name (TARGET-CURRENT). Plugins that still read window.slopsmithDesktop are renamed in their own PRs; nothing ships until the next desktop build bundles them together, so there is no broken shipped artifact. Fixes the "Select DLC Songs Folder — No Browse" report (wizard + Settings, Mac + Windows). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01QbexxfTt8q2tAn436MqGWF * fix(preload): also expose bridge under legacy slopsmithDesktop name Keep plugins/community code built against the pre-rename bridge working after the rename. Same isMainFrame gating. See got-feedback/feedBack-desktop#41. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> --------- Co-authored-by: ChrisBeWithYou <chris@rifflarr.local> Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com> Co-authored-by: byrongamatos <xasiklas@gmail.com>
4.8 KiB
Capability Migration Notes
Slopsmith Desktop is moving first-party renderer integrations away from private
webview globals and legacy song:* events toward Slopsmith capability domains.
The goal is to keep desktop behavior aligned with core while avoiding raw local
filenames, device handles, or native transport objects in plugin-visible state.
Playback Identity For Tone Mappings
Core playback capability v1 emits redaction-safe lifecycle events such as
playback:loading, playback:ready, playback:stopped, and playback:ended.
The playback target contains two public identities:
targetId: arrangement-scoped playback target identity.settingsKey: per-song storage identity shaped likesettings-v1-....
Desktop tone switching now uses target.settingsKey as the primary song key for
localStorage.slopsmith-tone-mappings. Raw filenames from song:loading remain
only as a compatibility fallback when the embedded Slopsmith core does not expose
playback capability v1.
Storage Shape
The store shape does not change:
{
"global": {},
"songs": {
"settings-v1-abc1234": { "Clean": "Clean Preset" }
},
"midiPC": {
"settings-v1-abc1234": { "mode": "midi", "vstSlotId": 0, "mappings": {} }
}
}
Only the per-song bucket key changes. New mappings created on playback-capable
core builds are written under settingsKey. Existing filename-keyed buckets are
left in place so older core builds and older desktop releases can still read
them.
Automatic Migration For Existing Mappings
When playback capability v1 emits playback:loading, desktop receives both the
safe target.settingsKey and the legacy filename fallback. If
localStorage.slopsmith-tone-mappings contains a filename-keyed songs or
midiPC bucket for the legacy filename and the corresponding settingsKey
bucket is missing or empty, desktop copies that bucket to the safe key and leaves
the original bucket untouched.
This migration is intentionally copy-only:
- Existing
settingsKeybuckets win and are not overwritten. - Filename-keyed buckets remain available to older desktop builds or embedded Slopsmith cores without playback capability v1.
- Corrupt or missing mapping stores fall back to the normal empty-store behavior.
For review/debugging, open a song with an existing filename-keyed mapping and
inspect localStorage.slopsmith-tone-mappings: the same mapping should appear
under both the old filename key and the new settings-v1-... key after the
playback:loading event.
Audio Effects Executor
Core audio-effects owns provider selection, policy, safe diagnostics, and the
slopsmith.audio_effects.chain_plan.v1 schema. Desktop owns the trusted physical
executor. Renderer plugins may pass a core-resolved chain plan plus a private
trusted asset map to window.feedBackDesktop.audioEffects.loadChainPlan(...);
desktop validates the schema, authorization, stage kinds, stage counts, opaque
asset references, local asset paths, and extension/kind compatibility before it
builds the native preset JSON and calls the existing native loadPreset path.
The preload surface is:
loadChainPlan(request)— validates and loads a chain plan through the native engine. The request must includeauthorization: "user-action","restore-selection", or"playback-session".inspectRoute(routeKey)— returns a redaction-safe route summary: route key, provider id, plan id, state, stage-kind counts, active segment, and last outcome.activateSegment(request)— applies a segment by translating plan stage ids to loaded native slot ids and callingsetMultiBypass.setStageBypass(request)andsetStageParameter(request)— route stage-level controls to the loaded native slot.
The executor accepts raw paths only inside the trusted asset map passed to
desktop. It never echoes local paths, filenames, model names, IR names, VST state
blobs, native preset JSON, handles, callbacks, DOM nodes, audio buffers, samples,
or waveforms in its public outcomes. Failed rich-provider loads return structured
failed, degraded, unavailable, or no-target outcomes so NAM Tone or core can
fall back cleanly instead of leaving a partially described chain in public state.
Maintainer Checklist
When migrating more desktop integrations to capabilities:
- Prefer capability events and snapshots over
window.playSong,window.stopSong, and rawsong:*events. - Use
target.settingsKeyfor local per-song plugin settings. - Use
targetIdonly for arrangement/session correlation, not persistent per-song settings. - Route effect-chain execution through
window.feedBackDesktop.audioEffectsrather than passing raw native preset JSON through plugin-visible capability state. - Keep raw filename fallback code behind a capability-version check.
- Add static migration guards under
tests/for any removed global wrapper or new capability declaration.