Skip to content

Make public site deployment explicit#88

Merged
Techris93 merged 1 commit into
mainfrom
codex/deploy-public-site-installers
Jul 25, 2026
Merged

Make public site deployment explicit#88
Techris93 merged 1 commit into
mainfrom
codex/deploy-public-site-installers

Conversation

@Techris93

Copy link
Copy Markdown
Owner

Summary

  • explicitly deploy the reviewed website/ directory to the website Cloudflare Pages project
  • use the existing GitHub-managed Cloudflare account token
  • verify both public installer bodies after every relevant production deployment
  • fail the deploy if either endpoint returns HTML or lacks its release marker

Why

The Cloudflare Git integration reported success but the custom domain still returned its SPA fallback for the new Hermes installer. This workflow makes the deployment directory and post-deploy acceptance checks explicit.

Verification

  • focused installer tests pass
  • workflow YAML parses successfully
  • repository diff check passes

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying secopsai with  Cloudflare Pages  Cloudflare Pages

Latest commit: 66364af
Status: ✅  Deploy successful!
Preview URL: https://3a2de0ca.secopsai.pages.dev
Branch Preview URL: https://codex-deploy-public-site-ins.secopsai.pages.dev

View logs

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying website with  Cloudflare Pages  Cloudflare Pages

Latest commit: 66364af
Status: ✅  Deploy successful!
Preview URL: https://782c3404.website-bks.pages.dev
Branch Preview URL: https://codex-deploy-public-site-ins.website-bks.pages.dev

View logs

@Techris93
Techris93 merged commit 0fb8514 into main Jul 25, 2026
16 checks passed
@Techris93
Techris93 deleted the codex/deploy-public-site-installers branch July 25, 2026 21:58

@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: 66364afe1f

ℹ️ 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".

echo "${installer} did not return a POSIX shell installer" >&2
exit 1
fi
grep -F "$marker" "$output" >/dev/null

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Compare fetched installers with this deployment

When an installer changes without changing these release markers—for example, a fix to install.sh while remaining on v1.0.0—the previous deployment has the same shebang and marker, so this check succeeds even if the apex domain still serves stale content or points at the wrong deployment. The SHA query only makes the request URL unique; it does not prove the response came from that commit. Compare the downloaded body with website/${installer} or its digest instead.

Useful? React with 👍 / 👎.

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