Skip to content

Port gHashTag/BrowserOS:trios/agent-server/apps/server/src/api/services/mcp/mcp-server.ts (TypeScript, 1 function) to sp - #8108

Merged
gHashTag merged 2 commits into
masterfrom
queen-5478
Oct 12, 2026
Merged

gHashTag merged 2 commits into
masterfrom
queen-5478

Conversation

@gHashTag

@gHashTag gHashTag commented Oct 9, 2026

Copy link
Copy Markdown
Owner

Closes #5478

Written by a bee on queen-5478 and published by tools/queen/publish.py. The branch itself is the bee's; the second commit is the coordination entry every pull request must add, which a bee has no way to know about.

1 file changed, 210 insertions(+)

🤖 Generated with Claude Code

Trinity Bee and others added 2 commits October 1, 2026 23:29
The turn ended with these files edited and never committed. Uncommitted
work is invisible to the review - it reads the branch - so the attempt
would have been released as empty and the next bee would have started
beside this work rather than from it.

This commit is not a claim that the work is correct. It is the bee's
work, committed on its behalf, and it is judged exactly like any other:
the adversarial reviewer reads it, the compiler runs on it, and the
issue's own criteria are measured against it.

Issue: #5478
Turn: a2d285b6-5f03-465e-b72b-cafcda7bbf17
Ending: finished (the turn closed)
Committed: 1 path(s)
Left uncommitted: 0 path(s) outside the declared boundary
A pull request must add exactly one docs/now entry and a bee has no way
to know that: its brief names a boundary file and acceptance criteria,
and docs/now/ is neither. The publisher adds it rather than failing the
gate.

Closes #5478

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This was referenced Oct 9, 2026
@github-actions

github-actions Bot commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-10-09 10:24:01 UTC

Summary

Status Count
Total Open PRs 50
PRs with Failing Checks 33
PRs with All Checks Green 17
READY 0
FAILING 33
PENDING 0
NO CHECKS YET 0

These columns do not partition: 0 + 33 + 0 + 0 = 33, and there are 50 open PRs. A PR is being counted twice or not at all.

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=3c0ade9e73e4 != manifest seal=87e5cbd3ad94.
    The committed NMSE numbers were certified against an older compiler.rs.
    Run scripts/reseal-check.sh locally for the two-step reseal command (advisory; not a merge gate).

This was referenced Oct 10, 2026
@gHashTag
gHashTag enabled auto-merge (squash) October 12, 2026 10:41
@gHashTag
gHashTag merged commit 30fdda0 into master Oct 12, 2026
26 of 29 checks passed
gHashTag added a commit that referenced this pull request Oct 12, 2026
…le (#9256)

The port of trios/agent-server mcp-server.ts had every builder helper
implemented and tested but the one function the port exists for was a
stub returning undefined. Implement it as the wiring record the
original's McpServer instance ends up with: config and capabilities
from the builders, the SetLevel no-op handler attached, browser tools
registered with the built params, Klavis tools registered only for a
truthy handle.

should_register_klavis_tools modeled 'klavisRef?.handle' as a plain
null check; the original skips registration when the ref is present
with a falsy handle. Gate on handle != 0 as well. Four tests exercise
createMcpServer itself, covering the zero-handle case the null check
would have mis-registered.

Refs #5478 (supersedes the stub in PR #8108)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants