refactor: align Ring config and infrastructure with Staryl patterns#81
Merged
Conversation
- Centralize guild limit definitions in src/lib/common/limits.ts - Add lib/common and lib/types structure with #common and #types aliases - Migrate from @skyra/* v2 to @wolfstar/* v3 packages - Align setup lifecycle with Staryl (env in setup, setInvite, import order) - Add compose.dev.yml for local Postgres development - Update AGENTS.md and copilot-instructions with dev notes
|
Caution Review failedAn error occurred during the review process. Please try again later. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Combine main's dependency bumps (taze, tsdown, commitlint, eslint, oxlint) with the feature branch's @wolfstar/* v3 migration and TypeScript 5.9.
|
Caution Failed to replace (edit) comment. This is likely due to insufficient permissions or the comment being deleted. Error details |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Aligns Ring's configuration layer and project infrastructure with Staryl patterns while preserving Ring's role as the authoritative guild limits store and the existing
GET /guilds/:idAPI contract.Changes
Centralized guild limits (
src/lib/common/limits.ts)getMappings()src/lib/types/limits.ts(StarylLimits,WolfstarLimits,GuildLimits)/configcommandProject structure (Staryl-style)
src/lib/common/andsrc/lib/types/with barrel exportssrc/lib/types/augments.tsGuildtype re-exported fromsrc/lib/setup/prisma.ts#common/*,#types,#types/*tsdownupdated with index.ts fallback resolver anddts: truePackage migration
@skyra/*v2 →@wolfstar/*v3 across all source files~5.9.3Setup alignment
envRun()moved insidesetup()with Staryl import order (logger → prisma → fastify)setInvite()alongsidesetRepository("ring")Dev experience
compose.dev.ymlfor local PostgresAGENTS.mdand.github/copilot-instructions.mdwith dev notes and new aliasessrc/.envtemplate with.env.localguidance and Staryl token pairing/config getimprovementLimitDefinitionswhen no DB row exists (consistent with API behavior)API contract preserved
GET /guilds/:idresponse shape and auth flow are unchanged for Staryl compatibility:{ maximumYouTubeSubscriptions, maximumTwitchSubscriptions }with defaults3, 5Test plan
pnpm prisma:generatepnpm buildpnpm lint:fix