Skip to content

P14-T1: Bound per-session ID restore maps in broker transport#92

Merged
SoundBlaster merged 8 commits intomainfrom
feature/P14-T1-bound-per-session-id-restore-maps
Feb 20, 2026
Merged

P14-T1: Bound per-session ID restore maps in broker transport#92
SoundBlaster merged 8 commits intomainfrom
feature/P14-T1-bound-per-session-id-restore-maps

Conversation

@SoundBlaster
Copy link
Owner

Description

Bound broker per-session alias maps to active in-flight requests by:

  • releasing id_restore, string_id_map, and int_id_map entries after response routing, shutdown drain, and upstream-write rollback,
  • making local alias allocation wrap-safe by skipping active aliases,
  • adding regression coverage for map pruning and wrap behavior,
  • archiving P14-T1 FLOW artifacts.

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)

make test equivalent run details:

  • pytest / pytest --cov each have one pre-existing local-environment failure: tests/unit/test_broker_transport.py::TestSocketPermissions::test_socket_created_with_0600_permissions (OSError: AF_UNIX path too long).
  • Coverage is 91.33%.

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

@SoundBlaster SoundBlaster merged commit 0ce15ba into main Feb 20, 2026
9 checks passed
@SoundBlaster SoundBlaster deleted the feature/P14-T1-bound-per-session-id-restore-maps branch February 20, 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