howzit 2.1.28 → 2.1.30

data/lib/howzit/util.rb

@@ -118,7 +118,11 @@ module Howzit
       # Paginate the output
       def page(text)
         unless $stdout.isatty
-          puts text
+          begin
+            puts text
+          rescue Errno::EPIPE
+            # Pipe closed, ignore
+          end
           return
         end
 
@@ -181,7 +185,11 @@ module Howzit
         if options[:paginate] && Howzit.options[:paginate]
           page(output)
         else
-          puts output
+          begin
+            puts output
+          rescue Errno::EPIPE
+            # Pipe closed, ignore
+          end
         end
       end
 
@@ -219,7 +227,7 @@ module Howzit
       ##
       def os_paste
         os = RbConfig::CONFIG['target_os']
-        out = "{bg}Pasting from clipboard".c
+        out = '{bg}Pasting from clipboard'.c
         case os
         when /darwin.*/i
           Howzit.console.debug("#{out} (macOS){x}".c)

data/lib/howzit/version.rb

@@ -3,5 +3,5 @@
 # Primary module for this gem.
 module Howzit
   # Current Howzit version.
-  VERSION = '2.1.28'
+  VERSION = '2.1.30'
 end

data/lib/howzit.rb

@@ -38,6 +38,11 @@ require_relative 'howzit/stringutils'
 
 require_relative 'howzit/console_logger'
 require_relative 'howzit/config'
+require_relative 'howzit/script_comm'
+require_relative 'howzit/script_support'
+require_relative 'howzit/condition_evaluator'
+require_relative 'howzit/conditional_content'
+require_relative 'howzit/directive'
 require_relative 'howzit/task'
 require_relative 'howzit/topic'
 require_relative 'howzit/buildnote'

data/spec/condition_evaluator_spec.rb

@@ -0,0 +1,261 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Howzit::ConditionEvaluator do
+  describe '.evaluate' do
+    context 'with string comparisons' do
+      it 'evaluates == correctly' do
+        expect(described_class.evaluate('"test" == "test"', {})).to be true
+        expect(described_class.evaluate('"test" == "other"', {})).to be false
+      end
+
+      it 'evaluates =~ regex correctly' do
+        expect(described_class.evaluate('"test" =~ /es/', {})).to be true
+        expect(described_class.evaluate('"test" =~ /xyz/', {})).to be false
+      end
+
+      it 'evaluates *= (contains) correctly' do
+        expect(described_class.evaluate('"testing" *= "est"', {})).to be true
+        expect(described_class.evaluate('"testing" *= "xyz"', {})).to be false
+      end
+
+      it 'evaluates ^= (starts with) correctly' do
+        expect(described_class.evaluate('"testing" ^= "test"', {})).to be true
+        expect(described_class.evaluate('"testing" ^= "ing"', {})).to be false
+      end
+
+      it 'evaluates $= (ends with) correctly' do
+        expect(described_class.evaluate('"testing" $= "ing"', {})).to be true
+        expect(described_class.evaluate('"testing" $= "test"', {})).to be false
+      end
+
+      it 'evaluates **= (fuzzy match) correctly' do
+        expect(described_class.evaluate('"fluffy" **= "ffy"', {})).to be true
+        expect(described_class.evaluate('"fluffy" **= "fly"', {})).to be true
+        expect(described_class.evaluate('"fluffy" **= "fuf"', {})).to be true
+        expect(described_class.evaluate('"fluffy" **= "xyz"', {})).to be false
+        expect(described_class.evaluate('"fluffy" **= "fxy"', {})).to be false
+      end
+    end
+
+    context 'with numeric comparisons' do
+      it 'evaluates == correctly' do
+        expect(described_class.evaluate('5 == 5', {})).to be true
+        expect(described_class.evaluate('5 == 3', {})).to be false
+      end
+
+      it 'evaluates != correctly' do
+        expect(described_class.evaluate('5 != 3', {})).to be true
+        expect(described_class.evaluate('5 != 5', {})).to be false
+      end
+
+      it 'evaluates > correctly' do
+        expect(described_class.evaluate('5 > 3', {})).to be true
+        expect(described_class.evaluate('3 > 5', {})).to be false
+      end
+
+      it 'evaluates >= correctly' do
+        expect(described_class.evaluate('5 >= 5', {})).to be true
+        expect(described_class.evaluate('5 >= 3', {})).to be true
+        expect(described_class.evaluate('3 >= 5', {})).to be false
+      end
+
+      it 'evaluates < correctly' do
+        expect(described_class.evaluate('3 < 5', {})).to be true
+        expect(described_class.evaluate('5 < 3', {})).to be false
+      end
+
+      it 'evaluates <= correctly' do
+        expect(described_class.evaluate('3 <= 3', {})).to be true
+        expect(described_class.evaluate('3 <= 5', {})).to be true
+        expect(described_class.evaluate('5 <= 3', {})).to be false
+      end
+    end
+
+    context 'with negation' do
+      it 'handles not prefix' do
+        expect(described_class.evaluate('not "test" == "other"', {})).to be true
+        expect(described_class.evaluate('not "test" == "test"', {})).to be false
+      end
+
+      it 'handles ! prefix' do
+        expect(described_class.evaluate('! "test" == "other"', {})).to be true
+        expect(described_class.evaluate('! "test" == "test"', {})).to be false
+      end
+    end
+
+    context 'with positional arguments' do
+      before do
+        Howzit.arguments = %w[arg1 arg2 arg3]
+      end
+
+      after do
+        Howzit.arguments = nil
+      end
+
+      it 'evaluates $1, $2, etc.' do
+        expect(described_class.evaluate('$1 == "arg1"', {})).to be true
+        expect(described_class.evaluate('$2 == "arg2"', {})).to be true
+        expect(described_class.evaluate('$1 == "other"', {})).to be false
+      end
+    end
+
+    context 'with named arguments' do
+      before do
+        Howzit.named_arguments = { test: 'value', other: 'thing' }
+      end
+
+      after do
+        Howzit.named_arguments = nil
+      end
+
+      it 'evaluates named arguments' do
+        expect(described_class.evaluate('test == "value"', {})).to be true
+        expect(described_class.evaluate('other == "thing"', {})).to be true
+        expect(described_class.evaluate('test == "other"', {})).to be false
+      end
+
+      it 'evaluates named arguments with ${} syntax' do
+        Howzit.named_arguments = { var: 'val', env: 'production' }
+        expect(described_class.evaluate('${var} == "val"', {})).to be true
+        expect(described_class.evaluate('${env} == "production"', {})).to be true
+        expect(described_class.evaluate('${var} == "other"', {})).to be false
+      end
+    end
+
+    context 'with metadata' do
+      it 'evaluates metadata keys' do
+        context = { metadata: { 'key1' => 'value1', 'key2' => 'value2' } }
+        expect(described_class.evaluate('key1 == "value1"', context)).to be true
+        expect(described_class.evaluate('key2 == "value2"', context)).to be true
+        expect(described_class.evaluate('key1 == "other"', context)).to be false
+      end
+    end
+
+    context 'with environment variables' do
+      it 'evaluates environment variables' do
+        ENV['TEST_VAR'] = 'test_value'
+        expect(described_class.evaluate('TEST_VAR == "test_value"', {})).to be true
+        ENV.delete('TEST_VAR')
+      end
+    end
+
+    context 'with special conditions' do
+      it 'evaluates git dirty' do
+        # Just test that it doesn't crash
+        result = described_class.evaluate('git dirty', {})
+        expect([true, false]).to include(result)
+      end
+
+      it 'evaluates git clean' do
+        result = described_class.evaluate('git clean', {})
+        expect([true, false]).to include(result)
+      end
+
+      it 'evaluates file exists' do
+        expect(described_class.evaluate('file exists /dev/null', {})).to be true
+        expect(described_class.evaluate('file exists /nonexistent/file', {})).to be false
+      end
+
+      it 'evaluates dir exists' do
+        expect(described_class.evaluate('dir exists /tmp', {})).to be true
+        expect(described_class.evaluate('dir exists /nonexistent/dir', {})).to be false
+      end
+
+      it 'evaluates cwd' do
+        result = described_class.evaluate('cwd', {})
+        expect(result).to be true
+      end
+
+      it 'evaluates working directory' do
+        result = described_class.evaluate('working directory', {})
+        expect(result).to be true
+      end
+
+      it 'evaluates cwd with string comparisons' do
+        cwd = Dir.pwd
+        expect(described_class.evaluate("cwd == \"#{cwd}\"", {})).to be true
+        expect(described_class.evaluate("cwd =~ /#{Regexp.escape(File.basename(cwd))}/", {})).to be true
+        expect(described_class.evaluate("cwd *= \"#{File.basename(cwd)}\"", {})).to be true
+        expect(described_class.evaluate("cwd ^= \"#{cwd[0..10]}\"", {})).to be true
+        expect(described_class.evaluate("cwd $= \"#{cwd[-10..-1]}\"", {})).to be true
+      end
+    end
+
+    context 'with file contents conditions' do
+      let(:test_file) { 'test_version.txt' }
+
+      before do
+        File.write(test_file, '0.1.2')
+      end
+
+      after do
+        File.delete(test_file) if File.exist?(test_file)
+      end
+
+      it 'evaluates file contents ^= (starts with) correctly' do
+        expect(described_class.evaluate("file contents #{test_file} ^= 0.", {})).to be true
+        expect(described_class.evaluate("file contents #{test_file} ^= 1.", {})).to be false
+      end
+
+      it 'evaluates file contents *= (contains) correctly' do
+        expect(described_class.evaluate("file contents #{test_file} *= 1.2", {})).to be true
+        expect(described_class.evaluate("file contents #{test_file} *= 2.3", {})).to be false
+      end
+
+      it 'evaluates file contents $= (ends with) correctly' do
+        expect(described_class.evaluate("file contents #{test_file} $= .2", {})).to be true
+        expect(described_class.evaluate("file contents #{test_file} $= .3", {})).to be false
+      end
+
+      it 'evaluates file contents == correctly' do
+        expect(described_class.evaluate("file contents #{test_file} == 0.1.2", {})).to be true
+        expect(described_class.evaluate("file contents #{test_file} == 1.0.0", {})).to be false
+      end
+
+      it 'evaluates file contents != correctly' do
+        expect(described_class.evaluate("file contents #{test_file} != 1.0.0", {})).to be true
+        expect(described_class.evaluate("file contents #{test_file} != 0.1.2", {})).to be false
+      end
+
+      it 'evaluates file contents **= (fuzzy match) correctly' do
+        expect(described_class.evaluate("file contents #{test_file} **= 012", {})).to be true
+        expect(described_class.evaluate("file contents #{test_file} **= 021", {})).to be false
+      end
+
+      it 'evaluates file contents =~ (regex) correctly' do
+        expect(described_class.evaluate("file contents #{test_file} =~ /0\\.\\d+\\.\\d+/", {})).to be true
+        expect(described_class.evaluate("file contents #{test_file} =~ /1\\.\\d+\\.\\d+/", {})).to be false
+      end
+
+      it 'returns false for non-existent file' do
+        expect(described_class.evaluate('file contents nonexistent.txt ^= 0.', {})).to be false
+      end
+
+      it 'handles file path as variable' do
+        context = { metadata: { 'version_file' => test_file } }
+        expect(described_class.evaluate('file contents version_file ^= 0.', context)).to be true
+      end
+    end
+
+    context 'with simple existence checks' do
+      before do
+        Howzit.named_arguments = { defined_var: 'value' }
+      end
+
+      after do
+        Howzit.named_arguments = nil
+      end
+
+      it 'returns true for defined variables' do
+        expect(described_class.evaluate('defined_var', {})).to be true
+      end
+
+      it 'returns false for undefined variables' do
+        expect(described_class.evaluate('undefined_var', {})).to be false
+      end
+    end
+  end
+end
+

data/spec/conditional_blocks_integration_spec.rb

@@ -0,0 +1,159 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe 'Conditional Blocks Integration' do
+  before do
+    Howzit.options[:include_upstream] = false
+    Howzit.options[:default] = true
+    Howzit.options[:matching] = 'partial'
+    Howzit.options[:multiple_matches] = 'choose'
+  end
+
+  after do
+    FileUtils.rm_f('builda.md')
+  end
+
+  describe 'with conditional blocks in topics' do
+    it 'includes content when @if condition is true' do
+      note = <<~EONOTE
+        # Test
+
+        ## Test Topic
+
+        @if "test" == "test"
+        This should be included
+        @end
+      EONOTE
+      File.open('builda.md', 'w') { |f| f.puts note }
+      Howzit.instance_variable_set(:@buildnote, nil) # Force reload
+      topic = Howzit.buildnote.find_topic('Test Topic')[0]
+      expect(topic).not_to be_nil
+      output = topic.print_out
+      expect(output.join("\n")).to include('This should be included')
+      expect(output.join("\n")).not_to include('@if')
+      expect(output.join("\n")).not_to include('@end')
+    end
+
+    it 'excludes content when @if condition is false' do
+      note = <<~EONOTE
+        # Test
+
+        ## Test Topic
+
+        @if "test" == "other"
+        This should NOT be included
+        @end
+      EONOTE
+      File.open('builda.md', 'w') { |f| f.puts note }
+      Howzit.instance_variable_set(:@buildnote, nil) # Force reload
+      topic = Howzit.buildnote.find_topic('Test Topic')[0]
+      expect(topic).not_to be_nil
+      output = topic.print_out
+      expect(output.join("\n")).not_to include('This should NOT be included')
+    end
+
+    it 'includes tasks when condition is true' do
+      note = <<~EONOTE
+        # Test
+
+        ## Test Topic
+
+        @if 1 == 1
+        @run(echo "test") Test Command
+        @end
+      EONOTE
+      File.open('builda.md', 'w') { |f| f.puts note }
+      Howzit.instance_variable_set(:@buildnote, nil) # Force reload
+      topic = Howzit.buildnote.find_topic('Test Topic')[0]
+      expect(topic).not_to be_nil
+      expect(topic.tasks.count).to eq(1)
+      expect(topic.tasks[0].action).to include('echo "test"')
+    end
+
+    it 'excludes tasks when condition is false' do
+      note = <<~EONOTE
+        # Test
+
+        ## Test Topic
+
+        @if 1 == 2
+        @run(echo "hidden") Hidden Command
+        @end
+      EONOTE
+      File.open('builda.md', 'w') { |f| f.puts note }
+      Howzit.instance_variable_set(:@buildnote, nil) # Force reload
+      topic = Howzit.buildnote.find_topic('Test Topic')[0]
+      expect(topic).not_to be_nil
+      expect(topic.tasks.count).to eq(0)
+    end
+
+    it 'handles nested conditional blocks' do
+      note = <<~EONOTE
+        # Test
+
+        ## Test Topic
+
+        @if "outer" == "outer"
+        Outer content
+        @if "inner" == "inner"
+        Inner content
+        @end
+        More outer content
+        @end
+      EONOTE
+      File.open('builda.md', 'w') { |f| f.puts note }
+      Howzit.instance_variable_set(:@buildnote, nil) # Force reload
+      topic = Howzit.buildnote.find_topic('Test Topic')[0]
+      expect(topic).not_to be_nil
+      output = topic.print_out
+      expect(output.join("\n")).to include('Outer content')
+      expect(output.join("\n")).to include('Inner content')
+      expect(output.join("\n")).to include('More outer content')
+    end
+
+    it 'handles metadata in conditions' do
+      note = <<~EONOTE
+        env: production
+
+        # Test
+
+        ## Test Topic
+
+        @if env == "production"
+        Production content
+        @end
+      EONOTE
+      File.open('builda.md', 'w') { |f| f.puts note }
+      Howzit.instance_variable_set(:@buildnote, nil) # Force reload
+      topic = Howzit.buildnote.find_topic('Test Topic')[0]
+      expect(topic).not_to be_nil
+      output = topic.print_out
+      expect(output.join("\n")).to include('Production content')
+    end
+
+    it 'handles @unless blocks correctly' do
+      note = <<~EONOTE
+        # Test
+
+        ## Test Topic
+
+        @unless "test" == "other"
+        This should be included
+        @end
+
+        @unless "test" == "test"
+        This should NOT be included
+        @end
+      EONOTE
+      File.open('builda.md', 'w') { |f| f.puts note }
+      Howzit.instance_variable_set(:@buildnote, nil) # Force reload
+      topic = Howzit.buildnote.find_topic('Test Topic')[0]
+      expect(topic).not_to be_nil
+      output = topic.print_out
+      expect(output.join("\n")).to include('This should be included')
+      expect(output.join("\n")).not_to include('This should NOT be included')
+    end
+  end
+end
+