Skip to content

Drop the device name prefix from the media player entity names #63

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
bharvey88 merged 1 commit into from
Jul 31, 2026
Merged

Drop the device name prefix from the media player entity names #63

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions Integrations/ESPHome/Core.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -335,11 +335,11 @@ media_source:
media_player:
- platform: sendspin
id: sendspin_group_media_player
name: "Apollo CAST-1 Sendspin Player"
name: "Sendspin Player"

- platform: speaker_source
id: external_media_player
name: "Apollo CAST-1 Player"
name: "Player"
Comment on lines +338 to +342

@coderabbitai coderabbitai Bot Jul 31, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🗄️ Data Integrity & Integration | 🟡 Minor | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/usr/bin/env bash
set -euo pipefail

# Find old entity-name or entity-ID references outside the renamed definition.
rg -n -i \
  --glob '!Integrations/ESPHome/Core.yaml' \
  'Apollo[[:space:]]+CAST-1|media_player\.apollo_cast_1' . || true

# Confirm both new names and stable internal IDs.
rg -n \
  'id: (sendspin_group_media_player|external_media_player)|name: "(Sendspin Player|Player)"' \
  Integrations/ESPHome/Core.yaml

# Locate both hardware variants for follow-up validation.
fd -t f -a 'CAST-1_(W|ETH)\.yaml$' .

Repository: ApolloAutomation/CAST-1

Length of output: 1110

🏁 Script executed:

#!/usr/bin/env bash
set -euo pipefail

echo "== relevant Core.yaml device section =="
sed -n '300,355p' Integrations/ESPHome/Core.yaml

echo
echo "== ESPHome include references =="
rg -n "Core\.yaml|CAST-1_(W|ETH)\.yaml|id: (sendspin_group_media_player|external_media_player)|platform: speaker_source|speaker:" Integrations/ESPHome -g '*.yaml'

echo
echo "== broader Home Assistant-style references in repo =="
rg -n -i \
  --glob '!node_modules/**' \
  --glob '!dist/**' \
  --glob '!build/**' \
  'apollo_cast_1|caster_cast_1|wizmote|sender|media_player|sendspin|player' . || true

Repository: ApolloAutomation/CAST-1

Length of output: 18837

🏁 Script executed:

#!/usr/bin/env bash
set -euo pipefail

python3 - <<'PY'
from pathlib import Path
import re

core = Path("Integrations/ESPHome/Core.yaml").read_text()
# Extract speaker_source blocks that include id/name in same block.
blocks = re.findall(r'-\s*platform:\s*speaker_source\s+(.*?)\n(?:\n|$)', core, flags=re.S)
for i, b in enumerate(blocks, 1):
    print(f"Speaker source block {i}:")
    for line in b.strip().splitlines():
        print(" ", line)

# Extract nearby internal IDs for the two media player definitions by line ranges.
matches = list(re.finditer(r'id:\s*(sendspin_group_media_player|external_media_player)', core))
for m in matches:
    start=max(0,m.start()-200); end=min(len(core), m.end()+200)
    print(f"\nContext for {m.group(1)}:")
    print(core[start:end])
PY

echo
echo "== YAML device include references (shell) =="
rg -n "Core\.yaml|CAST-1_(W|ETH)\.yaml|esp32|esp8266|project:|platform: speaker_source|speaker:" Integrations/ESPHome -g '*.yaml'

Repository: ApolloAutomation/CAST-1

Length of output: 2951

Update user-facing references for the renamed media player entities.

The internal CSP IDs sendspin_group_media_player and external_media_player stay unchanged, but Home Assistant will expose the names as Sendspin Player and Player. Update all automations, scripts, dashboards, and docs that target the previous Apollo CAST-1 device-prefixed entity IDs. Validate the intended entity names for both the W and ETH variants.

🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@Integrations/ESPHome/Core.yaml` around lines 338 - 342, Update all
automations, scripts, dashboards, and documentation that reference the previous
Apollo CAST-1-prefixed media-player entity IDs to use the Home Assistant
entities exposed by sendspin_group_media_player (“Sendspin Player”) and
external_media_player (“Player”). Keep both internal IDs unchanged, and verify
the references resolve correctly for both W and ETH variants.

volume_increment: 0.05

announcement_pipeline:
Expand Down
Loading