Skip to content

ci: move the production docs deploy to Blacksmith and its build cache - #8127

Merged
saddlepaddle merged 1 commit into
mainfrom
saddlepaddle/deploy-prod-followup
Oct 4, 2026
Merged

saddlepaddle merged 1 commit into
mainfrom
saddlepaddle/deploy-prod-followup

Conversation

@saddlepaddle

@saddlepaddle saddlepaddle commented Oct 4, 2026 •

Copy link
Copy Markdown
Collaborator

Why

#8101 missed the production docs deploy. It stayed on a GitHub runner with no Next build cache, and the preview docs job lost its cache step because I wrongly believed production had no docs deploy.

What changes

  • Deploy Docs to Vercel runs on Blacksmith 4 vCPU and writes the Next build cache, like the other four apps.
  • The preview docs job restores that cache.
  • The five production deploy jobs install the Vercel CLI with bun, as the previews already do.

Verified

  • This PR's preview run is the first one after production wrote the Next build cache on main, so it also measures the warm path for API, Web, Marketing and Admin.

Not verified

  • The production side only runs on push to main.

Summary by cubic

Finishes the production deploy move that #8101 started, which missed the docs job. Docs now deploys on Blacksmith and writes a Next build cache that the preview job restores, and all five production deploy jobs install the Vercel CLI with bun instead of npm.

The production side only runs on push to main, so the warm cache path is only verified by the next production run.

Written for commit f2a7a64. Summary will update on new commits.

Review in cubic

Summary by CodeRabbit

  • Chores
    • Updated deployment workflows to reuse documentation build cache and install the deployment tool consistently. These changes affect deployment processes only; no user-facing functionality changed.

#8101 missed the production docs deploy: it stayed on a GitHub runner
with no Next build cache, and the preview docs job lost its cache step
on the belief that production had none. Docs now runs on Blacksmith
and writes the cache that the preview reads, like the other four apps.

The five production jobs also install the Vercel CLI with bun, as the
previews do.
@coderabbitai

coderabbitai Bot commented Oct 4, 2026

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 37a847c4-d11c-432a-bd0f-4d9452950fc7
📥 Commits

Reviewing files that changed from the base of the PR and between e1347c7 and f2a7a64.

📒 Files selected for processing (2)
  • .github/workflows/deploy-preview.yml
  • .github/workflows/deploy-production.yml
 _____________________________________________________________________________________________________________________________
< If debugging is the process of removing bugs, then programming must be the process of putting them in. - Edsger W. Dijkstra >
 -----------------------------------------------------------------------------------------------------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · 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

Comment @coderabbitai help to get the list of available commands.

@greptile-apps

greptile-apps Bot commented Oct 4, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

[High risk] Changes production deployment infrastructure and build tooling.

The PR appears safe to merge based on the reviewed workflow changes.

Summary

The PR moves the production docs deploy to Blacksmith, adds a production-written docs build cache for preview restores, and switches five production Vercel CLI installations to Bun.

  • The cache steps follow the pattern already used by the other deployed apps.
  • No actionable regression was established.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[Main push: production docs build] --> B[Save docs Next.js cache]
  B --> C[PR preview: restore docs cache]
  C --> D[Build and deploy docs preview]
Loading

Reviews (1) · Last reviewed commit: "ci: finish moving the production deploys..."

@github-actions

github-actions Bot commented Oct 4, 2026 •

Copy link
Copy Markdown
Contributor

🧹 Preview Cleanup Complete

The following preview resources have been cleaned up:

  • ✅ Neon database branch

Thank you for your contribution! 🎉

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

Copy link
Copy Markdown
Contributor

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

Architecture diagram
sequenceDiagram
    participant GH as GitHub Actions
    participant BS as Blacksmith Runner
    participant CA as Cache Store
    participant VE as Vercel CLI
    participant VAPI as Vercel API

    Note over GH,BS: Production Deploy (push to main)

    GH->>BS: Trigger deploy-docs job
    BS->>BS: Setup Bun runtime
    BS->>VE: Install Vercel CLI (bun add --global)
    BS->>CA: Save Next.js build cache (key: next-docs-{sha})
    BS->>BS: Build docs app
    BS->>VE: Run vercel deploy --prod
    VE->>VAPI: Deploy docs to production
    VAPI-->>VE: Deployment URL
    VE-->>BS: Success

    Note over GH,BS: Preview Deploy (PR)

    GH->>BS: Trigger preview docs job
    BS->>BS: Setup Bun runtime
    BS->>VE: Install Vercel CLI (bun add --global)
    BS->>CA: Restore Next.js build cache (restore-keys: next-docs-)
    alt Cache hit
        CA-->>BS: Restored cache
    else Cache miss
        CA-->>BS: No cache found (cold build)
    end
    BS->>BS: Build docs app
    BS->>VE: Run vercel deploy --preview
    VE->>VAPI: Deploy docs preview
    VAPI-->>VE: Preview URL
    VE-->>BS: Success
Loading

Re-trigger cubic

@saddlepaddle
saddlepaddle merged commit bad1dee into main Oct 4, 2026
18 of 19 checks passed

This branch was previously deployed

1 inactive deployment
preview — f2a7a64f Deployed Oct 4, 2026 by saddlepaddle via Deploy Database (Neon) #10997
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