Refactor _Controls component by removing unused DomInspector code and…

[guitavano]

Summary by CodeRabbit

• Refactor
  • Removed internal DOM inspector integration and associated runtime setup
  • Simplified component dependencies and imports for cleaner architecture
  • Updated component interface to support flags-based configuration with sensible defaults

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

The _Controls.tsx component removes runtime DOM inspector integration, simplifying dependencies and removing localhost-only logic. The component now accepts a flags parameter and adds a flags property to the Live interface.

Changes

Cohort / File(s)	Summary
DOM Inspector Removal & Type Updates website/components/_Controls.tsx	Removed DomInspector and DomInspectorActivators integration including runtime instance setup and script injection. Added flags: Flag[] property to Live interface and optional flags?: Flag[] to Props type. Updated LiveControls signature to accept flags with default value []. Simplified imports to remove inspector-related dependencies.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 Inspector be gone, the DOM runs free,
Flags now flutter where controls can see,
Localhost whispers fade to quiet,
Simpler types lead the diet! ✨

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The pull request description is completely empty, missing all required sections from the template including what the contribution is about, issue links, and demonstration details.	Add a complete description following the template: explain the refactoring rationale, link the relevant issue, and optionally provide a Loom video or demonstration link for reviewer context.
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title is partially related to the changeset, referring to a real aspect of the changes (removing DomInspector code), but is incomplete with an ellipsis and doesn't convey the full scope.

✨ 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 tavano/remove-dom-inspector

---

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9dd7e8a and e3038e1.

📒 Files selected for processing (1)

• website/components/_Controls.tsx

🔇 Additional comments (3)

website/components/_Controls.tsx (3)

81-81: LGTM! Appropriate default value for flags parameter.

The addition of the flags parameter with a default empty array is well-designed:

• Maintains backward compatibility (existing callers unaffected)
• Ensures window.LIVE.flags is always an array (never undefined)
• Aligns with the Props interface where flags is optional

---

1-97: The DOM inspector removal is safe—no dependent code found.

The refactoring successfully removes unused DOM inspector integration with no external dependencies in the codebase.

---

4-4: No issues found. The type imports (Flag and Site) from @deco/deco are correctly used throughout the component and are legitimate exports from the package, as confirmed by their usage across the codebase.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[github-actions]

Tagging Options

Should a new tag be published when this PR is merged?

• 👍 for Patch 0.133.22 update
• 🎉 for Minor 0.134.0 update
• 🚀 for Major 1.0.0 update