aye_var 0.1.1 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2e1eb735e2846d27610ccd41568c9022de487d62ccbf2c9d212af0019c72e1c9
-  data.tar.gz: 0ceb88f7fe73970d8c7c47d15671ecc2fc8350a8c05e29eed55d4ce6935828bd
+  metadata.gz: d1738b022486bc41367e9dee76d2fc4a0792a47dccb5059ac872a479b192ab8c
+  data.tar.gz: 41ba0ed417967bea1af88f4c828a7bae04b586d4749e93b50ce91972ad4e55bd
 SHA512:
-  metadata.gz: ac42f044cd02865cd63d7194b76bbe5111da98bbdf9286ebc1975ff460bad95c22a6070c29238a2c2fa6ffdca2597f8f28aad1e0dfd7a3235c6051e635d16fc1
-  data.tar.gz: 52673c9d24958003fc168ddf02075332aac4d3275c70e73d0d3d86aca54aa6bc17ae2226cbb0c75df8d5537f0f3f208b0ce980c49b3f23160cdab75440f56ee9
+  metadata.gz: 9626b16ccb6c22998e885816359a0c304db95adff5657c69e59cdc83d3549717928b8425f69beee37269e69b8f1152c43330153c88267b0e701faa7b378ab84b
+  data.tar.gz: '089409f803c9dbae0d834795c373617088eef192a1f12bbb719fec2af28cb810884f8c4c3b90275b34fd92f88b04c0152f3c1a400094a3121d5734156033d288'

data/README.md

@@ -1,22 +1,20 @@
 # AyeVar 🏴‍☠️
 
-Arrr, this gem be mighty fresh an’ experimental, me hearties! Th’ API be likely to shift with th’ tides.
+Avast ye, this gem be fresh off th’ plunderin’ an’ experimental as a new recruit, me hearties! Th’ API be shiftin’ like th’ treacherous seas beneath yer barnacled hull.
 
 ## What does it do?
 
-It prevents ye from usin’ undefined instance variables by transformin’ yer code as it be loaded, savvy?
-
-Instance variables must be declared in yer object’s initializer (even if they be initially set to `nil`). Only then can ye access ’em in th’ rest o’ yer code, ye scurvy dog!
+It keelhauls them scurvy undefined instance variables by transformin’ yer code as it loads into th’ hold, savvy? Davy Jones ’imself would approve!
 
 ## Setup
 
-Add this treasure to yer gemfile, arr!
+Add this precious booty to yer gemfile, shiver me timbers!
 
 ```ruby
 gem "aye_var", require: false
 ```
 
-Then require an’ initialize it in yer vessel as early as possible, ye hear? If ye be usin’ Bootsnap, it should be right after Bootsnap, or I’ll make ye walk th’ plank!
+Then be requirin’ an’ initializin’ it in yer galleon as early as ye can hoist sail, ye scallywag! If ye be usin’ Bootsnap, place it right after, or by the powers, ye’ll be swimmin’ with th’ fishes!
 
 ```ruby
 require "aye_var"
@@ -24,4 +22,4 @@ require "aye_var"
 AyeVar.init(include: ["#{Dir.pwd}/**/*"])
 ```
 
-Ye can pass in an array o’ globs to `include:` an’ `exclude:`, ye bilge rat!
+Ye can pass an array o’ globs to `include:` an’ `exclude:`, or I’ll feed ye to th’ kraken, ye mangy bilge-suckin’ swab!

data/lib/aye_var/version.rb

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

data/lib/aye_var.rb

@@ -4,8 +4,6 @@ require "require-hooks/setup"
 require "prism"
 
 module AyeVar
-	METHODS_PERMITTED_FOR_DEFINITION = Set[:initialize, :setup].freeze
-
 	NameError = Class.new(::NameError)
 
 	def self.init(include: [], exclude: [])
@@ -25,7 +23,7 @@ module AyeVar
 			visitor.annotations.sort_by(&:first).reverse_each do |offset, action, name|
 				case action
 				when :start
-					buffer.insert(offset, "((raise ::AyeVar::NameError.new('Undefined instance variable #{name}') unless defined?(#{name}));")
+					buffer.insert(offset, "((raise ::AyeVar::NameError.new('Undefined instance variable #{name}') unless defined?(#{name})); ")
 				when :end
 					buffer.insert(offset, ")")
 				else
@@ -37,9 +35,8 @@ module AyeVar
 		end
 
 		def initialize
-			@definition_context = true
 			@this = nil
-			@context = nil
+			@context = Set[]
 			@annotations = []
 		end
 
@@ -61,16 +58,16 @@ module AyeVar
 			new_context(node) { super }
 		end
 
-		def visit_def_node(node)
-			parent = @this
-
-			new_context(node) do
-				if METHODS_PERMITTED_FOR_DEFINITION.include?(node.name) || Prism::SelfNode === node.receiver || !(Prism::ClassNode === parent)
-					super
-				else
-					prevent_definitions { super }
-				end
+		def visit_defined_node(node)
+			if Prism::InstanceVariableReadNode === node.value
+				context << node.value.name
 			end
+
+			super
+		end
+
+		def visit_def_node(node)
+			new_context(node) { super }
 		end
 
 		def visit_if_node(node)
@@ -105,21 +102,6 @@ module AyeVar
 			super
 		end
 
-		def visit_instance_variable_write_node(node)
-			name = node.name
-
-			unless @definition_context || context.include?(name)
-				location = node.location
-
-				context << name
-
-				@annotations << [location.start_character_offset, :start, name]
-				@annotations << [location.end_character_offset, :end, name]
-			end
-
-			super
-		end
-
 		private def new_context(this)
 			original_this = @this
 			original_context = @context
@@ -150,16 +132,5 @@ module AyeVar
 		private def context
 			@context
 		end
-
-		private def prevent_definitions
-			original_definition_context = @definition_context
-
-			begin
-				@definition_context = false
-				yield
-			ensure
-				@definition_context = original_definition_context
-			end
-		end
 	end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: aye_var
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.2.0
 platform: ruby
 authors:
 - Joel Drapper