ops_team 0.14.3 → 0.17.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 39b95630f778d48b40cb40418de7283acb5b5f47aeabe0d01f0580d6b1c93e22
-  data.tar.gz: 11c608a31bece6e3785bc94c48277c8b2aa6b9227a80517cd7fc46e18b3d5643
+  metadata.gz: 1a18acecc72ca3ac11486856bf71b2c082d81e3dfa65931141c20c1bb4ec8bc5
+  data.tar.gz: 852ea38c8598ca1ed3d806b3f5a334cd191e6484827f7141a265d1324c15fe9d
 SHA512:
-  metadata.gz: 2d891bdf8088dfc4d5c709e6cfe57e4d807aa6993630c6dd1d96335749a475266ecee061b93584e086e5f8f7047e9928f448be1b43237d9ef3bae5f1b2af331c
-  data.tar.gz: 9c974c476d915087da8752dd0a60236401cc1c058418e42835e196d679ed879c97fb73d82e248be2c625fa6619fca6b9a3915e13fd81cfc56ab521b74938ba33
+  metadata.gz: 19c70546affd151d331225af93e807acd07c536eca7f4369a2c36df4961cae845cd0adea9eb7446041d6e45e66a82cd5980f8a5f979393fccf0538406dbc4100
+  data.tar.gz: 471c92c6afd006c0073be40773a568823e37f8795ef74f8aa76e54a4b22a47208187af34274ebe54044d5b59f4f41036621893570f79eab41a035694b771198e

data/lib/action_list.rb

@@ -28,11 +28,15 @@ class ActionList
 
 	private
 
+	def actions_list
+		@actions_list ||= []
+	end
+
 	def process_action_list
 		@actions = {}
 		@aliases = {}
 
-		@actions_list.each do |name, config|
+		actions_list.each do |name, config|
 			action = Action.new(config, @args)
 
 			@actions[name] = action

data/lib/app_config.rb

@@ -8,6 +8,14 @@ class AppConfig
 			new(app_config_path).load
 		end
 
+		def default_filename
+			config_path_for(Environment.environment)
+		end
+
+		def config_path_for(env)
+			"config/#{env}/config.json"
+		end
+
 		private
 
 		def app_config_path
@@ -20,7 +28,7 @@ class AppConfig
 	end
 
 	def initialize(filename = "")
-		@filename = filename.empty? ? default_filename : filename
+		@filename = filename.empty? ? AppConfig.default_filename : filename
 	end
 
 	def load
@@ -31,10 +39,6 @@ class AppConfig
 
 	private
 
-	def default_filename
-		"config/#{environment}/config.json"
-	end
-
 	def config
 		@config ||= file_contents ? YAML.safe_load(file_contents) : {}
 	rescue YAML::SyntaxError => e
@@ -48,8 +52,4 @@ class AppConfig
 			nil
 		end
 	end
-
-	def environment
-		ENV['environment']
-	end
 end

data/lib/builtin.rb

@@ -1,6 +1,8 @@
 # frozen_string_literal: true
 
 class Builtin
+	class ArgumentError < StandardError; end
+
 	attr_reader :args, :config
 
 	class << self

data/lib/builtins/envdiff.rb

@@ -0,0 +1,129 @@
+# frozen_string_literal: true
+
+require 'builtin'
+
+module Builtins
+	class EnvDiff < Builtin
+		class << self
+			def description
+				"compares keys present in config and secrets between different environments"
+			end
+		end
+
+		def run
+			check_args
+
+			if source_only_keys.empty? && dest_only_keys.empty?
+				Output.out("Environments '#{source_env}' and '#{dest_env}' define the same #{source_keys.length} key(s).")
+				return
+			end
+
+			output_key_summary(source_only_keys, source_env, dest_env) if source_only_keys.any?
+			output_key_summary(dest_only_keys, dest_env, source_env) if dest_only_keys.any?
+		end
+
+		private
+
+		def output_key_summary(keys, in_env, not_in_env)
+			Output.warn("Environment '#{in_env}' defines keys that '#{not_in_env}' does not:\n")
+			keys.each do |key|
+				Output.warn("   - #{key}")
+			end
+			Output.out("")
+		end
+
+		def source_only_keys
+			@source_only_keys ||= source_keys - dest_keys
+		end
+
+		def dest_only_keys
+			@dest_only_keys ||= dest_keys - source_keys
+		end
+
+		def source_keys
+			@source_keys ||= keys_for(source_env)
+		end
+
+		def dest_keys
+			@dest_keys ||= keys_for(dest_env)
+		end
+
+		def keys_for(env)
+			tagged_config_keys_for(env) + tagged_secrets_keys_for(env)
+		end
+
+		def tagged_config_keys_for(env)
+			config_keys_for(env).map do |key|
+				"[CONFIG] #{key}"
+			end
+		end
+
+		def tagged_secrets_keys_for(env)
+			secrets_keys_for(env).map do |key|
+				"[SECRET] #{key}"
+			end
+		end
+
+		def config_keys_for(env)
+			(config_for(env)["environment"]&.keys || []) - ignored_keys
+		end
+
+		def secrets_keys_for(env)
+			(secrets_for(env)["environment"]&.keys || []) - ignored_keys
+		end
+
+		def config_for(env)
+			YAML.load_file(config_path_for(env))
+		end
+
+		def secrets_for(env)
+			YAML.load_file(secrets_path_for(env))
+		end
+
+		def check_args
+			raise Builtin::ArgumentError, "Usage: ops envdiff <env_one> <env_two>" unless args.length == 2
+
+			check_environment(source_env)
+			check_environment(dest_env)
+		end
+
+		def source_env
+			args[0]
+		end
+
+		def dest_env
+			args[1]
+		end
+
+		def check_environment(name)
+			raise_missing_file_error(config_path_for(name)) unless config_file_exists?(name)
+			raise_missing_file_error(secrets_path_for(name)) unless secrets_file_exists?(name)
+		end
+
+		def raise_missing_file_error(path)
+			raise Builtin::ArgumentError, "File '#{path}' does not exist."
+		end
+
+		def config_file_exists?(env)
+			File.exist?(config_path_for(env))
+		end
+
+		def secrets_file_exists?(env)
+			File.exist?(secrets_path_for(env))
+		end
+
+		def config_path_for(env)
+			AppConfig.config_path_for(env)
+		end
+
+		def secrets_path_for(env)
+			Secrets.config_path_for(env)
+		end
+
+		def ignored_keys
+			Options.get("envdiff.ignored_keys") || []
+		end
+	end
+
+	Envdiff = EnvDiff
+end

data/lib/builtins/help.rb

@@ -54,12 +54,18 @@ module Builtins
 		def actions
 			return [] unless @config["actions"]
 
-			@config["actions"].map do |name, value|
-				format("%<name>-35s %<desc>s",
-					name: name.yellow,
-					desc: value["description"] || value["command"]
+			@config["actions"].map do |name, action_config|
+				format("%<name>-40s %<desc>s",
+					name: "#{name.yellow} #{alias_string_for(action_config)}",
+					desc: action_config["description"] || action_config["command"]
 				)
 			end.sort
 		end
+
+		def alias_string_for(action_config)
+			return "[#{action_config["alias"]}]" if action_config["alias"]
+
+			""
+		end
 	end
 end

data/lib/environment.rb

@@ -1,8 +1,18 @@
 # frozen_string_literal: true
 
 require 'version'
+require 'secrets'
+require 'app_config'
 
 class Environment
+	class << self
+		def environment
+			return 'dev' if ENV['environment'].nil? || ENV['environment'].empty?
+
+			ENV['environment']
+		end
+	end
+
 	def initialize(env_hash)
 		@env_hash = env_hash
 	end
@@ -13,22 +23,18 @@ class Environment
 		set_configured_variables
 	end
 
-	def environment
-		return 'dev' if ENV['environment'].nil? || ENV['environment'].empty?
-
-		ENV['environment']
-	end
-
 	private
 
 	def set_ops_variables
 		ENV["OPS_YML_DIR"] = Dir.pwd
 		ENV["OPS_VERSION"] = Version.version.to_s
+		ENV["OPS_SECRETS_FILE"] = Secrets.config_path_for(Environment.environment)
+		ENV["OPS_CONFIG_FILE"] = AppConfig.config_path_for(Environment.environment)
 	end
 
 	def set_environment_aliases
 		environment_aliases.each do |alias_name|
-			ENV[alias_name] = environment
+			ENV[alias_name] = Environment.environment
 		end
 	end
 

data/lib/ops.rb

@@ -28,6 +28,7 @@ class Ops
 	ERROR_LOADING_APP_CONFIG_EXIT_CODE = 66
 	MIN_VERSION_NOT_MET_EXIT_CODE = 67
 	ACTION_CONFIG_ERROR_EXIT_CODE = 68
+	BUILTIN_SYNTAX_ERROR_EXIT_CODE = 69
 
 	RECOMMEND_HELP_TEXT = "Run 'ops help' for a list of builtins and actions."
 
@@ -108,6 +109,9 @@ class Ops
 		do_before_action
 		Output.notice("Running '#{action}' from #{CONFIG_FILE} in environment '#{ENV['environment']}'...")
 		action.run
+	rescue Builtin::ArgumentError => e
+		Output.error("Error running builtin '#{@action_name}': #{e}")
+		exit(BUILTIN_SYNTAX_ERROR_EXIT_CODE)
 	rescue AppConfig::ParsingError => e
 		Output.error("Error parsing app config: #{e}")
 		exit(ERROR_LOADING_APP_CONFIG_EXIT_CODE)
@@ -157,8 +161,12 @@ class Ops
 
 	def config
 		@config ||= begin
-			Output.warn("File '#{CONFIG_FILE}' does not exist.") unless File.exist?(CONFIG_FILE)
-			YAML.load_file(CONFIG_FILE)
+			if File.exist?(CONFIG_FILE)
+				YAML.load_file(CONFIG_FILE)
+			else
+				Output.warn("File '#{CONFIG_FILE}' does not exist.") unless @action_name == "init"
+				{}
+			end
 		rescue StandardError => e
 			Output.warn("Error parsing '#{CONFIG_FILE}': #{e}")
 			{}
@@ -166,7 +174,7 @@ class Ops
 	end
 
 	def env_vars
-		@config.dig("options", "environment") || {}
+		config.dig("options", "environment") || {}
 	end
 
 	def environment

data/lib/secrets.rb

@@ -9,31 +9,35 @@ require 'options'
 
 class Secrets < AppConfig
 	class << self
+		def default_filename
+			config_path_for(Environment.environment)
+		end
+
+		def config_path_for(env)
+			File.exist?(ejson_path_for(env)) ? ejson_path_for(env) : json_path_for(env)
+		end
+
 		private
 
+		def ejson_path_for(env)
+			"config/#{env}/secrets.ejson"
+		end
+
+		def json_path_for(env)
+			"config/#{env}/secrets.json"
+		end
+
 		def app_config_path
 			expand_path(Options.get("secrets.path"))
 		end
 	end
 
 	def initialize(filename = "")
-		@filename = filename.empty? ? default_filename : actual_filename_for(filename)
+		@filename = filename.empty? ? Secrets.default_filename : actual_filename_for(filename)
 	end
 
 	private
 
-	def default_filename
-		File.exist?(default_ejson_filename) ? default_ejson_filename : default_json_filename
-	end
-
-	def default_ejson_filename
-		"config/#{environment}/secrets.ejson"
-	end
-
-	def default_json_filename
-		"config/#{environment}/secrets.json"
-	end
-
 	def actual_filename_for(filename)
 		File.exist?(filename) ? filename : filename.sub(".ejson", ".json")
 	end

data/ops_team.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |s|
 	s.name = 'ops_team'
-	s.version = '0.14.3'
+	s.version = '0.17.1'
 	s.authors = [
 		'nickthecook@gmail.com'
 	]

metadata

@@ -1,11 +1,11 @@
 --- !ruby/object:Gem::Specification
 name: ops_team
 version: !ruby/object:Gem::Version
-  version: 0.14.3
+  version: 0.17.1
 platform: ruby
 authors:
 - nickthecook@gmail.com
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
 date: 2020-08-12 00:00:00.000000000 Z
@@ -130,8 +130,8 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 1.1.6
-description: 
-email: 
+description:
+email:
 executables:
 - ops
 extensions: []
@@ -154,6 +154,7 @@ files:
 - lib/builtins/background_log.rb
 - lib/builtins/down.rb
 - lib/builtins/env.rb
+- lib/builtins/envdiff.rb
 - lib/builtins/exec.rb
 - lib/builtins/help.rb
 - lib/builtins/helpers/dependency_handler.rb
@@ -186,7 +187,7 @@ homepage: https://github.com/nickthecook/ops
 licenses:
 - GPL-3.0-only
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -202,7 +203,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.0.3
-signing_key: 
+signing_key:
 specification_version: 4
 summary: ops_team handles basic operations tasks for your project, driven by YAML
   config