slk 0.4.2 → 0.6.0

data/lib/slk/services/user_picker.rb

@@ -0,0 +1,68 @@
+# frozen_string_literal: true
+
+module Slk
+  module Services
+    # Disambiguates between multiple matching users. Prompts at a TTY; raises
+    # ApiError in non-interactive contexts so callers don't silently get the
+    # wrong user when name resolution is ambiguous.
+    class UserPicker
+      def initialize(stdin: $stdin, prompt_io: $stderr)
+        @stdin = stdin
+        @prompt_io = prompt_io
+      end
+
+      def pick(matches)
+        return matches.first['id'] if matches.size == 1
+
+        unless interactive?
+          raise ApiError,
+                "Ambiguous match (#{matches.size} users): #{ids(matches).join(', ')}. " \
+                'Use --pick N or --all to disambiguate non-interactively.'
+        end
+
+        list(matches)
+        matches[read_index(matches.size)]['id']
+      end
+
+      private
+
+      def interactive?
+        @stdin.respond_to?(:tty?) && @stdin.tty?
+      end
+
+      def list(matches)
+        @prompt_io.puts('Multiple users match — pick one:')
+        matches.each_with_index { |u, i| @prompt_io.puts("  [#{i + 1}] #{describe(u)}") }
+      end
+
+      def describe(user)
+        profile = user['profile'] || {}
+        name = profile['real_name'] || profile['display_name'] || user['name']
+        suffix = profile['title'].to_s.empty? ? '' : " — #{profile['title']}"
+        "#{name} (#{user['id']})#{suffix}#{flag_suffix(user)}"
+      end
+
+      def flag_suffix(user)
+        flags = []
+        flags << 'deactivated' if user['deleted']
+        flags << 'bot' if user['is_bot']
+        flags.empty? ? '' : " [#{flags.join(', ')}]"
+      end
+
+      def ids(matches)
+        matches.map { |u| u['id'] }
+      end
+
+      def read_index(count)
+        loop do
+          @prompt_io.print("Choice [1-#{count}]: ")
+          choice = @stdin.gets&.strip
+          raise ApiError, 'No selection made' if choice.nil? || choice.empty?
+
+          n = Integer(choice, exception: false)
+          return n - 1 if n&.between?(1, count)
+        end
+      end
+    end
+  end
+end

data/lib/slk/services/who_target_resolver.rb

@@ -0,0 +1,64 @@
+# frozen_string_literal: true
+
+module Slk
+  module Services
+    # Resolves the positional target for `slk who` into one or more user_ids.
+    class WhoTargetResolver
+      def initialize(workspace:, cache_store:, api_client:, output:, options:)
+        @workspace = workspace
+        @cache_store = cache_store
+        @api_client = api_client
+        @output = output
+        @options = options
+      end
+
+      def resolve(target)
+        return [self_user_id] if target.nil? || target == 'me'
+        return [target] if target.match?(/\A[UW][A-Z0-9]+\z/)
+
+        ids = resolve_by_name(target.delete_prefix('@'))
+        ids || (raise ApiError, "Could not resolve user: #{target}")
+      end
+
+      private
+
+      def resolve_by_name(name)
+        matches = lookup.find_all_by_name(name)
+        return select(matches) if matches.any?
+
+        cached = @cache_store.get_user_id_by_name(@workspace.name, name)
+        cached ? [cached] : nil
+      end
+
+      def select(matches)
+        return matches.map { |u| u['id'] } if @options[:all]
+        return [pick_by_index(matches)] if @options[:pick]
+
+        [UserPicker.new.pick(matches)]
+      end
+
+      def pick_by_index(matches)
+        idx = @options[:pick]
+        raise ApiError, "--pick #{idx} out of range (got #{matches.size} matches)" unless idx&.between?(1, matches.size)
+
+        matches[idx - 1]['id']
+      end
+
+      def lookup
+        UserLookup.new(
+          cache_store: @cache_store, workspace: @workspace,
+          api_client: @api_client, on_debug: ->(msg) { @output.debug(msg) }
+        )
+      end
+
+      def self_user_id
+        cached = @cache_store.get_meta(@workspace.name, 'self_user_id')
+        return cached if cached
+
+        user_id = Api::Client.new(@api_client, @workspace).auth_test['user_id']
+        @cache_store.set_meta(@workspace.name, 'self_user_id', user_id) if user_id
+        user_id
+      end
+    end
+  end
+end

data/lib/slk/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Slk
-  VERSION = '0.4.2'
+  VERSION = '0.6.0'
 end

data/lib/slk.rb

@@ -11,7 +11,35 @@ require 'io/console'
 # Slack CLI - A command-line interface for Slack
 module Slk
   class Error < StandardError; end
-  class ApiError < Error; end
+
+  # Errors from any Slack-API-shaped failure: HTTP errors, network errors,
+  # logical Slack errors (user_not_found, missing_scope, etc.), JSON parse
+  # failures. The optional `code` symbol lets callers match specific cases
+  # (e.g. `e.code == :user_not_found`) without parsing message strings.
+  #
+  # Codes used:
+  #   :network_error, :http_error, :unauthorized, :invalid_json,
+  #   :ratelimited, plus any literal Slack `error` value (`:user_not_found`,
+  #   `:missing_scope`, `:account_inactive`, etc.).
+  class ApiError < Error
+    attr_reader :code
+
+    def initialize(message, code: nil)
+      super(message)
+      @code = code
+    end
+  end
+
+  # Slack rate-limit error. Carries Retry-After in seconds when present.
+  class RateLimitError < ApiError
+    attr_reader :retry_after
+
+    def initialize(message, retry_after: nil)
+      super(message, code: :ratelimited)
+      @retry_after = retry_after
+    end
+  end
+
   class ConfigError < Error; end
   class EncryptionError < Error; end
   class TokenStoreError < Error; end
@@ -34,6 +62,8 @@ module Slk
     autoload :Preset, 'slk/models/preset'
     autoload :SearchResult, 'slk/models/search_result'
     autoload :SavedItem, 'slk/models/saved_item'
+    autoload :Profile, 'slk/models/profile'
+    autoload :ProfileField, 'slk/models/profile_field'
   end
 
   # Application services for configuration, caching, and API communication
@@ -49,13 +79,20 @@ module Slk
     autoload :ReactionEnricher, 'slk/services/reaction_enricher'
     autoload :GemojiSync, 'slk/services/gemoji_sync'
     autoload :EmojiDownloader, 'slk/services/emoji_downloader'
+    autoload :FileDownloader, 'slk/services/file_downloader'
     autoload :EmojiSearcher, 'slk/services/emoji_searcher'
     autoload :ActivityEnricher, 'slk/services/activity_enricher'
     autoload :UnreadMarker, 'slk/services/unread_marker'
     autoload :TargetResolver, 'slk/services/target_resolver'
     autoload :SetupWizard, 'slk/services/setup_wizard'
     autoload :UserLookup, 'slk/services/user_lookup'
+    autoload :UserMatcher, 'slk/services/user_matcher'
+    autoload :UserPicker, 'slk/services/user_picker'
+    autoload :WhoTargetResolver, 'slk/services/who_target_resolver'
     autoload :MessageResolver, 'slk/services/message_resolver'
+    autoload :ProfileBuilder, 'slk/services/profile_builder'
+    autoload :ProfileResolver, 'slk/services/profile_resolver'
+    autoload :MetaCache, 'slk/services/meta_cache'
   end
 
   # Output formatters for messages, durations, and emoji
@@ -74,6 +111,9 @@ module Slk
     autoload :SearchFormatter, 'slk/formatters/search_formatter'
     autoload :SavedItemFormatter, 'slk/formatters/saved_item_formatter'
     autoload :TextProcessor, 'slk/formatters/text_processor'
+    autoload :ProfileFormatter, 'slk/formatters/profile_formatter'
+    autoload :ProfileFieldRenderer, 'slk/formatters/profile_field_renderer'
+    autoload :ProfileRows, 'slk/formatters/profile_rows'
   end
 
   # CLI commands implementing user-facing functionality
@@ -95,6 +135,9 @@ module Slk
     autoload :Config, 'slk/commands/config'
     autoload :Help, 'slk/commands/help'
     autoload :Later, 'slk/commands/later'
+    autoload :Debug, 'slk/commands/debug'
+    autoload :Who, 'slk/commands/who'
+    autoload :Org, 'slk/commands/org'
   end
 
   # Thin wrappers around Slack API endpoints
@@ -110,6 +153,7 @@ module Slk
     autoload :Activity, 'slk/api/activity'
     autoload :Search, 'slk/api/search'
     autoload :Saved, 'slk/api/saved'
+    autoload :Team, 'slk/api/team'
   end
 
   # Utility classes for paths, parsing, and helpers

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: slk
 version: !ruby/object:Gem::Version
-  version: 0.4.2
+  version: 0.6.0
 platform: ruby
 authors:
 - Eric Boehs
@@ -34,6 +34,7 @@ files:
 - lib/slk/api/emoji.rb
 - lib/slk/api/saved.rb
 - lib/slk/api/search.rb
+- lib/slk/api/team.rb
 - lib/slk/api/threads.rb
 - lib/slk/api/usergroups.rb
 - lib/slk/api/users.rb
@@ -43,11 +44,13 @@ files:
 - lib/slk/commands/cache.rb
 - lib/slk/commands/catchup.rb
 - lib/slk/commands/config.rb
+- lib/slk/commands/debug.rb
 - lib/slk/commands/dnd.rb
 - lib/slk/commands/emoji.rb
 - lib/slk/commands/help.rb
 - lib/slk/commands/later.rb
 - lib/slk/commands/messages.rb
+- lib/slk/commands/org.rb
 - lib/slk/commands/presence.rb
 - lib/slk/commands/preset.rb
 - lib/slk/commands/search.rb
@@ -55,6 +58,7 @@ files:
 - lib/slk/commands/status.rb
 - lib/slk/commands/thread.rb
 - lib/slk/commands/unread.rb
+- lib/slk/commands/who.rb
 - lib/slk/commands/workspaces.rb
 - lib/slk/formatters/activity_formatter.rb
 - lib/slk/formatters/attachment_formatter.rb
@@ -66,6 +70,9 @@ files:
 - lib/slk/formatters/mention_replacer.rb
 - lib/slk/formatters/message_formatter.rb
 - lib/slk/formatters/output.rb
+- lib/slk/formatters/profile_field_renderer.rb
+- lib/slk/formatters/profile_formatter.rb
+- lib/slk/formatters/profile_rows.rb
 - lib/slk/formatters/reaction_formatter.rb
 - lib/slk/formatters/saved_item_formatter.rb
 - lib/slk/formatters/search_formatter.rb
@@ -74,6 +81,8 @@ files:
 - lib/slk/models/duration.rb
 - lib/slk/models/message.rb
 - lib/slk/models/preset.rb
+- lib/slk/models/profile.rb
+- lib/slk/models/profile_field.rb
 - lib/slk/models/reaction.rb
 - lib/slk/models/saved_item.rb
 - lib/slk/models/search_result.rb
@@ -88,9 +97,13 @@ files:
 - lib/slk/services/emoji_downloader.rb
 - lib/slk/services/emoji_searcher.rb
 - lib/slk/services/encryption.rb
+- lib/slk/services/file_downloader.rb
 - lib/slk/services/gemoji_sync.rb
 - lib/slk/services/message_resolver.rb
+- lib/slk/services/meta_cache.rb
 - lib/slk/services/preset_store.rb
+- lib/slk/services/profile_builder.rb
+- lib/slk/services/profile_resolver.rb
 - lib/slk/services/reaction_enricher.rb
 - lib/slk/services/setup_wizard.rb
 - lib/slk/services/target_resolver.rb
@@ -99,6 +112,9 @@ files:
 - lib/slk/services/token_store.rb
 - lib/slk/services/unread_marker.rb
 - lib/slk/services/user_lookup.rb
+- lib/slk/services/user_matcher.rb
+- lib/slk/services/user_picker.rb
+- lib/slk/services/who_target_resolver.rb
 - lib/slk/support/date_parser.rb
 - lib/slk/support/error_logger.rb
 - lib/slk/support/help_formatter.rb
@@ -132,7 +148,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.3
+rubygems_version: 4.0.10
 specification_version: 4
 summary: A command-line interface for Slack
 test_files: []