Skip to content

feat: blog post cold start improvements#2811

Open
levivannoort wants to merge 27 commits intomainfrom
CLO-4110
Open

feat: blog post cold start improvements#2811
levivannoort wants to merge 27 commits intomainfrom
CLO-4110

Conversation

@levivannoort
Copy link
Copy Markdown
Member

@levivannoort levivannoort commented Mar 17, 2026

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

  • Documentation
    • Added a new author profile for Levi van Noort.
    • Published a blog post detailing cold-start performance work for Appwrite Sites, including observed bottlenecks, experimental optimizations (faster extraction and reduced runtime footprint), measured improvements, and redeployment instructions.
    • Included quantified results and a “what’s next” section outlining future optimization plans.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 17, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: c6a68a03-2284-40c0-a8d7-f6b38038b362

📥 Commits

Reviewing files that changed from the base of the PR and between dc75779 and 7bf0c31.

📒 Files selected for processing (1)
  • src/routes/blog/post/reducing-cold-starts-appwrite-sites/+page.markdoc
✅ Files skipped from review due to trivial changes (1)
  • src/routes/blog/post/reducing-cold-starts-appwrite-sites/+page.markdoc

Walkthrough

Adds 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)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'feat: blog post cold start improvements' directly relates to the main changes: adding a new blog post about cold start optimizations and creating an author profile page.
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
  • Commit unit tests in branch CLO-4110
📝 Coding Plan
  • Generate coding plan for human review comments

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.

❤️ Share

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

@levivannoort levivannoort marked this pull request as ready for review March 18, 2026 14:58
Copy link
Copy Markdown
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.

🧹 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

📥 Commits

Reviewing files that changed from the base of the PR and between 0154be8 and dc75779.

⛔ Files ignored due to path filters (3)
  • bun.lock is excluded by !**/*.lock
  • static/images/avatars/levi.png is excluded by !**/*.png
  • static/images/blog/reducing-cold-starts-appwrite-sites/cover.png is excluded by !**/*.png
📒 Files selected for processing (2)
  • src/routes/blog/author/levi-van-noort/+page.markdoc
  • src/routes/blog/post/reducing-cold-starts-appwrite-sites/+page.markdoc

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Mar 18, 2026

Greptile Summary

This PR publishes a new blog post covering cold-start performance improvements for Appwrite Sites (igzip extraction, Node file tracing) and introduces a new author profile for Levi van Noort. It also adds a "contain" display mode to Image.svelte to prevent chart/graph images from being cropped to the standard 16:9 aspect ratio.

Confidence Score: 5/5

Safe to merge; only remaining finding is a minor design concern about the sentinel-string approach in Image.svelte

All previously flagged blocking issues (missing image file, run-on text) are resolved. The one new comment is P2 style feedback that does not affect correctness or production behavior.

src/markdoc/nodes/Image.svelte - the sentinel-string pattern is worth revisiting if more display modes are added in the future

Important Files Changed

Filename Overview
src/markdoc/nodes/Image.svelte Adds a contain display mode triggered by the title attribute sentinel value; functional but reserves the word contain as an image title forever
src/routes/blog/post/reducing-cold-starts-appwrite-sites/+page.markdoc New blog post on cold-start optimizations; previously flagged missing image replaced with two correctly named image files
src/routes/blog/author/levi-van-noort/+page.markdoc New author profile for Levi van Noort with correct fields and avatar path
.optimize-cache.json Cache hashes added for the new avatar and blog images; no issues

Reviews (8): Last reviewed commit: "Update src/routes/blog/post/reducing-col..." | Re-trigger Greptile

Comment thread src/routes/blog/post/reducing-cold-starts-appwrite-sites/+page.markdoc Outdated
Comment thread src/routes/blog/post/reducing-cold-starts-appwrite-sites/+page.markdoc Outdated
Comment thread src/routes/blog/post/reducing-cold-starts-appwrite-sites/+page.markdoc Outdated
Comment thread src/routes/blog/post/reducing-cold-starts-appwrite-sites/+page.markdoc Outdated
Comment thread src/routes/blog/post/reducing-cold-starts-appwrite-sites/+page.markdoc Outdated
Comment thread src/routes/blog/post/reducing-cold-starts-appwrite-sites/+page.markdoc Outdated
Comment thread src/routes/blog/post/reducing-cold-starts-appwrite-sites/+page.markdoc Outdated
Comment thread src/routes/blog/post/reducing-cold-starts-appwrite-sites/+page.markdoc Outdated
Comment thread src/routes/blog/post/reducing-cold-starts-appwrite-sites/+page.markdoc Outdated
Comment thread src/routes/blog/post/reducing-cold-starts-appwrite-sites/+page.markdoc Outdated
Comment thread src/routes/blog/post/reducing-cold-starts-appwrite-sites/+page.markdoc Outdated
Comment thread src/routes/blog/post/reducing-cold-starts-appwrite-sites/+page.markdoc Outdated
Comment thread src/routes/blog/post/reducing-cold-starts-appwrite-sites/+page.markdoc Outdated
levivannoort and others added 5 commits April 22, 2026 10:25
….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>
levivannoort and others added 5 commits April 22, 2026 10:27
….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>
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