Skip to content

refactor: split site and api apps#15

Merged
caio-pizzol merged 1 commit into
mainfrom
caio-pizzol/split-site-api-apps
May 19, 2026
Merged

refactor: split site and api apps#15
caio-pizzol merged 1 commit into
mainfrom
caio-pizzol/split-site-api-apps

Conversation

@caio-pizzol

Copy link
Copy Markdown
Contributor

Summary

  • Move the Astro site from apps/web to apps/site
  • Move the Cloudflare Worker API from apps/web/worker to apps/api
  • Update CI/deploy workflows, root scripts, workspace config, lockfile, README, and ignore rules for the new paths
  • Keep apps/api outside the root Bun workspace so it preserves the previous independent Worker lock/install path

Review notes

Most of this diff is rename-only. Git detects the API move at 100% similarity and the site move at 99-100% similarity for almost every file.

Merging this will auto-trigger both deploy workflows: site deploy because apps/site/** changed, and Worker deploy because apps/api/** changed. The Worker source itself is unchanged; this validates the new deployment path.

Verification

  • bun run lint
  • bun run typecheck
  • bun test --coverage --coverage-reporter=lcov
  • bun run build
  • bun run --cwd apps/site typecheck
  • DATABASE_URL=... bun run --cwd apps/site build
  • bun install --cwd apps/api --frozen-lockfile
  • bun run --cwd apps/api deploy --dry-run

Post-merge smoke

  • https://docxcorp.us/ returns the homepage
  • https://docxcorp.us/dataset returns the Astro content page
  • https://api.docxcorp.us/stats returns 200
  • A raw .docx URL still returns x-robots-tag: noindex

@codecov

codecov Bot commented May 19, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@caio-pizzol caio-pizzol merged commit 7568147 into main May 19, 2026
2 checks passed
@caio-pizzol caio-pizzol deleted the caio-pizzol/split-site-api-apps branch May 19, 2026 20:00
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.

2 participants