Skip to content

fix(api): normalize skill PATCH request bodies - #157

Open
dirtybits wants to merge 1 commit into
mainfrom
fix/normalize-skill-patch-body
Open

fix(api): normalize skill PATCH request bodies#157
dirtybits wants to merge 1 commit into
mainfrom
fix/normalize-skill-patch-body

Conversation

@dirtybits

Copy link
Copy Markdown
Owner

Summary

  • normalize literal null PATCH bodies to the route's existing missing-fields 400 contract
  • return a client-error 400 for malformed PATCH JSON instead of leaking it through the outer 500 handler
  • add regressions proving both inputs stop before database, signature, and chain work

Test plan

  • npm run format:check
  • npm run lint --workspace @agentvouch/web
  • npm run typecheck --workspace @agentvouch/web
  • npm run test --workspace @agentvouch/web -- --maxWorkers=1 --no-fileParallelism
  • npm exec --workspace @agentvouch/web -- next build --webpack

Notes

  • The webpack build passed. It emitted the repository's existing dynamic-dependency warning from ox/viem, and expected local build-time fallbacks because DATABASE_URL is absent.

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