Skip to content

Standardize CDP SQL numeric schemas#874

Open
LeeSee1999 wants to merge 1 commit into
Merit-Systems:mainfrom
LeeSee1999:fix/cdp-sql-number-schemas
Open

Standardize CDP SQL numeric schemas#874
LeeSee1999 wants to merge 1 commit into
Merit-Systems:mainfrom
LeeSee1999:fix/cdp-sql-number-schemas

Conversation

@LeeSee1999
Copy link
Copy Markdown

Fixes #45.

Summary

  • Add shared cdpSqlNumberSchema and cdpSqlIntegerSchema helpers for CDP SQL aggregate responses.
  • Update transfers aggregate result schemas to accept CDP stringified numerics while returning cache-safe JavaScript number values.
  • Remove the local one-off total_amount conversion in bucketed transfer stats now that the result schema handles the coercion consistently.

Verification

  • git diff --check
  • pnpm exec prettier --check <changed transfer service files>
  • pnpm --filter @x402scan/app types:check was attempted locally. It currently fails before a clean app check because the local checkout is missing generated Prisma/Next/workspace type artifacts and reports existing project-wide type errors outside this change. I filtered the output for the modified src/services/transfers paths and did not find errors from this patch.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 14, 2026

@LeeSee1999 is attempting to deploy a commit to the Merit Systems Team on Vercel.

A member of the Team first needs to authorize it.

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.

Standardize the return type of CDP SQL services

1 participant