pubid 2.0.4 → 2.0.5

data/lib/pubid/parg/model.rb

@@ -27,7 +27,7 @@ module Pubid
           source = all_parts ? input.sub(ALL_PARTS_PATTERN, "") : input
           hash = Artifact.for(flavor).parse_and_bind(entry, source)
           all_parts ? Backend.mark_all_parts(hash) : hash
-        rescue Parsanol::ParseFailed => e
+        rescue *ENGINE_ERRORS => e
           parse_error(e, input, flavor)
         end
 
@@ -38,12 +38,17 @@ module Pubid
           register(flavor, model_class, entry)
           model, all_parts = materialize(flavor, model_class, input)
           all_parts ? model.to_all_parts : model
-        rescue Parsanol::ParseFailed => e
+        rescue *ENGINE_ERRORS => e
           parse_error(e, input, flavor)
         end
 
         private
 
+        # The uniform error contract (CLAUDE.md "One error namespace"):
+        # every grammar/binding failure surfaces as Pubid::Errors::ParseError
+        # so `rescue Pubid::Errors::Error` stays exhaustive. Binding failures
+        # raise Parsanol::PARG::BindError (< PARG::Error, NOT ParseFailed) —
+        # left unrescued it escaped Pubid.parse raw (issue #646).
         def parse_error(error, input, flavor)
           raise Pubid::Errors::ParseError.new(
             error.message,

data/lib/pubid/parg.rb

@@ -1,9 +1,22 @@
 # frozen_string_literal: true
 
+require "parsanol"
+
 module Pubid
   module Parg
     autoload :Artifact, "pubid/parg/artifact"
     autoload :Backend, "pubid/parg/backend"
     autoload :Model, "pubid/parg/model"
+
+    # The engine errors a PG artifact can raise while parsing or binding an
+    # input. Every one of them must surface as Pubid::Errors::ParseError —
+    # the uniform error contract (CLAUDE.md "One error namespace") makes
+    # `rescue Pubid::Errors::Error` exhaustive, and a raw Parsanol class
+    # escaping Pubid.parse bypasses every such guard (issue #646: the JIS
+    # year forms raised Parsanol::PARG::BindError raw, which is NOT a
+    # Parsanol::ParseFailed). PARG::Error carries BindError
+    # (< ArtifactError < Error) and the PARG Parse/Compile/Artifact errors;
+    # Parsanol::ParseFailed is the ruby reference engine's parse failure.
+    ENGINE_ERRORS = [Parsanol::ParseFailed, Parsanol::PARG::Error].freeze
   end
 end

data/lib/pubid/tgpp/CLAUDE.md

@@ -41,3 +41,5 @@ keeps `#matches?(other, ignore:)`.
   paths and the strict comparison treats that `[]` and a nil alike — the
   same asymmetry the `initialize_empty` note above is about, seen from the
   matching side. relaton had two committed specs breaking on this.
+
+- **The flavor key is `:"3gpp"`, not the derived `:tgpp` (pubid#644)**: `PrefixesSupport#prefix_flavor_key` derives the key from the Ruby constant's name (`Pubid::Tgpp` → `:tgpp`), but the external key is "3gpp" — the registry registration, the `pubid:3gpp:*` `_type` namespace and the `urn:3gpp:` URN namespace all spell it with the digit. The derived mismatch made `Pubid.prefix_flavors` attribute the "3GPP" prefix to a key **nothing is registered under**, so the router's routing table resolved `Registry.get(:tgpp)` to `nil`, `candidates_by_prefix`'s `compact` silently dropped the owner, and `Pubid.parse("3GPP TS 38.101")` — which `Pubid::Tgpp.parse` handles fine (the prefixed spelling is a normalizing parse; canonical render drops the publisher, `with_publisher: true` puts it back) — lost its prefix-owner fallback and raised. The fix is a one-line `prefix_flavor_key` override in `lib/pubid/tgpp.rb` returning `:"3gpp"`; `Pubid.prefix_flavors["3GPP"]` now reads `[:\"3gpp\"]` and `Registry.canonical_name(Pubid::Tgpp)` answers the real key. The registry-driven tripwire `spec/pubid/parse_routing_spec.rb` ("resolves every prefix table entry to a registered flavor module") is the guard that turns any future key/registry mismatch of this shape red at the table level.

data/lib/pubid/tgpp.rb

@@ -16,6 +16,17 @@ module Pubid
     # excluded. The parser still accepts an optional leading "3GPP ".
     PREFIXES = ["3GPP"].freeze
 
+    # The external flavor key is "3gpp" (the registry key, the `pubid:3gpp:*`
+    # `_type` namespace and the `urn:3gpp:` URN namespace), but the default
+    # derivation spells it from the Ruby constant as :tgpp. That mismatch made
+    # {Pubid.prefix_flavors} attribute "3GPP" to a key nothing is registered
+    # under, so {Pubid}'s routing table resolved the prefix to no module and
+    # the "3GPP TS …" printed form lost its prefix-owner status (pubid#644).
+    # Return the real external key, matching the registry registration below.
+    def self.prefix_flavor_key
+      :"3gpp"
+    end
+
     autoload :Builder, "#{__dir__}/tgpp/builder"
     autoload :Identifier, "#{__dir__}/tgpp/identifier"
     autoload :Identifiers, "#{__dir__}/tgpp/identifiers"

data/lib/pubid/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Pubid
-  VERSION = "2.0.4"
+  VERSION = "2.0.5"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pubid
 version: !ruby/object:Gem::Version
-  version: 2.0.4
+  version: 2.0.5
 platform: ruby
 authors:
 - Ribose Inc.
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-11-02 00:00:00.000000000 Z
+date: 2026-11-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: lutaml-model
@@ -93,6 +93,7 @@ files:
 - data/parg/ieee.json
 - data/parg/ietf.json
 - data/parg/iho.json
+- data/parg/iho.parg
 - data/parg/ire.json
 - data/parg/ire.parg
 - data/parg/isbn.json
@@ -109,6 +110,7 @@ files:
 - data/parg/oiml.json
 - data/parg/omg.json
 - data/parg/parg.json
+- data/parg/parg.parg
 - data/parg/plateau.json
 - data/parg/sae.json
 - data/parg/tables/bipm_groups.yaml