Skip to content

Fix InputFile imports in Hugging Face templates - #355

Merged
ChiragAgg5k merged 1 commit into
mainfrom
fix/node-appwrite-input-file-import
Aug 3, 2026
Merged

Fix InputFile imports in Hugging Face templates#355
ChiragAgg5k merged 1 commit into
mainfrom
fix/node-appwrite-input-file-import

Conversation

@ChiragAgg5k

Copy link
Copy Markdown
Member

Fixes the Node builds for the Text to speech and Music generation Hugging Face templates.

InputFile is exported through the node-appwrite/file subpath in node-appwrite 14.x, not from the package root. The 1.0.0 release therefore fails both templates during npm run setup.

Tested with a local Appwrite Docker Compose stack and Playwright-driven template deployments:

  • Text to speech (node-22): ready
  • Music generation (node-22): ready
  • Prettier: passed
  • Direct ESM imports with installed lockfile dependencies: passed

Related: appwrite/appwrite#13071

@greptile-apps

greptile-apps Bot commented Aug 3, 2026

Copy link
Copy Markdown

Greptile Summary

This PR fixes module initialization in two Hugging Face templates by importing InputFile from the subpath exported by node-appwrite 14.x.

  • Updates the music-generation template’s InputFile import.
  • Applies the same correction to the text-to-speech template.

Confidence Score: 5/5

The PR appears safe to merge.

Both templates resolve a compatible node-appwrite 14.x release and load appwrite.js as native ESM, for which the new node-appwrite/file import is the supported path.

Important Files Changed

Filename Overview
node/music-generation-with-huggingface/src/appwrite.js Moves InputFile to the valid node-appwrite/file ESM subpath while retaining the remaining root imports.
node/text-to-speech-with-huggingface/src/appwrite.js Corrects the InputFile import for the template’s resolved node-appwrite 14.2.0 dependency.

Reviews (1): Last reviewed commit: "fix: import InputFile from node-appwrite..." | Re-trigger Greptile

@ChiragAgg5k
ChiragAgg5k merged commit 2eaf541 into main Aug 3, 2026
11 checks passed
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