rails_console_ai 0.29.0 → 0.31.0

data/app/models/rails_console_ai/skill.rb

@@ -0,0 +1,148 @@
+require 'rails_console_ai/skill_loader'
+
+module RailsConsoleAi
+  class Skill < ActiveRecord::Base
+    self.table_name = 'rails_console_ai_skills'
+
+    STATUS_PROPOSED = 'proposed'.freeze
+    STATUS_APPROVED = 'approved'.freeze
+    STATUSES = [STATUS_PROPOSED, STATUS_APPROVED].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 }
+    validate  :content_parses
+
+    before_validation :sync_name_from_content
+
+    scope :alphabetical, -> { order(Arel.sql('LOWER(name)')) }
+    scope :approved, -> { where(status: STATUS_APPROVED) }
+    scope :proposed, -> { where(status: STATUS_PROPOSED) }
+
+    def self.connection
+      klass = RailsConsoleAi.configuration.connection_class
+      if klass
+        klass = Object.const_get(klass) if klass.is_a?(String)
+        klass.connection
+      else
+        super
+      end
+    end
+
+    # 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 content=(value)
+      @parsed = nil
+      super
+    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,
+        'name'                      => name,
+        'description'               => description,
+        'body'                      => body,
+        'tags'                      => tags,
+        'bypass_guards_for_methods' => bypass_guards_for_methods,
+        'content'                   => content,
+        'status'                    => status,
+        'approved_by'               => approved_by,
+        'approved_at'               => approved_at,
+        'use_count'                 => use_count,
+        'last_used_at'              => last_used_at,
+        'source'                    => :db,
+        'updated_at'                => updated_at
+      }
+    end
+
+    # 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? && changes.key?('content')
+          self.status      = STATUS_PROPOSED
+          self.approved_by = nil
+          self.approved_at = nil
+        end
+
+        save!
+        RailsConsoleAi::SkillVersion.create!(
+          skill_id:    id,
+          name:        name,
+          content:     content,
+          status:      status,
+          edited_by:   edited_by,
+          change_note: change_note
+        )
+      end
+      self
+    end
+
+    def approve!(approved_by:)
+      raise ArgumentError, 'approved_by is required' if approved_by.to_s.strip.empty?
+
+      update_with_version!(
+        {
+          status:      STATUS_APPROVED,
+          approved_by: approved_by,
+          approved_at: Time.now.utc
+        },
+        edited_by: approved_by,
+        change_note: "Approved by #{approved_by}",
+        preserve_approval: true
+      )
+    end
+
+    private
+
+    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

@@ -0,0 +1,33 @@
+require 'rails_console_ai/skill_loader'
+
+module RailsConsoleAi
+  class SkillVersion < ActiveRecord::Base
+    self.table_name = 'rails_console_ai_skill_versions'
+
+    belongs_to :skill,
+               class_name: 'RailsConsoleAi::Skill',
+               foreign_key: :skill_id,
+               optional: true
+
+    scope :recent, -> { order(created_at: :desc) }
+
+    def self.connection
+      klass = RailsConsoleAi.configuration.connection_class
+      if klass
+        klass = Object.const_get(klass) if klass.is_a?(String)
+        klass.connection
+      else
+        super
+      end
+    end
+
+    def parsed
+      @parsed ||= (RailsConsoleAi::SkillLoader.parse(content.to_s) || {})
+    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/layouts/rails_console_ai/application.html.erb

@@ -70,14 +70,91 @@
     .pagination a:hover { background: #f8f9fa; text-decoration: none; }
     .pagination .disabled { padding: 6px 14px; font-size: 13px; color: #ccc; }
     .pagination .page-info { font-size: 13px; color: #888; }
+    .nav-links a { margin-left: 16px; }
+    .btn {
+      display: inline-block; padding: 7px 14px; border-radius: 4px;
+      background: #4a6fa5; color: #fff !important; border: 0;
+      font-size: 13px; cursor: pointer; text-decoration: none;
+    }
+    .btn:hover { background: #3a5a85; text-decoration: none; }
+    .btn-secondary { background: #6c757d; }
+    .btn-secondary:hover { background: #5a6268; }
+    .btn-danger { background: #dc3545; }
+    .btn-danger:hover { background: #c82333; }
+    .btn-bar { display: flex; gap: 8px; margin: 16px 0; flex-wrap: wrap; }
+    .source-db { background: #cce5ff; color: #004085; }
+    .source-file { background: #e2e3e5; color: #383d41; }
+    .source-builtin { background: #e0d4ff; color: #4a2d8c; }
+    .status-approved { background: #d4edda; color: #155724; }
+    .status-proposed { background: #fff3cd; color: #856404; }
+    .form-row { margin-bottom: 14px; }
+    .form-row label {
+      display: block; font-size: 12px; font-weight: 600;
+      color: #555; text-transform: uppercase; margin-bottom: 4px;
+    }
+    .form-row input[type="text"], .form-row textarea {
+      width: 100%; padding: 8px 10px; font-size: 14px;
+      border: 1px solid #ccc; border-radius: 4px;
+      font-family: -apple-system, BlinkMacSystemFont, sans-serif;
+    }
+    .form-row textarea {
+      font-family: "SF Mono", "Monaco", "Menlo", "Consolas", monospace;
+      font-size: 13px;
+    }
+    .form-row .hint { font-size: 12px; color: #888; margin-top: 4px; }
+    .form-card {
+      background: #fff; border-radius: 8px; padding: 24px;
+      box-shadow: 0 1px 3px rgba(0,0,0,0.1); margin-bottom: 24px;
+    }
+    .flash { padding: 12px 16px; border-radius: 4px; margin-bottom: 16px; font-size: 14px; }
+    .flash-notice { background: #d4edda; color: #155724; }
+    .flash-alert { background: #f8d7da; color: #721c24; }
+    .markdown-body {
+      background: #fff; border-radius: 8px; padding: 20px;
+      box-shadow: 0 1px 3px rgba(0,0,0,0.1); margin-bottom: 16px;
+      white-space: pre-wrap; font-family: "SF Mono", monospace; font-size: 13px;
+    }
+    .versions-list { background: #fff; border-radius: 8px; padding: 0; box-shadow: 0 1px 3px rgba(0,0,0,0.1); }
+    .versions-list .ver-row {
+      display: flex; align-items: center; justify-content: space-between;
+      padding: 10px 16px; border-bottom: 1px solid #f0f0f0; font-size: 13px;
+    }
+    .versions-list .ver-row:last-child { border-bottom: 0; }
+    .versions-list .ver-meta { color: #888; font-size: 12px; }
+    .diff-table {
+      width: 100%; border-collapse: collapse; background: #fff;
+      border-radius: 8px; overflow: hidden; box-shadow: 0 1px 3px rgba(0,0,0,0.1);
+    }
+    .diff-table th { background: #f8f9fa; padding: 8px 12px; text-align: left; font-size: 12px; }
+    .diff-table td {
+      vertical-align: top; padding: 0; width: 50%; border-bottom: 1px solid #f0f0f0;
+    }
+    .diff-table pre {
+      margin: 0; padding: 4px 12px; font-family: "SF Mono", monospace;
+      font-size: 12px; white-space: pre-wrap; word-break: break-word;
+    }
+    .diff-add { background: #d4edda; }
+    .diff-del { background: #f8d7da; }
+    .tag {
+      display: inline-block; padding: 2px 8px; margin-right: 4px;
+      background: #e9ecef; color: #495057; border-radius: 10px; font-size: 11px;
+    }
+    .inline-form { display: inline; }
   </style>
 </head>
 <body>
   <div class="header">
     <h1>RailsConsoleAi Admin</h1>
-    <a href="<%= rails_console_ai.root_path %>">Sessions</a>
+    <div class="nav-links">
+      <a href="<%= rails_console_ai.root_path %>">Sessions</a>
+      <a href="<%= rails_console_ai.skills_path %>">Skills</a>
+      <a href="<%= rails_console_ai.memories_path %>">Memories</a>
+      <a href="<%= rails_console_ai.agents_path %>">Agents</a>
+    </div>
   </div>
   <div class="container">
+    <% if flash[:notice] %><div class="flash flash-notice"><%= flash[:notice] %></div><% end %>
+    <% if flash[:alert]  %><div class="flash flash-alert"><%= flash[:alert] %></div><% end %>
     <%= yield %>
   </div>
 </body>

data/app/views/rails_console_ai/agent_versions/index.html.erb

@@ -0,0 +1,28 @@
+<%= link_to '← Back to agent', agent_path(@agent), class: 'back-link' %>
+
+<h2 style="margin-bottom:16px;">Versions — <%= @agent.name %></h2>
+
+<% if @versions.empty? %>
+  <div class="meta-card"><p class="text-muted">No versions recorded yet.</p></div>
+<% else %>
+  <div class="versions-list">
+    <% @versions.each do |v| %>
+      <div class="ver-row">
+        <div>
+          <strong>v<%= v.id %></strong>
+          <span class="ver-meta">— <%= v.created_at.strftime('%Y-%m-%d %H:%M:%S') %> by <%= v.edited_by || 'unknown' %></span>
+          <% if v.status == 'approved' %><span class="badge status-approved">APPROVED</span>
+          <% elsif v.status == 'proposed' %><span class="badge status-proposed">PROPOSED</span><% end %>
+          <% if v.change_note.present? %><div class="ver-meta"><%= v.change_note %></div><% end %>
+        </div>
+        <div>
+          <%= link_to 'View', agent_version_path(@agent, v) %>
+          ·
+          <%= link_to 'Diff vs current', diff_agents_path(agent_id: @agent.id, from: v.id) %>
+          ·
+          <%= button_to 'Restore', restore_agent_version_path(@agent, v), method: :post, class: 'btn btn-secondary', form: { style: 'display:inline' }, data: { confirm: "Restore version ##{v.id}? Restoring reverts the agent to PROPOSED and a new version row will be created." } %>
+        </div>
+      </div>
+    <% end %>
+  </div>
+<% end %>

data/app/views/rails_console_ai/agent_versions/show.html.erb

@@ -0,0 +1,25 @@
+<%= link_to '← All versions', agent_versions_path(@agent), class: 'back-link' %>
+
+<div class="meta-card">
+  <h2 style="margin-bottom:8px;"><%= @version.name %> <span class="badge source-db">v<%= @version.id %></span></h2>
+  <p class="text-muted"><%= @version.description %></p>
+  <div class="meta-grid" style="margin-top:16px;">
+    <div class="meta-item"><label>Edited by</label><span><%= @version.edited_by || '—' %></span></div>
+    <div class="meta-item"><label>At</label><span><%= @version.created_at.strftime('%Y-%m-%d %H:%M:%S') %></span></div>
+    <div class="meta-item"><label>Status</label><span><%= @version.status || '—' %></span></div>
+    <div class="meta-item"><label>Model</label><span><%= @version.model || '(default)' %></span></div>
+    <div class="meta-item"><label>Max rounds</label><span><%= @version.max_rounds || '(default)' %></span></div>
+    <div class="meta-item"><label>Tools</label><span><%= Array(@version.tools).join(', ').presence || '(default set)' %></span></div>
+  </div>
+  <% if @version.change_note.present? %>
+    <p style="margin-top:12px;"><strong>Change note:</strong> <%= @version.change_note %></p>
+  <% end %>
+</div>
+
+<div class="btn-bar">
+  <%= button_to "Restore this version", restore_agent_version_path(@agent, @version), method: :post, class: 'btn btn-danger', data: { confirm: "Restore version ##{@version.id}? Current agent will be overwritten and status reverted to PROPOSED." } %>
+  <%= link_to 'Diff vs current', diff_agents_path(agent_id: @agent.id, from: @version.id), class: 'btn btn-secondary' %>
+</div>
+
+<h3 style="margin-top:24px; margin-bottom:8px;">Body / instructions</h3>
+<div class="markdown-body"><%= @version.body %></div>

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

@@ -0,0 +1,40 @@
+<% 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>.
+  </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 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>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;">
+
+    <div class="form-row">
+      <label>Edited by</label>
+      <input type="text" name="edited_by" value="<%= params[:edited_by] %>" placeholder="Your name or handle">
+    </div>
+
+    <div class="form-row">
+      <label>Change note (optional)</label>
+      <input type="text" name="change_note" value="" placeholder="What did you change and why?">
+    </div>
+  </div>
+
+  <div class="btn-bar">
+    <%= f.submit submit_label, class: 'btn' %>
+    <%= link_to 'Cancel', cancel_path, class: 'btn btn-secondary' %>
+  </div>
+<% end %>

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

@@ -0,0 +1,15 @@
+<%= link_to '← Back to agent', agent_path(@agent), class: 'back-link' %>
+
+<h2 style="margin-bottom:8px;">Diff — <%= @agent.name %></h2>
+<p class="text-muted" style="margin-bottom:16px;">
+  Version #<%= @from.id %> (<%= @from.created_at.strftime('%Y-%m-%d %H:%M') %>) → <%= @to_label %>
+</p>
+
+<%= 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? %>
+    <%= button_to "Restore v##{@from.id}", restore_agent_version_path(@agent, @from), method: :post, class: 'btn btn-danger', data: { confirm: 'Restoring will overwrite the current agent (a new version row is created) and revert status to PROPOSED.' } %>
+  <% end %>
+  <%= link_to 'Back', agent_path(@agent), class: 'btn btn-secondary' %>
+</div>

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

@@ -0,0 +1,7 @@
+<%= link_to '← Back to agent', agent_path(@agent), class: 'back-link' %>
+<h2 style="margin-bottom:16px;">Edit agent</h2>
+<%= render 'form',
+      form_url: agent_path(@agent),
+      form_method: :patch,
+      submit_label: 'Save changes',
+      cancel_path: agent_path(@agent) %>

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

@@ -0,0 +1,80 @@
+<div class="btn-bar">
+  <%= link_to 'New agent', new_agent_path, class: 'btn' %>
+  <% if @sort == 'used' %>
+    <%= link_to 'Sort: alphabetical', agents_path(q: @q), class: 'btn btn-secondary' %>
+  <% else %>
+    <%= link_to 'Sort: most used', agents_path(q: @q, sort: 'used'), class: 'btn btn-secondary' %>
+  <% end %>
+  <form method="get" action="<%= agents_path %>" style="margin-left:auto;">
+    <input type="text" name="q" value="<%= @q %>" placeholder="Search agents…" style="padding:6px 10px; border:1px solid #ccc; border-radius:4px;">
+    <% if @sort.present? %><input type="hidden" name="sort" value="<%= @sort %>"><% end %>
+  </form>
+</div>
+
+<% proposed_count = @agents.count { |a| a['source'] == :db && a['status'] != 'approved' } %>
+<% if proposed_count > 0 %>
+  <div class="flash flash-alert" style="margin-bottom:16px;">
+    <strong><%= proposed_count %></strong> agent<%= proposed_count == 1 ? '' : 's' %> awaiting approval. The AI cannot invoke them via delegate_task until a human approves.
+  </div>
+<% end %>
+
+<% if @agents.empty? %>
+  <div class="meta-card">
+    <p class="text-muted">No agents found. <%= link_to 'Create one', new_agent_path %> or drop a Markdown file in <code>.rails_console_ai/agents/</code>.</p>
+  </div>
+<% else %>
+  <table>
+    <thead>
+      <tr>
+        <th>Name</th>
+        <th>Description</th>
+        <th>Model</th>
+        <th>Source / Status</th>
+        <th title="Number of times delegate_task invoked this agent (DB only)">Uses</th>
+        <th>Last used</th>
+        <th></th>
+      </tr>
+    </thead>
+    <tbody>
+      <% @agents.each do |a| %>
+        <% link_id = a['id'] || a['name'] %>
+        <tr>
+          <td><strong><%= link_to a['name'], agent_path(link_id) %></strong></td>
+          <td class="query-cell"><%= a['description'] %></td>
+          <td class="mono"><%= a['model'] || '—' %></td>
+          <td>
+            <% case a['source'] %>
+            <% when :db %>
+              <span class="badge source-db">DB</span>
+              <% if a['status'] == 'approved' %>
+                <span class="badge status-approved">APPROVED</span>
+              <% else %>
+                <span class="badge status-proposed">PROPOSED</span>
+              <% end %>
+            <% when :file %>
+              <span class="badge source-file">FILE</span>
+            <% when :builtin %>
+              <span class="badge source-builtin">BUILTIN</span>
+            <% end %>
+          </td>
+          <td>
+            <% if a['source'] == :db %>
+              <%= a['use_count'] || 0 %>
+            <% else %>
+              <span class="text-muted">—</span>
+            <% end %>
+          </td>
+          <td class="text-muted" style="font-size:12px;">
+            <%= a['last_used_at']&.strftime('%Y-%m-%d %H:%M') || '—' %>
+          </td>
+          <td>
+            <%= link_to 'View', agent_path(link_id) %>
+            <% if a['source'] == :db %>
+              · <%= link_to 'Edit', edit_agent_path(a['id']) %>
+            <% end %>
+          </td>
+        </tr>
+      <% end %>
+    </tbody>
+  </table>
+<% end %>

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

@@ -0,0 +1,8 @@
+<%= link_to '← All agents', agents_path, class: 'back-link' %>
+<h2 style="margin-bottom:16px;">New agent</h2>
+
+<%= render 'form',
+      form_url: agents_path,
+      form_method: :post,
+      submit_label: 'Create agent',
+      cancel_path: agents_path %>

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

@@ -0,0 +1,108 @@
+<%
+  is_db = @agent.is_a?(RailsConsoleAi::Agent)
+  is_builtin = !is_db && @agent['source'] == :builtin
+  is_file = !is_db && @agent['source'] == :file
+
+  name = is_db ? @agent.name : @agent['name']
+  description = is_db ? @agent.description : @agent['description']
+  body = is_db ? @agent.body : @agent['body']
+  max_rounds = is_db ? @agent.max_rounds : @agent['max_rounds']
+  model = is_db ? @agent.model : @agent['model']
+  tools = is_db ? Array(@agent.tools) : Array(@agent['tools'])
+  file_key = is_db ? nil : @agent['file_key']
+%>
+
+<%= link_to '← All agents', agents_path, class: 'back-link' %>
+
+<% if is_db && !@agent.approved? %>
+  <div class="flash flash-alert" style="margin-bottom:16px;">
+    <strong>Awaiting approval.</strong> The AI cannot invoke this agent via delegate_task until a human approves it.
+    Review the body and tools list below, then click <strong>Approve</strong>.
+  </div>
+<% end %>
+
+<% if is_builtin %>
+  <div class="flash flash-notice" style="margin-bottom:16px;">
+    This is a <strong>built-in agent</strong> shipped with the gem. It's read-only here.
+    To customize it, <%= link_to 'create a same-named DB override', new_agent_path(from_builtin: name) %> — your DB agent will shadow this one.
+  </div>
+<% end %>
+
+<div class="meta-card">
+  <h2 style="margin-bottom:8px;">
+    <%= name %>
+    <% if is_db %>
+      <span class="badge source-db">DB</span>
+      <% if @agent.approved? %>
+        <span class="badge status-approved">APPROVED</span>
+      <% else %>
+        <span class="badge status-proposed">PROPOSED</span>
+      <% end %>
+    <% elsif is_builtin %>
+      <span class="badge source-builtin">BUILTIN</span>
+    <% else %>
+      <span class="badge source-file">FILE</span>
+    <% end %>
+  </h2>
+  <p class="text-muted"><%= description %></p>
+  <div class="meta-grid" style="margin-top:16px;">
+    <div class="meta-item"><label>Model</label><span><%= model || '(default)' %></span></div>
+    <div class="meta-item"><label>Max rounds</label><span><%= max_rounds || '(default)' %></span></div>
+    <div class="meta-item"><label>Tool whitelist</label><span><%= tools.empty? ? '(default set)' : tools.join(', ') %></span></div>
+    <% if is_db %>
+      <div class="meta-item"><label>Updated</label><span><%= @agent.updated_at.strftime('%Y-%m-%d %H:%M') %></span></div>
+      <div class="meta-item"><label>Versions</label><span><%= @agent.versions.count %></span></div>
+      <div class="meta-item"><label>Times invoked</label><span><%= @agent.use_count %></span></div>
+      <div class="meta-item"><label>Last invoked</label><span><%= @agent.last_used_at&.strftime('%Y-%m-%d %H:%M') || 'never' %></span></div>
+      <% if @agent.approved? %>
+        <div class="meta-item"><label>Approved by</label><span><%= @agent.approved_by || '—' %></span></div>
+        <div class="meta-item"><label>Approved at</label><span><%= @agent.approved_at&.strftime('%Y-%m-%d %H:%M') || '—' %></span></div>
+      <% end %>
+    <% else %>
+      <div class="meta-item"><label>Path</label><span class="mono"><%= file_key %></span></div>
+    <% end %>
+  </div>
+</div>
+
+<div class="btn-bar">
+  <% if is_db %>
+    <% unless @agent.approved? %>
+      <%= form_with url: approve_agent_path(@agent), method: :post, local: true, style: 'display:inline' do %>
+        <input type="text" name="approved_by" placeholder="Your name (required)" style="padding:7px 10px; border:1px solid #ccc; border-radius:4px; margin-right:6px;">
+        <button type="submit" class="btn btn-danger" data-confirm="Approve this agent? The AI will then be able to invoke it via delegate_task with the configured tool whitelist.">Approve</button>
+      <% end %>
+    <% end %>
+    <%= link_to 'Edit', edit_agent_path(@agent), class: 'btn' %>
+    <%= link_to 'Versions', agent_versions_path(@agent), class: 'btn btn-secondary' %>
+    <%= button_to 'Delete', agent_path(@agent), method: :delete, class: 'btn btn-danger', form: { style: 'display:inline' }, data: { confirm: 'Delete this agent? Past versions remain in history.' } %>
+  <% elsif is_builtin %>
+    <%= link_to 'Create DB override', new_agent_path(from_builtin: name), class: 'btn' %>
+  <% end %>
+</div>
+
+<h3 style="margin-bottom:8px;">Body / instructions</h3>
+<div class="markdown-body"><%= body %></div>
+
+<% if is_db && @versions && @versions.any? %>
+  <h3 style="margin:24px 0 8px;">Recent versions</h3>
+  <div class="versions-list">
+    <% @versions.first(5).each do |v| %>
+      <div class="ver-row">
+        <div>
+          <strong>v<%= v.id %></strong>
+          <span class="ver-meta">— <%= v.created_at.strftime('%Y-%m-%d %H:%M') %> by <%= v.edited_by || 'unknown' %></span>
+          <% if v.status == 'approved' %><span class="badge status-approved">APPROVED</span><% end %>
+          <% if v.change_note.present? %><div class="ver-meta"><%= v.change_note %></div><% end %>
+        </div>
+        <div>
+          <%= link_to 'View', agent_version_path(@agent, v) %>
+          ·
+          <%= link_to 'Diff vs current', diff_agents_path(agent_id: @agent.id, from: v.id) %>
+        </div>
+      </div>
+    <% end %>
+  </div>
+  <% if @versions.size > 5 %>
+    <p style="margin-top:8px;"><%= link_to "See all #{@versions.size} versions →", agent_versions_path(@agent) %></p>
+  <% end %>
+<% end %>

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

@@ -0,0 +1,29 @@
+<%= form_with model: @memory, url: form_url, method: form_method, local: true do |f| %>
+  <div class="form-card">
+    <div class="form-row">
+      <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;">
+
+    <div class="form-row">
+      <label>Edited by</label>
+      <input type="text" name="edited_by" value="<%= params[:edited_by] %>" placeholder="Your name or handle">
+    </div>
+
+    <div class="form-row">
+      <label>Change note (optional)</label>
+      <input type="text" name="change_note" value="" placeholder="What did you change and why?">
+    </div>
+  </div>
+
+  <div class="btn-bar">
+    <%= f.submit submit_label, class: 'btn' %>
+    <%= link_to 'Cancel', cancel_path, class: 'btn btn-secondary' %>
+  </div>
+<% end %>

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

@@ -0,0 +1,15 @@
+<%= link_to '← Back to memory', memory_path(@memory), class: 'back-link' %>
+
+<h2 style="margin-bottom:8px;">Diff — <%= @memory.name %></h2>
+<p class="text-muted" style="margin-bottom:16px;">
+  Version #<%= @from.id %> (<%= @from.created_at.strftime('%Y-%m-%d %H:%M') %>) → <%= @to_label %>
+</p>
+
+<%= 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? %>
+    <%= button_to "Restore v##{@from.id}", restore_memory_version_path(@memory, @from), method: :post, class: 'btn btn-danger', data: { confirm: 'Restoring will overwrite the current memory (a new version row is created).' } %>
+  <% end %>
+  <%= link_to 'Back', memory_path(@memory), class: 'btn btn-secondary' %>
+</div>

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

@@ -0,0 +1,7 @@
+<%= link_to '← Back to memory', memory_path(@memory), class: 'back-link' %>
+<h2 style="margin-bottom:16px;">Edit memory</h2>
+<%= render 'form',
+      form_url: memory_path(@memory),
+      form_method: :patch,
+      submit_label: 'Save changes',
+      cancel_path: memory_path(@memory) %>