panko_serializer 0.8.2 → 0.8.3

data/docs/package.json

@@ -1,14 +1,21 @@
 {
   "scripts": {
     "examples": "docusaurus-examples",
-    "start": "docusaurus-start",
-    "build": "docusaurus-build",
-    "publish-gh-pages": "docusaurus-publish",
+    "start": "docusaurus start",
+    "build": "docusaurus build",
     "write-translations": "docusaurus-write-translations",
     "version": "docusaurus-version",
-    "rename-version": "docusaurus-rename-version"
+    "rename-version": "docusaurus-rename-version",
+    "swizzle": "docusaurus swizzle",
+    "deploy": "docusaurus deploy",
+    "docusaurus": "docusaurus"
   },
-  "devDependencies": {
-    "docusaurus": "^1.14.7"
+  "devDependencies": {},
+  "dependencies": {
+    "@docusaurus/core": "3.5.2",
+    "@docusaurus/preset-classic": "3.5.2",
+    "clsx": "^1.1.1",
+    "react": "^17.0.2",
+    "react-dom": "^17.0.2"
   }
-}
+}

data/docs/src/css/customTheme.css

@@ -0,0 +1,9 @@
+:root{
+  --ifm-color-primary-lightest: #D34A4B;
+  --ifm-color-primary-lighter: #CA3133;
+  --ifm-color-primary-light: #C22F30;
+  --ifm-color-primary: #B02B2C;
+  --ifm-color-primary-dark: #9E2728;
+  --ifm-color-primary-darker: #962525;
+  --ifm-color-primary-darkest: #7B1E1F;
+}

data/docs/static/CNAME

@@ -0,0 +1 @@
+panko.dev

data/ext/panko_serializer/attributes_writer/active_record.c

@@ -9,79 +9,118 @@ static ID delegate_hash_id;
 static ID value_before_type_cast_id;
 static ID type_id;
 
-static bool type_detection_ran = false;
-static bool is_lazy_attributes_set_defined = false;
-
-/***
- * Returns ActiveModel::LazyAttributeSet or ActiveModel::LazyAttributesHash
- */
-VALUE panko_read_attributes_container(VALUE object) {
-  volatile VALUE attributes_set, lazy_attributes_hash;
-
-  attributes_set = rb_ivar_get(object, attributes_id);
-  if (NIL_P(attributes_set)) {
-    return Qnil;
+static ID fetch_id;
+
+// ActiveRecord::Result::IndexedRow
+static VALUE ar_result_indexed_row = Qundef;
+static int fetched_ar_result_indexed_row = 0;
+
+VALUE fetch_ar_result_indexed_row_type() {
+  if (fetched_ar_result_indexed_row == 1) {
+    return ar_result_indexed_row;
   }
 
-  if (is_lazy_attributes_set_defined) {
-    return attributes_set;
+  fetched_ar_result_indexed_row = 1;
+
+  VALUE ar, ar_result;
+
+  ar = rb_const_get_at(rb_cObject, rb_intern("ActiveRecord"));
+
+  // ActiveRecord::Result
+  ar_result = rb_const_get_at(ar, rb_intern("Result"));
+
+  if (rb_const_defined_at(ar_result, rb_intern("IndexedRow")) == (int)Qtrue) {
+    ar_result_indexed_row = rb_const_get_at(ar_result, rb_intern("IndexedRow"));
   }
 
-  lazy_attributes_hash = rb_ivar_get(attributes_set, attributes_id);
-  return lazy_attributes_hash;
+  return ar_result_indexed_row;
 }
 
 struct attributes {
+  // Hash
   VALUE attributes_hash;
+  size_t attributes_hash_size;
 
+  // Hash
   VALUE types;
+  // Hash
   VALUE additional_types;
+  // heuristics
+  bool tryToReadFromAdditionalTypes;
+
+  // Rails <8: Hash
+  // Rails >=8: ActiveRecord::Result::IndexedRow
   VALUE values;
 
-  // heuristicts
-  bool tryToReadFromAdditionalTypes;
+  // Hash
+  VALUE indexed_row_column_indexes;
+  // Array or NIL
+  VALUE indexed_row_row;
+  bool is_indexed_row;
 };
 
 struct attributes init_context(VALUE obj) {
-  struct attributes attributes_ctx;
-  attributes_ctx.attributes_hash = Qnil;
-  attributes_ctx.values = Qnil;
-  attributes_ctx.types = Qnil;
-  attributes_ctx.additional_types = Qnil;
+  volatile VALUE attributes_set = rb_ivar_get(obj, attributes_id);
+  volatile VALUE attributes_hash = rb_ivar_get(attributes_set, attributes_id);
+
+  struct attributes attrs = (struct attributes){
+      .attributes_hash =
+          PANKO_EMPTY_HASH(attributes_hash) ? Qnil : attributes_hash,
+      .attributes_hash_size = 0,
+
+      .types = rb_ivar_get(attributes_set, types_id),
+      .additional_types = rb_ivar_get(attributes_set, additional_types_id),
+      .tryToReadFromAdditionalTypes =
+          PANKO_EMPTY_HASH(rb_ivar_get(attributes_set, additional_types_id)) ==
+          false,
+
+      .values = rb_ivar_get(attributes_set, values_id),
+      .is_indexed_row = false,
+      .indexed_row_column_indexes = Qnil,
+      .indexed_row_row = Qnil,
+  };
+
+  if (attrs.attributes_hash != Qnil) {
+    attrs.attributes_hash_size = RHASH_SIZE(attrs.attributes_hash);
+  }
 
-  attributes_ctx.tryToReadFromAdditionalTypes = false;
+  if (CLASS_OF(attrs.values) == fetch_ar_result_indexed_row_type()) {
+    volatile VALUE indexed_row_column_indexes =
+        rb_ivar_get(attrs.values, rb_intern("@column_indexes"));
 
-  volatile VALUE attributes_container = panko_read_attributes_container(obj);
+    volatile VALUE indexed_row_row =
+        rb_ivar_get(attrs.values, rb_intern("@row"));
 
-  if (RB_TYPE_P(attributes_container, T_HASH)) {
-    attributes_ctx.attributes_hash = attributes_container;
-  } else {
-    if (is_lazy_attributes_set_defined == false) {
-      volatile VALUE delegate_hash =
-          rb_ivar_get(attributes_container, delegate_hash_id);
+    attrs.indexed_row_column_indexes = indexed_row_column_indexes;
+    attrs.indexed_row_row = indexed_row_row;
+    attrs.is_indexed_row = true;
+  }
 
-      if (PANKO_EMPTY_HASH(delegate_hash) == false) {
-        attributes_ctx.attributes_hash = delegate_hash;
-      }
-    } else {
-      volatile VALUE attributes_hash =
-          rb_ivar_get(attributes_container, attributes_id);
+  return attrs;
+}
 
-      if (PANKO_EMPTY_HASH(attributes_hash) == false) {
-        attributes_ctx.attributes_hash = attributes_hash;
-      }
-    }
+VALUE _read_value_from_indexed_row(struct attributes attributes_ctx,
+                                   volatile VALUE member) {
+  volatile VALUE value = Qnil;
 
-    attributes_ctx.types = rb_ivar_get(attributes_container, types_id);
-    attributes_ctx.values = rb_ivar_get(attributes_container, values_id);
+  if (NIL_P(attributes_ctx.indexed_row_column_indexes) ||
+      NIL_P(attributes_ctx.indexed_row_row)) {
+    return value;
+  }
+
+  volatile VALUE column_index =
+      rb_hash_aref(attributes_ctx.indexed_row_column_indexes, member);
+
+  if (NIL_P(column_index)) {
+    return value;
+  }
 
-    attributes_ctx.additional_types =
-        rb_ivar_get(attributes_container, additional_types_id);
-    attributes_ctx.tryToReadFromAdditionalTypes =
-        PANKO_EMPTY_HASH(attributes_ctx.additional_types) == false;
+  volatile VALUE row = attributes_ctx.indexed_row_row;
+  if (NIL_P(row)) {
+    return value;
   }
 
-  return attributes_ctx;
+  return RARRAY_AREF(row, NUM2INT(column_index));
 }
 
 VALUE read_attribute(struct attributes attributes_ctx, Attribute attribute,
@@ -91,7 +130,11 @@ VALUE read_attribute(struct attributes attributes_ctx, Attribute attribute,
   member = attribute->name_str;
   value = Qnil;
 
-  if (!NIL_P(attributes_ctx.attributes_hash)) {
+  if (
+      // we have attributes_hash
+      !NIL_P(attributes_ctx.attributes_hash)
+      // It's not empty
+      && (attributes_ctx.attributes_hash_size > 0)) {
     volatile VALUE attribute_metadata =
         rb_hash_aref(attributes_ctx.attributes_hash, member);
 
@@ -105,7 +148,11 @@ VALUE read_attribute(struct attributes attributes_ctx, Attribute attribute,
   }
 
   if (NIL_P(value) && !NIL_P(attributes_ctx.values)) {
-    value = rb_hash_aref(attributes_ctx.values, member);
+    if (attributes_ctx.is_indexed_row == true) {
+      value = _read_value_from_indexed_row(attributes_ctx, member);
+    } else {
+      value = rb_hash_aref(attributes_ctx.values, member);
+    }
   }
 
   if (NIL_P(attribute->type) && !NIL_P(value)) {
@@ -125,31 +172,9 @@ VALUE read_attribute(struct attributes attributes_ctx, Attribute attribute,
   return value;
 }
 
-VALUE detect_active_model_changes(VALUE v) {
-  if (type_detection_ran == true) {
-    return Qundef;
-  }
-
-  type_detection_ran = true;
-
-  volatile VALUE active_model_type =
-      rb_const_get_at(rb_cObject, rb_intern("ActiveModel"));
-
-  is_lazy_attributes_set_defined =
-      rb_const_defined(active_model_type, rb_intern("LazyAttributeSet")) > 0;
-
-  return Qundef;
-}
-
 void active_record_attributes_writer(VALUE obj, VALUE attributes,
                                      EachAttributeFunc write_value,
                                      VALUE writer) {
-  if (type_detection_ran == false) {
-    // If ActiveModel can't be found it will throw error
-    int isErrored;
-    rb_protect(detect_active_model_changes, Qnil, &isErrored);
-  }
-
   long i;
   struct attributes attributes_ctx = init_context(obj);
   volatile VALUE record_class = CLASS_OF(obj);
@@ -174,4 +199,10 @@ void init_active_record_attributes_writer(VALUE mPanko) {
   additional_types_id = rb_intern("@additional_types");
   type_id = rb_intern("@type");
   value_before_type_cast_id = rb_intern("@value_before_type_cast");
+  fetch_id = rb_intern("fetch");
 }
+
+void panko_init_active_record(VALUE mPanko) {
+  init_active_record_attributes_writer(mPanko);
+  panko_init_type_cast(mPanko);
+}

data/ext/panko_serializer/attributes_writer/active_record.h

@@ -13,3 +13,5 @@ extern void active_record_attributes_writer(VALUE object, VALUE attributes,
                                             VALUE writer);
 
 void init_active_record_attributes_writer(VALUE mPanko);
+
+void panko_init_active_record(VALUE mPanko);

data/gemfiles/7.0.0.gemfile

@@ -2,14 +2,15 @@
 
 source "https://rubygems.org"
 
-gem "standard", group: [:development, :test]
+gem "sqlite3", "~> 1.4"
 gem "activesupport", "~> 7.0.0"
 gem "activemodel", "~> 7.0.0"
 gem "activerecord", "~> 7.0.0", group: :test
 
 group :benchmarks do
-  gem "sqlite3", "~> 1.4"
-  gem "pg", ">= 0.18", "< 2.0"
+  gem "vernier"
+  gem "stackprof"
+  gem "pg"
   gem "benchmark-ips"
   gem "active_model_serializers", "~> 0.10"
   gem "terminal-table"
@@ -27,4 +28,12 @@ group :development do
   gem "rake-compiler"
 end
 
+group :development, :test do
+  gem "rubocop"
+  gem "standard"
+  gem "standard-performance"
+  gem "rubocop-performance"
+  gem "rubocop-rspec"
+end
+
 gemspec path: "../"

data/gemfiles/7.0.0.gemfile.lock

@@ -1,22 +1,22 @@
 PATH
   remote: ..
   specs:
-    panko_serializer (0.8.1)
+    panko_serializer (0.8.3)
       activesupport
       oj (> 3.11.0, < 4.0.0)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    actionpack (7.0.8.1)
-      actionview (= 7.0.8.1)
-      activesupport (= 7.0.8.1)
+    actionpack (7.0.8.5)
+      actionview (= 7.0.8.5)
+      activesupport (= 7.0.8.5)
       rack (~> 2.0, >= 2.2.4)
       rack-test (>= 0.6.3)
       rails-dom-testing (~> 2.0)
       rails-html-sanitizer (~> 1.0, >= 1.2.0)
-    actionview (7.0.8.1)
-      activesupport (= 7.0.8.1)
+    actionview (7.0.8.5)
+      activesupport (= 7.0.8.5)
       builder (~> 3.1)
       erubi (~> 1.4)
       rails-dom-testing (~> 2.0)
@@ -26,12 +26,12 @@ GEM
       activemodel (>= 4.1)
       case_transform (>= 0.2)
       jsonapi-renderer (>= 0.1.1.beta1, < 0.3)
-    activemodel (7.0.8.1)
-      activesupport (= 7.0.8.1)
-    activerecord (7.0.8.1)
-      activemodel (= 7.0.8.1)
-      activesupport (= 7.0.8.1)
-    activesupport (7.0.8.1)
+    activemodel (7.0.8.5)
+      activesupport (= 7.0.8.5)
+    activerecord (7.0.8.5)
+      activemodel (= 7.0.8.5)
+      activesupport (= 7.0.8.5)
+    activesupport (7.0.8.5)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (>= 1.6, < 2)
       minitest (>= 5.1)
@@ -41,38 +41,42 @@ GEM
       rake
       thor (>= 0.14.0)
     ast (2.4.2)
-    benchmark-ips (2.13.0)
-    builder (3.2.4)
+    benchmark-ips (2.14.0)
+    bigdecimal (3.1.8)
+    builder (3.3.0)
     byebug (11.1.3)
     case_transform (0.2)
       activesupport
-    concurrent-ruby (1.2.3)
+    concurrent-ruby (1.3.4)
     crass (1.0.6)
     diff-lcs (1.5.1)
-    erubi (1.12.0)
-    faker (3.2.3)
+    erubi (1.13.0)
+    faker (3.5.1)
       i18n (>= 1.8.11, < 2)
-    i18n (1.14.4)
+    i18n (1.14.6)
       concurrent-ruby (~> 1.0)
-    json (2.7.1)
+    json (2.7.5)
     jsonapi-renderer (0.2.2)
     language_server-protocol (3.17.0.3)
     lint_roller (1.1.0)
     loofah (2.22.0)
       crass (~> 1.0.2)
       nokogiri (>= 1.12.0)
-    memory_profiler (1.0.1)
-    minitest (5.22.3)
-    nokogiri (1.16.3-arm64-darwin)
+    memory_profiler (1.1.0)
+    minitest (5.25.4)
+    nokogiri (1.16.7-arm64-darwin)
       racc (~> 1.4)
-    oj (3.16.1)
-    parallel (1.24.0)
-    parser (3.3.0.5)
+    oj (3.16.7)
+      bigdecimal (>= 3.0)
+      ostruct (>= 0.2)
+    ostruct (0.6.0)
+    parallel (1.26.3)
+    parser (3.3.5.1)
       ast (~> 2.4.1)
       racc
-    pg (1.5.5)
-    racc (1.7.3)
-    rack (2.2.8.1)
+    pg (1.5.9)
+    racc (1.8.1)
+    rack (2.2.10)
     rack-test (2.1.0)
       rack (>= 1.3)
     rails-dom-testing (2.2.0)
@@ -84,59 +88,61 @@ GEM
       nokogiri (~> 1.14)
     rainbow (3.1.1)
     rake (13.2.1)
-    rake-compiler (1.2.1)
+    rake-compiler (1.2.8)
       rake
-    regexp_parser (2.9.0)
-    rexml (3.2.6)
+    regexp_parser (2.9.2)
     rspec (3.13.0)
       rspec-core (~> 3.13.0)
       rspec-expectations (~> 3.13.0)
       rspec-mocks (~> 3.13.0)
-    rspec-core (3.13.0)
+    rspec-core (3.13.2)
       rspec-support (~> 3.13.0)
-    rspec-expectations (3.13.0)
+    rspec-expectations (3.13.3)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.13.0)
-    rspec-mocks (3.13.0)
+    rspec-mocks (3.13.2)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.13.0)
     rspec-support (3.13.1)
-    rubocop (1.62.1)
+    rubocop (1.66.1)
       json (~> 2.3)
       language_server-protocol (>= 3.17.0)
       parallel (~> 1.10)
       parser (>= 3.3.0.2)
       rainbow (>= 2.2.2, < 4.0)
-      regexp_parser (>= 1.8, < 3.0)
-      rexml (>= 3.2.5, < 4.0)
-      rubocop-ast (>= 1.31.1, < 2.0)
+      regexp_parser (>= 2.4, < 3.0)
+      rubocop-ast (>= 1.32.2, < 2.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 2.4.0, < 3.0)
-    rubocop-ast (1.31.2)
-      parser (>= 3.3.0.4)
-    rubocop-performance (1.20.2)
+    rubocop-ast (1.33.0)
+      parser (>= 3.3.1.0)
+    rubocop-performance (1.22.1)
       rubocop (>= 1.48.1, < 2.0)
-      rubocop-ast (>= 1.30.0, < 2.0)
+      rubocop-ast (>= 1.31.1, < 2.0)
+    rubocop-rspec (3.2.0)
+      rubocop (~> 1.61)
     ruby-progressbar (1.13.0)
     sqlite3 (1.7.3-arm64-darwin)
-    standard (1.34.0)
+    stackprof (0.2.26)
+    standard (1.41.1)
       language_server-protocol (~> 3.17.0.2)
       lint_roller (~> 1.0)
-      rubocop (~> 1.60)
+      rubocop (~> 1.66.0)
       standard-custom (~> 1.0.0)
-      standard-performance (~> 1.3)
+      standard-performance (~> 1.5)
     standard-custom (1.0.2)
       lint_roller (~> 1.0)
       rubocop (~> 1.50)
-    standard-performance (1.3.1)
+    standard-performance (1.5.0)
       lint_roller (~> 1.1)
-      rubocop-performance (~> 1.20.2)
+      rubocop-performance (~> 1.22.0)
     terminal-table (3.0.2)
       unicode-display_width (>= 1.1.1, < 3)
-    thor (1.3.1)
+    thor (1.3.2)
     tzinfo (2.0.6)
       concurrent-ruby (~> 1.0)
-    unicode-display_width (2.5.0)
+    unicode-display_width (2.6.0)
+    vernier (1.3.0)
 
 PLATFORMS
   arm64-darwin-23
@@ -152,13 +158,19 @@ DEPENDENCIES
   faker
   memory_profiler
   panko_serializer!
-  pg (>= 0.18, < 2.0)
+  pg
   rake
   rake-compiler
   rspec (~> 3.0)
+  rubocop
+  rubocop-performance
+  rubocop-rspec
   sqlite3 (~> 1.4)
+  stackprof
   standard
+  standard-performance
   terminal-table
+  vernier
 
 BUNDLED WITH
    2.4.6

data/gemfiles/7.1.0.gemfile

@@ -2,14 +2,15 @@
 
 source "https://rubygems.org"
 
-gem "standard", group: [:development, :test]
-gem "activesupport", "~> 7.1.0"
-gem "activemodel", "~> 7.1.0"
-gem "activerecord", "~> 7.1.0", group: :test
+gem "sqlite3", "~> 1.4"
+gem "activesupport", "~> 7.1.5"
+gem "activemodel", "~> 7.1.5"
+gem "activerecord", "~> 7.1.5", group: :test
 
 group :benchmarks do
-  gem "sqlite3", "~> 1.4"
-  gem "pg", ">= 0.18", "< 2.0"
+  gem "vernier"
+  gem "stackprof"
+  gem "pg"
   gem "benchmark-ips"
   gem "active_model_serializers", "~> 0.10"
   gem "terminal-table"
@@ -27,4 +28,12 @@ group :development do
   gem "rake-compiler"
 end
 
+group :development, :test do
+  gem "rubocop"
+  gem "standard"
+  gem "standard-performance"
+  gem "rubocop-performance"
+  gem "rubocop-rspec"
+end
+
 gemspec path: "../"