fix: resolve 1 bugs in MicroAI-Paygate

[saurabhhhcodes]

Description

This PR fixes real bugs found in the codebase:

• Added rejection handler to Promise.all: an unhandled rejection in any input promise previously crashed silently.

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

Note

Catch unhandled promise rejections in wallet-widget.tsx

Adds a .catch handler to the Promise.all chain in wallet-widget.tsx so rejections are logged to console.error with the prefix "Promise.all failed:" instead of remaining unhandled.

^{Macroscope summarized 3164f86.}

Summary by CodeRabbit

• Bug Fixes
  • Improved error handling for wallet-related operations.
  • Added clearer reporting when wallet actions fail, helping issues surface more reliably.

[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 wallet widget adds a standalone .catch handler after isUserRejection. The handler is detached from a promise expression and makes the file syntactically invalid.

Changes

Wallet widget error handling

Layer / File(s)	Summary
Promise rejection handler syntax web/src/components/wallet-widget.tsx	Adds a "Promise.all failed:" error logger through a detached .catch handler after isUserRejection.

Estimated code review effort: 2 (Simple) | ~5 minutes

Possibly related PRs

• AnkanMisra/MicroAI-Paygate#337: Adds the same unattached .catch logging handler in wallet-widget.tsx.
• AnkanMisra/MicroAI-Paygate#361: Modifies wallet-widget.tsx with a .catch handler for Promise.all failures.

Suggested reviewers: ankanmisra

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description explains the bug fix and testing, but it omits the required template sections for affected areas, exact verification commands, screenshots, and reviewer notes.	Use the repository template and add the affected area, exact test commands with results, screenshot applicability, and notes for reviewers.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title identifies a bug fix in MicroAI-Paygate, but it has a minor grammar error and does not identify the affected wallet flow.
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

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 Biome (2.5.5)

web/src/components/wallet-widget.tsx

File contains syntax errors that prevent linting: Line 263: Expected a statement but instead found '.catch(err => console.error("Promise.all failed:", err))'.

---

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

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@web/src/components/wallet-widget.tsx`:
- Around line 262-263: The .catch handler is currently a standalone statement
that breaks TypeScript compilation because it lacks a promise receiver. Move the
.catch(err => console.error("Promise.all failed:", err)) handler to the end of
the Promise.all(...) expression on the preceding line so that it forms a
complete promise chain.

🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: e7be84ae-ceb3-462b-b7e0-d4480211098e

📥 Commits

Reviewing files that changed from the base of the PR and between e8481a3 and 3164f86.

📒 Files selected for processing (1)

• web/src/components/wallet-widget.tsx

[coderabbitai]

🎯 Functional Correctness | 🔴 Critical | ⚡ Quick win

Attach .catch(...) to the Promise.all(...) chain.

Line 263 is a standalone member-access expression. TypeScript cannot parse it without a promise receiver, so web/src/components/wallet-widget.tsx fails to compile. Move this handler to the end of the existing Promise.all(...) expression. This currently blocks WalletWidget and the shared navigation from rendering.

🧰 Tools

🪛 Biome (2.5.5)

[error] 263-263: Expected a statement but instead found '.catch(err => console.error("Promise.all failed:", err))'.

(parse)

🪛 GitHub Actions: Web Lint & Build / 0_web-lint-build.txt

[error] 263-263: ESLint parsing error: Declaration or statement expected. The 'bun run lint' command failed with exit code 1.

🪛 GitHub Actions: Web Lint & Build / web-lint-build

[error] 263-263: ESLint parsing error: Declaration or statement expected. The 'bun run lint' command failed with exit code 1.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@web/src/components/wallet-widget.tsx` around lines 262 - 263, The .catch
handler is currently a standalone statement that breaks TypeScript compilation
because it lacks a promise receiver. Move the .catch(err =>
console.error("Promise.all failed:", err)) handler to the end of the
Promise.all(...) expression on the preceding line so that it forms a complete
promise chain.

Source: Linters/SAST tools