moxml 0.5.131 → 0.5.132

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2de6081e549de3acc9fe0fe2697d0f44e137673eaa311dc9beec4de6346dfb54
-  data.tar.gz: 026a76efc77132825449057e48559626e3f3ec4166d54fb5ea817d51accb830e
+  metadata.gz: 6e9c4fec61c2e21eec26368889b888c8cdf1b3c553161bafc0334861cef93529
+  data.tar.gz: 0d32dcd44ab21957f09078af75e147005d1da6a67a72a17626d488757c6a305c
 SHA512:
-  metadata.gz: 369b3b0ca0527711f87e4b18f9b268c503612c9aed6aa8ea4fcfcb6dd9ad835be5cbb80b9ff00b0606b3df908bb150a0ef22bba6041157bfdb41a0bd8264a153
-  data.tar.gz: a86491ce96b541420ddf36fc2cbe1324c2f283d212d0271cb2eb61129bca15258dd7d92dbe7a022c6e3e4055140fb395111c96e8518ebd5b5d882a036186ae33
+  metadata.gz: 1d8da6f131247d4a9b61bc119e85ec19096bdc8769e35ec545f35da657d23556fb22b9d7b8e00c68684aa875a46134fb3d8845526d123f7d4379ef174ff70a8e
+  data.tar.gz: a8d0a826db5dd0de0febc05189048a86f7c23a2f8cb028395d6add3500f331b18d104498a1880c7e28e8786b81aab8beb2b9af8ae9e101b1c13755a3fdbe03ac

data/lib/moxml/builder.rb

@@ -7,7 +7,7 @@ module Moxml
     attr_reader :document
     alias_method :doc, :document
 
-    def initialize(context)
+    def initialize(context, strict: nil)
       @context = context
       @current = @document = context.create_document
       @namespaces = {}
@@ -22,6 +22,22 @@ module Moxml
       @face_counts = Hash.new(0)
       @face_deferred = []
       @face_entity_markers = false
+      # #371: strict mode — unknown methods raise instead of
+      # silently becoming empty elements. Per-build override
+      # (Context#build(strict:)), else the context's
+      # config.builder_strict.
+      @strict = strict.nil? ? context.config.builder_strict : strict
+    end
+
+    def strict?
+      @strict == true
+    end
+
+    # The tag-creation face NamespacedTags calls — the private
+    # create_element_node stays private.
+    def create_tag(tag_name, attrs = {}, text_content: nil, block: nil)
+      create_element_node(tag_name, attrs, text_content: text_content,
+                                           block: block, eval_block: false)
     end
 
     def build(&block)
@@ -30,6 +46,61 @@ module Moxml
       @document
     end
 
+    # Namespace-scoped tag syntax (#371, Nokogiri's xml["p"].tag
+    # parity): xml["w"].p creates <w:p>. The proxy creates through
+    # the same face as plain method_missing — caller-context
+    # blocks, text/attributes arguments, trailing-underscore
+    # escape — and honors the strict mode.
+    def [](namespace_prefix)
+      raise ArgumentError, "namespace prefix required" if namespace_prefix.nil? || namespace_prefix.to_s.empty?
+
+      NamespacedTags.new(self, namespace_prefix.to_s)
+    end
+
+    # Tag proxy bound to one namespace prefix. method_missing maps
+    # tag names to prefixed element creation; explicit Builder
+    # methods (text, element, ...) stay reachable by holding the
+    # builder — the proxy is tags-only by design.
+    class NamespacedTags
+      def initialize(builder, prefix)
+        @builder = builder
+        @prefix = prefix
+      end
+
+      def [](nested_prefix)
+        NamespacedTags.new(@builder, "#{@prefix}:#{nested_prefix}")
+      end
+
+      def method_missing(method_name, *args, &block)
+        # Trailing-underscore escape, same as plain Builder tags
+        # (type_ → type under the prefix).
+        tag = "#{@prefix}:#{method_name.to_s.chomp('_')}"
+        if @builder.strict?
+          raise NoMethodError,
+                "builder strict mode: unknown tag #{tag.inspect} " \
+                "would have silently created an empty element (#371)"
+        end
+
+        text_content = args.first.is_a?(String) ? args.shift : nil
+        attrs = args.first.is_a?(Hash) ? args.shift : {}
+        unless args.empty?
+          raise ArgumentError,
+                "unexpected arguments for #{tag}: #{args.inspect}"
+        end
+        if text_content && block
+          raise ArgumentError,
+                "#{tag}: cannot combine text content with a block"
+        end
+
+        @builder.create_tag(tag, attrs,
+                            text_content: text_content, block: block)
+      end
+
+      def respond_to_missing?(_method_name, _include_private = false)
+        true
+      end
+    end
+
     # Element ops inside a face-accumulated subtree cannot touch the
     # tree yet (nothing exists until the flush): PIs ride the raw
     # hatch in exact position; namespace/entity/declaration/doctype
@@ -139,6 +210,13 @@ module Moxml
     def method_missing(method_name, *args, &block)
       return super if RESERVED_METHOD_PATTERN.match?(method_name.to_s)
 
+      if @strict
+        raise NoMethodError,
+              "builder strict mode: unknown method #{method_name.inspect} " \
+              "would have silently created an empty <#{method_name}> element " \
+              "(#371); call a Builder method or use the explicit element face"
+      end
+
       text_content = args.first.is_a?(String) ? args.shift : nil
       attrs = args.first.is_a?(Hash) ? args.shift : {}
 

data/lib/moxml/config.rb

@@ -114,6 +114,7 @@ module Moxml
 
     attr_reader :adapter_name, :default_line_ending
     attr_accessor :strict_parsing,
+                  :builder_strict,
                   :default_encoding,
                   :default_indent,
                   :restore_entities,
@@ -173,6 +174,11 @@ module Moxml
       @entity_provider = nil
       @namespace_validation_mode = :strict
       @entity_restoration_mode = :lenient
+      # #371: Builder method_missing creates elements for unknown
+      # methods (the DSL contract). builder_strict flips that to
+      # raising NoMethodError — external helper calls stop
+      # silently becoming empty elements.
+      @builder_strict = false
       @entity_mode = :expand
     end
 

data/lib/moxml/context.rb

@@ -275,8 +275,11 @@ module Moxml
       config.adapter.sax_parse(xml, handler)
     end
 
-    def build(&block)
-      Builder.new(self).build(&block)
+    # strict: (issue #371) flips unknown method_missing calls to
+    # NoMethodError instead of silently creating empty elements —
+    # nil defers to config.builder_strict.
+    def build(strict: nil, &block)
+      Builder.new(self, strict: strict).build(&block)
     end
 
     # Frozen serialization defaults, memoized per context — Node#to_xml

data/lib/moxml/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Moxml
-  VERSION = "0.5.131"
+  VERSION = "0.5.132"
 end

data/spec/moxml/builder_namespaces_spec.rb

@@ -0,0 +1,95 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+# #371: Builder sharp edges — the namespace-scoped tag proxy
+# (Nokogiri's xml["p"].tag parity) and the strict mode where
+# unknown methods raise instead of silently becoming empty
+# elements.
+RSpec.describe "Builder namespaces + strict mode (#371)" do
+  describe "Builder#[] namespace-scoped tags" do
+    it "creates prefixed elements with text and attributes" do
+      doc = Moxml.new.build do
+        element("root") do
+          self["w"].p("text", "a" => 1)
+        end
+      end
+      expect(doc.root.children.first.name).to eq("w:p")
+      expect(doc.to_xml).to include(%(<w:p a="1">text</w:p>))
+    end
+
+    it "nests blocks in the caller context" do
+      doc = Moxml.new.build do
+        element("root") do
+          self["w"].sub { self["w"].deep("x") }
+        end
+      end
+      sub = doc.root.children.first
+      expect(sub.name).to eq("w:sub")
+      expect(sub.children.first.name).to eq("w:deep")
+      expect(sub.children.first.text).to eq("x")
+    end
+
+    it "strips the trailing-underscore escape under the prefix" do
+      doc = Moxml.new.build do
+        element("root") { self["epub"].type_ }
+      end
+      expect(doc.to_xml).to include("<epub:type>")
+    end
+
+    it "composes prefixes through nested []" do
+      doc = Moxml.new.build do
+        element("root") { self["w"]["ext"].node }
+      end
+      expect(doc.to_xml).to include("<w:ext:node>")
+    end
+
+    it "rejects a missing prefix" do
+      expect { Moxml.new.build { self[nil] } }
+        .to raise_error(ArgumentError, /prefix/)
+    end
+  end
+
+  describe "strict mode" do
+    it "raises NoMethodError for unknown methods (the silent-corruption fix)" do
+      expect do
+        Moxml.new.build(strict: true) do
+          element("root") { build_chart_type_unused }
+        end
+      end.to raise_error(NoMethodError, /build_chart_type_unused/)
+    end
+
+    it "raises for unknown namespaced tags too" do
+      expect do
+        Moxml.new.build(strict: true) do
+          element("root") { self["w"].chart_helper }
+        end
+      end.to raise_error(NoMethodError, /w:chart_helper/)
+    end
+
+    it "still builds known tags under strict" do
+      doc = Moxml.new.build(strict: true) do
+        element("root") { text "ok" }
+      end
+      expect(doc.to_xml).to include("<root>ok</root>")
+    end
+
+    it "defaults off and honors config.builder_strict" do
+      expect(Moxml.new.config.builder_strict).to be(false)
+      ctx = Moxml.new
+      ctx.config.builder_strict = true
+      expect do
+        ctx.build { element("root") { helperish } }
+      end.to raise_error(NoMethodError)
+    end
+
+    it "a per-build false overrides a strict config" do
+      ctx = Moxml.new
+      ctx.config.builder_strict = true
+      doc = ctx.build(strict: false) do
+        element("root") { helperish }
+      end
+      expect(doc.to_xml).to include("<helperish>")
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: moxml
 version: !ruby/object:Gem::Version
-  version: 0.5.131
+  version: 0.5.132
 platform: ruby
 authors:
 - Ribose Inc.
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-10-12 00:00:00.000000000 Z
+date: 2026-10-13 00:00:00.000000000 Z
 dependencies: []
 description: |
   Moxml is a unified XML manipulation library that provides a common API
@@ -473,6 +473,7 @@ files:
 - spec/moxml/at_search_spec.rb
 - spec/moxml/attribute_resolver_spec.rb
 - spec/moxml/attribute_spec.rb
+- spec/moxml/builder_namespaces_spec.rb
 - spec/moxml/builder_spec.rb
 - spec/moxml/c14n/adapter_delegation_spec.rb
 - spec/moxml/c14n/api_spec.rb