Skip to content

fix: publish docs preview artifact from CI - #233

Merged
mynameistito merged 1 commit into
mainfrom
fix/preview-ci-artifact
Oct 31, 2026
Merged

mynameistito merged 1 commit into
mainfrom
fix/preview-ci-artifact

Conversation

@mynameistito

@mynameistito mynameistito commented Oct 31, 2026 •

Copy link
Copy Markdown
Owner

Documentation previews fail because successful PR CI runs never upload the alchemy-worker artifact required by the deployment action. Build the exact PR head in the credential-free build job and upload its static site with a minimal asset-serving Worker. Configure preview asset routing to match production.

Run 41182441264 contained no artifacts; the download in run 41182489899 failed before deployment. Existing runs cannot be repaired by rerunning only the preview: the PR must receive a fresh CI run containing this workflow.

Validation: bun run build, bun run typecheck, bun run check, and the preview Worker regression test.

Created with gpt-6.1-terra in Codex.


Summary by cubic

Fixes documentation PR previews failing after successful CI runs, which never uploaded the alchemy-worker artifact required by the deployment job.

The build task now checks out the exact PR head SHA and uploads the built site together with a minimal asset-serving Worker (.github/preview-worker.js), and preview asset routing now matches production's htmlHandling and notFoundHandling configuration. Existing broken previews can't be repaired by rerunning the preview alone; the PR needs a fresh CI run containing this change.

Written for commit 2f18bd9. Summary will update on new commits.

View guided diff Turn on auto-fix

@changeset-bot

changeset-bot Bot commented Oct 31, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 2f18bd9

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@coderabbitai

coderabbitai Bot commented Oct 31, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 47 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 1f0cfc8d-574a-44b6-b863-20c232d931a8
Commits

Reviewing files that changed from the base of the PR and between f8a2930 and 2f18bd9.

Files selected for processing: 4
  • .github/preview-worker.js
  • .github/scripts/__tests__/preview-worker.test.js
  • .github/workflows/ci.yml
  • .github/workflows/deploy-preview.yml
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@github-actions github-actions Bot added github-actions Automatically managed pull request metadata size/s Automatically managed pull request metadata labels Oct 31, 2026
@codecov

codecov Bot commented Oct 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@mynameistito
mynameistito merged commit 1ce98bf into main Oct 31, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github-actions Automatically managed pull request metadata size/s Automatically managed pull request metadata

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant