pwn 0.5.643 → 0.5.650

data/lib/pwn/ai/agent/tools/ruby_eval.rb

@@ -39,19 +39,42 @@ PWN::AI::Agent::Registry.register(
       # INTENTIONAL: this IS the pwn-ai → PWN bridge
       # As YTCracker says, "It ain't a bug, it's a featcha."
       # https://www.youtube.com/watch?v=2nALqqSqdDw
-      # val = eval(code, TOPLEVEL_BINDING, '(pwn_eval)')
+      #
+      # File label '(pwn_eval)' (not __FILE__) so SyntaxError / NameError
+      # messages cite the *payload* line, not ruby_eval.rb:49 glued to source
+      # (e.g. "ruby_eval:49results = probes.map do |R|").
+      # Rescue ScriptError (SyntaxError, LoadError, NotImplementedError) as
+      # well as StandardError — Dispatch only catches StandardError, so an
+      # unrescued SyntaxError previously escaped the tool loop. Returning a
+      # structured error lets the model self-heal (fix code and retry).
+      # rubocop:disable Style/EvalWithLocation -- intentional file label
+      # '(pwn_eval)' so SyntaxError/NameError messages cite the payload, not
+      # this library line (specs + model self-heal depend on that string).
       proc = eval(
-        "proc { #{code} }",
+        "proc { #{code}\n}",
         TOPLEVEL_BINDING,
-        __FILE__,
-        __LINE__ - 3
+        '(pwn_eval)',
+        1
       )
+      # rubocop:enable Style/EvalWithLocation
       val = proc.call
       # rubocop:enable Security/Eval
       # rubocop:enable Style/DocumentDynamicEvalDefinition
       { stdout: buf.string, value: val.inspect }
-
-      # TODO: A rescue here may enable self-healing of the agent if the model emits code that raises an exception. The model could then be prompted to fix the code and try again.
+    rescue ScriptError, StandardError => e
+      {
+        stdout: buf.string,
+        error: "#{e.class}: #{e.message}",
+        backtrace: Array(e.backtrace).first(5),
+        # Hint common payload footguns the model keeps emitting
+        hint: (
+          if e.is_a?(SyntaxError) && e.message.match?(/formal argument cannot be a constant/)
+            'Block parameters must be local variables (lowercase), e.g. |r| not |R|. Also close every do/end.'
+          elsif e.is_a?(SyntaxError)
+            'Payload failed to parse. Check matching do/end, braces, and that no line-number prefix was glued onto the code.'
+          end
+        )
+      }
     ensure
       $stdout = old_stdout
     end

data/lib/pwn/cron.rb

@@ -231,8 +231,11 @@ module PWN
         )
       end
 
-      # P3 — backfill ORM/PRM labels when local :failure_only introspect is on
-      unless names.include?('curriculum_offline_judge')
+      # P3 — backfill ORM/PRM labels when local :failure_only introspect is on.
+      # P13 — treat legacy alias offline_judge_nightly as the same job so
+      # install_defaults never double-seeds the 30 3 * * * slot.
+      offline_names = %w[curriculum_offline_judge offline_judge_nightly]
+      unless names.intersect?(offline_names)
         seeded << create(
           name: 'curriculum_offline_judge',
           schedule: '30 3 * * *',
@@ -241,6 +244,15 @@ module PWN
           enabled: true
         )
       end
+      # Disable duplicate alias if both exist (idempotent cleanup).
+      if names.include?('curriculum_offline_judge') && names.include?('offline_judge_nightly')
+        begin
+          dup = jobs.values.find { |j| j[:name].to_s == 'offline_judge_nightly' }
+          disable(id: dup[:id]) if dup && dup[:enabled]
+        rescue StandardError
+          nil
+        end
+      end
 
       # M1/M3 — nightly memory GC so the injected MEMORY block stays high-signal
       unless names.include?('learning_consolidate_nightly')

data/lib/pwn/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module PWN
-  VERSION = '0.5.643'
+  VERSION = '0.5.650'
 end