Skip to content

fix(api): validate public agent route pubkeys - #170

Merged
dirtybits merged 1 commit into
mainfrom
fix/validate-agent-public-route-pubkeys
Sep 5, 2026
Merged

fix(api): validate public agent route pubkeys#170
dirtybits merged 1 commit into
mainfrom
fix/validate-agent-public-route-pubkeys

Conversation

@dirtybits

Copy link
Copy Markdown
Owner

Summary

  • reject malformed Solana wallet path parameters in public agent identity and trust GET routes before downstream work
  • reuse the configured Solana context and shared isValidChainAddress boundary helper
  • add no-side-effect regressions and replace invalid success fixtures with a valid Solana public key
  • record the focused implementation and verification in .agents/plans/validate-agent-public-route-pubkeys.plan.md

Test plan

  • npm test --workspace @agentvouch/web -- __tests__/api/agent-identity-route.test.ts __tests__/api/agent-trust-route.test.ts --maxWorkers=1 --no-fileParallelism (17 tests)
  • npm run format:check
  • npm run lint:web
  • npm run typecheck
  • npm test --workspace @agentvouch/web -- --maxWorkers=1 --no-fileParallelism (128 files / 929 tests)
  • npm exec --workspace @agentvouch/web -- next build --webpack
  • git diff --check

Notes

No live Solana RPC, database, browser, wallet, or deployment flow was run. The webpack build passed with the repository's existing ox dynamic-dependency warning and expected static-generation DATABASE_URL fallback logs because local database credentials are absent.

@dirtybits
dirtybits merged commit ccb432b into main Sep 5, 2026
3 checks 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.

1 participant