Skip to content

feat!: remove registry - #77

Merged
gameroman merged 2 commits into
mainfrom
remove-registry
Sep 5, 2026
Merged

gameroman merged 2 commits into
mainfrom
remove-registry

Conversation

@gameroman

@gameroman gameroman commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Removals

    • Removed the registry app’s user-facing pages, navigation, authentication link, error pages, styling, and deployment configuration.
    • Removed the registry app’s build settings, environment template, package manifest, and supporting UI components.
  • Documentation

    • Updated the USTS README link to point to the Rolldown page on npmx.
  • Chores

    • Updated the shared TypeScript configuration reference.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 5, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
❌ Deployment failed
View logs
u c455150 Sep 05 2026, 09:49 PM

@coderabbitai

coderabbitai Bot commented Sep 5, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

Next included review available in 57 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: d77f7b24-4898-4717-be7f-91b706f63464

📥 Commits

Reviewing files that changed from the base of the PR and between 23c229e and c455150.

⛔ Files ignored due to path filters (1)
  • bun.lock is excluded by !**/*.lock
📒 Files selected for processing (2)
  • .github/workflows/ci.yml
  • package.json

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: f27c05f1-d4dd-4bf3-a945-03f1b3da268d

📥 Commits

Reviewing files that changed from the base of the PR and between 6b97a61 and 23c229e.

⛔ Files ignored due to path filters (1)
  • bun.lock is excluded by !**/*.lock
📒 Files selected for processing (18)
  • apps/registry/.env.example
  • apps/registry/README.md
  • apps/registry/astro.config.ts
  • apps/registry/package.json
  • apps/registry/public/.assetsignore
  • apps/registry/src/components/TopBar.astro
  • apps/registry/src/components/ui/Button.tsx
  • apps/registry/src/components/ui/Link.tsx
  • apps/registry/src/layouts/Layout.astro
  • apps/registry/src/pages/404.astro
  • apps/registry/src/pages/500.astro
  • apps/registry/src/pages/index.astro
  • apps/registry/src/styles/global.css
  • apps/registry/tailwind.config.ts
  • apps/registry/tsconfig.json
  • apps/registry/wrangler.jsonc
  • packages/usts/README.md
  • tsconfig.json
💤 Files with no reviewable changes (15)
  • apps/registry/src/components/ui/Button.tsx
  • apps/registry/wrangler.jsonc
  • apps/registry/public/.assetsignore
  • apps/registry/tsconfig.json
  • apps/registry/src/pages/index.astro
  • apps/registry/src/styles/global.css
  • apps/registry/astro.config.ts
  • apps/registry/src/pages/500.astro
  • apps/registry/package.json
  • apps/registry/tailwind.config.ts
  • apps/registry/src/pages/404.astro
  • apps/registry/src/layouts/Layout.astro
  • apps/registry/src/components/TopBar.astro
  • apps/registry/.env.example
  • apps/registry/src/components/ui/Link.tsx

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The registry application and its supporting configuration are removed. The root TypeScript configuration now extends a different shared config path. The USTS README updates its rolldown link.

Changes

Registry application removal

Layer / File(s) Summary
Remove registry pages and components
apps/registry/src/pages/*, apps/registry/src/layouts/Layout.astro, apps/registry/src/components/*
The registry home page, error pages, layout, TopBar, Button, and Link components are deleted.
Remove registry build and runtime configuration
apps/registry/.env.example, apps/registry/package.json, apps/registry/astro.config.ts, apps/registry/tsconfig.json, apps/registry/tailwind.config.ts, apps/registry/wrangler.jsonc, apps/registry/public/.assetsignore, apps/registry/src/styles/global.css
The registry package, Astro and Wrangler settings, TypeScript settings, environment template, Tailwind configuration, asset exclusions, and Tailwind stylesheet import are removed.

Repository metadata updates

Layer / File(s) Summary
Update repository metadata
tsconfig.json, packages/usts/README.md
The root TypeScript configuration extends @gameroman/config/ts. The README links to https://npmx.dev/rolldown.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to 23c22

This change intentionally removes the registry application and updates repository metadata. No concrete issue remains that would prevent merging.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the primary change: removal of the registry app.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch remove-registry

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@gameroman
gameroman merged commit 75b3f26 into main Sep 5, 2026
2 of 3 checks passed
@gameroman
gameroman deleted the remove-registry branch September 5, 2026 21:51
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