Skip to content

fix: native app build and change app name#1911

Merged
kushagrasarathe merged 1 commit intodevfrom
fix/native-build
Apr 29, 2026
Merged

fix: native app build and change app name#1911
kushagrasarathe merged 1 commit intodevfrom
fix/native-build

Conversation

@kushagrasarathe
Copy link
Copy Markdown
Contributor

No description provided.

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 29, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
peanut-wallet Building Building Preview, Comment Apr 29, 2026 10:53am

Request Review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 29, 2026

Caution

Review failed

Pull request was closed or merged during review

Walkthrough

The pull request updates the Android application identifier from me.peanut.app to me.peanut.wallet across configuration files, resets versioning to 1.0.0, aligns Java and resource package namespaces, updates Android App Links data with a new certificate fingerprint, and refactors the onramp quote API call to use authentication headers instead of direct environment variable access.

Changes

Cohort / File(s) Summary
App Identifier & Version Updates
android/app/build.gradle
Updates namespace and applicationId from me.peanut.app to me.peanut.wallet; changes version from code 10 / name 1.0.9 to code 1 / name 1.0.0.
Package Namespace Alignment
android/app/src/main/java/me/peanut/wallet/MainActivity.java, android/app/src/main/res/values/strings.xml, capacitor.config.ts
Updates package references and app identifiers from me.peanut.app to me.peanut.wallet across Java package declaration, string resources, and Capacitor configuration.
Android App Links Configuration
public/.well-known/assetlinks.json
Updates Android App Links declaration with new package name me.peanut.wallet and replaces one SHA-256 certificate fingerprint while retaining others.
API Authentication Refactoring
src/app/actions/onramp-quote.ts
Removes 'use server' directive, switches from hardcoded API key header to dynamic authentication headers via getAuthHeaders(), and eliminates pre-flight configuration validation for PEANUT_API_KEY.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No pull request description was provided by the author, making it impossible to assess whether the description relates to the changeset. Add a pull request description that explains the changes, such as the app package renaming from 'me.peanut.app' to 'me.peanut.wallet' and version/build configuration updates.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title 'fix: native app build and change app name' directly describes the main changes: fixing the native app build configuration and rebranding the app identifier/name from 'me.peanut.app' to 'me.peanut.wallet'.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch

Review rate limit: 4/5 reviews remaining, refill in 12 minutes.

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown
Contributor

Code-analysis diff

Painscore total: 5542.9 → 5542.94 (+0.04)
Findings: 0 net (+1 new, -1 resolved)

🆕 New findings (1)

  • medium complexity — src/app/actions/onramp-quote.ts — CC 6, MI 57.7, SLOC 27

✅ Resolved (1)

  • src/app/actions/onramp-quote.ts — CC 8, MI 55.16, SLOC 33

@github-actions
Copy link
Copy Markdown
Contributor

🧪 UI test report — ✅ all green

Suites

  • unit: 932 ran, 0 failed, 0 skipped, 16.3s

📊 Coverage (unit)

metric %
statements 43.0%
branches 22.4%
functions 22.4%
lines 42.7%
⏱ 10 slowest test cases
time test
0.3s src/app/actions/__tests__/api-headers.test.ts › should include Content-Type in updateUserById
0.3s src/app/actions/__tests__/api-headers-extended.test.ts › should not include apiKey in updateUserById body
0.1s src/app/(mobile-ui)/qr-pay/__tests__/qr-pay-states.test.tsx › SimpleFi WebSocket waiting shows processing indicator
0.1s src/app/(mobile-ui)/qr-pay/__tests__/qr-pay-states.test.tsx › Manteca PIX form ready shows merchant card + amount input + pay button
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle valid 9-digit US account
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle valid UK IBAN with spaces
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle too long for US account
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle minimum length (6 digits) US account
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle valid ETH address in lowercase
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle invalid ETH address (missing 0x prefix)
📍 Inline annotations are in the **Unit test report** check above. Coverage artifact: `coverage-unit`. Generated by `.github/workflows/tests.yml`.

@kushagrasarathe kushagrasarathe merged commit 3c57e56 into dev Apr 29, 2026
10 of 12 checks passed
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