Skip to content

Implement first batch of code restructure#622

Merged
Lucaci-Andrei merged 3 commits intomainfrom
feat/code-restructure-1
Feb 12, 2026
Merged

Implement first batch of code restructure#622
Lucaci-Andrei merged 3 commits intomainfrom
feat/code-restructure-1

Conversation

@Lucaci-Andrei
Copy link
Contributor

@Lucaci-Andrei Lucaci-Andrei commented Feb 11, 2026

Summary by CodeRabbit

  • Chores
    • Removed the demo-office demonstration application and associated configuration files.
    • Updated documentation and test configuration paths for the map-template package.
    • Cleared the demos package index placeholder.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 11, 2026

📝 Walkthrough

Walkthrough

This pull request removes the demo-office package entirely, including its React components, styles, configuration files, and documentation. Additionally, it updates test configuration paths and removes references to the demos package from root documentation.

Changes

Cohort / File(s) Summary
Demo-office Package Removal
packages/demo-office/... (all files removed)
Entire demo-office package deleted, including React components (App, Header, LocationDetails, Search), stylesheets, configuration files (.eslintrc, .gitignore, vite.config.js), environment examples, package.json, and utilities (hooks, tools, fake data).
Root and Demos Documentation
README.md, packages/demos/index.html
Removed reference to demos package from root README; cleared content of packages/demos/index.html.
Test Path Configuration Updates
packages/map-template/tests/README.md, packages/map-template/tests/react-component/package.json, packages/map-template/tests/web-component/package.json
Updated file path references in test documentation and test package configurations; changed dependency path from file:../../map-template/dist to file:../../dist and adjusted dev script paths accordingly.
Test Bootstrap Code Cleanup
packages/tests/react-component/src/main.jsx
Removed React application initialization code including imports and ReactDOM.createRoot().render() block.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Suggested reviewers

  • matbmapspeople

Poem

🐰 Off hops the demo-office, down the rabbit hole,
No more blue dots or office scrolls,
The packages shrink, the tests align,
A cleaner burrow, reorganized and fine! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the primary intent of the changeset - a code restructure affecting multiple packages and removing the demo-office package entirely.
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 docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/code-restructure-1

No actionable comments were generated in the recent review. 🎉


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 and usage tips.

Copy link
Collaborator

@matbmapspeople matbmapspeople left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. It also works in the browser 👍

@Lucaci-Andrei Lucaci-Andrei merged commit 6341c80 into main Feb 12, 2026
3 checks passed
@Lucaci-Andrei Lucaci-Andrei deleted the feat/code-restructure-1 branch February 12, 2026 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants