Skip to content

arch: remove local web control plane#110

Merged
benvinegar merged 2 commits into
mainfrom
bentlegen/remove-control-plane
Feb 22, 2026
Merged

arch: remove local web control plane#110
benvinegar merged 2 commits into
mainfrom
bentlegen/remove-control-plane

Conversation

@benvinegar
Copy link
Copy Markdown
Member

Summary

Remove the local in-repo web control plane while keeping all operational CLI commands intact.

What changed

  • deleted local web control plane implementation:
    • bin/control-plane.sh
    • control-plane/server.mjs
    • control-plane/server.test.mjs
    • control-plane/package.json + lockfile
  • removed control-plane references from tests/CI:
    • package.json (test:js, test:coverage)
    • vitest.config.mjs
    • bin/test.sh
    • bin/ci/setup-ubuntu.sh
    • bin/ci/setup-arch.sh
  • removed control-plane docs/config mentions:
    • docs/operations.md
    • CONFIGURATION.md
    • SECURITY.md
    • AGENTS.md
  • removed obsolete firewall comment tied to local control-plane port.

Why

Architecture is moving to baudbot-services as the control plane (encrypted, pull-only). Keeping a separate local dashboard in this repo duplicates control-plane surface and ownership.

Validation

  • npm test

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Feb 22, 2026

Greptile Summary

Removed the local web control plane implementation and all references to it from tests, CI, docs, and configuration. The PR deletes 2,600+ lines including the entire control-plane/ directory (server implementation, tests, package files), launcher script, and all documentation references. The architecture is shifting to baudbot-services as the external control plane.

Confidence Score: 4/5

  • Safe to merge after removing the leftover reference in .c8rc.json
  • Clean deletion PR with comprehensive removal of control-plane code and documentation. One leftover reference found in .c8rc.json that needs cleanup. All test configuration and CI scripts properly updated.
  • .c8rc.json contains a stale reference to the deleted control-plane server

Important Files Changed

Filename Overview
bin/test.sh Removed control-plane test from JS test array
control-plane/server.mjs Deleted entire control-plane server implementation (675 lines)
control-plane/server.test.mjs Deleted entire control-plane test suite (208 lines)
package.json Removed control-plane tests from test:js and test:coverage, removed express dependency
vitest.config.mjs Removed control-plane test from vitest include list

Last reviewed commit: bc5a4b2

Copy link
Copy Markdown

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

16 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Feb 22, 2026

Additional Comments (1)

.c8rc.json
leftover reference to deleted control-plane/server.mjs

Prompt To Fix With AI
This is a comment left during a code review.
Path: .c8rc.json
Line: 5

Comment:
leftover reference to deleted `control-plane/server.mjs`

```suggestion
```

How can I resolve this? If you propose a fix, please make it concise.

@benvinegar
Copy link
Copy Markdown
Member Author

Addressed the review feedback: removed the stale control-plane/server.mjs entry from .c8rc.json and pushed commit 328a30d.

Also re-ran coverage command to validate the config path update.

Responded by dev-agent using openai/gpt-5.

@benvinegar benvinegar merged commit 93bbcd6 into main Feb 22, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant