fix: resolve 3 bugs in MicroAI-Paygate

[saurabhhhcodes]

Description

This PR fixes real bugs found in the codebase:

• Removed return undefined: bare return conveys the same intent without the redundant literal.
• Removed return undefined: bare return conveys the same intent without the redundant literal.
• Removed return undefined: bare return conveys the same intent without the redundant literal.

Type of Change

• Bug fix (non-breaking change fixing an issue)

How Has This Been Tested?

• Local manual testing

Checklist

• My code follows the style guidelines
• I have performed a self-review

Related Issue

Ref: #403

Summary by CodeRabbit

• Refactor
  • Simplified internal handling of missing request details and authorization data.
  • No user-visible behavior or public API changes.

Note

Fix bare return bugs in MicroAI-Paygate client, protocol, and error handling

Fixes three functions that incorrectly returned undefined explicitly instead of using a bare return. Affected functions are PaygateClient.serializeRequestBody in client.ts, MicroAIPaygateProtocol.getPayer in microai.ts, and sanitizeDetail in errors.ts.

^{Macroscope summarized 849a4e8.}

[vercel]

Someone is attempting to deploy a commit to the ankanmisra's projects Team on Vercel.

A member of the Team first needs to authorize it.

[coderabbitai]

📝 Walkthrough

Walkthrough

The change replaces three explicit return undefined statements with bare returns in TypeScript request serialization, payer extraction, and web error-detail sanitization. Observable behavior remains unchanged.

Changes

Implicit undefined returns

Layer / File(s)	Summary
Replace explicit undefined returns sdk/typescript/src/client.ts, sdk/typescript/src/protocol/microai.ts, web/src/lib/errors.ts	The affected early-return branches now use bare returns when no value is available. Their behavior remains unchanged.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

• AnkanMisra/MicroAI-Paygate#340: Applies the same changes to the same functions.
• AnkanMisra/MicroAI-Paygate#350: Replaces redundant return undefined statements with bare returns.
• AnkanMisra/MicroAI-Paygate#396: Applies the same changes to serializeRequestBody, getPayer, and sanitizeDetail.

Suggested reviewers: ankanmisra

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title identifies the three MicroAI-Paygate changes and the affected functionality, although the changes are behavior-preserving refactors.
Description check	✅ Passed	The description explains the changes, type, testing, checklist, and issue reference, but it omits affected areas and exact verification commands.

✨ Finishing Touches 💡 1

🛠️ Fix failing CI checks 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}