fix: make project limit alert program-aware

[HarshMN2345]

What does this PR do?

(Provide a description of what this PR does.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

Summary by CodeRabbit

• Bug Fixes
  • Refined free-plan alert display logic to show under more specific conditions and now dynamically displays the correct project limit for the user's plan instead of a hard-coded value.

[appwrite]

Console (appwrite/console)

Project ID: 688b7bf400350cbd60e9

Sites (1)

Site	Status	Logs	Preview	QR
console-stage 688b7cf6003b1842c9dc	Ready	View Logs	Preview URL

Tip

SSR frameworks are fully supported with configurable build runtimes

[coderabbitai]

Walkthrough

A Svelte component for an organization console page is modified to refine the free-plan alert display logic. An additional guard condition is added that checks if data.program is falsy before rendering the alert, further restricting its visibility. The alert message is updated to dynamically display the project limit from data.currentPlan?.projects rather than a hard-coded value. Alert actions and dismissal behavior remain unchanged. The net change spans 3 additions and 3 removals across the single file.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main change: adding program awareness to the project limit alert logic to determine when it displays.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix-project-limit-alert-not-hidden-for-special-programs

---

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 and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@src/routes/`(console)/organization-[organization]/+page.svelte:
- Around line 222-226: The conditional renders an alert when data.currentPlan
may be undefined because `data.currentPlan?.projects !== 0` is true for
undefined; change the guard to ensure projects is present and non-zero (e.g.,
check `data.currentPlan?.projects` truthiness or explicitly
`data.currentPlan?.projects > 0`) so the Alert.Inline/Tyography.Text never shows
"up to undefined projects"; update the condition in the same expression that
uses isCloud, data.currentPlan?.projects, projectsToArchive.length,
freePlanAlertDismissed and keep dismissFreePlanAlert behavior unchanged.

[coderabbitai]

⚠️ Potential issue | 🟡 Minor

Minor: undefined !== 0 is true — the guard doesn't protect against a missing currentPlan.

If data.currentPlan were ever null/undefined, the check data.currentPlan?.projects !== 0 evaluates to undefined !== 0 → true, so the alert would render with the text "up to undefined projects". This is likely safe in practice (cloud layouts should always load currentPlan), but a small tweak makes it more robust:

Suggested defensive tweak

-    {`#if` isCloud && !data.program && data.currentPlan?.projects !== 0 && projectsToArchive.length === 0 && !freePlanAlertDismissed}
+    {`#if` isCloud && !data.program && data.currentPlan?.projects && projectsToArchive.length === 0 && !freePlanAlertDismissed}

Using truthiness (data.currentPlan?.projects is falsy for both 0 and undefined/null) collapses the two failure modes into one check and avoids rendering "undefined" in the message.

🤖 Prompt for AI Agents

In `@src/routes/`(console)/organization-[organization]/+page.svelte around lines
222 - 226, The conditional renders an alert when data.currentPlan may be
undefined because `data.currentPlan?.projects !== 0` is true for undefined;
change the guard to ensure projects is present and non-zero (e.g., check
`data.currentPlan?.projects` truthiness or explicitly
`data.currentPlan?.projects > 0`) so the Alert.Inline/Tyography.Text never shows
"up to undefined projects"; update the condition in the same expression that
uses isCloud, data.currentPlan?.projects, projectsToArchive.length,
freePlanAlertDismissed and keep dismissFreePlanAlert behavior unchanged.