bandcamp-discover 0.7.0 → 0.7.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0e4dc43f9146747097ff3c34b9a581e02e96ad6e3dc812b84e7c77f1dd459147
-  data.tar.gz: 208fdb4ae22424fb73d93722bbbb1158703a89c4626a4897dbfb79a5dffa32b1
+  metadata.gz: '094dffa15caad4bbabef29bd4b3d87789425e4736c2d06461d6917efc8fe600d'
+  data.tar.gz: de1a1d20425f1ecb984878b10541208849abbf7300cd46f80834e5197730ef1c
 SHA512:
-  metadata.gz: 615dd28ade655b5c351249c4e98d1088102bb85d9ad002ca4e0396f49b60bd33c5fe058198f366dab1300f825e2e858a721e5ce715f866a77ce6a6f3afac8b32
-  data.tar.gz: 6cc60964b92fd4cbc56544c13606397632fd526b4871cd1ae28feaa3cfcde54b1cbf7c6483c9c613e10e09a876a582d42f9d7890575cb0fb097fd5e351cdd3ed
+  metadata.gz: a6a057d4869da1039b00b9c2ffd5b4019123c4ee055cf392fd3fefc4b8fa7f427d2346081554e960a5416c95b52d507d9244912112dada5d0ddedec06820ac33
+  data.tar.gz: ae37374ebc23a193aedaed32cd741f01af048502cb9853596d6e9bf1438ec2040fcacdaee67f226951accf6bd9d674191faf8667f5bf86cc3b9216d84a408bbc

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    bandcamp-discover (0.7.0)
+    bandcamp-discover (0.7.2)
       async
       base64
       concurrent-ruby

data/lib/bandcamp-discover/analyzer.rb

@@ -3,6 +3,10 @@ require_relative "configuration"
 
 module BandcampDiscover
   class Analyzer
+    # The model replied, but not with the object asked for. Carries the reply so
+    # a failed job says what was said instead of where JSON.parse gave up.
+    class NoAnswer < StandardError; end
+
     # A per-call model still wins over the configured one so existing callers
     # keep their behaviour.
     def initialize(description, model = nil, credits: [])
@@ -17,10 +21,14 @@ module BandcampDiscover
       @description.to_s.match?(/label|platform|records/i)
     end
 
+    # The prompt says to default to false; a reply that never reached an answer
+    # is that default, not a reason to lose the whole scrape.
     def accepts_demos?
       return false unless llm?
 
       ask(BandcampDiscover.configuration.demos_prompt)
+    rescue NoAnswer
+      false
     end
 
     private
@@ -43,7 +51,15 @@ module BandcampDiscover
         extras: {response_format: {type: "json_object"}}
       )
 
-      JSON.parse(response["choices"][0]["message"]["content"])["answer"]&.to_s&.downcase == "true"
+      answer(response["choices"][0]["message"]["content"])["answer"]&.to_s&.downcase == "true"
+    end
+
+    # response_format is advisory on OpenRouter: Anthropic models return the
+    # object inside a ```json fence, and some prepend a sentence.
+    def answer(content)
+      JSON.parse(content[/\{.*\}/m] || content)
+    rescue JSON::ParserError
+      raise NoAnswer, "model replied without a JSON object: #{content.to_s.strip[0, 200].inspect}"
     end
 
     # The bio alone cannot tell one person releasing under aliases from a

data/lib/bandcamp-discover/version.rb

@@ -1,3 +1,3 @@
 module BandcampDiscover
-  VERSION = "0.7.0"
+  VERSION = "0.7.2"
 end

data/test/analyzer_test.rb

@@ -23,12 +23,12 @@ module OpenRouter
 
   class Client
     class << self
-      attr_accessor :requests, :answer
+      attr_accessor :requests, :answer, :raw
     end
 
     def complete(messages, model:, extras:)
       self.class.requests << {messages: messages, model: model, extras: extras}
-      {"choices" => [{"message" => {"content" => JSON.generate("answer" => self.class.answer)}}]}
+      {"choices" => [{"message" => {"content" => self.class.raw || JSON.generate("answer" => self.class.answer)}}]}
     end
   end
 end
@@ -41,6 +41,7 @@ class AnalyzerTest < Minitest::Test
     OpenRouter.configuration.access_token = "token"
     OpenRouter::Client.requests = []
     OpenRouter::Client.answer = true
+    OpenRouter::Client.raw = nil
   end
 
   def teardown
@@ -99,6 +100,31 @@ class AnalyzerTest < Minitest::Test
     assert_equal "", OpenRouter::Client.requests.last[:messages].last[:content]
   end
 
+  def test_reads_the_answer_out_of_a_markdown_fence
+    OpenRouter::Client.raw = "```json\n{\"answer\": false}\n```"
+
+    refute Analyzer.new("bio").label?
+  end
+
+  def test_reads_the_answer_out_of_surrounding_prose
+    OpenRouter::Client.raw = "Sure. {\"answer\": true} Hope that helps."
+
+    assert Analyzer.new("bio").label?
+  end
+
+  def test_label_raises_a_named_error_carrying_a_reply_without_an_object
+    OpenRouter::Client.raw = "I don't see enough information to decide."
+
+    error = assert_raises(BandcampDiscover::Analyzer::NoAnswer) { Analyzer.new("bio").label? }
+    assert_includes error.message, "I don't see enough information"
+  end
+
+  def test_demos_defaults_to_false_on_a_reply_without_an_object
+    OpenRouter::Client.raw = "I don't see any mention of demos here."
+
+    refute Analyzer.new("bio").accepts_demos?
+  end
+
   def test_parses_the_answer
     OpenRouter::Client.answer = false
 

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: bandcamp-discover
 version: !ruby/object:Gem::Version
-  version: 0.7.0
+  version: 0.7.2
 platform: ruby
 authors:
 - Julian RUbisch
 bindir: bin
 cert_chain: []
-date: 2026-09-03 00:00:00.000000000 Z
+date: 2026-09-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake