[js] fix incorrect output name mapping

fs-eire · fs-eire · commit cee5ce4f765d · 2026-01-07T17:21:50.000-08:00
diff --git a/js/node/src/inference_session_wrap.cc b/js/node/src/inference_session_wrap.cc
@@ -216,7 +216,7 @@ Napi::Value InferenceSessionWrap::Run(const Napi::CallbackInfo& info) {
       Napi::Object result = Napi::Object::New(env);
 
       for (size_t i = 0; i < outputIndex; i++) {
-        result.Set(outputNames_[i], OrtValueToNapiValue(env, std::move(outputValues[i])));
+        result.Set(outputNames_cstr[i], OrtValueToNapiValue(env, std::move(outputValues[i])));
       }
       return scope.Escape(result);
     } else {
@@ -244,7 +244,7 @@ Napi::Value InferenceSessionWrap::Run(const Napi::CallbackInfo& info) {
 
       Napi::Object result = Napi::Object::New(env);
       for (size_t i = 0; i < outputIndex; i++) {
-        result.Set(outputNames_[i], OrtValueToNapiValue(env, std::move(outputs[i])));
+        result.Set(outputNames_cstr[i], OrtValueToNapiValue(env, std::move(outputs[i])));
       }
       return scope.Escape(result);
     }

Original file line number	Diff line number	Diff line change
`@@ -216,7 +216,7 @@ Napi::Value InferenceSessionWrap::Run(const Napi::CallbackInfo& info) {`
`216`	`216`	`Napi::Object result = Napi::Object::New(env);`
`217`	`217`
`218`	`218`	`for (size_t i = 0; i < outputIndex; i++) {`
`219`		`- result.Set(outputNames_[i], OrtValueToNapiValue(env, std::move(outputValues[i])));`
	`219`	`+ result.Set(outputNames_cstr[i], OrtValueToNapiValue(env, std::move(outputValues[i])));`
`220`	`220`	`}`
`221`	`221`	`return scope.Escape(result);`
`222`	`222`	`} else {`
`@@ -244,7 +244,7 @@ Napi::Value InferenceSessionWrap::Run(const Napi::CallbackInfo& info) {`
`244`	`244`
`245`	`245`	`Napi::Object result = Napi::Object::New(env);`
`246`	`246`	`for (size_t i = 0; i < outputIndex; i++) {`
`247`		`- result.Set(outputNames_[i], OrtValueToNapiValue(env, std::move(outputs[i])));`
	`247`	`+ result.Set(outputNames_cstr[i], OrtValueToNapiValue(env, std::move(outputs[i])));`
`248`	`248`	`}`
`249`	`249`	`return scope.Escape(result);`
`250`	`250`	`}`