Skip to content

feat(web): sell Company Brain on Max as well as Scale - #1440

Merged
graphite-app[bot] merged 1 commit into
mainfrom
08-08-feat_web_sell_company_brain_on_max_as_well_as_scale
Aug 10, 2026
Merged

feat(web): sell Company Brain on Max as well as Scale#1440
graphite-app[bot] merged 1 commit into
mainfrom
08-08-feat_web_sell_company_brain_on_max_as_well_as_scale

Conversation

@MaheshtheDev

Copy link
Copy Markdown
Member

Company Brain workspaces could only buy Scale at $399/mo, which is roughly eight times what the median team uses. Adds the $100/mo Max card to the Company Brain plan picker, notes what a Scale trial loses on the way down, and flags that Scale is cheaper above about $400/mo of credits.

Copy link
Copy Markdown
Member Author

How to use the Graphite Merge Queue

Add the label Main to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@MaheshtheDev
MaheshtheDev marked this pull request as ready for review August 8, 2026 08:22
@claude

claude Bot commented Aug 8, 2026

Copy link
Copy Markdown

Claude finished @MaheshtheDev's task in 1m 37s —— View job


PR Review Complete

  • Fetch PR diff and existing review comments
  • Read relevant source files for context
  • Analyze changes for bugs, security issues, and logic errors
  • Submit review findings

Summary: Reviewed the PR changes for bugs and security issues. The code is clean — all issues flagged by previous reviewers have been correctly addressed. The trial button logic fix correctly handles the edge case where Max would appear disabled during Scale trial. No inline comments needed.

Score: 9/10 — Ready to ship.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 8, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  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 59b148e Aug 10 2026, 12:12 AM

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 8, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  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 59b148e Commit Preview URL Aug 10 2026, 12:12 AM

Comment thread apps/web/components/settings/billing.tsx

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 4da6fd9684

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread apps/web/components/settings/billing.tsx
Comment thread apps/web/components/settings/billing.tsx

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overview: Adds the $100/mo Max plan to Company Brain billing options and includes informational text about switching from trial to Max.

Issues found:

Bug: Trial users cannot actually select Max (lines 1491-1497)

The new informational text tells trial users "Moving to Max keeps the agent, shared memory and unlimited seats..." — but the getPlanCardAction function (around line 968) will show a disabled "Included with Scale" button for Max because:

  • During a Scale trial, currentPlan = "scale"
  • isIncludedPlan = PLAN_RANK["scale"] > PLAN_RANK["max"] = 3 > 2 = true
  • This triggers the early return with the disabled "Included with X" button

The Scale card has special handling (line 934) to show "Activate Scale" during trial, bypassing the isIncludedPlan check. Max needs equivalent handling.

Suggested fix: Add trial handling for Max before the isIncludedPlan check (around line 950):

if (isCompanyBrain && isOnTrial && plan.id === "max") {
  return (
    <button
      type="button"
      onClick={() => handleUpgrade("api_max")}
      disabled={disabled}
      className={cn(
        dmSans125ClassName(),
        PLAN_CARD_ACTION_CLASS,
        "bg-[#0054AD] text-[#FAFAFA] hover:bg-[#0B65C9]",
      )}
    >
      {disabled ? <LoaderIcon className="size-4 animate-spin" /> : null}
      Switch to Max
    </button>
  )
}

Score: 6/10

The PR correctly adds the Max plan card and updates the feature lists, but the button logic prevents trial users from actually selecting Max — which contradicts the UI text explaining how to move to Max. This needs fixing before merge.

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overview: Adds the $100/mo Max plan to Company Brain billing options and fixes connector tier gating so GitHub is Scale-only while ensuring higher tiers satisfy lower requirements.

Issues found: None — this looks good to ship.

The critical bug identified by previous reviewers (trial users couldn't select Max due to isIncludedPlan logic) has been correctly fixed at lines 933-952 by adding a special-case handler before the isIncludedPlan check runs. The fix mirrors the existing Scale trial handling.

Other changes verified:

  • Scale features list now includes User Insights (line 236), matching the downgrade copy
  • GitHub connector moved from requiredPlan="max" to requiredPlan="scale" (step-sources.tsx:1195-1196)
  • isLocked function correctly implements tier hierarchy: Scale > Max > Pro (line 376: !(hasMax || hasScale))
  • use-connector-access.ts exports hasScale for the new gating logic

Score: 9/10

Clean fix addressing all review feedback. Minor deduction only because this is the second iteration after reviewers caught the original button-disabled bug.

@graphite-app

graphite-app Bot commented Aug 10, 2026

Copy link
Copy Markdown

Merge activity

Company Brain workspaces could only buy Scale at $399/mo, which is roughly eight times what the median team uses. Adds the $100/mo Max card to the Company Brain plan picker, notes what a Scale trial loses on the way down, and flags that Scale is cheaper above about $400/mo of credits.
@graphite-app
graphite-app Bot force-pushed the 08-08-feat_web_sell_company_brain_on_max_as_well_as_scale branch from ad06686 to 59b148e Compare August 10, 2026 00:10
@graphite-app
graphite-app Bot merged commit 59b148e into main Aug 10, 2026
6 of 8 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.

2 participants