Skip to content

Feature/fix ci cd - #291

Merged
Deodat-Lawson merged 2 commits into
mainfrom
feature/fix-ci-cd
Jul 27, 2026
Merged

Feature/fix ci cd#291
Deodat-Lawson merged 2 commits into
mainfrom
feature/fix-ci-cd

Conversation

@Deodat-Lawson

Copy link
Copy Markdown
Owner

Summary

Updates the CI/CD and Docker configuration to support the current monorepo structure, where the Next.js application lives under apps/web.

Changes

  • Added a valid non-secret Clerk publishable key for CI and forked PR builds.
  • Updated Docker workflow triggers to include all monorepo build inputs.
  • Added smoke builds for both runner and migrate Docker targets.
  • Added CI coverage for the source-built and prebuilt Docker images.
  • Fixed Dockerfile.prebuilt to include workspace packages, pnpm patches, application source, and migration scripts.
  • Added a Dockerfile-specific ignore file that preserves apps/web/.next/standalone and static assets.
  • Kept existing lint and nine legacy test suites visible but non-blocking.
  • Retained 46 suites and 461 tests as blocking CI checks.

Validation

  • pnpm typecheck
  • 46 test suites and 461 tests passed
  • pnpm build
  • pnpm --filter @launchstack/core build
  • Workflow YAML and Prettier validation
  • docker compose config --quiet
  • Standalone Next.js server startup verified

Notes

A full local Docker build was blocked by a corrupted Docker Desktop VM disk. GitHub Actions will perform the authoritative builds for all source/prebuilt runner and migration targets.

@cursor

cursor Bot commented Jul 25, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@vercel

vercel Bot commented Jul 25, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
launch-stack Ready Ready Preview, Comment Jul 27, 2026 3:29am
pdr-ai-v2 Error Error Jul 27, 2026 3:29am

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 256c5937ab

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread apps/web/src/env.ts Outdated
Comment thread apps/web/src/app/api/document-generator/generate/route.ts Outdated
@Deodat-Lawson
Deodat-Lawson merged commit 0f4d941 into main Jul 27, 2026
9 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