fix(b20): correct link_method for candidate wallet match#131
Merged
Conversation
…ToAgent Case 3 (issuer wallet in agent wallet list but not manifest-declared) was incorrectly setting method to "manifest". Changed to "none" — the agent link is still captured via agentName/confidence:candidate. deriveManifestStatus correctly returns "candidate" via the agentName null-check regardless. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01RHDXdEbGQsn88zks713gye
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
linkTokenToAgentcase 3 (issuer wallet found in agent's wallet list, but not manifest-declared) was incorrectly settingmethod: "manifest"— making it look in the DB and UI like a confirmed manifest link when it is only a candidate matchmethod: "none"— the agent association is still captured viaagentName+confidence: "candidate"deriveManifestStatuswas already correct: it gates"attributed"onconfidence === "confirmed"ANDmethod ∈ {manifest, known_token}, so no financial data was ever affected — only thelink_methodmetadata label was wrongData integrity impact: None — financial attribution was never at risk. This is a labelling fix caught during the post-B20-V1 verification pass.
Test plan
link_methodin DB is"none",link_confidenceis"candidate",manifest_statusis"candidate"b20-token-analysisskill returnsmanifest_status: "candidate"not"attributed"for this token🤖 Generated with Claude Code
https://claude.ai/code/session_01RHDXdEbGQsn88zks713gye
Generated by Claude Code