Skip to content

fix: add Daytona cloud logo#3274

Merged
louisgv merged 3 commits intoOpenRouterTeam:mainfrom
AhmedTMM:fix/daytona-logo
Apr 12, 2026
Merged

fix: add Daytona cloud logo#3274
louisgv merged 3 commits intoOpenRouterTeam:mainfrom
AhmedTMM:fix/daytona-logo

Conversation

@AhmedTMM
Copy link
Copy Markdown
Collaborator

Summary

  • Adds Daytona logo (128x128 PNG from their GitHub org avatar) to assets/clouds/daytona.png
  • Adds icon field to Daytona cloud entry in manifest.json
  • Cloud picker will now show the Daytona logo instead of a text "D" placeholder

🤖 Generated with Claude Code

Adds the Daytona icon (from their GitHub org avatar) so the cloud
picker shows a proper logo instead of a text "D" placeholder.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown
Member

@louisgv louisgv left a comment

Choose a reason for hiding this comment

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

Security Review

Verdict: APPROVED ✓
Commit: 762562e

Summary

This PR adds a logo for the Daytona cloud provider. Changes are limited to:

  • Adding assets/clouds/daytona.png (2KB PNG file)
  • Adding source URL to assets/clouds/.sources.json
  • Adding icon field to manifest.json (daytona cloud entry)

Security Analysis

  • Asset source: GitHub avatar (https://avatars.githubusercontent.com/u/130513197) - legitimate GitHub CDN URL
  • Icon URL pattern: Follows established convention (raw.githubusercontent.com/OpenRouterTeam/spawn/main/assets/clouds/)
  • JSON structure: Valid, consistent with other cloud entries (hetzner, aws, digitalocean, gcp, sprite)
  • File size: 2KB PNG - reasonable for a logo
  • No code changes: Only static assets and configuration

Tests

  • JSON validation: PASS
  • Test suite: PASS (2104 tests, 0 failures)
  • Icon URL consistency: PASS (matches pattern used by 5 other clouds)

Findings

No security concerns identified.


-- security/pr-reviewer

@louisgv louisgv merged commit d927770 into OpenRouterTeam:main Apr 12, 2026
5 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.

2 participants