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: #395

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 8b85bdf.}

Summary by CodeRabbit

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

[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]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: d6e121b6-2fcb-4e06-90aa-90250f032037

📥 Commits

Reviewing files that changed from the base of the PR and between e8481a3 and 8b85bdf.

📒 Files selected for processing (3)

• sdk/typescript/src/client.ts
• sdk/typescript/src/protocol/microai.ts
• web/src/lib/errors.ts

---

📝 Walkthrough

Walkthrough

Three early returns now use implicit undefined returns in request serialization, payer lookup, and error detail sanitization. Runtime behavior remains unchanged.

Changes

Return syntax updates

Layer / File(s)	Summary
Equivalent early-return simplification sdk/typescript/src/client.ts, sdk/typescript/src/protocol/microai.ts, web/src/lib/errors.ts	Replaces explicit undefined returns with bare returns in three early-exit paths. Behavior remains unchanged.

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

Possibly related PRs

• AnkanMisra/MicroAI-Paygate#340: Makes the same three return changes.
• AnkanMisra/MicroAI-Paygate#350: Modifies the same three functions to use implicit undefined returns.
• AnkanMisra/MicroAI-Paygate#372: Makes the same changes to the same three functions.

Suggested reviewers: ankanmisra

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description summarizes the change and type, but it omits required template sections and exact verification commands with results.	Use the repository template headings, identify affected areas, complete the contributor checklist, and list exact test commands with their results.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly identifies the primary change as fixing three MicroAI-Paygate bugs.
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.

✨ 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.}