idml 0.2.1 → 0.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1b87a74424cdcbf39668b9d617232a939d3d93b5bdc925d9b5ff48a327a2b1f2
-  data.tar.gz: 3059b9c662c5b979b82989899e0a2048e45b510d96ba4006a1c384ee7688bec1
+  metadata.gz: 52c92d7618628647f3ad6a9a791f4e5236514463b25c860390a046b5538df11d
+  data.tar.gz: 16786debbbceae219fc8933d96ad06b02c73f2efae0f4f03162deefd4d98c112
 SHA512:
-  metadata.gz: 9eabdd81a468d04168b9c5602e3f3c547753430b5e7467c71805f7d74d3e976073ffce7cd85209cbdd2b7f06b83b485c9cc00c070ddbf783e471364accb604e7
-  data.tar.gz: 88894e18b9f58717958cfbba80021b73a6e3f6b2accdab35993ff82d29312450730d6e255ff906e0c07f61272a63205169cbb75192845080c6226427250aef18
+  metadata.gz: 10756f2694a6e5b7d6543552d03f33770bb43ceaf085f3365348f919ae7ba54b9f45f2073cfaadad96d3679d93a966a0e67e758e667f5999a810780756a0a15a
+  data.tar.gz: 8efeee5edb697daee0839492011ae90a28c0db9edf155ea417c75b58fe353105733d93c30a73a32e3d63e5442957198e3dccee2d6b3276f76d303bf5eac383da

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    idml (0.2.1)
+    idml (0.2.3)
       bigdecimal
       fontisan
       lutaml-model (~> 0.8.18)
@@ -12,7 +12,7 @@ PATH
 PATH
   remote: /Users/mulgogi/src/claricle/pdfrb
   specs:
-    pdfrb (0.18.0)
+    pdfrb (0.19.0)
       lutaml-model (>= 0.7.0)
       thor
 
@@ -204,7 +204,7 @@ CHECKSUMS
   ffi (1.17.4-x86_64-linux-gnu) sha256=9d3db14c2eae074b382fa9c083fe95aec6e0a1451da249eab096c34002bc752d
   ffi (1.17.4-x86_64-linux-musl) sha256=3fdf9888483de005f8ef8d1cf2d3b20d86626af206cbf780f6a6a12439a9c49e
   fontisan (0.4.45) sha256=2337ca0205f806647b24b8cac57aa34f3769473026ae6f52dc36bde0feed6c62
-  idml (0.2.1)
+  idml (0.2.3)
   json (2.21.2) sha256=1f1d3b7cf2b3ba1a69beca0bb6db13d5438b80bff3cd54cdaaa620b9b07c1c6a
   language_server-protocol (3.17.0.6) sha256=5ef2c0c138f8267e1bc631d3328347d354f96724b0af22f2c79516120443b7f0
   lint_roller (1.1.0) sha256=2c0c845b632a7d172cb849cc90c1bce937a28c5c8ccccb50dfd46a485003cc87
@@ -226,7 +226,7 @@ CHECKSUMS
   paint (2.3.0) sha256=327d623e4038619d5bd99ae5db07973859cd78400c7f0329eea283cef8e83be5
   parallel (2.1.0) sha256=b35258865c2e31134c5ecb708beaaf6772adf9d5efae28e93e99260877b09356
   parser (3.3.12.0) sha256=21a6d7f755d5a24dfbdc6e6b772e4e879a52e7631a88bc5a3a134606052c9828
-  pdfrb (0.18.0)
+  pdfrb (0.19.0)
   prism (1.9.0) sha256=7b530c6a9f92c24300014919c9dcbc055bf4cdf51ec30aed099b06cd6674ef85
   racc (1.8.1) sha256=4a7f6929691dbec8b5209a0b373bc2614882b55fc5d2e447a21aaa691303d62f
   rainbow (3.1.1) sha256=039491aa3a89f42efa1d6dec2fc4e62ede96eb6acd95e52f1ad581182b79bc6a

data/TODO.pdf/64-pipeline-cli-optimizations.md

@@ -0,0 +1,23 @@
+# TODO PDF 64: Pipeline optimizations and CLI enhancements
+
+## Status: DONE
+
+## What was implemented
+
+1. **Pipeline SpreadRenderer caching**: `render_spread_pages` creates
+   one SpreadRenderer per spread (was one per page). The renderer is
+   stateless and can be reused across pages within a spread.
+
+2. **CLI --verbose / -v flag**: Prints progress during rendering:
+   - Input and output paths
+   - Spread count
+   - Goes to stderr (doesn't interfere with stdout piping)
+
+3. **render_options extraction**: CLI's render_options hash extracted
+   as a method (DRY).
+
+## Acceptance criteria
+
+- [x] SpreadRenderer created once per spread (not per page)
+- [x] CLI --verbose flag prints progress to stderr
+- [x] All tests pass

data/TODO.pdf/65-pdfrb-019-integration.md

@@ -0,0 +1,29 @@
+# TODO PDF 65: pdfrb 0.19.0 feature integration
+
+## Status: PARTIALLY DONE
+
+## What was implemented
+
+1. **Canvas#text_lines**: TextFrameRenderer uses `canvas.text_lines`
+   instead of N separate `canvas.text` calls per line. Single batch
+   call for all lines within a run.
+
+## What remains (pdfrb bugs)
+
+2. **glyph_width for TTF**: pdfrb 0.19.0 has `Fonts#glyph_width` but
+   returns 0 for TrueType fonts (bug: "undefined method glyph_id_for
+   for String"). Standard 14 Type1 fonts work correctly. Cannot
+   replace FontMetrics/Fontisan until this is fixed.
+
+3. **Pdfrb::FontResolver**: Returns nil for common fonts (Helvetica.ttc
+   on macOS). Cannot replace Fontisan-based resolver yet.
+
+4. **Font subsetting**: `Fonts#add` accepts `**opts` but `subset: true`
+   behavior unverified. Standard 14 fonts don't need FontFile embedding.
+
+## Acceptance criteria
+
+- [x] TextFrameRenderer uses canvas.text_lines
+- [ ] FontMetrics replaced with pdfrb glyph_width (blocked by TTF bug)
+- [ ] FontResolver replaced with Pdfrb::FontResolver (blocked by .ttc)
+- [ ] Font subsetting verified

data/lib/idml/cli.rb

@@ -58,7 +58,7 @@ module Idml
       end
     end
 
-    desc "render PATH [-o OUTPUT] [--pdf-a] [--tagged] [--font-path DIR]",
+    desc "render PATH [-o OUTPUT] [--pdf-a] [--tagged] [-v]",
          "Convert IDML to PDF"
     method_option :output, aliases: "-o", type: :string, required: true
     method_option :font_path, type: :array, default: [],
@@ -66,9 +66,12 @@ module Idml
     method_option :pdf_a, type: :boolean, default: false,
                           desc: "Produce PDF/A-2a compliant output"
     method_option :tagged, type: :boolean, default: false,
-                           desc: "Produce tagged PDF for accessibility (PDF/UA)"
+                           desc: "Produce tagged PDF (PDF/UA)"
+    method_option :verbose, aliases: "-v", type: :boolean, default: false,
+                            desc: "Print progress"
     def render(path)
       pkg = package(path)
+      print_verbose(pkg) if options[:verbose]
       Idml::Render.render(package: pkg, to: options[:output],
                           **render_options)
       puts "Wrote #{options[:output]}"
@@ -97,6 +100,11 @@ module Idml
       }
     end
 
+    def print_verbose(pkg)
+      warn "Rendering to #{options[:output]}"
+      warn "  #{pkg.spreads.length} spread(s)"
+    end
+
     def print_validation_results(results)
       results.partition(&:ok?).then do |ok, failed|
         ok.each { |r| puts "  OK   #{r.part_name}" }

data/lib/idml/render/pipeline.rb

@@ -43,23 +43,27 @@ module Idml
                               font_ref_resolver, font_name)
         pages = spread.spread.flat_map(&:page)
         image_refs = collect_images(writer, spread, base_dir, layer_filter)
+        renderer = build_renderer(layer_filter, font_ref_resolver, font_name)
 
         pages.each do |page|
           dims = page_dimensions_for(page)
           canvas = writer.add_page(width: dims[:width], height: dims[:height])
-          renderer = SpreadRenderer.new(
-            font_resolver: @font_resolver,
-            font_ps_name: font_name,
-            package: @package,
-            layer_filter: layer_filter,
-            font_ref_resolver: font_ref_resolver,
-          )
           renderer.render(canvas, spread, page_width: dims[:width],
                                           page_height: dims[:height],
                                           image_refs: image_refs)
         end
       end
 
+      def build_renderer(layer_filter, font_ref_resolver, font_name)
+        SpreadRenderer.new(
+          font_resolver: @font_resolver,
+          font_ps_name: font_name,
+          package: @package,
+          layer_filter: layer_filter,
+          font_ref_resolver: font_ref_resolver,
+        )
+      end
+
       def collect_images(writer, spread, base_dir, layer_filter)
         refs = []
         spread.each_page_item do |item|

data/lib/idml/render/renderers/text_frame_renderer.rb

@@ -95,15 +95,21 @@ module Idml
             glyphs: glyphs, frame_width: box[:width],
           )
 
+          line_texts = []
           lines.each do |line|
             break if baseline_y < box[:y]
 
-            line_text = line.glyphs.map { |g| [g.codepoint].pack("U") }.join
-            canvas.text(line_text, at: [box[:x], baseline_y],
-                                   font: context.font_ps_name,
-                                   size: size)
+            line_texts << line.glyphs.map { |g| [g.codepoint].pack("U") }.join
             baseline_y -= size * LEADING_FACTOR
           end
+
+          if line_texts.any?
+            canvas.text_lines(line_texts,
+                              font: context.font_ps_name,
+                              size: size,
+                              at: [box[:x], box[:y] + box[:height] - size],
+                              leading: size * LEADING_FACTOR)
+          end
           baseline_y
         end
         private_class_method :render_run_lines

data/lib/idml/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Idml
-  VERSION = "0.2.1"
+  VERSION = "0.2.3"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: idml
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.3
 platform: ruby
 authors:
 - Ribose
@@ -204,6 +204,8 @@ files:
 - TODO.pdf/61-known-limitations.md
 - TODO.pdf/62-pdfrb-015-migration.md
 - TODO.pdf/63-replace-fontmetrics-with-pdfrb.md
+- TODO.pdf/64-pipeline-cli-optimizations.md
+- TODO.pdf/65-pdfrb-019-integration.md
 - TODO.pdf/README.md
 - exe/idml
 - idml.gemspec