775 feature restructure blocks and UI components by domain grouping#790
Merged
tomaszpacior merged 14 commits intomainfrom Mar 17, 2026
Merged
Conversation
Update workspace and generator/eject tooling for nested block domains. Add domain docs and a changeset for the restructuring rollout. Made-with: Cursor
Reorganize all block packages under packages/blocks/<domain>/<block>. Update block package metadata and per-block README domain sections. Made-with: Cursor
Fix block API responses by flattening observable-like payloads in logger interception. Add defensive guards in block renderers to prevent SSR crashes on incomplete data. Made-with: Cursor
Avoid unsafe access to table action labels by using a stable local reference. This removes the TypeScript undefined check failure during monorepo build. Made-with: Cursor
Move @o2s/ui components to grouped paths under components/<group>/<Component>. Update package exports and TS config for nested component imports. Made-with: Cursor
Update frontend, block renderers, docs, and templates to use grouped @o2s/ui component paths. Made-with: Cursor
Contributor
|
Important Review skippedToo many files! This PR contains 299 files, which is 149 over the limit of 150. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (299)
You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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 Tip You can disable the changed files summary in the walkthrough.Disable the |
…ents-by-domain-grouping
…ents-by-domain-grouping
…ain-grouping' of github.com:o2sdev/openselfservice into 775-feature-restructure-blocks-and-ui-components-by-domain-grouping
…ents-by-domain-grouping
…ents-by-domain-grouping
Contributor
Coverage Report for packages/configs/vitest-config
File Coverage
|
||||||||||||||||||||||||||||||||||||||
marcinkrasowski
approved these changes
Mar 17, 2026
tomaszpacior
deleted the
775-feature-restructure-blocks-and-ui-components-by-domain-grouping
branch
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
2 participants
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.
What does this PR do?
Related Ticket(s)
Key Changes
packages/blocksinto a domain-based layout:packages/blocks/<domain>/<block>.turbo/generators/config.ts, block templates),eject-block, CLI block discovery),repository.directory,homepage),ConfigModuleimports in selected API harmonization modules,QuickLinks,ProductList,ArticleSearch),packages/ui/src/componentsinto grouped domains:layout,data,feedback,cards,charts,commerce,content,navigation,forms,actions,media(breaking cleanup; no legacy proxy paths).
@o2s/uiexports, TS config, and UI docs.Side effects / impact
@o2s/ui/components/<group>/<Component>.blocks,ui,frontend,docs, tooling), so merge conflicts are more likely on imports/docs files.nullfor incomplete block payloads instead of crashing at runtime.