tag_ripper 0.1.0 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3f55c43ce02431ffc69bd9284ec5875678d743e3c14544f3ebb0710e440921b8
-  data.tar.gz: 10b1baa79118d897442fd87e49ee7da2b9259172802246f22e0ea5a3b3855d72
+  metadata.gz: 375a528cbf8942e0dd48c51ffe10c005a6b3672b414e82b5d768011f3355e350
+  data.tar.gz: f3e74892414a1587e5dfabeba4cda6839f2dd500f800bda6303fa2ea97f18042
 SHA512:
-  metadata.gz: ab28505f7eb7784a4a90d7613960263251b43f9d8dcbca34570c360828f7c651e3b60d5e518b0d98e6722e0fc214a53e27f8280dfe3713962474c5914074fb5c
-  data.tar.gz: c38bd4b3c85a1a6b02f7fe57ab61e69ea729f031d9f6a04fd40a85e71e978c820d7577d63a060079f3309a704452b779fa994d7fca6cc09d547dbcbdbb8622af
+  metadata.gz: 49ababa76c465f7fa6a691f968290a6668762d884e0be15ca3790e6b6b884853aa1ff1bbbe78c9afdeef293b85ab4f3470357d635c9dda531257f26e81d4700b
+  data.tar.gz: 07311f3a4768f8643c0fafcab84f3b6318392a5c1726bcfccf2fcfd1291db35e4039d4493adeb4e0241765b086e56bd2fbb3feee4e9065452c4880df3a9d88f9

data/.rubocop.yml

@@ -8,7 +8,9 @@ AllCops:
   TargetRubyVersion: 3.1
   NewCops: enable
   SuggestExtensions: true
-
+  Exclude:
+    - bin/*
+    - test/fixtures/**/*  # These files have particular styles that should be preserved
 Metrics/AbcSize:
   Exclude:
     - test/**/*
@@ -30,4 +32,3 @@ Style/TrivialAccessors:
 
 Style/StringLiteralsInInterpolation:
   EnforcedStyle: double_quotes
-

data/README.md

@@ -17,15 +17,26 @@ class User
 end
 
 TagRipper.new(File.read('user.rb')).taggables
-# => [
-# <id=22040, @name=some_method_that_isnt_tested, tags={"warning" => #<Set: {"untested"}>},parent=#<TagRipper::TaggableEntity:0x00000001203bf468>>,
-#  <id=22224, @name=User, tags={"domain" => #<Set: {"Auth"}>},parent=>nil
-# ]
+# (Beautified output)
+# ---
+# - 
+#   id: 2221
+#   name: some_method_that_isnt_tested
+#   tags:
+#     warning:
+#       - "untested
+#   parent: 22224,
+# - 
+#   id: 22224
+#   name: User
+#   tags:
+#     domain: 
+#       - "Auth"
+#    parent: nil
 ```
 
 ## Installation
 
-TODO: Replace `tag_ripper` with your gem name right after releasing it to RubyGems.org. Please do not do it earlier due to security reasons. Alternatively, replace this section with instructions to install your gem from git if you don't plan to release to RubyGems.org.
 
 Install the gem and add to the application's Gemfile by executing:
 

data/lib/tag_ripper/lexical_token.rb

@@ -49,6 +49,10 @@ module TagRipper
       comment? && token.match?(TAG_REGEX)
     end
 
+    def double_colon?
+      token == "::"
+    end
+
     def keyword?
       type == :on_kw
     end

data/lib/tag_ripper/taggable_entity.rb

@@ -9,22 +9,42 @@ module TagRipper
   # entity is spawned. This creates a sort of recursion that allows a taggable
   # entity to be flexible to any amount of code nesting.
   class TaggableEntity
+    # Unable to move transition from one state to another
     class IllegalStateTransitionError < StandardError
       def initialize(from:, to:)
         super("Cannot transition from #{from} to #{to}")
       end
     end
 
+    # Attempting to set status to an unknown value
+    class InvalidStatusError < ArgumentError; end
+
+    # TODO: define naming state, to represent a partial name token
+    # The valid statuses that a TaggableEntity can move through.
+    # @return [Array<Symbol>]
+    VALID_STATUSES = %i[
+      pending
+      tagged
+      awaiting_name
+      named
+      closed
+    ].freeze
+
+    # Statuses that represent an open lexical scope.
+    # @return [Array<Symbol>]
+    OPENED_STATUSES = %i[tagged awaiting_name named].freeze
+
     def initialize(name: nil, parent: nil)
       @name = name
       @tags = Hash.new { |hash, key| hash[key] = Set.new }
       @parent = parent
       @type = nil
-      @status = :pending
+      self.status = :pending
     end
 
+    alias id object_id
+
     def send_event(event_name, lex)
-      puts "send_event: #{event_name} - #{lex} #(#{@status})"
       if respond_to?(event_name, true)
         send(event_name, lex)
       else
@@ -39,6 +59,9 @@ module TagRipper
     def type
       @type
     end
+
+    # The fully-qualified name of this entity (e.g. +"Foo::Bar::MyClass"+)
+    # @return [String]
     def fqn
       return nil unless named?
       return name if fqn_names.size == 1
@@ -51,14 +74,10 @@ module TagRipper
     end
     alias fully_qualified_name fqn
 
-    def pending? = @status == :pending
-
-    def tagged? = @status == :tagged
-
-    def awaiting_name? = @status == :awaiting_name
-
-    OPENED_STATUSES = %i[tagged awaiting_name named].freeze
-
+    # Have we opened a new lexical scope? (e.g. evaluating within the body
+    # of a class, rather than comments before the class)
+    #
+    # @return [Boolean]
     def open?
       OPENED_STATUSES.include?(@status)
     end
@@ -68,7 +87,7 @@ module TagRipper
         raise IllegalStateTransitionError.new(from: @status, to: :tagged)
       end
 
-      @status = :tagged
+      self.status = :tagged
 
       add_tag(tag_name, tag_value)
     end
@@ -78,7 +97,7 @@ module TagRipper
         raise IllegalStateTransitionError.new(from: @status, to: :awaiting_name)
       end
 
-      @status = :awaiting_name
+      self.status = :awaiting_name
     end
 
     def name=(name)
@@ -87,17 +106,19 @@ module TagRipper
       end
 
       @name = name.to_s
-      @status = :named
+      self.status = :named
     end
 
     def close!
       @open = false
-      @status = :closed
+      self.status = :closed
       freeze
     end
 
-    def closed?
-      @status == :closed
+    VALID_STATUSES.each do |status|
+      define_method(:"#{status}?") do
+        @status == status
+      end
     end
 
     def may_tag?
@@ -128,13 +149,28 @@ module TagRipper
       @name.to_s.dup
     end
 
-    def type=(type)
-      @type = type.to_sym
+    def parent
+      @parent
     end
 
     protected
 
-    alias id object_id
+    def append_name!(string)
+      raise StandardError, "Cannot append #{string} to nil" unless @name
+
+      @name.concat(string.to_s)
+    end
+
+    def status=(status)
+      status = status.to_sym
+      raise InvalidStatusError unless VALID_STATUSES.include?(status)
+
+      @status = status
+    end
+
+    def type=(type)
+      @type = type.to_sym
+    end
 
     def fqn_names
       return [name] if parent.nil?
@@ -142,10 +178,6 @@ module TagRipper
       [*parent.fqn_names, name]
     end
 
-    def parent
-      @parent
-    end
-
     def add_tag(name, value)
       @tags[name].add(value)
     end
@@ -178,6 +210,8 @@ module TagRipper
       send(event_token_method_name, lex)
     end
 
+    ##
+    # Lex is a keyword (e.g. def, class, module)
     def on_new_taggable_context_kw(lex)
       returnable_entity = named? ? build_child : self
 
@@ -191,21 +225,41 @@ module TagRipper
     alias on_kw_module on_new_taggable_context_kw
     alias on_kw_class on_new_taggable_context_kw
 
-    IGNORED_IDENT_KEYWORDS = %w[require private class_eval instance_eval define_method].freeze
+    IGNORED_IDENT_KEYWORDS = %w[require private class_eval instance_eval
+                                define_method].freeze
     private_constant :IGNORED_IDENT_KEYWORDS
 
-    def name_from_lex(lex)
+    def name_from_lex(lex) # rubocop:disable Metrics
       return self if IGNORED_IDENT_KEYWORDS.include?(lex.token)
-      return self if named?
-      return self unless may_name?
+      # TODO: Simplify this logic
+      return self if named? && !@name.end_with?("::")
+      return self unless may_name? || @name.end_with?("::")
+
+      # self.status = :awaiting_name # TODO: Fix this with a proper state
+      if named? && @name.end_with?("::")
+        append_name!(lex.token)
+      else
+        self.name = "#{name}#{lex.token}"
+      end
 
-      self.name = lex.token
       self
-    end
+    end # rubocop:enable Metrics
 
+    ##
+    # Matches names of constants: module names, const names, etc.
     alias on_const name_from_lex
+
+    ##
+    # Matches tokens like: private, method names, argument names
     alias on_ident name_from_lex
 
+    def on_op(lex)
+      if lex.double_colon?
+        append_name!(lex.token)
+      end
+      self
+    end
+
     def on_kw_end(_lex)
       close!
       parent

data/lib/tag_ripper/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module TagRipper
-  VERSION = "0.1.0"
+  VERSION = "0.2.0"
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: tag_ripper
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
 - Gavin Morrice
 bindir: exe
 cert_chain: []
-date: 2025-04-03 00:00:00.000000000 Z
+date: 2025-06-19 00:00:00.000000000 Z
 dependencies: []
 description: Add tags to your Ruby code comments and then Rip the as lexical tokens
 email:
@@ -30,13 +30,13 @@ files:
 - lib/tag_ripper/ripper.rb
 - lib/tag_ripper/taggable_entity.rb
 - lib/tag_ripper/version.rb
-homepage: https://github.com/Bodacious/tag-ripper/
+homepage: https://github.com/Bodacious/tag_ripper/
 licenses:
 - MIT
 metadata:
-  homepage_uri: https://github.com/Bodacious/tag-ripper/
-  source_code_uri: https://github.com/Bodacious/tag-ripper/
-  changelog_uri: https://github.com/Bodacious/tag-ripper/
+  homepage_uri: https://github.com/Bodacious/tag_ripper/
+  source_code_uri: https://github.com/Bodacious/tag_ripper/
+  changelog_uri: https://github.com/Bodacious/tag_ripper/
   rubygems_mfa_required: 'true'
 rdoc_options: []
 require_paths: