rails_console_ai 0.30.0 → 0.31.0

data/app/models/rails_console_ai/memory_version.rb

@@ -1,4 +1,4 @@
-require 'json'
+require 'rails_console_ai/tools/memory_tools'
 
 module RailsConsoleAi
   class MemoryVersion < ActiveRecord::Base
@@ -21,26 +21,11 @@ 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))
-    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 tags;        Array(parsed['tags']); end
   end
 end

data/app/models/rails_console_ai/skill.rb

@@ -1,4 +1,4 @@
-require 'json'
+require 'rails_console_ai/skill_loader'
 
 module RailsConsoleAi
   class Skill < 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 skill 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 tags bypass_guards_for_methods].freeze
-
     has_many :versions,
              -> { order(created_at: :desc) },
              class_name: 'RailsConsoleAi::SkillVersion',
              foreign_key: :skill_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,42 +35,38 @@ module RailsConsoleAi
       end
     end
 
-    # Manual JSON accessors keep us off Rails-version-specific `serialize` syntax
-    # (positional coder in Rails 5–6, keyword coder in Rails 7+).
-    def tags
-      decode_json_array(read_attribute(:tags))
-    end
-
-    def tags=(value)
-      write_attribute(:tags, encode_json_array(value))
-    end
-
-    def bypass_guards_for_methods
-      decode_json_array(read_attribute(:bypass_guards_for_methods))
-    end
-
-    def bypass_guards_for_methods=(value)
-      write_attribute(:bypass_guards_for_methods, encode_json_array(value))
-    end
-
-    # Defensive accessors — if `ai_db_migrate` hasn't been run yet, the status
-    # / approval columns may be missing on an older table. Return safe defaults
-    # instead of blowing up with NameError.
-    def status
-      has_attribute_status? ? read_attribute(:status) : STATUS_PROPOSED
+    # Parsed view of the raw markdown content. Memoized per-instance and cleared
+    # on assignment. Returns {} for invalid content so callers don't need to nil-guard.
+    def parsed
+      @parsed ||= (RailsConsoleAi::SkillLoader.parse(content.to_s) || {})
     end
 
-    def approved_by
-      has_attribute?(:approved_by) ? read_attribute(:approved_by) : nil
+    def content=(value)
+      @parsed = nil
+      super
     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 tags;        Array(parsed['tags']); end
+    def bypass_guards_for_methods; Array(parsed['bypass_guards_for_methods']); end
 
     def proposed?; status.to_s == STATUS_PROPOSED; end
     def approved?; status.to_s == STATUS_APPROVED; end
 
+    # Atomically bump use_count + last_used_at without firing callbacks /
+    # validations / updated_at. Safe to call from concurrent AI tool calls.
+    def self.record_use!(id)
+      where(id: id).update_all([
+        'use_count = COALESCE(use_count, 0) + 1, last_used_at = ?',
+        Time.now.utc
+      ])
+      true
+    rescue ::ActiveRecord::ActiveRecordError => e
+      RailsConsoleAi.logger.warn("RailsConsoleAi::Skill.record_use!(#{id.inspect}) failed: #{e.message}")
+      false
+    end
+
     def to_hash
       {
         'id'                        => id,
@@ -80,6 +75,7 @@ module RailsConsoleAi
         'body'                      => body,
         'tags'                      => tags,
         'bypass_guards_for_methods' => bypass_guards_for_methods,
+        'content'                   => content,
         'status'                    => status,
         'approved_by'               => approved_by,
         'approved_at'               => approved_at,
@@ -90,66 +86,14 @@ module RailsConsoleAi
       }
     end
 
-    def has_attribute_status?
-      has_attribute?(:status)
-    end
-
-    # Atomically bump use_count + last_used_at without firing callbacks /
-    # validations / updated_at. Safe to call from concurrent AI tool calls.
-    # No-op (returns false) if the table doesn't have the columns yet — that
-    # keeps older installs working until they run ai_db_migrate.
-    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
-      ])
-      true
-    rescue ::ActiveRecord::ActiveRecordError => e
-      RailsConsoleAi.logger.warn("RailsConsoleAi::Skill.record_use!(#{id.inspect}) failed: #{e.message}")
-      false
-    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.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 SkillVersion snapshot of the post-save state.
-    # Every save produces exactly one version row, so the version log is a complete history
-    # including the current state (the most recent version mirrors `self`).
-    #
-    # If `preserve_approval` is false (the default), any change to a content attribute
-    # reverts the skill back to "proposed" and clears the approver. Pass true from the
-    # approve! flow so approval doesn't reset itself.
+    # Assigns attrs, saves, and records one SkillVersion snapshot.
+    # Any change to `content` reverts approval back to "proposed" unless
+    # `preserve_approval: true` is passed (approve! does this).
     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
@@ -157,22 +101,17 @@ module RailsConsoleAi
 
         save!
         RailsConsoleAi::SkillVersion.create!(
-          skill_id:                   id,
-          name:                       name,
-          description:                description,
-          body:                       body,
-          tags:                       tags,
-          bypass_guards_for_methods:  bypass_guards_for_methods,
-          status:                     status,
-          edited_by:                  edited_by,
-          change_note:                change_note
+          skill_id:    id,
+          name:        name,
+          content:     content,
+          status:      status,
+          edited_by:   edited_by,
+          change_note: change_note
         )
       end
       self
     end
 
-    # Marks the current head as approved. Logs a version row with the approver name
-    # so the audit trail captures the approval moment.
     def approve!(approved_by:)
       raise ArgumentError, 'approved_by is required' if approved_by.to_s.strip.empty?
 
@@ -190,9 +129,20 @@ module RailsConsoleAi
 
     private
 
-    # Did any content-bearing attribute change in this assign_attributes pass?
-    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?  # presence validator handles blank
+      hash = RailsConsoleAi::SkillLoader.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/skill_version.rb

@@ -1,4 +1,4 @@
-require 'json'
+require 'rails_console_ai/skill_loader'
 
 module RailsConsoleAi
   class SkillVersion < ActiveRecord::Base
@@ -21,34 +21,13 @@ module RailsConsoleAi
       end
     end
 
-    def tags
-      decode_json_array(read_attribute(:tags))
+    def parsed
+      @parsed ||= (RailsConsoleAi::SkillLoader.parse(content.to_s) || {})
     end
 
-    def tags=(value)
-      write_attribute(:tags, encode_json_array(value))
-    end
-
-    def bypass_guards_for_methods
-      decode_json_array(read_attribute(:bypass_guards_for_methods))
-    end
-
-    def bypass_guards_for_methods=(value)
-      write_attribute(:bypass_guards_for_methods, 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 tags;        Array(parsed['tags']); end
+    def bypass_guards_for_methods; Array(parsed['bypass_guards_for_methods']); end
   end
 end

data/app/views/rails_console_ai/agents/_form.html.erb

@@ -1,47 +1,23 @@
 <% if @agent.persisted? && @agent.respond_to?(:approved?) && @agent.approved? %>
   <div class="flash flash-alert" style="margin-bottom:16px;">
-    Heads up: this agent is currently <strong>approved</strong>. Editing it will revert its status to <strong>proposed</strong> and the AI won't be able to invoke it again until someone re-approves it.
+    Heads up: this agent is currently <strong>approved</strong>. Editing it will revert its status to <strong>proposed</strong>.
   </div>
 <% elsif !@agent.persisted? %>
   <div class="flash flash-notice" style="margin-bottom:16px;">
-    New DB agents start as <strong>proposed</strong>. After saving, an approver (likely you) needs to click the Approve button on the show page before the AI can invoke this agent via delegate_task.
+    New DB agents start as <strong>proposed</strong>. After saving, an approver (likely you) needs to click the Approve button before delegate_task can invoke it.
   </div>
 <% end %>
 
 <%= form_with model: @agent, url: form_url, method: form_method, local: true do |f| %>
   <div class="form-card">
     <div class="form-row">
-      <label>Name</label>
-      <%= f.text_field :name, value: @agent.name, required: true %>
-    </div>
-
-    <div class="form-row">
-      <label>Description</label>
-      <%= f.text_field :description, value: @agent.description, placeholder: 'One-line description of what this agent specializes in' %>
-    </div>
-
-    <div class="form-row">
-      <label>Body / system instructions (markdown)</label>
-      <%= f.text_area :body, value: @agent.body, rows: 18 %>
-      <div class="hint">Persona + strategy + rules. Be specific about what tools/approaches the sub-agent should use and what format its summary should take.</div>
-    </div>
-
-    <div class="form-row">
-      <label>Max rounds (optional)</label>
-      <%= f.number_field :max_rounds, value: @agent.max_rounds, min: 1, max: 100 %>
-      <div class="hint">Maximum tool-loop iterations for the sub-agent. Defaults to the global <code>config.sub_agent_max_rounds</code>. Smaller = tighter scope, fewer tokens.</div>
-    </div>
-
-    <div class="form-row">
-      <label>Model (optional)</label>
-      <%= f.text_field :model, value: @agent.model, placeholder: 'e.g. claude-haiku-4-5' %>
-      <div class="hint">Model override. Leave blank to use the global model. Useful for cheap/fast agents.</div>
-    </div>
-
-    <div class="form-row">
-      <label>Tool whitelist (one per line, optional)</label>
-      <%= f.text_area :tools, value: Array(@agent.tools).join("\n"), rows: 4 %>
-      <div class="hint">Tool names the sub-agent is allowed to call (e.g. <code>execute_code</code>, <code>search_code</code>, <code>read_file</code>). Leave blank to use the default tool set. Adding tools here can broaden what the sub-agent can do — this is why DB agents need approval.</div>
+      <label>Markdown (frontmatter + body)</label>
+      <%= f.text_area :content, value: @agent.content, rows: 28, required: true, style: 'font-family: ui-monospace, SFMono-Regular, Menlo, monospace; font-size: 13px;' %>
+      <div class="hint">
+        YAML frontmatter between <code>---</code> lines, then the agent persona / instructions. Required:
+        <code>name</code>, <code>description</code>. Optional: <code>max_rounds</code>, <code>model</code>,
+        <code>tools</code> (whitelist; empty/omitted = default set).
+      </div>
     </div>
 
     <hr style="margin: 20px 0; border: 0; border-top: 1px solid #eee;">
@@ -49,7 +25,6 @@
     <div class="form-row">
       <label>Edited by</label>
       <input type="text" name="edited_by" value="<%= params[:edited_by] %>" placeholder="Your name or handle">
-      <div class="hint">Recorded with this version. Free text — the gem does not assume authentication beyond the admin login.</div>
     </div>
 
     <div class="form-row">

data/app/views/rails_console_ai/agents/diff.html.erb

@@ -5,11 +5,7 @@
   Version #<%= @from.id %> (<%= @from.created_at.strftime('%Y-%m-%d %H:%M') %>) → <%= @to_label %>
 </p>
 
-<h3 style="margin:16px 0 6px;">Body</h3>
-<%= render_text_diff(@from.body, @to_body, left_label: "v##{@from.id}", right_label: @to_label) %>
-
-<h3 style="margin:24px 0 6px;">Tool whitelist</h3>
-<%= render_json_diff(Array(@from.tools), @to_tools, left_label: "v##{@from.id}", right_label: @to_label) %>
+<%= render_text_diff(@from.content, @to_content, left_label: "v##{@from.id}", right_label: @to_label) %>
 
 <div class="btn-bar" style="margin-top:24px;">
   <% if @to.nil? %>

data/app/views/rails_console_ai/agents/new.html.erb

@@ -1,22 +1,6 @@
 <%= link_to '← All agents', agents_path, class: 'back-link' %>
 <h2 style="margin-bottom:16px;">New agent</h2>
 
-<%= form_with url: import_agents_path, method: :post, local: true do %>
-  <div class="form-card" style="background:#f8f9fa;">
-    <div class="form-row" style="margin-bottom:8px;">
-      <label>Paste a .md file (optional)</label>
-      <textarea name="content" rows="14" placeholder="---&#10;name: My agent&#10;description: What it specializes in&#10;max_rounds: 10&#10;model: claude-haiku-4-5&#10;tools: [execute_code, search_code]&#10;---&#10;&#10;Persona, strategy, rules …"></textarea>
-      <div class="hint">
-        Paste the full contents of a Markdown file (YAML frontmatter + body) — same
-        format as <code>.rails_console_ai/agents/*.md</code> or the built-in agents
-        shipped with the gem. Useful for moving an existing on-disk agent into the
-        versioned DB store.
-      </div>
-    </div>
-    <button type="submit" class="btn btn-secondary">Parse pasted content ↓</button>
-  </div>
-<% end %>
-
 <%= render 'form',
       form_url: agents_path,
       form_method: :post,

data/app/views/rails_console_ai/memories/_form.html.erb

@@ -1,19 +1,12 @@
 <%= form_with model: @memory, url: form_url, method: form_method, local: true do |f| %>
   <div class="form-card">
     <div class="form-row">
-      <label>Name</label>
-      <%= f.text_field :name, value: @memory.name, required: true %>
-    </div>
-
-    <div class="form-row">
-      <label>Description</label>
-      <%= f.text_area :description, value: @memory.description, rows: 14 %>
-      <div class="hint">The fact or pattern you're persisting. Markdown is fine.</div>
-    </div>
-
-    <div class="form-row">
-      <label>Tags (comma-separated)</label>
-      <%= f.text_field :tags, value: Array(@memory.tags).join(', '), placeholder: 'database, sharding' %>
+      <label>Markdown (frontmatter + body)</label>
+      <%= f.text_area :content, value: @memory.content, rows: 22, required: true, style: 'font-family: ui-monospace, SFMono-Regular, Menlo, monospace; font-size: 13px;' %>
+      <div class="hint">
+        YAML frontmatter between <code>---</code> lines (required: <code>name</code>; optional: <code>tags</code>),
+        then the memory body — the fact or pattern you're persisting.
+      </div>
     </div>
 
     <hr style="margin: 20px 0; border: 0; border-top: 1px solid #eee;">

data/app/views/rails_console_ai/memories/diff.html.erb

@@ -5,11 +5,7 @@
   Version #<%= @from.id %> (<%= @from.created_at.strftime('%Y-%m-%d %H:%M') %>) → <%= @to_label %>
 </p>
 
-<h3 style="margin:16px 0 6px;">Description</h3>
-<%= render_text_diff(@from.description, @to_description, left_label: "v##{@from.id}", right_label: @to_label) %>
-
-<h3 style="margin:24px 0 6px;">Tags</h3>
-<%= render_json_diff(Array(@from.tags), @to_tags, left_label: "v##{@from.id}", right_label: @to_label) %>
+<%= render_text_diff(@from.content, @to_content, left_label: "v##{@from.id}", right_label: @to_label) %>
 
 <div class="btn-bar" style="margin-top:24px;">
   <% if @to.nil? %>

data/app/views/rails_console_ai/memories/new.html.erb

@@ -1,21 +1,6 @@
 <%= link_to '← All memories', memories_path, class: 'back-link' %>
 <h2 style="margin-bottom:16px;">New memory</h2>
 
-<%= form_with url: import_memories_path, method: :post, local: true do %>
-  <div class="form-card" style="background:#f8f9fa;">
-    <div class="form-row" style="margin-bottom:8px;">
-      <label>Paste a .md file (optional)</label>
-      <textarea name="content" rows="12" placeholder="---&#10;name: My memory&#10;tags: [database]&#10;---&#10;&#10;The fact or pattern …"></textarea>
-      <div class="hint">
-        Paste the full contents of a Markdown file (YAML frontmatter + body) — same
-        format as <code>.rails_console_ai/memories/*.md</code>. The fields below will
-        be prefilled from the paste.
-      </div>
-    </div>
-    <button type="submit" class="btn btn-secondary">Parse pasted content ↓</button>
-  </div>
-<% end %>
-
 <%= render 'form',
       form_url: memories_path,
       form_method: :post,

data/app/views/rails_console_ai/skills/_form.html.erb

@@ -11,36 +11,14 @@
 <%= form_with model: @skill, url: form_url, method: form_method, local: true do |f| %>
   <div class="form-card">
     <div class="form-row">
-      <label>Name</label>
-      <%= f.text_field :name, value: @skill.name, required: true %>
-    </div>
-
-    <div class="form-row">
-      <label>Description</label>
-      <%= f.text_field :description, value: @skill.description, placeholder: 'One-line description of when to use this skill' %>
-    </div>
-
-    <div class="form-row">
-      <label>Body (markdown)</label>
-      <%= f.text_area :body, value: @skill.body, rows: 18 %>
-      <div class="hint">Include "## When to use", "## Recipe" (numbered steps with code blocks), "## Notes".</div>
-    </div>
-
-    <div class="form-row">
-      <label>Tags (comma-separated)</label>
-      <%= f.text_field :tags, value: Array(@skill.tags).join(', '), placeholder: 'booking-page, admin' %>
-    </div>
-
-    <div class="form-row">
-      <label>Bypass guards for methods (one per line)</label>
-      <%= f.text_area :bypass_guards_for_methods, value: Array(@skill.bypass_guards_for_methods).join("\n"), rows: 4 %>
+      <label>Markdown (frontmatter + body)</label>
+      <%= f.text_area :content, value: @skill.content, rows: 28, required: true, style: 'font-family: ui-monospace, SFMono-Regular, Menlo, monospace; font-size: 13px;' %>
       <div class="hint">
-        Two forms accepted — anything else is ignored:
-        <ul style="margin:6px 0 0 18px; padding:0;">
-          <li><code>ClassName#method</code> — instance method, e.g. <code>BookingPage#save!</code></li>
-          <li><code>ClassName.method</code> — class (singleton) method, e.g. <code>User.create_admin!</code></li>
-        </ul>
-        Namespaced classes work: <code>Acme::Booking#destroy</code>. The <code>!</code> is part of the match, so <code>save</code> and <code>save!</code> are different specs. While the skill is active, calls to these methods skip the configured safety guards (DB writes, HTTP mutations, email, etc.) <strong>for that tool-call only</strong>.
+        YAML frontmatter between <code>---</code> lines, then the markdown body. Required frontmatter:
+        <code>name</code>, <code>description</code>. Optional: <code>tags</code>,
+        <code>bypass_guards_for_methods</code> (entries shaped like <code>ClassName#method</code> or
+        <code>ClassName.method</code> — while the skill is active, calls to these methods skip the
+        configured safety guards for that tool-call only).
       </div>
     </div>
 

data/app/views/rails_console_ai/skills/diff.html.erb

@@ -5,14 +5,7 @@
   Version #<%= @from.id %> (<%= @from.created_at.strftime('%Y-%m-%d %H:%M') %>) → <%= @to_label %>
 </p>
 
-<h3 style="margin:16px 0 6px;">Body</h3>
-<%= render_text_diff(@from.body, @to_body, left_label: "v##{@from.id}", right_label: @to_label) %>
-
-<h3 style="margin:24px 0 6px;">Tags</h3>
-<%= render_json_diff(Array(@from.tags), @to_tags, left_label: "v##{@from.id}", right_label: @to_label) %>
-
-<h3 style="margin:24px 0 6px;">Bypass guards</h3>
-<%= render_json_diff(Array(@from.bypass_guards_for_methods), @to_bypass, left_label: "v##{@from.id}", right_label: @to_label) %>
+<%= render_text_diff(@from.content, @to_content, left_label: "v##{@from.id}", right_label: @to_label) %>
 
 <div class="btn-bar" style="margin-top:24px;">
   <% if @to.nil? %>

data/app/views/rails_console_ai/skills/new.html.erb

@@ -1,23 +1,6 @@
 <%= link_to '← All skills', skills_path, class: 'back-link' %>
 <h2 style="margin-bottom:16px;">New skill</h2>
 
-<%= form_with url: import_skills_path, method: :post, local: true do %>
-  <div class="form-card" style="background:#f8f9fa;">
-    <div class="form-row" style="margin-bottom:8px;">
-      <label>Paste a .md file (optional)</label>
-      <textarea name="content" rows="14" placeholder="---&#10;name: My skill&#10;description: What it does&#10;tags: [admin]&#10;bypass_guards_for_methods: []&#10;---&#10;&#10;## When to use&#10;…&#10;&#10;## Recipe&#10;1. …"></textarea>
-      <div class="hint">
-        Paste the full contents of a Markdown file (YAML frontmatter between
-        <code>---</code> lines, then a markdown body) — same format as
-        <code>.rails_console_ai/skills/*.md</code>. The fields below will be
-        prefilled from the paste. You can review and tweak before clicking
-        <strong>Create skill</strong>.
-      </div>
-    </div>
-    <button type="submit" class="btn btn-secondary">Parse pasted content ↓</button>
-  </div>
-<% end %>
-
 <%= render 'form',
       form_url: skills_path,
       form_method: :post,

data/config/routes.rb

@@ -8,7 +8,6 @@ RailsConsoleAi::Engine.routes.draw do
     end
     collection do
       get :diff
-      post :import
     end
     resources :versions, only: [:index, :show], controller: 'skill_versions' do
       member do
@@ -20,7 +19,6 @@ RailsConsoleAi::Engine.routes.draw do
   resources :memories do
     collection do
       get :diff
-      post :import
     end
     resources :versions, only: [:index, :show], controller: 'memory_versions' do
       member do
@@ -35,7 +33,6 @@ RailsConsoleAi::Engine.routes.draw do
     end
     collection do
       get :diff
-      post :import
     end
     resources :versions, only: [:index, :show], controller: 'agent_versions' do
       member do

data/lib/rails_console_ai/agent_loader.rb

@@ -159,16 +159,10 @@ module RailsConsoleAi
       key = agent_key(name)
       existing = load_agent_file(key)
 
-      frontmatter = {
-        'name' => name,
-        'description' => description
-      }
-      frontmatter['max_rounds'] = max_rounds if max_rounds
-      frontmatter['model'] = model if model
-      tool_list = Array(tools)
-      frontmatter['tools'] = tool_list unless tool_list.empty?
-
-      content = "---\n#{YAML.dump(frontmatter).sub("---\n", '').strip}\n---\n\n#{body}\n"
+      content = self.class.dump(
+        name: name, description: description, body: body,
+        max_rounds: max_rounds, model: model, tools: tools
+      )
       @storage.write(key, content)
 
       path = @storage.respond_to?(:root_path) ? File.join(@storage.root_path, key) : key
@@ -210,5 +204,19 @@ module RailsConsoleAi
     rescue Psych::SyntaxError
       nil
     end
+
+    # Inverse of parse: emit a canonical .md (frontmatter + body) string from
+    # structured attrs.
+    def self.dump(name:, description:, body:, max_rounds: nil, model: nil, tools: nil)
+      frontmatter = {
+        'name' => name,
+        'description' => description
+      }
+      frontmatter['max_rounds'] = max_rounds if max_rounds
+      frontmatter['model'] = model if model
+      tool_list = Array(tools)
+      frontmatter['tools'] = tool_list unless tool_list.empty?
+      "---\n#{YAML.dump(frontmatter).sub("---\n", '').strip}\n---\n\n#{body}\n"
+    end
   end
 end