Release/1.0 - WIP #242

robertqin86 · 2025-04-10T16:17:31Z

This will be the release cut from beta that contains all the latest changes that we want to release to production.

TODO: Add changelog

Summary by CodeRabbit

New Features
- Introduced a secure account deletion flow with confirmation prompts.
- Added a funding wallet selection interface for smoother transaction management.
- Enhanced wallet displays with improved feedback and support for optional transaction memos.
- Upgraded saved prompts with advanced filtering and pagination.
- Updated the application version to 1.0.0 with new STABLE branding.

…error fix: resolve corepack signature issue (#221)

…e-code-debug feat: support server side code debugging

…e-code-debug-vscode feat: add launch.json to enable debug mode on vscode

…-empty-accounts feat: allow users delete empty accounts

…epage Feat/add prompt filtering on homepage

…lets

…t funding modal

…ana and embedded wallets to funding wallets inteface

…nected-wallets feat: enable privy connected wallets to pay EAP

vercel · 2025-04-10T16:17:37Z

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
neur-app	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 10, 2025 4:21pm

coderabbitai · 2025-04-10T16:17:41Z

Walkthrough

This pull request introduces multiple configuration updates and new features. It adds entries and files to define the development environment (e.g., .gitignore, .nvmrc, VSCode configurations, Docker settings, and local development instructions). Furthermore, it enhances the application's functionality by implementing new components for account deletion, funding wallet selection, and wallet card displays. Backend changes include the addition of a deleteUser function and improvements in wallet transfer logic. Additional updates include UI tweaks, new hooks for wallet balance management, and expanded constants and environment configurations.

Changes

Files	Change Summary
`.gitignore`, `.nvmrc`, `.vscode/launch.json`, `.vscode/settings.json`, `Dockerfile.local`, `docker-compose.yml`, `LOCAL_DEV.md`, `package.json`	• Configuration updates for development and debugging • Added `.idea` ignore, Node version file, VSCode debug configs, disabled format on save, Corepack installation command, updated docker port, local dev instructions, version bump, new scripts and dependencies in package.json
`src/app/(user)/account/account-content.tsx`, `src/app/(user)/account/delete-account-dialog.tsx`, `src/app/(user)/home/components/select-funding-wallet.tsx`, `src/app/(user)/home/data/funding-wallets.ts`, `src/app/(user)/home/home-content.tsx`	• Enhanced account management and wallet interactions • Integrated account deletion with new states and a confirmation dialog • Introduced funding wallet selection component and related data models • Updated filtering logic and purchase handling in home content
`src/app/(user)/saved-prompts/components/filter-dropdown.tsx`, `src/app/(user)/saved-prompts/page.tsx`, `src/components/dashboard/app-sidebar.tsx`	• UI improvements in saved prompts and dashboard • Added optional `displayAtHomePage` prop to filter dropdown • Centralized filter options via import • Updated sidebar label from "BETA" to "STABLE"
`src/components/dashboard/wallet-card-eap.tsx`, `src/components/dashboard/wallet-card.tsx`, `src/components/provider-auth.tsx`, `src/components/transfer-dialog.tsx`, `src/components/ui/checkbox.tsx`	• New and modified UI components for wallet display and interactions • Introduced `WalletCardEap` for funding wallet info • Added `disableFund` property for controlled funding • Added dynamic provider configuration and a reusable Checkbox component • Minor formatting changes
`src/hooks/use-wallets.ts`, `src/lib/constants.ts`, `src/lib/solana/index.ts`	• Added new helper functions to fetch embedded wallet balances and check for non-empty wallets • Introduced new constants for filter options, EAP price, Solana cluster configuration, and Phantom wallet select • Updated Solana lib with enhanced wallet type handling and transfer signature improvements
`src/server/actions/ai.ts`, `src/server/actions/user.ts`	• Updated SolanaAgentKit configuration with an optional `HELIUS_RPC_URL` and added support for a memo in transfers • Added a new asynchronous `deleteUser` function including necessary validations, transaction management, and error handling

Sequence Diagram(s)

sequenceDiagram
    participant U as User
    participant AC as AccountContent
    participant DAD as DeleteAccountDialog
    participant S as Server (deleteUser)

    U->>AC: Click "Delete Account"
    AC->>DAD: Show deletion confirmation dialog
    U->>DAD: Confirm deletion action
    DAD->>AC: Trigger delete account process
    AC->>S: Call deleteUser API
    S-->>AC: Return success or error response
    AC->>U: Notify outcome and perform logout on success

Possibly related PRs

feat: allow users delete empty accounts #239: Focuses on enabling users to delete empty accounts, which directly relates to the new deleteUser functionality and account deletion flow introduced in this PR.

Suggested labels

enhancement

Suggested reviewers

slimeonmyhead

Poem

Oh, what a hop in our code today,
Where bugs are chased and glitches stray.
I, a clever rabbit, nibble through each line,
With deletion and wallets now looking so fine.
🐰 Cheers to new features in every array!
Leaping through changes in a joyful display.

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.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

src/app/(user)/account/account-content.tsx

Oops! Something went wrong! :(

ESLint: 9.18.0

ESLint couldn't find the plugin "eslint-plugin-react-hooks".

(The package "eslint-plugin-react-hooks" was not found when loaded as a Node module from the directory "".)

It's likely that the plugin isn't installed correctly. Try reinstalling by running the following:

npm install eslint-plugin-react-hooks@latest --save-dev

The plugin "eslint-plugin-react-hooks" was referenced from the config file in " » eslint-config-next/core-web-vitals » /node_modules/.pnpm/[email protected][email protected][email protected][email protected]/node_modules/eslint-config-next/index.js".

If you still can't figure out the problem, please see https://eslint.org/docs/latest/use/troubleshooting.

src/app/(user)/account/delete-account-dialog.tsx

Oops! Something went wrong! :(

ESLint: 9.18.0

ESLint couldn't find the plugin "eslint-plugin-react-hooks".

(The package "eslint-plugin-react-hooks" was not found when loaded as a Node module from the directory "".)

It's likely that the plugin isn't installed correctly. Try reinstalling by running the following:

npm install eslint-plugin-react-hooks@latest --save-dev

The plugin "eslint-plugin-react-hooks" was referenced from the config file in " » eslint-config-next/core-web-vitals » /node_modules/.pnpm/[email protected][email protected][email protected][email protected]/node_modules/eslint-config-next/index.js".

If you still can't figure out the problem, please see https://eslint.org/docs/latest/use/troubleshooting.

src/app/(user)/home/home-content.tsx

Oops! Something went wrong! :(

ESLint: 9.18.0

ESLint couldn't find the plugin "eslint-plugin-react-hooks".

(The package "eslint-plugin-react-hooks" was not found when loaded as a Node module from the directory "".)

It's likely that the plugin isn't installed correctly. Try reinstalling by running the following:

npm install eslint-plugin-react-hooks@latest --save-dev

The plugin "eslint-plugin-react-hooks" was referenced from the config file in " » eslint-config-next/core-web-vitals » /node_modules/.pnpm/[email protected][email protected][email protected][email protected]/node_modules/eslint-config-next/index.js".

If you still can't figure out the problem, please see https://eslint.org/docs/latest/use/troubleshooting.

15 others

✨ Finishing Touches

📝 Generate Docstrings

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

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

Review comments: Directly reply to a review comment made by CodeRabbit. Example:
- I pushed a fix in commit <commit_id>, please review it.
- Generate unit testing code for this file.
- Open a follow-up GitHub issue for this discussion.
Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
- @coderabbitai generate unit testing code for this file.
- @coderabbitai modularize this function.
PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
- @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
- @coderabbitai read src/utils.ts and generate unit testing code.
- @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
- @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

@coderabbitai pause to pause the reviews on a PR.
@coderabbitai resume to resume the paused reviews.
@coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
@coderabbitai full review to do a full review from scratch and review all the files again.
@coderabbitai summary to regenerate the summary of the PR.
@coderabbitai generate docstrings to generate docstrings for this PR.
@coderabbitai resolve resolve all the CodeRabbit review comments.
@coderabbitai plan to trigger planning for file edits and PR creation.
@coderabbitai configuration to show the current CodeRabbit configuration for the repository.
@coderabbitai help to get help.

Other keywords and placeholders

Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (`.coderabbit.yaml`)

You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
Please see the configuration documentation for more information.
If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

Visit our Documentation for detailed information on how to use CodeRabbit.
Join our Discord Community to get help, request features, and share feedback.
Follow us on X/Twitter for updates and announcements.

coderabbitai

Actionable comments posted: 4

🔭 Outside diff range comments (1)

src/app/(user)/account/account-content.tsx (1)
92-105: 🛠️ Refactor suggestion

Potential one-way logic in balance check.
When hasBalance is false, isEmptyAccount is set to true. However, if the user later acquires a balance, this code does not unset isEmptyAccount. Consider updating to:
- if (!hasBalance) {
-   setIsEmptyAccount(true);
- }
+ setIsEmptyAccount(!hasBalance);
to ensure the flag remains correct as wallet balances change.

🧹 Nitpick comments (11)

src/components/dashboard/app-sidebar.tsx (1)

37-40: Potential Logic Mismatch in Beta Status Display.
The sidebar header now displays "STABLE" when IS_BETA is truthy. Typically, one might expect a beta indicator (e.g., "BETA") when the app is in beta mode. If this change is a deliberate part of the release branding, consider adding an inline comment to clarify the intent and avoid confusion.
docker-compose.yml (1)
63-63: Port mapping change is appropriately documented.

Changing the host port from 5432 to 5433 is a good practice to avoid conflicts with local PostgreSQL instances. The comment clearly explains the rationale.

However, there's a trailing space at the end of the comment line that should be removed:
-      - "5433:5432" # Map container port to a different host port to prevent conflicts with local postgres instance 
+      - "5433:5432" # Map container port to a different host port to prevent conflicts with local postgres instance
🧰 Tools

🪛 YAMLlint (1.35.1)

[error] 63-63: trailing spaces

(trailing-spaces)
local_app_dev.sh (1)
1-12: New development script properly configures the PostgreSQL connection.

The script correctly sets up environment variables for the database connection using port 5433, which is consistent with the docker-compose.yml change. The default credentials are provided but can be overridden through the .env file, which is good for developer flexibility.

Consider adding basic error handling and file permission instructions:
 #!/bin/bash

+# Ensure PostgreSQL is running before starting the application
+if ! nc -z localhost 5433 &>/dev/null; then
+  echo "Warning: PostgreSQL doesn't appear to be running on port 5433"
+  echo "Make sure to start Docker containers with: docker-compose up -d"
+fi
+
 # change the default POSTGRES_USER and POSTGRES_PASSWORD 
 # to the actual value defined in your .env
 export POSTGRES_USER=${POSTGRES_USER:-"admin"}
 export POSTGRES_PASSWORD=${POSTGRES_PASSWORD:-"admin"}

 DB_URL="postgres://${POSTGRES_USER}:${POSTGRES_PASSWORD}@localhost:5433/neurdb"
 export DATABASE_URL=$DB_URL
 export DIRECT_URL=$DB_URL

+# Ensure the dev:local script exists
+if ! pnpm run --list | grep -q "dev:local"; then
+  echo "Error: dev:local script not found in package.json"
+  exit 1
+fi
+
 pnpm run dev:local
Don't forget to make the script executable with:
chmod +x local_app_dev.sh
LOCAL_DEV.md (1)
93-110: Add language specifiers to the code blocks.

The markdown fenced code blocks are missing language specifiers, which affects syntax highlighting and is flagged by markdownlint.
-```
+```bash
chmod u+x local_app_dev.sh
- +bash
./local_app_dev.sh
The debugging instructions are otherwise clear and helpful for developers who need to debug server-side code.

🧰 Tools

🪛 markdownlint-cli2 (0.17.2)

97-97: Fenced code blocks should have a language specified
null

(MD040, fenced-code-language)

103-103: Fenced code blocks should have a language specified
null

(MD040, fenced-code-language)
src/lib/constants.ts (1)
39-39: Consider adding documentation for EAP_PRICE.

While the constant is straightforward, adding a comment explaining what EAP stands for (Early Access Program) and the purpose of this constant would improve code clarity.
-export const EAP_PRICE = 1.0;
+/** Price in SOL for Early Access Program participation */
+export const EAP_PRICE = 1.0;
src/app/(user)/account/delete-account-dialog.tsx (2)
16-21: Fix typo in interface property name.

There's a spelling error in the property name "eligibililty" (extra 'l').
interface DeletePromptDialogProps {
  displayPrompt: boolean;
- eligibililty: boolean;
+ eligibility: boolean;
  onConfirm: () => void;
  onCancel: () => void;
}
23-28: Fix typo in component prop name.

Ensure the prop name is corrected in the component parameter destructuring as well.
export const DeleteAccountDialog = ({
  displayPrompt,
- eligibililty,
+ eligibility,
  onCancel,
  onConfirm,
}: DeletePromptDialogProps) => {
src/hooks/use-wallets.ts (1)
10-27: Consider adding error logging.
Inside the catch block at lines 24-26, you return undefined without logging the error. Logging could help with troubleshooting.
   } catch (error) {
+    console.error("Failed to retrieve embedded wallet balance:", error);
     return undefined;
   }
src/components/dashboard/wallet-card-eap.tsx (1)

1-171: Overall functionality looks solid.
The wallet card’s logic for funding and paying EAP is well structured. Consider removing or clarifying the commented-out code at line 72 if it’s no longer needed.
src/lib/solana/index.ts (1)
132-132: Avoid using this inside static methods.
Using this in a static context can be confusing. Prefer referencing the class name (SolanaUtils) directly for clarity.
- if (this.isConnectedSolanaWallet(wallet)) {
+ if (SolanaUtils.isConnectedSolanaWallet(wallet)) {

- const balance = await this.connection.getBalance(fromPubkey);
+ const balance = await SolanaUtils.connection.getBalance(fromPubkey);

- const { blockhash } = await this.connection.getLatestBlockhash('confirmed');
+ const { blockhash } = await SolanaUtils.connection.getLatestBlockhash('confirmed');

- const signature = await this.connection.sendRawTransaction(signedTransaction.serialize(), ...);
+ const signature = await SolanaUtils.connection.sendRawTransaction(signedTransaction.serialize(), ...);

- const balance = await this.connection.getBalance(fromPubkey);
+ const balance = await SolanaUtils.connection.getBalance(fromPubkey);
Also applies to: 145-145, 177-177, 185-185, 223-223

🧰 Tools

🪛 Biome (1.9.4)

[error] 132-132: Using this in a static context can be confusing.

this refers to the class.
Unsafe fix: Use the class name instead.

(lint/complexity/noThisInStatic)
src/app/(user)/account/account-content.tsx (1)
316-327: Avatar and user info rendering.
Displays user’s Twitter profile picture if available; fallback logic is correct. Optional: include an alt attribute for accessibility on the AvatarImage.
<AvatarImage
  src={userData.twitter?.profilePictureUrl || undefined}
+ alt="User avatar"
  className="rounded-lg object-cover"
/>

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 4b9dfba and 4b95518.

⛔ Files ignored due to path filters (1)

pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml

📒 Files selected for processing (27)

.gitignore (1 hunks)
.nvmrc (1 hunks)
.vscode/launch.json (1 hunks)
.vscode/settings.json (1 hunks)
Dockerfile.local (1 hunks)
LOCAL_DEV.md (2 hunks)
docker-compose.yml (1 hunks)
local_app_dev.sh (1 hunks)
package.json (3 hunks)
src/app/(user)/account/account-content.tsx (11 hunks)
src/app/(user)/account/delete-account-dialog.tsx (1 hunks)
src/app/(user)/home/components/select-funding-wallet.tsx (1 hunks)
src/app/(user)/home/data/funding-wallets.ts (1 hunks)
src/app/(user)/home/home-content.tsx (10 hunks)
src/app/(user)/saved-prompts/components/filter-dropdown.tsx (3 hunks)
src/app/(user)/saved-prompts/page.tsx (2 hunks)
src/components/dashboard/app-sidebar.tsx (1 hunks)
src/components/dashboard/wallet-card-eap.tsx (1 hunks)
src/components/dashboard/wallet-card.tsx (4 hunks)
src/components/provider-auth.tsx (2 hunks)
src/components/transfer-dialog.tsx (1 hunks)
src/components/ui/checkbox.tsx (1 hunks)
src/hooks/use-wallets.ts (2 hunks)
src/lib/constants.ts (2 hunks)
src/lib/solana/index.ts (5 hunks)
src/server/actions/ai.ts (3 hunks)
src/server/actions/user.ts (1 hunks)

🧰 Additional context used

🧬 Code Graph Analysis (10)

src/components/dashboard/wallet-card.tsx (2)

src/types/db.ts (1)

EmbeddedWallet (12-22)

src/lib/constants.ts (1)

solanaCluster (43-45)

src/app/(user)/home/components/select-funding-wallet.tsx (8)

src/types/db.ts (1)

EmbeddedWallet (12-22)

src/app/(user)/home/data/funding-wallets.ts (3)

FundingWallet (7-12)

SolanaConnectedFundingWallet (31-41)

EmbeddedFundingWallet (17-29)

src/hooks/use-wallets.ts (1)

useEmbeddedWallets (29-39)

src/hooks/use-user.ts (1)

useUser (122-230)

src/components/ui/dialog.tsx (5)

Dialog (113-113)

DialogContent (118-118)

DialogHeader (119-119)

DialogTitle (121-121)

DialogDescription (122-122)

src/lib/utils.ts (1)

cn (17-19)

src/components/dashboard/wallet-card-eap.tsx (1)

WalletCardEap (29-171)

src/lib/constants.ts (1)

solanaCluster (43-45)

src/app/(user)/home/data/funding-wallets.ts (3)

src/types/helius/fungibleToken.ts (1)

FungibleToken (8-23)

src/types/helius/nonFungibleToken.ts (1)

NonFungibleToken (5-20)

src/types/db.ts (1)

EmbeddedWallet (12-22)

src/app/(user)/account/delete-account-dialog.tsx (4)

src/components/ui/dialog.tsx (5)

Dialog (113-113)

DialogContent (118-118)

DialogHeader (119-119)

DialogTitle (121-121)

DialogFooter (120-120)

src/components/ui/checkbox.tsx (1)

Checkbox (39-39)

src/components/ui/label.tsx (1)

Label (27-27)

src/components/ui/button.tsx (1)

Button (58-58)

src/components/provider-auth.tsx (1)

src/lib/constants.ts (2)

solanaCluster (43-45)

RPC_URL (9-11)

src/hooks/use-wallets.ts (3)

src/types/db.ts (1)

EmbeddedWallet (12-22)

src/lib/solana/helius.ts (1)

searchWalletAssets (97-270)

src/types/helius/portfolio.ts (1)

SOL_MINT (4-4)

src/components/ui/checkbox.tsx (1)

src/lib/utils.ts (1)

cn (17-19)

src/app/(user)/account/account-content.tsx (3)

src/hooks/use-wallets.ts (1)

hasWalletWithBalance (41-46)

src/app/(user)/account/delete-account-dialog.tsx (1)

DeleteAccountDialog (23-109)

src/server/actions/user.ts (1)

deleteUser (319-363)

src/lib/constants.ts (1)

src/app/(user)/saved-prompts/types/prompt.ts (1)

FilterOption (1-4)

src/components/dashboard/wallet-card-eap.tsx (7)

src/app/(user)/home/data/funding-wallets.ts (1)

FundingWallet (7-12)

src/lib/solana/helius.ts (1)

searchWalletAssets (97-270)

src/types/helius/portfolio.ts (1)

SOL_MINT (4-4)

src/components/ui/copyable-text.tsx (1)

CopyableText (17-68)

src/components/ui/skeleton.tsx (1)

Skeleton (15-15)

src/lib/constants.ts (1)

EAP_PRICE (39-39)

src/components/ui/button.tsx (1)

Button (58-58)

🪛 markdownlint-cli2 (0.17.2)

LOCAL_DEV.md

97-97: Fenced code blocks should have a language specified
null

(MD040, fenced-code-language)

103-103: Fenced code blocks should have a language specified
null

(MD040, fenced-code-language)

🪛 Biome (1.9.4)

src/lib/solana/index.ts

[error] 132-132: Using this in a static context can be confusing.