ligarb 0.5.0 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8e697032ff94ff3c4a910343f29d6a977bd386afaa92dbcf4dbb0ca0108e70b2
-  data.tar.gz: d142ecba5fb0cb7287f2206e31309c40dff8016b216f79053669f5fa8bc59c47
+  metadata.gz: 4aa971885dcefac0f440b02705cc3cb0adb88fcbf21c38cc23bed207887f30dc
+  data.tar.gz: 155c53e158b727d4e7c5ef5b012b980e32106e54e737c8c55b4f3953c08caeb2
 SHA512:
-  metadata.gz: 6a24c126ae03426cd2361885244882a08b5e666cd112fa77b437225a48d954adb0ec2ea59d3c620b317a9549ea78efa17bfcde367c8aed9bd69be79f6b7f45b9
-  data.tar.gz: b77a95db5ec9f3541349d999b9c936ddad7b65df844a3ec15f0059698e05f8973cabde8ac2b3fe60598acfa43ab866c70162bee0c1ce46bc7350be41ec195253
+  metadata.gz: a04d10091801975a7029c762a81fed663b685710634a29320b8f6cd0a149297627e1c6fc2d28efd0c69d6fd44576ec5c535764b88fdcd62411ad0ec4f06dae35
+  data.tar.gz: b60a0faef98596347c5c4e6b25b2737a00a47b9998124a562d2d1b918648e1ad46560008e84426cc216d286d6e432d528e14d9f3f1ac3be595cab1463c5fbc54

data/assets/review.css

@@ -203,6 +203,23 @@
   margin-top: 4px;
 }
 
+.ligarb-file-path {
+  font-size: 10px;
+  color: var(--color-text-muted, #999);
+  margin-top: 2px;
+}
+
+.ligarb-file-path summary {
+  cursor: pointer;
+  opacity: 0.5;
+}
+
+.ligarb-file-path code {
+  font-size: 10px;
+  word-break: break-all;
+  user-select: all;
+}
+
 /* ── Messages ── */
 
 .ligarb-messages {

data/assets/review.js

@@ -269,9 +269,11 @@
 
   function renderReview(review) {
     var ctx = review.context || {};
+    var filePath = review.file_path ? '<details class="ligarb-file-path"><summary>debug</summary><code>' + escapeHTML(review.file_path) + '</code></details>' : '';
     panel.querySelector('.ligarb-context').innerHTML =
       '<div class="ligarb-selected-text">"' + escapeHTML(ctx.selected_text || '') + '"</div>' +
-      '<div class="ligarb-meta">Chapter: ' + escapeHTML(ctx.chapter_slug || '') + '</div>';
+      '<div class="ligarb-meta">Chapter: ' + escapeHTML(ctx.chapter_slug || '') + '</div>' +
+      filePath;
 
     var msgsEl = panel.querySelector('.ligarb-messages');
     msgsEl.innerHTML = '';
@@ -331,7 +333,7 @@
     var patches = '';
 
     parts.forEach(function(part) {
-      var m = part.match(/<patch(?:\s+file="([^"]*)")?>[\s\S]*?<<<\n([\s\S]*?)\n===\n([\s\S]*?)\n>>>\s*<\/patch>/);
+      var m = part.match(/<patch(?:\s+file="([^"]*)")?>\s*<<<[ \t]*\r?\n([\s\S]*?)\r?\n===[ \t]*\r?\n([\s\S]*?)\r?\n>>>[ \t]*\s*<\/patch>/);
       if (m) {
         hasPatch = true;
         var fileLabel = m[1] ? '<div class="ligarb-patch-file">' + escapeHTML(m[1]) + '</div>' : '';
@@ -505,6 +507,7 @@
       }
 
       body.innerHTML = '';
+      reviews.reverse();
       reviews.forEach(function(r) {
         var item = document.createElement('div');
         item.className = 'ligarb-list-item ligarb-list-' + r.status;

data/assets/serve.js

@@ -53,6 +53,27 @@
             });
           }
           window.scrollTo(0, scrollY);
+
+          // Re-initialize syntax highlighting and special blocks
+          if (typeof hljs !== 'undefined') hljs.highlightAll();
+          if (typeof mermaid !== 'undefined') {
+            var unrendered = oldMain.querySelectorAll('.mermaid:not([data-processed])');
+            if (unrendered.length > 0) mermaid.run({nodes: unrendered});
+          }
+          if (typeof katex !== 'undefined') {
+            oldMain.querySelectorAll('.math-block[data-math]').forEach(function(el) {
+              if (el.childNodes.length === 0) {
+                try { katex.render(el.getAttribute('data-math'), el, {displayMode: true, throwOnError: false}); }
+                catch(e) { el.textContent = el.getAttribute('data-math'); }
+              }
+            });
+            oldMain.querySelectorAll('.math-inline[data-math]').forEach(function(el) {
+              if (el.childNodes.length === 0) {
+                try { katex.render(el.getAttribute('data-math'), el, {displayMode: false, throwOnError: false}); }
+                catch(e) { el.textContent = el.getAttribute('data-math'); }
+              }
+            });
+          }
         }
         refreshing = false;
         hideReloadButton();

data/assets/style.css

@@ -447,6 +447,7 @@ mark.search-highlight {
   --color-accent-light: #1c3a5c;
   --color-code-bg: #0f172a;
   --color-code-border: #2d3748;
+  --color-highlight: rgba(255, 243, 205, 0.15);
 }
 
 [data-theme="dark"] .admonition-note {
@@ -658,6 +659,17 @@ mark.search-highlight {
   text-decoration: underline;
 }
 
+.ligarb-highlight {
+  border-radius: 4px;
+  padding: 4px 6px;
+  animation: ligarb-highlight-fade 3s ease-out forwards;
+}
+
+@keyframes ligarb-highlight-fade {
+  0% { background: var(--color-highlight, #fff3cd); }
+  100% { background: transparent; }
+}
+
 .cite-ref {
   font-size: 0.8em;
 }
@@ -815,3 +827,27 @@ mark.search-highlight {
     }
   }
 }
+
+/* === Mermaid source toggle === */
+details.mermaid-source {
+  margin-top: 0.3em;
+  font-size: 0.8em;
+}
+details.mermaid-source summary {
+  cursor: pointer;
+  color: #888;
+  user-select: none;
+}
+details.mermaid-source pre {
+  margin-top: 0.3em;
+  padding: 0.5em;
+  background: #f5f5f5;
+  border: 1px solid #ddd;
+  border-radius: 4px;
+  overflow-x: auto;
+  white-space: pre-wrap;
+}
+.dark-mode details.mermaid-source pre {
+  background: #2a2a2a;
+  border-color: #444;
+}

data/lib/ligarb/asset_manager.rb

@@ -9,7 +9,7 @@ module Ligarb
   class AssetManager
     ASSETS = {
       highlight: {
-        fence_pattern: /language-(?!mermaid|math)(\w+)/,
+        fence_pattern: /language-(?!mermaid|math|functionplot)(\w+)/,
         files: {
           "js/highlight.min.js" => "https://cdnjs.cloudflare.com/ajax/libs/highlight.js/11.11.1/highlight.min.js",
           "css/highlight.css" => "https://cdnjs.cloudflare.com/ajax/libs/highlight.js/11.11.1/styles/github.min.css",
@@ -28,6 +28,13 @@ module Ligarb
           "css/katex.min.css" => "https://cdn.jsdelivr.net/npm/katex@0.16/dist/katex.min.css",
         },
       },
+      functionplot: {
+        fence_pattern: /class="functionplot"/,
+        files: {
+          "js/d3.min.js" => "https://cdn.jsdelivr.net/npm/d3@7/dist/d3.min.js",
+          "js/function-plot.min.js" => "https://cdn.jsdelivr.net/npm/function-plot@1/dist/function-plot.js",
+        },
+      },
     }.freeze
 
     def initialize(output_path)
@@ -60,6 +67,8 @@ module Ligarb
 
     private
 
+    MAX_ASSET_SIZE = 5 * 1024 * 1024 # 5MB
+
     def download(url, dest)
       FileUtils.mkdir_p(File.dirname(dest))
       $stderr.print "Downloading #{File.basename(dest)}... "
@@ -68,6 +77,9 @@ module Ligarb
       response = fetch_with_redirects(uri)
 
       if response.is_a?(Net::HTTPSuccess)
+        if response.body.bytesize > MAX_ASSET_SIZE
+          abort "Error: downloaded file too large: #{File.basename(dest)} (#{response.body.bytesize} bytes, limit #{MAX_ASSET_SIZE})"
+        end
         File.write(dest, response.body)
         $stderr.puts "done"
       else
@@ -77,11 +89,14 @@ module Ligarb
 
     def fetch_with_redirects(uri, limit = 5)
       raise "Too many redirects" if limit == 0
+      raise "Only HTTPS URLs are supported for asset downloads" unless uri.scheme == "https"
 
       response = Net::HTTP.get_response(uri)
       case response
       when Net::HTTPRedirection
-        fetch_with_redirects(URI(response["location"]), limit - 1)
+        new_uri = URI(response["location"])
+        raise "Redirect to non-HTTPS URL: #{new_uri}" unless new_uri.scheme == "https"
+        fetch_with_redirects(new_uri, limit - 1)
       else
         response
       end

data/lib/ligarb/chapter.rb

@@ -166,14 +166,18 @@ module Ligarb
     end
 
     def convert_special_code_blocks(html)
-      html.gsub(%r{<pre><code class="language-(mermaid|math)">(.*?)</code></pre>}m) do
+      html.gsub(%r{<pre><code class="language-(mermaid|math|functionplot)">(.*?)</code></pre>}m) do
         lang = $1
         raw = decode_entities($2)
         case lang
         when "mermaid"
-          %(<div class="mermaid">\n#{raw}</div>)
+          escaped = raw.gsub("&", "&amp;").gsub("<", "&lt;").gsub(">", "&gt;")
+          %(<div class="mermaid">\n#{raw}</div>) +
+            %(<details class="mermaid-source"><summary>mermaid source</summary><pre>#{escaped}</pre></details>)
         when "math"
           %(<div class="math-block" data-math="#{encode_attr(raw)}"></div>)
+        when "functionplot"
+          %(<div class="functionplot" data-plot="#{encode_attr(raw)}"></div>)
         end
       end
     end

data/lib/ligarb/claude_runner.rb

@@ -6,14 +6,21 @@ require_relative "cli"
 
 module Ligarb
   class ClaudeRunner
-    PATCH_RE = %r{<patch(?:\s+file="([^"]*)")?>\s*<<<\n(.*?)\n===\n(.*?)\n>>>\s*</patch>}m
+    PATCH_RE = %r{<patch(?:\s+file="([^"]*)")?>\s*<<<[ \t]*\r?\n(.*?)\r?\n===[ \t]*\r?\n(.*?)\r?\n>>>[ \t]*\s*</patch>}m
 
     def initialize(config)
       @config = config
     end
 
     def installed?
-      system("claude", "--version", out: File::NULL, err: File::NULL)
+      claude_path = which("claude")
+      return :not_found unless claude_path
+
+      if system(claude_path, "--version", out: File::NULL, err: File::NULL)
+        true
+      else
+        :version_failed
+      end
     end
 
     # Run claude -p with the given prompt. Returns the text response.
@@ -88,9 +95,14 @@ module Ligarb
 
     # Extract patches from the last assistant message and apply them.
     # Supports cross-chapter patches via the file attribute.
+    # Uses transactional approach: all patches applied in memory first,
+    # then written to disk. On build failure, changes are rolled back.
     def apply_patches(review)
       patches = extract_patches(review)
-      return { "error" => "No patches found in the conversation" } if patches.empty?
+      if patches.empty?
+        hint = has_unmatched_patches?(review) ? " (patch tags found but format didn't match)" : ""
+        return { "error" => "No patches found in the conversation#{hint}" }
+      end
 
       default_source = review.dig("context", "source_file")
 
@@ -110,37 +122,62 @@ module Ligarb
 
       return { "error" => "No valid target files found for patches" } if file_patches.empty?
 
+      use_git = git_available?
+      target_files = file_patches.keys
+
+      # Check for uncommitted changes when git is available
+      if use_git
+        dirty = git_dirty_files(target_files)
+        unless dirty.empty?
+          return { "error" => "Cannot apply patches: uncommitted changes in #{dirty.join(", ")}" }
+        end
+      end
+
+      # Phase 1: Apply all patches in memory
       applied = 0
       total = patches.size
+      results = {} # file => new_content
+      backups = {} # file => original_content
 
       file_patches.each do |file, file_patch_list|
-        unless File.exist?(file)
-          next
-        end
+        next unless File.exist?(file)
 
         content = File.read(file)
-        changed = false
+        backups[file] = content
 
         file_patch_list.each do |old_text, new_text|
           if content.include?(old_text)
             content = content.sub(old_text, new_text)
             applied += 1
-            changed = true
           end
         end
 
-        File.write(file, content) if changed
+        results[file] = content if content != backups[file]
       end
 
       return { "error" => "No patches matched the source files (0/#{total})" } if applied == 0
 
-      # Rebuild
+      # Phase 2: Write all files at once
+      results.each { |file, content| File.write(file, content) }
+
+      # Phase 3: Rebuild
       config_path = File.join(@config.base_dir, "book.yml")
       require_relative "builder"
       begin
         Builder.new(config_path).build
       rescue SystemExit => e
-        return { "error" => "Applied #{applied}/#{total} patch(es) but rebuild failed: #{e.message}" }
+        # Rollback on build failure
+        if use_git
+          git_rollback_files(results.keys)
+        else
+          backups.each { |file, content| File.write(file, content) if results.key?(file) }
+        end
+        return { "error" => "Applied #{applied}/#{total} patch(es) but rebuild failed (rolled back): #{e.message}" }
+      end
+
+      # Phase 4: Commit on success
+      if use_git
+        git_commit_patches(results.keys, review)
       end
 
       { "text" => "Applied #{applied}/#{total} patch(es) and rebuilt." }
@@ -171,11 +208,102 @@ module Ligarb
       []
     end
 
+    # Check if assistant messages contain <patch> tags that didn't match PATCH_RE.
+    def has_unmatched_patches?(review)
+      (review["messages"] || [])
+        .select { |m| m["role"] == "assistant" }
+        .any? { |m| m["content"].include?("<patch") && m["content"].include?("</patch>") }
+    end
+
+    # Check if git is available in the project directory.
+    def git_available?
+      system("git", "rev-parse", "--git-dir",
+             chdir: @config.base_dir, out: File::NULL, err: File::NULL)
+    end
+
     private
 
+    # Return list of target files that have uncommitted changes.
+    def git_dirty_files(files)
+      files.select do |file|
+        rel = relative_to_base(file)
+        next false unless rel
+        out, = Open3.capture2("git", "status", "--porcelain", "--", rel, chdir: @config.base_dir)
+        !out.strip.empty?
+      end
+    end
+
+    # Commit patched files with a message including review context.
+    def git_commit_patches(files, review)
+      rel_files = files.filter_map { |f| relative_to_base(f) }
+      return if rel_files.empty?
+
+      system("git", "add", "--", *rel_files, chdir: @config.base_dir)
+
+      message = build_commit_message(review, rel_files)
+      system("git", "commit", "-m", message, chdir: @config.base_dir,
+             out: File::NULL, err: File::NULL)
+    end
+
+    # Rollback files using git checkout.
+    def git_rollback_files(files)
+      rel_files = files.filter_map { |f| relative_to_base(f) }
+      return if rel_files.empty?
+
+      system("git", "checkout", "--", *rel_files, chdir: @config.base_dir,
+             out: File::NULL, err: File::NULL)
+    end
+
+    def build_commit_message(review, rel_files)
+      messages = review["messages"] || []
+      user_msg = messages.find { |m| m["role"] == "user" }&.dig("content").to_s
+      assistant_msg = messages.select { |m| m["role"] == "assistant" }.last&.dig("content").to_s
+
+      source = review.dig("context", "source_file") || "unknown"
+      source_rel = relative_to_base(source) || source
+
+      lines = ["[ligarb] Review: #{source_rel}"]
+      lines << ""
+      lines << "User: #{truncate_message(user_msg)}" unless user_msg.empty?
+      lines << "Claude: #{truncate_message(assistant_msg)}" unless assistant_msg.empty?
+      lines << ""
+      lines << "Files: #{rel_files.join(", ")}"
+      lines.join("\n")
+    end
+
+    def truncate_message(text, max_lines: 3, max_chars: 200)
+      truncated = text.lines.first(max_lines).join.strip
+      truncated = truncated[0, max_chars] + "..." if truncated.length > max_chars
+      truncated
+    end
+
+    def relative_to_base(file)
+      abs = File.expand_path(file)
+      base = File.expand_path(@config.base_dir)
+      return nil unless abs.start_with?(base + "/")
+      abs[(base.length + 1)..]
+    end
+
+    def which(cmd)
+      ENV["PATH"].to_s.split(File::PATH_SEPARATOR).each do |dir|
+        path = File.join(dir, cmd)
+        return path if File.executable?(path)
+      end
+      nil
+    end
+
     # Resolve a relative path from a patch to an absolute path.
+    # Prevents path traversal outside base_dir.
     def resolve_patch_file(rel_path)
-      absolute = File.join(@config.base_dir, rel_path)
+      absolute = File.expand_path(rel_path, @config.base_dir)
+      base_dir = File.expand_path(@config.base_dir)
+
+      # Reject paths that escape base_dir (e.g. "../../etc/passwd")
+      unless absolute.start_with?(base_dir + "/")
+        $stderr.puts "Warning: patch path '#{rel_path}' resolves outside project directory, skipping"
+        return nil
+      end
+
       return absolute if File.exist?(absolute)
 
       # Try matching by basename against all chapter paths

data/lib/ligarb/cli.rb

@@ -22,8 +22,18 @@ module Ligarb
         config_paths = ["book.yml"] if config_paths.empty?
         port_idx = args.index("--port")
         port = port_idx ? args[port_idx + 1].to_i : 3000
+        abort "Error: port must be 1-65535" unless (1..65535).include?(port)
+        multi = args.include?("--multi")
         require_relative "server"
-        Server.new(config_paths, port: port).start
+        Server.new(config_paths, port: port, multi: multi).start
+      when "librarium"
+        config_paths = Dir.glob("*/book.yml").sort
+        abort "Error: no */book.yml found in current directory" if config_paths.empty?
+        port_idx = args.index("--port")
+        port = port_idx ? args[port_idx + 1].to_i : 3000
+        abort "Error: port must be 1-65535" unless (1..65535).include?(port)
+        require_relative "server"
+        Server.new(config_paths, port: port, multi: true).start
       when "write"
         require_relative "writer"
         begin
@@ -59,6 +69,7 @@ module Ligarb
           ligarb init [DIRECTORY]  Create a new book project
           ligarb build [CONFIG]    Build the HTML book (default CONFIG: book.yml)
           ligarb serve [CONFIG]   Serve the book with live reload and review UI
+          ligarb librarium       Serve all */book.yml as a multi-book library
           ligarb write [BRIEF]         Generate a book with AI from brief.yml
           ligarb write --init [DIR]    Create DIR/brief.yml template
           ligarb help              Show detailed specification (for AI integration)
@@ -127,9 +138,10 @@ module Ligarb
                                 Multiple CONFIG paths can be given to serve multiple books.
                                 Options:
                                   --port PORT  Server port (default: 3000)
-                                Single book mode (1 CONFIG):
+                                  --multi      Force multi-book mode (even with 1 CONFIG)
+                                Single book mode (1 CONFIG, without --multi):
                                 - Serves the built HTML book at http://localhost:PORT
-                                Multi-book mode (2+ CONFIGs):
+                                Multi-book mode (2+ CONFIGs, or --multi):
                                 - Top page (/) shows a book index with links
                                 - Each book is served at /<directory-name>/
                                 - "Write a new book" button on the index page to generate
@@ -146,6 +158,12 @@ module Ligarb
                                 - Review patches can span multiple chapters and the
                                   bibliography file (Claude reads book.yml to find all files)
 
+        ligarb librarium        Start a multi-book library server.
+                                Automatically discovers */book.yml in the current directory.
+                                Equivalent to: ligarb serve --multi */book.yml
+                                Options:
+                                  --port PORT  Server port (default: 3000)
+
         ligarb help             Show this detailed specification.
 
         ligarb --help           Show short usage summary.
@@ -292,9 +310,11 @@ module Ligarb
         and build/css/.
 
         ```ruby, ```python, etc.   Syntax highlighting (highlight.js, BSD-3-Clause)
-        ```mermaid                  Diagrams: flowcharts, sequence, class, etc.
+        ```mermaid                  Diagrams: flowcharts, sequence, bar/line/pie charts,
+                                    gantt, mindmap, etc.
                                     (mermaid, MIT)
         ```math                     LaTeX math equations (KaTeX, MIT)
+        ```functionplot             Function graphs (function-plot + d3, MIT)
 
         These are rendered visually in the output HTML — use them freely.
 
@@ -315,6 +335,64 @@ module Ligarb
                 Server-->>Client: Response
             ```
 
+        Mermaid example (bar chart):
+
+            ```mermaid
+            xychart
+                title "Monthly Sales"
+                x-axis ["Jan", "Feb", "Mar", "Apr", "May"]
+                y-axis "Revenue" 0 --> 500
+                bar [120, 230, 180, 350, 410]
+                line [120, 230, 180, 350, 410]
+            ```
+
+        Mermaid example (line chart, line only):
+
+            ```mermaid
+            xychart
+                title "Temperature"
+                x-axis ["Jan", "Feb", "Mar", "Apr", "May", "Jun"]
+                y-axis "°C" -5 --> 30
+                line [2, 4, 10, 16, 22, 26]
+            ```
+
+        Mermaid example (pie chart):
+
+            ```mermaid
+            pie title Browser Share
+                "Chrome" : 65
+                "Safari" : 19
+                "Firefox" : 4
+                "Other" : 12
+            ```
+
+        Mermaid example (gantt chart):
+
+            ```mermaid
+            gantt
+                title Project Plan
+                dateFormat YYYY-MM-DD
+                section Design
+                    Requirements :a1, 2025-01-01, 14d
+                    Architecture :a2, after a1, 10d
+                section Dev
+                    Implementation :b1, after a2, 21d
+                    Testing        :b2, after b1, 14d
+            ```
+
+        Mermaid example (mindmap):
+
+            ```mermaid
+            mindmap
+                root((Project))
+                    Frontend
+                        React
+                        CSS
+                    Backend
+                        API
+                        Database
+            ```
+
         Math example (KaTeX, LaTeX syntax):
 
             ```math
@@ -332,6 +410,28 @@ module Ligarb
         - Content inside <code> and <pre> is not affected
         - The content is rendered with KaTeX (displayMode: false)
 
+        Function plot example:
+
+            ```functionplot
+            y = sin(x)
+            y = x^2 - 1
+            range: [-2pi, 2pi]
+            yrange: [-3, 3]
+            ```
+
+        Function plot syntax:
+        - y = <expr>                   Standard function (e.g. y = sin(x))
+        - r = <expr>                   Polar function (e.g. r = cos(2*theta))
+        - parametric: <x>, <y>         Parametric curve (e.g. parametric: cos(t), sin(t))
+        - Bare expression              Treated as y = <expr>
+        Options (one per line):
+        - range / xrange: [min, max]   X-axis range (supports pi, e.g. [-2pi, 2pi])
+        - yrange: [min, max]           Y-axis range
+        - width: <pixels>              Plot width (default: 600)
+        - height: <pixels>             Plot height (default: 400)
+        - title: <text>                Plot title
+        - grid: true                   Show grid lines
+
         == Images ==
 
         Place image files in the 'images/' directory next to book.yml.

data/lib/ligarb/initializer.rb

@@ -34,6 +34,7 @@ module Ligarb
       File.write(book_yml, generate_book_yml(title, chapter_paths))
       File.write(File.join(target, "images", ".gitkeep"), "")
 
+      init_git(target)
       print_success(target, chapter_paths)
     end
 
@@ -86,6 +87,25 @@ module Ligarb
       puts "  Run 'ligarb build' to generate HTML"
     end
 
+    def init_git(target)
+      unless system("git", "rev-parse", "--git-dir", out: File::NULL, err: File::NULL, chdir: target)
+        write_gitignore(target)
+        system("git", "init", chdir: target)
+        puts "Initialized git repository"
+      end
+    end
+
+    def write_gitignore(target)
+      gitignore = File.join(target, ".gitignore")
+      return if File.exist?(gitignore)
+
+      File.write(gitignore, <<~IGNORE)
+        # Generated by ligarb - remove lines as needed
+        build/
+        .ligarb/
+      IGNORE
+    end
+
     def relative_path(target)
       if @directory && @directory != "."
         @directory.start_with?("/") ? @directory : "./#{@directory}"