yaml_smith 0.1.0 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d7ff73d942d1f809bbac011b168ef43cd3ce7b8d193292c661e6d2d65a358498
-  data.tar.gz: 4e6fe2d3391a53d6fd9d364d0b87061342409033ba9be4e29520eb8ab75b4e20
+  metadata.gz: 03e70b6c6bd6522e6a35265fc2906e5fdc05e1e5a3bb497024e7805eb89cdc2e
+  data.tar.gz: a3b8572f3a9ead73fc96d1eb4795a995ac17a2ade613dc54334ee8c7b2e59843
 SHA512:
-  metadata.gz: 0da49b2b19add33c7a5137624bb6347c39c87ea6bc203c7ad416a77e026bf3c387b3ca5f0fbb10e550716f50c6729592c4ffa8b1204ef255d808e5c15782baf4
-  data.tar.gz: 241861d560bdce07bdfe246c11618f83750a00f31d338a140c820147faf9482df223ff115df8fcf0fe6b29db93f4d1544cb82efd19be7422ba304a8b59662609
+  metadata.gz: 49168721a967966498b840fc55af20beaad3be787c756cfb28aa4d08c55011f1d8cfbef4f0af5216278aceee1efce14c2f89b9cb39ff24d6bc0379e57afe09cb
+  data.tar.gz: b7b6467e1f366f016ca038e5a7ecd2e54b55f72b9e6b071bd53ffaf2edf0726f46faa3b635d9dbb5e55d13b6e4e93ee943f52fec3bc95493c2404330b0bec03c

data/AGENTS.md

@@ -20,3 +20,4 @@
 - Load existing files and values with `comments: true`.
 - Values passed on the command line are YAML and should retain their YAML types.
 - File edits are written through a same-directory temporary file and rename; preserve this atomic-write behavior.
+- In tests, use squiggly heredocs (`<<~YAML`) for multiline YAML fixtures and expected YAML output so the structure remains readable.

data/README.md

@@ -34,12 +34,25 @@ Set or replace a top-level key:
 yaml-smith set config.yml timeout 60
 ```
 
-Delete a top-level key:
+Delete matching keys recursively:
 
 ```bash
 yaml-smith delete config.yml timeout
 ```
 
+Delete searches mappings nested inside arrays and other mappings. For example,
+this removes the `WEB_CONCURRENCY` entry from a nested `envVars` list:
+
+```bash
+yaml-smith delete render.yaml WEB_CONCURRENCY
+```
+
+Use `--top-level-only` to restrict deletion to the document's top-level keys:
+
+```bash
+yaml-smith delete config.yml timeout --top-level-only
+```
+
 ## Development
 
 After checking out the repo, run `bin/setup` to install dependencies. Then,

data/Rakefile

@@ -1,11 +1,11 @@
 # frozen_string_literal: true
 
-require "bundler/gem_tasks"
-require "minitest/test_task"
+require 'bundler/gem_tasks'
+require 'minitest/test_task'
 
 Minitest::TestTask.create
 
-require "rubocop/rake_task"
+require 'rubocop/rake_task'
 
 RuboCop::RakeTask.new
 

data/lib/yaml_smith/cli.rb

@@ -10,7 +10,7 @@ module YamlSmith
       'set' => SetKey,
       'delete' => DeleteKey
     }.freeze
-    USAGE = 'Usage: yaml-smith COMMAND FILE KEY [VALUE]'
+    USAGE = 'Usage: yaml-smith COMMAND FILE KEY [VALUE] [--top-level-only]'
 
     def self.call(arguments)
       new(arguments).call
@@ -24,7 +24,7 @@ module YamlSmith
       command = COMMANDS[@arguments.first]
       return usage unless valid_arguments?(command)
 
-      command.call(*command_arguments)
+      command.call(*command_arguments, **command_options)
     rescue Error => e
       warn e.message
       1
@@ -43,7 +43,13 @@ module YamlSmith
     end
 
     def command_arguments
-      @arguments.drop(1)
+      @arguments.drop(1).reject { |argument| argument == '--top-level-only' }
+    end
+
+    def command_options
+      return {} unless delete_command?
+
+      { top_level_only: @arguments.include?('--top-level-only') }
     end
 
     def usage

data/lib/yaml_smith/delete_key.rb

@@ -1,16 +1,33 @@
 # frozen_string_literal: true
 
 module YamlSmith
-  # Deletes a top-level key from a YAML file.
+  # Deletes matching keys from a YAML file.
   class DeleteKey < FileCommand
-    def call
+    def call(top_level_only: false)
       document = load_file(@path)
 
       ensure_mapping(document)
-      raise Error, "key does not exist: #{@key}" unless document.key?(@key)
+      deleted = top_level_only ? !document.delete(@key).nil? : delete_key(document)
+      raise Error, "key does not exist: #{@key}" unless deleted
 
-      document.delete(@key)
       write(Psych::Pure.dump(document))
     end
+
+    private
+
+    def delete_key(value)
+      return delete_from_array(value) if value.instance_of?(Psych::Pure::LoadedArray)
+      return false unless value.instance_of?(Psych::Pure::LoadedHash)
+
+      deleted = value.key?(@key)
+      value.delete(@key)
+      value.each_value { |child| deleted = delete_key(child) || deleted }
+      deleted
+    end
+
+    def delete_from_array(value)
+      deleted = value.delete_if { |child| child.instance_of?(Psych::Pure::LoadedHash) && child.key?(@key) }.any?
+      value.any? { |child| delete_key(child) } || deleted
+    end
   end
 end

data/lib/yaml_smith/version.rb

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: yaml_smith
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
 - Andy Waite