Skip to content

docs: correct Cloudflare Pages project name and root redirect - #47

Merged
amondnet merged 2 commits into
mainfrom
docs/fix-pages-project-name
Oct 13, 2026
Merged

amondnet merged 2 commits into
mainfrom
docs/fix-pages-project-name

Conversation

@amondnet

@amondnet amondnet commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Follow-up to #43.

  • Pages project name. The docs site deploys to the Cloudflare Pages project code-style, not pleaseai-code-style. This fixes the name in AGENTS.md, ADR-0003, and apps/docs/wrangler.jsonc, so bun run docs:deploy (wrangler pages deploy) targets the real project. The Git-integrated Pages build now runs from the repo root and doesn't read apps/docs/wrangler.jsonc, so CI deploys are unaffected.
  • Root redirect. ADR-0003 said / is served as a meta-refresh page. Nimbus copies Astro's redirects into dist/_redirects when it detects a Cloudflare deploy, so Pages answers / with a 301 to /en, and the meta-refresh index.html is never served. The ADR's constraint, i18n, and URL consequence sections now describe the 301. The comment on redirects in apps/docs/astro.config.ts is corrected the same way.

Related issue

Follow-up to #43.

Checklist

  • PR title follows Conventional Commits
  • Packages build (bun run build): not run; docs and config only
  • Lint/format pass (bun run lint): lint-staged ESLint passed on commit
  • Documentation updated if behavior changed
  • No breaking change

Summary by cubic

Corrects the Cloudflare Pages project name in the docs so bun run docs:deploy targets the real code-style project instead of pleaseai-code-style, and updates ADR-0003 to describe the root URL as a 301 redirect to /en rather than a meta-refresh page. The Git-integrated Pages build runs from the repo root and is unaffected.

Written for commit 1ac4d77. Summary will update on new commits.

Turn on auto-fix

The docs site deploys to the Pages project `code-style`, not
`pleaseai-code-style`. Fix the name in AGENTS.md, ADR-0003, and
apps/docs/wrangler.jsonc so `bun run docs:deploy` targets the real project.

ADR-0003 described `/` as a meta-refresh page. Nimbus writes Astro's
`redirects` into `dist/_redirects`, so Cloudflare Pages answers `/` with a
301 to `/en`. Update the ADR to match.
@gemini-code-assist

Copy link
Copy Markdown

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

Deploying code-style with  Cloudflare Pages  Cloudflare Pages

Latest commit: 1ac4d77
Status: ✅  Deploy successful!
Preview URL: https://b777a4e8.code-style.pages.dev
Branch Preview URL: https://docs-fix-pages-project-name.code-style.pages.dev

View logs

@codacy-production

codacy-production Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@sonarqubecloud

Copy link
Copy Markdown

@amondnet
amondnet marked this pull request as ready for review October 13, 2026 07:31

@cubic-dev-ai cubic-dev-ai 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.

No issues found across 4 files

Architecture diagram
sequenceDiagram
    participant Dev as Developer
    participant CLI as Bun/Wrangler CLI
    participant CF as Cloudflare Pages
    participant Edge as Cloudflare Edge
    participant DNS as DNS (code-style.pages.dev)
    participant User as Site Visitor

    Note over Dev,CF: Deploy Flow (bun run docs:deploy)
    Dev->>CLI: run docs:deploy
    CLI->>CLI: Build Astro site (dist/ output)
    CLI->>CF: wrangler pages deploy (project: code-style)
    CF->>CF: Store static assets (dist/ contents)
    CF-->>CLI: Deployment successful
    CLI-->>Dev: Success message

    Note over User,Edge: Runtime Request Flow
    User->>DNS: GET https://code-style.pages.dev/
    DNS-->>User: Route to Cloudflare Pages edge

    alt Request for "/" (root)
        User->>Edge: GET /
        Edge->>CF: Look up _redirects rules
        CF-->>Edge: 301 redirect to /en
        Edge-->>User: 301 Location: /en
        Note over Edge,CF: Nimbus generated _redirects from Astro redirects config
    end

    alt Request for "/en"
        User->>Edge: GET /en
        Edge->>CF: Fetch static file /en/index.html
        CF-->>Edge: 200 OK (HTML)
        Edge-->>User: Serve English landing page
    end

    alt Request for "/ko" or locale path
        User->>Edge: GET /ko
        Edge->>CF: Fetch static file /ko/index.html
        CF-->>Edge: 200 OK (HTML)
        Edge-->>User: Serve Korean landing page
    end

    Note over Edge,CF: Astro's meta-refresh index.html exists but is never served by Pages
Loading

Turn on auto-fix | Re-trigger cubic

@greptile-apps

greptile-apps Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

[Low impact] The PR appears safe to merge; no actionable defects were found.

Summary

Corrects the docs deployment target and explains the root redirect.

  • Manual docs deploys target the code-style Pages project.
  • Docs explain that Cloudflare Pages redirects / to /en.

Reviews (1) · Last reviewed commit: "docs: correct root redirect comment in a..." · Reviewed by Greptile

@amondnet
amondnet merged commit 620b69e into main Oct 13, 2026
13 checks passed
@amondnet
amondnet deleted the docs/fix-pages-project-name branch October 13, 2026 07:37
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