Skip to content

Conversation

@Meldiron
Copy link
Contributor

@Meldiron Meldiron commented Oct 21, 2025

Reverts #2477

Summary by CodeRabbit

  • Removed Features

    • Removed the LLM prompt starter kit banner from platform creation flows across Android, Apple, Flutter, React Native, and Web. The "Clone starter" section is now streamlined.
  • Chores

    • Removed associated analytics tracking for prompt starter kit copy interactions.

@railway-app
Copy link

railway-app bot commented Oct 21, 2025

This PR was not deployed automatically as @Meldiron does not have access to the Railway project.

In order to get automatic PR deploys, please add @Meldiron to your workspace on Railway.

@appwrite
Copy link

appwrite bot commented Oct 21, 2025

Console

Project ID: 688b7bf400350cbd60e9

Sites (2)
Site Status Logs Preview QR
 console-stage
688b7cf6003b1842c9dc
Ready Ready View Logs Preview URL QR Code
 console-cloud
688b7c18002b9b871a8f
Ready Ready View Logs Preview URL QR Code

Tip

Appwrite has a Discord community with over 16 000 members.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 21, 2025

Walkthrough

This pull request removes the LLM banner feature from the platform creation workflow. The changes include deleting the LlmBanner component file, removing its imports and usages from all platform creation files (Android, Apple, Flutter, React Native, Web), deleting related prompt generation and configuration utilities from the store module, and removing the associated analytics event for copying the starter kit prompt.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

The changes follow a consistent removal pattern: LlmBanner component deletion across multiple platform files (Android, Apple, Flutter, React Native, Web), store export removals (LLMPromptConfig, generatePromptFromConfig, PlatformConfig, platformConfigs, buildPlatformConfig), and analytics enum entry removal. While spread across 8 files, the homogeneous nature of these deletions—removing the same component and related utilities—reduces cognitive load. Review effort focuses on verifying complete removal with no orphaned references and confirming that deleted store exports and analytics events are not referenced elsewhere in the codebase.

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The pull request title "Revert 'copy prompt'" directly and accurately describes the main objective of the changeset. The PR explicitly reverts a previous feature implementation (#2477) by removing the LlmBanner component, related analytics tracking, prompt generation utilities, and supporting types across multiple files. The title is concise, specific enough to convey the primary change (reverting the copy prompt feature), and avoids vague or misleading language. A teammate reviewing the commit history would clearly understand that this PR removes a previously added copy prompt capability.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch revert-2477-copy-prompt

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b49f884 and 5649842.

📒 Files selected for processing (8)
  • src/lib/actions/analytics.ts (1 hunks)
  • src/routes/(console)/project-[region]-[project]/overview/platforms/createAndroid.svelte (0 hunks)
  • src/routes/(console)/project-[region]-[project]/overview/platforms/createApple.svelte (0 hunks)
  • src/routes/(console)/project-[region]-[project]/overview/platforms/createFlutter.svelte (0 hunks)
  • src/routes/(console)/project-[region]-[project]/overview/platforms/createReactNative.svelte (0 hunks)
  • src/routes/(console)/project-[region]-[project]/overview/platforms/createWeb.svelte (1 hunks)
  • src/routes/(console)/project-[region]-[project]/overview/platforms/llmBanner.svelte (0 hunks)
  • src/routes/(console)/project-[region]-[project]/overview/platforms/store.ts (0 hunks)
💤 Files with no reviewable changes (6)
  • src/routes/(console)/project-[region]-[project]/overview/platforms/store.ts
  • src/routes/(console)/project-[region]-[project]/overview/platforms/createApple.svelte
  • src/routes/(console)/project-[region]-[project]/overview/platforms/createFlutter.svelte
  • src/routes/(console)/project-[region]-[project]/overview/platforms/createReactNative.svelte
  • src/routes/(console)/project-[region]-[project]/overview/platforms/llmBanner.svelte
  • src/routes/(console)/project-[region]-[project]/overview/platforms/createAndroid.svelte
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: build
🔇 Additional comments (2)
src/lib/actions/analytics.ts (1)

198-198: No dangling references found—removal is clean.

Comprehensive search across the codebase confirms no remaining references to CopyPromptStarterKitClick or click_copy_prompt_starter_kit. The removal is consistent with reverting the LLM banner feature, and the formatting adjustment removing the trailing comma from WebsiteOpenClick is appropriate as it's now the last enum member.

src/routes/(console)/project-[region]-[project]/overview/platforms/createWeb.svelte (1)

49-49: Verification complete: cleanup is clean and consistent across all platform files.

The removal of LLMPromptConfig and LlmBanner has been successfully completed:

  • No LLMPromptConfig references found anywhere in the codebase
  • No LlmBanner references found anywhere in the codebase
  • All platform creation files (createWeb, createFlutter, createApple, createAndroid, createReactNative) maintain consistent import patterns from ./store
  • store.ts no longer exports the removed functionality

The file at line 49 of createWeb.svelte correctly imports only the necessary types and functions, consistent with other platform files.


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

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

@Meldiron Meldiron merged commit 4171e0d into main Oct 21, 2025
5 checks passed
@Meldiron Meldiron deleted the revert-2477-copy-prompt branch October 21, 2025 15:44
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