panko_serializer 0.8.2 → 0.8.4

data/docs/package.json

@@ -1,14 +1,18 @@
 {
   "scripts": {
     "examples": "docusaurus-examples",
-    "start": "docusaurus-start",
-    "build": "docusaurus-build",
-    "publish-gh-pages": "docusaurus-publish",
+    "start": "docusaurus start",
+    "build": "docusaurus build",
+    "serve": "docusaurus serve",
     "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"
+  "dependencies": {
+    "@docusaurus/core": "^3.9.2",
+    "@docusaurus/preset-classic": "^3.9.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,91 @@ 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;
-  }
-
-  if (is_lazy_attributes_set_defined) {
-    return attributes_set;
-  }
-
-  lazy_attributes_hash = rb_ivar_get(attributes_set, attributes_id);
-  return lazy_attributes_hash;
-}
+static ID fetch_id;
 
 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 (strcmp(rb_class2name(CLASS_OF(attrs.values)),
+             "ActiveRecord::Result::IndexedRow") == 0) {
+    volatile VALUE indexed_row_column_indexes =
+        rb_ivar_get(attrs.values, rb_intern("@column_indexes"));
+    volatile VALUE indexed_row_row =
+        rb_ivar_get(attrs.values, rb_intern("@row"));
 
-  volatile VALUE attributes_container = panko_read_attributes_container(obj);
+    attrs.indexed_row_column_indexes = indexed_row_column_indexes;
+    attrs.indexed_row_row = indexed_row_row;
+    attrs.is_indexed_row = true;
+  }
 
-  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);
+  return attrs;
+}
 
-      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);
+VALUE _read_value_from_indexed_row(struct attributes attributes_ctx,
+                                   volatile VALUE member) {
+  volatile VALUE value = Qnil;
 
-      if (PANKO_EMPTY_HASH(attributes_hash) == false) {
-        attributes_ctx.attributes_hash = attributes_hash;
-      }
-    }
+  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);
 
-    attributes_ctx.types = rb_ivar_get(attributes_container, types_id);
-    attributes_ctx.values = rb_ivar_get(attributes_container, values_id);
+  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 +103,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 +121,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 +145,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 +172,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/ext/panko_serializer/attributes_writer/type_cast/type_cast.c

@@ -288,7 +288,7 @@ VALUE cast_date_time_type(VALUE value) {
   return Qundef;
 }
 
-VALUE rescue_func() { return Qfalse; }
+VALUE rescue_func(VALUE _arg, VALUE _data) { return Qfalse; }
 
 VALUE parse_json(VALUE value) {
   return rb_funcall(oj_type, oj_sc_parse_id, 2, rb_cObject, value);

data/gemfiles/{6.1.0.gemfile → 7.2.0.gemfile}

@@ -2,14 +2,16 @@
 
 source "https://rubygems.org"
 
-gem "standard", group: [:development, :test]
-gem "activesupport", "~> 6.1.0"
-gem "activemodel", "~> 6.1.0"
-gem "activerecord", "~> 6.1.0", group: :test
+gem "activesupport", "~> 7.2.0"
+gem "activemodel", "~> 7.2.0"
+gem "activerecord", "~> 7.2.0", group: :test
+gem "trilogy"
+gem "sqlite3", "~> 1.4"
 
 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"
@@ -18,6 +20,7 @@ end
 
 group :test do
   gem "faker"
+  gem "temping"
 end
 
 group :development do
@@ -27,4 +30,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.2.0.gemfile.lock

@@ -0,0 +1,221 @@
+PATH
+  remote: ..
+  specs:
+    panko_serializer (0.8.3)
+      activesupport
+      oj (> 3.11.0, < 4.0.0)
+
+GEM
+  remote: https://rubygems.org/
+  specs:
+    actionpack (7.2.3)
+      actionview (= 7.2.3)
+      activesupport (= 7.2.3)
+      cgi
+      nokogiri (>= 1.8.5)
+      racc
+      rack (>= 2.2.4, < 3.3)
+      rack-session (>= 1.0.1)
+      rack-test (>= 0.6.3)
+      rails-dom-testing (~> 2.2)
+      rails-html-sanitizer (~> 1.6)
+      useragent (~> 0.16)
+    actionview (7.2.3)
+      activesupport (= 7.2.3)
+      builder (~> 3.1)
+      cgi
+      erubi (~> 1.11)
+      rails-dom-testing (~> 2.2)
+      rails-html-sanitizer (~> 1.6)
+    active_model_serializers (0.10.15)
+      actionpack (>= 4.1)
+      activemodel (>= 4.1)
+      case_transform (>= 0.2)
+      jsonapi-renderer (>= 0.1.1.beta1, < 0.3)
+    activemodel (7.2.3)
+      activesupport (= 7.2.3)
+    activerecord (7.2.3)
+      activemodel (= 7.2.3)
+      activesupport (= 7.2.3)
+      timeout (>= 0.4.0)
+    activesupport (7.2.3)
+      base64
+      benchmark (>= 0.3)
+      bigdecimal
+      concurrent-ruby (~> 1.0, >= 1.3.1)
+      connection_pool (>= 2.2.5)
+      drb
+      i18n (>= 1.6, < 2)
+      logger (>= 1.4.2)
+      minitest (>= 5.1)
+      securerandom (>= 0.3)
+      tzinfo (~> 2.0, >= 2.0.5)
+    appraisal (2.5.0)
+      bundler
+      rake
+      thor (>= 0.14.0)
+    ast (2.4.3)
+    base64 (0.3.0)
+    benchmark (0.5.0)
+    benchmark-ips (2.14.0)
+    bigdecimal (3.3.1)
+    builder (3.3.0)
+    byebug (12.0.0)
+    case_transform (0.2)
+      activesupport
+    cgi (0.5.0)
+    concurrent-ruby (1.3.5)
+    connection_pool (2.5.4)
+    crass (1.0.6)
+    diff-lcs (1.6.2)
+    drb (2.2.3)
+    erubi (1.13.1)
+    faker (3.5.2)
+      i18n (>= 1.8.11, < 2)
+    i18n (1.14.7)
+      concurrent-ruby (~> 1.0)
+    json (2.13.2)
+    jsonapi-renderer (0.2.2)
+    language_server-protocol (3.17.0.5)
+    lint_roller (1.1.0)
+    logger (1.7.0)
+    loofah (2.24.1)
+      crass (~> 1.0.2)
+      nokogiri (>= 1.12.0)
+    memory_profiler (1.1.0)
+    mini_portile2 (2.8.9)
+    minitest (5.26.2)
+    nokogiri (1.18.9-arm64-darwin)
+      racc (~> 1.4)
+    nokogiri (1.18.9-x86_64-linux-gnu)
+      racc (~> 1.4)
+    oj (3.16.11)
+      bigdecimal (>= 3.0)
+      ostruct (>= 0.2)
+    ostruct (0.6.3)
+    parallel (1.27.0)
+    parser (3.3.9.0)
+      ast (~> 2.4.1)
+      racc
+    pg (1.6.1-arm64-darwin)
+    pg (1.6.1-x86_64-linux)
+    prism (1.4.0)
+    racc (1.8.1)
+    rack (3.1.16)
+    rack-session (2.1.1)
+      base64 (>= 0.1.0)
+      rack (>= 3.0.0)
+    rack-test (2.2.0)
+      rack (>= 1.3)
+    rails-dom-testing (2.3.0)
+      activesupport (>= 5.0.0)
+      minitest
+      nokogiri (>= 1.6)
+    rails-html-sanitizer (1.6.2)
+      loofah (~> 2.21)
+      nokogiri (>= 1.15.7, != 1.16.7, != 1.16.6, != 1.16.5, != 1.16.4, != 1.16.3, != 1.16.2, != 1.16.1, != 1.16.0.rc1, != 1.16.0)
+    rainbow (3.1.1)
+    rake (13.3.0)
+    rake-compiler (1.3.0)
+      rake
+    regexp_parser (2.11.2)
+    rspec (3.13.1)
+      rspec-core (~> 3.13.0)
+      rspec-expectations (~> 3.13.0)
+      rspec-mocks (~> 3.13.0)
+    rspec-core (3.13.5)
+      rspec-support (~> 3.13.0)
+    rspec-expectations (3.13.5)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.13.0)
+    rspec-mocks (3.13.5)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.13.0)
+    rspec-support (3.13.5)
+    rubocop (1.75.8)
+      json (~> 2.3)
+      language_server-protocol (~> 3.17.0.2)
+      lint_roller (~> 1.1.0)
+      parallel (~> 1.10)
+      parser (>= 3.3.0.2)
+      rainbow (>= 2.2.2, < 4.0)
+      regexp_parser (>= 2.9.3, < 3.0)
+      rubocop-ast (>= 1.44.0, < 2.0)
+      ruby-progressbar (~> 1.7)
+      unicode-display_width (>= 2.4.0, < 4.0)
+    rubocop-ast (1.46.0)
+      parser (>= 3.3.7.2)
+      prism (~> 1.4)
+    rubocop-performance (1.25.0)
+      lint_roller (~> 1.1)
+      rubocop (>= 1.75.0, < 2.0)
+      rubocop-ast (>= 1.38.0, < 2.0)
+    rubocop-rspec (3.6.0)
+      lint_roller (~> 1.1)
+      rubocop (~> 1.72, >= 1.72.1)
+    ruby-progressbar (1.13.0)
+    securerandom (0.4.1)
+    sqlite3 (1.7.3)
+      mini_portile2 (~> 2.8.0)
+    stackprof (0.2.27)
+    standard (1.50.0)
+      language_server-protocol (~> 3.17.0.2)
+      lint_roller (~> 1.0)
+      rubocop (~> 1.75.5)
+      standard-custom (~> 1.0.0)
+      standard-performance (~> 1.8)
+    standard-custom (1.0.2)
+      lint_roller (~> 1.0)
+      rubocop (~> 1.50)
+    standard-performance (1.8.0)
+      lint_roller (~> 1.1)
+      rubocop-performance (~> 1.25.0)
+    temping (4.3.0)
+      activerecord (>= 6.0, < 8.1)
+      activesupport (>= 6.0, < 8.1)
+    terminal-table (4.0.0)
+      unicode-display_width (>= 1.1.1, < 4)
+    thor (1.4.0)
+    timeout (0.4.4)
+    trilogy (2.9.0)
+    tzinfo (2.0.6)
+      concurrent-ruby (~> 1.0)
+    unicode-display_width (3.1.5)
+      unicode-emoji (~> 4.0, >= 4.0.4)
+    unicode-emoji (4.0.4)
+    useragent (0.16.11)
+    vernier (1.8.0)
+
+PLATFORMS
+  arm64-darwin
+  x86_64-linux
+
+DEPENDENCIES
+  active_model_serializers (~> 0.10)
+  activemodel (~> 7.2.0)
+  activerecord (~> 7.2.0)
+  activesupport (~> 7.2.0)
+  appraisal
+  benchmark-ips
+  byebug
+  faker
+  memory_profiler
+  panko_serializer!
+  pg
+  rake
+  rake-compiler
+  rspec (~> 3.0)
+  rubocop
+  rubocop-performance
+  rubocop-rspec
+  sqlite3 (~> 1.4)
+  stackprof
+  standard
+  standard-performance
+  temping
+  terminal-table
+  trilogy
+  vernier
+
+BUNDLED WITH
+   2.7.2

data/gemfiles/{7.0.0.gemfile → 8.0.0.gemfile}

@@ -2,14 +2,16 @@
 
 source "https://rubygems.org"
 
-gem "standard", group: [:development, :test]
-gem "activesupport", "~> 7.0.0"
-gem "activemodel", "~> 7.0.0"
-gem "activerecord", "~> 7.0.0", group: :test
+gem "activesupport", "~> 8.0.0"
+gem "activemodel", "~> 8.0.0"
+gem "activerecord", "~> 8.0.0", group: :test
+gem "trilogy"
+gem "sqlite3", ">= 2.1"
 
 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"
@@ -18,6 +20,7 @@ end
 
 group :test do
   gem "faker"
+  gem "temping"
 end
 
 group :development do
@@ -27,4 +30,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: "../"