improve architecture

[mad-raccoon]

Why is this pull-request needed?

Improve architecture

Summary by CodeRabbit

Release Notes

• Documentation
  • Updated architecture documentation with clearer project structure and component organization.
  • Added comprehensive guidance for component patterns, state handling, and data management practices.
  • Enhanced developer workflow documentation for creating new components with updated examples and best practices.

[coderabbitai]

Walkthrough

Documentation update to ARCHITECTURE.md that restructures component pattern guidance, introducing explicit definition.ts and ComponentName.emb.ts file conventions while reorganizing project structure examples and adding detailed examples for state handling, data queries, and component composition.

Changes

Cohort / File(s)

Summary

Architecture Documentation ARCHITECTURE.md

Comprehensive reorganization of component architecture patterns, replacing previous charts/shared layout with granular components/editors/theme/utils structure. Introduces two-file component definition pattern (definition.ts and .emb.ts), expands guidance on state handling, multi-query data fetching, and strong typing, includes updated workflow for adding new components and new code examples throughout.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The pull request description is largely incomplete, containing only a restatement of the title without addressing the required template sections.	Complete the description by adding 'Main changes' section detailing the new component pattern and project structure updates, and 'Test evidence' section documenting how the documentation changes were verified.
Title check	❓ Inconclusive	The title 'improve architecture' is vague and generic, failing to convey the specific nature of the architectural changes made to the documentation.	Consider a more specific title like 'Refactor ARCHITECTURE.md to document new component pattern and project structure' to better reflect the actual changes.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch RUI-000_improve_architecture

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@ARCHITECTURE.md`:
- Line 7: The fenced code block opening (``` ) in ARCHITECTURE.md is missing a
language tag and triggers markdownlint MD040; update the opening fence to
include a language (for example change ``` to ```text or ```bash) for the tree
block so the markdown linter recognizes the language and the rule is satisfied.

---

ℹ️ Review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Cache: Disabled due to Reviews > Disable Cache setting

Knowledge base: Disabled due to data retention organization setting

📥 Commits

Reviewing files that changed from the base of the PR and between be029d6 and 31351b4.

📒 Files selected for processing (1)

• ARCHITECTURE.md