rubik 0.3.5 → 0.4.1

data/lib/rubik/repl/step_formatter.rb

@@ -145,7 +145,7 @@ module Rubik
       # blocks stripped; nil when nothing human-readable remains. Shared
       # with UI::Renderer (Plan 51, Decision D2).
       def thought_first_line(content)
-        clean_content = content.to_s.sub(/```(?:json)?\s*\{.*?\}\s*```/m, "").strip
+        clean_content = content.to_s.sub(/```(?:json)?\s*\{[^`]*\}\s*```/m, "").strip
         return nil if clean_content.empty?
 
         clean_content.lines.first.to_s.strip
@@ -243,6 +243,7 @@ module Rubik
       # with UI::Renderer (Plan 51, Decision D2).
       def format_llm_thought(step_info)
         return nil if @level == LEVEL_QUIET || @level == LEVEL_NORMAL
+        return nil if step_info[:streamed] == true
 
         first_line = thought_first_line(step_info[:content])
         return nil if first_line.nil? || first_line.empty?

data/lib/rubik/runtime/diagnostic_engine.rb

@@ -144,7 +144,7 @@ module Rubik
             :dependency_missing
           elsif combined.match?(/(syntax error|unexpected token|compilation error|error\[E\d+\]|parse error|fatal error)/i)
             :syntax_or_compile_error
-          elsif combined.match?(/(failures:|assertions|tests failed|failed tests|FAILED \(|FAIL:|\d+ failures, \d+ errors)/i)
+          elsif combined.match?(/(failures:|assertions|tests failed|failed tests|FAILED \(|FAIL:|\b\d+ failures, \d+ errors\b)/i)
             :test_failure
           else
             :unknown_failure

data/lib/rubik/security/blocked_guidance.rb

@@ -164,7 +164,7 @@ module Rubik
       def self.segment_annotation(segment)
         rule = segment[:rule] || "unknown"
         reason = segment[:reason].to_s.strip
-        return "#{rule} — option #{segment[:option]} overrides to DANGEROUS" if segment[:option] && reason.match?(/overridden by option .+ to dangerous\z/i)
+        return "#{rule} — option #{segment[:option]} overrides to DANGEROUS" if segment[:option] && reason.match?(/overridden by option \S+ to dangerous\z/i)
 
         # Base-rule reasons restate matched_rule ("base rule: rm (…)") — the
         # rule is already displayed; keep only the elevation evidence.

data/lib/rubik/security/classifier/classifier.rb

@@ -22,6 +22,8 @@ module Rubik
         DEV_PATHS = ["/dev/null", "/dev/stdin", "/dev/stdout", "/dev/stderr", "/dev/fd/"].freeze
         WRITE_OPERATORS = [">", ">>", "2>", "&>", ">&", "2>>", "*>"].freeze
         READ_OPERATORS = ["<"].freeze
+        # PERF: Hoist home token array to avoid per-path array allocations in system_path? check.
+        HOME_TOKENS = ["~", "~/", "$HOME", "${HOME}"].freeze
 
         def self.default_database
           @default_database ||= CommandDatabase.new
@@ -351,7 +353,7 @@ module Rubik
 
           return true if p =~ %r{\A/(?:\*|\.+)?/?\z}
           return true if p =~ %r{\A[a-zA-Z]:(?:[\\/]|\*|\.+)?\z}i
-          return true if ["~", "~/", "$HOME", "${HOME}"].include?(p)
+          return true if HOME_TOKENS.include?(p)
 
           if p =~ %r{\A([a-zA-Z]):[/\\](.*)\z}
             parts = []

data/lib/rubik/security/redactor.rb

@@ -14,19 +14,19 @@ module Rubik
         /AIzaSy[a-zA-Z0-9_-]{33}/,
         /\bbearer\s+[a-zA-Z0-9_\-.]{20,}/i,
         /-----BEGIN [A-Z0-9_\s-]+PRIVATE KEY-----[\s\S]*?-----END [A-Z0-9_\s-]+PRIVATE KEY-----/,
-        %r{(https?://[^:]+:)([^@]+)@}
+        %r{(https?://[^\s/@:]+:)([^\s/@:]+)@}
       ].freeze
 
       ASSIGNMENT_KEY_REGEX = /\b(password|passwd|api_key|secret)(\s*[:=]\s*)(['"][^'"]*['"]|[^\s,;'"]+)/i
       ASSIGNMENT_TOKEN_REGEX = /\b(token)(\s*[:=]\s*)(['"][^'"]*['"]|[^\s,;'"]+)/i
 
       # PERF: Pre-compiled constants to avoid re-allocation inside hot loops
-      URL_PATTERN = %r{(https?://[^:]+:)([^@]+)@}
+      URL_PATTERN = %r{(https?://[^\s/@:]+:)([^\s/@:]+)@}
       URL_MASK = "\\1#{MASK_STRING}@".freeze
       STATIC_PATTERNS = PATTERNS.reject { |pattern| pattern == URL_PATTERN }.freeze
       # Skip static replacement passes for assignment-only and URL-only inputs.
       STATIC_PATTERNS_COMBINED_REGEX = Regexp.union(STATIC_PATTERNS).freeze
-      QUICK_CHECK_REGEX = %r{sk-|AKIA|ghp_|github_pat_|AIza|bearer|PRIVATE KEY|://[^:]+:|password|passwd|api_key|secret|token}i
+      QUICK_CHECK_REGEX = %r{sk-|AKIA|ghp_|github_pat_|AIza|bearer|PRIVATE KEY|://[^\s/@:]+:|password|passwd|api_key|secret|token}i
 
       # Anahtar-maskeleme desenleri (Session::Recorder#mask_payload ile aynı
       # aile): sır görünümlü anahtarların değerleri desen eşleşmese bile

data/lib/rubik/storage/layout.rb

@@ -48,7 +48,9 @@ module Rubik
 
       # [a-z0-9._-] dışını tireye çevirir; boş kalırsa "default".
       def slugify(name)
-        slug = name.to_s.downcase.gsub(/[^a-z0-9._-]+/, "-").gsub(/\A[-.]+|[-.]+\z/, "")
+        slug = name.to_s.downcase.gsub(/[^a-z0-9._-]+/, "-")
+        slug = slug.delete_prefix("-").delete_prefix(".") while slug.start_with?("-", ".")
+        slug = slug.delete_suffix("-").delete_suffix(".") while slug.end_with?("-", ".")
         slug.empty? ? "default" : slug
       end
 

data/lib/rubik/storage/render.rb

@@ -38,7 +38,7 @@ module Rubik
         return header << "_(empty — no #{type} entities yet)_\n" if summaries.empty?
 
         summaries.each do |s|
-          title = s[:title].to_s.gsub("|", "\\|").slice(0, 80)
+          title = s[:title].to_s.gsub("\\", "\\\\\\\\").gsub("|", "\\|").slice(0, 80)
           header << "| #{s[:row]} | `#{s[:id]}` | #{s[:status]} | #{s[:updated_at]} | #{title} |\n"
         end
         header

data/lib/rubik/storage/vakanuvis/vocabulary.rb

@@ -42,8 +42,11 @@ module Rubik
         # çözümleyicisi "wiki.42" gibi iki parçalı tip.satır formlarını ref
         # alias'tan ÖNCE dener — çarpışma yanlış satırı ezmesi demekti.
         def slug(text)
-          s = text.to_s.downcase.gsub(/[^a-z0-9]+/, "-").gsub(/\A-+|-+\z/, "")
-          s = s[0, SLUG_MAX].to_s.gsub(/\A-+|-+\z/, "")
+          s = text.to_s.downcase.gsub(/[^a-z0-9]+/, "-")
+          s = s.delete_prefix("-") while s.start_with?("-")
+          s = s.delete_suffix("-") while s.end_with?("-")
+          s = s[0, SLUG_MAX].to_s
+          s = s.delete_suffix("-") while s.end_with?("-")
           s = "n#{s}" if s.match?(/\A\d+\z/)
           s.empty? ? "untitled" : s
         end

data/lib/rubik/tools/base.rb

@@ -55,7 +55,8 @@ module Rubik
         filter = raw.to_s.strip.tr("\\", "/").sub(%r{^\./}, "")
         return ["", nil] if filter.empty? || filter == "." || filter == "/"
 
-        norm_root = root_dir.to_s.tr("\\", "/").sub(%r{/+\z}, "")
+        norm_root = root_dir.to_s.tr("\\", "/")
+        norm_root = norm_root.delete_suffix("/") while norm_root.end_with?("/")
         if filter.downcase == norm_root.downcase
           return ["", nil]
         elsif filter.downcase.start_with?("#{norm_root.downcase}/")

data/lib/rubik/tools/truncator.rb

@@ -41,7 +41,10 @@ module Rubik
         when :head_only
           kept_lines = lines.first(max_l)
           kept_str = kept_lines.join
-          kept_str = kept_str.byteslice(0, max_c) if kept_str.bytesize > max_c
+          if kept_str.bytesize > max_c
+            kept_str = kept_str.byteslice(0, max_c)
+            kept_str = kept_str.scrub("") unless kept_str.valid_encoding?
+          end
           kept_line_count = kept_str.empty? ? 0 : kept_str.lines(sep).size
           omitted_lines = [lines.size - kept_line_count, 0].max
           omitted_chars = [str.bytesize - kept_str.bytesize, 0].max
@@ -51,7 +54,10 @@ module Rubik
         when :tail_only
           kept_lines = lines.last(max_l)
           kept_str = kept_lines.join
-          kept_str = kept_str.byteslice(-max_c, max_c) if kept_str.bytesize > max_c
+          if kept_str.bytesize > max_c
+            kept_str = kept_str.byteslice(-max_c, max_c)
+            kept_str = kept_str.scrub("") unless kept_str.valid_encoding?
+          end
           kept_line_count = kept_str.empty? ? 0 : kept_str.lines(sep).size
           omitted_lines = [lines.size - kept_line_count, 0].max
           omitted_chars = [str.bytesize - kept_str.bytesize, 0].max
@@ -62,7 +68,10 @@ module Rubik
           if max_l <= 1
             kept_lines = lines.first(max_l)
             kept_str = kept_lines.join
-            kept_str = kept_str.byteslice(0, max_c) if kept_str.bytesize > max_c
+            if kept_str.bytesize > max_c
+              kept_str = kept_str.byteslice(0, max_c)
+              kept_str = kept_str.scrub("") unless kept_str.valid_encoding?
+            end
             kept_line_count = kept_str.empty? ? 0 : kept_str.lines(sep).size
             omitted_lines = [lines.size - kept_line_count, 0].max
             omitted_chars = [str.bytesize - kept_str.bytesize, 0].max
@@ -93,8 +102,14 @@ module Rubik
           head_str = head_lines.join
           tail_str = tail_lines.join
 
-          head_str = head_str.byteslice(0, head_chars) if head_str.bytesize > head_chars
-          tail_str = tail_str.byteslice(-tail_chars, tail_chars) if tail_str.bytesize > tail_chars
+          if head_str.bytesize > head_chars
+            head_str = head_str.byteslice(0, head_chars)
+            head_str = head_str.scrub("") unless head_str.valid_encoding?
+          end
+          if tail_str.bytesize > tail_chars
+            tail_str = tail_str.byteslice(-tail_chars, tail_chars)
+            tail_str = tail_str.scrub("") unless tail_str.valid_encoding?
+          end
 
           omitted_lines = if lines.size == 1
                             0
@@ -106,9 +121,7 @@ module Rubik
           omitted_chars = [str.bytesize - (head_str.bytesize + tail_str.bytesize), 0].max
           notice = "\n\n... [Truncated: #{omitted_lines} lines / #{omitted_chars} chars omitted to preserve token budget] ...\n\n"
 
-          head_final = head_str.valid_encoding? ? head_str : head_str.scrub("")
-          tail_final = tail_str.valid_encoding? ? tail_str : tail_str.scrub("")
-          { text: "#{head_final}#{notice}#{tail_final}", truncated: true,
+          { text: "#{head_str}#{notice}#{tail_str}", truncated: true,
             omitted_lines: omitted_lines, omitted_chars: omitted_chars }
         end
       end

data/lib/rubik/ui/renderer.rb

@@ -172,6 +172,7 @@ module Rubik
       def render_llm_thought(step_info)
         @spinner.stop
         return nil if level < Rubik::REPL::StepFormatter::LEVEL_VERBOSE
+        return nil if step_info[:streamed] == true
 
         first_line = @legacy.thought_first_line(step_info[:content])
         return nil if first_line.nil? || first_line.empty?

data/lib/rubik/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Rubik
-  VERSION = "0.3.5"
+  VERSION = "0.4.1"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rubik
 version: !ruby/object:Gem::Version
-  version: 0.3.5
+  version: 0.4.1
 platform: ruby
 authors:
 - Özgür Ulukır