Add Python Support

.github/workflows/unit_tests.yml

@@ -56,7 +56,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        backends: [ V8, JavaScriptCore, Lua, Empty ]
+        backends: [ V8, JavaScriptCore, Lua, Python, Empty ]
         build_type:
           - Debug
           - Release

backend/JavaScriptCore/JscEngine.cc

@@ -19,6 +19,7 @@
 #include "../../src/Native.hpp"
 #include "JscEngine.hpp"
 #include "JscHelper.h"
+#include "../../src/utils/Helper.hpp"
 
 namespace script::jsc_backend {
 
@@ -177,6 +178,27 @@ script::Local<script::Value> JscEngine::eval(const script::Local<script::String>
   return eval(script, {});
 }
 
+Local<Value> JscEngine::loadFile(const Local<String>& scriptFile) {
+  if(scriptFile.toString().empty())
+    throw Exception("script file no found");
+  Local<Value> content = internal::readAllFileContent(scriptFile);
+  if(content.isNull())
+    throw Exception("can't load script file");
+
+  std::string sourceFilePath = scriptFile.toString();
+  std::size_t pathSymbol = sourceFilePath.rfind("/");
+  if(pathSymbol != -1)
+    sourceFilePath = sourceFilePath.substr(pathSymbol + 1);
+  else
+  {
+    pathSymbol = sourceFilePath.rfind("\\");
+    if(pathSymbol != -1)
+      sourceFilePath = sourceFilePath.substr(pathSymbol + 1);
+  }
+  Local<String> sourceFileName = String::newString(sourceFilePath);
+  return eval(content.asString(), sourceFileName);
+}
+
 std::shared_ptr<utils::MessageQueue> JscEngine::messageQueue() { return messageQueue_; }
 
 void JscEngine::gc() {

backend/JavaScriptCore/JscEngine.h

@@ -86,6 +86,8 @@ class JscEngine : public ::script::ScriptEngine {
   Local<Value> eval(const Local<String>& script) override;
   using ScriptEngine::eval;
 
+  Local<Value> loadFile(const Local<String>& scriptFile) override;
+
   std::shared_ptr<utils::MessageQueue> messageQueue() override;
 
   void gc() override;

backend/JavaScriptCore/JscUtils.cc

@@ -73,7 +73,7 @@ std::string_view StringHolder::stringView() const {
 std::string StringHolder::string() const {
   jsc_backend::initString(internalHolder_);
   internalHolder_.inited = false;
-  return std::move(internalHolder_.stringContent);
+  return internalHolder_.stringContent;
 }
 
 #if defined(__cpp_char8_t)

backend/Lua/LuaEngine.cc

@@ -28,6 +28,7 @@
 #include "LuaHelper.hpp"
 #include "LuaReference.hpp"
 #include "LuaScope.hpp"
+#include "../../src/utils/Helper.hpp"
 
 // ref https://www.lua.org/manual/5.1/manual.html
 // https://www.lua.org/wshop14/Zykov.pdf
@@ -259,6 +260,27 @@ Local<Value> LuaEngine::eval(const Local<String>& script, const Local<Value>& so
   return lua_backend::callFunction({}, {}, 0, nullptr);
 }
 
+Local<Value> LuaEngine::loadFile(const Local<String>& scriptFile) {
+  if(scriptFile.toString().empty())
+    throw Exception("script file no found");
+  Local<Value> content = internal::readAllFileContent(scriptFile);
+  if(content.isNull())
+    throw Exception("can't load script file");
+
+  std::string sourceFilePath = scriptFile.toString();
+  std::size_t pathSymbol = sourceFilePath.rfind("/");
+  if(pathSymbol != -1)
+    sourceFilePath = sourceFilePath.substr(pathSymbol + 1);
+  else
+  {
+    pathSymbol = sourceFilePath.rfind("\\");
+    if(pathSymbol != -1)
+      sourceFilePath = sourceFilePath.substr(pathSymbol + 1);
+  }
+  Local<String> sourceFileName = String::newString(sourceFilePath);
+  return eval(content.asString(), sourceFileName);
+}
+
 Arguments LuaEngine::makeArguments(LuaEngine* engine, int stackBase, size_t paramCount,
                                    bool isInstanceFunc) {
   lua_backend::ArgumentsData argumentsData{engine, stackBase, paramCount, isInstanceFunc};

backend/Lua/LuaEngine.h

@@ -77,6 +77,8 @@ class LuaEngine : public ScriptEngine {
   Local<Value> eval(const Local<String>& script) override;
   using ScriptEngine::eval;
 
+  Local<Value> loadFile(const Local<String>& scriptFile) override;
+
   std::shared_ptr<utils::MessageQueue> messageQueue() override;
 
   void gc() override;

backend/Lua/LuaLocalReference.cc

@@ -43,6 +43,28 @@ void ensureNonnull(int index) {
   throw Exception("NullPointerException");
 }
 
+bool judgeIsArray(int index)
+{
+  auto lua = currentLua();
+  int currectArrIndex = 0;
+
+  lua_pushnil(lua); 
+
+  while (lua_next(lua, index))
+  {
+      // Copy current key and judge it's type
+      lua_pushvalue(lua, -2);
+      if(!lua_isnumber(lua,-1) || lua_tonumber(lua,-1) != ++currectArrIndex)
+      {
+        lua_pop(lua, 3);
+        return false;
+      }
+      lua_pop(lua, 2);
+  }
+  return true;
+}
+
+
 }  // namespace lua_backend
 
 #define REF_IMPL_BASIC_FUNC(ValueType)                                                           \
@@ -149,8 +171,11 @@ ValueKind Local<Value>::getKind() const {
   } else if (isByteBuffer()) {
     return ValueKind::kByteBuffer;
   } else if (type == LUA_TTABLE) {
-    // lua don't have array type, the are all tables
-    return ValueKind::kObject;
+    // Traverse the table to judge whether it is an array
+    if(isArray())
+      return ValueKind::kArray;
+    else
+      return ValueKind::kObject;
   } else {
     return ValueKind::kUnsupported;
   }
@@ -176,7 +201,11 @@ bool Local<Value>::isFunction() const {
   return val_ != 0 && lua_type(lua_backend::currentLua(), val_) == LUA_TFUNCTION;
 }
 
-bool Local<Value>::isArray() const { return isObject(); }
+bool Local<Value>::isArray() const { 
+  if(val_ == 0 || lua_type(lua_backend::currentLua(), val_) != LUA_TTABLE)
+    return false;
+  return lua_backend::judgeIsArray(val_);
+}
 
 bool Local<Value>::isByteBuffer() const {
   auto engine = lua_backend::currentEngine();
@@ -437,4 +466,4 @@ void Local<ByteBuffer>::commit() const {}
 
 void Local<ByteBuffer>::sync() const {}
 
-}  // namespace script
+}  // namespace script

backend/Python/CMakeLists.txt

@@ -1 +1,16 @@
-message(FATAL_ERROR "${SCRIPTX_BACKEND} is to be implemented.")
+target_sources(ScriptX PRIVATE
+        ${CMAKE_CURRENT_LIST_DIR}/PyEngine.cc
+        ${CMAKE_CURRENT_LIST_DIR}/PyEngine.h
+        ${CMAKE_CURRENT_LIST_DIR}/PyException.cc
+        ${CMAKE_CURRENT_LIST_DIR}/PyHelper.h
+        ${CMAKE_CURRENT_LIST_DIR}/PyHelper.hpp
+        ${CMAKE_CURRENT_LIST_DIR}/PyHelper.cc
+        ${CMAKE_CURRENT_LIST_DIR}/PyLocalReference.cc
+        ${CMAKE_CURRENT_LIST_DIR}/PyNative.cc
+        ${CMAKE_CURRENT_LIST_DIR}/PyNative.hpp
+        ${CMAKE_CURRENT_LIST_DIR}/PyReference.hpp
+        ${CMAKE_CURRENT_LIST_DIR}/PyScope.h
+        ${CMAKE_CURRENT_LIST_DIR}/PyScope.cc
+        ${CMAKE_CURRENT_LIST_DIR}/PyUtils.cc
+        ${CMAKE_CURRENT_LIST_DIR}/PyValue.cc
+        )