mdbxmou 0.3.4 → 0.3.5

package/package.json

@@ -65,7 +65,7 @@
   },
   "gypfile": true,
   "name": "mdbxmou",
-  "version": "0.3.4",
+  "version": "0.3.5",
   "description": "Node bindings for mdbx",
   "repository": {
     "type": "git",

package/src/async/envmou_keys.cpp

@@ -30,7 +30,7 @@ static Napi::Value write_row(Napi::Env env, const keys_line& row)
     for (std::uint32_t j = 0; j < param.size(); ++j) {
         const auto& item = param[j];
         Napi::Value key_value;
-        if (key_mode.val & key_mode::ordinal) {
+        if (mdbx::is_ordinal(key_mode)) {
             if (key_flag.val & base_flag::number) {
                 key_value = Napi::Number::New(env, static_cast<double>(item.id_buf));
             } else {
@@ -105,9 +105,9 @@ void async_keys::do_keys_batch(txnmou_managed& txn,
     mdbx::map_handle dbi, keys_line& arg0)
 {
     auto& item = arg0.item;
-    const bool is_ordinal = mdbx::is_ordinal(arg0.key_mod);
-    
-    auto cursor = dbi::get_cursor(txn, dbi);
+    auto is_ordinal = mdbx::is_ordinal(arg0.key_mod);
+
+    auto cursor = dbi::open_cursor(txn, dbi);
 
     // Буфер для batch - MDBXMOU_BATCH_LIMIT/2 пар (key, value)
 #ifndef MDBXMOU_BATCH_LIMIT

package/src/async/envmou_query.cpp

@@ -39,7 +39,7 @@ static Napi::Value write_row(Napi::Env env, const query_line& row)
         const auto& item = param[j];
         Napi::Value key_value;
         Napi::Object js_item = Napi::Object::New(env);
-        if (key_mode.val & key_mode::ordinal) {
+        if (mdbx::is_ordinal(key_mode)) {
             if (key_flag.val & base_flag::number) {
                 key_value = Napi::Number::New(env, static_cast<double>(item.id_buf));
             } else {

package/src/cursormou.cpp

@@ -80,7 +80,7 @@ namespace mdbxmou
         auto value_flag = dbi_->get_value_flag();
 
         // Ключ
-        if (key_mode.val & key_mode::ordinal) {
+        if (mdbx::is_ordinal(key_mode)) {
             if (key_flag.val & base_flag::bigint) {
                 result.Set("key", key.to_bigint(env));
             } else {
@@ -210,7 +210,7 @@ namespace mdbxmou
 
         auto key_mode = dbi_->get_key_mode();
 
-        keymou key = (key_mode.val & key_mode::ordinal) ? 
+        keymou key = (mdbx::is_ordinal(key_mode)) ? 
             keymou::from(info[0], env, key_num_) : 
             keymou::from(info[0], env, key_buf_);
 
@@ -231,7 +231,7 @@ namespace mdbxmou
         auto value_flag = dbi_->get_value_flag();
 
         // Ключ
-        if (key_mode.val & key_mode::ordinal) {
+        if (mdbx::is_ordinal(key_mode)) {
             if (key_flag.val & base_flag::bigint) {
                 result.Set("key", key.to_bigint(env));
             } else {
@@ -275,7 +275,7 @@ namespace mdbxmou
 
         auto key_mode = dbi_->get_key_mode();
 
-        keymou key = (key_mode.val & key_mode::ordinal) ? 
+        keymou key = (mdbx::is_ordinal(key_mode)) ? 
             keymou::from(info[0], env, key_num_) : 
             keymou::from(info[0], env, key_buf_);
 
@@ -354,7 +354,7 @@ namespace mdbxmou
             auto result = Napi::Object::New(env);
 
             // Ключ
-            if (key_mode.val & key_mode::ordinal) {
+            if (mdbx::is_ordinal(key_mode)) {
                 if (key_flag.val & base_flag::bigint) {
                     result.Set("key", key.to_bigint(env));
                 } else {

package/src/dbi.cpp

@@ -47,10 +47,10 @@ bool dbi::del(MDBX_txn* txn, const keymou& key)
 
 cursormou_managed dbi::open_cursor(MDBX_txn* txn) const
 {
-    return get_cursor(txn, mdbx::map_handle{ id_ });
+    return open_cursor(txn, mdbx::map_handle{id_});
 }
 
-cursormou_managed dbi::get_cursor(MDBX_txn* txn, mdbx::map_handle dbi)
+cursormou_managed dbi::open_cursor(MDBX_txn *txn, mdbx::map_handle dbi)
 {
     MDBX_cursor* cursor_ptr;
     auto rc = mdbx_cursor_open(txn, dbi.dbi, &cursor_ptr);

package/src/dbi.hpp

@@ -39,7 +39,7 @@ public:
     cursormou_managed open_cursor(MDBX_txn* txn) const;
     
     // Static version for map_handle
-    static cursormou_managed get_cursor(MDBX_txn* txn, mdbx::map_handle dbi);
+    static cursormou_managed open_cursor(MDBX_txn *txn, mdbx::map_handle dbi);
 
     // Drop database or delete it
     void drop(MDBX_txn* txn, bool delete_db = false);

package/src/dbimou.cpp

@@ -48,7 +48,7 @@ Napi::Value dbimou::put(const Napi::CallbackInfo& info)
     auto txn = Napi::ObjectWrap<txnmou>::Unwrap(arg0);
     try {
         std::uint64_t t;
-        auto key = (key_mode_.val & key_mode::ordinal) ?
+        auto key = mdbx::is_ordinal(key_mode_) ?
             keymou::from(info[1], env, t) : 
             keymou::from(info[1], env, key_buf_);
 
@@ -77,7 +77,7 @@ Napi::Value dbimou::get(const Napi::CallbackInfo& info)
 
     try {
         std::uint64_t t;
-        auto key = (key_mode_.val & key_mode::ordinal) ?
+        auto key = mdbx::is_ordinal(key_mode_) ?
             keymou::from(info[1], env, t) : 
             keymou::from(info[1], env, key_buf_);
         
@@ -111,7 +111,7 @@ Napi::Value dbimou::del(const Napi::CallbackInfo& info)
 
     try {
         std::uint64_t t;
-        auto key = (key_mode_.val & key_mode::ordinal) ?
+        auto key = mdbx::is_ordinal(key_mode_) ?
             keymou::from(info[1], env, t) : 
             keymou::from(info[1], env, key_buf_);
         
@@ -140,7 +140,7 @@ Napi::Value dbimou::has(const Napi::CallbackInfo& info)
 
     try {
         std::uint64_t t;
-        auto key = (key_mode_.val & key_mode::ordinal) ?
+        auto key = mdbx::is_ordinal(key_mode_) ?
             keymou::from(info[1], env, t) : 
             keymou::from(info[1], env, key_buf_);
 
@@ -173,16 +173,16 @@ Napi::Value dbimou::for_each(const Napi::CallbackInfo& info)
         auto fn = info[1].As<Napi::Function>();
         
         try {
-            auto cursor = dbi::open_cursor(*txn);
-            auto stat = dbi::get_stat(*txn);
+            auto stat = get_stat(*txn);
             
             // Проверяем, есть ли записи в базе данных
             if (stat.ms_entries == 0) {
                 return Napi::Number::New(env, 0);
             }
-            
+
+            auto cursor = open_cursor(*txn);
             uint32_t index{};
-            if (key_mode_.val & key_mode::ordinal) {
+            if (mdbx::is_ordinal(key_mode_)) {
                 cursor.scan([&](const mdbx::pair& f) {
                     keymou key{f.key};
                     valuemou val{f.value};
@@ -238,7 +238,8 @@ Napi::Value dbimou::for_each(const Napi::CallbackInfo& info)
     }
 }
 
-Napi::Value dbimou::for_each_from(const Napi::CallbackInfo& info) {
+Napi::Value dbimou::for_each_from(const Napi::CallbackInfo& info) 
+{
     Napi::Env env = info.Env();
     auto arg_len = info.Length();
     
@@ -274,7 +275,7 @@ Napi::Value dbimou::for_each_from(const Napi::CallbackInfo& info) {
         
         // Парсим начальный ключ
         std::uint64_t t;
-        keymou from_key = (key_mode_.val & key_mode::ordinal) ?
+        keymou from_key = (mdbx::is_ordinal(key_mode_)) ?
             keymou::from(info[1], env, t) : 
             keymou::from(info[1], env, key_buf_);
         
@@ -308,7 +309,7 @@ Napi::Value dbimou::for_each_from(const Napi::CallbackInfo& info) {
         std::size_t index{};
         bool is_key_equal_mode = (cursor_mode == move_operation::key_equal || 
                                   cursor_mode == move_operation::multi_exactkey_value_equal);
-        if (key_mode_.val & key_mode::ordinal) {
+        if (mdbx::is_ordinal(key_mode_)) {
             cursor.scan_from([&](const mdbx::pair& f) {
                 keymou key{f.key};
                 valuemou val{f.value};
@@ -367,7 +368,8 @@ Napi::Value dbimou::for_each_from(const Napi::CallbackInfo& info) {
     }
 }
 
-Napi::Value dbimou::stat(const Napi::CallbackInfo& info) {
+Napi::Value dbimou::stat(const Napi::CallbackInfo& info) 
+{
     Napi::Env env = info.Env();
     auto arg_len = info.Length();
     if (arg_len < 1) {
@@ -405,7 +407,8 @@ Napi::Value dbimou::stat(const Napi::CallbackInfo& info) {
     return env.Undefined();
 }
 
-Napi::Value dbimou::keys(const Napi::CallbackInfo& info) {
+Napi::Value dbimou::keys(const Napi::CallbackInfo& info) 
+{
     Napi::Env env = info.Env();
     auto arg_len = info.Length();
     if (arg_len < 1) {
@@ -437,9 +440,9 @@ Napi::Value dbimou::keys(const Napi::CallbackInfo& info) {
         std::array<mdbx::slice, MDBXMOU_BATCH_LIMIT> pairs;
 
         uint32_t index{};
-        const bool is_ordinal = key_mode_.val & key_mode::ordinal;
-        const bool is_bigint = key_flag_.val & base_flag::bigint;
-        const bool is_string = key_flag_.val & base_flag::string;
+        auto is_ordinal = mdbx::is_ordinal(key_mode_);
+        auto is_bigint = key_flag_.val & base_flag::bigint;
+        auto is_string = key_flag_.val & base_flag::string;
         
         // Первый вызов с MDBX_FIRST
         size_t count = cursor.get_batch(pairs, MDBX_FIRST);
@@ -469,7 +472,8 @@ Napi::Value dbimou::keys(const Napi::CallbackInfo& info) {
     return env.Undefined();
 }
 
-Napi::Value dbimou::keys_from(const Napi::CallbackInfo& info) {
+Napi::Value dbimou::keys_from(const Napi::CallbackInfo& info) 
+{
     Napi::Env env = info.Env();
     auto arg_len = info.Length();
     if (arg_len < 2) {
@@ -487,7 +491,7 @@ Napi::Value dbimou::keys_from(const Napi::CallbackInfo& info) {
         
         // Парсим аргументы: txn, from, limit, cursorMode
         std::uint64_t t;
-        keymou from_key = (key_mode_.val & key_mode::ordinal) ?
+        keymou from_key = (mdbx::is_ordinal(key_mode_)) ?
             keymou::from(info[1], env, t) : 
             keymou::from(info[1], env, key_buf_);
         
@@ -528,7 +532,7 @@ Napi::Value dbimou::keys_from(const Napi::CallbackInfo& info) {
         bool is_key_equal_mode = (cursor_mode == move_operation::key_equal || 
                                   cursor_mode == move_operation::multi_exactkey_value_equal);
         
-        if (key_mode_.val & key_mode::ordinal) {
+        if (mdbx::is_ordinal(key_mode_)) {
             cursor.scan_from([&](const mdbx::pair& f) {
                 if (index >= count) {
                     return true; // останавливаем сканирование
@@ -580,7 +584,8 @@ Napi::Value dbimou::keys_from(const Napi::CallbackInfo& info) {
     return env.Undefined();
 }
 
-Napi::Value dbimou::drop(const Napi::CallbackInfo& info) {
+Napi::Value dbimou::drop(const Napi::CallbackInfo& info) 
+{
     Napi::Env env = info.Env();
     if (info.Length() < 1) {
         throw Napi::TypeError::New(env, "First argument must be a transaction");

package/src/querymou.cpp

@@ -27,7 +27,7 @@ void async_key::parse(const async_common& common, const Napi::Value& item)
     auto key_flag = common.key_flag;
 
     keymou key{};
-    if (common.key_mod.val & key_mode::ordinal) {
+    if (mdbx::is_ordinal(common.key_mod)) {
         if (item.IsBigInt()) {
             key = keymou{item.As<Napi::BigInt>(), id_buf};
         } else if (item.IsNumber()) {

package/src/txnmou.cpp

@@ -70,21 +70,102 @@ void txnmou::dec_counter() noexcept
     }
 }
 
-Napi::Value txnmou::get_dbi(const Napi::CallbackInfo& info, db_mode db_mode)
+Napi::Value txnmou::get_dbi(const char* name, base_flag key_flag, 
+    base_flag value_flag, key_mode key_mode, value_mode value_mode, 
+    db_mode db_mode)
 {
-    Napi::Env env = info.Env();
+    Napi::Env env = Env();
 
     if ((mode_.val & txn_mode::ro) && (db_mode.val & db_mode::create)) {
         throw Napi::Error::New(env, "dbi: cannot open DB in read-only transaction");
     }
 
+    if (!txn_) {
+        throw Napi::Error::New(env, "txn already completed");
+    }
+
+    MDBX_dbi dbi{};
+    auto flags = static_cast<MDBX_db_flags_t>(db_mode.val|key_mode.val|value_mode.val);
+    auto rc = mdbx_dbi_open(*this, (name && name[0]) ? name : nullptr, flags, &dbi);
+    if (rc != MDBX_SUCCESS) {
+        throw Napi::Error::New(env, std::string("mdbx_dbi_open: ") + mdbx_strerror(rc));
+    }
+    // создаем новый объект dbi
+    auto obj = dbimou::ctor.New({});
+    auto ptr = dbimou::Unwrap(obj);
+    ptr->attach(dbi, db_mode, key_mode, 
+        value_mode, key_flag, value_flag);
+    return obj;
+}
+
+Napi::Value txnmou::get_dbi(const Napi::Object& arg0, db_mode db_mode)
+{
+    auto env = arg0.Env();
+    auto conf = get_env_userctx(*env_);
+    auto key_flag = conf->key_flag;
+    auto value_flag = conf->value_flag;
     key_mode key_mode{};
     value_mode value_mode{};
     std::string db_name{};
+
+    if (arg0.Has("name")) {
+        auto value = arg0.Get("name");
+        if (!value.IsUndefined() && !value.IsNull()) {
+            if (!value.IsString()) {
+                throw Napi::Error::New(env, "dbi: name must be string");
+            }
+            db_name = value.As<Napi::String>().Utf8Value();
+        }
+    }
+
+    if (arg0.Has("keyFlag")) {
+        auto value = arg0.Get("keyFlag");
+        if (!value.IsUndefined() && !value.IsNull()) {
+            key_flag = base_flag::parse_key(value);
+        }
+    }
+
+    if (arg0.Has("valueFlag")) {
+        auto value = arg0.Get("valueFlag");
+        if (!value.IsUndefined() && !value.IsNull()) {
+            value_flag = base_flag::parse_value(value);
+        }
+    }
+
+    if (arg0.Has("keyMode")) {
+        auto value = arg0.Get("keyMode");
+        if (!value.IsUndefined() && !value.IsNull()) {
+            key_mode = parse_key_mode(env, value, key_flag);
+        }
+    }
+
+    if (arg0.Has("valueMode")) {
+        auto value = arg0.Get("valueMode");
+        if (!value.IsUndefined() && !value.IsNull()) {
+            value_mode = value_mode::parse(value);
+        }
+    }
+
+    return get_dbi(db_name.empty() ? nullptr : db_name.c_str(), 
+        key_flag, value_flag, key_mode, value_mode, db_mode);
+}
+
+Napi::Value txnmou::get_dbi(const Napi::CallbackInfo& info, db_mode db_mode)
+{
+    Napi::Env env = info.Env();
+
     auto conf = get_env_userctx(*env_);
     auto key_flag = conf->key_flag;
     auto value_flag = conf->value_flag;
+    key_mode key_mode{};
+    value_mode value_mode{};
+    std::string db_name{};
     auto arg_count = info.Length();
+
+    if (arg_count == 1 && info[0].IsObject()) {
+        return get_dbi(info[0].As<Napi::Object>(), db_mode);
+    }
+
     if (arg_count == 3) {
         auto arg0 = info[0]; // db_name
         auto arg1 = info[1]; // key_mode
@@ -94,8 +175,8 @@ Napi::Value txnmou::get_dbi(const Napi::CallbackInfo& info, db_mode db_mode)
         value_mode = value_mode::parse(arg2);
     } else if (arg_count == 2) {
         // db_name + key_mode || key_mode + value_mode
-        auto arg0 = info[0]; 
-        auto arg1 = info[1]; 
+        auto arg0 = info[0];
+        auto arg1 = info[1];
         if (arg0.IsString()) {
             db_name = arg0.As<Napi::String>().Utf8Value();
             key_mode = parse_key_mode(env, arg1, key_flag);
@@ -116,24 +197,10 @@ Napi::Value txnmou::get_dbi(const Napi::CallbackInfo& info, db_mode db_mode)
             throw Napi::Error::New(env, "Invalid argument type: expected string (db_name) or number (key_mode)");
         }
     }
-    // arg_count == 0: используем значения по умолчанию (строковый ключ, default db)
-
-    if (!txn_) {
-        throw Napi::Error::New(env, "txn already completed");
-    }
+    // arg_count == 0: значения по умолчанию (key/value = buffer, если env не задавал флаги)
 
-    MDBX_dbi dbi{};
-    auto flags = static_cast<MDBX_db_flags_t>(db_mode.val|key_mode.val|value_mode.val);
-    auto rc = mdbx_dbi_open(*this, db_name.empty() ? nullptr : db_name.c_str(), flags, &dbi);
-    if (rc != MDBX_SUCCESS) {
-        throw Napi::Error::New(env, std::string("mdbx_dbi_open: ") + mdbx_strerror(rc));
-    }
-    // создаем новый объект dbi
-    auto obj = dbimou::ctor.New({});
-    auto ptr = dbimou::Unwrap(obj);
-    ptr->attach(dbi, db_mode, key_mode, 
-        value_mode, key_flag, value_flag);
-    return obj;
+    return get_dbi(db_name.empty() ? nullptr : db_name.c_str(), 
+        key_flag, value_flag, key_mode, value_mode, db_mode);
 }
 
 Napi::Value txnmou::open_cursor(const Napi::CallbackInfo& info) {
@@ -178,4 +245,4 @@ void txnmou::attach(envmou& env, MDBX_txn* txn, txn_mode mode)
     txn_.reset(txn);
 }
 
-} // namespace mdbxmou
+} // namespace mdbxmou

package/src/txnmou.hpp

@@ -27,7 +27,10 @@ private:
     // Уменьшает счетчик транзакций
     void dec_counter() noexcept;
 
-    Napi::Value get_dbi(const Napi::CallbackInfo&, db_mode);
+    Napi::Value get_dbi(const char* name, base_flag key_flag, base_flag value_flag,
+        key_mode key_mode, value_mode value_mode, db_mode db_mode);
+    Napi::Value get_dbi(const Napi::Object& arg0, db_mode db_mode);
+    Napi::Value get_dbi(const Napi::CallbackInfo& info, db_mode db_mode);
 
 public:    
     static Napi::FunctionReference ctor;

package/src/typemou.hpp

@@ -246,7 +246,7 @@ static inline key_mode parse_key_mode(Napi::Env env, const Napi::Value& arg0, ba
             throw Napi::Error::New(env, "BigInt value lossless conversion failed");
         }
         mode.val = static_cast<int>(value);
-        if (mode.val & key_mode::ordinal) {
+        if (mdbx::is_ordinal(mode)) {
             // только если цифровой key_flag не был задан
             if (key_flag.val <= base_flag::string) {
                 key_flag.val = base_flag::bigint;
@@ -254,7 +254,7 @@ static inline key_mode parse_key_mode(Napi::Env env, const Napi::Value& arg0, ba
         }
     } else if (arg0.IsNumber()) {
         mode = key_mode::parse(arg0);
-        if (mode.val & key_mode::ordinal) {
+        if (mdbx::is_ordinal(mode)) {
             // только если цифровой key_flag не был задан
             if (key_flag.val <= base_flag::string) {
                 key_flag.val = base_flag::number;