mt-lang 0.3.9 → 0.3.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 61a19112e2810978ecf1ce56b1d1d1290a26bcb933b9e89857ffca77d1323392
-  data.tar.gz: f68d1016a49c1cf195b5d24c430e4bbccde2c1ad9c7b82523b9e51d9f899d2b6
+  metadata.gz: efa0075ce73ffd0c92161f778399c599c6443e4375194a8bfeda883b3c29ea09
+  data.tar.gz: 978ab66cd06f4261edb72629f528c020890250ed7b57bbd75bee8cbe01e7e6a1
 SHA512:
-  metadata.gz: d7ffd6ea3d812428f451bbf304662a36a05986eb7ff2ec9a207364be7f37f81d84379e40325a155fd9565885d648503c73ab34fb1e851adbb1d5e53b193d696c
-  data.tar.gz: caa0c4acb4264740d58d857214ff68c3b726bf9a08d5f894ddc25466066e898567fd45c3b1c0e0634902be9a993acec7f1006b146af811ec0b918c778f60b234
+  metadata.gz: b0516d11fd07ed4c2db9bfaa4d59735821cfc3245aca280118f87593e7fa1fb2fec3be0c8208efc87b91c4cd79215e3a593602b43e1d9811572fcdcced93c280
+  data.tar.gz: 1a27eb0e66b9b15330ae28273b59ddfa430a6e7439811c2ca196b685cb6e8379b56016bb3a6eed60b347582fd013d8ba29f0c7024fbec0c13b67592aff3e0eba

data/README.md

@@ -30,6 +30,7 @@ Package manifests, build workflow, and run workflow are documented separately in
 - External files are the dedicated raw ABI surface, usually for generated or low-level `std.c.*` bindings.
 - Module lookup resolves `a.b.c` to `a/b/c.mt`.
 - Inside a package, the file path relative to `package.source_root` defines the module name; platform-specific files such as `name.linux.mt` still map to module `name`.
+- Circular module imports are supported via forward-declaration bindings and two-pass checking. Cycle members see each other's type declarations in the first pass and full type information in the second pass.
 - In ordinary files, `import` statements appear only at the top.
 - In external files, leading `import` statements are allowed after `external`.
 - Only external files accept `include`, `link`, and `compiler_flag` directives.
@@ -314,6 +315,7 @@ Rules:
 - Compile-time reflection over validated attributes uses `has_attribute`, `attribute_of`, `attribute_arg[T]`, `field_of`, and `callable_of`.
 - `field_of(...)`, `callable_of(...)`, and `attribute_of(...)` produce compile-time handle values with source-visible handle types `field_handle`, `callable_handle`, and `attribute_handle`.
 - The current C backend lowers `packed` / `align(...)` attributes with GNU-style `__attribute__((...))`, so these layout controls currently require a Clang/GCC-family compiler. On Windows that means Clang or GCC-family toolchains such as MinGW; `cl.exe` is not a supported backend for these attributes today. On wasm/browser targets the same feature works through Emscripten `emcc`, which is Clang-based.
+- Keywords and reserved words are accepted as variant arm names, struct/union field names, enum/flags member names, lifetime parameters (`@return`), type parameters (`Box[return]`), and named constructor arguments (`Meta(return = 1)`). Primitive type names (`int`, `str`, `ref`, etc.) remain reserved and cannot be used in these positions.
 
 Enum and flags values support the full set of comparison operators (`==`, `!=`, `<`, `<=`, `>`, `>=`) against values of the same enum type and against their backing integer type. Comparisons use the underlying integer backing values. Flags also support bitwise operators (`|`, `&`, `^`, `~`).
 

data/docs/language-design.md

@@ -490,7 +490,7 @@ Milk Tea has first-class compiler support for multithreading. The threading mode
 parallel for i in 0..entity_count:
     positions[i] += velocities[i] * dt
 
-# Structured fork-join: each do block runs on a separate thread
+# Structured fork-join: each statement runs on a separate thread
 parallel:
     textures = load_textures(path)
     sounds = load_sounds(path)
@@ -501,7 +501,6 @@ Design choices:
 - **Structured, not fire-and-forget.** All `parallel for` and `parallel:` blocks are synchronous barriers — the calling thread blocks until all work completes. This guarantees that captured local variables remain alive for the duration and eliminates lifetime concerns without ownership annotations.
 - **Compile-time safety.** `ref[T]` captures are rejected because mutable aliases across thread boundaries create data races. The `parallel:` block enforces single-writer-or-multiple-readers: if one statement writes a variable, no other block may read or write it.
 - **Value capture, pointer-based arrays.** Scalars and spans are captured by value (the span's data pointer still references the original storage). Arrays are captured as pointers to their first element, so writes in the worker affect the original array.
-- **`do` is a keyword.** It is recognized inside `parallel:` blocks to introduce each concurrent unit of work.
 - **Real OS threads via libuv.** Thread dispatch uses `uv_thread_create` / `uv_thread_join`, consistent with `std.thread` and `std.sync`. CPU count is detected at runtime via `uv_cpu_info`. The first chunk runs on the calling thread to avoid unnecessary dispatch when the workload is small.
 - **No forced dependency.** The build system automatically detects `parallel for` and `parallel:` usage via a sema-level flag and links libuv only when needed.
 

data/docs/language-manual.md

@@ -51,6 +51,7 @@ Rules:
 - Valid platform filename suffixes are `linux`, `windows`, and `wasm`.
 - The platform suffix is not part of the module name. `import a.b.c` stays the same on every target.
 - Milk Tea does not have a source-level conditional compilation syntax such as `#if`, `#ifdef`, or per-declaration platform attributes.
+- Circular module imports are supported. When two modules import each other, the compiler uses forward-declaration bindings for the first pass and full type-checked bindings for the second pass, enabling cross-module type references and constructors in cycle groups.
 
 ## 2. Lexical Rules
 
@@ -360,6 +361,17 @@ Compile-time reflection over validated attributes uses `has_attribute`, `attribu
 
 The current C backend lowers `packed` / `align(...)` attributes with GNU-style `__attribute__((...))`, so these layout controls currently require a Clang/GCC-family compiler. On Windows that means Clang or GCC-family toolchains such as MinGW; `cl.exe` is not a supported backend for these attributes today. On wasm/browser targets the same feature works through Emscripten `emcc`, which is Clang-based.
 
+Language-level keywords and reserved words are accepted as:
+- variant arm names (`return(val: int)`, `if(arg: bool)`)
+- struct and union field names (`return: int`, `let: str`)
+- enum and flags member names (`return = 1`, `if = 6`)
+- variant arm payload field names (`execute(return: int)`)
+- lifetime parameter names (`struct Buf[@return]`)
+- type parameter names (`struct Box[return]`, `identity[type]`)
+- named arguments in struct/variant literals (`Meta(return = 1)`)
+
+Primitive type names (`int`, `str`, `bool`, `ref`, `ptr`, `span`, `type`, etc.) remain reserved and cannot be used as names.
+
 ### 3.4a Enum and flags operators
 
 Enum and flags values support the full set of comparison operators (`==`, `!=`, `<`, `<=`, `>`, `>=`) against values of the same enum type and against their backing integer type. Comparisons use the underlying integer backing values.
@@ -746,7 +758,6 @@ parallel:
 Rules:
 
 - A `parallel:` block must contain at least two statements.
-- `do` is a contextual keyword — only recognized inside `parallel:` blocks, not reserved globally.
 - Each statement must not contain `break`, `continue`, `return`, or `defer`.
 - The compiler enforces single-writer-or-multiple-readers: if a variable is written in one statement, no other block may access it.
 - Captured `ref[T]` values are rejected at compile time.

data/lib/milk_tea/base.rb

@@ -3,7 +3,7 @@
 require "pathname"
 
 module MilkTea
-  VERSION = "0.3.9"
+  VERSION = "0.3.11"
 
   def self.root
     @root ||= Pathname.new(File.expand_path("../..", __dir__))
@@ -23,7 +23,6 @@ module MilkTea
   def self.installed_as_gem?
     root.to_s.match?(%r{/gems/mt-lang-})
   end
-  private_class_method :installed_as_gem?
 
   def self.writable_root_for(root)
     resolved = Pathname.new(File.expand_path(root.to_s))

data/lib/milk_tea/bindings/bindgen/ast_parser.rb

@@ -4,8 +4,6 @@ module MilkTea
   module Bindgen
     class Generator
       module GeneratorAstParser
-        private
-
         def dump_ast
           Tempfile.create(["milk-tea-bindgen", ".c"]) do |translation_unit|
             @translation_unit_path = translation_unit.path

data/lib/milk_tea/bindings/bindgen/declaration.rb

@@ -4,8 +4,6 @@ module MilkTea
   module Bindgen
     class Generator
       module GeneratorDeclaration
-        private
-
         def select_record_declarations(nodes)
           selected = {}
           type_aliases = []

data/lib/milk_tea/bindings/bindgen/emitter.rb

@@ -4,8 +4,6 @@ module MilkTea
   module Bindgen
     class Generator
       module GeneratorEmitter
-        private
-
         def emitted_name(raw_name)
           return raw_name unless @strip_leading_underscores
 

data/lib/milk_tea/bindings/bindgen/overrides.rb

@@ -4,8 +4,6 @@ module MilkTea
   module Bindgen
     class Generator
       module GeneratorOverrides
-        private
-
         def function_param_type_override(function_name, param_name)
           @function_param_type_overrides.dig(function_name, param_name)
         end

data/lib/milk_tea/bindings/bindgen/type_mapper.rb

@@ -4,64 +4,62 @@ module MilkTea
   module Bindgen
     class Generator
       module GeneratorTypeMapper
-        private
-
         def map_c_type(qual_type, context:)
           normalized = normalize_c_type(qual_type)
           raise BindgenError, "missing C type for #{context}" if normalized.empty?
           normalized, nullability = extract_top_level_nullability(normalized)
           mapped_type = if array_type?(normalized)
-                          map_array_type(normalized, context:)
-                        elsif function_pointer_type?(normalized)
-                          map_function_pointer_type(normalized, context:)
-                        else
-                          pointer_candidate = strip_pointer_suffix_qualifiers(normalized)
-                          if pointer_type?(pointer_candidate)
-                            if va_list_pointer?(pointer_candidate)
-                              synthesize_typedef_dependency("va_list")
-                              "va_list"
-                            elsif c_string_pointer?(pointer_candidate)
-                              "cstr"
-                            else
-                              pointee = pointer_candidate.sub(/\s*\*\z/, "")
-                              pointer_name = top_level_const_qualified?(pointee) ? "const_ptr" : "ptr"
-                              "#{pointer_name}[#{map_c_type(pointee, context:)}]"
-                            end
-                          else
-                            unqualified = strip_qualifiers(normalized)
-                            if unqualified == "__va_list_tag" || unqualified == "struct __va_list_tag"
-                              synthesize_typedef_dependency("__va_list_tag")
-                              "__va_list_tag"
-                            elsif standard_typedef_primitive(unqualified)
-                              standard_typedef_primitive(unqualified)
-                            elsif PRIMITIVE_TYPE_MAP.key?(unqualified)
-                              PRIMITIVE_TYPE_MAP.fetch(unqualified)
-                            elsif @type_overrides.key?(unqualified)
-                              @type_overrides.fetch(unqualified)
-                            elsif unqualified.start_with?("long") || unqualified.start_with?("unsigned long") || unqualified.start_with?("signed long")
-                              map_long_type(unqualified, context:)
-                            elsif unqualified.start_with?("struct ") || unqualified.start_with?("union ") || unqualified.start_with?("enum ")
-                              tag_name = unqualified.split.last
-                              @type_overrides.fetch(tag_name) do
-                                if unqualified.start_with?("struct ")
-                                  record_name_for(unqualified)
-                                elsif unqualified.start_with?("union ")
-                                  record_name_for(unqualified)
-                                else
-                                  enum_name_for(unqualified)
-                                end
-                              end
-                            elsif unqualified.match?(/\A[A-Za-z_][A-Za-z0-9_]*\z/)
-                              if known_generated_type_name?(unqualified, @visible_typedef_names)
-                                visible_type_name(unqualified)
-                              else
-                                raise BindgenError, "unknown referenced C type #{unqualified.inspect} for #{context}"
-                              end
-                            else
-                              raise BindgenError, "unsupported C type #{qual_type.inspect} for #{context}"
-                            end
-                          end
-                        end
+            map_array_type(normalized, context:)
+          elsif function_pointer_type?(normalized)
+            map_function_pointer_type(normalized, context:)
+          else
+            pointer_candidate = strip_pointer_suffix_qualifiers(normalized)
+            if pointer_type?(pointer_candidate)
+              if va_list_pointer?(pointer_candidate)
+                synthesize_typedef_dependency("va_list")
+                "va_list"
+              elsif c_string_pointer?(pointer_candidate)
+                "cstr"
+              else
+                pointee = pointer_candidate.sub(/\s*\*\z/, "")
+                pointer_name = top_level_const_qualified?(pointee) ? "const_ptr" : "ptr"
+                "#{pointer_name}[#{map_c_type(pointee, context:)}]"
+              end
+            else
+              unqualified = strip_qualifiers(normalized)
+              if unqualified == "__va_list_tag" || unqualified == "struct __va_list_tag"
+                synthesize_typedef_dependency("__va_list_tag")
+                "__va_list_tag"
+              elsif standard_typedef_primitive(unqualified)
+                standard_typedef_primitive(unqualified)
+              elsif PRIMITIVE_TYPE_MAP.key?(unqualified)
+                PRIMITIVE_TYPE_MAP.fetch(unqualified)
+              elsif @type_overrides.key?(unqualified)
+                @type_overrides.fetch(unqualified)
+              elsif unqualified.start_with?("long") || unqualified.start_with?("unsigned long") || unqualified.start_with?("signed long")
+                map_long_type(unqualified, context:)
+              elsif unqualified.start_with?("struct ") || unqualified.start_with?("union ") || unqualified.start_with?("enum ")
+                tag_name = unqualified.split.last
+                @type_overrides.fetch(tag_name) do
+                  if unqualified.start_with?("struct ")
+                    record_name_for(unqualified)
+                  elsif unqualified.start_with?("union ")
+                    record_name_for(unqualified)
+                  else
+                    enum_name_for(unqualified)
+                  end
+                end
+              elsif unqualified.match?(/\A[A-Za-z_][A-Za-z0-9_]*\z/)
+                if known_generated_type_name?(unqualified, @visible_typedef_names)
+                  visible_type_name(unqualified)
+                else
+                  raise BindgenError, "unknown referenced C type #{unqualified.inspect} for #{context}"
+                end
+              else
+                raise BindgenError, "unsupported C type #{qual_type.inspect} for #{context}"
+              end
+            end
+          end
 
           apply_nullability(mapped_type, nullability)
         end

data/lib/milk_tea/bindings/imported_bindings/generator.rb

@@ -4,8 +4,6 @@ module MilkTea
   module ImportedBindings
     class Generator
       module GeneratorPolicy
-        private
-
         def load_policy
           JSON.parse(File.read(@policy_path))
         rescue Errno::ENOENT
@@ -914,13 +912,13 @@ module MilkTea
           signature = +""
           signature << "public " if visibility == :public
           signature << case kind
-                       when :static
-                         "static function "
-                       when :editable
-                         "editable function "
-                       else
-                         "function "
-                       end
+          when :static
+            "static function "
+          when :editable
+            "editable function "
+          else
+            "function "
+          end
           signature << name
           signature << render_type_params(type_params)
           signature << "(#{params.join(', ')}) -> #{return_type}:"

data/lib/milk_tea/bindings/imported_bindings/method_source.rb

@@ -4,8 +4,6 @@ module MilkTea
   module ImportedBindings
     class Generator
       module GeneratorMethodSource
-        private
-
         def load_method_sources(method_specs)
           method_specs.each_with_object({}) do |spec, sources|
             next unless spec[:module_name]

data/lib/milk_tea/bindings/imported_bindings/naming.rb

@@ -85,8 +85,6 @@ module MilkTea
           uniform
         ].freeze
 
-        private
-
         def normalize_opengl_snake_case(name)
           tokens = name.to_s.split("_").reject(&:empty?)
           normalized = []
@@ -175,12 +173,12 @@ module MilkTea
 
         def normalize_opengl_terminal_suffix(name, raw_name)
           normalized = if name.end_with?("indexedfv")
-                         "#{name.delete_suffix('indexedfv')}indexed_float_values"
-                       elsif name.end_with?("indexedf")
-                         "#{name.delete_suffix('indexedf')}indexed_float"
-                       else
-                         name
-                       end
+            "#{name.delete_suffix('indexedfv')}indexed_float_values"
+          elsif name.end_with?("indexedf")
+            "#{name.delete_suffix('indexedf')}indexed_float"
+          else
+            name
+          end
 
           raw_text = raw_name.to_s
           if raw_text.end_with?("i") && normalized.end_with?("i")

data/lib/milk_tea/bindings/imported_bindings.rb

@@ -205,8 +205,6 @@ module MilkTea
         Formatter.format_source(source, path: generated_module_path, mode: :tidy)
       end
 
-      private
-
       include GeneratorPolicy
       include GeneratorMethodSource
       include GeneratorNaming

data/lib/milk_tea/bindings/opengl_registry.rb

@@ -23,18 +23,16 @@ module MilkTea
     FeatureEntry = Data.define(:name, :api, :number, :blocks)
     ExtensionEntry = Data.define(:name, :supported, :blocks)
 
-    module_function
-
-    def helper_header_path(root: MilkTea.root)
+    def self.helper_header_path(root: MilkTea.root)
       root.join("std/c/#{HELPER_HEADER_BASENAME}")
     end
 
-    def source(root: MilkTea.root)
+    def self.source(root: MilkTea.root)
       UpstreamSources.default_sources(root:).find { |entry| entry.name == SOURCE_NAME } ||
         raise(Error, "missing upstream source definition for #{SOURCE_NAME}")
     end
 
-    def prepare!(root: MilkTea.root, api: DEFAULT_API, version: DEFAULT_VERSION, profile: DEFAULT_PROFILE, extensions: [])
+    def self.prepare!(root: MilkTea.root, api: DEFAULT_API, version: DEFAULT_VERSION, profile: DEFAULT_PROFILE, extensions: [])
       registry_source = source(root:)
       registry_source.bootstrap!
 
@@ -90,8 +88,6 @@ module MilkTea
         )
       end
 
-      private
-
       def parse_registry
         document = REXML::Document.new(@xml_source)
         root = document.root

data/lib/milk_tea/bindings/steamworks.rb

@@ -38,26 +38,24 @@ module MilkTea
       windows: "steam_api64.dll",
     }.freeze
 
-    module_function
-
-    def helper_header_path(root: MilkTea.root)
+    def self.helper_header_path(root: MilkTea.root)
       root.join("std/c/#{HELPER_HEADER_BASENAME}")
     end
 
-    def source(root: MilkTea.root)
+    def self.source(root: MilkTea.root)
       UpstreamSources.default_sources(root:).find { |entry| entry.name == SOURCE_NAME } ||
         raise(Error, "missing upstream source definition for #{SOURCE_NAME}")
     end
 
-    def host_platform
+    def self.host_platform
       MilkTea.host_platform
     end
 
-    def default_link_libraries(platform: host_platform)
+    def self.default_link_libraries(platform: host_platform)
       [LINK_LIBRARY_NAME_BY_PLATFORM.fetch(platform)]
     end
 
-    def sdk_root(root: MilkTea.root, env: ENV, bootstrap: false)
+    def self.sdk_root(root: MilkTea.root, env: ENV, bootstrap: false)
       candidates = []
       env_root = env[SDK_ROOT_ENV_VAR]
       candidates << canonical_sdk_root(env_root) if env_root && !env_root.empty?
@@ -76,7 +74,7 @@ module MilkTea
       candidates.find { |path| sdk_layout_present?(path) }
     end
 
-    def api_json_path(root: MilkTea.root, env: ENV, bootstrap: false)
+    def self.api_json_path(root: MilkTea.root, env: ENV, bootstrap: false)
       explicit_path = env[API_JSON_ENV_VAR]
       if explicit_path && !explicit_path.empty?
         candidate = Pathname.new(File.expand_path(explicit_path))
@@ -94,7 +92,7 @@ module MilkTea
       raise Error, "Steamworks metadata not found under #{resolved_sdk_root}"
     end
 
-    def redistributable_directory(root: MilkTea.root, env: ENV, platform: host_platform, bootstrap: false)
+    def self.redistributable_directory(root: MilkTea.root, env: ENV, platform: host_platform, bootstrap: false)
       resolved_sdk_root = sdk_root(root:, env:, bootstrap:)
       return unless resolved_sdk_root
 
@@ -104,7 +102,7 @@ module MilkTea
       nil
     end
 
-    def import_library_path(root: MilkTea.root, env: ENV, platform: host_platform, bootstrap: false)
+    def self.import_library_path(root: MilkTea.root, env: ENV, platform: host_platform, bootstrap: false)
       directory = redistributable_directory(root:, env:, platform:, bootstrap:)
       return unless directory
 
@@ -114,7 +112,7 @@ module MilkTea
       nil
     end
 
-    def runtime_library_path(root: MilkTea.root, env: ENV, platform: host_platform, bootstrap: false)
+    def self.runtime_library_path(root: MilkTea.root, env: ENV, platform: host_platform, bootstrap: false)
       directory = redistributable_directory(root:, env:, platform:, bootstrap:)
       return unless directory
 
@@ -124,7 +122,7 @@ module MilkTea
       nil
     end
 
-    def prepare!(root: MilkTea.root, env: ENV)
+    def self.prepare!(root: MilkTea.root, env: ENV)
       path = helper_header_path(root:)
       existing = File.exist?(path)
       json_path = api_json_path(root:, env:, bootstrap: !existing)
@@ -139,7 +137,7 @@ module MilkTea
       path.to_s
     end
 
-    def canonical_sdk_root(path)
+    def self.canonical_sdk_root(path)
       root = Pathname.new(File.expand_path(path.to_s))
       sdk_child = root.join("sdk")
       return sdk_child if sdk_layout_present?(sdk_child)
@@ -148,7 +146,7 @@ module MilkTea
     end
     private_class_method :canonical_sdk_root
 
-    def sdk_layout_present?(root)
+    def self.sdk_layout_present?(root)
       File.file?(root.join(API_JSON_RELATIVE_PATH))
     end
     private_class_method :sdk_layout_present?
@@ -280,8 +278,6 @@ module MilkTea
         lines.join("\n") + "\n"
       end
 
-      private
-
       def metadata
         @metadata ||= JSON.parse(@json_source)
       rescue JSON::ParserError => e

data/lib/milk_tea/bindings/upstream_sources.rb

@@ -38,8 +38,6 @@ module MilkTea
         end
       end
 
-      private
-
       def run_git!(*args)
         stdout, stderr, status = Open3.capture3("git", *args)
         return if status.success?
@@ -51,13 +49,11 @@ module MilkTea
       end
     end
 
-    module_function
-
-    def revisions_path(root: MilkTea.root)
+    def self.revisions_path(root: MilkTea.root)
       Pathname.new(File.expand_path(root.to_s)).join(CONFIG_DIR, REVISIONS_FILE)
     end
 
-    def load_version_overrides(root: MilkTea.root)
+    def self.load_version_overrides(root: MilkTea.root)
       path = revisions_path(root:)
       return {} unless File.file?(path)
 
@@ -78,7 +74,7 @@ module MilkTea
       overrides
     end
 
-    def load_sentinel_overrides(root: MilkTea.root)
+    def self.load_sentinel_overrides(root: MilkTea.root)
       path = revisions_path(root:)
       return {} unless File.file?(path)
 
@@ -95,7 +91,7 @@ module MilkTea
       overrides
     end
 
-    def write_version_override(name, revision, root: MilkTea.root)
+    def self.write_version_override(name, revision, root: MilkTea.root)
       path = revisions_path(root:)
       data = File.file?(path) ? (YAML.safe_load(File.read(path)) || {}) : {}
       existing = data[name.to_s]
@@ -108,7 +104,7 @@ module MilkTea
       File.write(path, data.to_yaml)
     end
 
-    def remove_version_override(name, root: MilkTea.root)
+    def self.remove_version_override(name, root: MilkTea.root)
       path = revisions_path(root:)
       return unless File.file?(path)
 
@@ -121,7 +117,7 @@ module MilkTea
       end
     end
 
-    def resolve_ref(repository_url, ref)
+    def self.resolve_ref(repository_url, ref)
       return ref if ref.match?(/\A[0-9a-f]{40}\z/i)
 
       output, status = Open3.capture2("git", "ls-remote", "--refs", repository_url, ref.to_s)
@@ -140,11 +136,11 @@ module MilkTea
       lines.first.split(/\s+/, 2).first
     end
 
-    def find_source(name, root: MilkTea.root)
+    def self.find_source(name, root: MilkTea.root)
       default_sources(root:).find { |source| source.name == name.to_s }
     end
 
-    def default_sources(root: MilkTea.root)
+    def self.default_sources(root: MilkTea.root)
       data = MilkTea.writable_root_for(root)
       sources = [
         Source.new(
@@ -338,11 +334,11 @@ module MilkTea
       sources
     end
 
-    def bootstrap_all!(root: MilkTea.root)
+    def self.bootstrap_all!(root: MilkTea.root)
       default_sources(root:).map(&:bootstrap!)
     end
 
-    def bootstrap_one(name, root: MilkTea.root)
+    def self.bootstrap_one(name, root: MilkTea.root)
       source = find_source(name, root:)
       raise Error, "unknown upstream source: #{name}" unless source
 

data/lib/milk_tea/bindings/vendored_box2d.rb

@@ -22,9 +22,7 @@ module MilkTea
       -lm
     ].freeze
 
-    module_function
-
-    def library(root: MilkTea.root)
+    def self.library(root: MilkTea.root)
       resolved_root = Pathname.new(File.expand_path(root.to_s))
       @libraries ||= {}
       @libraries[resolved_root.to_s] ||= VendoredCLibrary::CMake.new(
@@ -40,39 +38,39 @@ module MilkTea
       )
     end
 
-    def source_root(root: MilkTea.root)
+    def self.source_root(root: MilkTea.root)
       MilkTea.writable_root_for(root).join("third_party/box2d-upstream")
     end
 
-    def include_root(root: MilkTea.root)
+    def self.include_root(root: MilkTea.root)
       source_root(root:).join("include")
     end
 
-    def header_root(root: MilkTea.root)
+    def self.header_root(root: MilkTea.root)
       include_root(root:).join("box2d")
     end
 
-    def build_root(root: MilkTea.root)
+    def self.build_root(root: MilkTea.root)
       MilkTea.writable_root_for(root).join("tmp/vendored-box2d")
     end
 
-    def install_root(root: MilkTea.root)
+    def self.install_root(root: MilkTea.root)
       MilkTea.writable_root_for(root).join("tmp/vendored-box2d-prefix")
     end
 
-    def archive_path(root: MilkTea.root)
+    def self.archive_path(root: MilkTea.root)
       install_root(root:).join("lib/libbox2d.a")
     end
 
-    def include_flags(root: MilkTea.root)
+    def self.include_flags(root: MilkTea.root)
       library(root:).include_flags
     end
 
-    def link_flags(root: MilkTea.root)
+    def self.link_flags(root: MilkTea.root)
       library(root:).link_flags
     end
 
-    def prepare!(root: MilkTea.root, **kwargs)
+    def self.prepare!(root: MilkTea.root, **kwargs)
       library(root:).prepare!(**kwargs)
     end
   end

data/lib/milk_tea/bindings/vendored_cjson.rb

@@ -12,9 +12,7 @@ module MilkTea
       -lm
     ].freeze
 
-    module_function
-
-    def library(root: MilkTea.root)
+    def self.library(root: MilkTea.root)
       resolved_root = Pathname.new(File.expand_path(root.to_s))
       @libraries ||= {}
       @libraries[resolved_root.to_s] ||= VendoredCLibrary::Archive.new(
@@ -29,23 +27,23 @@ module MilkTea
       )
     end
 
-    def source_root(root: MilkTea.root)
+    def self.source_root(root: MilkTea.root)
       MilkTea.writable_root_for(root).join("third_party/cjson-upstream")
     end
 
-    def build_root(root: MilkTea.root)
+    def self.build_root(root: MilkTea.root)
       MilkTea.writable_root_for(root).join("tmp/vendored-cjson")
     end
 
-    def archive_path(root: MilkTea.root)
+    def self.archive_path(root: MilkTea.root)
       build_root(root:).join("libcjson.a")
     end
 
-    def link_flags(root: MilkTea.root)
+    def self.link_flags(root: MilkTea.root)
       library(root:).link_flags
     end
 
-    def prepare!(root: MilkTea.root, **kwargs)
+    def self.prepare!(root: MilkTea.root, **kwargs)
       library(root:).prepare!(**kwargs)
     end
   end