sxn 0.2.0 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8c4ab07a7db971653928d5f28ad031c215d1366fa3317d8a4ab3c53fe6c10935
-  data.tar.gz: ece376145596157b9f4c2be71508334f80ce85f542ee40ca9f5af17db6ee3202
+  metadata.gz: b9758f35bc9f122c019b15dd8a46f712cda58eb86495c6e5bb8c63791b10525e
+  data.tar.gz: 628449e55edf8b33ade3f83cdfab4369234bd22e26e5625cc3890b3c46a74702
 SHA512:
-  metadata.gz: 508ece263bcb4fe72595aded4fee96243fe7f440f6ee1c6d91dcc9bfe619de98dac48dd8fd5d717ee3a11e866d057c7cfb8c40d0b1ec6383ad19d9f6f6d2cd20
-  data.tar.gz: 0e319461455ed3e5212dd1f67e76f94aff6c6a8292d4f3512d4d5a0be81d3b71aabe803b6444cc744f03f23814087b25816e433234c3551c150f3445387e21d0
+  metadata.gz: 7970fbb77d1b7b21da3b93d566bb54a42219c787ca081b2319ec6c3dcf7b31dee35761339da42322ac2acf393232fa458325e3573dfebec546caa2e14f9da0eb
+  data.tar.gz: bd4f880ea4b2aa55b1a0de1d87d6e6bcc41a787218d29c104c644c87d07d2c43048b77217ba6ceae3f3df66a2da63c531a6b7f6cc96781d1454f0a2622232d1a

data/.parallel_rspec

@@ -0,0 +1,2 @@
+--format progress
+--format ParallelTests::RSpec::RuntimeLogger --out tmp/parallel_runtime_rspec.log

data/.parallel_rspec_options

@@ -0,0 +1,3 @@
+spec/unit
+spec/integration
+spec/performance

data/.rspec

@@ -1,4 +1,5 @@
 --require spec_helper
 --color
 --format documentation
---order random
+--order random
+--pattern '**/*_spec.rb'

data/CHANGELOG.md

@@ -5,6 +5,14 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.2.1] - 2025-01-20
+
+### Fixed
+- Fixed SQLite3 datatype mismatch error when listing sessions
+- Fixed `sxn list` showing no sessions even when sessions exist
+- Improved type coercion for database parameters
+- Enhanced error logging for SQLite3 errors
+
 ## [0.2.0] - 2025-01-20
 
 ### Added

data/Gemfile

@@ -10,8 +10,9 @@ gem "rake", "~> 13.0"
 
 gem "rubocop", "~> 1.21"
 
-# Test coverage
+# Test coverage and parallel testing
 group :test do
+  gem "parallel_tests", "~> 4.0"
   gem "simplecov", "~> 0.22", require: false
   gem "simplecov-console", require: false
 end

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    sxn (0.2.0)
+    sxn (0.2.2)
       async (~> 2.0)
       bcrypt (~> 3.1)
       dry-configurable (~> 1.0)
@@ -146,6 +146,8 @@ GEM
     openssl (3.3.0)
     ostruct (0.6.3)
     parallel (1.27.0)
+    parallel_tests (4.10.1)
+      parallel
     parser (3.3.9.0)
       ast (~> 2.4.1)
       racc
@@ -311,6 +313,7 @@ DEPENDENCIES
   faker (~> 3.2)
   irb
   memory_profiler (~> 1.0)
+  parallel_tests (~> 4.0)
   rake (~> 13.0)
   rbs (~> 3.4)
   rbs_rails (~> 0.12)

data/README.md

@@ -223,3 +223,7 @@ Bug reports and pull requests are welcome on GitHub at https://github.com/yourus
 ## License
 
 The gem is available as open source under the terms of the [MIT License](LICENSE.txt).
+
+## Author's Note
+
+This is a personal project that leverages Claude Code as the primary and active developer. As we continue to refine the development process and iron out any kinks, you can expect builds to gradually become more stable. Your patience and feedback are greatly appreciated as we evolve this tool together.

data/Rakefile

@@ -13,6 +13,47 @@ rescue LoadError
   # RSpec not available
 end
 
+# Parallel testing tasks
+begin
+  require "parallel_tests/tasks"
+rescue LoadError
+  # parallel_tests not available
+end
+
+namespace :parallel do
+  desc "Run specs in parallel"
+  task :spec do
+    sh "bundle exec parallel_rspec spec/unit spec/integration spec/performance --runtime-log tmp/parallel_runtime_rspec.log"
+  end
+
+  desc "Run specs in parallel with coverage"
+  task :spec_with_coverage do
+    ENV["ENABLE_SIMPLECOV"] = "true"
+    sh "bundle exec parallel_rspec spec/unit spec/integration spec/performance --runtime-log tmp/parallel_runtime_rspec.log"
+  end
+
+  desc "Setup parallel test databases (if needed)"
+  task :setup do
+    puts "No database setup needed for this project"
+  end
+
+  desc "Run specs in parallel with custom processor count"
+  task :spec_custom, [:processors] do |_t, args|
+    processors = args[:processors] || 4
+    sh "bundle exec parallel_rspec -n #{processors} spec/unit spec/integration spec/performance --runtime-log tmp/parallel_runtime_rspec.log"
+  end
+
+  desc "Generate parallel test runtime log"
+  task :generate_runtime do
+    puts "Generating runtime log for balanced test distribution..."
+    ENV["TEST_ENV_NUMBER"] = "1"
+    specs = "spec/unit spec/integration spec/performance"
+    sh "bundle exec rspec --format progress --format ParallelTests::RSpec::RuntimeLogger " \
+       "--out tmp/parallel_runtime_rspec.log #{specs}"
+    puts "Runtime log generated at tmp/parallel_runtime_rspec.log"
+  end
+end
+
 # Type checking tasks
 namespace :rbs do
   desc "Validate RBS files syntax"

data/docs/parallel_testing.md

@@ -0,0 +1,124 @@
+# Parallel Testing Guide
+
+## Overview
+
+The sxn project uses `parallel_tests` gem to run RSpec tests in parallel, significantly reducing test execution time from ~35 seconds to ~11 seconds.
+
+## Configuration
+
+### `.parallel_rspec` File
+
+This file configures RSpec formatters for parallel execution:
+
+```
+--format progress
+--format ParallelTests::RSpec::RuntimeLogger --out tmp/parallel_runtime_rspec.log
+```
+
+The RuntimeLogger records execution times for each test file, enabling balanced test distribution across parallel processes.
+
+### Runtime-Based Test Distribution
+
+Tests are distributed across parallel processes based on their historical execution times, ensuring all processes finish at roughly the same time.
+
+## Local Usage
+
+### Run Tests in Parallel
+
+```bash
+# Default parallel execution (uses all available CPUs)
+bundle exec rake parallel:spec
+
+# With custom processor count
+bundle exec rake parallel:spec_custom[2]  # Use 2 processes
+
+# With coverage
+bundle exec rake parallel:spec_with_coverage
+```
+
+### Generate/Update Runtime Log
+
+```bash
+# Generate initial runtime log or update existing one
+bundle exec rake parallel:generate_runtime
+```
+
+The runtime log is stored at `tmp/parallel_runtime_rspec.log` and contains execution times for each spec file.
+
+### Direct parallel_rspec Command
+
+```bash
+# Run with runtime balancing
+bundle exec parallel_rspec spec/unit spec/integration spec/performance --runtime-log tmp/parallel_runtime_rspec.log
+
+# Verbose output to see how tests are distributed
+bundle exec parallel_rspec spec --verbose-command --runtime-log tmp/parallel_runtime_rspec.log
+```
+
+## CI/CD Integration
+
+GitHub Actions is configured to:
+
+1. **Cache Runtime Log**: The runtime log is cached between runs using the spec files' hash as the cache key
+2. **Parallel Matrix Jobs**: Tests run across 4 parallel jobs
+3. **Automatic Generation**: If no cached runtime log exists, CI generates one automatically
+4. **Result Aggregation**: Test results from all parallel jobs are collected and summarized
+
+### CI Configuration
+
+The workflow uses a matrix strategy to run tests in parallel:
+
+```yaml
+matrix:
+  ruby: ['3.4.5']
+  ci_node_total: [4]
+  ci_node_index: [0, 1, 2, 3]
+```
+
+Each job runs a subset of tests based on the runtime log distribution.
+
+## Benefits
+
+- **Speed**: ~3x faster test execution (35s → 11s)
+- **Balanced Distribution**: Tests are distributed based on execution time, not file count
+- **CI Optimization**: Parallel jobs in CI reduce overall build time
+- **Coverage Support**: SimpleCov properly merges results from parallel processes
+
+## Troubleshooting
+
+### Unbalanced Test Distribution
+
+If tests aren't evenly distributed:
+
+1. Regenerate the runtime log: `bundle exec rake parallel:generate_runtime`
+2. Check the log file exists: `ls -la tmp/parallel_runtime_rspec.log`
+3. Verify the format (should be `path/to/spec.rb:execution_time`)
+
+### Missing Runtime Log
+
+The system will fall back to file-size-based distribution if no runtime log is found. To ensure runtime-based distribution:
+
+```bash
+# Generate if missing
+[ -f tmp/parallel_runtime_rspec.log ] || bundle exec rake parallel:generate_runtime
+```
+
+### Coverage Issues
+
+Ensure SimpleCov is configured for parallel tests in `spec/spec_helper.rb`:
+
+```ruby
+if ENV["TEST_ENV_NUMBER"]
+  SimpleCov.command_name "RSpec_#{ENV['TEST_ENV_NUMBER']}"
+  SimpleCov.merge_timeout 3600
+end
+```
+
+## Performance Metrics
+
+Current performance with 4 parallel processes:
+- Sequential execution: ~35 seconds
+- Parallel execution: ~11 seconds
+- Speedup: ~3.2x
+
+The runtime-based distribution ensures all processes finish within 1-2 seconds of each other, maximizing efficiency.

data/lib/sxn/CLI.rb

@@ -32,16 +32,8 @@ module Sxn
       # steep:ignore:start - Thor dynamic argument validation handled at runtime
       # Thor framework uses metaprogramming for argument parsing that can't be statically typed.
       # Runtime validation ensures type safety through Thor's built-in validation.
-      # Validate arguments, filtering out nil values for optional arguments
-      args_for_validation = [folder].compact
-      expected_arg_count = folder.nil? ? 0 : 1
-
-      RuntimeValidations.validate_thor_arguments("init", args_for_validation, options, {
-                                                   args: { count: [expected_arg_count], types: [String] },
-                                                   options: { force: :boolean, auto_detect: :boolean, quiet: :boolean }
-                                                 })
-
       Commands::Init.new.init(folder)
+      # steep:ignore:end
     rescue Sxn::Error => e
       handle_error(e)
     end
@@ -78,6 +70,30 @@ module Sxn
       handle_error(e)
     end
 
+    desc "remove [SESSION_NAME]", "Remove a session (shortcut for 'sxn sessions remove')"
+    option :force, type: :boolean, aliases: "-f", desc: "Force removal even with uncommitted changes"
+    def remove(session_name = nil)
+      cmd = Commands::Sessions.new
+      cmd.options = options
+      cmd.remove(session_name)
+    rescue Sxn::Error => e
+      handle_error(e)
+    end
+
+    desc "archive [SESSION_NAME]", "Archive a session (shortcut for 'sxn sessions archive')"
+    def archive(session_name = nil)
+      Commands::Sessions.new.archive(session_name)
+    rescue Sxn::Error => e
+      handle_error(e)
+    end
+
+    desc "activate [SESSION_NAME]", "Activate an archived session (shortcut for 'sxn sessions activate')"
+    def activate(session_name = nil)
+      Commands::Sessions.new.activate(session_name)
+    rescue Sxn::Error => e
+      handle_error(e)
+    end
+
     desc "projects SUBCOMMAND", "Manage project configurations"
     def projects(subcommand = nil, *args)
       Commands::Projects.start([subcommand, *args].compact)
@@ -200,18 +216,14 @@ module Sxn
 
       # steep:ignore:start - Safe integer to string coercion for UI display
       # These integer values are safely converted to strings for display purposes.
-      # Runtime validation ensures proper type handling.
-      @ui.key_value("Total Sessions",
-                    RuntimeValidations.validate_and_coerce_type(sessions.size, String, "session count display"))
-      @ui.key_value("Total Projects",
-                    RuntimeValidations.validate_and_coerce_type(projects.size, String, "project count display"))
+      @ui.key_value("Total Sessions", sessions.size.to_s)
+      @ui.key_value("Total Projects", projects.size.to_s)
 
       # Active worktrees
       if current_session
         worktree_manager = Sxn::Core::WorktreeManager.new(config_manager, session_manager)
         worktrees = worktree_manager.list_worktrees(session_name: current_session)
-        @ui.key_value("Active Worktrees",
-                      RuntimeValidations.validate_and_coerce_type(worktrees.size, String, "worktree count display"))
+        @ui.key_value("Active Worktrees", worktrees.size.to_s)
       end
 
       @ui.newline

data/lib/sxn/commands/sessions.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require "thor"
+require "time"
 
 module Sxn
   module Commands
@@ -79,7 +80,8 @@ module Sxn
           name = @prompt.select("Select session to remove:", choices)
         end
 
-        unless @prompt.confirm_deletion(name, "session")
+        # Skip confirmation if force flag is used
+        if !options[:force] && !@prompt.confirm_deletion(name, "session")
           @ui.info("Cancelled")
           return
         end
@@ -111,7 +113,7 @@ module Sxn
         begin
           sessions = @session_manager.list_sessions(
             status: options[:status],
-            limit: options[:limit]
+            limit: options[:limit]&.to_i || 50
           )
 
           @ui.section("Sessions")
@@ -259,8 +261,8 @@ module Sxn
 
         @ui.key_value("Linear Task", session[:linear_task]) if session[:linear_task]
 
-        @ui.key_value("Created", session[:created_at] || "Unknown")
-        @ui.key_value("Updated", session[:updated_at] || "Unknown")
+        @ui.key_value("Created", format_timestamp(session[:created_at]))
+        @ui.key_value("Updated", format_timestamp(session[:updated_at]))
 
         if verbose && session[:projects]&.any?
           @ui.newline
@@ -295,6 +297,19 @@ module Sxn
         @ui.newline
         @ui.recovery_suggestion("Create your first session with 'sxn add <session-name>'")
       end
+
+      def format_timestamp(timestamp)
+        return "Unknown" if timestamp.nil? || timestamp.empty?
+
+        # Parse the ISO8601 timestamp and convert to local time
+        time = Time.parse(timestamp)
+        local_time = time.localtime
+
+        # Format as "YYYY-MM-DD HH:MM:SS AM/PM Timezone"
+        local_time.strftime("%Y-%m-%d %I:%M:%S %p %Z")
+      rescue ArgumentError
+        timestamp # Return original if parsing fails
+      end
     end
   end
 end

data/lib/sxn/database/session_database.rb

@@ -25,7 +25,7 @@ module Sxn
     # - Bulk operations: < 100ms for 100 sessions
     class SessionDatabase
       # Current database schema version for migrations
-      SCHEMA_VERSION = 1
+      SCHEMA_VERSION = 2
 
       # Default database path relative to sxn config directory
       DEFAULT_DB_PATH = ".sxn/sessions.db"
@@ -118,8 +118,11 @@ module Sxn
       # @param offset [Integer] Results offset for pagination (default: 0)
       # @return [Array<Hash>] Array of session hashes
       def list_sessions(filters: {}, sort: {}, limit: 100, offset: 0)
-        # Ensure filters is a Hash
+        # Ensure filters is a Hash and parameters are correct types
         filters ||= {}
+        limit = limit.to_i if limit
+        offset = offset.to_i if offset
+
         query_parts = ["SELECT * FROM sessions"]
         params = []
 
@@ -132,9 +135,9 @@ module Sxn
         sort_order = sort[:order] || :desc
         query_parts << "ORDER BY #{sort_field} #{sort_order.to_s.upcase}"
 
-        # Add pagination
+        # Add pagination - ensure these are integers
         query_parts << "LIMIT ? OFFSET ?"
-        params.push(limit, offset)
+        params.push(limit || 100, offset || 0)
 
         sql = query_parts.join(" ")
 
@@ -417,6 +420,19 @@ module Sxn
       def setup_database
         current_version = get_schema_version
 
+        # Check if this is an old database without version info
+        # If sessions table exists but version is 0, it's likely an old database
+        if current_version.zero? && table_exists?("sessions")
+          # Check if it's the old schema (without worktrees/projects columns)
+          columns = connection.execute("PRAGMA table_info(sessions)").map { |col| col["name"] }
+          if !columns.include?("worktrees") || !columns.include?("projects")
+            # This is an old database, set it to version 1 so we can migrate it
+            Sxn.logger.info "Detected old database schema without version info, treating as version 1"
+            set_schema_version(1)
+            current_version = 1
+          end
+        end
+
         if current_version.zero?
           create_initial_schema
           set_schema_version(SCHEMA_VERSION)
@@ -425,6 +441,14 @@ module Sxn
         end
       end
 
+      # Check if a table exists
+      def table_exists?(table_name)
+        result = connection.execute("SELECT name FROM sqlite_master WHERE type='table' AND name=?", table_name)
+        !result.empty?
+      rescue SQLite3::Exception
+        false
+      end
+
       # Get current schema version from database
       def get_schema_version
         result = connection.execute("PRAGMA user_version").first
@@ -498,12 +522,41 @@ module Sxn
       end
 
       # Run database migrations from old version to current
-      def run_migrations(_from_version)
-        # Future migrations will be implemented here
-        # For now, we only have version 1
+      def run_migrations(from_version)
+        Sxn.logger.info "Running database migrations from version #{from_version} to #{SCHEMA_VERSION}"
+
+        # Run each migration in sequence
+        migrate_to_v1 if from_version < 1
+
+        migrate_to_v2 if from_version < 2
+
         set_schema_version(SCHEMA_VERSION)
       end
 
+      # Migration: Initial schema (v1)
+      def migrate_to_v1
+        # This is handled by create_initial_schema
+        create_initial_schema
+      end
+
+      # Migration: Add worktrees and projects columns (v2)
+      def migrate_to_v2
+        Sxn.logger.info "Migrating database to version 2: Adding worktrees and projects columns"
+
+        # Check if columns already exist
+        columns = connection.execute("PRAGMA table_info(sessions)").map { |col| col["name"] }
+
+        unless columns.include?("worktrees")
+          connection.execute("ALTER TABLE sessions ADD COLUMN worktrees TEXT")
+          Sxn.logger.info "Added worktrees column to sessions table"
+        end
+
+        return if columns.include?("projects")
+
+        connection.execute("ALTER TABLE sessions ADD COLUMN projects TEXT")
+        Sxn.logger.info "Added projects column to sessions table"
+      end
+
       # Generate secure, unique session ID
       def generate_session_id
         SecureRandom.hex(16) # 32 character hex string
@@ -620,7 +673,27 @@ module Sxn
 
       # Execute query with parameters and return results
       def execute_query(sql, params = [])
-        connection.execute(sql, params)
+        # Ensure params are properly typed for SQLite3
+        sanitized_params = params.map do |param|
+          case param
+          when Integer, Float, String, NilClass
+            param
+          when TrueClass
+            1
+          when FalseClass
+            0
+          else
+            param.to_s
+          end
+        end
+
+        connection.execute(sql, sanitized_params)
+      rescue SQLite3::MismatchException => e
+        # Log the error with details for debugging
+        warn "SQLite3 datatype mismatch: #{e.message}"
+        warn "SQL: #{sql}"
+        warn "Params: #{sanitized_params.inspect}"
+        raise e
       end
 
       # Transaction wrapper with rollback support

data/lib/sxn/runtime_validations.rb

@@ -6,6 +6,10 @@ module Sxn
     class << self
       # Validate Thor command arguments at runtime
       def validate_thor_arguments(command_name, args, options, validations)
+        # Ensure args and options are not nil
+        args ||= []
+        options ||= {}
+
         # Validate argument count
         if validations[:args]
           count_range = validations[:args][:count]
@@ -23,7 +27,7 @@ module Sxn
         end
 
         # Validate options
-        if validations[:options]
+        if validations[:options] && options.respond_to?(:each)
           options.each do |key, value|
             validate_option_type(command_name, key, value, validations[:options][key.to_sym]) if validations[:options][key.to_sym]
           end

data/lib/sxn/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Sxn
-  VERSION = "0.2.0"
+  VERSION = "0.2.2"
 end

data/sig/sxn/cli.rbs

@@ -21,6 +21,12 @@ module Sxn
 
     def current: () -> void
 
+    def remove: (?String? session_name) -> void
+
+    def archive: (?String? session_name) -> void
+
+    def activate: (?String? session_name) -> void
+
     def projects: (?String? subcommand, *String args) -> void
 
     def sessions: (?String? subcommand, *String args) -> void

data/sig/sxn/commands/sessions.rbs

@@ -57,6 +57,10 @@ module Sxn
 
       # Suggest creating a session
       def suggest_create_session: () -> void
+
+      # Format a timestamp string to local timezone
+      # @param timestamp ISO8601 timestamp string
+      def format_timestamp: (String? timestamp) -> String
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sxn
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.2
 platform: ruby
 authors:
 - Ernest Sim
@@ -488,6 +488,8 @@ files:
 - ".gem_rbs_collection/sqlite3/2.0/.rbs_meta.yaml"
 - ".gem_rbs_collection/sqlite3/2.0/database.rbs"
 - ".gem_rbs_collection/sqlite3/2.0/pragmas.rbs"
+- ".parallel_rspec"
+- ".parallel_rspec_options"
 - ".rspec"
 - ".rubocop.yml"
 - ".simplecov"
@@ -499,6 +501,7 @@ files:
 - Rakefile
 - Steepfile
 - bin/sxn
+- docs/parallel_testing.md
 - lib/sxn.rb
 - lib/sxn/CLI.rb
 - lib/sxn/commands.rb