Skip to content

Feat/571 mistyped key - #619

Merged
Weegy merged 1 commit into
byte5ai:feat/mcp-2026-07-28-wave0-wave1from
sneumannb5:feat/571-mistyped-key
Aug 7, 2026
Merged

Feat/571 mistyped key#619
Weegy merged 1 commit into
byte5ai:feat/mcp-2026-07-28-wave0-wave1from
sneumannb5:feat/571-mistyped-key

Conversation

@sneumannb5

@sneumannb5 sneumannb5 commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator

What

Closes #571: a mistyped key_id/agent_id no longer yields a 201 Created dead-but-configured-looking binding. Unknown agent → 400; unknown key → warning; list flags already-dead rows.

Why

Neither id is a foreign key (keys in vault, agents in-process registry, not Postgres), so a typo reached zero tools forever while looking identical to a working binding. Existence is now checked in the app layer against the same sources a real request uses — reusing createVerifyOnlyApiKeyStore, so "known == what the endpoint would accept" by construction. Fail-honest: unreadable source = "cannot tell", never a false reject.

Test plan

Risk / blast radius


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@Weegy
Weegy changed the base branch from main to feat/mcp-2026-07-28-wave0-wave1 August 7, 2026 05:37
@Weegy
Weegy merged commit 2138c94 into byte5ai:feat/mcp-2026-07-28-wave0-wave1 Aug 7, 2026
1 check passed
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.

A mistyped key id or agent id yields a fully-configured-looking dead binding

2 participants