Skip to content

fix(chat): show an album cover that lands after the songs - #271

Merged
ralyodio merged 1 commit into
masterfrom
fix/album-late-cover
Nov 10, 2026
Merged

ralyodio merged 1 commit into
masterfrom
fix/album-late-cover

Conversation

@ralyodio

Copy link
Copy Markdown
Collaborator

In a live /album run the cover sat at #7 in the image queue after the songs finished (about 16 minutes), so the album card showed no cover. After an album finishes without one, the chat now checks /api/v1/album/<id>/cover every 30 s for up to 2 h and shows it when it lands. The zip download includes it once InfernetProtocol/infernet-protocol#186 is live. tsc --noEmit is clean.

🤖 Generated with Claude Code

The image queue can be slower than the songs (live: cover #7 in line after a 16 min album). After an album finishes without a cover, keep checking /api/v1/album/<id>/cover for up to 2 h and show it. Pairs with InfernetProtocol/infernet-protocol#186.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

ThreatCrush Security Scan

13 finding(s)

HIGH/CRITICAL: 1 | MEDIUM: 8 | LOW: 4

Severity Rule Location
HIGH js-ssrf-outbound-request apps/web/public/sw.js:24
MEDIUM js-unescaped-html-sink apps/web/src/app/blog/[slug]/page.tsx:53
MEDIUM js-unescaped-html-sink apps/web/src/app/layout.tsx:89
MEDIUM sh-remote-script-execution scripts/dev-setup.sh:25
MEDIUM sh-eval-expansion scripts/dev-setup.sh:35
MEDIUM sh-remote-script-execution scripts/install.sh:181
MEDIUM sh-remote-script-execution scripts/install.sh:206
MEDIUM sh-remote-script-execution scripts/install.sh:404
MEDIUM sh-remote-script-execution scripts/install.sh:446
LOW py-ssrf-outbound-request examples/business-strategist-agent/chat.py:37
LOW py-ssrf-outbound-request examples/business-strategist-agent/train.py:54
LOW secret-generic-credential node/crates/c0mpute-core/src/webhook.rs:1171
LOW py-path-traversal node/crates/c0mpute-core/tests/fixtures/fake-infernet-text:29

Snippets are redacted; ThreatCrush never prints matched credential material.

@ralyodio
ralyodio merged commit 94e4498 into master Nov 10, 2026
12 checks passed
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.

1 participant