mtg_card_maker 0.2.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7fa254f0344ffc52479fb2be5aae9a6f0486b9ebff42b684156260bfe4814879
-  data.tar.gz: 34cdc3ed5192c5e89e7a4e0d58260f5276fbfa920a48e286c9638757e301342c
+  metadata.gz: da380956bcd066726ee25156f61eda1ea82071f07a5fa18a10b6eaaf00802351
+  data.tar.gz: 3f246516bc68c71485d7edcec625391329a2445b9219f7b79d1e5d28824bb57d
 SHA512:
-  metadata.gz: cf7b3b64a9c76fcb695571a3320cc5fdf9746c7719f411eb3ebaaa9d64573aec3862990f99b0818b754ed6b62191ff40b73d7997536328fc85e35f2b9a286626
-  data.tar.gz: ebc7ed18847e53e800e9fe2b7156e82ad470ac22984c2f418a8e662680f63a1bd46693808f804295408c185014760e05a3a0e7898517ae7fa07dbd29777591a7
+  metadata.gz: fe45049d937ba11f0f33c5dc37c663107f50df23a68602ceaa22c1a830a2c151ecda86a272e0bfa32b3f4940332eb1add45a0338f37548d28a4ab13835eea668
+  data.tar.gz: f006407754e7a65a85aa6d1eecf52c45e1739174ae95f999d9c3f6d7e9d8b8d5389d628b44a4712df5ba1843dd329e9f5b04cb16fdd8f96c4fa695cef642f4b4

data/CHANGELOG.md

@@ -1,5 +1,15 @@
 ## [Unreleased]
 
+## [0.2.1] - 2026-10-11
+
+### 🔄 Changed
+- **All card text fields are optional again.** `name`, `type_line`, and `rules_text` may be omitted or blank in the Ruby API, YAML configuration, and CLI, so vanilla creatures, tokens, and blank frames can be generated. 0.2.0 required all three. When `name` or `type_line` is missing, a warning is printed ("Card has no name or type line") and the card is still generated.
+- **The CLI rejects unknown options.** A misspelled option such as `--nmae` now fails with `Unknown switches "--nmae"` and a suggestion, instead of being ignored and producing a blank card.
+- `add_card` uses the key `card` (then `card_1`, `card_2`, ...) when no name is given.
+
+### 🐛 Fixed
+- `generate_sprite` reports `YAML file not found: <path>` for a missing YAML file again, instead of the generic "Failed to generate sprite sheet" error.
+
 ## [0.2.0] - 2026-10-11
 
 ### 📜 License
@@ -9,16 +19,34 @@
 - Replaced the bundled Goudy Mediaeval DemiBold font, which is licensed for personal use only, with Goudy Bookletter 1911. Embedded card text renders slightly differently, and SVGs with embedded fonts are about 38 KB larger.
 
 ### ⚠️ Breaking Changes
+- **Card validation.** The Ruby API and YAML configuration now raise `ArgumentError: Missing required fields` when `name`, `type_line`, or `rules_text` is missing or blank. In 0.1.0 these cards rendered without the missing text. These requirements were removed again in 0.2.1.
 - **Dropped support for Ruby 3.2.** Ruby 3.3 or higher is now required (Ruby 3.2 reached end of life in March 2026).
 
 ### ✨ Features
-- Hybrid and Phyrexian mana symbol support, with a new `IconFilenameService` for symbol mapping
-- SVG to WebP conversion script (experimental)
-- Italicized flavor text
-- Improved SVG initialization and viewBox scaling in card rendering
+- **Symbols:** hybrid mana (`{W/U}`, `{2/W}`, `{C/W}`), Phyrexian mana (`{R/P}`, `{W/B/P}`), snow (`{S}`), energy (`{E}`), tap and untap (`{T}`, `{Q}`), `{X}`, and numeric symbols `{0}` through `{99}`, in mana costs and in rules text. Symbols in rules text are rendered inline.
+- **Font embedding option:** `--embed-font` / `--no-embed-font` on the CLI and `embed_font:` in the Ruby API and YAML configuration.
+- **Italic flavor text.**
+- **Quieter, consistent CLI output:** messages go through a new `MtgCardMaker::Logger` (success and info to stdout, warnings and errors to stderr).
+- **viewBox on generated SVGs**, so cards scale correctly when embedded or resized.
+- SVG to WebP conversion script (`bin/svg_to_webp`, experimental, in the source repository only).
+
+### 🔄 Changed
+- **Fonts are embedded by default.** Generated cards now embed the card font as base64 (`embed_font` defaults to `true`; pass `--no-embed-font` or `embed_font: false` to opt out). Single-card SVGs are about 95 KB larger. Sprite sheets embed the font once, and only when a card sets `embed_font: true`.
+- **Generated SVG markup:** the rules text CSS class `card-description` is now `card-rules-text`, and the text box gradient ids `<scheme>_description_gradient` are now `<scheme>_text_box_gradient`. Stylesheets that target the old names need updating.
+- **Public Ruby API renames (affects direct users of these classes):**
+  - `ColorScheme#description_gradient_*` → `#text_box_gradient_*`
+  - `SvgGradientService.description_gradient_id` / `.define_description_gradient` → `.text_box_gradient_id` / `.define_text_box_gradient`
+  - `IconService#available_colors` and `#icon_svg` → `#available_icon_types` and `#symbol_svg`; `IconService::ICON_SETS` → `IconService.icon_sets`
+  - `LayerConfig#text_positioning_config` → `#text_box_config`; `#mana_cost_font_size`, `#mana_cost_icon_opacity`, `#mana_cost_text_y_offset`, and `#icon_opacity_config` were removed
+  - `ColorScheme.silver` and `ManaCost::COLOR_MAP` were removed
+
+### 🐛 Fixed
+- Snow and energy symbols no longer corrupt the SVG output.
+- Mana costs combining `X` and a number (for example `X3R`) render correctly.
+- An empty YAML file passed to `generate_sprite` now exits with "Failed to generate sprite sheet" instead of `undefined method 'empty?' for nil`.
 
 ### 📚 Documentation
-- README updated with new symbols and Markdown docs support; YARD documentation refreshed
+- README updated with hybrid, Phyrexian, snow, energy, tap, and numeric symbol references, and Markdown docs support; YARD documentation refreshed.
 
 ## [0.1.0] - 2025-07-12
 

data/README.md

@@ -81,11 +81,11 @@ gem install mtg_card_maker
 
 **Card Properties:**
 
-Optionals without a default hide when omitted.
+Every property is optional and hides when omitted. A warning is shown if `name` or `type_line` is missing.
 ```yaml
-name: "Card Name"                 # Required
-type_line: "Creature - Dragon"    # Required
-rules_text: "Card rules text"     # Required
+name: "Card Name"                 # Optional (warns when omitted)
+type_line: "Creature - Dragon"    # Optional (warns when omitted)
+rules_text: "Card rules text"     # Optional
 mana_cost: 2RR                    # Optional
 flavor_text: "Flavor text"        # Optional
 power: 4                          # Optional
@@ -244,6 +244,8 @@ mtg_card_maker add_card deck.yml \
 - `g` or `gc` for `generate_card`
 - `a` or `ac` for `add_card`
 
+*Misspelled options are rejected* (for example `--nmae` fails with `Unknown switches "--nmae"` and suggests `--name`) rather than silently producing a blank card.
+
 ### Generate a Sprite Sheet
 
 For printing or displaying multiple cards on a webpage, using a sprite sheet is recommended. This approach embeds fonts and gradients once per sheet instead of repeating them for each individual card, improving file size and loading efficiency.
@@ -269,7 +271,7 @@ mtg_card_maker generate_sprite deck.yml sprite_sheet.svg \
 
 **⚠️ Experimental Feature - Mac Only**
 
-The gem includes experimental support for converting SVG cards to WebP format using Chrome's headless mode. This feature is currently **Mac-only** and requires specific dependencies.
+The source repository includes an experimental script, `bin/svg_to_webp`, for converting SVG cards to WebP format using Chrome's headless mode. It is a development script and is **not** part of the published gem, so clone the repository to use it. This feature is currently **Mac-only** and requires specific dependencies.
 
 **Why?:**
 It seems a lot more user-friendly for use in things like this README.md file. The file size is bigger and it can't scale so I wouldn't use this option unless something in the SVG isn't working with whatever you are trying to do. You should also report that at https://github.com/joe-sharp/mtg_card_maker/issues . Thanks in advance!
@@ -278,7 +280,7 @@ It seems a lot more user-friendly for use in things like this README.md file. Th
 - **macOS**: Chrome must be installed at `/Applications/Google Chrome.app/Contents/MacOS/Google Chrome`
 - **cwebp**: Google's WebP encoder must be installed (install via Homebrew: `brew install webp`)
 
-**Usage:**
+**Usage** (from a clone of the repository):
 ```bash
 # Convert a single SVG card to WebP
 bin/svg_to_webp output_card.svg

data/lib/mtg_card_maker/base_card.rb

@@ -111,10 +111,10 @@ module MtgCardMaker
     # Initialize a new card with the given configuration
     #
     # @param config [Hash] the card configuration
-    # @option config [String] :name the card name
+    # @option config [String] :name the card name (optional, warns when omitted)
     # @option config [String] :mana_cost the mana cost in MTG notation
     # @option config [String] :type_line the card type
-    # @option config [String] :rules_text the card rules text
+    # @option config [String] :rules_text the card rules text (optional)
     # @option config [String] :flavor_text the flavor text
     # @option config [String] :power the power value for creatures
     # @option config [String] :toughness the toughness value for creatures
@@ -124,7 +124,7 @@ module MtgCardMaker
     # @option config [Boolean] :embed_font whether to embed the font as base64 (default: true)
     def initialize(config)
       assign_attributes(config)
-      validate_required_fields
+      warn_about_missing_fields
       @template = Template.new(embed_font: config.fetch(:embed_font, true))
       add_layers
     end
@@ -187,13 +187,11 @@ module MtgCardMaker
       @art = config[:art]
     end
 
-    def validate_required_fields
-      required_fields = %i[name type_line rules_text]
-      missing_fields = required_fields.select { |field| send(field).nil? || send(field).to_s.strip.empty? }
+    def warn_about_missing_fields
+      missing = { name: 'name', type_line: 'type line' }.select { |field, _label| public_send(field).to_s.strip.empty? }
+      return if missing.empty?
 
-      return if missing_fields.empty?
-
-      raise ArgumentError, "Missing required fields: #{missing_fields.join(', ')}"
+      Logger.new.warn("Card has no #{missing.values.join(' or ')}")
     end
 
     def define_art_window_mask

data/lib/mtg_card_maker/cli.rb

@@ -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)' },
@@ -58,10 +61,10 @@ module MtgCardMaker
 
     # 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
@@ -90,6 +93,8 @@ 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)
+      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)
@@ -114,7 +119,7 @@ module MtgCardMaker
 
       # Save back to file
       File.write(yaml_file, config.to_yaml)
-      logger.success("Added card '#{options[:name]}' to #{yaml_file}!")
+      logger.success("Added card '#{options[:name] || card_key}' to #{yaml_file}!")
       logger.info("🎴 Key: #{card_key}")
     end
 
@@ -158,17 +163,11 @@ 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)
@@ -177,10 +176,12 @@ module MtgCardMaker
         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)
@@ -209,16 +210,15 @@ module MtgCardMaker
     end
 
     def generate_card_key(name, config)
-      base_key = name.downcase.gsub(/[^a-z0-9]/, '_').squeeze('_').chomp('_')
+      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 config.key?(base_key)
-        counter = 1
-        counter += 1 while 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/version.rb

@@ -3,6 +3,6 @@
 module MtgCardMaker
   MAJOR = 0
   MINOR = 2
-  PATCH = 0
+  PATCH = 1
   VERSION = [MAJOR, MINOR, PATCH].join('.')
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mtg_card_maker
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors:
 - joe-sharp
@@ -192,7 +192,7 @@ licenses:
 metadata:
   license: MIT
   homepage_uri: https://github.com/joe-sharp/mtg_card_maker
-  source_code_uri: https://github.com/joe-sharp/mtg_card_maker/tree/v0.2.0
+  source_code_uri: https://github.com/joe-sharp/mtg_card_maker/tree/v0.2.1
   bug_tracker_uri: https://github.com/joe-sharp/mtg_card_maker/issues
   changelog_uri: https://github.com/joe-sharp/mtg_card_maker/blob/main/CHANGELOG.md
   rubygems_mfa_required: 'true'