NO-ISSUE upd

Author: Yang-33

README.md

@@ -122,6 +122,7 @@ end
 
 > **Note:** You can still use the individual clients (e.g. `Line::Bot::V2::MessagingApi::ApiClient`) if you prefer.
 > For channel access token operations, use `Line::Bot::V2::ChannelAccessToken::ApiClient` directly.
+> For module attach operations, use `Line::Bot::V2::ModuleAttach::ApiClient` directly.
 
 ### Main classes
 You may use this classes to use LINE Messaging API features.

lib/line/bot/v2/client.rb

@@ -15,12 +15,13 @@
 module Line
   module Bot
     module V2
-      # A single client for all LINE Bot APIs, except channel access token management.
+      # A single client for all LINE Bot APIs, except channel access token and module attach management.
       #
       # It wraps the individual generated API clients and exposes their methods directly,
       # so callers can work with one object instead of juggling several client instances.
       #
       # For channel access token operations, use {Line::Bot::V2::ChannelAccessToken::ApiClient} directly.
+      # For module attach operations, use {Line::Bot::V2::ModuleAttach::ApiClient} directly.
       class Client
         # @param channel_access_token [String] Channel access token issued for your LINE Official Account.
         # @param api_base_url [String, nil] Base URL for Messaging API style endpoints. Defaults to https://api.line.me.

migration_from_v1_to_v2_guide.md

@@ -47,7 +47,7 @@ However, it's not recommended to always use it as it could lead to migration err
 ## Clients
 
 ### Unified Client (recommended)
-`Line::Bot::V2::Client` wraps all API clients into a single object. You only need one client for MessagingApi, Insight, Liff, ManageAudience, Module, and Shop.
+`Line::Bot::V2::Client` wraps supported API clients into a single object. ChannelAccessToken and ModuleAttach are not included; use their individual clients directly.
 
 ```rb
 client = Line::Bot::V2::Client.new(
@@ -71,6 +71,7 @@ You can also use individual clients directly. There are several types depending
   and so on.
 
 For channel access token operations, use `Line::Bot::V2::ChannelAccessToken::ApiClient` directly (not included in the unified client).
+For module attach operations, use `Line::Bot::V2::ModuleAttach::ApiClient` directly (not included in the unified client).
 
 
 ## Migration examples

scripts/unified-client-generator/main.rb

@@ -20,12 +20,13 @@ module UnifiedClientGenerator
   BUILTINS = Set['Array', 'File', 'Float', 'Hash', 'Integer', 'Object', 'String', 'Symbol'].freeze
 
   CLIENT_CLASS_COMMENT = <<~COMMENT
-    A single client for all LINE Bot APIs, except channel access token management.
+    A single client for all LINE Bot APIs, except channel access token and module attach management.
 
     It wraps the individual generated API clients and exposes their methods directly,
     so callers can work with one object instead of juggling several client instances.
 
     For channel access token operations, use {Line::Bot::V2::ChannelAccessToken::ApiClient} directly.
+    For module attach operations, use {Line::Bot::V2::ModuleAttach::ApiClient} directly.
   COMMENT
 
   CLIENT_INITIALIZE_COMMENT = <<~COMMENT
@@ -74,16 +75,30 @@ def run(root_dir = File.expand_path('../..', __dir__))
 
   # --- Validation ---
 
+  SUPPORTED_BASE_URLS = Set['https://api.line.me', 'https://api-data.line.me'].freeze
+
   def validate_clients!(clients)
     raise 'No client files were found.' if clients.empty?
 
     check_collisions!('Delegate name', clients.map(&:delegate_name))
     check_collisions!('Method name', clients.flat_map { |c| c.delegated_methods.map(&:name) })
 
     allowed = Set['base_url', 'channel_access_token', 'http_options']
+
     clients.each do |c|
-      extra = Set.new(c.constructor_required_keywords + c.constructor_optional_keywords) - allowed
+      req = Set.new(c.constructor_required_keywords)
+      opt = Set.new(c.constructor_optional_keywords)
+      all = req | opt
+
+      extra = all - allowed
       raise "Unsupported constructor keys in #{c.qualified_class_name}: #{extra.to_a.join(', ')}" unless extra.empty?
+
+      raise "Missing channel_access_token in #{c.qualified_class_name}" unless all.include?('channel_access_token')
+
+      unless SUPPORTED_BASE_URLS.include?(c.default_base_url)
+        raise "Unsupported default_base_url #{c.default_base_url.inspect} in #{c.qualified_class_name}. " \
+              "Supported: #{SUPPORTED_BASE_URLS.to_a.join(', ')}"
+      end
     end
   end
 

scripts/unified-client-generator/renderer.rb

@@ -106,14 +106,23 @@ def build_generated_delegates(
     RUBY
   end
 
+  BASE_URL_MAP = {
+    'https://api.line.me' => 'api_base_url',
+    'https://api-data.line.me' => 'data_api_base_url'
+  }.freeze
+
   def render_factory_assignment(client)
-    base_url_kw = { 'https://api.line.me' => 'api_base_url',
-                    'https://api-data.line.me' => 'data_api_base_url' }[client.default_base_url]
+    base_url_kw = BASE_URL_MAP[client.default_base_url]
+
+    supports = lambda do |key|
+      client.constructor_required_keywords.include?(key) ||
+        client.constructor_optional_keywords.include?(key)
+    end
 
     args = []
-    args << "base_url: #{base_url_kw}" if client.constructor_optional_keywords.include?('base_url') && base_url_kw
-    args << 'channel_access_token: channel_access_token' if client.constructor_required_keywords.include?('channel_access_token')
-    args << 'http_options: http_options' if client.constructor_optional_keywords.include?('http_options')
+    args << "base_url: #{base_url_kw}" if supports.call('base_url') && base_url_kw
+    args << 'channel_access_token: channel_access_token' if supports.call('channel_access_token')
+    args << 'http_options: http_options' if supports.call('http_options')
 
     "#{client.ivar_name} = #{client.qualified_class_name}.\n  new(#{args.join(', ')})"
   end

sig/line/bot/v2/client.rbs

@@ -7,12 +7,13 @@ module Line
   module Bot
     module V2
       class Client
-        # A single client for all LINE Bot APIs, except channel access token management.
+        # A single client for all LINE Bot APIs, except channel access token and module attach management.
         #
         # It wraps the individual generated API clients and exposes their methods directly,
         # so callers can work with one object instead of juggling several client instances.
         #
         # For channel access token operations, use {Line::Bot::V2::ChannelAccessToken::ApiClient} directly.
+        # For module attach operations, use {Line::Bot::V2::ModuleAttach::ApiClient} directly.
         @insight: Line::Bot::V2::Insight::ApiClient
         @liff: Line::Bot::V2::Liff::ApiClient
         @manage_audience: Line::Bot::V2::ManageAudience::ApiClient