jekyll-chatgpt-translate 0.0.31 → 0.0.33

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8c50e5ce734d7a12ddcedbbcc4602453d5c0a7e78352a6e12bfbe04abfcf149e
-  data.tar.gz: 66f6a5668e0d37fd32a1f9573e34036b3155cf864238afb48116c8be4cd19e64
+  metadata.gz: bcbe51d1f51e425e01eeb95267bfe08ebb5e57eba3add1b071fd5ba96b43c0de
+  data.tar.gz: 00d09ef020ba4fea3f6029a3b78d89ce9595428bf4621fa6f20e66f87e6c6266
 SHA512:
-  metadata.gz: b5adeb2316daff862f639e9318f5636d3651c69ea5fd50c42c3ddc86e13a7ac97406491fe7f0acf5bfe5d0550135a0568abeed790d1b70ccb3a593a89fc279b6
-  data.tar.gz: cf233bb2b67710883367515aaffffe4304d4c7f2b4c9571e276ed1323c90793bbddef1045067d5e84ca04dbb93fb946614bb5d05bbe71571bbfc589011af3c66
+  metadata.gz: 8c685004776a719fca54ecfcf4e8a66e27b728ad1af9d2e40efdc9cfd8cf531cbc98d89103089636da67285c2596537b992159de78838d8045016dd622faaf50
+  data.tar.gz: 3430c908c2273a2b8c28d628334b4333ed80d61b52498b5dd365e27fec1631a0260fe54bff159e9e1ecc8b669bfb20f464df63d02abcce1f734f31a57d4631e7

data/.rubocop.yml

@@ -22,7 +22,7 @@ Metrics/BlockLength:
 Metrics/CyclomaticComplexity:
   Max: 30
 Metrics/PerceivedComplexity:
-  Max: 30
+  Max: 40
 Layout/EmptyLineAfterGuardClause:
   Enabled: false
 Naming/FileName:

data/Gemfile

@@ -32,4 +32,4 @@ gem 'rake', '13.0.6', require: false
 gem 'rubocop', '1.56.2', require: false
 gem 'rubocop-rspec', '2.23.2', require: false
 gem 'simplecov', '0.22.0', require: false
-gem 'webmock', '3.18.1', require: false
+gem 'webmock', '3.19.1', require: false

data/features/cli.feature

@@ -168,3 +168,41 @@ Feature: Simple site building
     And File "_site/2023/01/01/hello.html" contains "/bye.html"
     And File "_site/2023/02/02/bye.html" exists
     And File "_site/2023/02/02/bye.html" contains "/hello.html"
+
+  Scenario: No translation at all
+    Given I have a "_config.yml" file with content:
+    """
+    url: https://www.yegor256.com
+    markdown: kramdown
+    plugins:
+      - jekyll-chatgpt-translate
+    chatgpt-translate:
+      source: en
+      threshold: 0
+      api_key: "it-is-not-used, because EN to EN translation"
+      layout: default
+      targets:
+        -
+          language: en
+          permalink: :slug.html
+    """
+    And I have a "_layouts/default.html" file with content:
+    """
+    {{ content }}
+    """
+    And I have a "_posts/2023-01-01-hello.md" file with content:
+    """
+    ---
+    title: foo
+    ---
+    {% if page.chatgpt-translate.model %}
+    TRANSLATED :(
+    {% else %}
+    NO TRANSLATION! :)
+    {% endif %}
+    """
+    Then I build Jekyll site
+    And Exit code is zero
+    And Stdout contains "The page is absent, need to translate"
+    And File "_site/2023/01/01/hello.html" exists
+    And File "_site/2023/01/01/hello.html" contains "NO TRANSLATION!"

data/jekyll-chatgpt-translate.gemspec

@@ -28,7 +28,7 @@ Gem::Specification.new do |s|
   s.required_rubygems_version = Gem::Requirement.new('>= 0') if s.respond_to? :required_rubygems_version=
   s.required_ruby_version = '>= 2.6'
   s.name = 'jekyll-chatgpt-translate'
-  s.version = '0.0.31'
+  s.version = '0.0.33'
   s.license = 'MIT'
   s.summary = 'Translate Jekyll Pages Through ChatGPT'
   s.description = [

data/lib/jekyll-chatgpt-translate/generator.rb

@@ -60,7 +60,7 @@ class GptTranslate::Generator < Jekyll::Generator
     copied = 0
     model = config['model'] || 'gpt-3.5-turbo'
     marker = "Translated by ChatGPT #{model}#{version.empty? ? '' : "/#{version}"}"
-    site.posts.docs.shuffle.each do |doc|
+    site.posts.docs.shuffle.each_with_index do |doc, pos|
       plain = GptTranslate::Plain.new(doc.content).to_s
       config['targets'].each do |target|
         start = Time.now
@@ -86,12 +86,14 @@ class GptTranslate::Generator < Jekyll::Generator
         )
         html = config['no_download'].nil? ? GptTranslate::Ping.new(site, link).download : nil
         needed = false
+        added = false
         if html.nil?
           Jekyll.logger.info("The page is absent, need to translate #{link.inspect}")
           needed = true
         else
           copied += 1
           site.static_files << DownloadedFile.new(site, link, html)
+          added = true
           if version.empty?
             Jekyll.logger.info("Re-translation not required, since version is empty: #{link.inspect}")
           elsif html.include?(marker)
@@ -103,7 +105,7 @@ class GptTranslate::Generator < Jekyll::Generator
           end
         end
         if translated >= threshold
-          Jekyll.logger.info("We are over the threshold: #{translated} > #{threshold}")
+          Jekyll.logger.info("Page ##{pos} is ignored, we are over the threshold of #{threshold}: #{link}")
         elsif needed
           gpt = GptTranslate::ChatGPT.new(
             key,
@@ -124,10 +126,12 @@ class GptTranslate::Generator < Jekyll::Generator
           )
           site.pages << Jekyll::Page.new(site, site.source, File.dirname(path), File.basename(path))
           site.static_files.delete_if { |f| f.is_a?(DownloadedFile) && f.link == link }
+          added = true
           translated += 1
           Jekyll.logger.info("Translated via ChatGPT \
 in #{(Time.now - start).round(2)}s: #{path} (#{File.size(path)} bytes)")
         end
+        next unless added
         doc.data['chatgpt-translate'] ||= {}
         doc.data['chatgpt-translate']['model'] ||= model
         doc.data['chatgpt-translate']['urls'] ||= {}

data/lib/jekyll-chatgpt-translate/ping.rb

@@ -57,8 +57,7 @@ class GptTranslate::Ping
       html = response.body if response.is_a?(Net::HTTPSuccess)
       Jekyll.logger.debug("GET #{uri.inspect}: #{response.code}")
     rescue Errno::ECONNREFUSED, Errno::EADDRNOTAVAIL => e
-      Jekyll.logger.debug("Failed to ping #{uri.inspect}: #{e.message}")
-      Jekyll.logger.info("The page is absent (#{e.class.name}): #{uri.inspect}")
+      Jekyll.logger.debug("Failed to ping #{uri.inspect} (#{e.class.name}): #{e.message}")
     end
     html
   end

data/lib/jekyll-chatgpt-translate/prompt.rb

@@ -54,6 +54,10 @@ class GptTranslate::Prompt
       to[3],
       ', don\'t translate technical terms and proper nouns'
     ].join
-    "#{head}:\n\n#{@par}"
+    if @par.include?('"') || @par.split.count >= 8
+      "#{head}:\n\n#{@par}"
+    else
+      "#{head}: \"#{@par}\""
+    end
   end
 end

data/lib/jekyll-chatgpt-translate/version.rb

@@ -23,5 +23,5 @@
 # SOFTWARE.
 
 module GptTranslate
-  VERSION = '0.0.31'
+  VERSION = '0.0.33'
 end

data/test/test_prompt.rb

@@ -31,24 +31,24 @@ require_relative '../lib/jekyll-chatgpt-translate/prompt'
 # Copyright:: Copyright (c) 2023 Yegor Bugayenko
 # License:: MIT
 class GptTranslate::PromptTest < Minitest::Test
-  def par(body, source, target)
+  def head(source, target)
     [
       'Please, translate the following Markdown paragraph',
       " from #{source} to #{target},",
-      " don't translate technical terms and proper nouns:\n\n#{body}"
+      " don't translate technical terms and proper nouns"
     ].join
   end
 
   def test_english_to_russian
     assert_equal(
-      par('Hello, dude!', 'English', 'Russian'),
-      GptTranslate::Prompt.new('Hello, dude!', 'en', 'ru').to_s
+      "#{head('English', 'Russian')}:\n\nHello, dude, how are you doing today in this fair city?",
+      GptTranslate::Prompt.new('Hello, dude, how are you doing today in this fair city?', 'en', 'ru').to_s
     )
   end
 
   def test_english_to_chinese
     assert_equal(
-      par('Hello, Jeff!', 'English', 'Chinese'),
+      "#{head('English', 'Chinese')}: \"Hello, Jeff!\"",
       GptTranslate::Prompt.new('Hello, Jeff!', 'en', 'zh').to_s
     )
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-chatgpt-translate
 version: !ruby/object:Gem::Version
-  version: 0.0.31
+  version: 0.0.33
 platform: ruby
 authors:
 - Yegor Bugayenko
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-08-29 00:00:00.000000000 Z
+date: 2023-08-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: iri