rails_console_ai 0.30.0 → 0.31.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7455c7e1d55abd75c69bfa99621f1f83ec53a5faa2313e3095b3d80b88ec9938
-  data.tar.gz: 0a12cdef783d5f101d0c45b7fce9876e54bf31b1874fc4178a5d2103064ecb0a
+  metadata.gz: '083adc4ffd8b69f950fbc4ab18f2ac056e8642c6946c8ba46e67e19e4299e0f2'
+  data.tar.gz: a01fef0fe45dfa157d638a7d4edb1ee5c8d9753be3a79956b3ebde220d3ee939
 SHA512:
-  metadata.gz: 7433c944553f7c683b61e58653e76138e389e41ff29dff45c28fb344913a50514d2a58783fca2c916ff6f86a46a4d71cbad47803625e8078aa25895a2b12a057
-  data.tar.gz: 6cb61503f513cf4864fd918e95719f167733fc94ef001bb61290f6e0b262be0b2a3d83a936904ea7100096103c6c5c8145f74f5e99536088e45ad3aeb90d5a99
+  metadata.gz: 56b8fdcd0cf445d9480d4e56e252271241af017c8ca51e2a11af36d2158ce4a3a1fdf9670685f931b8cbd0994e6828e6bac08d10247e8bec1af8c66ad83c47f4
+  data.tar.gz: 6ccb72b8b8e9b157b28722d1e532d2abf5df8037f4ba4f1f3635bb37ad8a3441e48d832d903f6f92b6dce80106082215368dde15626eb0fc41d93da60100f261

data/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 All notable changes to this project will be documented in this file.
 
+## [0.31.0]
+
+- Simplify DB-backed skills, sub-agents, and memories to a single content field per record, replacing the separate frontmatter/body columns and streamlining the models, controllers, and forms
+- Add options for running agents
+- Fix the `/script/release` script
+
 ## [0.30.0]
 
 - Add background agent support — new `run_agent` mechanism to launch agents asynchronously alongside an HTTP API channel

data/README.md

@@ -521,6 +521,23 @@ RailsConsoleAi.get_agent_response(id)
 
 `run_agent` enqueues a row in the sessions table with `mode='agent_api'` and `status='queued'`. A separate long-running rake task picks them up and runs each in its own thread using the same engine that powers `ai "..."` in the console.
 
+### Per-run options
+
+`run_agent` accepts two extra keyword arguments to tune individual runs:
+
+```ruby
+RailsConsoleAi.run_agent(
+  "Trace why nightly billing is double-charging some accounts",
+  use_thinking_model: true,        # run on the thinking-tier model (e.g. Opus)
+  max_wall_clock_seconds: 1800     # hard kill after 30 minutes; pass nil for no cap
+)
+```
+
+- `use_thinking_model:` (default `false`) — switches the run to `config.thinking_model` (or the provider default thinking model) for the duration of the agent. Useful for harder, multi-step problems.
+- `max_wall_clock_seconds:` (default `600`) — hard ceiling on wall-clock time. If the run exceeds the cap, the worker thread is killed and the session is marked `status='failed'` with `error_message: "exceeded max_wall_clock_seconds (Ns)"`. Pass `nil` to opt out of any cap.
+
+These (along with any future per-run options) are stored in a JSON `options` column on the session row, so they survive the handoff to the background runner.
+
 ### Running the background runner
 
 ```bash
@@ -535,7 +552,7 @@ Run it separately from your web server, alongside (or instead of) the Slack bot.
 
 ### Requirements
 
-- `RailsConsoleAi.setup!` must have been run so the sessions table has the `status`, `result`, and `error_message` columns. `ai_db_setup` (or `ai_db_migrate` on existing installs) handles this.
+- `RailsConsoleAi.setup!` must have been run so the sessions table has the `status`, `result`, `error_message`, and `options` columns. `ai_db_setup` (or `ai_db_migrate` on existing installs) handles this.
 - `session_logging` must be enabled (it is by default).
 
 ### Behavior notes

data/app/controllers/rails_console_ai/agent_versions_controller.rb

@@ -13,14 +13,7 @@ module RailsConsoleAi
     def restore
       version = @agent.versions.find(params[:id])
       @agent.update_with_version!(
-        {
-          name:        version.name,
-          description: version.description,
-          body:        version.body,
-          max_rounds:  version.max_rounds,
-          model:       version.model,
-          tools:       Array(version.tools)
-        },
+        { content: version.content },
         edited_by: params[:edited_by].presence || 'web',
         change_note: "Restored from version ##{version.id}"
       )

data/app/controllers/rails_console_ai/agents_controller.rb

@@ -32,14 +32,17 @@ module RailsConsoleAi
           a['source'] == :builtin && a['name'].to_s.downcase == params[:from_builtin].to_s.downcase
         }
         if builtin
-          @agent.name        = builtin['name']
-          @agent.description = builtin['description']
-          @agent.body        = builtin['body']
-          @agent.max_rounds  = builtin['max_rounds']
-          @agent.model       = builtin['model']
-          @agent.tools       = Array(builtin['tools'])
+          @agent.content = AgentLoader.dump(
+            name: builtin['name'],
+            description: builtin['description'],
+            body: builtin['body'],
+            max_rounds: builtin['max_rounds'],
+            model: builtin['model'],
+            tools: Array(builtin['tools'])
+          )
         end
       end
+      @agent.content ||= new_agent_template
     end
 
     def create
@@ -57,34 +60,6 @@ module RailsConsoleAi
       end
     end
 
-    # POST /agents/import — parse a pasted .md blob and re-render `new` with fields prefilled.
-    def import
-      content = params[:content].to_s
-      if content.strip.empty?
-        redirect_to new_agent_path, alert: 'Nothing to parse — paste the .md content into the box first.'
-        return
-      end
-
-      parsed = AgentLoader.parse(content)
-      if parsed.nil? || parsed['name'].to_s.strip.empty?
-        redirect_to new_agent_path,
-                    alert: 'Could not parse. Expected YAML frontmatter (between `---` lines) with at least a `name` field, followed by the agent body.'
-        return
-      end
-
-      @agent = Agent.new(
-        name: parsed['name'],
-        description: parsed['description'],
-        body: parsed['body'],
-        max_rounds: parsed['max_rounds'],
-        model: parsed['model']
-      )
-      @agent.tools = Array(parsed['tools'])
-
-      flash.now[:notice] = "Parsed \"#{parsed['name']}\" from pasted content. Review the fields below and click Create agent to save to the DB."
-      render :new
-    end
-
     def edit
       redirect_to agents_path, alert: read_only_message and return unless @agent.is_a?(RailsConsoleAi::Agent)
     end
@@ -136,9 +111,8 @@ module RailsConsoleAi
       @agent = Agent.find(params[:agent_id])
       @from = @agent.versions.find(params[:from])
       @to   = params[:to].present? ? @agent.versions.find(params[:to]) : nil
-      @to_label = @to ? "Version ##{@to.id}" : 'Current'
-      @to_body  = @to ? @to.body : @agent.body
-      @to_tools = @to ? Array(@to.tools) : Array(@agent.tools)
+      @to_label   = @to ? "Version ##{@to.id}" : 'Current'
+      @to_content = @to ? @to.content : @agent.content
     end
 
     private
@@ -163,22 +137,25 @@ module RailsConsoleAi
     end
 
     def agent_params
-      {
-        name:        params.require(:agent)[:name],
-        description: params[:agent][:description],
-        body:        params[:agent][:body],
-        max_rounds:  params[:agent][:max_rounds].presence&.to_i,
-        model:       params[:agent][:model].presence,
-        tools:       split_lines(params[:agent][:tools])
-      }
+      { content: params.require(:agent)[:content].to_s }
     end
 
     def edited_by_param
       params[:edited_by].presence || 'web'
     end
 
-    def split_lines(str)
-      str.to_s.split(/[\r\n,]+/).map(&:strip).reject(&:empty?)
+    def new_agent_template
+      <<~MD
+        ---
+        name:
+        description:
+        max_rounds:
+        model:
+        tools: []
+        ---
+
+        Persona, strategy, rules…
+      MD
     end
 
     def slugify(name)

data/app/controllers/rails_console_ai/memories_controller.rb

@@ -25,32 +25,7 @@ module RailsConsoleAi
     end
 
     def new
-      @memory = Memory.new
-    end
-
-    # POST /memories/import — parse a pasted .md blob and re-render `new` with fields prefilled.
-    def import
-      content = params[:content].to_s
-      if content.strip.empty?
-        redirect_to new_memory_path, alert: 'Nothing to parse — paste the .md content into the box first.'
-        return
-      end
-
-      parsed = Tools::MemoryTools.parse(content)
-      if parsed.nil? || parsed['name'].to_s.strip.empty?
-        redirect_to new_memory_path,
-                    alert: 'Could not parse. Expected YAML frontmatter (between `---` lines) with at least a `name` field, followed by the memory body.'
-        return
-      end
-
-      @memory = Memory.new(
-        name: parsed['name'],
-        description: parsed['description']
-      )
-      @memory.tags = Array(parsed['tags'])
-
-      flash.now[:notice] = "Parsed \"#{parsed['name']}\" from pasted content. Review the fields below and click Create memory to save to the DB."
-      render :new
+      @memory = Memory.new(content: new_memory_template)
     end
 
     def create
@@ -102,9 +77,8 @@ module RailsConsoleAi
       @memory = Memory.find(params[:memory_id])
       @from = @memory.versions.find(params[:from])
       @to   = params[:to].present? ? @memory.versions.find(params[:to]) : nil
-      @to_label = @to ? "Version ##{@to.id}" : 'Current'
-      @to_description = @to ? @to.description : @memory.description
-      @to_tags = @to ? Array(@to.tags) : Array(@memory.tags)
+      @to_label   = @to ? "Version ##{@to.id}" : 'Current'
+      @to_content = @to ? @to.content : @memory.content
     end
 
     private
@@ -130,19 +104,22 @@ module RailsConsoleAi
     end
 
     def memory_params
-      {
-        name:        params.require(:memory)[:name],
-        description: params[:memory][:description],
-        tags:        split_csv(params[:memory][:tags])
-      }
+      { content: params.require(:memory)[:content].to_s }
     end
 
     def edited_by_param
       params[:edited_by].presence || 'web'
     end
 
-    def split_csv(str)
-      str.to_s.split(',').map(&:strip).reject(&:empty?)
+    def new_memory_template
+      <<~MD
+        ---
+        name:
+        tags: []
+        ---
+
+        The fact or pattern you're persisting.
+      MD
     end
 
     def slugify(name)

data/app/controllers/rails_console_ai/memory_versions_controller.rb

@@ -13,11 +13,7 @@ module RailsConsoleAi
     def restore
       version = @memory.versions.find(params[:id])
       @memory.update_with_version!(
-        {
-          name:        version.name,
-          description: version.description,
-          tags:        Array(version.tags)
-        },
+        { content: version.content },
         edited_by: params[:edited_by].presence || 'web',
         change_note: "Restored from version ##{version.id}"
       )

data/app/controllers/rails_console_ai/skill_versions_controller.rb

@@ -13,13 +13,7 @@ module RailsConsoleAi
     def restore
       version = @skill.versions.find(params[:id])
       @skill.update_with_version!(
-        {
-          name:                      version.name,
-          description:               version.description,
-          body:                      version.body,
-          tags:                      Array(version.tags),
-          bypass_guards_for_methods: Array(version.bypass_guards_for_methods)
-        },
+        { content: version.content },
         edited_by: params[:edited_by].presence || 'web',
         change_note: "Restored from version ##{version.id}"
       )

data/app/controllers/rails_console_ai/skills_controller.rb

@@ -26,37 +26,7 @@ module RailsConsoleAi
     end
 
     def new
-      @skill = Skill.new
-    end
-
-    # POST /skills/import — accepts a pasted .md blob in params[:content], parses
-    # YAML frontmatter + body, and re-renders `new` with the fields pre-populated.
-    # The user reviews + clicks Create skill to actually persist (normal proposed-
-    # status + version-row flow applies).
-    def import
-      content = params[:content].to_s
-      if content.strip.empty?
-        redirect_to new_skill_path, alert: 'Nothing to parse — paste the .md content into the box first.'
-        return
-      end
-
-      parsed = SkillLoader.parse(content)
-      if parsed.nil? || parsed['name'].to_s.strip.empty?
-        redirect_to new_skill_path,
-                    alert: 'Could not parse. Expected YAML frontmatter (between `---` lines) with at least a `name` field, followed by a markdown body.'
-        return
-      end
-
-      @skill = Skill.new(
-        name: parsed['name'],
-        description: parsed['description'],
-        body: parsed['body']
-      )
-      @skill.tags = Array(parsed['tags'])
-      @skill.bypass_guards_for_methods = Array(parsed['bypass_guards_for_methods'])
-
-      flash.now[:notice] = "Parsed \"#{parsed['name']}\" from pasted content. Review the fields below and click Create skill to save to the DB."
-      render :new
+      @skill = Skill.new(content: new_skill_template)
     end
 
     def create
@@ -130,10 +100,8 @@ module RailsConsoleAi
       @from = @skill.versions.find(params[:from])
       @to   = params[:to].present? ? @skill.versions.find(params[:to]) : nil
       # If `to` is omitted, diff against the current skill.
-      @to_label = @to ? "Version ##{@to.id}" : 'Current'
-      @to_body  = @to ? @to.body : @skill.body
-      @to_tags  = @to ? Array(@to.tags) : Array(@skill.tags)
-      @to_bypass = @to ? Array(@to.bypass_guards_for_methods) : Array(@skill.bypass_guards_for_methods)
+      @to_label   = @to ? "Version ##{@to.id}" : 'Current'
+      @to_content = @to ? @to.content : @skill.content
     end
 
     private
@@ -165,25 +133,28 @@ module RailsConsoleAi
     end
 
     def skill_params
-      {
-        name:                       params.require(:skill)[:name],
-        description:                params[:skill][:description],
-        body:                       params[:skill][:body],
-        tags:                       split_csv(params[:skill][:tags]),
-        bypass_guards_for_methods:  split_lines(params[:skill][:bypass_guards_for_methods])
-      }
+      { content: params.require(:skill)[:content].to_s }
     end
 
     def edited_by_param
       params[:edited_by].presence || 'web'
     end
 
-    def split_csv(str)
-      str.to_s.split(',').map(&:strip).reject(&:empty?)
-    end
+    def new_skill_template
+      <<~MD
+        ---
+        name:
+        description:
+        tags: []
+        bypass_guards_for_methods: []
+        ---
+
+        ## When to use
+
+        ## Recipe
 
-    def split_lines(str)
-      str.to_s.split(/[\r\n]+/).map(&:strip).reject(&:empty?)
+        ## Notes
+      MD
     end
 
     def slugify(name)

data/app/models/rails_console_ai/agent.rb

@@ -1,4 +1,4 @@
-require 'json'
+require 'rails_console_ai/agent_loader'
 
 module RailsConsoleAi
   class Agent < ActiveRecord::Base
@@ -8,19 +8,18 @@ module RailsConsoleAi
     STATUS_APPROVED = 'approved'.freeze
     STATUSES = [STATUS_PROPOSED, STATUS_APPROVED].freeze
 
-    # Attributes that, if changed, invalidate the current approval and revert
-    # the agent back to "proposed". Status / approver columns are excluded so
-    # that an explicit approve! call doesn't reset its own approval.
-    CONTENT_ATTRIBUTES = %w[name description body max_rounds model tools].freeze
-
     has_many :versions,
              -> { order(created_at: :desc) },
              class_name: 'RailsConsoleAi::AgentVersion',
              foreign_key: :agent_id,
              dependent: :nullify
 
+    validates :content, presence: true
     validates :name, presence: true, uniqueness: { case_sensitive: false }
-    validates :status, inclusion: { in: STATUSES }, if: :has_attribute_status?
+    validates :status, inclusion: { in: STATUSES }
+    validate  :content_parses
+
+    before_validation :sync_name_from_content
 
     scope :alphabetical, -> { order(Arel.sql('LOWER(name)')) }
     scope :approved, -> { where(status: STATUS_APPROVED) }
@@ -36,43 +35,25 @@ module RailsConsoleAi
       end
     end
 
-    # Manual JSON accessor for `tools` — same approach we use for skill tags,
-    # avoids Rails-version-specific `serialize` API.
-    def tools
-      decode_json_array(read_attribute(:tools))
-    end
-
-    def tools=(value)
-      write_attribute(:tools, encode_json_array(value))
+    def parsed
+      @parsed ||= (RailsConsoleAi::AgentLoader.parse(content.to_s) || {})
     end
 
-    # Defensive accessors — if `ai_db_migrate` hasn't been run yet, the status
-    # / approval columns may be missing on an older table.
-    def status
-      has_attribute_status? ? read_attribute(:status) : STATUS_PROPOSED
+    def content=(value)
+      @parsed = nil
+      super
     end
 
-    def approved_by
-      has_attribute?(:approved_by) ? read_attribute(:approved_by) : nil
-    end
-
-    def approved_at
-      has_attribute?(:approved_at) ? read_attribute(:approved_at) : nil
-    end
+    def description; parsed['description']; end
+    def body;        parsed['body']; end
+    def max_rounds;  parsed['max_rounds']; end
+    def model;       parsed['model']; end
+    def tools;       Array(parsed['tools']); end
 
     def proposed?; status.to_s == STATUS_PROPOSED; end
     def approved?; status.to_s == STATUS_APPROVED; end
 
-    def use_count
-      has_attribute?(:use_count) ? (read_attribute(:use_count) || 0) : 0
-    end
-
-    def last_used_at
-      has_attribute?(:last_used_at) ? read_attribute(:last_used_at) : nil
-    end
-
     def self.record_use!(id)
-      return false unless connection.column_exists?(table_name, :use_count)
       where(id: id).update_all([
         'use_count = COALESCE(use_count, 0) + 1, last_used_at = ?',
         Time.now.utc
@@ -92,6 +73,7 @@ module RailsConsoleAi
         'max_rounds'   => max_rounds,
         'model'        => model,
         'tools'        => tools,
+        'content'      => content,
         'status'       => status,
         'approved_by'  => approved_by,
         'approved_at'  => approved_at,
@@ -102,33 +84,11 @@ module RailsConsoleAi
       }
     end
 
-    def has_attribute_status?
-      has_attribute?(:status)
-    end
-
-    def self.decode_json_array(raw)
-      return [] if raw.nil? || (raw.respond_to?(:empty?) && raw.empty?)
-      return raw if raw.is_a?(Array)
-      JSON.parse(raw)
-    rescue JSON::ParserError
-      []
-    end
-
-    def self.encode_json_array(value)
-      JSON.dump(Array(value))
-    end
-
-    def decode_json_array(raw); self.class.decode_json_array(raw); end
-    def encode_json_array(value); self.class.encode_json_array(value); end
-
-    # Assigns attrs, saves, and records one AgentVersion snapshot of the post-save state.
-    # If `preserve_approval` is false (the default), any change to a content attribute
-    # reverts the agent back to "proposed" and clears the approver.
     def update_with_version!(attrs, edited_by: nil, change_note: nil, preserve_approval: false)
       transaction do
         assign_attributes(attrs)
 
-        if !preserve_approval && approved? && content_dirty?
+        if !preserve_approval && approved? && changes.key?('content')
           self.status      = STATUS_PROPOSED
           self.approved_by = nil
           self.approved_at = nil
@@ -138,11 +98,7 @@ module RailsConsoleAi
         RailsConsoleAi::AgentVersion.create!(
           agent_id:    id,
           name:        name,
-          description: description,
-          body:        body,
-          max_rounds:  max_rounds,
-          model:       model,
-          tools:       tools,
+          content:     content,
           status:      status,
           edited_by:   edited_by,
           change_note: change_note
@@ -168,8 +124,20 @@ module RailsConsoleAi
 
     private
 
-    def content_dirty?
-      CONTENT_ATTRIBUTES.any? { |a| changes.key?(a) }
+    def sync_name_from_content
+      return if content.to_s.strip.empty?
+      parsed_name = parsed['name'].to_s.strip
+      self.name = parsed_name unless parsed_name.empty?
+    end
+
+    def content_parses
+      return if content.to_s.strip.empty?
+      hash = RailsConsoleAi::AgentLoader.parse(content.to_s)
+      if hash.nil?
+        errors.add(:content, "could not be parsed — expected YAML frontmatter between `---` lines followed by a markdown body")
+      elsif hash['name'].to_s.strip.empty?
+        errors.add(:content, "frontmatter is missing a `name:` field")
+      end
     end
   end
 end

data/app/models/rails_console_ai/agent_version.rb

@@ -1,4 +1,4 @@
-require 'json'
+require 'rails_console_ai/agent_loader'
 
 module RailsConsoleAi
   class AgentVersion < ActiveRecord::Base
@@ -21,26 +21,14 @@ module RailsConsoleAi
       end
     end
 
-    def tools
-      decode_json_array(read_attribute(:tools))
+    def parsed
+      @parsed ||= (RailsConsoleAi::AgentLoader.parse(content.to_s) || {})
     end
 
-    def tools=(value)
-      write_attribute(:tools, encode_json_array(value))
-    end
-
-    private
-
-    def decode_json_array(raw)
-      return [] if raw.nil? || (raw.respond_to?(:empty?) && raw.empty?)
-      return raw if raw.is_a?(Array)
-      JSON.parse(raw)
-    rescue JSON::ParserError
-      []
-    end
-
-    def encode_json_array(value)
-      JSON.dump(Array(value))
-    end
+    def description; parsed['description']; end
+    def body;        parsed['body']; end
+    def max_rounds;  parsed['max_rounds']; end
+    def model;       parsed['model']; end
+    def tools;       Array(parsed['tools']); end
   end
 end

data/app/models/rails_console_ai/memory.rb

@@ -1,4 +1,4 @@
-require 'json'
+require 'rails_console_ai/tools/memory_tools'
 
 module RailsConsoleAi
   class Memory < ActiveRecord::Base
@@ -10,7 +10,11 @@ module RailsConsoleAi
              foreign_key: :memory_id,
              dependent: :nullify
 
+    validates :content, presence: true
     validates :name, presence: true, uniqueness: { case_sensitive: false }
+    validate  :content_parses
+
+    before_validation :sync_name_from_content
 
     scope :alphabetical, -> { order(Arel.sql('LOWER(name)')) }
 
@@ -24,24 +28,21 @@ module RailsConsoleAi
       end
     end
 
-    def tags
-      decode_json_array(read_attribute(:tags))
+    def parsed
+      @parsed ||= (RailsConsoleAi::Tools::MemoryTools.parse(content.to_s) || {})
     end
 
-    def tags=(value)
-      write_attribute(:tags, encode_json_array(value))
+    def content=(value)
+      @parsed = nil
+      super
     end
 
-    def use_count
-      has_attribute?(:use_count) ? (read_attribute(:use_count) || 0) : 0
-    end
-
-    def last_used_at
-      has_attribute?(:last_used_at) ? read_attribute(:last_used_at) : nil
-    end
+    # Memories don't have a separate description vs body — the markdown body
+    # IS the memory. Parser exposes it under 'description'.
+    def description; parsed['description']; end
+    def tags;        Array(parsed['tags']); end
 
     def self.record_use!(id)
-      return false unless connection.column_exists?(table_name, :use_count)
       where(id: id).update_all([
         'use_count = COALESCE(use_count, 0) + 1, last_used_at = ?',
         Time.now.utc
@@ -58,6 +59,7 @@ module RailsConsoleAi
         'name'         => name,
         'description'  => description,
         'tags'         => tags,
+        'content'      => content,
         'use_count'    => use_count,
         'last_used_at' => last_used_at,
         'source'       => :db,
@@ -72,8 +74,7 @@ module RailsConsoleAi
         RailsConsoleAi::MemoryVersion.create!(
           memory_id:   id,
           name:        name,
-          description: description,
-          tags:        tags,
+          content:     content,
           edited_by:   edited_by,
           change_note: change_note
         )
@@ -83,16 +84,20 @@ module RailsConsoleAi
 
     private
 
-    def decode_json_array(raw)
-      return [] if raw.nil? || (raw.respond_to?(:empty?) && raw.empty?)
-      return raw if raw.is_a?(Array)
-      JSON.parse(raw)
-    rescue JSON::ParserError
-      []
+    def sync_name_from_content
+      return if content.to_s.strip.empty?
+      parsed_name = parsed['name'].to_s.strip
+      self.name = parsed_name unless parsed_name.empty?
     end
 
-    def encode_json_array(value)
-      JSON.dump(Array(value))
+    def content_parses
+      return if content.to_s.strip.empty?
+      hash = RailsConsoleAi::Tools::MemoryTools.parse(content.to_s)
+      if hash.nil?
+        errors.add(:content, "could not be parsed — expected YAML frontmatter between `---` lines followed by a markdown body")
+      elsif hash['name'].to_s.strip.empty?
+        errors.add(:content, "frontmatter is missing a `name:` field")
+      end
     end
   end
 end