purl_fetcher-client 0.5.0 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b80b63b29ffbed55055e58d2d063f4496f84015b1130a6908e82e4d91a18c94d
-  data.tar.gz: 3d9691d8e9939f17a8d4f3a66a03bb3644463a3ffcc9604b8558242614f4e0a6
+  metadata.gz: cb1a08fc0010f80045db352cb2bfb2735dd2b7447513ba366281cc4d488895e4
+  data.tar.gz: a2213eda4272694dcf1e889ed9185956b0ba1da24e98056e2360a242e9eb6a0b
 SHA512:
-  metadata.gz: '0484a2254521f201fed8d95cf65ab85a3310111c6be46c6d13ecb4e7c40cec3d807861b42c47dc389f34c242918733dce7c21b66beb872d062112bbfe8674dd0'
-  data.tar.gz: ff5cdc4fe8d78eedeba1594e8aa47e87bfb6186337d4f1042f928a56f0a6b854487e7acfba5398e306401a123cb710ae0ac4bc6b9b11b157a81d20550b789570
+  metadata.gz: f8a0200ff798158ca1b5c58c120c94b1cb3e8f172cec1c28b30b1a1cae47291483dc5d681467a17f08d7ef650a14bf9fffc0c8d33895fd36565e2b2ea7e70ac9
+  data.tar.gz: 4600d53676d35892f3485f9cbc280afb4cc1fd8cedd9028794376cb20b9a49ad6d3981279e84e89b56549b87173fdc106aaa09f459d1d461a72675b2c2c2ca12

data/.github/workflows/ruby.yml

@@ -2,9 +2,9 @@ name: CI
 
 on:
   push:
-    branches: [ master ]
+    branches: [ main ]
   pull_request:
-    branches: [ master ]
+    branches: [ main ]
 
 jobs:
   tests:

data/.rubocop.yml

@@ -0,0 +1,5 @@
+inherit_gem:
+  rubocop-rails-omakase: rubocop.yml
+
+AllCops:
+  TargetRubyVersion: 3.2

data/Gemfile

@@ -1,6 +1,9 @@
 source "https://rubygems.org"
 
-git_source(:github) {|repo_name| "https://github.com/#{repo_name}" }
+git_source(:github) { |repo_name| "https://github.com/#{repo_name}" }
 
 # Specify your gem's dependencies in purl_fetcher-client.gemspec
 gemspec
+
+gem "rubocop-rails-omakase", require: false, group: [ :development ]
+gem "debug"

data/README.md

@@ -22,7 +22,25 @@ Or install it yourself as:
 
 ## Usage
 
-TODO: Write usage instructions here
+### Uploading a file
+
+```ruby
+PurlFetcher::Client.configure(url:'http://127.0.0.1:3000', token: 'abc123')
+
+PurlFetcher::Client::UploadFiles.upload(
+  file_metadata: {
+    'file1.txt' => PurlFetcher::Client::DirectUploadRequest.new(
+      checksum: '123',
+      byte_size: 10_000,
+      content_type: 'image/tiff',
+      filename: 'image.tiff'
+    )
+  },
+  filepath_map: {
+    'file1.txt' => File.expand_path('Gemfile.lock')
+  }
+)
+```
 
 ## Development
 
@@ -36,4 +54,4 @@ Bug reports and pull requests are welcome on GitHub at https://github.com/[USERN
 
 ## Code of Conduct
 
-Everyone interacting in the PurlFetcher::Client project’s codebases, issue trackers, chat rooms and mailing lists is expected to follow the [code of conduct](https://github.com/[USERNAME]/purl_fetcher-client/blob/master/CODE_OF_CONDUCT.md).
+Everyone interacting in the PurlFetcher::Client project’s codebases, issue trackers, chat rooms and mailing lists is expected to follow the [code of conduct](https://github.com/[USERNAME]/purl_fetcher-client/blob/main/CODE_OF_CONDUCT.md).

data/Rakefile

@@ -1,6 +1,6 @@
-require "bundler/gem_tasks"
-require "rspec/core/rake_task"
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
 
 RSpec::Core::RakeTask.new(:spec)
 
-task :default => :spec
+task default: :spec

data/bin/console

@@ -1,7 +1,8 @@
 #!/usr/bin/env ruby
 
-require "bundler/setup"
-require "purl_fetcher/client"
+require 'bundler/setup'
+require 'purl_fetcher/client'
+require 'debug'
 
 # You can add fixtures and/or initialization code here to make experimenting
 # with your gem easier. You can also use a different console, if you like.
@@ -10,5 +11,5 @@ require "purl_fetcher/client"
 # require "pry"
 # Pry.start
 
-require "irb"
+require 'irb'
 IRB.start(__FILE__)

data/lib/purl_fetcher/client/direct_upload_request.rb

@@ -0,0 +1,39 @@
+# frozen_string_literal: true
+
+require "digest"
+
+module PurlFetcher
+  class Client
+    # This models the JSON that we send to the server.
+    DirectUploadRequest = Data.define(:checksum, :byte_size, :content_type, :filename) do
+      def self.from_file(hexdigest:, byte_size:, file_name:, content_type:)
+        new(checksum: hex_to_base64_digest(hexdigest),
+            byte_size: byte_size,
+            content_type: clean_content_type(content_type),
+            filename: file_name)
+      end
+
+      def to_h
+        {
+          blob: { filename: filename, byte_size: byte_size, checksum: checksum,
+                  content_type: self.class.clean_content_type(content_type) }
+        }
+      end
+
+      def to_json(*_args)
+        JSON.generate(to_h)
+      end
+
+      def self.clean_content_type(content_type)
+        return "application/octet-stream" if content_type.blank?
+
+        # ActiveStorage is expecting "application/x-stata-dta" not "application/x-stata-dta;version=14"
+        content_type.split(";").first
+      end
+
+      def self.hex_to_base64_digest(hexdigest)
+        [ [ hexdigest ].pack("H*") ].pack("m0")
+      end
+    end
+  end
+end

data/lib/purl_fetcher/client/direct_upload_response.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+module PurlFetcher
+  class Client
+    DirectUploadResponse = Data.define(:id, :key, :checksum, :byte_size, :content_type,
+                                      :filename, :metadata, :created_at, :direct_upload,
+                                      :signed_id, :service_name) do
+                                        def with_filename(filename)
+                                          self.class.new(**deconstruct_keys(nil).merge(filename:))
+                                        end
+                                      end
+  end
+end

data/lib/purl_fetcher/client/reader.rb

@@ -1,77 +1,48 @@
 class PurlFetcher::Client::Reader
   include Enumerable
-  attr_reader :input_stream, :settings, :range
+  attr_reader :host, :conn, :range
 
-  def initialize(input_stream, settings = {})
-    @settings = settings
-    @input_stream = input_stream
+  def initialize(host: "https://purl-fetcher.stanford.edu", conn: nil)
+    @host = host
+    @conn = conn || Faraday.new(host) do |f|
+              f.response :json
+            end
     @range = {}
   end
 
-  def each
-    return to_enum(:each) unless block_given?
-
-    changes(first_modified: first_modified, target: target).each do |change, meta|
-      next unless target.nil? || (change['true_targets'] && change['true_targets'].include?(target))
-
-      public_xml = PurlFetcher::Client::PublicXmlRecord.new(change['druid'].sub('druid:', ''), settings)
-
-      yield public_xml, change, self
-    end
-  end
-
   def collection_members(druid)
     return to_enum(:collection_members, druid) unless block_given?
 
-    paginated_get("/collections/druid:#{druid.sub(/^druid:/, '')}/purls", 'purls').each do |obj, _meta|
-      yield PurlFetcher::Client::PublicXmlRecord.new(obj['druid'].sub('druid:', ''), settings), obj, self
+    paginated_get("/collections/druid:#{druid.delete_prefix('druid:')}/purls", "purls").each do |obj, _meta|
+      yield obj["druid"].delete_prefix("druid:")
     end
   end
 
   private
 
-  def first_modified
-    settings['purl_fetcher.first_modified']
-  end
-
-  def target
-    settings['purl_fetcher.target']
-  end
-
-  ##
-  # @return [Enumerator]
-  def changes(params = {})
-    paginated_get('/docs/changes', 'changes', params)
-  end
-
-  ##
-  # @return [Enumerator]
-  def deletes(params = {})
-    paginated_get('/docs/deletes', 'deletes', params)
-  end
-
   ##
   # @return [Hash] a parsed JSON hash
-  def get(path, params = {})
-    JSON.parse(fetch(settings.fetch('purl_fetcher.api_endpoint', 'https://purl-fetcher.stanford.edu') + path, params))
-  end
+  def fetch(path, params)
+    response = conn.get(path, params: params)
 
-  def fetch(url, params)
-    if defined?(Manticore)
-      Manticore.get(url, query: params).body
-    else
-      HTTP.get(url, params: params).body
+    unless response.success?
+      if defined?(Honeybadger)
+        Honeybadger.context({ path:, params:, response_code: response.code, body: response.body })
+      end
+      raise PurlFetcher::Client::ResponseError, "Unsuccessful response from purl-fetcher"
     end
+
+    response.body
   end
 
   ##
   # For performance, and enumberable object is returned.
   #
   # @example operating on each of the results as they come in
-  #   paginated_get('/docs/changes', 'changes').map { |v| puts v.inspect }
+  #   paginated_get('/docs/collections/druid:123', 'purls').map { |v| puts v.inspect }
   #
   # @example getting all of the results and converting to an array
-  #   paginated_get('/docs/changes', 'changes').to_a
+  #   paginated_get('/docs/collections/druid:123', 'purls').to_a
   #
   # @return [Enumerator] an enumberable object
   def paginated_get(path, accessor, options = {})
@@ -83,8 +54,8 @@ class PurlFetcher::Client::Reader
       total    = 0
 
       loop do
-        data = get(path, { per_page: per_page, page: page }.merge(params))
-        @range = data['range']
+        data = fetch(path, { per_page: per_page, page: page }.merge(params))
+        @range = data["range"]
 
         total += data[accessor].length
 
@@ -92,7 +63,7 @@ class PurlFetcher::Client::Reader
           yielder.yield element, self
         end
 
-        page = data['pages']['next_page']
+        page = data["pages"]["next_page"]
 
         break if page.nil? || total >= max
       end

data/lib/purl_fetcher/client/upload_files.rb

@@ -0,0 +1,70 @@
+# frozen_string_literal: true
+
+module PurlFetcher
+  class Client
+    # The file uploading part of a transfer
+    class UploadFiles
+      # @param [Hash<String,DirectUploadRequest>] file_metadata map of relative filepaths to file metadata
+      # @param [Hash<String,String>] filepath_map map of relative filepaths to absolute filepaths
+      def self.upload(file_metadata:, filepath_map:)
+        new(file_metadata: file_metadata, filepath_map: filepath_map).upload
+      end
+
+      # @param [Hash<String,DirectUploadRequest>] file_metadata map of relative filepaths to file metadata
+      # @param [Hash<String,String>] filepath_map map of relative filepaths to absolute filepaths
+      def initialize(file_metadata:, filepath_map:)
+        @file_metadata = file_metadata
+        @filepath_map = filepath_map
+      end
+
+      # @return [Array<DirectUploadResponse>] the responses from the server for the uploads
+      def upload
+        file_metadata.map do |filepath, metadata|
+          direct_upload(metadata.to_json).tap do |response|
+            # ActiveStorage modifies the filename provided in response, so setting here with the relative filename
+            response = response.with_filename(filepath)
+            upload_file(response)
+            logger.info("Upload of #{filepath} complete")
+          end
+        end
+      end
+
+      private
+
+      attr_reader :file_metadata, :filepath_map
+
+      def logger
+        Client.config.logger
+      end
+
+      def client
+        Client.instance
+      end
+
+      def path
+        "/v1/direct_uploads"
+      end
+
+      def direct_upload(metadata_json)
+        logger.info("Starting an upload request: #{metadata_json}")
+        response = client.post(path: path, body: metadata_json)
+
+        logger.info("Response from server: #{response}")
+        DirectUploadResponse.new(**response.symbolize_keys)
+      end
+
+      def upload_file(response)
+        logger.info("Uploading `#{response.filename}' to #{response.direct_upload.fetch('url')}")
+
+        client.put(
+          path: response.direct_upload.fetch("url"),
+          body: ::File.open(filepath_map[response.filename]),
+          headers: {
+            "content-type" => response.content_type,
+            "content-length" => response.byte_size.to_s
+          }
+        )
+      end
+    end
+  end
+end

data/lib/purl_fetcher/client/version.rb

@@ -1,5 +1,5 @@
 module PurlFetcher
-  module Client
-    VERSION = "0.5.0"
+  class Client
+    VERSION = "1.1.0"
   end
 end

data/lib/purl_fetcher/client.rb

@@ -1,15 +1,92 @@
+require "active_support"
+require "active_support/core_ext"
+require "faraday"
+require "singleton"
+require "logger"
+
 require "purl_fetcher/client/version"
-require 'http'
-begin
-  require 'manticore' if defined? JRUBY_VERSION
-rescue LoadError
-end
+require "purl_fetcher/client/reader"
+require "purl_fetcher/client/upload_files"
+require "purl_fetcher/client/direct_upload_request"
+require "purl_fetcher/client/direct_upload_response"
 
 module PurlFetcher
-  module Client
-    require 'purl_fetcher/client/public_xml_record'
-    require 'purl_fetcher/client/reader'
-    require 'purl_fetcher/client/deletes_reader'
-    # Your code goes here...
+  class Client
+    # General error originating in PurlFetcher::Client
+    class Error < StandardError; end
+
+    # Raised when the response from the server is not successful
+    class ResponseError < Error; end
+
+    include Singleton
+    class << self
+      def configure(url:, logger: default_logger, token: nil)
+        instance.config = Config.new(
+          url: url,
+          logger: logger,
+          token: token
+        )
+
+        instance
+      end
+
+      def default_logger
+        Logger.new($stdout)
+      end
+
+      delegate :config, to: :instance
+    end
+
+    attr_accessor :config
+
+    # Send an POST request
+    # @param path [String] the path for the API request
+    # @param body [String] the body of the POST request
+    def post(path:, body:)
+      response = connection.post(path) do |request|
+        request.body = body
+      end
+
+      raise "unexpected response: #{response.status} #{response.body}" unless response.success?
+
+      response.body
+    end
+
+    # Send an PUT request
+    # @param path [String] the path for the API request
+    # @param body [String] the body of the POST request
+    # @param headers [Hash] extra headers to add to the SDR API request
+    def put(path:, body:, headers: {})
+      response = connection.put(path) do |request|
+        request.body = body
+        request.headers = default_headers.merge(headers)
+      end
+
+      raise "unexpected response: #{response.status} #{response.body}" unless response.success?
+
+      response.body
+    end
+
+    private
+
+    Config = Data.define(:url, :logger, :token)
+
+    def connection
+      Faraday.new(
+        url: config.url,
+        headers: default_headers
+      ) do |conn|
+        conn.response :json
+      end
+    end
+
+    def default_headers
+      {
+        accept: "application/json",
+        content_type: "application/json"
+      }.tap do |headers|
+        headers[:authorization] = "Bearer #{config.token}" if config.token
+      end
+    end
   end
 end

data/purl_fetcher-client.gemspec

@@ -1,32 +1,30 @@
-
-lib = File.expand_path("../lib", __FILE__)
+lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require "purl_fetcher/client/version"
+require 'purl_fetcher/client/version'
 
 Gem::Specification.new do |spec|
-  spec.name          = "purl_fetcher-client"
+  spec.name          = 'purl_fetcher-client'
   spec.version       = PurlFetcher::Client::VERSION
-  spec.authors       = ["Chris Beer"]
-  spec.email         = ["cabeer@stanford.edu"]
+  spec.authors       = [ "Chris Beer" ]
+  spec.email         = [ "cabeer@stanford.edu" ]
 
   spec.summary       = 'Traject-compatible reader implementation for streaming data from purl-fetcher'
 
   # Specify which files should be added to the gem when it is released.
   # The `git ls-files -z` loads the files in the RubyGem that have been added into git.
-  spec.files         = Dir.chdir(File.expand_path('..', __FILE__)) do
+  spec.files         = Dir.chdir(File.expand_path(__dir__)) do
     `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
   end
-  spec.bindir        = "exe"
+  spec.bindir        = 'exe'
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
-  spec.require_paths = ["lib"]
+  spec.require_paths = [ "lib" ]
 
-  spec.add_dependency 'http'
-  spec.add_dependency 'nokogiri'
-  spec.add_dependency 'stanford-mods'
-  spec.add_dependency 'dor-rights-auth'
-  spec.add_dependency 'mods_display', '>= 1.0.0.alpha1'
+  spec.add_dependency 'activesupport'
+  spec.add_dependency 'faraday', '~> 2.1'
 
-  spec.add_development_dependency "bundler"
-  spec.add_development_dependency "rake"
-  spec.add_development_dependency "rspec", "~> 3.0"
+  spec.add_development_dependency 'bundler'
+  spec.add_development_dependency 'debug'
+  spec.add_development_dependency 'rake'
+  spec.add_development_dependency 'rspec', '~> 3.0'
+  spec.add_development_dependency 'webmock'
 end

metadata

@@ -1,17 +1,17 @@
 --- !ruby/object:Gem::Specification
 name: purl_fetcher-client
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 1.1.0
 platform: ruby
 authors:
 - Chris Beer
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2022-01-13 00:00:00.000000000 Z
+date: 2024-05-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: http
+  name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -25,27 +25,27 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: nokogiri
+  name: faraday
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.1'
 - !ruby/object:Gem::Dependency
-  name: stanford-mods
+  name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-  type: :runtime
+  type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
@@ -53,13 +53,13 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: dor-rights-auth
+  name: debug
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-  type: :runtime
+  type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
@@ -67,35 +67,35 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: mods_display
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.0.0.alpha1
-  type: :runtime
+        version: '0'
+  type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.0.0.alpha1
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: bundler
+  name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.0'
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: webmock
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -108,20 +108,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: rspec
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.0'
 description:
 email:
 - cabeer@stanford.edu
@@ -132,6 +118,7 @@ files:
 - ".github/workflows/ruby.yml"
 - ".gitignore"
 - ".rspec"
+- ".rubocop.yml"
 - CODE_OF_CONDUCT.md
 - Gemfile
 - README.md
@@ -139,9 +126,10 @@ files:
 - bin/console
 - bin/setup
 - lib/purl_fetcher/client.rb
-- lib/purl_fetcher/client/deletes_reader.rb
-- lib/purl_fetcher/client/public_xml_record.rb
+- lib/purl_fetcher/client/direct_upload_request.rb
+- lib/purl_fetcher/client/direct_upload_response.rb
 - lib/purl_fetcher/client/reader.rb
+- lib/purl_fetcher/client/upload_files.rb
 - lib/purl_fetcher/client/version.rb
 - purl_fetcher-client.gemspec
 homepage:
@@ -162,7 +150,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.32
+rubygems_version: 3.4.19
 signing_key:
 specification_version: 4
 summary: Traject-compatible reader implementation for streaming data from purl-fetcher

data/lib/purl_fetcher/client/deletes_reader.rb

@@ -1,21 +0,0 @@
-class PurlFetcher::Client::DeletesReader < PurlFetcher::Client::Reader
-  # Enumerate objects that should be deleted.
-  def each
-    return to_enum(:each) unless block_given?
-
-    deletes(first_modified: first_modified).each do |change|
-
-      public_xml = PurlFetcher::Client::PublicXmlRecord.new(change['druid'].sub('druid:', ''), settings)
-
-      yield public_xml, change, self
-    end
-
-    changes(first_modified: first_modified, target: target).each do |change|
-      public_xml = PurlFetcher::Client::PublicXmlRecord.new(change['druid'].sub('druid:', ''), settings)
-
-      next unless target.nil? || (change['false_targets'] && change['false_targets'].include?(target)) || (settings['skip_if_catkey'] && record.catkey)
-
-      yield public_xml, change, self
-    end
-  end
-end

data/lib/purl_fetcher/client/public_xml_record.rb

@@ -1,205 +0,0 @@
-require 'nokogiri'
-require 'stanford-mods'
-require 'mods_display'
-require 'dor/rights_auth'
-
-module PurlFetcher::Client
-  class PublicXmlRecord
-    attr_reader :druid, :options
-
-    def self.fetch(url)
-      if defined?(JRUBY_VERSION)
-        response = Manticore.get(url)
-        response.body if response.code == 200
-      else
-        response = HTTP.get(url)
-        response.body if response.status.ok?
-      end
-    end
-
-    def initialize(druid, options = {})
-      @druid = druid
-      @options = options
-    end
-
-    def searchworks_id
-      catkey.nil? ? druid : catkey
-    end
-
-    # @return catkey value from the DOR identity_metadata, or nil if there is no catkey
-    def catkey
-      get_value(public_xml_doc.xpath("/publicObject/identityMetadata/otherId[@name='catkey']"))
-    end
-
-    # @return objectLabel value from the DOR identity_metadata, or nil if there is no barcode
-    def label
-      get_value(public_xml_doc.xpath('/publicObject/identityMetadata/objectLabel'))
-    end
-
-    def get_value(node)
-      (node && node.first) ? node.first.content : nil
-    end
-
-    def stanford_mods
-      @smods_rec ||= Stanford::Mods::Record.new.tap do |smods_rec|
-        smods_rec.from_str(mods.to_s)
-      end
-    end
-
-    def mods_display
-      @mods_display ||= ModsDisplay::HTML.new(stanford_mods)
-    end
-
-    def public_xml
-      @public_xml ||= self.class.fetch(purl_base_url + "/#{druid}.xml")
-    end
-
-    def public_xml?
-      !!public_xml
-    end
-
-    def public_xml_doc
-      @public_xml_doc ||= Nokogiri::XML(public_xml)
-    end
-
-    def mods
-      @mods ||= if public_xml_doc.xpath('/publicObject/mods:mods', mods: 'http://www.loc.gov/mods/v3').any?
-        public_xml_doc.xpath('/publicObject/mods:mods', mods: 'http://www.loc.gov/mods/v3').first
-      else
-        if defined?(Honeybadger)
-          Honeybadger.notify(
-            'Unable to find MODS in the public xml; falling back to stand-along mods document',
-            context: { druid: druid }
-          )
-        end
-
-        Nokogiri::XML(self.class.fetch(purl_base_url + "/#{druid}.mods"))
-      end
-    end
-
-    # @return true if the identityMetadata has <objectType>collection</objectType>, false otherwise
-    def is_collection
-      object_type_nodes = public_xml_doc.xpath('//objectType')
-      object_type_nodes.find_index { |n| %w(collection set).include? n.text.downcase }
-    end
-
-    # value is used to tell SearchWorks UI app of specific display needs for objects
-    # this comes from the <thumb> element in publicXML or the first image found (as parsed by discovery-indexer)
-    # @return [String] filename or nil if none found
-    def thumb
-      return if is_collection
-      encoded_thumb if %w(book image manuscript map webarchive-seed).include?(dor_content_type)
-    end
-
-    # the value of the type attribute for a DOR object's contentMetadata
-    #  more info about these values is here:
-    #    https://consul.stanford.edu/display/chimera/DOR+content+types%2C+resource+types+and+interpretive+metadata
-    #    https://consul.stanford.edu/display/chimera/Summary+of+Content+Types%2C+Resource+Types+and+their+behaviors
-    # @return [String]
-    def dor_content_type
-      public_xml_doc.xpath('//contentMetadata/@type').text
-    end
-
-    # the thumbnail in publicXML, falling back to the first image if no thumb node is found
-     # @return [String] thumb filename with druid prepended, e.g. oo000oo0001/filename withspace.jp2
-     def parse_thumb
-       unless public_xml_doc.nil?
-         thumb = public_xml_doc.xpath('//thumb')
-         # first try and parse what is in the thumb node of publicXML, but fallback to the first image if needed
-         if thumb.size == 1
-           thumb.first.content
-         elsif thumb.size == 0 && parse_sw_image_ids.size > 0
-           parse_sw_image_ids.first
-         else
-           nil
-         end
-       end
-     end
-
-     # the druid and id attribute of resource/file and objectId and fileId of the
-     # resource/externalFile elements that match the image, page, or thumb resource type, including extension
-     # Also, prepends the corresponding druid and / specifically for Searchworks use
-     # @return [Array<String>] filenames
-     def parse_sw_image_ids
-       public_xml_doc.xpath('//resource[@type="page" or @type="image" or @type="thumb"]').map do |node|
-         node.xpath('./file[@mimetype="image/jp2"]/@id').map{ |x| "#{@druid.gsub('druid:','')}/" + x } << node.xpath('./externalFile[@mimetype="image/jp2"]').map do |y|
-           "#{y.attributes['objectId'].text.split(':').last}" + "/" + "#{y.attributes['fileId']}"
-         end
-       end.flatten
-     end
-
-     def collections
-       @collections ||= predicate_druids('isMemberOfCollection').map do |druid|
-         PublicXmlRecord.new(druid, options)
-       end
-     end
-
-     def constituents
-       @constituents ||= predicate_druids('isConstituentOf').map do |druid|
-         PublicXmlRecord.new(druid, options)
-       end
-     end
-
-     def items(&block)
-       return [] unless is_collection
-
-       purl_fetcher_client.collection_members(druid, &block)
-     end
-
-     # the thumbnail in publicXML properly URI encoded, including the slash separator
-     # @return [String] thumb filename with druid prepended, e.g. oo000oo0001%2Ffilename%20withspace.jp2
-    def encoded_thumb
-      thumb=parse_thumb
-      return unless thumb
-      thumb_druid=thumb.split('/').first # the druid (before the first slash)
-      thumb_filename=thumb.split(/[a-zA-Z]{2}[0-9]{3}[a-zA-Z]{2}[0-9]{4}[\/]/).last # everything after the druid
-      "#{thumb_druid}%2F#{ERB::Util.url_encode(thumb_filename)}"
-    end
-
-    # get the druids from predicate relationships in rels-ext from public_xml
-    # @return [Array<String>, nil] the druids (e.g. ww123yy1234) from the rdf:resource of the predicate relationships, or nil if none
-    def predicate_druids(predicate, predicate_ns = 'info:fedora/fedora-system:def/relations-external#')
-      ns_hash = { 'rdf' => 'http://www.w3.org/1999/02/22-rdf-syntax-ns#', 'pred_ns' => predicate_ns }
-      xpth = "/publicObject/rdf:RDF/rdf:Description/pred_ns:#{predicate}/@rdf:resource"
-      pred_nodes = public_xml_doc.xpath(xpth, ns_hash)
-      pred_nodes.reject { |n| n.value.empty? }.map do |n|
-        n.value.split('druid:').last
-      end
-    end
-
-    def druid_tree
-      druid.match(/(..)(...)(..)(....)/).captures.join('/')
-    end
-
-    def rights_xml
-      @rights_xml ||= public_xml_doc.xpath('//rightsMetadata').to_s
-    end
-
-    def rights
-      @rights ||= ::Dor::RightsAuth.parse(rights_xml)
-    end
-
-    def public?
-      rights.world_unrestricted?
-    end
-
-    def stanford_only?
-      rights.stanford_only_unrestricted?
-    end
-
-    def purl_base_url
-      options[:purl_url]&.sub(%r{/$}, '') || 'https://purl.stanford.edu'
-    end
-
-    def purl_fetcher_api_endpoint
-      options[:purl_fetcher_url] || 'https://purl-fetcher.stanford.edu'
-    end
-
-    def purl_fetcher_client
-      @purl_fetcher_client ||= PurlFetcher::Client::Reader.new(
-        nil,
-        'purl_fetcher.api_endpoint' => purl_fetcher_api_endpoint
-      )
-    end
-  end
-end