activeproject 0.1.0 → 0.1.1

data/lib/active_project/adapters/trello_adapter.rb

@@ -13,8 +13,6 @@ module ActiveProject
     # Implements the interface defined in ActiveProject::Adapters::Base.
     # API Docs: https://developer.atlassian.com/cloud/trello/rest/
     class TrelloAdapter < Base
-      BASE_URL = "https://api.trello.com/1/"
-
       # Computes the expected Trello webhook signature.
       # @param callback_url [String] The exact URL registered for the webhook.
       # @param response_body [String] The raw response body received from Trello.
@@ -28,22 +26,12 @@ module ActiveProject
 
       attr_reader :config
 
-      # Initializes the Trello Adapter.
-      # @param config [Configurations::TrelloConfiguration] The configuration object for Trello.
-      # @raise [ArgumentError] if required configuration options (:api_key, :api_token) are missing.
-      def initialize(config:)
-        unless config.is_a?(ActiveProject::Configurations::TrelloConfiguration)
-          raise ArgumentError, "TrelloAdapter requires a TrelloConfiguration object"
-        end
-        @config = config
-
-        unless @config.api_key && !@config.api_key.empty? && @config.api_token && !@config.api_token.empty?
-          raise ArgumentError, "TrelloAdapter configuration requires :api_key and :api_token"
-        end
-
-        @connection = initialize_connection
-      end
-
+      include Trello::Connection
+      include Trello::Projects
+      include Trello::Issues
+      include Trello::Comments
+      include Trello::Lists
+      include Trello::Webhooks
 
       # --- Resource Factories ---
 
@@ -59,318 +47,6 @@ module ActiveProject
         ResourceFactory.new(adapter: self, resource_class: Resources::Issue)
       end
 
-      # --- Implementation of Base methods ---
-
-      # Lists Trello boards accessible by the configured token.
-      # @return [Array<ActiveProject::Resources::Project>] An array of project resources.
-      def list_projects
-        path = "members/me/boards"
-        query = { fields: "id,name,desc" }
-        boards_data = make_request(:get, path, nil, query)
-
-        return [] unless boards_data.is_a?(Array)
-
-        boards_data.map do |board_data|
-          Resources::Project.new(self, # Pass adapter instance
-            id: board_data["id"],
-            key: nil,
-            name: board_data["name"],
-            adapter_source: :trello,
-            raw_data: board_data
-          )
-        end
-      end
-
-      # Finds a specific Trello Board by its ID.
-      # @param board_id [String] The ID of the Trello Board.
-      # @return [ActiveProject::Resources::Project] The project resource.
-      def find_project(board_id)
-         path = "boards/#{board_id}"
-         query = { fields: "id,name,desc" }
-         board_data = make_request(:get, path, nil, query)
-
-         Resources::Project.new(self, # Pass adapter instance
-           id: board_data["id"],
-           key: nil,
-           name: board_data["name"],
-           adapter_source: :trello,
-           raw_data: board_data
-         )
-      end
-
-      # Creates a new board in Trello.
-      # @param attributes [Hash] Board attributes. Required: :name. Optional: :description, :default_lists.
-      # @return [ActiveProject::Resources::Project] The created project resource.
-      def create_project(attributes)
-        unless attributes[:name] && !attributes[:name].empty?
-          raise ArgumentError, "Missing required attribute for Trello board creation: :name"
-        end
-
-        path = "boards/"
-        query_params = {
-          name: attributes[:name],
-          desc: attributes[:description],
-          defaultLists: attributes.fetch(:default_lists, true) # Default to creating lists
-          # Add other board options here if needed (e.g., idOrganization)
-        }.compact
-
-        board_data = make_request(:post, path, nil, query_params)
-
-        Resources::Project.new(self, # Pass adapter instance
-          id: board_data["id"],
-          key: nil,
-          name: board_data["name"],
-          adapter_source: :trello,
-          raw_data: board_data
-        )
-      end
-
-      # Creates a new list on a Trello board.
-      # @param board_id [String] The ID of the board.
-      # @param attributes [Hash] List attributes. Required: :name. Optional: :pos.
-      # @return [Hash] The raw data hash of the created list.
-      def create_list(board_id, attributes)
-        unless attributes[:name] && !attributes[:name].empty?
-          raise ArgumentError, "Missing required attribute for Trello list creation: :name"
-        end
-
-        path = "boards/#{board_id}/lists"
-        query_params = {
-          name: attributes[:name],
-          pos: attributes[:pos]
-        }.compact
-
-        make_request(:post, path, nil, query_params)
-      end
-
-      # Deletes a board in Trello.
-      # WARNING: This is a permanent deletion.
-      # @param board_id [String] The ID of the board to delete.
-      # @return [Boolean] true if deletion was successful (API returns 200).
-      # @raise [NotFoundError] if the board is not found.
-      # @raise [AuthenticationError] if credentials lack permission.
-      # @raise [ApiError] for other errors.
-      def delete_project(board_id)
-        path = "/boards/#{board_id}"
-        make_request(:delete, path) # DELETE returns 200 OK on success
-        true # Return true if make_request doesn't raise an error
-      end
-
-
-
-
-      # Lists Trello cards on a specific board.
-      # @param board_id [String] The ID of the Trello board.
-      # @param options [Hash] Optional filtering options.
-      # @return [Array<ActiveProject::Resources::Issue>] An array of issue resources.
-      def list_issues(board_id, options = {})
-        path = "boards/#{board_id}/cards"
-        # Fetch idMembers and list name for potential name mapping fallback
-        query = { fields: "id,name,desc,closed,idList,idBoard,due,dueComplete,idMembers", list: true }
-        query[:filter] = options[:filter] if options[:filter]
-
-        cards_data = make_request(:get, path, nil, query)
-        return [] unless cards_data.is_a?(Array)
-
-        cards_data.map { |card_data| map_card_data(card_data, board_id) }
-      end
-
-      # Finds a specific Card by its ID.
-      # @param card_id [String] The ID of the Trello Card.
-      # @param context [Hash] Optional context (ignored).
-      # @return [ActiveProject::Resources::Issue] The issue resource.
-      def find_issue(card_id, context = {})
-        path = "cards/#{card_id}"
-        # Fetch idMembers and list name for potential name mapping fallback
-        query = { fields: "id,name,desc,closed,idList,idBoard,due,dueComplete,idMembers", list: true }
-        card_data = make_request(:get, path, nil, query)
-        map_card_data(card_data, card_data["idBoard"])
-      end
-
-      # Creates a new Card in Trello.
-      # @param _board_id [String] Ignored (context).
-      # @param attributes [Hash] Card attributes. Required: :list_id, :title. Optional: :description, :assignee_ids, :due_on.
-      # @return [ActiveProject::Resources::Issue] The created issue resource.
-      def create_issue(_board_id, attributes)
-        list_id = attributes[:list_id]
-        title = attributes[:title]
-
-        unless list_id && title && !title.empty?
-          raise ArgumentError, "Missing required attributes for Trello card creation: :list_id, :title"
-        end
-
-        path = "cards"
-        query_params = {
-          idList: list_id,
-          name: title,
-          desc: attributes[:description],
-          # Use assignee_ids (expects an array of Trello member IDs)
-          idMembers: attributes[:assignee_ids]&.join(","),
-          due: attributes[:due_on]&.iso8601
-        }.compact
-
-        card_data = make_request(:post, path, nil, query_params)
-        map_card_data(card_data, card_data["idBoard"])
-      end
-
-      # Updates an existing Card in Trello.
-      # @param card_id [String] The ID of the Trello Card.
-      # @param attributes [Hash] Attributes to update (e.g., :title, :description, :list_id, :closed, :due_on, :assignee_ids, :status).
-      # @param context [Hash] Optional context (ignored).
-      # @return [ActiveProject::Resources::Issue] The updated issue resource.
-      def update_issue(card_id, attributes, context = {})
-        # Make a mutable copy of attributes
-        update_attributes = attributes.dup
-
-        # Handle :status mapping to :list_id
-        if update_attributes.key?(:status)
-          target_status = update_attributes.delete(:status) # Remove status key
-
-          # Fetch board_id efficiently if not already known
-          # We need the board_id to look up the correct status mapping
-          board_id = update_attributes[:board_id] || begin
-            find_issue(card_id).project_id # Fetch the issue to get its board_id
-          rescue NotFoundError
-            # Re-raise NotFoundError if the card itself doesn't exist
-            raise NotFoundError, "Trello card with ID '#{card_id}' not found."
-          end
-
-          unless board_id
-            # This should theoretically not happen if find_issue succeeded or board_id was passed
-            raise ApiError, "Could not determine board ID for card '#{card_id}' to perform status mapping."
-          end
-
-          # Use stored config for status mappings
-          board_mappings = @config.status_mappings[board_id]
-          unless board_mappings
-            raise ConfigurationError, "Trello status mapping not configured for board ID '#{board_id}'. Cannot map status ':#{target_status}'."
-          end
-
-          # Find the target list ID by looking up the status symbol in the board's mappings.
-          # We iterate through the mappings hash { list_id => status_symbol }
-          target_list_id = board_mappings.key(target_status)
-
-          unless target_list_id
-            raise ConfigurationError, "Target status ':#{target_status}' not found in configured Trello status mappings for board ID '#{board_id}'."
-          end
-
-          # Add the resolved list_id to the attributes to be updated
-          update_attributes[:list_id] = target_list_id
-        end
-
-
-        path = "cards/#{card_id}"
-
-        # Build query parameters from the potentially modified update_attributes
-        query_params = {}
-        query_params[:name] = update_attributes[:title] if update_attributes.key?(:title)
-        query_params[:desc] = update_attributes[:description] if update_attributes.key?(:description)
-        query_params[:closed] = update_attributes[:closed] if update_attributes.key?(:closed)
-        query_params[:idList] = update_attributes[:list_id] if update_attributes.key?(:list_id) # Use the mapped list_id if status was provided
-        query_params[:due] = update_attributes[:due_on]&.iso8601 if update_attributes.key?(:due_on)
-        query_params[:dueComplete] = update_attributes[:dueComplete] if update_attributes.key?(:dueComplete)
-        # Use assignee_ids (expects an array of Trello member IDs)
-        query_params[:idMembers] = update_attributes[:assignee_ids]&.join(",") if update_attributes.key?(:assignee_ids)
-
-        # If after processing :status, there are no actual changes, just return the current issue state
-        return find_issue(card_id, context) if query_params.empty?
-
-        # Make the PUT request to update the card
-        card_data = make_request(:put, path, nil, query_params.compact)
-
-        # Return the updated issue resource, mapped with potentially new status
-        map_card_data(card_data, card_data["idBoard"])
-      end
-
-      # Adds a comment to a Card in Trello.
-      # @param card_id [String] The ID of the Trello Card.
-      # @param comment_body [String] The comment text (Markdown).
-      # @param context [Hash] Optional context (ignored).
-      # @return [ActiveProject::Resources::Comment] The created comment resource.
-      def add_comment(card_id, comment_body, context = {})
-        path = "cards/#{card_id}/actions/comments"
-        query_params = { text: comment_body }
-        comment_data = make_request(:post, path, nil, query_params)
-        map_comment_action_data(comment_data, card_id)
-      end
-
-      # Parses an incoming Trello webhook payload.
-      # @param request_body [String] The raw JSON request body.
-      # @param headers [Hash] Request headers (unused).
-      # @return [ActiveProject::WebhookEvent, nil] Parsed event or nil if unhandled.
-      def parse_webhook(request_body, headers = {})
-        payload = JSON.parse(request_body) rescue nil
-        return nil unless payload.is_a?(Hash) && payload["action"].is_a?(Hash)
-
-        action = payload["action"]
-        action_type = action["type"]
-        actor_data = action.dig("memberCreator")
-        timestamp = Time.parse(action["date"]) rescue nil
-        board_id = action.dig("data", "board", "id")
-        card_data = action.dig("data", "card")
-        comment_text = action.dig("data", "text")
-        old_data = action.dig("data", "old") # For updateCard events
-
-        event_type = nil
-        object_kind = nil
-        event_object_id = nil
-        object_key = nil
-        changes = nil
-        object_data = nil
-
-        case action_type
-        when "createCard"
-          event_type = :issue_created
-          object_kind = :issue
-          event_object_id = card_data["id"]
-          object_key = card_data["idShort"]
-        when "updateCard"
-          event_type = :issue_updated
-          object_kind = :issue
-          event_object_id = card_data["id"]
-          object_key = card_data["idShort"]
-          # Parse changes for updateCard
-          if old_data.is_a?(Hash)
-            changes = {}
-            old_data.each do |field, old_value|
-              # Find the corresponding new value in the card data if possible
-              new_value = card_data[field]
-              changes[field.to_sym] = [ old_value, new_value ]
-            end
-          end
-        when "commentCard"
-          event_type = :comment_added
-          object_kind = :comment
-          event_object_id = action["id"] # Action ID is comment ID
-          object_key = nil
-        when "addMemberToCard", "removeMemberFromCard"
-          event_type = :issue_updated
-          object_kind = :issue
-          event_object_id = card_data["id"]
-          object_key = card_data["idShort"]
-          changes = { assignees: true } # Indicate assignees changed, specific diff not easily available
-        else
-          return nil # Unhandled action type
-        end
-
-        WebhookEvent.new(
-          event_type: event_type,
-          object_kind: object_kind,
-          event_object_id: event_object_id,
-          object_key: object_key,
-          project_id: board_id,
-          actor: map_user_data(actor_data), # Use helper
-          timestamp: timestamp,
-          adapter_source: :trello,
-          changes: changes,
-          object_data: object_data, # Keep nil for now
-          raw_data: payload
-        )
-      rescue JSON::ParserError
-        nil # Ignore unparseable payloads
-      end
-
-
       # Retrieves details for the currently authenticated user.
       # @return [ActiveProject::Resources::User] The user object.
       # @raise [ActiveProject::AuthenticationError] if authentication fails.
@@ -390,18 +66,9 @@ module ActiveProject
         false
       end
 
-
       private
 
       # Initializes the Faraday connection object.
-      def initialize_connection
-        Faraday.new(url: BASE_URL) do |conn|
-          conn.request :retry
-          conn.headers["Accept"] = "application/json"
-          conn.response :raise_error
-          conn.headers["User-Agent"] = ActiveProject.user_agent
-        end
-      end
 
       # Helper method for making requests.
       def make_request(method, path, body = nil, query_params = {})
@@ -417,6 +84,7 @@ module ActiveProject
         end
 
         return nil if response.status == 204 || response.body.empty?
+
         JSON.parse(response.body)
       rescue Faraday::Error => e
         handle_faraday_error(e)
@@ -440,11 +108,14 @@ module ActiveProject
         when 400, 422
           if status == 400 && message&.include?("invalid id")
             raise NotFoundError, "Trello resource not found (Status: 400, Message: #{message})"
-          else
-            raise ValidationError.new("Trello validation failed (Status: #{status}): #{message}", status_code: status, response_body: body)
           end
+
+          raise ValidationError.new("Trello validation failed (Status: #{status}): #{message}", status_code: status,
+                                                                                                response_body: body)
+
         else
-          raise ApiError.new("Trello API error (Status: #{status || 'N/A'}): #{message}", original_error: error, status_code: status, response_body: body)
+          raise ApiError.new("Trello API error (Status: #{status || 'N/A'}): #{message}", original_error: error,
+                                                                                          status_code: status, response_body: body)
         end
       end
 
@@ -461,48 +132,54 @@ module ActiveProject
           status = board_mappings[list_id]
         # Fallback: Try mapping by List Name if ID mapping failed and list data is present
         elsif board_mappings && card_data["list"] && board_mappings.key?(card_data["list"]["name"])
-           status = board_mappings[card_data["list"]["name"]]
+          status = board_mappings[card_data["list"]["name"]]
         end
 
         # Override status if the card is archived (closed)
         status = :closed if card_data["closed"]
 
-        created_timestamp = Time.at(card_data["id"][0..7].to_i(16)) rescue nil
-        due_on = card_data["due"] ? Date.parse(card_data["due"]) : nil rescue nil
+        created_timestamp = begin
+          Time.at(card_data["id"][0..7].to_i(16))
+        rescue StandardError
+          nil
+        end
+        due_on = begin
+          card_data["due"] ? Date.parse(card_data["due"]) : nil
+        rescue StandardError
+          nil
+        end
 
         Resources::Issue.new(self, # Pass adapter instance
-          id: card_data["id"],
-          key: nil,
-          title: card_data["name"],
-          description: card_data["desc"],
-          status: status, # Use the determined status
-          assignees: map_member_ids_to_users(card_data["idMembers"]), # Use new method
-          reporter: nil, # Trello cards don't have a distinct reporter
-          project_id: board_id,
-          created_at: created_timestamp,
-          updated_at: nil, # Trello API doesn't provide a standard updated_at for cards easily
-          due_on: due_on,
-          priority: nil, # Trello doesn't have priority
-          adapter_source: :trello,
-          raw_data: card_data
-        )
+                             id: card_data["id"],
+                             key: nil,
+                             title: card_data["name"],
+                             description: card_data["desc"],
+                             status: status, # Use the determined status
+                             assignees: map_member_ids_to_users(card_data["idMembers"]), # Use new method
+                             reporter: nil, # Trello cards don't have a distinct reporter
+                             project_id: board_id,
+                             created_at: created_timestamp,
+                             updated_at: nil, # Trello API doesn't provide a standard updated_at for cards easily
+                             due_on: due_on,
+                             priority: nil, # Trello doesn't have priority
+                             adapter_source: :trello,
+                             raw_data: card_data)
       end
 
       # Maps raw Trello comment action data hash to a Comment resource.
       def map_comment_action_data(action_data, card_id)
-        author_data = action_data.dig("memberCreator")
+        author_data = action_data["memberCreator"]
         comment_text = action_data.dig("data", "text")
 
         Resources::Comment.new(self, # Pass adapter instance
-          id: action_data["id"],
-          body: comment_text,
-          author: map_user_data(author_data), # Use user mapping
-          created_at: action_data["date"] ? Time.parse(action_data["date"]) : nil,
-          updated_at: nil,
-          issue_id: card_id,
-          adapter_source: :trello,
-          raw_data: action_data
-        )
+                               id: action_data["id"],
+                               body: comment_text,
+                               author: map_user_data(author_data), # Use user mapping
+                               created_at: action_data["date"] ? Time.parse(action_data["date"]) : nil,
+                               updated_at: nil,
+                               issue_id: card_id,
+                               adapter_source: :trello,
+                               raw_data: action_data)
       end
 
       # Maps an array of Trello member IDs to an array of User resources.
@@ -511,6 +188,7 @@ module ActiveProject
       # @return [Array<Resources::User>]
       def map_member_ids_to_users(member_ids)
         return [] unless member_ids.is_a?(Array)
+
         member_ids.map do |id|
           Resources::User.new(self, id: id, adapter_source: :trello, raw_data: { id: id })
         end
@@ -521,13 +199,13 @@ module ActiveProject
       # @return [Resources::User, nil]
       def map_user_data(member_data)
         return nil unless member_data && member_data["id"]
+
         Resources::User.new(self, # Pass adapter instance
-          id: member_data["id"],
-          name: member_data["fullName"], # Trello uses fullName
-          email: nil, # Trello API often doesn't provide email directly here
-          adapter_source: :trello,
-          raw_data: member_data
-        )
+                            id: member_data["id"],
+                            name: member_data["fullName"], # Trello uses fullName
+                            email: nil, # Trello API often doesn't provide email directly here
+                            adapter_source: :trello,
+                            raw_data: member_data)
       end
     end
   end

data/lib/active_project/association_proxy.rb

@@ -32,6 +32,7 @@ module ActiveProject
       # Ensure owner.id is accessed correctly
       owner_id = @owner.respond_to?(:id) ? @owner.id : nil
       raise "Owner object #{@owner.inspect} does not have an ID for association call." unless owner_id
+
       @adapter.send(list_method, owner_id, options)
     end
 
@@ -79,6 +80,8 @@ module ActiveProject
       @target_resource_class.new(@adapter, merged_attrs)
     end
 
+    alias new build
+
     # Creates and saves a new associated resource.
     # Example: project.issues.create(title: 'New', list_id: '...')
     # @param attributes [Hash] Attributes for the new resource.
@@ -86,12 +89,13 @@ module ActiveProject
     # @raise [NotImplementedError] Currently raises because #save is not fully implemented on resource.
     def create(attributes = {})
       create_method = determine_create_method
-      context = determine_context # Get owner context
+      determine_context # Get owner context
       # Pass owner ID/context first, then attributes
       owner_id = @owner.respond_to?(:id) ? @owner.id : nil
       raise "Owner object #{@owner.inspect} does not have an ID for association call." unless owner_id
+
       @adapter.send(create_method, owner_id, attributes)
-      # Note: This currently returns the result from the adapter directly.
+      # NOTE: This currently returns the result from the adapter directly.
       # A full implementation would likely build and then save, or re-fetch.
     end
 
@@ -113,6 +117,7 @@ module ActiveProject
       unless @adapter.respond_to?(method_name)
         raise NotImplementedError, "#{@adapter.class.name} does not implement ##{method_name}"
       end
+
       method_name
     end
 
@@ -125,6 +130,7 @@ module ActiveProject
       unless @adapter.respond_to?(method_name)
         raise NotImplementedError, "#{@adapter.class.name} does not implement ##{method_name}"
       end
+
       method_name
     end
 
@@ -136,6 +142,7 @@ module ActiveProject
       unless @adapter.respond_to?(method_name)
         raise NotImplementedError, "#{@adapter.class.name} does not implement ##{method_name}"
       end
+
       method_name
     end
   end

data/lib/active_project/configuration.rb

@@ -29,9 +29,7 @@ module ActiveProject
     # @param options [Hash] Configuration options for the adapter (e.g., site, api_key, token).
     # @yield [BaseAdapterConfiguration] Yields an adapter-specific configuration object if a block is given.
     def add_adapter(adapter_type, instance_name = :primary, options = {}, &block)
-      unless adapter_type.is_a?(Symbol)
-        raise ArgumentError, "Adapter type must be a Symbol (e.g., :basecamp)"
-      end
+      raise ArgumentError, "Adapter type must be a Symbol (e.g., :basecamp)" unless adapter_type.is_a?(Symbol)
 
       # Handle the case where instance_name is actually the options hash
       if instance_name.is_a?(Hash) && options.empty?

data/lib/active_project/configurations/trello_configuration.rb

@@ -20,9 +20,7 @@ module ActiveProject
 
       def freeze
         # Ensure nested hashes are also frozen
-        @status_mappings.each do |board_id, mappings|
-          mappings.freeze
-        end
+        @status_mappings.each_value(&:freeze)
         @status_mappings.freeze
         super
       end

data/lib/active_project/resource_factory.rb

@@ -23,13 +23,11 @@ module ActiveProject
       # Call adapter method with appropriate arguments
       if primary_arg
         @adapter.send(list_method, primary_arg, options)
-      else
+      elsif @adapter.method(list_method).arity.zero?
         # Handle case where list method might not take options (like list_projects)
-        if @adapter.method(list_method).arity == 0
-           @adapter.send(list_method)
-        else
-           @adapter.send(list_method, options)
-        end
+        @adapter.send(list_method)
+      else
+        @adapter.send(list_method, options)
       end
     end
 
@@ -89,7 +87,7 @@ module ActiveProject
     def create(attributes = {})
       # Determine the correct adapter create method based on resource type
       create_method = determine_create_method
-      # Note: Assumes create methods on adapters take attributes hash directly
+      # NOTE: Assumes create methods on adapters take attributes hash directly
       # Context like project_id needs to be part of the attributes hash if required by adapter
       @adapter.send(create_method, attributes)
       # A full implementation would likely involve build then save:
@@ -106,7 +104,11 @@ module ActiveProject
       when "ActiveProject::Resources::Issue" then :list_issues
       else raise "Cannot determine list method for #{@resource_class.name}"
       end
-      raise NotImplementedError, "#{@adapter.class.name} does not implement ##{method_name}" unless @adapter.respond_to?(method_name)
+      unless @adapter.respond_to?(method_name)
+        raise NotImplementedError,
+              "#{@adapter.class.name} does not implement ##{method_name}"
+      end
+
       method_name
     end
 
@@ -116,14 +118,22 @@ module ActiveProject
       when "ActiveProject::Resources::Issue" then :find_issue
       else raise "Cannot determine find method for #{@resource_class.name}"
       end
-      raise NotImplementedError, "#{@adapter.class.name} does not implement ##{method_name}" unless @adapter.respond_to?(method_name)
+      unless @adapter.respond_to?(method_name)
+        raise NotImplementedError,
+              "#{@adapter.class.name} does not implement ##{method_name}"
+      end
+
       method_name
     end
 
     def determine_create_method
       singular_name = @resource_class.name.split("::").last.downcase.to_sym
       method_name = :"create_#{singular_name}"
-      raise NotImplementedError, "#{@adapter.class.name} does not implement ##{method_name}" unless @adapter.respond_to?(method_name)
+      unless @adapter.respond_to?(method_name)
+        raise NotImplementedError,
+              "#{@adapter.class.name} does not implement ##{method_name}"
+      end
+
       method_name
     end
   end

data/lib/active_project/resources/issue.rb

@@ -7,8 +7,6 @@ module ActiveProject
       def_members :id, :key, :title, :description, :status, :assignees,
                   :reporter, :project_id, :created_at, :updated_at, :due_on,
                   :priority, :adapter_source
-      # raw_data and adapter are inherited from BaseResource
-
 
       # Saves the issue (creates if new, updates if existing).
       # Placeholder - Full implementation requires attribute tracking and adapter delegation.
@@ -28,7 +26,6 @@ module ActiveProject
         raise NotImplementedError, "#update not yet implemented for #{self.class.name}"
       end
 
-
       # Returns an association proxy for accessing comments on this issue.
       # @return [AssociationProxy<Resources::Comment>]
       def comments

data/lib/active_project/resources/project.rb

@@ -7,7 +7,6 @@ module ActiveProject
       def_members :id, :key, :name, :adapter_source
       # raw_data and adapter are inherited from BaseResource
 
-
       # Returns an association proxy for accessing issues within this project.
       # @return [AssociationProxy<Resources::Issue>]
       def issues