Skip to content

fix(ci): correct GitHub Pages URL from 4444j99 to organvm throughout#153

Merged
4444J99 merged 1 commit into
mainfrom
limen/gen-organvm-portfolio-ci-green-0626-3ef7
Jun 26, 2026
Merged

fix(ci): correct GitHub Pages URL from 4444j99 to organvm throughout#153
4444J99 merged 1 commit into
mainfrom
limen/gen-organvm-portfolio-ci-green-0626-3ef7

Conversation

@4444J99

@4444J99 4444J99 commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • The portfolio repo is deployed at https://organvm.github.io/portfolio/ (confirmed via GitHub Pages API) but every config file and script hardcoded 4444j99.github.io
  • The post-deploy smoke test was hitting https://4444j99.github.io/portfolio/ (wrong host) causing all 5 URL checks to return 404 and the CI to fail on every push to main
  • Updated 20 files: astro.config.mjs, src/utils/paths.ts, src/layouts/Layout.astro, src/components/head/SchemaOrg.astro, src/pages/feed.xml.ts, src/pages/github-pages.xml.ts, OG image text, utility scripts, and the smoke test fallback URL
  • Also wired DEPLOY_URL: ${{ steps.deployment.outputs.page_url }} into the CI workflow so the smoke test uses the actual deployment URL automatically in the future

Test plan

  • npm run lint — passes (biome, no errors)
  • npm run typecheck:strict — 0 errors, 0 hints
  • npm run test — 272/272 pass

🤖 Generated with Claude Code

The repo is deployed at organvm.github.io/portfolio but all config and
scripts hardcoded 4444j99.github.io, causing the post-deploy smoke test
to hit the wrong host (all 404). Pass DEPLOY_URL from the deployment
step output so future URL changes are picked up automatically.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@4444J99 4444J99 merged commit eaf9b1b into main Jun 26, 2026
1 check passed
@4444J99 4444J99 deleted the limen/gen-organvm-portfolio-ci-green-0626-3ef7 branch June 26, 2026 07:21
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