pr feedback

Author: guybedford

src/workerd/io/worker-fs.c++

@@ -1757,32 +1757,30 @@ void writeStdio(jsg::Lock& js, VirtualFileSystem::Stdio type, kj::ArrayPtr<const
   size_t endPos = chars.size();
   if (endPos > 0 && chars[endPos - 1] == '\n') endPos--;
 
-
   KJ_IF_SOME(console, js.global().get(js, "console"_kj).tryCast<jsg::JsObject>()) {
     auto method = console.get(js, "log"_kj);
     if (method.isFunction()) {
       v8::Local<v8::Value> methodVal(method);
-      auto methodFunc = methodVal.As<v8::Function>();
+      auto methodFunc = jsg::JsFunction(methodVal.As<v8::Function>());
 
       kj::String outputStr;
       auto processStdioPrefixed = Worker::Isolate::from(js).getProcessStdioPrefixed();
       if (processStdioPrefixed) {
         if (endPos == 0) {
-          v8::Local<v8::Value> args[] = {
-            js.str(type == VirtualFileSystem::Stdio::OUT ? "stdout:"_kj : "stderr:"_kj)};
-          jsg::check(methodFunc->Call(js.v8Context(), console, 1, args));
+          methodFunc.call(js, console,
+              js.str(type == VirtualFileSystem::Stdio::OUT ? "stdout:"_kj : "stderr:"_kj));
         } else {
-          v8::Local<v8::Value> args[] = {
-            js.str(type == VirtualFileSystem::Stdio::OUT ? "stdout:"_kj : "stderr:"_kj),
-            js.str(chars.first(endPos))};
-          jsg::check(methodFunc->Call(js.v8Context(), console, 2, args));
+          methodFunc.call(js, console,
+              js.str(type == VirtualFileSystem::Stdio::OUT ? "stdout:"_kj : "stderr:"_kj),
+              js.str(chars.first(endPos)));
         }
       } else {
-        v8::Local<v8::Value> args[] = {js.str(chars.first(endPos))};
-        jsg::check(methodFunc->Call(js.v8Context(), console, 1, args));
+        methodFunc.call(js, console, js.str(chars.first(endPos)));
       }
+      return;
     }
   }
+  KJ_LOG(WARNING, "No console.log implementation available for stdio logging");
 }
 
 // An StdioFile is a special file implementation used to represent stdin,