slk 0.12.1 → 0.13.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 709675ee0926af3aee4e201318382d1558d6a46e4ebd8158a6353ee0da49f670
-  data.tar.gz: 8b82f996a687811516b17c6fe164a0dcedffc0e199de0eacadab61f37918955d
+  metadata.gz: cbd57363408aa79edfa6cbbe3e875034d2fbcb9e3c2a7ec8ea1e9acea30c9167
+  data.tar.gz: bf9ab8b668ad9d04191ddb9b63a1c502d3652059c3b5bbc16113dff1d94eb8bc
 SHA512:
-  metadata.gz: 2ab37ace8886dd600f194faf26f96597e422f2d1252aaba09fafacf83fff77e549bd9bc69be059cd567c787a3b2b2e69e0fc84360ac42f186916c1ea8930fe5d
-  data.tar.gz: 68f5dcd8ca42484c6c9783de5c35d02e95755046a69b1b0184b62ed4a84aaedfe0a2067821d18572e5b1f821ad98fe45a2af8b438f44930626df433a5806c5e3
+  metadata.gz: 71d05dc4a9f5b46c49e5a14cad5c5e9b79042ece234142bb16f53dde80da67a5beafd1a2af0f30ef33b2bb8ef5381d64393979e640c996810bb536005e9a8180
+  data.tar.gz: 49956b5f79adb5e03f4bac033109626a315f9a1326904e321d4dcf2e17d189d7ac500bb820e2dab52199243d916c204b6ecef94b0c445c8287d591bdd1bfaa20

data/CHANGELOG.md

@@ -7,6 +7,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.13.0] - 2026-10-09
+
+### Added
+
+- `slk later` now lists each saved message's files (`[File: name]`) and attachment images (`[Image: name]`), and prints an "N files not downloaded" hint like `slk messages`. Pass `--fetch-attachments` to download them to `~/.cache/slk/files/` and show the local paths. With `--json`, it also adds a `file_paths` map (file ID to local path).
+- With `--fetch-attachments`, `slk later` draws downloaded images inline under their `[File:]` line in Ghostty, Kitty, iTerm2 and WezTerm, including inside tmux. This uses [chafa](https://hpjansson.org/chafa/) when it's installed. In other terminals, without chafa, or when output is piped, only the local path is shown. Pass `--no-image-preview` or set `SLK_IMAGE_PREVIEW=0` to download without drawing images.
+
 ## [0.12.1] - 2026-10-06
 
 ### Fixed

data/lib/slk/commands/later.rb

@@ -9,6 +9,17 @@ module Slk
     # rubocop:disable Metrics/ClassLength
     class Later < Base
       include Support::InlineImages
+      include Support::AttachmentFetching
+      include Support::ImagePreview
+
+      BOOLEAN_FLAGS = {
+        '--completed' => :completed,
+        '--in-progress' => :in_progress,
+        '--counts' => :counts,
+        '--no-content' => :no_content,
+        '--no-image-preview' => :no_image_preview,
+        '--workspace-emoji' => :workspace_emoji
+      }.freeze
 
       def execute
         result = validate_options
@@ -86,6 +97,7 @@ module Slk
           unless @options[:no_content]
             message = fetch_message_content(workspace, item)
             json_item[:message] = message if message
+            add_json_file_paths(json_item, workspace, item, message)
           end
 
           json_item
@@ -93,7 +105,13 @@ module Slk
       end
       # rubocop:enable Metrics/MethodLength
 
-      # rubocop:disable Metrics/MethodLength
+      def add_json_file_paths(json_item, workspace, item, message)
+        return unless @options[:fetch_attachments]
+
+        paths = attachment_paths_for(workspace, item, message)
+        json_item[:file_paths] = paths unless paths.empty?
+      end
+
       def display_formatted(workspace, items)
         if items.empty?
           puts 'No saved items found.'
@@ -101,37 +119,93 @@ module Slk
         end
 
         @fetch_failures = 0
-        # Skip workspace emoji inline images in markdown mode (they're terminal escape sequences)
-        if @options[:workspace_emoji] && inline_images_supported? && !@options[:markdown]
-          display_with_workspace_emoji(workspace, items)
-        else
-          display_without_workspace_emoji(workspace, items)
-        end
+        @displayed_messages = []
+        display_each_item(workspace, items)
         show_fetch_failure_summary
+        print_file_summary(@displayed_messages) unless @options[:fetch_attachments]
       end
-      # rubocop:enable Metrics/MethodLength
 
-      def display_without_workspace_emoji(workspace, items)
-        formatter = build_formatter(output)
+      def display_each_item(workspace, items)
+        printer = buffered_display? ? method(:display_item_buffered) : method(:display_item_direct)
         items.each do |item|
           message = @options[:no_content] ? nil : fetch_message_content(workspace, item)
-          formatter.display_item(item, workspace, message: message, width: display_width, truncate: @options[:truncate])
+          printer.call(item, workspace, message, attachment_paths_for(workspace, item, message))
         end
       end
 
-      def display_with_workspace_emoji(workspace, items)
-        # Capture output to StringIO, then reprint with workspace emoji
+      def display_item_direct(item, workspace, message, file_paths)
+        @direct_formatter ||= build_formatter(output)
+        format_item(@direct_formatter, item, workspace, message, file_paths)
+      end
+
+      # Capture one item's output, then reprint it line by line so workspace
+      # emoji and downloaded images can be drawn inline as each item arrives.
+      def display_item_buffered(item, workspace, message, file_paths)
         buffer = StringIO.new
-        buffer_output = create_buffer_output(buffer)
-        formatter = build_formatter(buffer_output)
+        format_item(build_formatter(create_buffer_output(buffer)), item, workspace, message, file_paths)
+        previews = preview_paths(file_paths)
+        buffer.string.each_line do |line|
+          print_buffered_line(line.chomp, workspace)
+          print_line_previews(line, previews)
+        end
+      end
 
-        items.each do |item|
-          message = @options[:no_content] ? nil : fetch_message_content(workspace, item)
-          formatter.display_item(item, workspace, message: message, width: display_width, truncate: @options[:truncate])
+      def format_item(formatter, item, workspace, message, file_paths)
+        formatter.display_item(item, workspace, message: message, width: display_width,
+                                                truncate: @options[:truncate], file_paths: file_paths)
+      end
+
+      def print_buffered_line(line, workspace)
+        if workspace_emoji_images?
+          print_with_workspace_emoji(line, workspace)
+        else
+          puts line
         end
+      end
+
+      # Draw a preview under the [File: ...] / [Image: ...] line that names it
+      def print_line_previews(line, previews)
+        previews.each do |path|
+          next unless line.include?("[File: #{path}]") || line.include?("[Image: #{path}]")
 
-        # Reprint each line with workspace emoji replacement
-        buffer.string.each_line { |line| print_with_workspace_emoji(line.chomp, workspace) }
+          previews.delete(path)
+          print_image_preview(path)
+        end
+      end
+
+      def preview_paths(file_paths)
+        return [] unless attachment_previews?
+
+        file_paths.values.uniq.select { |path| previewable_image?(path) }
+      end
+
+      # Workspace emoji are terminal escape sequences, so skip them in markdown mode
+      def workspace_emoji_images?
+        @options[:workspace_emoji] && !@options[:markdown] && inline_images_supported?
+      end
+
+      def attachment_previews?
+        return false unless @options[:fetch_attachments]
+        return false if @options[:markdown] || @options[:quiet] || @options[:no_image_preview]
+
+        image_previews_supported?
+      end
+
+      def buffered_display?
+        workspace_emoji_images? || attachment_previews?
+      end
+
+      # Track the message for the "N files not downloaded" summary and, with
+      # --fetch-attachments, download its files/images to the local cache.
+      # @return [Hash] file_id / "att_<ts>_<idx>" => local path
+      def attachment_paths_for(workspace, item, message)
+        return {} unless message
+
+        model = Models::Message.from_api(message, channel_id: item.channel_id)
+        @displayed_messages&.push(model)
+        return {} unless @options[:fetch_attachments]
+
+        fetch_attachment_files([model], workspace)
       end
 
       def show_fetch_failure_summary
@@ -187,20 +261,19 @@ module Slk
           in_progress: false,
           counts: false,
           no_content: false,
+          no_image_preview: false,
           truncate: false,
           workspace_emoji: false
         )
       end
 
       def handle_option(arg, args, remaining)
-        case arg
-        when '-n', '--limit' then @options[:limit] = args.shift.to_i
-        when '--completed' then @options[:completed] = true
-        when '--in-progress' then @options[:in_progress] = true
-        when '--counts' then @options[:counts] = true
-        when '--no-content' then @options[:no_content] = true
-        when '--workspace-emoji' then @options[:workspace_emoji] = true
-        else super
+        if %w[-n --limit].include?(arg)
+          @options[:limit] = args.shift.to_i
+        elsif BOOLEAN_FLAGS.key?(arg)
+          @options[BOOLEAN_FLAGS[arg]] = true
+        else
+          super
         end
       end
 
@@ -221,6 +294,7 @@ module Slk
       def help_text
         help = Support::HelpFormatter.new('slk later [options]')
         help.description('Show saved "Later" items from Slack.')
+        help.note('Set SLK_IMAGE_PREVIEW=0 to always skip inline image previews.')
         add_options_section(help)
         help.render
       end
@@ -233,6 +307,8 @@ module Slk
           s.option('--in-progress', 'Show in-progress items')
           s.option('--counts', 'Show summary counts only')
           s.option('--no-content', 'Skip fetching message text')
+          s.option('--fetch-attachments', 'Download files to ~/.cache/slk/files/ (inline image previews need chafa)')
+          s.option('--no-image-preview', 'Download attachments but skip inline image previews')
           s.option('--workspace-emoji', 'Show workspace emoji as inline images')
           s.option('--no-emoji', 'Show :emoji: codes instead of unicode')
           s.option('--width N', 'Wrap text at N columns')

data/lib/slk/commands/messages.rb

@@ -9,6 +9,7 @@ module Slk
     # rubocop:disable Metrics/ClassLength
     class Messages < Base
       include Support::InlineImages
+      include Support::AttachmentFetching
 
       # rubocop:disable Metrics/MethodLength
       def execute
@@ -346,28 +347,6 @@ module Slk
         end
       end
 
-      def print_file_summary(messages)
-        file_count = messages.sum { |m| m.files.size + downloadable_attachment_count(m) }
-        return if file_count.zero?
-
-        label = file_count == 1 ? '1 file' : "#{file_count} files"
-        puts
-        info("#{label} not downloaded. Use --fetch-attachments to download.")
-      end
-
-      def downloadable_attachment_count(message)
-        message.attachments.count { |a| a['image_url'] || a['thumb_url'] }
-      end
-
-      def fetch_attachment_files(messages, workspace)
-        paths = Support::XdgPaths.new
-        downloader = Services::FileDownloader.new(
-          cache_dir: paths.cache_dir,
-          on_debug: ->(msg) { debug(msg) }
-        )
-        downloader.download_message_files(messages, workspace)
-      end
-
       def find_workspace_emoji(workspace_name, emoji_name)
         return nil if emoji_name.empty?
 

data/lib/slk/formatters/saved_item_formatter.rb

@@ -3,6 +3,7 @@
 module Slk
   module Formatters
     # Formats saved/later items for terminal display
+    # rubocop:disable Metrics/ClassLength
     class SavedItemFormatter
       def initialize(output:, mention_replacer:, text_processor:, on_debug: nil)
         @output = output
@@ -13,7 +14,9 @@ module Slk
 
       # Display a single saved item
       # @param truncate [Boolean] if true, truncate to single line at width instead of wrapping
-      def display_item(item, workspace, message: nil, width: nil, truncate: false)
+      # @param file_paths [Hash] file_id / "att_<ts>_<idx>" => local path for downloaded attachments
+      # rubocop:disable Metrics/ParameterLists
+      def display_item(item, workspace, message: nil, width: nil, truncate: false, file_paths: {})
         status_badge = format_status_badge(item)
         due_info = format_due_info(item)
 
@@ -22,9 +25,13 @@ module Slk
         @output.puts header_parts.join(' | ') unless header_parts.empty?
 
         # Message content
-        display_message(message, workspace, width: width, truncate: truncate) if message
+        if message
+          display_message(message, workspace, width: width, truncate: truncate)
+          display_attachments(message, file_paths || {})
+        end
         @output.puts # blank line between items
       end
+      # rubocop:enable Metrics/ParameterLists
 
       private
 
@@ -113,6 +120,34 @@ module Slk
         lines[1..].each { |line| @output.puts "  #{line.rstrip}" } if lines.length > 1
       end
 
+      def display_attachments(message, file_paths)
+        display_files(message['files'] || [], file_paths)
+        display_attachment_images(message['attachments'] || [], message['ts'], file_paths)
+      end
+
+      def display_files(files, file_paths)
+        files.each do |file|
+          label = file_paths[file['id']] || file['name'] || file['title'] || 'file'
+          @output.puts "  #{@output.blue("[File: #{label}]")}"
+        end
+      end
+
+      def display_attachment_images(attachments, message_ts, file_paths)
+        attachments.each_with_index do |att, idx|
+          url = att['image_url'] || att['thumb_url']
+          next unless url
+
+          label = file_paths["att_#{message_ts}_#{idx}"] || att['title'] || image_filename(url)
+          @output.puts "  #{@output.blue("[Image: #{label}]")}"
+        end
+      end
+
+      def image_filename(url)
+        File.basename(URI.parse(url).path)
+      rescue URI::InvalidURIError
+        'image'
+      end
+
       def resolve_message_author(message, workspace)
         if message['user']
           @mentions.lookup_user_name(workspace, message['user']) || message['user']
@@ -140,5 +175,6 @@ module Slk
         text.length > max_length ? "#{text[0...max_length]}..." : text
       end
     end
+    # rubocop:enable Metrics/ClassLength
   end
 end

data/lib/slk/support/attachment_fetching.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+module Slk
+  module Support
+    # Shared --fetch-attachments behavior for commands that display messages.
+    # Expects the including command to provide #debug and #info.
+    module AttachmentFetching
+      private
+
+      # Download files and attachment images for the given messages.
+      # @return [Hash] file_id / "att_<ts>_<idx>" => local path
+      def fetch_attachment_files(messages, workspace)
+        downloader = Services::FileDownloader.new(
+          cache_dir: Support::XdgPaths.new.cache_dir,
+          on_debug: ->(msg) { debug(msg) }
+        )
+        downloader.download_message_files(messages, workspace)
+      end
+
+      def downloadable_file_count(messages)
+        messages.sum { |m| m.files.size + downloadable_attachment_count(m) }
+      end
+
+      def downloadable_attachment_count(message)
+        message.attachments.count { |a| a['image_url'] || a['thumb_url'] }
+      end
+
+      def print_file_summary(messages)
+        file_count = downloadable_file_count(messages)
+        return if file_count.zero?
+
+        label = file_count == 1 ? '1 file' : "#{file_count} files"
+        puts
+        info("#{label} not downloaded. Use --fetch-attachments to download.")
+      end
+    end
+  end
+end

data/lib/slk/support/image_preview.rb

@@ -0,0 +1,93 @@
+# frozen_string_literal: true
+
+require 'open3'
+
+module Slk
+  module Support
+    # Renders downloaded image attachments inline using chafa, which handles
+    # Kitty graphics (Ghostty/Kitty) and iTerm2 images, including tmux
+    # passthrough with Unicode placeholders so images scroll with the pane.
+    # Without a capable terminal or chafa, callers just show the file path.
+    # Set SLK_IMAGE_PREVIEW=0 (or false/no/off) to turn previews off.
+    # Expects Support::InlineImages to be included alongside.
+    module ImagePreview
+      PREVIEW_MAX_COLS = 100
+      PREVIEW_MAX_ROWS = 15
+      PREVIEWABLE_EXTENSIONS = %w[png jpg jpeg gif webp bmp tif tiff heic].freeze
+      DISABLED_VALUES = %w[0 false no off].freeze
+
+      private
+
+      def image_previews_supported?
+        return @image_previews_supported if defined?(@image_previews_supported)
+
+        @image_previews_supported = !image_previews_disabled_by_env? && $stdout.tty? &&
+                                    !image_preview_format.nil? && chafa_available?
+      end
+
+      def image_previews_disabled_by_env?
+        DISABLED_VALUES.include?(ENV.fetch('SLK_IMAGE_PREVIEW', '').strip.downcase)
+      end
+
+      def image_preview_format
+        if kitty_graphics_supported?
+          'kitty'
+        elsif iterm2_protocol_supported?
+          'iterm'
+        end
+      end
+
+      def previewable_image?(path)
+        PREVIEWABLE_EXTENSIONS.include?(File.extname(path.to_s).delete('.').downcase)
+      end
+
+      # Print an inline preview of the image at path, indented by indent columns.
+      # @return [Boolean] whether a preview was printed
+      def print_image_preview(path, indent: 2) # rubocop:disable Naming/PredicateMethod
+        return false unless previewable_image?(path) && File.exist?(path)
+
+        rendered = render_image_preview(path, indent)
+        return false unless rendered
+
+        $stdout.print(' ' * indent)
+        $stdout.print(rendered)
+        $stdout.print("\n") unless rendered.end_with?("\n", "\e[?25h")
+        $stdout.flush
+        true
+      end
+
+      def render_image_preview(path, indent)
+        stdout, _stderr, status = Open3.capture3(*chafa_args(path, indent))
+        status.success? && !stdout.empty? ? stdout : nil
+      rescue SystemCallError => e
+        debug("chafa failed: #{e.message}") if respond_to?(:debug, true)
+        nil
+      end
+
+      def chafa_args(path, indent)
+        args = ['chafa', "--format=#{image_preview_format}"]
+        args << '--passthrough=tmux' if in_tmux?
+        args + ["--size=#{preview_cols(indent)}x#{PREVIEW_MAX_ROWS}",
+                '--animate=off', '--probe=off', '--relative=off', path]
+      end
+
+      def preview_cols(indent)
+        [terminal_columns - indent - 1, PREVIEW_MAX_COLS].min.clamp(10, PREVIEW_MAX_COLS)
+      end
+
+      def terminal_columns
+        require 'io/console'
+        cols = IO.console&.winsize&.last.to_i
+        cols.positive? ? cols : 80
+      rescue StandardError
+        80
+      end
+
+      def chafa_available?
+        ENV.fetch('PATH', '').split(File::PATH_SEPARATOR).any? do |dir|
+          File.executable?(File.join(dir, 'chafa'))
+        end
+      end
+    end
+  end
+end

data/lib/slk/version.rb

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

data/lib/slk.rb

@@ -194,6 +194,8 @@ module Slk
     autoload :XdgPaths, 'slk/support/xdg_paths'
     autoload :SlackUrlParser, 'slk/support/slack_url_parser'
     autoload :InlineImages, 'slk/support/inline_images'
+    autoload :AttachmentFetching, 'slk/support/attachment_fetching'
+    autoload :ImagePreview, 'slk/support/image_preview'
     autoload :HelpFormatter, 'slk/support/help_formatter'
     autoload :ErrorLogger, 'slk/support/error_logger'
     autoload :UserResolver, 'slk/support/user_resolver'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: slk
 version: !ruby/object:Gem::Version
-  version: 0.12.1
+  version: 0.13.0
 platform: ruby
 authors:
 - Eric Boehs
@@ -136,11 +136,13 @@ files:
 - lib/slk/services/user_matcher.rb
 - lib/slk/services/user_picker.rb
 - lib/slk/services/who_target_resolver.rb
+- lib/slk/support/attachment_fetching.rb
 - lib/slk/support/check_in_time.rb
 - lib/slk/support/date_parser.rb
 - lib/slk/support/decimal_timestamp.rb
 - lib/slk/support/error_logger.rb
 - lib/slk/support/help_formatter.rb
+- lib/slk/support/image_preview.rb
 - lib/slk/support/inline_images.rb
 - lib/slk/support/interactive_prompt.rb
 - lib/slk/support/platform.rb