Skip to content

test: cover BIF MCP server startup#9

Merged
hummbl-dev merged 1 commit into
mainfrom
test/codex/bif-mcp-server
May 13, 2026
Merged

test: cover BIF MCP server startup#9
hummbl-dev merged 1 commit into
mainfrom
test/codex/bif-mcp-server

Conversation

@hummbl-dev
Copy link
Copy Markdown
Owner

Summary

  • add a subprocess stdio test that initializes mcp_server.py, lists tools, and calls bif_start_session
  • document BIF_SESSIONS_DIR and MCP startup/client configuration in README

Closes #8

Validation

  • Not run locally; awaiting GitHub Actions.

@hummbl-dev
Copy link
Copy Markdown
Owner Author

REVIEW VERDICT: APPROVE at current head bf58142.

This addresses #8 directly: it adds a subprocess stdio MCP test that initializes the server, lists tools, calls bif_start_session, and verifies session persistence under BIF_SESSIONS_DIR; README now documents startup and MCP client configuration.

Verified:

  • GitHub CI passes on Python 3.11 and 3.12.
  • Local C:\Users\Owner\bin\python.cmd -m pytest tests/ -v passes: 96 passed.
  • Local git diff --check origin/main...HEAD passes.
  • Working tree remained clean after local verification.

No blocking findings. Normal peer/operator merge gate still applies.

@hummbl-dev hummbl-dev merged commit 8983f0e into main May 13, 2026
2 checks passed
@hummbl-dev hummbl-dev deleted the test/codex/bif-mcp-server branch May 13, 2026 21:03
@hummbl-dev
Copy link
Copy Markdown
Owner Author

REVIEW VERDICT: APPROVE

The PR directly covers #8's MCP server startup/test/documentation gap.

Verified:

  • GitHub test matrix passes on Python 3.11 and 3.12.
  • Local python3 -m pytest tests/ -v passes: 96 tests, including the new subprocess MCP server test.
  • Local git diff --check origin/main...HEAD is clean.
  • README now documents BIF_SESSIONS_DIR, python mcp_server.py, client configuration, and supported JSON-RPC methods.

No blocking findings.

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.

Add tests and documentation for the MCP server integration

1 participant