mcp_authorization 0.6.0 → 0.6.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 63bcf798beee003e44d7894f8e2e80177a6ddb1e7357b68507a74464c2665ce6
-  data.tar.gz: 047560ce4c65b3b960f7c66904ca48c70b513e90510d7714edbd93996c39f919
+  metadata.gz: d09c532c9c775775e5b5dbea5d44df18cb9a693212641fff9b86621101806e4a
+  data.tar.gz: 18c5e6cc485efd2e73265e22d4ab037f3b21d32f72c377285794a644c8d4a225
 SHA512:
-  metadata.gz: 41b2c6a57ffeda77b7c74e2d2a849da6653d6678f78e9a76b9dec0cdaef5059b4ea00449063c57254e526c9cf6ebc780a7f43b59bfefeb57fabcc12e2f966842
-  data.tar.gz: 7dc31d4854025087e495bc9963ee46079a35f3cc6cc6fcbd3a904b00383017e6f68b7718db51a49fb78585657d86158e637fcbd5a182a72b1ad6d09ec83f362e
+  metadata.gz: 27959d3f62484456e2e6978f51cef90d8879584a978c4f4937e52fa599451c01ad41cfea2cb79607726ab2dcba166f50917c46b0bc0057639d1fba5aa4b4a249
+  data.tar.gz: cbdc8f8187594e942afc039248a8f8db638325b33f257972d85e5d8cbb62ec8d64f6e9f57b8e1ca022f9e1cbae1f7a0d590fa3cb5ea5784ad9cb87e20072e20a

data/CHANGELOG.md

@@ -4,6 +4,11 @@ All notable changes to this gem are documented here. The format follows
 [Keep a Changelog](https://keepachangelog.com/en/1.1.0/) and this project
 adheres to [Semantic Versioning](https://semver.org/).
 
+## [0.6.1] - 2026-07-01
+
+### Fixed
+- **Single-line string-literal-union type aliases (e.g. `type logic = "AND" | "OR"`) lost every member after the first.** Both the `# @rbs type` inline-comment parser and the shared `sig/shared/*.rbs` file parser only scanned *subsequent lines* for `| "value"` continuations, so a union written entirely on one line — the common shape for short enums like `"AND" | "OR"` — resolved to `{type: "string", enum: ["AND"]}` instead of `["AND", "OR"]`. Multi-line unions (`"low"\n| "medium"\n| "high"`) were unaffected. Literal unions written directly inline on a record field (not behind a named alias) were also unaffected, since those go through the RBS-parser union visitor rather than the alias collector.
+
 ## [0.6.0] - 2026-06-25
 
 Per-request work is now scoped to what each MCP method needs, and the

data/lib/mcp_authorization/rbs_schema_compiler.rb

@@ -1364,7 +1364,7 @@ module McpAuthorization
             current_name = $1.to_s
             current_base = nil
             current_body = "{"
-          elsif stripped =~ /\Atype (\w+) = "([^"]+)"/
+          elsif stripped =~ /\Atype (\w+) = ("[^"]*"(?:\s*\|\s*"[^"]*")*)/
             aliases[$1.to_s] = parse_rbs_string_union($2.to_s, line, content)
           elsif current_name
             current_body << strip_rbs_comment(stripped)
@@ -1380,15 +1380,24 @@ module McpAuthorization
         aliases
       end
 
-      # Parse a multi-line string literal union from an .rbs file:
+      # Parse a string literal union from an .rbs file, either written on
+      # one line:
+      #   type logic = "AND" | "OR"
+      # or continued across multiple lines:
       #   type priority = "low"
       #                 | "medium"
       #                 | "high"
       #
+      # +first_segment+ is everything captured after the +=+ on the
+      # opening line, which may itself already contain the full
+      # +"a" | "b" | "c"+ union — a single-line union has no continuation
+      # lines, so capturing only its first quoted literal (the prior
+      # behavior) silently dropped every member after the first.
+      #
       # @return [Hash] +{type: "string", enum: ["low", "medium", "high"]}+
       #: (String, String, String) -> Hash[Symbol, untyped]
-      def parse_rbs_string_union(first_value, line, content)
-        values = [first_value]
+      def parse_rbs_string_union(first_segment, line, content)
+        values = first_segment.scan(/"([^"]*)"/).flatten
         content.each_line.drop_while { |l| l != line }.drop(1).each do |next_line|
           if next_line =~ /^\s*\|\s*"([^"]+)"/
             values << $1.to_s
@@ -1491,7 +1500,7 @@ module McpAuthorization
               current_name = nil
               current_body = +""
             end
-          elsif line =~ /#\s*@rbs type (\w+)\s*=\s*"([^"]+)"/
+          elsif line =~ /#\s*@rbs type (\w+)\s*=\s*("[^"]*"(?:\s*\|\s*"[^"]*")*)/
             aliases[$1.to_s] = parse_string_union($2.to_s, line, content)
           elsif current_name
             stripped = strip_rbs_comment(line.strip.sub(/^#\s*/, ""))
@@ -1961,15 +1970,24 @@ module McpAuthorization
         str.count("{") == str.count("}")
       end
 
-      # Parse a multi-line string literal union from handler source comments:
+      # Parse a string literal union from handler source comments, either
+      # written on one line:
+      #   # @rbs type logic = "AND" | "OR"
+      # or continued across multiple lines:
       #   # @rbs type priority = "low"
       #   #                    | "medium"
       #   #                    | "high"
       #
+      # +first_segment+ is everything captured after the +=+ on the
+      # opening line, which may itself already contain the full
+      # +"a" | "b" | "c"+ union — a single-line union has no continuation
+      # lines, so capturing only its first quoted literal (the prior
+      # behavior) silently dropped every member after the first.
+      #
       # @return [Hash] +{type: "string", enum: ["low", "medium", "high"]}+
       #: (String, String, String) -> Hash[Symbol, untyped]
-      def parse_string_union(first_value, line, content)
-        values = [first_value]
+      def parse_string_union(first_segment, line, content)
+        values = first_segment.scan(/"([^"]*)"/).flatten
         content.each_line.drop_while { |l| l != line }.drop(1).each do |next_line|
           if next_line =~ /^\s*#\s*\|\s*"([^"]+)"/
             values << $1.to_s

data/lib/mcp_authorization/version.rb

@@ -1,3 +1,3 @@
 module McpAuthorization
-  VERSION = "0.6.0"
+  VERSION = "0.6.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mcp_authorization
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 0.6.1
 platform: ruby
 authors:
 - AndyGauge
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-06-25 00:00:00.000000000 Z
+date: 2026-07-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails