linear_api 0.3.2 → 0.5.0

data/lib/linear_api.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+require 'logger'
 require_relative 'linear_api/version'
 require_relative 'linear_api/client'
 require_relative 'linear_api/result'
@@ -19,23 +20,59 @@ module LinearApi
   class Error < StandardError; end
   class AuthenticationError < Error; end
   class NotFoundError < Error; end
-  class RateLimitError < Error; end
+
+  class RateLimitError < Error
+    attr_reader :retry_after
+
+    def initialize(message = 'Rate limited', retry_after: nil)
+      @retry_after = retry_after
+      super(message)
+    end
+  end
+
+  # Sensitive key patterns to redact from issue descriptions
+  SENSITIVE_KEY_PATTERN = /password|secret|token|key|credential|ssn|api_key|authorization/i
 
   class << self
-    attr_accessor :api_key, :team_id
+    attr_accessor :api_key, :team_id, :workspace_slug
+    attr_writer :logger
 
     def configure
       yield self
     end
 
+    def logger
+      @logger ||= if defined?(Rails) && Rails.respond_to?(:logger) && Rails.logger
+                     Rails.logger
+                   else
+                     Logger.new($stdout, level: Logger::WARN)
+                   end
+    end
+
     def client
-      raise AuthenticationError, 'API key not configured' unless api_key
+      @client_mutex ||= Mutex.new
+      @client_mutex.synchronize do
+        raise AuthenticationError, 'API key not configured' unless api_key
 
-      @client ||= Client.new(api_key: api_key, team_id: team_id)
+        @client ||= Client.new(api_key: api_key, team_id: team_id)
+      end
     end
 
     def reset_client!
-      @client = nil
+      @client_mutex ||= Mutex.new
+      @client_mutex.synchronize do
+        @client = nil
+      end
+    end
+
+    # Quick health check against the Linear API
+    #
+    # @return [Boolean] true if the API is reachable and authenticated
+    def healthy?
+      result = client.query('query { viewer { id } }')
+      result.success?
+    rescue StandardError
+      false
     end
 
     # Rails calls eager_load! on all registered eager_load_namespaces.

data/lib/tasks/linear_api.rake

@@ -24,7 +24,7 @@ namespace :linear do
 
   desc 'Sync all metadata from Linear API (labels, projects, states, users)'
   task sync_cache: :environment do
-    puts 'Syncing Linear metadata from API...'
+    puts 'Syncing Linear metadata from API (single batched request)...'
 
     results = LinearApi::CacheSync.sync_all
 
@@ -70,12 +70,17 @@ namespace :linear do
   task export_cache: :environment do
     output_path = ENV['LINEAR_EXPORT_PATH'] || Rails.root.join('linear_cache.json')
 
+    # Fetch team info for export (avoids hardcoded values)
+    team_data = { id: LinearApi.team_id }
+    team_result = LinearApi.client.get_team
+    if team_result.success?
+      team_data[:key] = team_result.data.key
+      team_data[:name] = team_result.data.name
+    end
+
     data = {
       last_updated: Time.current.iso8601,
-      team: {
-        id: LinearApi.team_id,
-        key: 'TOS'
-      },
+      team: team_data,
       workflow_states: export_states,
       labels: export_labels,
       projects: export_projects,
@@ -86,10 +91,10 @@ namespace :linear do
     puts "Exported cache to #{output_path}"
   end
 
-  desc 'Refresh stale synced issues from Linear'
+  desc 'Refresh stale synced issues from Linear (batched)'
   task refresh_issues: :environment do
     stale_count = LinearApi::SyncedIssue.stale.count
-    puts "Refreshing #{stale_count} stale issues..."
+    puts "Refreshing #{stale_count} stale issues (batched)..."
 
     LinearApi::SyncedIssue.refresh_stale_issues!
 
@@ -119,8 +124,20 @@ namespace :linear do
     end
   end
 
-  # Helper methods for export
-  def export_states
+  desc 'Health check: verify Linear API connectivity'
+  task health: :environment do
+    puts 'Checking Linear API connectivity...'
+
+    if LinearApi.healthy?
+      puts '  ✓ Linear API is reachable and authenticated'
+    else
+      puts '  ✗ Cannot reach Linear API (check LINEAR_API_KEY)'
+      exit 1
+    end
+  end
+
+  # Helper methods for export (defined as module methods to avoid polluting global scope)
+  def self.export_states
     LinearApi::CachedMetadata.states.each_with_object({}) do |state, hash|
       hash[state.key] = {
         id: state.linear_id,
@@ -130,7 +147,7 @@ namespace :linear do
     end
   end
 
-  def export_labels
+  def self.export_labels
     LinearApi::CachedMetadata.labels.group_by(&:category).transform_values do |labels|
       labels.each_with_object({}) do |label, hash|
         key = label.key.split(':').last
@@ -143,7 +160,7 @@ namespace :linear do
     end
   end
 
-  def export_projects
+  def self.export_projects
     LinearApi::CachedMetadata.projects.each_with_object({}) do |project, hash|
       hash[project.key] = {
         id: project.linear_id,
@@ -153,7 +170,7 @@ namespace :linear do
     end
   end
 
-  def export_users
+  def self.export_users
     LinearApi::CachedMetadata.users.each_with_object({}) do |user, hash|
       hash[user.key] = {
         id: user.linear_id,

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: linear_api
 version: !ruby/object:Gem::Version
-  version: 0.3.2
+  version: 0.5.0
 platform: ruby
 authors:
 - TheOwnerStack
@@ -29,6 +29,26 @@ dependencies:
     - - "<"
       - !ruby/object:Gem::Version
         version: '3.0'
+- !ruby/object:Gem::Dependency
+  name: faraday-retry
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '1.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '3.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '1.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '3.0'
 - !ruby/object:Gem::Dependency
   name: rails
   requirement: !ruby/object:Gem::Requirement
@@ -36,7 +56,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '7.0'
-  type: :runtime
+  type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements: