fix(mdbx): correct read_recent limit and fix test timestamps

- read_recent now reads full range then takes the last limit records

  instead of delegating limit to read_range which returns oldest first

- Test uses LOGIT_CURRENT_TIMESTAMP_MS() as now for period and limit checks

- Add limited_desc assertion to verify Descending with limit

Co-Authored-By: Claude Opus 4.8 <[email protected]>

Author: LimiNode

include/logit_cpp/logit/loggers/MdbxLogger.hpp

@@ -224,7 +224,12 @@ namespace logit {
                 LogReadOrder order = LogReadOrder::Ascending) const override {
             const int64_t now_ms = LOGIT_CURRENT_TIMESTAMP_MS();
             const int64_t from_ms = (period_ms > 0) ? (now_ms - period_ms) : 0;
-            auto records = read_range(from_ms, now_ms + 1, limit);
+            auto records = read_range(from_ms, now_ms + 1, 0);
+            if (limit > 0 && records.size() > limit) {
+                records.erase(
+                    records.begin(),
+                    records.begin() + static_cast<std::ptrdiff_t>(records.size() - limit));
+            }
             if (order == LogReadOrder::Descending && !records.empty()) {
                 std::reverse(records.begin(), records.end());
             }

tests/mdbx_logger_test.cpp

@@ -265,9 +265,10 @@ void test_read_recent() {
         config.async = false;
 
         logit::MdbxLogger logger(config);
-        logger.log(make_record(logit::LogLevel::LOG_LVL_INFO, 1000, 70), "old");
-        logger.log(make_record(logit::LogLevel::LOG_LVL_INFO, 2000, 71), "mid");
-        logger.log(make_record(logit::LogLevel::LOG_LVL_INFO, 3000, 72), "new");
+        const int64_t now = LOGIT_CURRENT_TIMESTAMP_MS();
+        logger.log(make_record(logit::LogLevel::LOG_LVL_INFO, now - 3000, 70), "old");
+        logger.log(make_record(logit::LogLevel::LOG_LVL_INFO, now - 1000, 71), "mid");
+        logger.log(make_record(logit::LogLevel::LOG_LVL_INFO, now, 72), "new");
 
         auto all_asc = logger.read_recent(0, 0, logit::LogReadOrder::Ascending);
         assert(all_asc.size() == 3);
@@ -281,6 +282,13 @@ void test_read_recent() {
 
         auto limited = logger.read_recent(2, 0, logit::LogReadOrder::Ascending);
         assert(limited.size() == 2);
+        assert(limited[0].message == "mid");
+        assert(limited[1].message == "new");
+
+        auto limited_desc = logger.read_recent(2, 0, logit::LogReadOrder::Descending);
+        assert(limited_desc.size() == 2);
+        assert(limited_desc[0].message == "new");
+        assert(limited_desc[1].message == "mid");
 
         auto period = logger.read_recent(10, 1500, logit::LogReadOrder::Ascending);
         assert(period.size() == 2);