Repository navigation
refactor: restructure zod, testing, mcp, migrate packages to multi-file DDD - #74
Merged
Merged
Conversation
…le DDD - Remove stale package-level LICENSE.md files (central root LICENSE.md applies) - Split each package into domain-based directories with index.ts barrel files - Replace any export * with explicit named exports in barrels - Fix tsup dts build for composite tsconfig: dts: false + tsc declaration step - All tests pass, typechecks clean, lint green across all four packages
|
Caution Review failedThe pull request is closed.
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. |
size-limit report 📦
|
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
Restructure four packages (@graph-om/zod, @graph-om/testing, @graph-om/mcp, @graph-om/migrate) from single monolithic index.ts files into multi-file domain-driven designs. Each package's index.ts now serves only as a barrel that manually selects and re-exports public API (no
export *).Changes
License cleanup
DDD restructure per package
@graph-om/zod — schema/ + validation/ domains
@graph-om/testing — driver/, matchers/, fixtures/, generators/ domains
@graph-om/mcp — config, server/, tools/, resources/ domains
@graph-om/migrate — model/, runner/, schema/, version/, template/ domains
Each domain has its own index.ts barrel with explicit named exports.
Build fix
dts: trueis incompatible withcomposite: true+ multi-file sourcesdts: falsein tsup configs and addedtsc --declarationdeclaration step to build scripts (matches existing SDK pattern)Verification
Test Plan
Run
pnpm --filter @graph-om/{zod,testing,mcp,migrate} test— all green.