xmi 0.5.6 → 0.5.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 62e6b5232df619d790a5ca7b69454d3ed6f2762bb457808436be572162867f16
-  data.tar.gz: 4adbc6c45fd9e75a2fb6612f60cb35ef47a91afe44cca8c366f8c9b17237aeab
+  metadata.gz: efc69e1cb868e8f245cb943d45b7ceaf007b31c59f6c6ff63f6439d999d4385e
+  data.tar.gz: 5bb58e282bd3eb3a565e8d407ce71ba602add350e8ed3ccf5ade429deafe01e8
 SHA512:
-  metadata.gz: fad7e8f844a110c38395e3d6b90f5506832cf4d971830b3dee23294d7268c6183b5a07ffafaa0832aba65b13ae1a2ad7dae7176b93d333c1607ef5607079f7af
-  data.tar.gz: e81076d61606ae1742f08fffa61aa7b0efd999c29dcbffd1c80ee5a34b81adaeb79bb988021f33a07cb51c8c16d023a09cdf5b884905dfb80b4070634731dbdd
+  metadata.gz: 0eaafdbea1e4fc8d9cd306c9b17ba169a660675b11094fa971f729c5a20126753c3d57e31a31cbae0c394215c26879e63b7c05d0e0a80479fc969bb48fbafb0b
+  data.tar.gz: ec71dd2f1fa4f19238b6c8f0a464bac681f05989f54aed68bc43ff6c212ee8b3e493b26624613b4244121841d422aaa12353c452fbd87cae9845ddcbebd01d68

data/lib/xmi/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Xmi
-  VERSION = "0.5.6"
+  VERSION = "0.5.7"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: xmi
 version: !ruby/object:Gem::Version
-  version: 0.5.6
+  version: 0.5.7
 platform: ruby
 authors:
 - Ribose Inc.
@@ -57,11 +57,6 @@ files:
 - Gemfile
 - README.adoc
 - Rakefile
-- TODO.perf/01-eliminate-duplicate-nokogiri-parse.md
-- TODO.perf/02-read-only-fast-mode.md
-- TODO.perf/03-register-fallback-idempotency.md
-- TODO.perf/04-pipeline-hash-mutation.md
-- TODO.perf/05-ea-root-single-parse.md
 - bin/console
 - bin/setup
 - docs/migration.md

data/TODO.perf/01-eliminate-duplicate-nokogiri-parse.md

@@ -1,33 +0,0 @@
-# 01: Eliminate Duplicate Nokogiri Parse
-
-## Impact: HIGH (~30-50% parse time for large files)
-
-## Problem
-
-`ParserPipeline` parses XML twice:
-1. `NamespaceDetector.detect_versions` → `Nokogiri::XML(xml_content)` + `collect_namespaces` — parses entire doc just to read root namespace URIs
-2. `from_xml` → adapter.parse(xml_content) — parses again via lutaml-model
-
-For the 3.5MB large fixture, this is the dominant cost.
-
-## Fix
-
-Extract namespace URIs from the first ~4KB of the XML string using regex, avoiding Nokogiri entirely for version detection.
-
-```ruby
-# Instead of:
-doc = Nokogiri::XML(xml_content)
-doc.collect_namespaces
-
-# Use:
-NS_REGEX = /xmlns:?(\\w*)=["']([^"']+)["']/
-xml_content[0, 4096].scan(NS_REGEX).to_h { |prefix, uri| [prefix, uri] }
-```
-
-The namespace declarations are always on or near the root element, so scanning the first 4KB is sufficient.
-
-## Files
-
-- `lib/xmi/namespace_detector.rb` — replace `extract_namespace_uris` Nokogiri parse with regex
-- `lib/xmi/namespace_detector.rb` — verify `detect_versions` still works
-- `spec/xmi/parser_pipeline_spec.rb` — verify tests pass

data/TODO.perf/02-read-only-fast-mode.md

@@ -1,38 +0,0 @@
-# 02: Read-Only Fast Mode (Skip Namespace Declaration Plan)
-
-## Impact: HIGH (eliminates full tree walk before mapping starts)
-
-## Problem
-
-In lutaml-model's `ModelTransform#data_to_model` (lines 71-76), `build_input_declaration_plan` calls `collect_element_namespaces` which recursively walks **every element** in the parsed document — solely for round-trip namespace fidelity (preserving xmlns declarations for `to_xml`).
-
-Most XMI consumers only parse (read) and never call `to_xml`. This full tree walk is wasted work.
-
-## Fix
-
-### In lutaml-model
-
-Add a `parse_only: true` option to `from_xml` that skips namespace declaration plan collection:
-
-```ruby
-# model_transform.rb line 71
-unless options.key?(:lutaml_parent) || options[:parse_only]
-  input_declaration_plan = build_input_declaration_plan(root_element)
-  # ...
-end
-```
-
-### In xmi gem
-
-Pass `parse_only: true` in `ParserPipeline::ParseXml` step:
-
-```ruby
-model_class.from_xml(ctx[:xml], register: register, parse_only: true)
-```
-
-If the user later calls `to_xml`, it works but without preserved input namespace declarations (acceptable for read-only use). If they need full round-trip, they opt out of `parse_only`.
-
-## Files
-
-- `lutaml-model/lib/lutaml/xml/model_transform.rb` — gate `build_input_declaration_plan` behind `parse_only` option
-- `lib/xmi/parser_pipeline.rb` — pass `parse_only: true` in ParseXml step

data/TODO.perf/03-register-fallback-idempotency.md

@@ -1,20 +0,0 @@
-# 03: Register Fallback Idempotency Guard
-
-## Impact: MEDIUM (prevents cache invalidation on every parse)
-
-## Problem
-
-`extend_fallback_for_mixed_namespaces` is called on every `parse_xml` for mixed-namespace documents. It calls `primary_register.add_fallback(reg.id)` which invalidates internal caches even when the fallback was already added on a previous parse.
-
-## Fix
-
-Add a guard before calling `add_fallback`:
-
-```ruby
-next if primary_register.fallback.include?(reg.id)
-primary_register.add_fallback(reg.id)
-```
-
-## Files
-
-- `lib/xmi/namespace_detector.rb` or `lib/xmi/version_registry.rb` — find where `extend_fallback_for_mixed_namespaces` is called and add the guard

data/TODO.perf/04-pipeline-hash-mutation.md

@@ -1,31 +0,0 @@
-# 04: Pipeline Hash Mutation
-
-## Impact: LOW (saves 4 intermediate Hash allocations per parse)
-
-## Problem
-
-Each pipeline step returns `ctx.merge(key: value)` creating intermediate Hash objects.
-
-## Fix
-
-Use `ctx[:key] = value` mutation instead:
-
-```ruby
-# Before
-def self.call(ctx)
-  xml = ctx[:xml]
-  # ...process...
-  ctx.merge(xml: fixed_xml)
-
-# After
-def self.call(ctx)
-  xml = ctx[:xml]
-  # ...process...
-  ctx[:xml] = fixed_xml
-  ctx
-end
-```
-
-## Files
-
-- `lib/xmi/parser_pipeline.rb` — all 4 step modules

data/TODO.perf/05-ea-root-single-parse.md

@@ -1,29 +0,0 @@
-# 05: EaRoot Single Parse for Extension Loading
-
-## Impact: LOW (only affects load_extension, not parse_xml hot path)
-
-## Problem
-
-`EaRoot.load_extension(xml_path)` reads and parses the XML file twice:
-1. `derive_module_name` in `ea_root.rb:54-56` — `Nokogiri::XML(File.read(xml_path))`
-2. `build_extension` in `code_generation.rb:8` — `Nokogiri::XML(File.read(xml_path))`
-
-## Fix
-
-Parse once in `load_extension`, pass the Nokogiri doc to both:
-
-```ruby
-def load_extension(xml_path)
-  xmi_doc = Nokogiri::XML(File.read(xml_path))
-  extension_id = get_module_name(xmi_doc)
-  # ...guard...
-  build_extension_from_doc(xmi_doc)
-  update_mappings(extension_id)
-  loaded_extensions[extension_id] = xml_path
-end
-```
-
-## Files
-
-- `lib/xmi/ea_root.rb` — parse once, pass doc
-- `lib/xmi/ea_root/code_generation.rb` — accept doc parameter instead of file path