chore: remove "open" method

Author: davidturnbull

README.md

@@ -44,18 +44,6 @@ engine = LingoDotDev::Engine.new(api_key: 'your-api-key')
 # Localize text
 result = engine.localize_text('Hello world', target_locale: 'es')
 puts result # => "Hola mundo"
-
-# Clean up when done
-engine.close
-```
-
-For automatic resource management, use the block form:
-
-```ruby
-LingoDotDev::Engine.open(api_key: 'your-api-key') do |engine|
-  result = engine.localize_text('Hello world', target_locale: 'es')
-  puts result
-end
 ```
 
 ## Usage
@@ -354,19 +342,8 @@ Returns information about the authenticated user.
 
 - **Returns:** Hash with `:email` and `:id` keys, or `nil` if authentication fails
 
-#### `close`
-
-Closes the engine and cleans up resources.
-
 ### Class methods
 
-#### `Engine.open(api_key:, **options, &block)`
-
-Creates an engine instance with automatic resource cleanup.
-
-- **Parameters:** Same as `Engine.new`
-- **Returns:** Engine instance (or yields to block and returns block result)
-
 #### `Engine.quick_translate(content, api_key:, target_locale:, source_locale: nil, fast: true, api_url: 'https://engine.lingo.dev')`
 
 One-off translation without managing engine lifecycle.

examples/ruby-on-rails/app/controllers/translate_controller.rb

@@ -6,9 +6,8 @@ class TranslateController < ApplicationController
   def translate
     api_key = ENV['LINGODOTDEV_API_KEY'] || 'your-api-key-here'
 
-    translated = LingoDotDev::Engine.open(api_key: api_key) do |engine|
-      engine.localize_text('Hello world', target_locale: 'es')
-    end
+    engine = LingoDotDev::Engine.new(api_key: api_key)
+    translated = engine.localize_text('Hello world', target_locale: 'es')
 
     render json: {
       original: 'Hello world',

lib/lingodotdev.rb

@@ -90,21 +90,6 @@ def initialize(api_key:, api_url: 'https://engine.lingo.dev', batch_size: 25, id
       @config.send(:validate!)
     end
 
-    def self.open(api_key:, **options)
-      engine = new(api_key: api_key, **options)
-      return engine unless block_given?
-
-      begin
-        yield engine
-      ensure
-        engine.close
-      end
-    end
-
-    def close
-      @client = nil
-    end
-
     def localize_text(text, target_locale:, source_locale: nil, fast: nil, reference: nil, on_progress: nil, concurrent: false, &block)
       raise ValidationError, 'Target locale is required' if target_locale.nil? || target_locale.empty?
       raise ValidationError, 'Text cannot be nil' if text.nil?
@@ -272,53 +257,51 @@ def whoami
     end
 
     def self.quick_translate(content, api_key:, target_locale:, source_locale: nil, fast: true, api_url: 'https://engine.lingo.dev')
-      open(api_key: api_key, api_url: api_url) do |engine|
-        case content
-        when String
-          engine.localize_text(
-            content,
-            target_locale: target_locale,
-            source_locale: source_locale,
-            fast: fast
-          )
-        when Hash
-          engine.localize_object(
-            content,
-            target_locale: target_locale,
-            source_locale: source_locale,
-            fast: fast,
-            concurrent: true
-          )
-        else
-          raise ValidationError, 'Content must be a String or Hash'
-        end
+      engine = new(api_key: api_key, api_url: api_url)
+      case content
+      when String
+        engine.localize_text(
+          content,
+          target_locale: target_locale,
+          source_locale: source_locale,
+          fast: fast
+        )
+      when Hash
+        engine.localize_object(
+          content,
+          target_locale: target_locale,
+          source_locale: source_locale,
+          fast: fast,
+          concurrent: true
+        )
+      else
+        raise ValidationError, 'Content must be a String or Hash'
       end
     end
 
     def self.quick_batch_translate(content, api_key:, target_locales:, source_locale: nil, fast: true, api_url: 'https://engine.lingo.dev')
-      open(api_key: api_key, api_url: api_url) do |engine|
-        case content
-        when String
-          engine.batch_localize_text(
+      engine = new(api_key: api_key, api_url: api_url)
+      case content
+      when String
+        engine.batch_localize_text(
+          content,
+          target_locales: target_locales,
+          source_locale: source_locale,
+          fast: fast,
+          concurrent: true
+        )
+      when Hash
+        target_locales.map do |target_locale|
+          engine.localize_object(
             content,
-            target_locales: target_locales,
+            target_locale: target_locale,
             source_locale: source_locale,
             fast: fast,
             concurrent: true
           )
-        when Hash
-          target_locales.map do |target_locale|
-            engine.localize_object(
-              content,
-              target_locale: target_locale,
-              source_locale: source_locale,
-              fast: fast,
-              concurrent: true
-            )
-          end
-        else
-          raise ValidationError, 'Content must be a String or Hash'
         end
+      else
+        raise ValidationError, 'Content must be a String or Hash'
       end
     end
 

spec/lingo_dot_dev/engine_spec.rb

@@ -34,30 +34,6 @@
     end
   end
 
-  describe '.open' do
-    it 'returns an engine without a block' do
-      engine = described_class.open(api_key: api_key)
-      expect(engine).to be_a(described_class)
-    end
-
-    it 'yields engine to block and closes after' do
-      result = nil
-      described_class.open(api_key: api_key) do |engine|
-        expect(engine).to be_a(described_class)
-        result = engine
-      end
-      expect(result.instance_variable_get(:@client)).to be_nil
-    end
-  end
-
-  describe '#close' do
-    it 'clears the http client' do
-      engine = described_class.new(api_key: api_key)
-      engine.close
-      expect(engine.instance_variable_get(:@client)).to be_nil
-    end
-  end
-
   describe '#localize_text' do
     it 'localizes text to target locale' do
       engine = described_class.new(api_key: api_key)