Skip to content

feat: all fixes + self-hosted Google Fonts with Next.js parity #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
blitss wants to merge 4 commits into
base: main
Choose a base branch
from
Open

feat: all fixes + self-hosted Google Fonts with Next.js parity #1

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
fa420b7
feat(fonts): self-hosted Google Fonts with fallback metrics (Next.js …
blitss Apr 2, 2026
6ad8817
fix(deploy): patch dist/server/wrangler.json with env configs from us…
blitss Apr 2, 2026
3e9dd8c
Revert "fix(deploy): patch dist/server/wrangler.json with env configs…
blitss Apr 2, 2026
6bc5c67
feat: user changes to rsc-entry, index, and image shim
blitss Apr 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/preview-release.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ permissions: {}
jobs:
publish:
# Skip for forks — they don't have write access needed by pkg-pr-new.
if: github.repository_owner == 'cloudflare'
if: github.repository_owner == 'cloudflare' || github.repository_owner == 'streamloop'
Comment on lines 13 to +14
Copy link
Copy Markdown

@coderabbitai coderabbitai bot Apr 2, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

🧩 Analysis chain

🏁 Script executed:

find . -name "preview-release.yml" -type f

Repository: streamloop/vinext

Length of output: 100

🏁 Script executed:

cat -n .github/workflows/preview-release.yml

Repository: streamloop/vinext

Length of output: 696

🌐 Web query:

GitHub Actions pull_request event fork head.repo.full_name documentation

💡 Result:

In GitHub Actions workflows triggered by the pull_request event, the full name of the fork repository (head repository) is available in the github context as github.event.pull_request.head.repo.full_name. This is a string in the format "owner/repo" for the repository containing the head branch of the pull request. This property is particularly useful for distinguishing pull requests from forks. For example, to skip a job if the PR originates from a fork: jobs: example: if: github.event.pull_request.head.repo.full_name == github.repository runs-on: ubuntu-latest steps: - ... The github.event context mirrors the webhook payload for the pull_request event, where head.repo is a repository object containing full_name, fork (boolean), name, etc. Official GitHub documentation confirms github.event is the full event webhook payload, and community/official action docs reference this path explicitly for fork detection in pull_request events. Note: This is only populated for pull_request and pull_request_target events. For forks, head.repo.full_name differs from github.repository (the base repo).

Citations:

Skip fork PRs explicitly, not just non-owned repos.

For pull_request events, GitHub sends fork PR events to the base repository. The condition github.repository_owner == 'cloudflare' || github.repository_owner == 'streamloop' will always evaluate to true for PRs to this repository regardless of fork origin, because github.repository_owner refers to the base repository owner, not the PR's source. This means fork PRs still enter the publish job, contradicting the Line 13 comment intent.

To properly skip fork PRs, guard the job with both an owner check and a fork detection:

Suggested fix 
-    if: github.repository_owner == 'cloudflare' || github.repository_owner == 'streamloop'
+    if: >
+      (github.repository_owner == 'cloudflare' || github.repository_owner == 'streamloop') &&
+      (
+        github.event_name != 'pull_request' ||
+        github.event.pull_request.head.repo.full_name == github.repository
+      )
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
# Skip for forks — they don't have write access needed by pkg-pr-new.
if: github.repository_owner == 'cloudflare'
if: github.repository_owner == 'cloudflare' || github.repository_owner == 'streamloop'
# Skip for forks — they don't have write access needed by pkg-pr-new.
if: >
(github.repository_owner == 'cloudflare' || github.repository_owner == 'streamloop') &&
(
github.event_name != 'pull_request' ||
github.event.pull_request.head.repo.full_name == github.repository
)
🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In @.github/workflows/preview-release.yml around lines 13 - 14, The current
workflow condition only checks repository owner and therefore still runs for
forked PRs; update the job's if condition (the same guarded block that
references pkg-pr-new in the workflow) to additionally detect and skip forked
pull requests by requiring either that this is not a pull_request event or that
github.event.pull_request.head.repo.fork == false; in practice replace the
existing if with a combined expression that keeps the owner check
(github.repository_owner == 'cloudflare' || github.repository_owner ==
'streamloop') AND also requires (github.event_name != 'pull_request' ||
github.event.pull_request.head.repo.fork == false) so fork PRs are excluded.

runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v6
Expand Down
737 changes: 736 additions & 1 deletion packages/vinext/README.md
View file
Open in desktop

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions packages/vinext/package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,7 @@
"dev": "vp pack --watch"
},
"dependencies": {
"@capsizecss/metrics": "catalog:",
"@unpic/react": "catalog:",
"@vercel/og": "catalog:",
"magic-string": "catalog:",
Expand Down
4 changes: 3 additions & 1 deletion packages/vinext/src/entries/app-rsc-entry.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1699,7 +1699,9 @@ async function _handleRequest(request, __reqCtx, _mwCtx) {
const __imgResult = validateImageUrl(url.searchParams.get("url"), request.url);
if (__imgResult instanceof Response) return __imgResult;
// In dev, redirect to the original asset URL so Vite's static serving handles it.
return Response.redirect(new URL(__imgResult, url.origin).href, 302);
// Use a relative Location header so the current scheme is preserved behind
// HTTPS-terminating proxies during local development.
return new Response(null, { status: 302, headers: { Location: __imgResult } });
}

// Handle metadata routes (sitemap.xml, robots.txt, manifest.webmanifest, etc.)
Expand Down
Loading