Skip to content

Update Wasp Studio (deps, Express, tsx) #2848

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

infomiho
Copy link
Contributor

@infomiho infomiho commented Jun 18, 2025

This PR upgrade Wasp Studio:

  • Replaces ts-node with tsx (dev mode was broken, so I went with tsx which has a watch mode so I can remove nodemon)
  • Removes nodemon since it's no longer neded
  • Replaces Fastify with Express (we use Express is Wasp, so this reduces the amount of libs team members need to be aware of)
  • Deletes the studio/client/node_modules to keep the binary size down
  • Bumps the dep versions

As a side-effect, all this fixed the issues with Wasp Studio not working in Firefox.

Closes #2774

@infomiho infomiho requested a review from cprecioso June 18, 2025 13:45
@@ -2,7 +2,6 @@
<html lang="en">
<head>
<meta charset="UTF-8" />
<link rel="icon" type="image/svg+xml" href="/vite.svg" />
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

vite.svg didn't exist, so this just ended up being a 404 in the network tab

@@ -8,28 +8,25 @@
"main": "index.js",
"keywords": [],
"scripts": {
"build": "npm run build:client && rm -rf dist && tsc && cp -r ./public ./dist/public",
"build": "npm run build:client && rm -rf ./client/node_modules && rm -rf dist && tsc && cp -r ./public ./dist/public",
Copy link
Contributor Author

@infomiho infomiho Jun 18, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This removes the client's node_modules (this should help with keeping the build size smaller) @sodic

Related to #2718

"@fastify/cors": "^8.3.0",
"@fastify/static": "^6.11.2",
"commander": "^11.0.0",
"dotenv": "^16.0.3",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We didn't use .env files so I've removed dotenv as dep.

root: new URL("./public", import.meta.url).pathname,
});

app.use(morgan("dev"));
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Logging requests

@infomiho infomiho requested a review from sodic June 18, 2025 13:48
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.

wasp studio is broken on main in Firefox
1 participant