Remove redundant credits section

[ved015]

Replaced add more credtis with buy credits

Modal colour scheme matches rest of the modals

Primary button and secondary button colour scheme matched

Primary button and secondary button colour scheme matched

[cloudflare-workers-and-pages]

Deploying with    Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status	Name	Latest Commit	Updated (UTC)
✅ Deployment successful! View logs	supermemory-mcp	6268353	May 29 2026, 02:50 PM

[claude]

Claude finished @ved015's task in 55s —— View job

---

• Gather context and review the PR changes
• Verify the previous bug fix is still in place
• Analyze the diff for any new issues
• Submit review

Review Complete

Overview: This PR removes redundant usage display from the Credits section and simplifies the dialog structure.

Changes:

• Removed duplicate "Usage this period" display from Credits section (already shown in Billing & Subscription section)
• Moved the credits Dialog component outside of nested structure for cleaner organization
• Simplified Credits card to show only top-up balance and purchase button
• Removed "(optional)" label from "Top-up credits" heading
• Updated modal styling to match other dialogs in the app
• Refactored PillButton variants (danger → ghost) in org-context

Previous issue status: The debug artifact true || hasPaidPlan from the previous review has been fixed (commit 582009a). The Credits section now correctly uses hasPaidPlan at line 940.

Issues found: None. The refactoring is clean and the bug has been addressed.

Score: 9/10 — Clean refactoring that reduces redundancy and improves code organization. Ready to merge.

---

[cloudflare-workers-and-pages]

Deploying with    Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status	Name	Latest Commit	Preview URL	Updated (UTC)
✅ Deployment successful! View logs	supermemory-app	6268353	Commit Preview URL Branch Preview URL	May 29 2026, 02:51 PM

[claude]

Overview: Removes the redundant usage display from the Credits section and simplifies the credits dialog structure.

Issues found:

• Line 936: Debug artifact true || hasPaidPlan will always show the Credits section to all users regardless of subscription status. Should be just hasPaidPlan.

Score: 7/10

The refactoring looks clean, but the debug artifact must be removed before merge — it would expose the credits section to free-tier users who shouldn't see it.

[ved015]

@claude pls review

[claude]

Claude finished @ved015's task in 44s —— View job

---

• Gather context and review the PR changes
• Check if the previously identified bug was fixed
• Analyze the diff for any remaining issues
• Submit review

Review Complete

Overview: This PR removes redundant usage display from the Credits section and simplifies the dialog structure.

Changes:

• Removed duplicate "Usage this period" display from Credits section (already shown in Billing & Subscription section)
• Moved the credits Dialog component outside of nested structure for cleaner organization
• Simplified Credits card to show only top-up balance and purchase button
• Removed "(optional)" label from "Top-up credits" heading

Previous issue status: The debug artifact true || hasPaidPlan from the previous review has been fixed (commit 582009a). The Credits section now correctly uses hasPaidPlan at line 917.

Issues found: None. The refactoring is clean and the bug has been addressed.

Score: 9/10 — Clean refactoring that reduces redundancy and improves code organization. Ready to merge.

---

[claude]

Overview: This PR removes redundant usage display from the Credits section, restructures the Dialog component, and updates button styling.

Issues found:

• Line 940: The debug artifact true || hasPaidPlan was fixed in commit 582009a but reintroduced in the latest commit 49ea8f9. This will show the Credits section to all users regardless of their subscription status.

Score: 7/10

The refactoring is clean otherwise, but the debug artifact must be removed before merge.

[ved015]

@claude pls review the pr

[claude]

Claude finished @ved015's task in 48s —— View job

---

• Gather context and review the PR changes
• Verify the true || hasPaidPlan bug fix is in place
• Analyze the diff for code quality and issues
• Submit review

Review Complete

Overview: This PR removes redundant usage display from the Credits section, restructures the dialog component, and updates styling for consistency.

Changes:

• Removed duplicate "Usage this period" display from Credits section (already shown in Billing & Subscription)
• Moved the Credits Dialog component outside the nested section structure for cleaner organization
• Simplified Credits card to show only top-up balance and purchase button
• Removed "(optional)" label from "Top-up credits" heading
• Updated modal styling to match other dialogs (SURFACE_SHADOW, rounded corners, colors)
• Refactored PillButton variants in org-context (danger → ghost) for consistency

Bug fix verified: The debug artifact true || hasPaidPlan has been correctly removed. Line 921 now shows {hasPaidPlan ? ( which properly gates the Credits section to paid plan users only.

Issues found: None

Score: 9/10 — Clean refactoring that reduces redundancy and improves code organization. The previous bug has been addressed. Ready to merge.

---

View job

[MaheshtheDev]

LGTM

.