featureparity 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 695b5f522e34a1e886e07177cbbe5ca87b515caf6845e15b50ca9e1c4fb3735e
-  data.tar.gz: 18ff57dfb177341dd4a6f2d8dc05e862a53d8e24eb0c2da4172e022986533cc1
+  metadata.gz: 7fec4ef9bda94cba09b19e5209ed867ff87d6a5312c6c6441234691e406bd2a2
+  data.tar.gz: 7fd7a58df29f9a69ce342ed60e5b7ff60b796f286404b81a56108be1f5c83f1b
 SHA512:
-  metadata.gz: 9851ef3ea41669276c356fbe2e5a200ec06d59cea82a9d852eaca71a20297319b4eeb32e5a684921cfba3b298bf1e8931f45bc89ad0c59d802128a5f5940b157
-  data.tar.gz: 1e075a99c1b02368908639649ff45ba78be33f8e0a03756b0f7782275758f326e01992488508c6930d1f5ac65e47eb6ef7b5e24438482e6eac2b855fe06b2557
+  metadata.gz: b198114877fb8ca6695f14c38796334b2cf4269e4164509a97a7b0d12cf4fa7f86ecbd4036028fad15fb80eb00d95da5a410542eda6cdcbd7366b33e6c0dab11
+  data.tar.gz: 7a6a248d554a4fa13bc5b33591be0868f2714dfc483c81da0ceafaa4fa951f5f915c229ac8096fbda59efae8319c39e135e4b36e4403944c6ceec80cc7f74832

data/lib/fp/cli.rb

@@ -12,6 +12,7 @@ module Fp
       'report' => Commands::Report,
       'matrix' => Commands::Matrix,
       'profile' => Commands::Profile,
+      'config' => Commands::ConfigCmd,
       'setup' => Commands::Setup,
       'version' => Commands::Version,
       'help' => Commands::Help
@@ -32,17 +33,19 @@ module Fp
           exit 1
         end
 
-        # Profile commands don't need auth
-        needs_auth = !%w[profile setup version help].include?(command_name)
+        # Profile/config commands don't need auth
+        needs_auth = !%w[profile config setup version help].include?(command_name)
 
         if needs_auth
-          config = Config.new(profile: global_opts[:profile])
+          config = Config.new(profile: global_opts[:profile], api_url: global_opts[:api_url])
           unless config.valid?
             output = Output.new(json: global_opts[:json])
             output.error(config.validation_error.strip)
             exit 1
           end
           client = Client.new(config)
+        else
+          config = Config.new(profile: global_opts[:profile], api_url: global_opts[:api_url])
         end
 
         output = Output.new(json: global_opts[:json])
@@ -60,7 +63,7 @@ module Fp
       private
 
       def extract_global_options(args)
-        opts = { json: false, profile: nil }
+        opts = { json: false, profile: nil, api_url: nil }
 
         # Extract --json flag
         if args.delete('--json')
@@ -89,6 +92,17 @@ module Fp
           opts[:profile] = args.delete_at(idx)
         end
 
+        # Extract --api-url flag
+        if (idx = args.index('--api-url'))
+          args.delete_at(idx)
+          value = args[idx]
+          if value.nil? || value.start_with?('-')
+            $stderr.puts "Error: --api-url requires a value"
+            exit 1
+          end
+          opts[:api_url] = args.delete_at(idx)
+        end
+
         opts
       end
     end

data/lib/fp/commands/config_cmd.rb

@@ -0,0 +1,130 @@
+# frozen_string_literal: true
+
+module Fp
+  module Commands
+    # fp config get|set|unset|list
+    # Manage global CLI settings (stored in ~/.config/fp/config.yml)
+    class ConfigCmd < Base
+      ALLOWED_SETTINGS = %w[api_url].freeze
+
+      def run(args)
+        subcommand = args.shift
+
+        case subcommand
+        when 'set'
+          set_setting(args)
+        when 'get'
+          get_setting(args)
+        when 'unset'
+          unset_setting(args)
+        when 'list'
+          list_settings
+        else
+          show_usage
+        end
+      end
+
+      private
+
+      def set_setting(args)
+        key = args.shift
+        value = args.shift
+
+        unless key
+          output.error('Usage: fp config set <key> <value>')
+          exit 1
+        end
+
+        unless ALLOWED_SETTINGS.include?(key)
+          output.error("Unknown setting: '#{key}'. Allowed: #{ALLOWED_SETTINGS.join(', ')}")
+          exit 1
+        end
+
+        unless value
+          output.error("Value is required. Usage: fp config set #{key} <value>")
+          exit 1
+        end
+
+        Config.set_setting(key, value)
+
+        output.success({ key: key, value: value }) do
+          puts "Set #{key} = #{value}"
+        end
+      end
+
+      def get_setting(args)
+        key = args.shift
+
+        unless key
+          output.error('Usage: fp config get <key>')
+          exit 1
+        end
+
+        value = Config.get_setting(key)
+
+        if value
+          output.success({ key: key, value: value }) do
+            puts value
+          end
+        else
+          output.success({ key: key, value: nil }) do
+            puts "(not set)"
+          end
+        end
+      end
+
+      def unset_setting(args)
+        key = args.shift
+
+        unless key
+          output.error('Usage: fp config unset <key>')
+          exit 1
+        end
+
+        Config.set_setting(key, nil)
+
+        output.success({ key: key }) do
+          puts "Unset #{key}"
+        end
+      end
+
+      def list_settings
+        settings = {}
+        ALLOWED_SETTINGS.each do |key|
+          value = Config.get_setting(key)
+          settings[key] = value if value
+        end
+
+        output.success({ settings: settings }) do
+          if settings.empty?
+            puts "No global settings configured."
+            puts
+            puts "Set one with:"
+            puts "  fp config set api_url https://api.dev.featureparity.dev"
+          else
+            puts "Global settings:"
+            settings.each { |k, v| puts "  #{k} = #{v}" }
+          end
+        end
+      end
+
+      def show_usage
+        output.error('Usage: fp config <set|get|unset|list>')
+        puts
+        puts 'Commands:'
+        puts '  fp config set <key> <value>   Set a global setting'
+        puts '  fp config get <key>           Get a setting value'
+        puts '  fp config unset <key>         Remove a setting'
+        puts '  fp config list                Show all global settings'
+        puts
+        puts 'Available settings:'
+        puts '  api_url   Base URL for the FeatureParity API'
+        puts
+        puts 'Examples:'
+        puts '  fp config set api_url https://api.dev.featureparity.dev'
+        puts '  fp config get api_url'
+        exit 1
+      end
+    end
+  end
+end

data/lib/fp/commands/help.rb

@@ -20,6 +20,7 @@ module Fp
             report <slug> --project <slug>... Report evidence for a requirement
             matrix --project <slug>           Show the parity matrix
             profile add|list                  Manage named profiles
+            config set|get|unset|list         Manage global settings
             setup                             Interactive first-time setup
             version                           Show version
             help                              Show this help
@@ -28,6 +29,7 @@ module Fp
             --json                            Output as JSON envelope
             --profile <name>                  Use a named profile
             -p <name>                         Alias for --profile
+            --api-url <url>                   Override API URL for this command
 
           AUTHENTICATION
             Set FP_API_KEY environment variable, or use a profile:

data/lib/fp/commands/list.rb

@@ -52,11 +52,36 @@ module Fp
           if requirements.empty?
             puts "No #{status_filter} requirements found."
           else
+            # Group into tree structure: parents first, then children indented
+            parents = requirements.select { |r| r['parent_id'].nil? || r['parent_id'] == '' }
+            children_by_parent = requirements
+              .select { |r| r['parent_id'] && r['parent_id'] != '' }
+              .group_by { |r| r['parent_id'] }
+
+            # Orphaned children (parent not in filtered set) show at top level
+            parent_ids = parents.map { |p| p['id'] }.to_set
+            orphans = requirements.select { |r| r['parent_id'] && r['parent_id'] != '' && !parent_ids.include?(r['parent_id']) }
+
             headers = %w[SLUG TITLE SURFACES STATUS]
-            rows = requirements.map do |r|
+            rows = []
+
+            parents.each do |r|
               surfaces = (r['required_surfaces'] || []).join(', ')
-              [r['slug'], truncate(r['title'], 40), surfaces.empty? ? '-' : surfaces, r['status']]
+              rows << [r['slug'], truncate(r['title'], 40), surfaces.empty? ? '-' : surfaces, r['status']]
+
+              # Add children indented
+              (children_by_parent[r['id']] || []).each do |child|
+                child_surfaces = (child['required_surfaces'] || []).join(', ')
+                rows << ["  └ #{child['slug']}", truncate(child['title'], 36), child_surfaces.empty? ? '-' : child_surfaces, child['status']]
+              end
             end
+
+            # Show orphans at top level
+            orphans.each do |r|
+              surfaces = (r['required_surfaces'] || []).join(', ')
+              rows << [r['slug'], truncate(r['title'], 40), surfaces.empty? ? '-' : surfaces, r['status']]
+            end
+
             output.table(headers, rows)
           end
         end

data/lib/fp/commands/matrix.rb

@@ -68,8 +68,6 @@ module Fp
             output.table(headers, rows)
             puts
             puts 'Legend: ✅ = passing, ⬜ = required (no evidence), - = not required'
-            puts
-            puts '(Evidence status will show when #1214 is complete)'
           end
         end
       end

data/lib/fp/commands/propose.rb

@@ -2,7 +2,7 @@
 
 module Fp
   module Commands
-    # fp propose --project <slug> --slug X --name "..." --why "..." --required a,b,c --acceptance "..."
+    # fp propose --project <slug> --slug X --name "..." --why "..." --required a,b,c --acceptance "..." --parent <slug>
     # Create a new requirement as draft
     class Propose < Base
       KNOWN_FLAGS = {
@@ -11,7 +11,8 @@ module Fp
         name: :string,
         why: :string,
         required: :string,
-        acceptance: :string
+        acceptance: :string,
+        parent: :string
       }.freeze
 
       def run(args)
@@ -38,6 +39,13 @@ module Fp
         }
         params[:why] = opts[:why] if opts[:why]
         params[:acceptance] = opts[:acceptance] if opts[:acceptance]
+
+        # Resolve parent slug to parent_id if provided
+        if opts[:parent]
+          parent_id = resolve_parent_id(workspace_id, opts[:parent])
+          params[:parent_id] = parent_id
+        end
+
         # Note: status is NOT sent - the API enforces draft for agents
 
         result = client.create_requirement(params)
@@ -55,12 +63,34 @@ module Fp
           puts "  Title:    #{requirement['title']}"
           puts "  Status:   #{requirement['status']}"
           puts "  Surfaces: #{(requirement['required_surfaces'] || []).join(', ')}"
+          puts "  Parent:   #{opts[:parent] || '(none)'}" if opts[:parent]
           puts
           puts '⚠️  This requirement is a DRAFT.'
           puts '   A human must activate it in the web app before it appears in the parity matrix.'
           puts '   Agents cannot activate requirements.'
         end
       end
+
+      private
+
+      # Resolve a parent requirement slug to its ID
+      def resolve_parent_id(workspace_id, parent_slug)
+        result = client.list_requirements(workspace_id: workspace_id)
+        unless result[:ok]
+          output.error("Failed to resolve parent slug '#{parent_slug}': #{result[:error]}")
+          exit 1
+        end
+
+        requirements = result[:data]['requirements'] || []
+        parent = requirements.find { |r| r['slug'] == parent_slug }
+
+        unless parent
+          output.error("Parent requirement '#{parent_slug}' not found in this workspace")
+          exit 1
+        end
+
+        parent['id']
+      end
     end
   end
 end

data/lib/fp/commands/setup.rb

@@ -22,13 +22,16 @@ module Fp
 
         api_key = resolve_api_key(opts)
         profile_name = resolve_profile_name(opts)
-        api_url = opts[:api_url]
+        api_url = opts[:api_url] || Config.get_setting('api_url')
 
         puts "Validating API key..."
         validate_key!(api_key, api_url)
 
         save_profile!(profile_name, api_key, api_url)
 
+        effective_url = api_url || Config::DEFAULT_API_URL
+        domain = URI.parse(effective_url).host.sub(/^api\./, '') rescue 'featureparity.dev'
+
         puts
         puts "✅ Setup complete!"
         puts
@@ -54,7 +57,11 @@ module Fp
           exit 1
         end
 
-        puts "Get your API key from: https://featureparity.dev"
+        api_url = opts[:api_url] || Config.get_setting('api_url')
+        effective_url = api_url || Config::DEFAULT_API_URL
+        domain = URI.parse(effective_url).host.sub(/^api\./, '') rescue 'featureparity.dev'
+
+        puts "Get your API key from: https://#{domain}"
         puts "(Workspace Settings → API Keys → Mint Key)"
         puts
         print "Paste your API key (starts with fp_): "

data/lib/fp/commands/show.rb

@@ -38,6 +38,7 @@ module Fp
         puts
         puts "Slug:     #{req['slug']}"
         puts "Status:   #{req['status']}"
+        puts "Parent:   #{req['parent_id'] || '(none)'}"
         puts
 
         surfaces = req['required_surfaces'] || []

data/lib/fp/commands.rb

@@ -9,6 +9,7 @@ require_relative 'commands/propose'
 require_relative 'commands/report'
 require_relative 'commands/matrix'
 require_relative 'commands/profile'
+require_relative 'commands/config_cmd'
 require_relative 'commands/setup'
 require_relative 'commands/version'
 require_relative 'commands/help'

data/lib/fp/config.rb

@@ -5,7 +5,17 @@ require 'fileutils'
 
 module Fp
   # Handles configuration: profiles, API keys, API URL.
-  # Priority: FP_API_KEY env var > --profile flag > FP_PROFILE env var > default profile
+  #
+  # API URL resolution priority (highest wins):
+  #   1. --api-url CLI flag (passed via constructor)
+  #   2. FP_API_URL env var
+  #   3. Profile-specific api_url (in profile config)
+  #   4. Global api_url setting (top-level in config.yml)
+  #   5. DEFAULT_API_URL
+  #
+  # API key resolution priority:
+  #   1. FP_API_KEY env var
+  #   2. Profile api_key (--profile flag > FP_PROFILE env var)
   class Config
     DEFAULT_API_URL = 'https://api.featureparity.dev'
     CONFIG_DIR = File.expand_path('~/.config/fp')
@@ -13,8 +23,9 @@ module Fp
 
     attr_reader :api_key, :api_url, :profile_name
 
-    def initialize(profile: nil)
+    def initialize(profile: nil, api_url: nil)
       @profile_name = resolve_profile_name(profile)
+      @explicit_api_url = api_url
       @api_key = resolve_api_key
       @api_url = resolve_api_url
     end
@@ -33,34 +44,51 @@ module Fp
     end
 
     class << self
-      def load_profiles
+      def load_config
         return {} unless File.exist?(CONFIG_FILE)
 
-        config = YAML.safe_load_file(CONFIG_FILE, permitted_classes: [Symbol]) || {}
-        config['profiles'] || {}
+        YAML.safe_load_file(CONFIG_FILE, permitted_classes: [Symbol]) || {}
       rescue StandardError => e
         warn "Warning: Could not load config file: #{e.message}"
         {}
       end
 
+      def load_profiles
+        config = load_config
+        config['profiles'] || {}
+      end
+
       def save_profile(name, api_key:, api_url: nil)
         FileUtils.mkdir_p(CONFIG_DIR)
+        ensure_config_file!
 
-        # Ensure config file has restricted permissions
-        unless File.exist?(CONFIG_FILE)
-          File.write(CONFIG_FILE, "---\nprofiles: {}\n")
-          File.chmod(0o600, CONFIG_FILE)
-        end
-
-        config = YAML.safe_load_file(CONFIG_FILE, permitted_classes: [Symbol]) || {}
+        config = load_config
         config['profiles'] ||= {}
-        config['profiles'][name] = {
-          'api_key' => api_key
-        }
+        config['profiles'][name] = { 'api_key' => api_key }
         config['profiles'][name]['api_url'] = api_url if api_url
 
-        File.write(CONFIG_FILE, YAML.dump(config))
-        File.chmod(0o600, CONFIG_FILE)
+        write_config!(config)
+      end
+
+      # Get a global setting (top-level key in config.yml)
+      def get_setting(key)
+        config = load_config
+        config[key.to_s]
+      end
+
+      # Set a global setting (top-level key in config.yml)
+      def set_setting(key, value)
+        FileUtils.mkdir_p(CONFIG_DIR)
+        ensure_config_file!
+
+        config = load_config
+        if value.nil?
+          config.delete(key.to_s)
+        else
+          config[key.to_s] = value
+        end
+
+        write_config!(config)
       end
 
       def list_profiles
@@ -70,12 +98,25 @@ module Fp
       def profile_exists?(name)
         load_profiles.key?(name)
       end
+
+      private
+
+      def ensure_config_file!
+        return if File.exist?(CONFIG_FILE)
+
+        File.write(CONFIG_FILE, "---\nprofiles: {}\n")
+        File.chmod(0o600, CONFIG_FILE)
+      end
+
+      def write_config!(config)
+        File.write(CONFIG_FILE, YAML.dump(config))
+        File.chmod(0o600, CONFIG_FILE)
+      end
     end
 
     private
 
     def resolve_profile_name(explicit_profile)
-      # Explicit --profile flag takes precedence over FP_PROFILE env var
       explicit_profile || ENV['FP_PROFILE']
     end
 
@@ -94,17 +135,24 @@ module Fp
     end
 
     def resolve_api_url
-      # FP_API_URL env var takes precedence
+      # 1. Explicit --api-url flag
+      return @explicit_api_url if @explicit_api_url && !@explicit_api_url.empty?
+
+      # 2. FP_API_URL env var
       return ENV['FP_API_URL'] if ENV['FP_API_URL'] && !ENV['FP_API_URL'].empty?
 
-      # Then profile-specific URL
+      # 3. Profile-specific URL
       if @profile_name
         profiles = self.class.load_profiles
         profile = profiles[@profile_name]
         return profile['api_url'] if profile && profile['api_url']
       end
 
-      # Finally, default
+      # 4. Global api_url setting
+      global_url = self.class.get_setting('api_url')
+      return global_url if global_url && !global_url.empty?
+
+      # 5. Default
       DEFAULT_API_URL
     end
   end

data/lib/fp/output.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require 'json'
-require 'csv'
 
 module Fp
   # Output formatting for CLI responses
@@ -61,6 +60,7 @@ module Fp
     end
 
     def csv(headers, rows)
+      require 'csv'
       csv_string = CSV.generate do |csv|
         csv << headers
         rows.each { |row| csv << row }

data/lib/fp/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Fp
-  VERSION = '0.0.1'
+  VERSION = '0.0.2'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: featureparity
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - Stowzilla
@@ -95,6 +95,7 @@ files:
 - lib/fp/client.rb
 - lib/fp/commands.rb
 - lib/fp/commands/base.rb
+- lib/fp/commands/config_cmd.rb
 - lib/fp/commands/help.rb
 - lib/fp/commands/list.rb
 - lib/fp/commands/matrix.rb