Skip to content

Commit 5f7a382

Browse files
justin808claude
andcommitted
Fix worker.ts to use serverBundleCachePath instead of deprecated bundlePath
The worker.ts file was still using the old bundlePath property name in two places: 1. When destructuring config from getConfig() 2. When constructing the upload destination path This was a critical runtime bug - when users only set serverBundleCachePath (the new property name), the code would receive undefined for bundlePath, breaking file upload functionality. Changes: - Line 121: Updated destructuring to use serverBundleCachePath - Line 154: Updated path.join call to use serverBundleCachePath 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
1 parent 62dcc72 commit 5f7a382

File tree

1 file changed

+2
-2
lines changed
  • react_on_rails_pro/packages/node-renderer/src

1 file changed

+2
-2
lines changed

react_on_rails_pro/packages/node-renderer/src/worker.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,7 @@ export default function run(config: Partial<Config>) {
118118
// getConfig():
119119
buildConfig(config);
120120

121-
const { bundlePath, logHttpLevel, port, fastifyServerOptions, workersCount } = getConfig();
121+
const { serverBundleCachePath, logHttpLevel, port, fastifyServerOptions, workersCount } = getConfig();
122122

123123
const app = fastify({
124124
http2: useHttp2 as true,
@@ -151,7 +151,7 @@ export default function run(config: Partial<Config>) {
151151
fileSize: Infinity,
152152
},
153153
onFile: async (part) => {
154-
const destinationPath = path.join(bundlePath, 'uploads', part.filename);
154+
const destinationPath = path.join(serverBundleCachePath, 'uploads', part.filename);
155155
// TODO: inline here
156156
await saveMultipartFile(part, destinationPath);
157157
// eslint-disable-next-line no-param-reassign

0 commit comments

Comments
 (0)