Skip to content

P3-T11: Add Stop broker/service control button to Web UI#133

Merged
SoundBlaster merged 10 commits intomainfrom
codex/feature/P3-T11-web-ui-stop-broker-control
Mar 3, 2026
Merged

P3-T11: Add Stop broker/service control button to Web UI#133
SoundBlaster merged 10 commits intomainfrom
codex/feature/P3-T11-web-ui-stop-broker-control

Conversation

@SoundBlaster
Copy link
Owner

Description

Add Web UI stop-control support for broker daemon runtime:

  • New control API endpoints (GET /api/control, POST /api/control/stop) with auth enforcement.
  • Dashboard Stop Service button that is shown only when backend reports stop capability.
  • Broker-daemon wiring in __main__.py to request graceful shutdown via delayed self-SIGTERM.
  • Unit tests for supported and unsupported stop flows, plus auth checks.
  • FLOW artifacts for task selection/plan/archive/review.

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Refactoring
  • CI/CD improvement

Quality Gates

Before submitting, ensure all quality gates pass:

make check

Or run individually:

  • make test - All tests pass with ≥90% coverage
  • make lint - No linting errors
  • make format - Code is properly formatted
  • make typecheck - Type checking passes
  • make doccheck - Documentation is synced with DocC (if docs changed)

Documentation Sync

If you modified files in docs/, ensure corresponding DocC files are also updated:

docs/ file DocC file
docs/installation.md mcpbridge-wrapper.docc/Installation.md
docs/cursor-setup.md mcpbridge-wrapper.docc/CursorSetup.md
docs/claude-setup.md mcpbridge-wrapper.docc/ClaudeCodeSetup.md
docs/codex-setup.md mcpbridge-wrapper.docc/CodexCLISetup.md
docs/troubleshooting.md mcpbridge-wrapper.docc/Troubleshooting.md
docs/architecture.md mcpbridge-wrapper.docc/Architecture.md
docs/environment-variables.md mcpbridge-wrapper.docc/EnvironmentVariables.md
README.md mcpbridge-wrapper.docc/mcpbridge-wrapper.md
  • Documentation changes are synced with DocC catalog (or N/A)

Testing

  • Added/updated tests for new functionality
  • All tests pass locally
  • Manually tested the changes

Checklist

  • Code follows the project's style guidelines
  • Self-review completed
  • Comments added for complex code
  • Documentation updated (if needed)
  • No new warnings generated
  • PR title is descriptive

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 132aef5da3

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@SoundBlaster SoundBlaster merged commit c55c934 into main Mar 3, 2026
10 checks passed
@SoundBlaster SoundBlaster deleted the codex/feature/P3-T11-web-ui-stop-broker-control branch March 3, 2026 07:01
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