mtg_card_maker 0.1.0 → 0.2.1

data/lib/mtg_card_maker/cli.rb

@@ -21,7 +21,7 @@ module MtgCardMaker
   #     --rules-text "Lightning Bolt deals 3 damage to any target." --color red
   #
   # @since 0.1.0
-  class CLI < Thor # rubocop:disable Metrics/ClassLength
+  class CLI < Thor
     package_name 'mtg_card_maker'
     map 'a'  => :add_card
     map 'ac' => :add_card
@@ -30,6 +30,9 @@ module MtgCardMaker
     map 'gcs'=> :generate_sprite
     map 'gs' => :generate_sprite
 
+    # Fail on misspelled options (for example --nmae) instead of ignoring them
+    check_unknown_options!
+
     def self.exit_on_failure?
       true
     end
@@ -39,11 +42,11 @@ module MtgCardMaker
     # @return [Hash] the card option definitions for Thor
     def self.card_options
       {
-        name: { type: :string, required: true, desc: 'Card name' },
+        name: { type: :string, desc: 'Card name (warns when omitted)' },
         mana_cost: { type: :string, aliases: ['mana', 'cost'], desc: 'Mana cost (e.g., "2RR", "XG")' },
-        type_line: { type: :string, required: true, aliases: ['type'],
-                     desc: 'Card type & subtype (e.g., "Creature - Dragon", "Instant")' },
-        rules_text: { type: :string, required: true, aliases: ['rules'], desc: 'Card rules text' },
+        type_line: { type: :string, aliases: ['type'],
+                     desc: 'Card type & subtype (e.g., "Creature - Dragon", "Instant"; warns when omitted)' },
+        rules_text: { type: :string, aliases: ['rules'], desc: 'Card rules text (optional)' },
         flavor_text: { type: :string, aliases: ['flavor'], desc: 'Flavor text (optional)' },
         power: { type: :string, desc: 'Power (for creatures)' },
         toughness: { type: :string, desc: 'Toughness (for creatures)' },
@@ -51,16 +54,17 @@ module MtgCardMaker
         color: { type: :string,
                  default: 'colorless',
                  desc: 'Card color (white, blue, black, red, green, colorless)' },
-        art: { type: :string, aliases: ['artwork', 'image'], desc: 'Image URL or path for card artwork' }
+        art: { type: :string, aliases: ['artwork', 'image'], desc: 'Image URL or path for card artwork' },
+        embed_font: { type: :boolean, default: true, desc: 'Embed font as base64 in SVG' }
       }
     end
 
     # Generate a single MTG card with specified parameters
     #
-    # @option options [String] :name the card name (required)
+    # @option options [String] :name the card name (optional, warns when omitted)
     # @option options [String] :mana_cost the mana cost in MTG notation
-    # @option options [String] :type_line the card type (required)
-    # @option options [String] :rules_text the card rules (required)
+    # @option options [String] :type_line the card type (optional, warns when omitted)
+    # @option options [String] :rules_text the card rules text (optional)
     # @option options [String] :flavor_text the flavor text
     # @option options [String] :power the power value for creatures
     # @option options [String] :toughness the toughness value for creatures
@@ -80,7 +84,7 @@ module MtgCardMaker
       config = config.transform_keys(&:to_sym)
       card = BaseCard.new(config)
       card.save(options[:output])
-      puts "✨ Generated #{options[:output]}! ✨"
+      logger.success("Generated #{options[:output]}!")
     end
 
     # @!method generate_sprite(yaml_file, output_file)
@@ -89,17 +93,11 @@ module MtgCardMaker
     option :cards_per_row, type: :numeric, default: 4, desc: 'Number of cards per row in sprite'
     option :spacing, type: :numeric, default: 30, desc: 'Spacing between cards in pixels'
     def generate_sprite(yaml_file, output_file)
-      validate_yaml_file(yaml_file)
-
-      begin
-        config = load_yaml_config(yaml_file)
-        sprite_service = create_sprite_service
-        process_sprite_generation(config, sprite_service, output_file)
-      rescue Psych::SyntaxError => e
-        handle_yaml_error(e)
-      rescue StandardError => e
-        handle_general_error(e)
-      end
+      abort "❌ YAML file not found: #{yaml_file}" unless File.exist?(yaml_file)
+
+      config = validate_yaml(yaml_file)
+      sprite_service = create_sprite_service
+      process_sprite_generation(config, sprite_service, output_file)
     end
 
     # @!method add_card(yaml_file)
@@ -107,11 +105,8 @@ module MtgCardMaker
     desc 'add_card YAML_FILE [OPTIONS]', 'Add a new card configuration to YAML file'
     card_options.each { |option, config| option option, config }
     def add_card(yaml_file)
-      # Create directory if it doesn't exist
-      FileUtils.mkdir_p(File.dirname(yaml_file))
-
-      # Load existing config or create new one
-      config = File.exist?(yaml_file) ? YAML.safe_load_file(yaml_file) : {}
+      # Create directory and load/validate config
+      config = create_directory_and_load_config(yaml_file)
 
       # Generate a unique key for the card
       card_key = generate_card_key(options[:name], config)
@@ -124,47 +119,40 @@ module MtgCardMaker
 
       # Save back to file
       File.write(yaml_file, config.to_yaml)
-      puts "✨ Added card '#{options[:name]}' to #{yaml_file}! ✨"
-      puts "🎴 Key: #{card_key}"
+      logger.success("Added card '#{options[:name] || card_key}' to #{yaml_file}!")
+      logger.info("🎴 Key: #{card_key}")
     end
 
     private
 
-    def validate_yaml_file(yaml_file)
-      return if File.exist?(yaml_file)
-
-      warn "❌ YAML file not found: #{yaml_file}"
-      exit 1
+    def logger
+      @logger ||= MtgCardMaker::Logger.new
     end
 
     def process_sprite_generation(config, sprite_service, output_file)
       if sprite_service.create_sprite_sheet(config, output_file)
         display_success_message(config, sprite_service, output_file)
       else
-        warn '❌ Failed to generate sprite sheet'
+        logger.error('Failed to generate sprite sheet')
         exit 1
       end
     end
 
     def handle_yaml_error(error)
-      warn "❌ Invalid YAML syntax: #{error.message}"
+      logger.error("Invalid YAML syntax: #{error.message}")
       exit 1
     end
 
     def handle_general_error(error)
-      warn "💥 Error: #{error.message}"
+      logger.error(error.message)
       exit 1
     end
 
     def display_success_message(config, sprite_service, output_file)
-      puts "✨ Generated #{output_file}! ✨"
+      logger.success("Generated #{output_file}!")
       width, height = sprite_service.sprite_dimensions(config.length)
-      puts "📏 Sprite dimensions: #{width}x#{height} pixels"
-      puts "🎨 Contains #{config.length} cards"
-    end
-
-    def load_yaml_config(yaml_file)
-      YAML.safe_load_file(yaml_file)
+      logger.info("📏 Sprite dimensions: #{width}x#{height} pixels")
+      logger.info("🎨 Contains #{config.length} cards")
     end
 
     def create_sprite_service
@@ -175,29 +163,25 @@ module MtgCardMaker
     end
 
     def build_card_config_from_options
-      config = build_required_config
-      add_optional_fields(config)
-      config
+      build_base_config.tap { |config| add_optional_fields(config) }
     end
 
-    def build_required_config
-      {
-        'name' => options[:name],
-        'type_line' => options[:type_line],
-        'rules_text' => process_newlines(options[:rules_text])
-      }
+    def build_base_config
+      { 'name' => options[:name], 'type_line' => options[:type_line] }.compact
     end
 
     def add_optional_fields(config)
-      optional_fields = %w[mana_cost power toughness border_color color art]
+      optional_fields = %w[mana_cost power toughness border_color color art embed_font]
       optional_fields.each do |field|
-        config[field] = options[field.to_sym] if options[field.to_sym]
+        config[field] = options[field.to_sym] if options.key?(field.to_sym)
       end
 
-      # Handle flavor_text separately to process newlines
-      return unless options[:flavor_text]
+      add_text_fields(config)
+    end
 
-      config['flavor_text'] = process_newlines(options[:flavor_text])
+    # Text fields are handled separately to process newlines
+    def add_text_fields(config)
+      %i[rules_text flavor_text].each { |f| config[f.to_s] = process_newlines(options[f]) if options[f] }
     end
 
     def process_newlines(text)
@@ -205,17 +189,36 @@ module MtgCardMaker
       text.gsub('\\n', "\n")
     end
 
-    def generate_card_key(name, existing_config)
-      base_key = name.downcase.gsub(/[^a-z0-9]/, '_').squeeze('_').chomp('_')
+    def create_directory_and_load_config(yaml_file)
+      # Create directory if it doesn't exist
+      FileUtils.mkdir_p(File.dirname(yaml_file))
+
+      # Load and validate config
+      validate_yaml(yaml_file)
+    end
+
+    def validate_yaml(yaml_file)
+      return {} unless File.exist?(yaml_file)
+
+      YAML.safe_load_file(yaml_file) || {}
+    rescue Psych::SyntaxError => e
+      logger.error("Invalid YAML syntax: #{e.message}")
+      exit 1
+    rescue StandardError => e
+      logger.error("Error reading YAML file: #{e.message}")
+      exit 1
+    end
+
+    def generate_card_key(name, config)
+      base_key = name.to_s.downcase.gsub(/[^a-z0-9]/, '_').squeeze('_').chomp('_')
+      base_key = 'card' if base_key.empty?
 
       # If key already exists, append a number
-      if existing_config.key?(base_key)
-        counter = 1
-        counter += 1 while existing_config.key?("#{base_key}_#{counter}")
-        "#{base_key}_#{counter}"
-      else
-        base_key
-      end
+      return base_key unless config.key?(base_key)
+
+      counter = 1
+      counter += 1 while config.key?("#{base_key}_#{counter}")
+      "#{base_key}_#{counter}"
     end
   end
 end

data/lib/mtg_card_maker/color_scheme.rb

@@ -17,14 +17,14 @@ module MtgCardMaker
   #   colorless_scheme = MtgCardMaker::ColorScheme.colorless
   #
   # @since 0.1.0
-  class ColorScheme # rubocop:disable Metrics/ClassLength
+  class ColorScheme
     # Predefined color schemes for all MTG colors and card types
     # @return [Hash] the predefined color scheme configurations
     SCHEMES = {
       colorless: {
         frame_gradient: ['#8B8B8B', '#6B6B6B', '#4A4A4A'],
         name_gradient: ['#F5F5F5', '#E8E8E8', '#D4D4D4'],
-        description_gradient: ['#F5F5F5', '#E8E8E8', '#D4D4D4'],
+        text_box_gradient: ['#F5F5F5', '#E8E8E8', '#D4D4D4'],
         card_gradient: ['#3D3D3D', '#111', '#1A1A1A'],
         primary: '#8B8B8B',
         background: '#E8E8E8',
@@ -37,7 +37,7 @@ module MtgCardMaker
       white: {
         frame_gradient: ['#FFF9C4', '#F5F5F5', '#BDB76B'],
         name_gradient: ['#FFFFFF', '#FFF9C4', '#E0E0E0'],
-        description_gradient: ['#FFFFFF', '#F5F5F5', '#E0E0E0'],
+        text_box_gradient: ['#FFFFFF', '#F5F5F5', '#E0E0E0'],
         card_gradient: ['#D7CCC8', '#BCAAA4', '#8D6E63'],
         primary: '#8B8B8B',
         background: '#FFFFFF',
@@ -47,7 +47,7 @@ module MtgCardMaker
       blue: {
         frame_gradient: ['#42A5F5', '#1565C0', '#263238'],
         name_gradient: ['#E3F2FD', '#BBDEFB', '#90CAF9'],
-        description_gradient: ['#F3F8FF', '#E3F2FD', '#BBDEFB'],
+        text_box_gradient: ['#F3F8FF', '#E3F2FD', '#BBDEFB'],
         card_gradient: ['#546E7A', '#37474F', '#263238'],
         primary: '#42A5F5',
         background: '#E3F2FD',
@@ -57,7 +57,7 @@ module MtgCardMaker
       black: {
         frame_gradient: ['#424242', '#212121', '#000'],
         name_gradient: ['#E0E0E0', '#BDBDBD', '#9E9E9E'],
-        description_gradient: ['#F5F5F5', '#E0E0E0', '#BDBDBD'],
+        text_box_gradient: ['#F5F5F5', '#E0E0E0', '#BDBDBD'],
         card_gradient: ['#424242', '#212121', '#000'],
         primary: '#424242',
         background: '#E0E0E0',
@@ -67,7 +67,7 @@ module MtgCardMaker
       red: {
         frame_gradient: ['#F44336', '#D32F2F', '#B71C1C'],
         name_gradient: ['#FFEBEE', '#FFCDD2', '#EF9A9A'],
-        description_gradient: ['#FFEBEE', '#FFCDD2', '#EF9A9A'],
+        text_box_gradient: ['#FFEBEE', '#FFCDD2', '#EF9A9A'],
         card_gradient: ['#8D6E63', '#6D4C41', '#4E342E'],
         primary: '#F44336',
         background: '#FFEBEE',
@@ -77,7 +77,7 @@ module MtgCardMaker
       green: {
         frame_gradient: ['#4CAF50', '#388E3C', '#2E7D32'],
         name_gradient: ['#E8F5E8', '#C8E6C9', '#A5D6A7'],
-        description_gradient: ['#F1F8E9', '#E8F5E8', '#C8E6C9'],
+        text_box_gradient: ['#F1F8E9', '#E8F5E8', '#C8E6C9'],
         card_gradient: ['#6D4C41', '#5D4037', '#4E342E'],
         primary: '#4CAF50',
         background: '#E8F5E8',
@@ -87,7 +87,7 @@ module MtgCardMaker
       gold: {
         frame_gradient: ['#FFD700', '#FFA500', '#FF8C00'],
         name_gradient: ['#F5DEB3', '#E1C16E', '#C9A13B'],
-        description_gradient: ['#FFF8DC', '#FFE4B5', '#FFDAB9'],
+        text_box_gradient: ['#FFF8DC', '#FFE4B5', '#FFDAB9'],
         card_gradient: ['#8B7355', '#6B4423', '#4A2C0A'],
         primary: '#FFD700',
         background: '#FFF8DC',
@@ -100,7 +100,7 @@ module MtgCardMaker
       artifact: {
         frame_gradient: ['#D2B48C', '#BC8F8F', '#A0522D'],
         name_gradient: ['#F5F5DC', '#DEB887', '#CD853F'],
-        description_gradient: ['#F5F5DC', '#DEB887', '#CD853F'],
+        text_box_gradient: ['#F5F5DC', '#DEB887', '#CD853F'],
         card_gradient: ['#8B7355', '#6B4423', '#4A2C0A'],
         primary: '#D2B48C',
         background: '#F5F5DC',
@@ -169,21 +169,21 @@ module MtgCardMaker
       config[:name_gradient]
     end
 
-    # Description gradient colors
-    def description_gradient_start
-      config[:description_gradient][0]
+    # Text box gradient colors
+    def text_box_gradient_start
+      config[:text_box_gradient][0]
     end
 
-    def description_gradient_middle
-      config[:description_gradient][1]
+    def text_box_gradient_middle
+      config[:text_box_gradient][1]
     end
 
-    def description_gradient_end
-      config[:description_gradient][2]
+    def text_box_gradient_end
+      config[:text_box_gradient][2]
     end
 
-    def description_gradient_colors
-      config[:description_gradient]
+    def text_box_gradient_colors
+      config[:text_box_gradient]
     end
 
     # Card gradient colors
@@ -296,10 +296,5 @@ module MtgCardMaker
     def self.artifact
       new(:artifact)
     end
-
-    # @return [ColorScheme] a silver color scheme
-    def self.silver
-      new(:silver)
-    end
   end
 end

data/lib/mtg_card_maker/css_service.rb

@@ -0,0 +1,72 @@
+# frozen_string_literal: true
+
+module MtgCardMaker
+  # Service class for managing CSS styles used across the application
+  class CssService
+    class << self
+      # Generate the font face declaration
+      #
+      # @param embed [Boolean] whether to embed the font as base64
+      # @return [String] the font face CSS
+      def font_face(embed: false)
+        return '' unless embed
+
+        font_path = File.join(__dir__, 'fonts', 'goudy_bookletter_base64.txt')
+        base64_font_data = File.read(font_path).strip
+        <<~CSS
+          @font-face {
+            font-family: 'Goudy Bookletter 1911';
+            src: url(data:font/truetype;charset=utf-8;base64,#{base64_font_data}) format('truetype');
+          }
+        CSS
+      end
+
+      # Generate all CSS classes
+      #
+      # @return [String] the CSS classes
+      def css_classes(embed: false)
+        <<~CSS
+          /* Font Classes */
+          .card-name, .card-type {
+            font-family: #{font_family(embed)};
+            font-weight: #{font_weight(embed)};
+          }
+
+          .card-rules-text, .mana-cost-text {
+            font-family: serif;
+          }
+
+          .card-flavor-text {
+            font-family: serif;
+            font-style: italic;
+          }
+
+          .card-power-toughness {
+            font-family: serif;
+            font-weight: bold;
+          }
+
+          .card-copyright {
+            font-family: sans-serif;
+          }
+        CSS
+      end
+
+      def font_family(embed)
+        embed ? "'Goudy Bookletter 1911', serif" : 'serif'
+      end
+
+      def font_weight(embed)
+        embed ? 'normal' : 'bold'
+      end
+
+      # Generate complete CSS styles
+      #
+      # @param embed [Boolean] whether to embed the font as base64
+      # @return [String] the complete CSS styles
+      def complete_styles(embed: false)
+        font_face(embed: embed) + css_classes(embed: embed)
+      end
+    end
+  end
+end

data/lib/mtg_card_maker/fonts/OFL.txt

@@ -0,0 +1,92 @@
+Copyright (c) 2010, Barry Schwartz (crudfactory.com)
+
+This Font Software is licensed under the SIL Open Font License, Version 1.1.
+This license is copied below, and is also available with a FAQ at:
+http://scripts.sil.org/OFL
+
+-----------------------------------------------------------
+SIL OPEN FONT LICENSE Version 1.1 - 26 February 2007
+-----------------------------------------------------------
+
+PREAMBLE
+The goals of the Open Font License (OFL) are to stimulate worldwide
+development of collaborative font projects, to support the font creation
+efforts of academic and linguistic communities, and to provide a free and
+open framework in which fonts may be shared and improved in partnership
+with others.
+
+The OFL allows the licensed fonts to be used, studied, modified and
+redistributed freely as long as they are not sold by themselves. The
+fonts, including any derivative works, can be bundled, embedded, 
+redistributed and/or sold with any software provided that any reserved
+names are not used by derivative works. The fonts and derivatives,
+however, cannot be released under any other type of license. The
+requirement for fonts to remain under this license does not apply
+to any document created using the fonts or their derivatives.
+
+DEFINITIONS
+"Font Software" refers to the set of files released by the Copyright
+Holder(s) under this license and clearly marked as such. This may
+include source files, build scripts and documentation.
+
+"Reserved Font Name" refers to any names specified as such after the
+copyright statement(s).
+
+"Original Version" refers to the collection of Font Software components as
+distributed by the Copyright Holder(s).
+
+"Modified Version" refers to any derivative made by adding to, deleting,
+or substituting -- in part or in whole -- any of the components of the
+Original Version, by changing formats or by porting the Font Software to a
+new environment.
+
+"Author" refers to any designer, engineer, programmer, technical
+writer or other person who contributed to the Font Software.
+
+PERMISSION & CONDITIONS
+Permission is hereby granted, free of charge, to any person obtaining
+a copy of the Font Software, to use, study, copy, merge, embed, modify,
+redistribute, and sell modified and unmodified copies of the Font
+Software, subject to the following conditions:
+
+1) Neither the Font Software nor any of its individual components,
+in Original or Modified Versions, may be sold by itself.
+
+2) Original or Modified Versions of the Font Software may be bundled,
+redistributed and/or sold with any software, provided that each copy
+contains the above copyright notice and this license. These can be
+included either as stand-alone text files, human-readable headers or
+in the appropriate machine-readable metadata fields within text or
+binary files as long as those fields can be easily viewed by the user.
+
+3) No Modified Version of the Font Software may use the Reserved Font
+Name(s) unless explicit written permission is granted by the corresponding
+Copyright Holder. This restriction only applies to the primary font name as
+presented to the users.
+
+4) The name(s) of the Copyright Holder(s) or the Author(s) of the Font
+Software shall not be used to promote, endorse or advertise any
+Modified Version, except to acknowledge the contribution(s) of the
+Copyright Holder(s) and the Author(s) or with their explicit written
+permission.
+
+5) The Font Software, modified or unmodified, in part or in whole,
+must be distributed entirely under this license, and must not be
+distributed under any other license. The requirement for fonts to
+remain under this license does not apply to any document created
+using the Font Software.
+
+TERMINATION
+This license becomes null and void if any of the above conditions are
+not met.
+
+DISCLAIMER
+THE FONT SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO ANY WARRANTIES OF
+MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT
+OF COPYRIGHT, PATENT, TRADEMARK, OR OTHER RIGHT. IN NO EVENT SHALL THE
+COPYRIGHT HOLDER BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
+INCLUDING ANY GENERAL, SPECIAL, INDIRECT, INCIDENTAL, OR CONSEQUENTIAL
+DAMAGES, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+FROM, OUT OF THE USE OR INABILITY TO USE THE FONT SOFTWARE OR FROM
+OTHER DEALINGS IN THE FONT SOFTWARE.