timothy/ersatztv
1
0
Fork 0
Code Issues 54 Pull Requests 6 Actions Packages Projects Releases 9 Wiki Activity

ersatztv skill states the stack names correctly but does not warn that two of the three fail silently #783

New Issue
Open
opened 2026-08-13 20:10:21 +02:00 by timothy · 0 comments
timothy commented 2026-08-13 20:10:21 +02:00
Owner
Copy Link
Copy Source

Small gap found while closing out #720 (/done vault + skill cross-check).

.claude/skills/ersatztv/SKILL.md is factually correct about the stacks — line 30's table has prod as Komodo jazz-media (source docker/jazz/stacks/media-servers/compose.yaml) and test as Komodo ersatztv, and line 36 correctly records auto_update: false for prod. So nothing here is wrong.

What it lacks is the warning, which is what #720 established is the actual hazard: of the three names in play, two are wrong and both fail silently.

  • DeployStack media-servers → a dead stack lingering on bumblebee. Succeeds, promotes nothing.
  • DeployStack ersatztv → the test channel (:latest, auto-updating). Succeeds, looks healthy, promotes nothing to prod.

An agent reading the table sees both names and has no reason to suspect that reaching for the obvious one is wrong. The skill is the first thing read before touching ErsatzTV, so this is the place the warning earns its keep.

Suggested fix

One or two lines near the stack table, carrying the rule already adopted in docs/ci-cd.md (#720) and ~/homelab-docs/Docker/ErsatzTV.md:

Resolve the prod stack by the container's com.docker.compose.project.config_files label, never by stack name — prod's points at …/stacks/jazz-media/…/media-servers/compose.yaml, test's at …/stacks/ersatztv/…/ersatztv/compose.yaml.

Why this was not just done inline

.claude/** is excluded from the docs-only auto-pass on review-verdict/h10 (#698), so even a one-line skill edit needs a real review verdict and a full CI cycle — materially more than the "just fix it" edits this class usually deserves. Filed rather than rushed at session end; it is genuinely low priority because the skill states no falsehood.

Worth folding into the next session that touches this skill for any other reason, rather than being its own PR.

Done-when

  • The skill warns that media-servers and ersatztv are both wrong-and-silent, and states the label-based resolution rule
  • Adversarial review passed
Small gap found while closing out #720 (`/done` vault + skill cross-check). `.claude/skills/ersatztv/SKILL.md` is **factually correct** about the stacks — line 30's table has prod as Komodo `jazz-media` (source `docker/jazz/stacks/media-servers/compose.yaml`) and test as Komodo `ersatztv`, and line 36 correctly records `auto_update: false` for prod. So nothing here is wrong. What it lacks is the **warning**, which is what #720 established is the actual hazard: of the three names in play, two are wrong and both fail *silently*. - `DeployStack media-servers` → a dead stack lingering on bumblebee. Succeeds, promotes nothing. - `DeployStack ersatztv` → the **test** channel (`:latest`, auto-updating). Succeeds, looks healthy, promotes nothing to prod. An agent reading the table sees both names and has no reason to suspect that reaching for the obvious one is wrong. The skill is the first thing read before touching ErsatzTV, so this is the place the warning earns its keep. ## Suggested fix One or two lines near the stack table, carrying the rule already adopted in `docs/ci-cd.md` (#720) and `~/homelab-docs/Docker/ErsatzTV.md`: > Resolve the prod stack by the container's `com.docker.compose.project.config_files` label, never by stack name — prod's points at `…/stacks/jazz-media/…/media-servers/compose.yaml`, test's at `…/stacks/ersatztv/…/ersatztv/compose.yaml`. ## Why this was not just done inline `.claude/**` is excluded from the docs-only auto-pass on `review-verdict/h10` (#698), so even a one-line skill edit needs a real review verdict and a full CI cycle — materially more than the "just fix it" edits this class usually deserves. Filed rather than rushed at session end; it is genuinely low priority because the skill states no falsehood. Worth folding into the next session that touches this skill for any other reason, rather than being its own PR. ## Done-when - [ ] The skill warns that `media-servers` and `ersatztv` are both wrong-and-silent, and states the label-based resolution rule - [ ] Adversarial review passed
timothy added the ci-cdpriority: low labels 2026-08-13 20:10:26 +02:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
ad-hoc
One-off / ad-hoc work not tracked by a dedicated issue
 api
REST API / HTTP endpoints
 bug
Something isn't working
 ci-cd
Build, test, deploy pipeline
 content
Channel content / schedules / playlists
 dependencies
Dependency updates (Renovate)
 enhancement
New feature or improvement
 frontend
ChicoryTV React SPA frontend
 in-progress
Claimed by an active session — do not pick up
 jellyfin
Jellyfin tuner / IPTV integration
 parked
Excluded from automatic queue pickup; work only when explicitly selected
 priority: high
priority: low
priority: medium
review
Adversarial review finding
 security
Security / vulnerability fix
No labels ci-cd priority: low
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
renovate timothy (Timothy)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: timothy/ersatztv#783