Skip to content

FU-P13-T17: Enable broker-hosted Web UI with shared multi-client telemetry#115

Merged
SoundBlaster merged 10 commits into
mainfrom
codex/feature/FU-P13-T17-broker-hosted-webui
Feb 28, 2026
Merged

FU-P13-T17: Enable broker-hosted Web UI with shared multi-client telemetry#115
SoundBlaster merged 10 commits into
mainfrom
codex/feature/FU-P13-T17-broker-hosted-webui

Conversation

@SoundBlaster
Copy link
Copy Markdown
Owner

Description

Implements FU-P13-T17 by enabling broker-hosted Web UI runtime and shared broker telemetry.

Key changes:

  • --broker-daemon --web-ui now starts broker transport and dashboard in one owner process.
  • --broker-spawn --web-ui ... now propagates Web UI args to spawned daemon via BrokerProxy.
  • Broker transport now records initialize identity and tools/call request/response telemetry into shared metrics/audit.
  • Added/updated unit coverage for main broker flow, proxy spawn args, and transport telemetry branches.

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
Copy Markdown

@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: 1d499a48d9

ℹ️ 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".

Comment thread src/mcpbridge_wrapper/broker/transport.py
@SoundBlaster SoundBlaster merged commit 78ff23e into main Feb 28, 2026
10 checks passed
@SoundBlaster SoundBlaster deleted the codex/feature/FU-P13-T17-broker-hosted-webui branch February 28, 2026 13:02
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