linear_api 0.3.2 → 0.5.0

data/lib/linear_api/issue.rb

@@ -3,6 +3,10 @@
 module LinearApi
   # Represents a Linear issue
   class Issue
+    # ==========================================================================
+    # GraphQL Mutations
+    # ==========================================================================
+
     CREATE_MUTATION = <<~GRAPHQL
       mutation CreateIssue($input: IssueCreateInput!) {
         issueCreate(input: $input) {
@@ -12,6 +16,9 @@ module LinearApi
             identifier
             title
             url
+            priority
+            dueDate
+            estimate
             state {
               id
               name
@@ -29,6 +36,9 @@ module LinearApi
             id
             identifier
             title
+            priority
+            dueDate
+            estimate
             state {
               id
               name
@@ -44,9 +54,55 @@ module LinearApi
       }
     GRAPHQL
 
-    SEARCH_QUERY = <<~GRAPHQL
-      query GetIssue($term: String!) {
-        searchIssues(term: $term, first: 1) {
+    ADD_COMMENT_MUTATION = <<~GRAPHQL
+      mutation AddComment($issueId: String!, $body: String!) {
+        commentCreate(input: { issueId: $issueId, body: $body }) {
+          success
+          comment {
+            id
+            body
+            createdAt
+          }
+        }
+      }
+    GRAPHQL
+
+    ARCHIVE_MUTATION = <<~GRAPHQL
+      mutation ArchiveIssue($id: String!) {
+        issueArchive(id: $id) {
+          success
+        }
+      }
+    GRAPHQL
+
+    UNARCHIVE_MUTATION = <<~GRAPHQL
+      mutation UnarchiveIssue($id: String!) {
+        issueUnarchive(id: $id) {
+          success
+        }
+      }
+    GRAPHQL
+
+    LINK_PR_MUTATION = <<~GRAPHQL
+      mutation LinkPR($issueId: String!, $url: String!) {
+        attachmentLinkURL(issueId: $issueId, url: $url) {
+          success
+          attachment {
+            id
+            url
+          }
+        }
+      }
+    GRAPHQL
+
+    # ==========================================================================
+    # GraphQL Queries
+    # ==========================================================================
+
+    # Direct lookup by identifier (exact match, replaces search-based lookup)
+    GET_BY_IDENTIFIER_QUERY = <<~GRAPHQL
+      query GetIssueByIdentifier($filter: IssueFilter!) {
+        issues(filter: $filter, first: 1) {
           nodes {
             id
             identifier
@@ -54,6 +110,8 @@ module LinearApi
             description
             url
             priority
+            dueDate
+            estimate
             state {
               id
               name
@@ -88,6 +146,43 @@ module LinearApi
       }
     GRAPHQL
 
+    # Full-text search (use search_issues for fuzzy matching)
+    SEARCH_QUERY = <<~GRAPHQL
+      query SearchIssues($term: String!, $first: Int!) {
+        searchIssues(term: $term, first: $first) {
+          nodes {
+            id
+            identifier
+            title
+            description
+            url
+            priority
+            dueDate
+            estimate
+            state {
+              id
+              name
+              type
+            }
+            labels {
+              nodes {
+                id
+                name
+              }
+            }
+            assignee {
+              id
+              name
+            }
+            project {
+              id
+              name
+            }
+          }
+        }
+      }
+    GRAPHQL
+
     LIST_QUERY = <<~GRAPHQL
       query ListIssues($teamId: String!, $first: Int!) {
         team(id: $teamId) {
@@ -97,6 +192,8 @@ module LinearApi
               identifier
               title
               priority
+              dueDate
+              estimate
               state {
                 id
                 name
@@ -114,21 +211,128 @@ module LinearApi
       }
     GRAPHQL
 
-    ADD_COMMENT_MUTATION = <<~GRAPHQL
-      mutation AddComment($issueId: String!, $body: String!) {
-        commentCreate(input: { issueId: $issueId, body: $body }) {
-          success
-          comment {
+    LIST_PAGINATED_QUERY = <<~GRAPHQL
+      query ListIssuesPaginated($teamId: String!, $first: Int!, $after: String) {
+        team(id: $teamId) {
+          issues(first: $first, after: $after, orderBy: updatedAt) {
+            nodes {
+              id
+              identifier
+              title
+              priority
+              dueDate
+              estimate
+              state {
+                id
+                name
+                type
+              }
+              labels {
+                nodes {
+                  id
+                  name
+                }
+              }
+            }
+            pageInfo {
+              hasNextPage
+              endCursor
+            }
+          }
+        }
+      }
+    GRAPHQL
+
+    GET_BY_ID_QUERY = <<~GRAPHQL
+      query GetIssueById($id: String!) {
+        issue(id: $id) {
+          id
+          identifier
+          title
+          description
+          url
+          priority
+          dueDate
+          estimate
+          state {
             id
-            body
-            createdAt
+            name
+            type
+          }
+          labels {
+            nodes {
+              id
+              name
+            }
+          }
+          assignee {
+            id
+            name
+          }
+          project {
+            id
+            name
+          }
+        }
+      }
+    GRAPHQL
+
+    BATCH_GET_QUERY = <<~GRAPHQL
+      query BatchGetIssues($filter: IssueFilter!) {
+        issues(filter: $filter) {
+          nodes {
+            id
+            identifier
+            title
+            priority
+            dueDate
+            estimate
+            state {
+              id
+              name
+              type
+            }
+            labels {
+              nodes {
+                id
+                name
+              }
+            }
+            assignee {
+              id
+              name
+            }
+          }
+        }
+      }
+    GRAPHQL
+
+    SUB_ISSUES_QUERY = <<~GRAPHQL
+      query GetSubIssues($id: String!) {
+        issue(id: $id) {
+          children {
+            nodes {
+              id
+              identifier
+              title
+              state {
+                id
+                name
+              }
+              priority
+            }
           }
         }
       }
     GRAPHQL
 
+    # ==========================================================================
+    # Model
+    # ==========================================================================
+
     attr_reader :id, :identifier, :title, :description, :url, :priority,
-                :state, :labels, :assignee, :project, :comments, :raw
+                :state, :labels, :assignee, :project, :comments,
+                :due_date, :estimate, :raw
 
     def initialize(data)
       @raw = data
@@ -138,6 +342,8 @@ module LinearApi
       @description = data['description']
       @url = data['url']
       @priority = data['priority']
+      @due_date = data['dueDate']
+      @estimate = data['estimate']
       @state = data['state']
       @labels = (data.dig('labels', 'nodes') || [])
       @assignee = data['assignee']
@@ -161,6 +367,8 @@ module LinearApi
         description: description,
         url: url,
         priority: priority,
+        due_date: due_date,
+        estimate: estimate,
         state: state_name,
         labels: label_names
       }

data/lib/linear_api/issue_tracker.rb

@@ -23,7 +23,7 @@ module LinearApi
         existing = SyncedIssue.find_open_for_fingerprint(fingerprint)
         return add_occurrence_comment(existing, exception, context) if existing
 
-        # Ensure auto-tracked project and label exist
+        # Ensure auto-tracked project and label exist (uses in-memory cache after first call)
         project_id = ensure_auto_tracked_project
         auto_label_id = ensure_auto_tracked_label
 
@@ -57,6 +57,10 @@ module LinearApi
         )
 
         Result.new(success: true, data: synced_issue)
+      rescue StandardError => e
+        # Never let error tracking failures propagate - log and return failure
+        LinearApi.logger.error { "LinearApi::IssueTracker: failed to track error: #{e.class} - #{e.message}" }
+        Result.new(success: false, error: "Error tracking failed: #{e.message}")
       end
 
       # Track a custom issue (not from exception)
@@ -98,10 +102,17 @@ module LinearApi
         )
 
         Result.new(success: true, data: synced_issue)
+      rescue StandardError => e
+        LinearApi.logger.error { "LinearApi::IssueTracker: failed to track issue: #{e.class} - #{e.message}" }
+        Result.new(success: false, error: "Issue tracking failed: #{e.message}")
       end
 
       # Setup auto-tracking infrastructure (project + label)
       def setup!
+        # Clear in-memory cache to force fresh lookup
+        @auto_tracked_project_id = nil
+        @auto_tracked_label_id = nil
+
         {
           project_id: ensure_auto_tracked_project,
           label_id: ensure_auto_tracked_label
@@ -127,11 +138,16 @@ module LinearApi
         labels.filter_map { |label| CachedMetadata.resolve_label_id(label) }
       end
 
-      # Ensure the "Auto-Tracked Errors" project exists
+      # Ensure the "Auto-Tracked Errors" project exists (with in-memory memoization)
       def ensure_auto_tracked_project
-        # Check cache first
+        return @auto_tracked_project_id if @auto_tracked_project_id
+
+        # Check DB cache first
         cached = CachedMetadata.projects.find_by(key: AUTO_TRACKED_PROJECT_KEY)
-        return cached.linear_id if cached
+        if cached
+          @auto_tracked_project_id = cached.linear_id
+          return @auto_tracked_project_id
+        end
 
         # Check if project exists in Linear
         result = LinearApi.client.list_projects
@@ -139,7 +155,8 @@ module LinearApi
           existing = result.data.find { |p| p.name == AUTO_TRACKED_PROJECT_NAME }
           if existing
             cache_project(existing)
-            return existing.id
+            @auto_tracked_project_id = existing.id
+            return @auto_tracked_project_id
           end
         end
 
@@ -153,14 +170,19 @@ module LinearApi
         return nil unless create_result.success?
 
         cache_project(create_result.data)
-        create_result.data.id
+        @auto_tracked_project_id = create_result.data.id
       end
 
-      # Ensure the "auto:tracked" label exists
+      # Ensure the "auto:tracked" label exists (with in-memory memoization)
       def ensure_auto_tracked_label
-        # Check cache first
+        return @auto_tracked_label_id if @auto_tracked_label_id
+
+        # Check DB cache first
         cached = CachedMetadata.labels.find_by(key: AUTO_TRACKED_LABEL_NAME)
-        return cached.linear_id if cached
+        if cached
+          @auto_tracked_label_id = cached.linear_id
+          return @auto_tracked_label_id
+        end
 
         # Check if label exists in Linear
         result = LinearApi.client.list_labels
@@ -168,7 +190,8 @@ module LinearApi
           existing = result.data.find { |l| l.name == AUTO_TRACKED_LABEL_NAME }
           if existing
             cache_label(existing)
-            return existing.id
+            @auto_tracked_label_id = existing.id
+            return @auto_tracked_label_id
           end
         end
 
@@ -182,7 +205,7 @@ module LinearApi
         return nil unless create_result.success?
 
         cache_label(create_result.data)
-        create_result.data.id
+        @auto_tracked_label_id = create_result.data.id
       end
 
       def cache_project(project)
@@ -253,16 +276,27 @@ module LinearApi
       end
 
       def sanitize_context(context)
-        context.transform_values do |value|
-          case value
-          when Time, DateTime then value.iso8601
-          when Date then value.to_s
-          when ActiveRecord::Base then "#{value.class.name}##{value.id}"
-          else value
+        context.each_with_object({}) do |(key, value), hash|
+          key_str = key.to_s
+          if LinearApi::SENSITIVE_KEY_PATTERN.match?(key_str)
+            hash[key_str] = '[REDACTED]'
+          else
+            hash[key_str] = sanitize_value(value)
           end
         end
       end
 
+      def sanitize_value(value)
+        case value
+        when Time, DateTime then value.iso8601
+        when Date then value.to_s
+        when ActiveRecord::Base then "#{value.class.name}##{value.id}"
+        when Hash then sanitize_context(value)
+        when Array then value.map { |v| sanitize_value(v) }
+        else value
+        end
+      end
+
       def add_occurrence_comment(synced_issue, exception, context)
         comment_body = <<~MARKDOWN
           ## New Occurrence

data/lib/linear_api/label.rb

@@ -18,6 +18,20 @@ module LinearApi
       }
     GRAPHQL
 
+    CREATE_MUTATION = <<~GRAPHQL
+      mutation CreateLabel($input: IssueLabelCreateInput!) {
+        issueLabelCreate(input: $input) {
+          success
+          issueLabel {
+            id
+            name
+            color
+            description
+          }
+        }
+      }
+    GRAPHQL
+
     attr_reader :id, :name, :color, :description, :raw
 
     def initialize(data)

data/lib/linear_api/project.rb

@@ -13,13 +13,31 @@ module LinearApi
               description
               state
               progress
+              url
             }
           }
         }
       }
     GRAPHQL
 
-    attr_reader :id, :name, :description, :state, :progress, :raw
+    CREATE_MUTATION = <<~GRAPHQL
+      mutation CreateProject($input: ProjectCreateInput!) {
+        projectCreate(input: $input) {
+          success
+          project {
+            id
+            name
+            description
+            url
+            color
+            state
+            progress
+          }
+        }
+      }
+    GRAPHQL
+
+    attr_reader :id, :name, :description, :state, :progress, :url, :color, :raw
 
     def initialize(data)
       @raw = data
@@ -28,6 +46,8 @@ module LinearApi
       @description = data['description']
       @state = data['state']
       @progress = data['progress']
+      @url = data['url']
+      @color = data['color']
     end
 
     def to_h
@@ -36,7 +56,9 @@ module LinearApi
         name: name,
         description: description,
         state: state,
-        progress: progress
+        progress: progress,
+        url: url,
+        color: color
       }
     end
   end

data/lib/linear_api/result.rb

@@ -2,6 +2,21 @@
 
 module LinearApi
   # Result object for API operations
+  #
+  # Always check success?/failure? before accessing data/error:
+  #
+  #   result = client.create_issue(title: 'Bug')
+  #   if result.success?
+  #     puts result.data.identifier
+  #   else
+  #     puts result.error
+  #   end
+  #
+  # Or use value! to raise on failure:
+  #
+  #   issue = client.create_issue(title: 'Bug').value!
+  #   puts issue.identifier
+  #
   class Result
     attr_reader :data, :error, :raw_response
 
@@ -20,17 +35,14 @@ module LinearApi
       !@success
     end
 
-    # Allow accessing data directly on result
-    def method_missing(method, *args, &block)
-      if data.respond_to?(method)
-        data.public_send(method, *args, &block)
-      else
-        super
-      end
-    end
+    # Returns data on success, raises LinearApi::Error on failure
+    #
+    # @return [Object] the result data
+    # @raise [LinearApi::Error] if the result is a failure
+    def value!
+      raise LinearApi::Error, error if failure?
 
-    def respond_to_missing?(method, include_private = false)
-      data.respond_to?(method, include_private) || super
+      data
     end
   end
 end

data/lib/linear_api/team.rb

@@ -23,6 +23,66 @@ module LinearApi
               name
               type
               position
+              color
+            }
+          }
+        }
+      }
+    GRAPHQL
+
+    USERS_QUERY = <<~GRAPHQL
+      query ListUsers($teamId: String!) {
+        team(id: $teamId) {
+          members {
+            nodes {
+              id
+              name
+              email
+              displayName
+              active
+            }
+          }
+        }
+      }
+    GRAPHQL
+
+    # Fetch all team metadata in a single query (for CacheSync)
+    ALL_METADATA_QUERY = <<~GRAPHQL
+      query FetchAllMetadata($teamId: String!) {
+        team(id: $teamId) {
+          labels {
+            nodes {
+              id
+              name
+              color
+              description
+            }
+          }
+          projects {
+            nodes {
+              id
+              name
+              description
+              state
+              progress
+            }
+          }
+          states {
+            nodes {
+              id
+              name
+              type
+              position
+              color
+            }
+          }
+          members {
+            nodes {
+              id
+              name
+              email
+              displayName
+              active
             }
           }
         }

data/lib/linear_api/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module LinearApi
-  VERSION = '0.3.2'
+  VERSION = '0.5.0'
 end