Repository navigation
feat: blog post cold start improvements - #2811
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
WalkthroughAdds two new Markdoc pages under src/routes/blog: an author profile for Levi van Noort with frontmatter metadata (layout, name, slug, role, bio, avatar), and a blog post "How we reduced cold start times on Appwrite Sites" with frontmatter and content describing observability, igzip-based extraction optimization, Node.js file-tracing to reduce artifacts, measured template size and cold-start reductions (reported ~30–50%), redeploy instructions, and next steps. Both files are new additions; no existing files were modified. Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Warning Tools execution failed with the following error: Failed to run tools: Stream setup permanently failed: 13 INTERNAL: Received RST_STREAM with code 2 (Internal server error) 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. Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
src/routes/blog/post/reducing-cold-starts-appwrite-sites/+page.markdoc (1)
48-48: Minor: Inconsistent dash formatting.The sentence has inconsistent spacing around dashes: "download phase - previously" (space-dash-space) vs "steps-dropped" (no spaces). Consider using em-dashes consistently:
✏️ Suggested fix
-Those smaller artifacts translated directly into faster cold starts. With build outputs reduced by up to 99%, the download phase - previously one of the more costly steps-dropped from multiple seconds to just 100–200 milliseconds. Extraction saw a similar improvement, going from regular 4–7 second spikes down to around 200–400 milliseconds. Together, these two phases went from dominating the cold-start timeline to being barely noticeable. +Those smaller artifacts translated directly into faster cold starts. With build outputs reduced by up to 99%, the download phase—previously one of the more costly steps—dropped from multiple seconds to just 100–200 milliseconds. Extraction saw a similar improvement, going from regular 4–7 second spikes down to around 200–400 milliseconds. Together, these two phases went from dominating the cold-start timeline to being barely noticeable.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@src/routes/blog/post/reducing-cold-starts-appwrite-sites/`+page.markdoc at line 48, The sentence has inconsistent dash spacing ("download phase - previously" vs "steps-dropped"); update the punctuation in the Markdoc content so both occurrences use a consistent em-dash style (e.g., change "download phase - previously" to "download phase—previously" and "steps-dropped" to "steps—dropped") ensuring the same em-dash usage across the paragraph.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@src/routes/blog/post/reducing-cold-starts-appwrite-sites/`+page.markdoc:
- Line 48: The sentence has inconsistent dash spacing ("download phase -
previously" vs "steps-dropped"); update the punctuation in the Markdoc content
so both occurrences use a consistent em-dash style (e.g., change "download phase
- previously" to "download phase—previously" and "steps-dropped" to
"steps—dropped") ensuring the same em-dash usage across the paragraph.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: bfeea003-300d-4c67-9e3b-dba3340513a1
⛔ Files ignored due to path filters (3)
bun.lockis excluded by!**/*.lockstatic/images/avatars/levi.pngis excluded by!**/*.pngstatic/images/blog/reducing-cold-starts-appwrite-sites/cover.pngis excluded by!**/*.png
📒 Files selected for processing (2)
src/routes/blog/author/levi-van-noort/+page.markdocsrc/routes/blog/post/reducing-cold-starts-appwrite-sites/+page.markdoc
Greptile Summary
Confidence Score: 5/5Safe to merge — only P2 findings, no logic bugs or security issues. All changes are additive (new blog post, new images, new Image.svelte feature flag). The only finding is an unreferenced image asset, which is cosmetic. No functional regressions identified.
Important Files Changed
Reviews (11): Last reviewed commit: "chore: run bun run optimize to update as..." | Re-trigger Greptile |
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Co-authored-by: Aditya Oberai <adityaoberai1@gmail.com>
….markdoc Co-authored-by: Aditya Oberai <adityaoberai1@gmail.com>
….markdoc Co-authored-by: Aditya Oberai <adityaoberai1@gmail.com>
….markdoc Co-authored-by: Aditya Oberai <adityaoberai1@gmail.com>
….markdoc Co-authored-by: Aditya Oberai <adityaoberai1@gmail.com>
….markdoc Co-authored-by: Aditya Oberai <adityaoberai1@gmail.com>
….markdoc Co-authored-by: Aditya Oberai <adityaoberai1@gmail.com>
….markdoc Co-authored-by: Aditya Oberai <adityaoberai1@gmail.com>
….markdoc Co-authored-by: Aditya Oberai <adityaoberai1@gmail.com>
….markdoc Co-authored-by: Aditya Oberai <adityaoberai1@gmail.com>
….markdoc Co-authored-by: Aditya Oberai <adityaoberai1@gmail.com>
|
@copilot run |
Agent-Logs-Url: https://github.com/appwrite/website/sessions/6f3c30ed-ff00-4a4c-8846-b93bbd1434fb Co-authored-by: atharvadeosthale <11805367+atharvadeosthale@users.noreply.github.com>
Done in 6e99f84. The script ran successfully and updated the optimize cache with the new avatar image. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
What does this PR do?
Adds a new blog to the website.
Test Plan
N/A
Related PRs and Issues
N/A
Have you read the Contributing Guidelines on issues?
Yes
Summary by CodeRabbit