Skip to content

fix: cloud api key for preview#219

Merged
Kinfe123 merged 1 commit into
mainfrom
fix/api-key-for-preview
May 25, 2026
Merged

fix: cloud api key for preview#219
Kinfe123 merged 1 commit into
mainfrom
fix/api-key-for-preview

Conversation

@Kinfe123
Copy link
Copy Markdown
Member

@Kinfe123 Kinfe123 commented May 25, 2026

Summary by cubic

Send GitHub repo metadata with preview requests so the Cloud API can validate the API key and return the correct preview URL. This makes cloud preview reliable across branches and subdirectories.

  • Bug Fixes
    • Collects repo owner, name, branch, and root directory via git and sends them in repository when calling /api/cloud/preview.
    • Supports git@github.com:owner/name.git and https://github.com/owner/name(.git) remotes.
    • Test sets up a temp repo, branch, and origin and asserts the request body includes correct metadata.

Written for commit 4e6919b. Summary will update on new commits. Review in cubic

@vercel
Copy link
Copy Markdown

vercel Bot commented May 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs-website Building Building Preview, Comment May 25, 2026 8:29pm

Copy link
Copy Markdown

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 25, 2026

Open in StackBlitz

@farming-labs/astro

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/astro@219

@farming-labs/astro-theme

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/astro-theme@219

@farming-labs/docs

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/docs@219

@farming-labs/theme

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/theme@219

@farming-labs/next

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/next@219

@farming-labs/nuxt

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/nuxt@219

@farming-labs/nuxt-theme

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/nuxt-theme@219

@farming-labs/svelte

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/svelte@219

@farming-labs/svelte-theme

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/svelte-theme@219

@farming-labs/tanstack-start

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/tanstack-start@219

commit: 4e6919b

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

Re-trigger cubic

@Kinfe123 Kinfe123 merged commit 10078c7 into main May 25, 2026
9 of 10 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