rubik 0.1.2 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a0ec56cafd854c2e2999c68cdfdf787322ca4d7d2a2866fac4b4e8ce9f1efac7
-  data.tar.gz: ec584d8ac5ee2039423d71968cce852afb65fcb072cbdbb87325ef94693e6a5d
+  metadata.gz: a3548e6b569bb11a7c6c6349bca95300d93cd375c65860095ff18d362cd289ed
+  data.tar.gz: 207e5ed22d5180f7b219b546d0322ac2f597703dec1157ec44da48feb899ab1e
 SHA512:
-  metadata.gz: 3a535d5c0f45c74363387e7df47da8c87b1296be897fdd09adc17e4adfcc576b48555066cb0b3618d27f1eeeba052c53fe1ccb7a670598a43cad57bb8a4cc942
-  data.tar.gz: d75bb114c57fb2221f17f9d89a1a0fe1811b9e36ba52289337030b822210ef5373118e7d8438c4000da9dfca65fb4dadf3b5f85bdb8ad117ee50d56469c1a782
+  metadata.gz: a6b485c11a72537a00fbe813889f74b92e18edaa1445ec59bc3834bf6e56c377cc60ac628f059878e6d3fc0a1e27d4f63fa1f6161bac84513af4b3640a37c4b6
+  data.tar.gz: 228ce20578fc055bde12ffe0883343f21b74a9b499e718c1e953fc40c9e5054d3c21a27f5e3b986890efba08d3ae23ec77a37498fd197e37a3449de5524d0943

data/CHANGELOG.md

@@ -10,8 +10,37 @@ The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
 Nothing yet.
 
+## [0.1.3] - 2026-10-12
+
+A robustness release straight from a field incident: the `!goal` loop
+no longer strands state when the provider fails mid-goal, and the
+redactor gets measurably cheaper on secret-free input.
+
+### Fixed
+
+- **`!goal` survives provider errors (#105):** an api read-timeout
+  `ClientError` escaped the goal handler (only `Interrupt` was
+  rescued), leaving `GoalState` stuck in `:running`, the ledger
+  unfinalized and the session silent while the batch spinner kept
+  cycling with the cursor hidden. One `fail_goal` channel now serves
+  the `:error` return plus `ClientError`/`StandardError` rescues
+  (`Interrupt` still means `:stopped`); both command-dispatch branches
+  kill the leaked spinner instead of masking the real failure; and
+  goal steps record their tool calls to the session like free-text
+  turns.
+
+### Changed
+
+- **Redactor short-circuits secret-free lines (#104):** pre-compiled
+  quick-check constants plus a trigger check before scrubbing cut
+  allocations by ~68% on a mixed redact workload (172,940 -> 54,864
+  objects per 20x2000-line pass) with byte-for-byte output parity;
+  invalid-UTF-8 input stays total (masked/scrubbed) instead of raising
+  ArgumentError.
+
 ## [0.1.2] - 2026-10-11
 
+
 The Anthropic correctness release: the provider now speaks the modern
 extended-thinking contract, so pinning any current Claude model works
 instead of failing with HTTP 400.

data/lib/rubik/repl/commands/agent_execution.rb

@@ -127,6 +127,16 @@ module Rubik
                 # Adım sayacı rubik://goal/state'e aksın: olaylar bir adım
                 # içinde tekrarlanabilir, GoalState monotoniktir (zararsız).
                 goal_state.update_steps(step_info[:step], step_info[:max_steps]) if step_info[:step].is_a?(Integer)
+                # Session teşhis edilebilirliği (field incident 2026-10-12):
+                # saatler süren !goal koşuları session dosyasına tek satır
+                # düşürüyordu — free-text turn simetrisinde tool adımları
+                # kayıtçıya akar; render ve LLM geçmişi değişmez.
+                case step_info[:type]&.to_sym
+                when :tool_call
+                  ctx.recorder&.record_tool_call(step_info[:tool], step_info[:args], call_id: step_info[:call_id])
+                when :tool_result
+                  ctx.recorder&.record_tool_result(step_info[:tool], step_info[:output], status: step_info[:status] || :ok, call_id: step_info[:call_id])
+                end
                 ctx.render_step_event(step_info)
               }
             )
@@ -143,8 +153,28 @@ module Rubik
             end
 
             # Pruner apply seam: hedef döngüsü de aynı sınırda tohumlanır —
-            # aşamalı plan varsa goal promptundan ÖNCE uygulanır.
+            # aşamalı budama planı varsa goal promptundan ÖNCE uygulanır.
             ctx.apply_staged_prune_plan!
+
+            # Tek hata kanalı (field incident 2026-10-12): eskiden yalnızca
+            # normal :error dönüşü başarısızlık eşlemesini yapıyordu; loop'tan
+            # kaçan bir ClientError (sağlayıcı read timeout) goal'u kayıtsız
+            # 'running' bırakıyor, ledger terminal görmiyor ve session
+            # dosyası sessiz kalıyordu. Rescue dalları da aynı kapanış
+            # sırasını buradan çalıştırır. Kayıtçı '!goal ...' user_turn'unu
+            # komut girişinde çoktan yazdı — assistant_response olmadan
+            # session restore EŞLEŞMEMİŞ bir 'Goal: ...' turu hidrate ederdi.
+            # Adım sayısı: loop'un truncation hatası :steps'siz döner —
+            # GoalState monotonik sayacı otoriter yedek.
+            fail_goal = lambda do |reason, tokens: nil, steps: nil|
+              sanitized = Rubik::Platform.safe_utf8(reason.to_s)
+              steps_used = steps || goal_state.step_count
+              goal_state.fail!(sanitized, tokens: tokens)
+              finalize_ledger.call(:failed)
+              ctx.out.puts "\n❌ Goal could not be completed: #{sanitized}"
+              record_goal_pair.call("[Goal attempt ended after #{steps_used} steps: #{sanitized}]", tokens: tokens)
+            end
+
             ctx.active_loop = orchestrator
             begin
               res = orchestrator.run(goal_text, goal_mode: true, conversation_history: ctx.conversation_history)
@@ -170,6 +200,16 @@ module Rubik
               finalize_ledger.call(:stopped)
               record_goal_pair.call("[Goal attempt interrupted after #{goal_state.step_count} steps]")
               raise
+            rescue Rubik::LLM::ClientError => e
+              # execute_free_text simetrisi: sağlayıcı hatası (read timeout,
+              # HTTP 4xx) kaçarsa da goal terminal görür. Yeniden yükseltmez —
+              # spinner temizliği dispatch katmanına (execute_command) aittir.
+              # next: rescue sonrası akış res'siz `if res[:ok]`'a düşmesin.
+              fail_goal.call(e.message)
+              next
+            rescue StandardError => e
+              fail_goal.call("#{e.class}: #{e.message}")
+              next
             ensure
               ctx.active_loop = nil
             end
@@ -197,17 +237,10 @@ module Rubik
                 ctx.out.puts "\nTip: Use `!diff` to review changes and `!apply` to commit to disk." if ctx.buffer_manager.dirty?
               end
             else
-              # Başarısızlık eşlemesi: kayıtçı '!goal ...' user_turn'unu komut
-              # girişinde çoktan yazdı — assistant_response olmadan session
-              # restore EŞLEŞMEMİŞ bir 'Goal: ...' turu hidrate ederdi.
-              # Adım sayısı: loop'un truncation hatası :steps'siz döner —
-              # GoalState monotonik sayacı otoriter yedek.
-              sanitized = Rubik::Platform.safe_utf8(res[:error].to_s)
-              steps_used = res[:steps] || goal_state.step_count
-              goal_state.fail!(sanitized, tokens: res[:tokens])
-              finalize_ledger.call(:failed)
-              ctx.out.puts "\n❌ Goal could not be completed: #{sanitized}"
-              record_goal_pair.call("[Goal attempt ended after #{steps_used} steps: #{sanitized}]", tokens: res[:tokens])
+              # Başarısızlık eşlemesi yukarıdaki tek hata kanalından akar:
+              # GoalState → ledger finalize → kayıtçı çifti, istisna yoluyla
+              # aynı sırayla.
+              fail_goal.call(res[:error], tokens: res[:tokens], steps: res[:steps])
             end
           end
         end

data/lib/rubik/repl/input_execution.rb

@@ -56,6 +56,11 @@ module Rubik
             rescue Interrupt
               Rubik::REPL::InterruptHandler.restore_console_after_interrupt
               @output.puts "\n⚠️ [Rubik] Komut kullanıcı tarafından iptal edildi (^C)."
+            ensure
+              # !goal dahil lifecycle komutları tracked dal ile aynı spinner
+              # garantisini taşır (field incident 2026-10-12: !goal'ten kaçan
+              # ClientError'da öksüz spinner terminali kilitli gösterdi).
+              kill_command_spinner!
             end
           else
             captured = StringIO.new
@@ -70,7 +75,7 @@ module Rubik
               @output.puts "\n⚠️ [Rubik] Komut kullanıcı tarafından iptal edildi (^C)."
             ensure
               # A command (!plan/!goal) may leave a live spinner if it died mid-wait.
-              @ui_renderer&.spinner&.kill
+              kill_command_spinner!
             end
             output_text = captured.string.strip
             if !cmd_interrupted && !output_text.empty?
@@ -297,11 +302,7 @@ module Rubik
         rescue StandardError => e
           warn "[rubik] markdown tail flush failed: #{e.class}: #{e.message}"
         end
-        begin
-          renderer&.spinner&.kill
-        rescue StandardError => e
-          warn "[rubik] spinner cleanup failed: #{e.class}: #{e.message}"
-        end
+        kill_spinner!(renderer&.spinner)
         flush_subagent_notices! if respond_to?(:flush_subagent_notices!)
         # Session review B3: notices stranded in the ended loop's inbox never
         # reached the model channel and owed no wake — an idle maestro learned
@@ -318,6 +319,21 @@ module Rubik
         end
         reap_dead_kernel_units! if respond_to?(:reap_dead_kernel_units!)
       end
+
+      # Shared spinner teardown. Cleanup must never mask the in-flight
+      # exception (AGENTS.md §5): a spinner whose tick thread died holding
+      # the mutex raises ThreadError from synchronize — report it to stderr
+      # and continue so the real failure surfaces.
+      def kill_spinner!(spinner)
+        spinner&.kill
+      rescue StandardError => e
+        warn "[rubik] spinner cleanup failed: #{e.class}: #{e.message}"
+      end
+
+      # Command-dispatch spinner cleanup (both branches).
+      def kill_command_spinner!
+        kill_spinner!(@ui_renderer&.spinner)
+      end
     end
   end
 end

data/lib/rubik/security/redactor.rb

@@ -20,6 +20,11 @@ module Rubik
       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_MASK = "\\1#{MASK_STRING}@".freeze
+      QUICK_CHECK_REGEX = %r{sk-|AKIA|ghp_|github_pat_|AIza|bearer|PRIVATE KEY|://[^:]+:|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
       # maskelenir; sayaç anahtarları (tokens, max_tokens) istisnadır.
@@ -33,10 +38,17 @@ module Rubik
         # scrub: redact ham geçmişe (tool çıktısı, prompt) dokunur — geçersiz
         # UTF-8 gsub!'ı ArgumentError ile düşürür; total olmak maskenin
         # kendisi bir arıza kaynağı olmasın demektir.
-        result = text.dup.scrub
+        result = text.valid_encoding? ? text.dup : text.scrub
+
+        # PERF: kısa devre — sır tetikleyicisi yoksa 11 gsub turu atlanır.
+        # Tetik testi scrub'lanmış sonuç üstünde yapılır: match? ham geçersiz
+        # UTF-8'de ArgumentError düşürür; scrub sonrası metin hep geçerlidir
+        # ve desenler zaten bu metne uygulanıyordu, dolayısıyla eşdeğerdir.
+        return result unless result.match?(QUICK_CHECK_REGEX)
+
         PATTERNS.each do |pattern|
-          if pattern == %r{(https?://[^:]+:)([^@]+)@}
-            result.gsub!(pattern, "\\1#{MASK_STRING}@")
+          if pattern == URL_PATTERN
+            result.gsub!(pattern, URL_MASK)
           else
             result.gsub!(pattern, MASK_STRING)
           end

data/lib/rubik/version.rb

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

metadata

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