🐛 Bug Report: Use Default Build Commands

[ItzNotABug]

👟 Reproduction steps

If there are no build commands passed, e.g. npm install, the build will succeed but the execution will fail because the dependencies are not loaded.

See -

1. Successful Build (check the build size, no dependencies are loaded)-

2. Execution Failed -

---

This makes the optional command, not so optional as we must pass a build command.

👍 Expected behavior

Runtime should check & handle this behaviour and simply call a relevant build command if one doesn't exist for the function.

👎 Actual Behavior

Execution fails.

🎲 Appwrite version

Version 1.4.x

💻 Operating system

Linux

🧱 Your Environment

No response

👀 Have you spent some time to check if this issue has been raised before?

• I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

• I have read the Code of Conduct

[0x2f0]

Hello maintainers,
I want to work on this issue.
Can I be assigned to this Issue?

[stnguyen90]

@ItzNotABug, thanks for raising this! Let me bring it up with the team.

[gewenyu99]

Why would you not pass in a build command 🤔 This doesn't feel like a bug. It's expected that if you don't build, you can't run a function 😅

[ItzNotABug]

@gewenyu99 The build commands are marked optional, if I remember correctly.
This doesn't really make sense as on one side it's optional but on the other hand, the function wouldn't just work if it has dependencies.

My suggestion was to check if there's a command passed. If yes then process that, if not then it wouldn't hurt to use a default command.

Its pretty comman where someone would just forget adding the commands when creating a function. This would save a trip to add the commands again and redeployment.