Skip to content

feat variables api#2915

Merged
atharvadeosthale merged 5 commits intomainfrom
worktree-variables-api
Apr 30, 2026
Merged

feat variables api#2915
atharvadeosthale merged 5 commits intomainfrom
worktree-variables-api

Conversation

@atharvadeosthale
Copy link
Copy Markdown
Member

No description provided.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 27, 2026

Greptile Summary

This PR introduces dedicated environment-variables documentation pages for Functions, Sites, and the new project-scoped Variables API, alongside a blog post and changelog entry for the announcement. The existing inline env-var sections in functions/develop and sites/develop are removed and replaced with links to the new pages. Previously flagged issues (missing import os in the Python "Read variables" snippet in functions/environment-variables, and variableId absent from functions.createVariable/sites.createVariable examples) remain open and should be addressed before merging.

Confidence Score: 4/5

Safe to merge once the two pre-existing flagged issues are resolved; no new critical problems found

Two P1 issues flagged in earlier review threads remain unresolved: a missing import os in the Python snippet that will cause a NameError for any user who copies it, and an inconsistency in the variableId parameter across the three createVariable code blocks. No new P0/P1 issues were found in this pass.

src/routes/docs/products/functions/environment-variables/+page.markdoc — Python "Read variables" snippet still missing import os

Important Files Changed

Filename Overview
src/routes/docs/products/functions/environment-variables/+page.markdoc New page with full SDK coverage for managing function variables; Python "Read variables" snippet is missing import os (flagged in previous review threads)
src/routes/docs/advanced/platform/environment-variables/+page.markdoc New page documenting project-scoped variables with full SDK coverage; variableId present in project createVariable examples (flagged in prior threads for inconsistency vs functions/sites pages)
src/routes/docs/products/sites/environment-variables/+page.markdoc New page for site variables with full SDK coverage; "Read variables" section correctly defers to framework adapters rather than providing a potentially broken code snippet
src/routes/blog/post/announcing-variables-api/+page.markdoc New announcement blog post; cover image confirmed present in this PR; scope names and SDK examples look consistent with the docs pages
src/routes/changelog/(entries)/2026-04-30.markdoc New changelog entry for the Variables API; project.read/project.write scope names are consistent with the blog post and platform docs
src/routes/docs/products/functions/develop/+page.markdoc Old inline env-var table and code examples replaced with a single arrow-link to the new dedicated page; refactoring is clean
src/routes/docs/products/sites/develop/+page.markdoc Removes the inline env-var section (now at the dedicated sites/environment-variables page); no issues found
src/partials/functions-env-vars.md New partial with Appwrite-injected variables table; adds APPWRITE_FUNCTION_CPUS and APPWRITE_FUNCTION_MEMORY vs the old inline table
src/routes/docs/advanced/platform/+layout.svelte Adds "Environment variables" nav item with isNewUntil('30 May 2026') badge; correct placement
src/routes/docs/products/functions/+layout.svelte Adds "Environment variables" nav item to the Functions sidebar
src/routes/docs/products/sites/+layout.svelte Adds "Environment variables" nav item to the Sites sidebar

Reviews (4): Last reviewed commit: "cover image" | Re-trigger Greptile

Comment thread src/routes/changelog/(entries)/2026-04-30.markdoc
Comment thread src/routes/blog/post/announcing-variables-api/+page.markdoc
@atharvadeosthale atharvadeosthale merged commit 21aea19 into main Apr 30, 2026
6 checks passed
@atharvadeosthale atharvadeosthale deleted the worktree-variables-api branch April 30, 2026 17:25
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