engineering/tpmjs
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 2.1k

fix(executor): declare packageName/exportName before try block

Browse source 
Same issue as startTime - these variables were destructured inside the
try block but referenced in the catch block for health reporting. If
JSON parsing or any early error occurred, the catch block would crash
with 'packageName is not defined'.

Now declares them with 'unknown' defaults before try, then assigns
the actual values inside.
This commit is contained in:
Ajax Davis 2025-12-12 06:14:08 +10:00
parent bd4878576e
commit 8f27d1cfa7
1 changed files with 6 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

7
apps/railway-executor/server.ts
View file

@ -379,9 +379,14 @@ async function loadAndDescribe(req: Request): Promise<Response> {
*/
async function executeTool(req: Request): Promise<Response> {
const startTime = Date.now();
// Declare these before try block so they're available in catch for error reporting
let packageName = 'unknown';
let exportName = 'unknown';
try {
const body = await req.json();
const { packageName, exportName, version, importUrl, params, env } = body;
const { packageName: pkg, exportName: exp, version, importUrl, params, env } = body;
packageName = pkg || 'unknown';
exportName = exp || 'unknown';
console.log('📥 Execute request:', {
packageName,