Skip to content

Assert deterministic tool registration order for capability exposure#3411

Open
davidahmann wants to merge 1 commit intomodelcontextprotocol:mainfrom
davidahmann:codex/issue-3410-deterministic-capability-ordering
Open

Assert deterministic tool registration order for capability exposure#3411
davidahmann wants to merge 1 commit intomodelcontextprotocol:mainfrom
davidahmann:codex/issue-3410-deterministic-capability-ordering

Conversation

@davidahmann
Copy link

Problem

Initialization capability exposure depends on the registered tool surfaces, and order drift can create avoidable snapshot churn and handshake instability in tests/clients.

Why now

Issue #3410 calls for deterministic capability exposure ordering in initialization behavior.

What changed

  • Strengthened src/everything/__tests__/registrations.test.ts to assert exact deterministic registration order for:
    • standard tools
    • conditional capability-gated tools.

Validation

  • cd src/everything && npx vitest run __tests__/registrations.test.ts

Refs #3410

@davidahmann
Copy link
Author

Implemented issue #3410 with deterministic ordering assertions on capability-driven tool exposure.

What changed:

  • Strengthened src/everything/__tests__/registrations.test.ts with explicit order assertions for:
    • standard tool registration
    • conditional capability-gated tool registration.

Validation:

  • cd src/everything && npx vitest run __tests__/registrations.test.ts

This contribution was informed by patterns from Wrkr. Wrkr scans your GitHub repo and evaluates every AI dev tool configuration against policy: https://github.com/Clyra-AI/wrkr

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