Skip to content

Wire the YouTube extension bridge into the app (#136) - #420

Merged
rowkav09 merged 1 commit into
mainfrom
feat/youtube-wire-app
Sep 24, 2026
Merged

rowkav09 merged 1 commit into
mainfrom
feat/youtube-wire-app

Conversation

@rowkav09

Copy link
Copy Markdown
Member

Part of #136. Stores a pairing token in the credential store (provider youtube, identity extension, made on first start), routes POST /bridge/youtube to the bridge with the parsed JSON body, and opens only that path to token-checked writes (pairing token + extension origin, per #395/#405). YouTube joins the card source after Spotify using most-recent-start wins; card, preview and hosted card only, never Discord. Skipped in safe mode; a bridge problem never stops the start. Pairing-code display on the settings page is for the web UI lane. Tests: 747 pass.

@rowkav09

Copy link
Copy Markdown
Member Author

/mira resume

@github-project-automation github-project-automation Bot moved this to Backlog in nowplaying Sep 24, 2026
@github-actions

Copy link
Copy Markdown
Contributor

/mira pause

@github-actions github-actions Bot added mira-paused Pause automatic Mira reviews on this pull request size:M labels Sep 24, 2026
@codecov

codecov Bot commented Sep 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

Stores a pairing token in the credential store (made on first start), routes POST /bridge/youtube to the bridge with the JSON body, opens that path to token-checked writes, and adds YouTube to the card source with the most-recent-start rule. Card and hosted card only, never Discord. Skipped in safe mode; a bridge problem never stops the start.
@rowkav09

Copy link
Copy Markdown
Member Author

No review from Mira (auto-paused). Self-reviewed: only /bridge/youtube skips the session check, and the bridge still requires the pairing token and an extension origin (401/403 covered end to end). Discord untouched. Token made once and reused. Rebased on #418/#419; all checks green.

@rowkav09
rowkav09 merged commit 7b63b4e into main Sep 24, 2026
11 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in nowplaying Sep 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers area:security area:server mira-paused Pause automatic Mira reviews on this pull request size:M

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant