RubyGems - epub_tools - Versions diffs - 0.4.1 → 0.5.0 - Mend

epub_tools 0.4.1 → 0.5.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (51) hide show

checksums.yaml +4 -4
data/.github/workflows/ci.yml +3 -0
data/.rubocop.yml +10 -17
data/CLAUDE.md +124 -0
data/Gemfile +4 -4
data/Gemfile.lock +39 -34
data/Rakefile +2 -0
data/bin/epub-tools +2 -0
data/epub_tools.gemspec +3 -1
data/lib/epub_tools/add_chapters.rb +47 -29
data/lib/epub_tools/chapter_validator.rb +40 -0
data/lib/epub_tools/cli/command_options_configurator.rb +115 -0
data/lib/epub_tools/cli/command_registry.rb +2 -0
data/lib/epub_tools/cli/option_builder.rb +5 -3
data/lib/epub_tools/cli/runner.rb +59 -110
data/lib/epub_tools/cli.rb +16 -29
data/lib/epub_tools/compile_book.rb +48 -65
data/lib/epub_tools/compile_workspace.rb +40 -0
data/lib/epub_tools/epub_configuration.rb +33 -0
data/lib/epub_tools/epub_file_writer.rb +57 -0
data/lib/epub_tools/epub_initializer.rb +83 -162
data/lib/epub_tools/epub_metadata_builder.rb +92 -0
data/lib/epub_tools/loggable.rb +2 -0
data/lib/epub_tools/pack_ebook.rb +28 -14
data/lib/epub_tools/split_chapters.rb +42 -17
data/lib/epub_tools/style_finder.rb +17 -6
data/lib/epub_tools/unpack_ebook.rb +20 -10
data/lib/epub_tools/version.rb +3 -1
data/lib/epub_tools/xhtml_cleaner.rb +1 -0
data/lib/epub_tools/xhtml_extractor.rb +20 -10
data/lib/epub_tools/xhtml_generator.rb +71 -0
data/lib/epub_tools.rb +2 -0
data/test/add_chapters_test.rb +49 -25
data/test/chapter_validator_test.rb +47 -0
data/test/cli/command_registry_test.rb +2 -0
data/test/cli/option_builder_test.rb +24 -14
data/test/cli/runner_test.rb +15 -15
data/test/cli_commands_test.rb +2 -0
data/test/cli_test.rb +2 -0
data/test/cli_version_test.rb +2 -0
data/test/compile_book_test.rb +17 -102
data/test/compile_workspace_test.rb +55 -0
data/test/epub_initializer_test.rb +55 -27
data/test/pack_ebook_test.rb +33 -9
data/test/split_chapters_test.rb +27 -7
data/test/style_finder_test.rb +2 -0
data/test/test_helper.rb +2 -0
data/test/unpack_ebook_test.rb +45 -20
data/test/xhtml_cleaner_test.rb +2 -0
data/test/xhtml_extractor_test.rb +3 -1
metadata +13 -3

data/lib/epub_tools/epub_initializer.rb CHANGED Viewed

@@ -1,8 +1,12 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 require 'fileutils'
-require 'time'
-require 'securerandom'
 require_relative 'loggable'
+require_relative 'xhtml_generator'
+require_relative 'epub_metadata_builder'
+require_relative 'epub_file_writer'
+require_relative 'epub_configuration'
 module EpubTools
   # Sets up a basic empty EPUB directory structure with the basic files created:
@@ -15,6 +19,7 @@ module EpubTools
   # - cover image (optionally)
   class EpubInitializer
     include Loggable
     # Initializes the class
     # @param options [Hash] Configuration options
     # @option options [String] :title Book title (required)
@@ -23,197 +28,113 @@ module EpubTools
     # @option options [String] :cover_image Optional path to the cover image
     # @option options [Boolean] :verbose Whether to print progress to STDOUT (default: false)
     def initialize(options = {})
-      @title = options.fetch(:title)
-      @author = options.fetch(:author)
-      @destination = File.expand_path(options.fetch(:destination))
-      @uuid = "urn:uuid:#{SecureRandom.uuid}"
-      @modified = Time.now.utc.iso8601
-      @cover_image_path = options[:cover_image]
-      @cover_image_fname = nil
-      @cover_image_media_type = nil
-      @verbose = options[:verbose] || false
+      @config = EpubConfiguration.new(options)
+      @verbose = @config.verbose
+      @xhtml_generator = create_xhtml_generator
+      @metadata_builder = create_metadata_builder
+      @file_writer = create_file_writer
     end
+    private
+    def create_xhtml_generator
+      XhtmlGenerator.new(title: @config.title, author: @config.author)
+    end
+    def create_metadata_builder
+      EpubMetadataBuilder.new(@config)
+    end
+    def create_file_writer
+      EpubFileWriter.new(@config.destination)
+    end
+    public
     # Creates the empty ebook and returns the directory
     def run
-      create_structure
-      write_mimetype
+      @file_writer.create_structure
+      @file_writer.write_mimetype
       write_title_page
-      write_container
-      write_cover if @cover_image_path
+      @file_writer.write_container
+      write_cover if @config.cover_image_path
       write_package_opf
       write_nav
-      write_style
-      log "Created empty ebook structure at: #{@destination}"
-      @destination
+      @file_writer.write_style
+      log "Created empty ebook structure at: #{@config.destination}"
+      @config.destination
     end
     private
-    def create_structure
-      FileUtils.mkdir_p("#{@destination}/META-INF")
-      FileUtils.mkdir_p("#{@destination}/OEBPS")
+    def write_title_page
+      content = @xhtml_generator.build_title_page
+      @file_writer.write_xhtml('title.xhtml', content)
     end
-    def write_mimetype
-      File.write("#{@destination}/mimetype", 'application/epub+zip')
-    end
+    # Copies the cover image into the EPUB structure and creates a cover.xhtml page
+    def write_cover
+      return unless cover_image_exists?
-    def write_title_page
-      content = <<~XHTML
-        <?xml version="1.0" encoding="UTF-8"?>
-        <html xmlns="http://www.w3.org/1999/xhtml" lang="en">
-          <head>
-            <meta charset="UTF-8" />
-            <title>#{@title}</title>
-            <link rel="stylesheet" type="text/css" href="style.css"/>
-          </head>
-          <body>
-            <h1 class="title">#{@title}</h1>
-            <p class="author">by #{@author}</p>
-          </body>
-        </html>
-      XHTML
-      File.write("#{@destination}/OEBPS/title.xhtml", content)
+      ext = File.extname(@config.cover_image_path).downcase
+      media_type = determine_media_type(ext)
+      return unless media_type
+      fname = "cover#{ext}"
+      @config.update_cover_info(fname, media_type)
+      @xhtml_generator.cover_image_fname = fname
+      update_metadata_builder_with_cover_info
+      copy_cover_image(ext)
+      write_cover_page
     end
-    def write_container
-      content = <<~XML
-        <?xml version="1.0" encoding="UTF-8"?>
-        <container version="1.0" xmlns="urn:oasis:names:tc:opendocument:xmlns:container">
-          <rootfiles>
-            <rootfile full-path="OEBPS/package.opf" media-type="application/oebps-package+xml"/>
-          </rootfiles>
-        </container>
-      XML
-      File.write("#{@destination}/META-INF/container.xml", content)
+    def cover_image_exists?
+      return true if File.exist?(@config.cover_image_path)
+      warn "Warning: cover image '#{@config.cover_image_path}' not found; skipping cover support."
+      false
     end
-    # Copies the cover image into the EPUB structure and creates a cover.xhtml page
-    def write_cover
-      path = @cover_image_path
-      unless File.exist?(path)
-        warn "Warning: cover image '#{path}' not found; skipping cover support."
-        return
+    def determine_media_type(ext)
+      case ext
+      when '.jpg', '.jpeg' then 'image/jpeg'
+      when '.png' then 'image/png'
+      when '.gif' then 'image/gif'
+      when '.svg' then 'image/svg+xml'
+      else
+        warn "Warning: unsupported cover image type '#{ext}'; skipping cover support."
+        nil
       end
-      ext = File.extname(path).downcase
-      @cover_image_media_type = case ext
-                                when '.jpg', '.jpeg' then 'image/jpeg'
-                                when '.png' then 'image/png'
-                                when '.gif' then 'image/gif'
-                                when '.svg' then 'image/svg+xml'
-                                else
-                                  warn "Warning: unsupported cover image type '#{ext}'; skipping cover support."
-                                  return
-                                end
-      @cover_image_fname = "cover#{ext}"
-      dest = File.join(@destination, 'OEBPS', @cover_image_fname)
-      FileUtils.cp(path, dest)
-      write_cover_page
     end
-    # Generates a cover.xhtml file displaying the cover image
-    def write_cover_page
-      content = <<~XHTML
-        <?xml version="1.0" encoding="UTF-8"?>
-        <html xmlns="http://www.w3.org/1999/xhtml" lang="en">
-          <head>
-            <meta charset="UTF-8" />
-            <title>Cover</title>
-            <link rel="stylesheet" type="text/css" href="style.css"/>
-          </head>
-          <body>
-            <div class="cover-image">
-              <img src="#{@cover_image_fname}" alt="Cover"/>
-            </div>
-          </body>
-        </html>
-      XHTML
-      File.write(File.join(@destination, 'OEBPS', 'cover.xhtml'), content)
+    def update_metadata_builder_with_cover_info
+      @metadata_builder = EpubMetadataBuilder.new(@config)
     end
-    def mitem(id, href, type, properties = nil)
-      xml = "<item id=\"#{id}\" href=\"#{href}\" media-type=\"#{type}\""
-      xml += " properties=\"#{properties}\"" if properties
-      "#{xml}/>"
+    def copy_cover_image(_ext)
+      dest = File.join(@config.destination, 'OEBPS', @config.cover_image_fname)
+      FileUtils.cp(@config.cover_image_path, dest)
+    end
+    # Generates a cover.xhtml file displaying the cover image
+    def write_cover_page
+      content = @xhtml_generator.build_cover_page
+      @file_writer.write_xhtml('cover.xhtml', content)
     end
     # Generates the package.opf with optional cover image entries
     def write_package_opf
-      manifest_items = []
-      spine_items = []
-      manifest_items << mitem('style', 'style.css', 'text/css')
-      manifest_items << mitem('nav', 'nav.xhtml', 'application/xhtml+xml', 'nav')
-      if @cover_image_fname
-        manifest_items << mitem('cover-image', @cover_image_fname, @cover_image_media_type, 'cover-image')
-        manifest_items << mitem('cover-page', 'cover.xhtml', 'application/xhtml+xml')
-        spine_items << '<itemref idref="cover-page"/>'
-      end
-      manifest_items << mitem('title', 'title.xhtml', 'application/xhtml+xml')
-      spine_items << '<itemref idref="title"/>'
-      metadata = []
-      metadata << %(<dc:identifier id="pub-id">#{@uuid}</dc:identifier>)
-      metadata << %(<dc:title>#{@title}</dc:title>)
-      metadata << %(<dc:creator>#{@author}</dc:creator>)
-      metadata << '<dc:language>en</dc:language>'
-      metadata << %(<meta property="dcterms:modified">#{@modified}</meta>)
-      metadata << %(<meta property="schema:accessMode">textual</meta>)
-      metadata << %(<meta property="schema:accessibilityFeature">unknown</meta>)
-      metadata << %(<meta property="schema:accessibilityHazard">none</meta>)
-      metadata << %(<meta property="schema:accessModeSufficient">textual</meta>)
-      metadata << %(<meta name="cover" content="cover-image"/>) if @cover_image_fname
-      content = <<~XML
-        <?xml version="1.0" encoding="utf-8"?>
-        <package xmlns="http://www.idpf.org/2007/opf" version="3.0" unique-identifier="pub-id" xml:lang="en">
-          <metadata xmlns:dc="http://purl.org/dc/elements/1.1/">
-        #{metadata.map { |line| "    #{line}" }.join("\n")}
-          </metadata>
-          <manifest>
-        #{manifest_items.map { |line| "    #{line}" }.join("\n")}
-          </manifest>
-          <spine>
-        #{spine_items.map { |line| "    #{line}" }.join("\n")}
-          </spine>
-        </package>
-      XML
-      File.write(File.join(@destination, 'OEBPS', 'package.opf'), content)
+      manifest_items, spine_items = @metadata_builder.build_manifest_and_spine
+      metadata = @metadata_builder.build_metadata
+      content = @metadata_builder.build_opf_xml(metadata, manifest_items, spine_items)
+      @file_writer.write_package_opf(content)
     end
     # Generates the initial navigation document (Table of Contents)
     def write_nav
-      content = <<~XHTML
-        <?xml version="1.0" encoding="utf-8"?>
-        <html xmlns="http://www.w3.org/1999/xhtml" xmlns:epub="http://www.idpf.org/2007/ops" lang="en">
-          <head>
-            <title>Table of Contents</title>
-          </head>
-          <body>
-            <nav epub:type="toc" id="toc">
-              <h1>Table of Contents</h1>
-              <ol>
-                <li><a href="title.xhtml">Title Page</a></li>
-              </ol>
-            </nav>
-          </body>
-        </html>
-      XHTML
-      File.write(File.join(@destination, 'OEBPS', 'nav.xhtml'), content)
-    end
-    def write_style
-      src = File.join(Dir.pwd, 'style.css')
-      dest = File.join(@destination, 'OEBPS', 'style.css')
-      unless File.exist?(src)
-        warn "Warning: style.css not found in project root (#{src}), skipping copy."
-        return
-      end
-      FileUtils.cp(src, dest)
+      content = @xhtml_generator.build_nav_page
+      @file_writer.write_xhtml('nav.xhtml', content)
     end
   end
 end

data/lib/epub_tools/epub_metadata_builder.rb ADDED Viewed

@@ -0,0 +1,92 @@
+# frozen_string_literal: true
+module EpubTools
+  # Builds metadata content for EPUB package.opf files
+  class EpubMetadataBuilder
+    def initialize(config)
+      @config = config
+    end
+    # Builds complete metadata array
+    def build_metadata
+      metadata = []
+      add_dublin_core_metadata(metadata)
+      add_schema_metadata(metadata)
+      add_cover_metadata(metadata) if @config.cover_image_fname
+      metadata
+    end
+    # Builds manifest and spine items
+    def build_manifest_and_spine
+      manifest_items = []
+      spine_items = []
+      add_base_manifest_items(manifest_items)
+      add_cover_items(manifest_items, spine_items) if @config.cover_image_fname
+      add_title_items(manifest_items, spine_items)
+      [manifest_items, spine_items]
+    end
+    # Builds complete OPF XML content
+    def build_opf_xml(metadata, manifest_items, spine_items)
+      <<~XML
+        <?xml version="1.0" encoding="utf-8"?>
+        <package xmlns="http://www.idpf.org/2007/opf" version="3.0" unique-identifier="pub-id" xml:lang="en">
+          <metadata xmlns:dc="http://purl.org/dc/elements/1.1/">
+        #{metadata.map { |line| "    #{line}" }.join("\n")}
+          </metadata>
+          <manifest>
+        #{manifest_items.map { |line| "    #{line}" }.join("\n")}
+          </manifest>
+          <spine>
+        #{spine_items.map { |line| "    #{line}" }.join("\n")}
+          </spine>
+        </package>
+      XML
+    end
+    private
+    def add_dublin_core_metadata(metadata)
+      metadata << %(<dc:identifier id="pub-id">#{@config.uuid}</dc:identifier>)
+      metadata << %(<dc:title>#{@config.title}</dc:title>)
+      metadata << %(<dc:creator>#{@config.author}</dc:creator>)
+      metadata << '<dc:language>en</dc:language>'
+      metadata << %(<meta property="dcterms:modified">#{@config.modified}</meta>)
+    end
+    def add_schema_metadata(metadata)
+      metadata << %(<meta property="schema:accessMode">textual</meta>)
+      metadata << %(<meta property="schema:accessibilityFeature">unknown</meta>)
+      metadata << %(<meta property="schema:accessibilityHazard">none</meta>)
+      metadata << %(<meta property="schema:accessModeSufficient">textual</meta>)
+    end
+    def add_cover_metadata(metadata)
+      metadata << %(<meta name="cover" content="cover-image"/>)
+    end
+    def add_base_manifest_items(manifest_items)
+      manifest_items << mitem('style', 'style.css', 'text/css')
+      manifest_items << mitem('nav', 'nav.xhtml', 'application/xhtml+xml', 'nav')
+    end
+    def add_cover_items(manifest_items, spine_items)
+      manifest_items << mitem('cover-image', @config.cover_image_fname, @config.cover_image_media_type, 'cover-image')
+      manifest_items << mitem('cover-page', 'cover.xhtml', 'application/xhtml+xml')
+      spine_items << '<itemref idref="cover-page"/>'
+    end
+    def add_title_items(manifest_items, spine_items)
+      manifest_items << mitem('title', 'title.xhtml', 'application/xhtml+xml')
+      spine_items << '<itemref idref="title"/>'
+    end
+    def mitem(id, href, type, properties = nil)
+      xml = "<item id=\"#{id}\" href=\"#{href}\" media-type=\"#{type}\""
+      xml += " properties=\"#{properties}\"" if properties
+      "#{xml}/>"
+    end
+  end
+end

data/lib/epub_tools/loggable.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 module EpubTools
   # Provides logging capability to classes that include it
   module Loggable

data/lib/epub_tools/pack_ebook.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 require 'zip'
 require 'fileutils'
 require 'pathname'
@@ -7,6 +9,7 @@ module EpubTools
   # Packages an EPUB directory into a .epub file
   class PackEbook
     include Loggable
     # Initializes the class
     # @param options [Hash] Configuration options
     # @option options [String] :input_dir Path to the EPUB directory (containing mimetype, META-INF, OEBPS) (required)
@@ -28,21 +31,9 @@ module EpubTools
     def run
       validate_input!
       Dir.chdir(@input_dir) do
-        # determine the output path: absolute stays as-is, otherwise sibling to input_dir
-        target = Pathname.new(@output_file).absolute? ? @output_file : File.join('..', @output_file)
+        target = determine_output_path
         FileUtils.rm_f(target)
-        Zip::File.open(target, Zip::File::CREATE) do |zip|
-          # Add mimetype first and uncompressed
-          add_mimetype(zip)
-          # Add all other files with compression, preserving paths
-          Dir.glob('**/*', File::FNM_DOTMATCH).sort.each do |entry|
-            next if ['.', '..', 'mimetype'].include?(entry)
-            next if File.directory?(entry)
-            zip.add(entry, entry)
-          end
-        end
+        create_zip_file(target)
       end
       log "EPUB created: #{@output_file}"
       @output_file
@@ -50,6 +41,29 @@ module EpubTools
     private
+    def determine_output_path
+      # determine the output path: absolute stays as-is, otherwise sibling to input_dir
+      Pathname.new(@output_file).absolute? ? @output_file : File.join('..', @output_file)
+    end
+    def create_zip_file(target)
+      Zip::File.open(target, create: true) do |zip|
+        # Add mimetype first and uncompressed
+        add_mimetype(zip)
+        add_content_files(zip)
+      end
+    end
+    def add_content_files(zip)
+      # Add all other files with compression, preserving paths
+      Dir.glob('**/*', File::FNM_DOTMATCH).sort.each do |entry|
+        next if ['.', '..', 'mimetype'].include?(entry)
+        next if File.directory?(entry)
+        zip.add(entry, entry)
+      end
+    end
     def validate_input!
       raise ArgumentError, "Directory '#{@input_dir}' does not exist." unless Dir.exist?(@input_dir)

data/lib/epub_tools/split_chapters.rb CHANGED Viewed

@@ -1,4 +1,6 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 require 'nokogiri'
 require 'yaml'
 require 'fileutils'
@@ -16,6 +18,7 @@ module EpubTools
   # - Saves those files to +output_dir+
   class SplitChapters
     include Loggable
     # Initializes the class
     # @param options [Hash] Configuration options
     # @option options [String] :input_file Path to the source XHTML (required)
@@ -60,21 +63,39 @@ module EpubTools
       current_fragment = nil
       doc.at('body').children.each do |node|
-        if (m = node.text.match(/Chapter\s+(\d+)/i)) && %w[p span h2 h3 h4].include?(node.name)
-          # start a new chapter (skip the marker node so title isn't duplicated)
-          chapters[current_number] = current_fragment.to_html if current_number
-          current_number = m[1].to_i
-          current_fragment = Nokogiri::HTML::DocumentFragment.parse('')
-        elsif prologue_marker?(node)
-          # start the prologue (skip the marker node)
-          chapters[current_number] = current_fragment.to_html if current_number
-          current_number = 0
-          current_fragment = Nokogiri::HTML::DocumentFragment.parse('')
-        else
-          current_fragment&.add_child(node.dup)
-        end
+        current_number, current_fragment = process_node(node, chapters, current_number, current_fragment)
+      end
+      finalize_chapters(chapters, current_number, current_fragment)
+    end
+    def process_node(node, chapters, current_number, current_fragment)
+      if chapter_marker?(node)
+        start_new_chapter(chapters, node, current_number, current_fragment)
+      elsif prologue_marker?(node)
+        start_prologue(chapters, current_number, current_fragment)
+      else
+        current_fragment&.add_child(node.dup)
+        [current_number, current_fragment]
       end
+    end
+    def chapter_marker?(node)
+      node.text.match?(/Chapter\s+\d+/i) && %w[p span h2 h3 h4].include?(node.name)
+    end
+    def start_new_chapter(chapters, node, current_number, current_fragment)
+      chapters[current_number] = current_fragment.to_html if current_number
+      chapter_number = node.text.match(/Chapter\s+(\d+)/i)[1].to_i
+      [chapter_number, Nokogiri::HTML::DocumentFragment.parse('')]
+    end
+    def start_prologue(chapters, current_number, current_fragment)
+      chapters[current_number] = current_fragment.to_html if current_number
+      [0, Nokogiri::HTML::DocumentFragment.parse('')]
+    end
+    def finalize_chapters(chapters, current_number, current_fragment)
       chapters[current_number] = current_fragment.to_html if current_number
       chapters
     end
@@ -91,7 +112,14 @@ module EpubTools
     def write_chapter_file(label, content)
       display_label = display_label(label)
       filename = File.join(@output_dir, "#{@output_prefix}_#{label}.xhtml")
-      File.write(filename, <<~HTML)
+      File.write(filename, build_xhtml_template(display_label, content))
+      XHTMLCleaner.new({ filename: filename }).run
+      log("Extracted: #{filename}")
+      filename
+    end
+    def build_xhtml_template(display_label, content)
+      <<~HTML
         <?xml version="1.0" encoding="UTF-8"?>
         <html xmlns="http://www.w3.org/1999/xhtml" lang="en">
           <head>
@@ -104,9 +132,6 @@ module EpubTools
           </body>
         </html>
       HTML
-      XHTMLCleaner.new({ filename: filename }).run
-      log("Extracted: #{filename}")
-      filename
     end
     def display_label(label)

data/lib/epub_tools/style_finder.rb CHANGED Viewed

@@ -1,4 +1,6 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 require 'nokogiri'
 require 'yaml'
 require_relative 'loggable'
@@ -9,6 +11,7 @@ module EpubTools
   # {SplitChapters}[rdoc-ref:EpubTools::SplitChapters].
   class StyleFinder
     include Loggable
     # Initializes the class
     # @param options [Hash] Configuration options
     # @option options [String] :file_path XHTML file to be analyzed (required)
@@ -24,18 +27,26 @@ module EpubTools
     # Runs the finder
     # @return [Hash] Data containing the extracted style classes (italics and bolds)
     def run
+      style_blocks = extract_style_blocks
+      italics, bolds = extract_style_classes(style_blocks)
+      generate_output(italics, bolds)
+    end
+    def extract_style_blocks
       doc = Nokogiri::HTML(File.read(@file_path))
-      style_blocks = doc.xpath('//style').map(&:text).join("\n")
+      doc.xpath('//style').map(&:text).join("\n")
+    end
+    def extract_style_classes(style_blocks)
       italics = extract_classes(style_blocks, /font-style\s*:\s*italic/)
-      bolds   = extract_classes(style_blocks, /font-weight\s*:\s*700/)
+      bolds = extract_classes(style_blocks, /font-weight\s*:\s*700/)
+      [italics, bolds]
+    end
+    def generate_output(italics, bolds)
       print_summary(italics, bolds) if @verbose
-      data = {
-        'italics' => italics,
-        'bolds' => bolds
-      }
+      data = { 'italics' => italics, 'bolds' => bolds }
       File.write(@output_path, data.to_yaml)
       data
     end

data/lib/epub_tools/unpack_ebook.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 require 'zip'
 require 'fileutils'
 require_relative 'loggable'
@@ -6,6 +8,7 @@ module EpubTools
   # Unpacks an EPUB (.epub file) into a directory
   class UnpackEbook
     include Loggable
     # Initializes the class
     # @param options [Hash] Configuration options
     # @option options [String] :epub_file Path to the .epub file to unpack (required)
@@ -23,22 +26,29 @@ module EpubTools
     def run
       validate!
       FileUtils.mkdir_p(@output_dir)
+      extract_entries
+      log "Unpacked #{File.basename(@epub_file)} to #{@output_dir}"
+      @output_dir
+    end
+    private
+    def extract_entries
       Zip::File.open(@epub_file) do |zip|
         zip.each do |entry|
-          dest_path = File.join(@output_dir, entry.name)
-          if entry.directory?
-            FileUtils.mkdir_p(dest_path)
-          else
-            FileUtils.mkdir_p(File.dirname(dest_path))
-            entry.extract(dest_path) { true }
-          end
+          extract_entry(entry)
         end
       end
-      log "Unpacked #{File.basename(@epub_file)} to #{@output_dir}"
-      @output_dir
     end
-    private
+    def extract_entry(entry)
+      if entry.directory?
+        FileUtils.mkdir_p(File.join(@output_dir, entry.name))
+      else
+        FileUtils.mkdir_p(File.join(@output_dir, File.dirname(entry.name)))
+        entry.extract(destination_directory: @output_dir) { true }
+      end
+    end
     def default_dir
       [File.dirname(@epub_file), File.basename(@epub_file, '.epub')].join('/')

data/lib/epub_tools/version.rb CHANGED Viewed

@@ -1,4 +1,6 @@
+# frozen_string_literal: true
 module EpubTools
   # Ruby Gem version number
-  VERSION = '0.4.1'.freeze
+  VERSION = '0.5.0'
 end

data/lib/epub_tools/xhtml_cleaner.rb CHANGED Viewed

@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 require 'nokogiri'
 require 'yaml'