newsman 1.2.1 → 1.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 28993b9f11754759d159a42a76b5979bec9c12980098de771abd5442fc124ccd
-  data.tar.gz: 1e98570b8d3b0cb9b0694472e5fefa7fe780ce6434efa12cd231dfa86a31c885
+  metadata.gz: 8bff52319f7db052d667bb38fa31764a4204ddab745eeaaa1daf090d36a73789
+  data.tar.gz: a7ccce39053631d627d210fea0897988b29ac24a62d9a17f98d83055777fbe40
 SHA512:
-  metadata.gz: a807c540f352124e1a89d485cccfb2f0cf6ff36300b42d57ebb591b5e3e4b03d12413325f69a75d7f99e309c0813d336b1c3e32bb71012e22dd1093129cee5a6
-  data.tar.gz: 81a635a5caba68596e9c3acd7a242fbf597da0cb06ce87c2103cdab84c1e2e89ef59ec9812e997ff970a0484a16ace406415a83b8c642f6db8ff0f82e571c111
+  metadata.gz: 638afa442dd92022d7966ea99405fea3838a6a92edd12c71f32d9a7cb8392c59287e647d8376524e9a97f61ea9ce3b74b5160b85b27940627d478528d76d3eef
+  data.tar.gz: c8ee4f4a28435a66132f865046f222a2370361884d81d29536e1748778429cfd1a60581e6027448c0bbf5a491d55fda13e0d3f45af293e25b5fbe9d9fff5d5c2

data/README.md

@@ -98,6 +98,25 @@ newsman --help
 ```
 And you should see a welcome message from newsman.
 
+## Releases
+
+Releases are handled by the [`release.yaml`](.github/workflows/release.yaml) GitHub Actions workflow. It runs whenever a tag matching `v*.*.*` (e.g. `v1.2.1`) is pushed, or manually via `workflow_dispatch`. On trigger it:
+
+1. Runs the test suite (`rake test`) and RuboCop (`rake rubocop`).
+2. Builds and installs the gem (`rake install`).
+3. Publishes the gem to [RubyGems.org](https://rubygems.org/gems/newsman) (`rake publish`), using the `RUBYGEMS_API_KEY` secret.
+4. Creates a GitHub Release for the pushed tag, using the `RELEASE_GITHUB_TOKEN` secret.
+
+### How to cut a release
+
+1. **Bump the version** in `newsman.gemspec` (`spec.version`) and commit it to `main`. RubyGems refuses to publish a version that's already live, so this step is mandatory - the workflow's `gem push` will silently fail otherwise (the publish step is `continue-on-error: true`, so the run can still show green even though nothing was actually published).
+2. **Tag the commit** with the same version, prefixed with `v`, and push the tag:
+   ```shell
+   git tag v1.2.1
+   git push origin v1.2.1
+   ```
+3. Watch the `Release Ruby Gem` workflow run in the Actions tab, and confirm the new version shows up on [RubyGems.org](https://rubygems.org/gems/newsman) and as a [GitHub Release](https://github.com/volodya-lombrozo/newsman/releases).
+
 ## Examples
 
 You can find examples of generated reports [here](https://volodya-lombrozo.github.io/newsman/)

data/lib/newsman/assistant.rb

@@ -45,7 +45,8 @@ class Assistant
     example = "repository-name:\n
       - To publish ABC package draft [#27]\n
       - To review first draft of the report [#56]\n
-      - To implement optimization for the class X [#125]"
+      - To implement optimization for the class X [#125]\n
+      - Updated dependencies: rubocop, ruby [#197, #199, #204]"
     prompt = <<~PROMPT
       Please compile a summary of the plans for the next week using the GitHub Issues.
       Each issue should be summarized in a single sentence.
@@ -53,6 +54,8 @@ class Assistant
       Pay attention, that you didn't loose any issue.
       Ensure that each sentence includes the corresponding issue number as an integer value. If an issue doesn't mention an issue number, just print [#chore].
       If several issues have the same number, combine them into a single sentence.
+      Dependency version-bump issues (e.g. titles like "chore(deps): update dependency X to vY" or similar renovate/dependabot style bumps) must all be grouped into a single combined bullet listing the dependency names and all associated issue numbers, instead of one bullet per issue.
+      Other substantive issues should keep their own individual bullet as usual.
       Please strictly adhere to the example template provided: "#{example}".
       List of GitHub Issues in JSON format: ```json #{issues}```.
     PROMPT
@@ -65,13 +68,16 @@ class Assistant
     example = "repository-name:\n
       - Added 100 new files to the Dataset [#168]\n
       - Fixed the deployment of XYZ [#169]\n
-      - Refined the requirements [#177]\n"
+      - Refined the requirements [#177]\n
+      - Updated dependencies: rubocop, ruby, nokogiri [#197, #199, #200, #202, #203, #204]\n"
     prompt = <<~PROMPT
       Please compile a summary of the work completed in the following Pull Requests (PRs).
       Each PR should be summarized in a single sentence, focusing on the PR title rather than the implementation details.
       Ensure no PR is omitted.
       Each sentence must include the corresponding issue number as an integer value. If a PR does not mention an issue number, use [#chore].
       If several PRs have the same number, combine them into a single sentence.
+      Dependency version-bump PRs (e.g. titles like "chore(deps): update dependency X to vY" or similar renovate/dependabot style bumps) must all be grouped into a single combined bullet listing the dependency names and all associated PR numbers, instead of one bullet per PR.
+      Other substantive PRs should keep their own individual bullet as usual.
       Combine the information from each PR into a concise and fluent sentence.
       Follow the provided example template strictly: "#{example}".
       List of Pull Requests in JSON format: ```json #{prs}```.
@@ -112,32 +118,32 @@ class Assistant
     send(prompt)
   end
 
-  # rubocop:disable Metrics/MethodLength
+  # OpenAI's reasoning-family models (o1, o3, o4-mini, gpt-5, ...) only accept
+  # the default temperature (1) and reject any explicit value with a 400.
+  # Their "-chat" flavors (e.g. gpt-5-chat-latest) are non-reasoning and
+  # support a custom temperature just like gpt-4o, gpt-3.5-turbo, etc.
+  FIXED_TEMPERATURE_MODELS = /\A(o[1-9]|gpt-5)/
+
   def send(request)
+    parameters = { model: @model, messages: messages(request) }
+    parameters[:temperature] = @temperature unless fixed_temperature?
+    @client.chat(parameters: parameters).dig('choices', 0, 'message', 'content')
+  end
+
+  def fixed_temperature?
+    @model.match?(FIXED_TEMPERATURE_MODELS) && !@model.include?('chat')
+  end
+
+  SYSTEM_PROMPT = 'You are a developer tasked with composing a concise report detailing'\
+    ' your activities and progress for the previous week, intended for submission to your supervisor.'
+
+  def messages(request)
     if @model == 'o1-preview'
-      @client.chat(
-        parameters: {
-          model: @model,
-          messages: [{ role: 'user', content: request.to_s }]
-        }
-      ).dig('choices', 0, 'message', 'content')
+      [{ role: 'user', content: request.to_s }]
     else
-      @client.chat(
-        parameters: {
-          model: @model,
-          messages: [
-            { role: 'system', content: 'You are a developer tasked'\
-  ' with composing a concise report detailing your activities'\
-  ' and progress for the previous week,'\
-  ' intended for submission to your supervisor.' },
-            { role: 'user', content: request.to_s }
-          ],
-          temperature: @temperature
-        }
-      ).dig('choices', 0, 'message', 'content')
+      [{ role: 'system', content: SYSTEM_PROMPT }, { role: 'user', content: request.to_s }]
     end
   end
-  # rubocop:enable Metrics/MethodLength
 
   def deprecated(method)
     warn "Warning! '#{method}' is deprecated and will be removed in future versions."

data/lib/newsman/docx_output.rb

@@ -30,7 +30,7 @@ class Docxout
   # Content types part, required by every OOXML package.
   CONTENT_TYPES = <<~XML
     <?xml version="1.0" encoding="UTF-8" standalone="yes"?>
-    <Types xmlns="http://schemas.openxmlformats.org/package/2006/content-types"><Default Extension="rels" ContentType="application/vnd.openxmlformats-package.relationships+xml"/><Default Extension="xml" ContentType="application/xml"/><Override PartName="/word/document.xml" ContentType="application/vnd.openxmlformats-officedocument.wordprocessingml.document.main+xml"/></Types>
+    <Types xmlns="http://schemas.openxmlformats.org/package/2006/content-types"><Default Extension="rels" ContentType="application/vnd.openxmlformats-package.relationships+xml"/><Default Extension="xml" ContentType="application/xml"/><Override PartName="/word/document.xml" ContentType="application/vnd.openxmlformats-officedocument.wordprocessingml.document.main+xml"/><Override PartName="/word/numbering.xml" ContentType="application/vnd.openxmlformats-officedocument.wordprocessingml.numbering+xml"/></Types>
   XML
 
   # Package-level relationships, pointing at the main document part.
@@ -39,6 +39,25 @@ class Docxout
     <Relationships xmlns="http://schemas.openxmlformats.org/package/2006/relationships"><Relationship Id="rId1" Type="http://schemas.openxmlformats.org/officeDocument/2006/relationships/officeDocument" Target="word/document.xml"/></Relationships>
   XML
 
+  # Document-level relationships, pointing at the numbering part used by lists.
+  DOCUMENT_RELS = <<~XML
+    <?xml version="1.0" encoding="UTF-8" standalone="yes"?>
+    <Relationships xmlns="http://schemas.openxmlformats.org/package/2006/relationships"><Relationship Id="rId1" Type="http://schemas.openxmlformats.org/officeDocument/2006/relationships/numbering" Target="numbering.xml"/></Relationships>
+  XML
+
+  # Numbering definitions: numId 1 is a bulleted list, numId 2 is a decimal (numbered) list.
+  NUMBERING = <<~XML
+    <?xml version="1.0" encoding="UTF-8" standalone="yes"?>
+    <w:numbering xmlns:w="http://schemas.openxmlformats.org/wordprocessingml/2006/main"><w:abstractNum w:abstractNumId="0"><w:lvl w:ilvl="0"><w:numFmt w:val="bullet"/><w:lvlText w:val="&#8226;"/><w:pPr><w:ind w:left="720" w:hanging="360"/></w:pPr></w:lvl></w:abstractNum><w:abstractNum w:abstractNumId="1"><w:lvl w:ilvl="0"><w:numFmt w:val="decimal"/><w:lvlText w:val="%1."/><w:pPr><w:ind w:left="720" w:hanging="360"/></w:pPr></w:lvl></w:abstractNum><w:num w:numId="1"><w:abstractNumId w:val="0"/></w:num><w:num w:numId="2"><w:abstractNumId w:val="1"/></w:num></w:numbering>
+  XML
+
+  # Matches markdown headers, e.g. "## Title" (levels 1-6).
+  HEADING = /^\s{0,3}(\#{1,6})\s+(.+?)\s*$/
+  # Matches markdown bullet list items, e.g. "- item" or "* item".
+  BULLET = /^\s{0,3}[-*]\s+(.+?)\s*$/
+  # Matches markdown numbered list items, e.g. "1. item".
+  NUMBERED = /^\s{0,3}\d+\.\s+(.+?)\s*$/
+
   FONT = 'Times New Roman'
   # Word expresses font size in half-points, so 12pt is 24.
   SIZE = '24'
@@ -50,11 +69,7 @@ class Docxout
   def print(report, reporter, model)
     puts "Create a docx file in a directory #{@root}"
     path = File.join(@root, filename(reporter, model))
-    Zip::File.open(path, create: true) do |zip|
-      zip.get_output_stream('[Content_Types].xml') { |f| f.write(CONTENT_TYPES) }
-      zip.get_output_stream('_rels/.rels') { |f| f.write(RELS) }
-      zip.get_output_stream('word/document.xml') { |f| f.write(document_xml(report)) }
-    end
+    write_package(path, report)
     puts "Report was successfully printed to a #{path}"
     File.basename(path)
   end
@@ -67,16 +82,63 @@ class Docxout
 
   private
 
+  def write_package(path, report)
+    Zip::File.open(path, create: true) do |zip|
+      zip.get_output_stream('[Content_Types].xml') { |f| f.write(CONTENT_TYPES) }
+      zip.get_output_stream('_rels/.rels') { |f| f.write(RELS) }
+      zip.get_output_stream('word/_rels/document.xml.rels') { |f| f.write(DOCUMENT_RELS) }
+      zip.get_output_stream('word/numbering.xml') { |f| f.write(NUMBERING) }
+      zip.get_output_stream('word/document.xml') { |f| f.write(document_xml(report)) }
+    end
+  end
+
   def document_xml(report)
-    body = paragraphs(report).map { |lines| paragraph_xml(lines) }.join
+    body = blocks(report).map { |block| block_xml(block) }.join
     <<~XML
       <?xml version="1.0" encoding="UTF-8" standalone="yes"?>
       <w:document xmlns:w="http://schemas.openxmlformats.org/wordprocessingml/2006/main"><w:body>#{body}<w:sectPr/></w:body></w:document>
     XML
   end
 
-  def paragraphs(report)
-    report.to_s.split(/\n{2,}/).map { |paragraph| paragraph.split("\n") }
+  # Splits the report into blocks: markdown headers and list items each become
+  # their own block, and consecutive plain-text lines are grouped together.
+  def blocks(report)
+    classified = report.to_s.split("\n", -1).map { |line| classify(line) }
+    classified.chunk_while { |prev, curr| prev[:type] == :text && curr[:type] == :text }
+              .reject { |group| group.first[:type] == :blank }
+              .map { |group| merge_text(group) }
+  end
+
+  def merge_text(group)
+    return group.first unless group.first[:type] == :text
+
+    { type: :text, lines: group.map { |block| block[:line] } }
+  end
+
+  def classify(line)
+    return { type: :blank } if line.strip.empty?
+    return { type: :heading, level: ::Regexp.last_match(1).length, text: ::Regexp.last_match(2) } if line =~ HEADING
+    return { type: :bullet, text: ::Regexp.last_match(1) } if line =~ BULLET
+    return { type: :numbered, text: ::Regexp.last_match(1) } if line =~ NUMBERED
+
+    { type: :text, line: line }
+  end
+
+  def block_xml(block)
+    case block[:type]
+    when :heading
+      "<w:p><w:pPr><w:pStyle w:val=\"Heading#{block[:level]}\"/></w:pPr>#{run_xml(block[:text])}</w:p>"
+    when :bullet
+      list_item_xml(block[:text], '1')
+    when :numbered
+      list_item_xml(block[:text], '2')
+    else
+      paragraph_xml(block[:lines])
+    end
+  end
+
+  def list_item_xml(text, num_id)
+    "<w:p><w:pPr><w:numPr><w:ilvl w:val=\"0\"/><w:numId w:val=\"#{num_id}\"/></w:numPr></w:pPr>#{run_xml(text)}</w:p>"
   end
 
   def paragraph_xml(lines)

data/test/test_assistant.rb

@@ -35,4 +35,31 @@ class TestAssistant < Minitest::Test
     assert_equal expected,
                  assistant.say_hello
   end
+
+  def test_fixed_temperature_for_reasoning_models
+    %w[o1 o1-preview o1-mini o3 o3-mini o4-mini gpt-5 gpt-5-mini gpt-5.6].each do |model|
+      assistant = Assistant.new('test-token', model: model)
+      assert(assistant.fixed_temperature?, "expected #{model} to have a fixed temperature")
+    end
+  end
+
+  def test_custom_temperature_for_chat_models
+    %w[gpt-3.5-turbo gpt-4o gpt-4.1 gpt-5-chat-latest].each do |model|
+      assistant = Assistant.new('test-token', model: model)
+      refute(assistant.fixed_temperature?, "expected #{model} to allow a custom temperature")
+    end
+  end
+
+  def test_o1_preview_skips_system_message
+    assistant = Assistant.new('test-token', model: 'o1-preview')
+    messages = assistant.messages('do something')
+    assert_equal([{ role: 'user', content: 'do something' }], messages)
+  end
+
+  def test_other_models_keep_system_message
+    assistant = Assistant.new('test-token', model: 'gpt-5.6')
+    messages = assistant.messages('do something')
+    assert_equal(2, messages.size)
+    assert_equal('system', messages.first[:role])
+  end
 end

data/test/test_docxout.rb

@@ -50,4 +50,50 @@ class TestDocxout < Minitest::Test
       assert_includes(document_xml, 'Issue description')
     end
   end
+
+  def test_renders_markdown_headers_as_word_headings
+    Dir.mktmpdir do |temp_dir|
+      output = Docxout.new(temp_dir)
+      report = "# Title\n\n## Subtitle\n\n### Section"
+      path = File.join(temp_dir, output.print(report, 'volodya-lombrozo', 'gpt-3.5-turbo'))
+      document_xml = Zip::File.open(path) { |zip| zip.read('word/document.xml') }
+      %w[Heading1 Heading2 Heading3].each { |style| assert_includes(document_xml, %(<w:pStyle w:val="#{style}"/>)) }
+      assert_includes(document_xml, '>Title<')
+      refute_includes(document_xml, '# Title')
+    end
+  end
+
+  def test_renders_markdown_bullet_lists_as_word_lists
+    Dir.mktmpdir do |temp_dir|
+      output = Docxout.new(temp_dir)
+      report = "List is here:\n\n- one\n- two\n* three"
+      path = File.join(temp_dir, output.print(report, 'volodya-lombrozo', 'gpt-3.5-turbo'))
+      document_xml = Zip::File.open(path) { |zip| zip.read('word/document.xml') }
+      assert_includes(document_xml, '<w:numId w:val="1"/>')
+      assert_includes(document_xml, '>one<')
+      refute_includes(document_xml, '>- one<')
+    end
+  end
+
+  def test_renders_markdown_numbered_lists_as_word_lists
+    Dir.mktmpdir do |temp_dir|
+      output = Docxout.new(temp_dir)
+      report = "1. first\n2. second"
+      path = File.join(temp_dir, output.print(report, 'volodya-lombrozo', 'gpt-3.5-turbo'))
+      document_xml = Zip::File.open(path) { |zip| zip.read('word/document.xml') }
+      assert_includes(document_xml, '<w:numId w:val="2"/>')
+      assert_includes(document_xml, '>first<')
+    end
+  end
+
+  def test_docx_package_includes_a_valid_numbering_part
+    Dir.mktmpdir do |temp_dir|
+      output = Docxout.new(temp_dir)
+      path = File.join(temp_dir, output.print("- one\n- two", 'volodya-lombrozo', 'gpt-3.5-turbo'))
+      Zip::File.open(path) do |zip|
+        assert(zip.find_entry('word/numbering.xml'))
+        assert(zip.find_entry('word/_rels/document.xml.rels'))
+      end
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: newsman
 version: !ruby/object:Gem::Version
-  version: 1.2.1
+  version: 1.2.3
 platform: ruby
 authors:
 - Volodya Lombrozo