rubyn 0.1.3 → 0.1.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4a406bccd1e8bba621fd7465b792b8630c2dd586118d9116e1ad178061fe39dd
-  data.tar.gz: a49480476f8293d38a2998a0df0f6b1144ccc477dec6982ac853038a93866d0a
+  metadata.gz: 36a16de7cb6802b6a29f1a5ac6f4b6688cd5761c5ac9d3cf90986842885f3e26
+  data.tar.gz: de2d885c2cd2ba5828817b4242de06b5c05296aa264546e017d3fdf06c9e9bac
 SHA512:
-  metadata.gz: a68aa5fac1c7c708e678976e89818bc0cc108ab1b4e902d6f32ec82d71872ddea82f060db656d9829fa095cba6655b9de3b24464ec8d151fcfbf4846fde8fe83
-  data.tar.gz: edb357abf5f2e8e0c106fe35a22866db7c4a70dda97d5edc9b90ff8c93077d99d1e148f780346b5ba90b7b88cdb9ba900ef8e610e0b62186dbdb4703258af8d6
+  metadata.gz: 0cdc367ce9ff58c530f3c6fcb5ddacddcc0e7af44763258e62c4226f5531fe5b8d5b8608fa305800be01a2397241c9191e83c31f75114589a3d814da68337c3d
+  data.tar.gz: 504ae8345f735d6476576f40b16d84cb5959e3732f29c7a145a06f0c5ae7b665abd3a25bd6d13c3513c8eec31d9caad3272510b155b7508ca23be6b530b02019

data/lib/rubyn/context/file_applier.rb

@@ -18,14 +18,6 @@ module Rubyn
         end
       end
 
-      def new_file?(path)
-        return false unless path
-
-        path != @original_file &&
-          !path.end_with?("/#{@original_file}") &&
-          !@original_file.end_with?("/#{path}")
-      end
-
       private
 
       def apply_single(code)
@@ -50,7 +42,7 @@ module Rubyn
         @formatter.info("This refactor produces #{file_blocks.length} file(s):")
         file_blocks.each do |block|
           path = block[:path] || @original_file
-          label = new_file?(path) ? "NEW" : "MODIFIED"
+          label = block[:tag] == "new" ? "NEW" : "MODIFIED"
           @formatter.info("  [#{label}] #{path}")
         end
 
@@ -69,7 +61,7 @@ module Rubyn
 
       def apply_selected(block)
         path = block[:path] || @original_file
-        label = new_file?(path) ? "NEW" : "MODIFIED"
+        label = block[:tag] == "new" ? "NEW" : "MODIFIED"
         print "  Apply [#{label}] #{path}? (y/n/diff) "
         choice = $stdin.gets&.strip&.downcase
 
@@ -77,7 +69,7 @@ module Rubyn
         when "y"
           write_block(block)
         when "diff"
-          existing = new_file?(path) ? "" : File.read(resolve_path(block))
+          existing = block[:tag] == "new" ? "" : File.read(resolve_path(block))
           Rubyn::Output::DiffRenderer.render(original: existing, modified: block[:code])
           print "  Apply? (y/n) "
           write_block(block) if $stdin.gets&.strip&.downcase == "y"

data/lib/rubyn/context/response_parser.rb

@@ -3,7 +3,7 @@
 module Rubyn
   module Context
     class ResponseParser
-      BOLD_HEADER = /\*\*(?:New|Updated|Modified)\s*(?:file)?:\s*([a-zA-Z0-9_\/\.\-]+\.rb)\*\*/i
+      BOLD_HEADER = /\*\*(New|Updated|Modified)\s*(?:file)?:\s*([a-zA-Z0-9_\/\.\-]+\.rb)\*\*/i
       BACKTICK_PATH = /`([a-zA-Z0-9_\/\.\-]+\.rb)`\s*\z/
       INLINE_COMMENT = /^#\s*([a-zA-Z0-9_\/\.\-]+\.rb)/
 
@@ -24,9 +24,9 @@ module Rubyn
 
           code = part.sub(/\A```ruby\n/, "").sub(/```\z/, "")
           preceding = i > 0 ? parts[i - 1] : ""
-          path = detect_path(preceding, code)
+          result = detect_path(preceding, code)
 
-          blocks << { path: path, code: code }
+          blocks << { path: result[:path], tag: result[:tag], code: code }
         end
 
         blocks
@@ -37,19 +37,29 @@ module Rubyn
       def detect_path(preceding_text, code)
         match_bold_header(preceding_text) ||
           match_backtick_path(preceding_text) ||
-          match_inline_comment(code)
+          match_inline_comment(code) ||
+          { path: nil, tag: nil }
       end
 
       def match_bold_header(text)
-        text.match(BOLD_HEADER)&.[](1)
+        m = text.match(BOLD_HEADER)
+        return nil unless m
+
+        { path: m[2], tag: m[1].downcase }
       end
 
       def match_backtick_path(text)
-        text.match(BACKTICK_PATH)&.[](1)
+        m = text.match(BACKTICK_PATH)
+        return nil unless m
+
+        { path: m[1], tag: nil }
       end
 
       def match_inline_comment(code)
-        code.lines.first&.strip&.match(INLINE_COMMENT)&.[](1)
+        m = code.lines.first&.strip&.match(INLINE_COMMENT)
+        return nil unless m
+
+        { path: m[1], tag: nil }
       end
     end
   end

data/lib/rubyn/engine/app/assets/javascripts/rubyn/application.js

@@ -297,20 +297,22 @@
     var fileChanges = codeBlocks.map(function(code, i) {
       var header = fileHeaders[i];
       var path = (header && header.path) ? header.path : file;
-      var isNew = !!(path && file && path !== file && path.indexOf(file) === -1 && file.indexOf(path) === -1);
+      var tag = header ? header.tag : null;
       return {
         path: path,
-        isNew: isNew,
+        tag: tag,
         code: code
       };
     });
 
-    // Show each code block with its file path and NEW badge if applicable
+    // Show each code block with its file path and tag badge
     fileChanges.forEach(function(change, i) {
       html += '<div class="rubyn-code-block">';
       html += '<div class="rubyn-code-block-header">';
-      if (change.isNew) {
+      if (change.tag === "new") {
         html += '<span class="rubyn-file-tag rubyn-tag-new">NEW</span>';
+      } else if (change.tag === "updated" || change.tag === "modified") {
+        html += '<span class="rubyn-file-tag rubyn-tag-updated">UPDATED</span>';
       }
       html += '<span class="rubyn-tool-filepath">' + escapeHtml(change.path) + '</span>';
       html += '<div class="rubyn-code-block-actions">';
@@ -488,11 +490,12 @@
       var code = parts[i].replace(/^```ruby\n/, "").replace(/```$/, "");
       var preceding = i > 0 ? parts[i - 1] : "";
       var path = null;
+      var tag = null;
 
       // Strategy 1: Bold header (**New file: path.rb**, **Updated file: path.rb**)
       // Only match when preceded by New/Updated/Modified to avoid matching bold .rb references in prose
-      var boldMatch = preceding.match(/\*\*(?:New|Updated|Modified)\s*(?:file)?:\s*([a-zA-Z0-9_\/\.\-]+\.rb)\*\*/i);
-      if (boldMatch) path = boldMatch[1];
+      var boldMatch = preceding.match(/\*\*(New|Updated|Modified)\s*(?:file)?:\s*([a-zA-Z0-9_\/\.\-]+\.rb)\*\*/i);
+      if (boldMatch) { tag = boldMatch[1].toLowerCase(); path = boldMatch[2]; }
 
       // Strategy 3: Backtick-wrapped path above code block
       if (!path) {
@@ -507,7 +510,7 @@
         if (commentMatch) path = commentMatch[1];
       }
 
-      headers.push({ path: path });
+      headers.push({ path: path, tag: tag });
     }
 
     return headers;

data/lib/rubyn/engine/app/assets/stylesheets/rubyn/application.css

@@ -735,6 +735,11 @@ body {
   color: var(--rubyn-success);
 }
 
+.rubyn-tag-updated {
+  background: var(--rubyn-primary-soft);
+  color: var(--rubyn-primary);
+}
+
 
 .rubyn-code-block .rubyn-tool-output {
   margin: 0;

data/lib/rubyn/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Rubyn
-  VERSION = "0.1.3"
+  VERSION = "0.1.5"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rubyn
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.5
 platform: ruby
 authors:
 - matthewsuttles