aidp 0.39.8 → 0.39.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5df865d45987ecd333e79480c0329acc64ef04ffec6c8810a0e43fd94b5b9236
-  data.tar.gz: '08b02ed42782161769e3ec9bc81e7d747b97d735cbcbb3031af01df6a94b3f5a'
+  metadata.gz: 48e9a4d5c62e99992a0b61d3547b20b2a7598c13dbc6ba8b23a4f8126cd74c35
+  data.tar.gz: 9427e78702251d548a938d5b93f466b9514686128162c4be5bb52eb8375013a0
 SHA512:
-  metadata.gz: 118d2e50c63cd1bc536d3c0acd8dd01b626f174328eed7e0614e3ba13cb29315472cdc4caa0a6dda6f8672498b0bbae404e00cb11fcef40d36643b195c8b7a72
-  data.tar.gz: a0667e973b6ed0a1eaf704a7ee5bb019870d4ad2163c00ba40c2b5cd3009add2dbf9c99172b4678e0588dfd7e2a821025a1bcbaf0311b99cf0282e33b4001600
+  metadata.gz: eb8f0d861f8b85f8df71034a2e6cbef875be34dcf28eacec009aaa65c8589527b904fa060598b43c9854e63c9b354ca0299bb70b96eb5da80e36c19b6ea0a033
+  data.tar.gz: a46a2734b9094b966c242a284b1673cd956df4281294557b813925d19f8204bdf470ad93d4a1c01e27a2846de2b6544258d062f0f36dacd29607827a7bc3049c

data/lib/aidp/jobs/background_runner.rb

@@ -13,6 +13,8 @@ module Aidp
     # Manages background execution of work loops
     # Runs harness in daemon process and tracks job metadata
     class BackgroundRunner
+      StartError = Class.new(StandardError)
+
       include Aidp::MessageDisplay
       include Aidp::RescueLogging
       include Aidp::SafeDirectory
@@ -34,9 +36,11 @@ module Aidp
       # Start a background job
       # Returns job_id
       def start(mode, options = {})
+        ensure_jobs_directory
         job_id = generate_job_id
-        log_file = File.join(@jobs_dir, job_id, "output.log")
-        pid_file = File.join(@jobs_dir, job_id, "job.pid")
+        log_file = job_file_path(job_id, "output.log")
+        pid_file = job_file_path(job_id, "job.pid")
+        raise_start_error unless log_file && pid_file
 
         # Create job directory
         FileUtils.mkdir_p(File.dirname(log_file))
@@ -109,6 +113,9 @@ module Aidp
         metadata = load_job_metadata(job_id)
         return nil unless metadata
 
+        log_file = job_log_path(job_id)
+        return nil unless log_file
+
         # Check if process is still running
         pid = metadata[:pid]
         running = pid && process_running?(pid)
@@ -125,7 +132,7 @@ module Aidp
           started_at: metadata[:started_at],
           completed_at: metadata[:completed_at],
           checkpoint: checkpoint,
-          log_file: File.join(@jobs_dir, job_id, "output.log")
+          log_file: log_file
         }
       end
 
@@ -169,12 +176,12 @@ module Aidp
 
       # Get job logs
       def job_logs(job_id, options = {})
-        log_file = File.join(@jobs_dir, job_id, "output.log")
+        log_file = job_log_path(job_id)
+        return nil unless log_file
         return nil unless File.exist?(log_file)
 
         if options[:tail]
-          lines = options[:lines] || 50
-          `tail -n #{lines} #{log_file}`
+          tail_job_logs(log_file, options[:lines])
         else
           File.read(log_file)
         end
@@ -182,7 +189,8 @@ module Aidp
 
       # Follow job logs in real-time
       def follow_job_logs(job_id)
-        log_file = File.join(@jobs_dir, job_id, "output.log")
+        log_file = job_log_path(job_id)
+        return unless log_file
         return unless File.exist?(log_file)
 
         # Use tail -f to follow logs
@@ -195,6 +203,10 @@ module Aidp
         @jobs_dir = safe_mkdir_p(@jobs_dir, component_name: "BackgroundRunner")
       end
 
+      def raise_start_error
+        raise StartError, "Unable to create or access background job directory: #{@jobs_dir}"
+      end
+
       def generate_job_id
         timestamp = Time.now.strftime("%Y%m%d_%H%M%S")
         random = SecureRandom.hex(4)
@@ -202,7 +214,8 @@ module Aidp
       end
 
       def save_job_metadata(job_id, pid, mode, options)
-        metadata_file = File.join(@jobs_dir, job_id, "metadata.yml")
+        metadata_file = job_metadata_path(job_id)
+        return unless metadata_file
 
         metadata = {
           job_id: job_id,
@@ -217,7 +230,8 @@ module Aidp
       end
 
       def load_job_metadata(job_id)
-        metadata_file = File.join(@jobs_dir, job_id, "metadata.yml")
+        metadata_file = job_metadata_path(job_id)
+        return nil unless metadata_file
         return nil unless File.exist?(metadata_file)
 
         # Return raw metadata with times as ISO8601 strings to avoid unsafe class loading
@@ -231,7 +245,8 @@ module Aidp
         return unless metadata
 
         metadata.merge!(updates)
-        metadata_file = File.join(@jobs_dir, job_id, "metadata.yml")
+        metadata_file = job_metadata_path(job_id)
+        return unless metadata_file
         File.write(metadata_file, metadata.to_yaml)
       end
 
@@ -298,6 +313,57 @@ module Aidp
           metadata[:status] || "completed"
         end
       end
+
+      def job_log_path(job_id)
+        job_file_path(job_id, "output.log")
+      end
+
+      def job_metadata_path(job_id)
+        job_file_path(job_id, "metadata.yml")
+      end
+
+      def job_file_path(job_id, file_name)
+        return unless Dir.exist?(@jobs_dir)
+
+        jobs_dir = File.realpath(@jobs_dir)
+        resolved_job_dir = resolved_job_dir_path(jobs_dir, job_id)
+        return unless resolved_job_dir
+        return unless resolved_job_dir.start_with?("#{jobs_dir}/")
+
+        resolved_job_file_path(resolved_job_dir, file_name)
+      end
+
+      def tail_job_logs(log_file, requested_lines)
+        lines = Integer(requested_lines || 50, exception: false)
+        lines = 50 unless lines&.positive?
+
+        IO.popen(["tail", "-n", lines.to_s, log_file], &:read)
+      end
+
+      def resolved_job_dir_path(jobs_dir, job_id)
+        job_dir = File.join(@jobs_dir, job_id.to_s)
+
+        if File.exist?(job_dir)
+          File.realpath(job_dir)
+        else
+          File.join(jobs_dir, job_id.to_s)
+        end
+      rescue Errno::ENOENT, Errno::EACCES
+        nil
+      end
+
+      def resolved_job_file_path(job_dir, file_name)
+        candidate = File.join(job_dir, file_name)
+        return candidate unless File.exist?(candidate) || File.symlink?(candidate)
+        return if File.symlink?(candidate)
+
+        resolved_file = File.realpath(candidate)
+        return unless resolved_file.start_with?("#{job_dir}/")
+
+        resolved_file
+      rescue Errno::ENOENT, Errno::EACCES
+        nil
+      end
     end
   end
 end

data/lib/aidp/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Aidp
-  VERSION = "0.39.8"
+  VERSION = "0.39.9"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: aidp
 version: !ruby/object:Gem::Version
-  version: 0.39.8
+  version: 0.39.9
 platform: ruby
 authors:
 - Bart Agapinan