8000 Add ElevenLabs text-to-speech sites template by adityaoberai · Pull Request #10782 · appwrite/appwrite · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@adityaoberai
Copy link
Member

What does this PR do?

Adds ElevenLabs text-to-speech sites template

Test Plan

Locally deploy Appwrite and setup the template

Related PRs and Issues

Checklist

  • Have you read the Contributing Guidelines on issues?
  • If the PR includes a change to an API's metadata (desc, label, params, etc.), does it also include updated API specs and example docs?

HarshMN2345 and others added 14 commits October 2, 2025 17:43
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: stnguyen90 <1477010+stnguyen90@users.noreply.github.com>
Co-authored-by: stnguyen90 <1477010+stnguyen90@users.noreply.github.com>
…github-sites-and-functions

Feat: update facts on GitHub sites and functions
…orkflow

Migrate issue-triage workflow from event-based to scheduled batch processing with duplicate detection
@coderabbitai
Copy link
Contributor
coderabbitai bot commented Nov 7, 2025
📝 Walkthrough

Walkthrough

Adds a public constant AI = 'artificial intelligence' to UseCases and updates app/config/templates/site.php: increases the score for the starter-for-tanstack-start template from 6 to 9, and appends a new template keyed text-to-speech. The new template is a Next.js entry named "Text-to-speech with ElevenLabs" (score 10), references UseCases::AI, defines dark/light screenshot paths, a NEXTJS framework entry with providerRootDirectory: './nextjs/text-to-speech', VCS/provider metadata (github, providerRepositoryId: 'templates-for-sites', providerOwner: 'appwrite', providerVersion: '0.6.*'), and a required password variable ELEVENLABS_API_KEY (placeholder sk_....., value '').

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Single configuration file modified with declarative additions and one constant.
  • No runtime logic or algorithmic changes.

Focus areas:

  • Confirm UseCases::AI naming/value matches existing use-case taxonomy.
  • Verify the text-to-speech template metadata (score, name, tagline, screenshot paths).
  • Validate framework override providerRootDirectory and providerVersion.
  • Confirm ELEVENLABS_API_KEY variable fields (type: password, required, placeholder).

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The pull request title accurately summarizes the main change: adding an ElevenLabs text-to-speech sites template.
Description check ✅ Passed The pull request description is directly related to the changeset, explaining what the PR does and providing test plan and related links.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ 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 add-elevenlabs-site-template

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a797951 and b06354b.

📒 Files selected for processing (1)
  • app/config/templates/site.php (3 hunks)
⏰ 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). (3)
  • GitHub Check: Setup & Build Appwrite Image
  • GitHub Check: Setup & Build Appwrite Image
  • GitHub Check: scan
🔇 Additional comments (3)
app/config/templates/site.php (3)

27-27: LGTM! New AI use case constant properly added.

The new constant follows the existing pattern and is correctly referenced in the text-to-speech template.


974-974: Clarify the score change for TanStack Start template.

The score was changed from 6 to 9, which violates the guideline to "avoid 1,2,3,8,9,10 if possible" and appears unrelated to this PR's objective of adding the ElevenLabs template. Please confirm whether this change is intentional or if it should be reverted.


1447-1474: LGTM! Text-to-speech template is well-configured.

The template configuration is complete and follows the established patterns. All concerns from previous reviews have been addressed:

  • The providerVersion is correctly set to 0.6.*
  • The API key variable uses an empty value for better UX
  • The placeholder correctly shows the sk_ prefix format

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.

@adityaoberai adityaoberai requested a review from Meldiron November 7, 2025 17:10
@github-actions
Copy link
github-actions bot commented Nov 7, 2025

Security Scan Results for PR

Docker Image Scan Results

Package Version Vulnerability Severity
binutils 2.44-r2 CVE-2025-5244 HIGH
binutils 2.44-r2 CVE-2025-5245 HIGH
libxml2 2.13.8-r0 CVE-2025-49794 CRITICAL
libxml2 2.13.8-r0 CVE-2025-49796 CRITICAL
libxml2 2.13.8-r0 CVE-2025-49795 HIGH
libxml2 2.13.8-r0 CVE-2025-6021 HIGH
pcre2 10.43-r1 CVE-2025-58050 CRITICAL
github.com/containerd/containerd/v2 v2.0.2 CVE-2024-25621 HIGH
golang.org/x/crypto v0.31.0 CVE-2025-22869 HIGH
golang.org/x/oauth2 v0.24.0 CVE-2025-22868 HIGH
stdlib 1.22.10 CVE-2025-47907 HIGH
stdlib 1.22.10 CVE-2025-47912 HIGH
stdlib 1.22.10 CVE-2025-58183 HIGH
stdlib 1.22.10 CVE-2025-58186 HIGH
stdlib 1.22.10 CVE-2025-58187 HIGH
stdlib 1.22.10 CVE-2025-58188 HIGH
stdlib 1.22.10 CVE-2025-61724 HIGH

Source Code Scan Results

🎉 No vulnerabilities found!

Copy link
Contributor
@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ebee879 and 469d085.

⛔ Files ignored due to path filters (2)
  • public/images/sites/templates/text-to-speech-dark.png is excluded by !**/*.png
  • public/images/sites/templates/text-to-speech-light.png is excluded by !**/*.png
📒 Files selected for processing (1)
  • app/config/templates/site.php (2 hunks)
⏰ 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). (3)
  • GitHub Check: Setup & Build Appwrite Image
  • GitHub Check: Setup & Build Appwrite Image
  • GitHub Check: scan
🔇 Additional comments (1)
app/config/templates/site.php (1)

22-22: LGTM! New AI use case category added.

The new AI constant is well-defined and follows the existing pattern. This appropriately extends the use case categories for AI-powered templates.

@github-actions
Copy link
github-actions bot commented Nov 7, 2025

✨ Benchmark results

  • Requests per second: 1,108
  • Requests with 200 status code: 199,505
  • P99 latency: 0.175398952

⚡ Benchmark Comparison

Metric This PR Latest version
RPS 1,108 1,142
200 199,505 205,659
P99 0.175398952 0.180275435

@adityaoberai adityaoberai changed the base branch from main to 1.8.x November 10, 2025 14:38
@adityaoberai adityaoberai changed the base branch from 1.8.x to main November 10, 2025 14:43
@adityaoberai adityaoberai changed the base branch from main to 1.8.x November 10, 2025 15:07
@Meldiron Meldiron merged commit 9db6f9c into 1.8.x Nov 10, 2025
41 checks passed
@adityaoberai adityaoberai deleted the add-elevenlabs-site-template branch November 10, 2025 16:03
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.

6 participants

0