fui 0.5.0 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ca2369b75631de675a012aa298db4edc1976fba82ca689411be60347be016525
-  data.tar.gz: 179e9195e0ededab573b0dee845cb4c60b766ce3e64cf001bf86d6be8a937c8e
+  metadata.gz: e1b8934ca5afcaf97a11129a196887f07aa1596113b40c18e061db018410df26
+  data.tar.gz: d8639e7b395f89c874c81cbe0d5613dea5e5db59a7e0ceb389e0ea32a361a0be
 SHA512:
-  metadata.gz: 8f2b6927e03089fee397a11ea73ce511c88ddbbe15652e106e4f1801fc677351f10f90cb39734bb2279cdda003e8c577108c9efab324a3dfe848b90c01133745
-  data.tar.gz: 2039608e5a2394039f3d576838da7e7324cdd8588bdca12cb15570a3d729446885568d2e1e3d05fd7777a9e9313d3f500c775368eb77a667a3495c470e5928cf
+  metadata.gz: 8bc77ab09250e68b6ff3f54111f107852d1a988cabfb875c167662363c3040cc904d37ac9133275741190aa11f72477e3bf19d5b6074d9ceffc6bfadc22ff5a1
+  data.tar.gz: 4397a5d1a4c8b19d7b4dc91b50808200c30d96083e311a3f7bf816df3b1b58922827e98bcfb77955fb02a4d977a8763bf03d5d59bf3c4ac45ff51f499b7ad72a

data/CHANGELOG.md

@@ -1,3 +1,13 @@
+### 0.6.0 (2026/04/12)
+
+* [#45](https://github.com/dblock/fui/pull/45): Migrated from Travis CI to GitHub Actions with danger-pr-comment workflow - [@dblock](https://github.com/dblock).
+* [#47](https://github.com/dblock/fui/pull/47): Fixed `ArgumentError: invalid byte sequence in UTF-8` when processing files with non-UTF-8 encoding - [@dblock](https://github.com/dblock).
+* [#48](https://github.com/dblock/fui/pull/48): Fixed `NameError: undefined local variable or method 'project_path'` in verbose mode - [@dblock](https://github.com/dblock).
+* [#49](https://github.com/dblock/fui/pull/49): Added summary output to `find` command: prints `Found N unused header(s).` or `No unused imports found.` - [@dblock](https://github.com/dblock).
+* [#50](https://github.com/dblock/fui/pull/50): Fixed `delete` command to also remove `.mm` (Objective-C++) implementation files - [@dblock](https://github.com/dblock).
+* [#51](https://github.com/dblock/fui/pull/51): Added support for Ruby 4.x - [@dblock](https://github.com/dblock).
+* [#52](https://github.com/dblock/fui/pull/52): Clarified README: fui finds unreferenced header files, not unused imports within files - [@dblock](https://github.com/dblock).
+
 ### 0.5.0 (2018/12/19)
 
 * [#29](https://github.com/dblock/fui/pull/29): Added support for ignoring paths through `-i`, `--ignore-path` - [@jeffctown](https://github.com/jeffctown).

data/README.md

@@ -2,10 +2,14 @@ Fui
 ==========
 
 [![Gem Version](https://badge.fury.io/rb/fui.svg)](https://badge.fury.io/rb/fui)
-[![Build Status](https://travis-ci.org/dblock/fui.svg)](https://travis-ci.org/dblock/fui)
+[![Test](https://github.com/dblock/fui/actions/workflows/test.yml/badge.svg)](https://github.com/dblock/fui/actions/workflows/test.yml)
 
 Find unused Objective-C imports.
 
+> **What fui does:** finds header (`.h`) files that are never imported anywhere in your project — i.e. classes that can potentially be deleted entirely.
+>
+> **What fui does not do:** find `#import` statements within a file that are unused by that file's code. For that, use a tool like [Clang's unused-includes warning](https://clang.llvm.org/docs/DiagnosticsReference.html) or AppCode.
+
 # Table of Contents
 
 - [Usage](#usage)
@@ -24,27 +28,27 @@ Find unused Objective-C imports.
 
 ## Usage
 
-```
+```sh
 gem install fui
 ```
 
 ### Get Help
 
-```
+```sh
 fui help
 ```
 
 ### Find Unused Classes in the Current Directory
 
-```
+```sh
 fui find
 ```
 
-The `find` command lists all the files that contain unused imports and exits with the number of files found.
+The `find` command lists all header (`.h`) files that are not imported anywhere in the project, and exits with the number of such files found.
 
 ### Find Unused Classes in any Path
 
-```
+```sh
 fui --path=~/source/project/Name find
 ```
 
@@ -52,7 +56,7 @@ fui --path=~/source/project/Name find
 
 Running `fui` with `-x` (or `--ignore-xib-files`) will, for example, mark `Foo.h` as unused when `Foo.xib` holds a reference to the `Foo` class and no other references to Foo.h exist.
 
-```
+```sh
 fui -x --path=~/source/project/Name find
 ```
 
@@ -60,7 +64,7 @@ fui -x --path=~/source/project/Name find
 
 Running `fui` with `-l` (or `--ignore-local-imports`) will, for example, mark `Foo.h` as unused when `Bar.h` contains a local (quotation syntax) import of `Foo.h` (eg. `#import Foo.h`).
 
-```
+```sh
 fui -l --path=~/source/project/Name find
 ```
 
@@ -68,7 +72,7 @@ fui -l --path=~/source/project/Name find
 
 Running `fui` with `-g` (or `--ignore-global-imports`) will, for example, mark `Foo.h` as unused when `Bar.h` contains a global (bracket syntax) import of `Foo.h` (eg. `#import <Framework/Foo.h>`).
 
-```
+```sh
 fui -g --path=~/source/project/Name find
 ```
 
@@ -76,7 +80,7 @@ fui -g --path=~/source/project/Name find
 
 Running `fui` with `-i` (or `--ignore-path`) will, for example, ignore a `Pods` folder when searching for headers or referencing files.
 
-```
+```sh
 fui --path=~/source/project/Name --ignore-path=Pods find
 ```
 
@@ -84,13 +88,13 @@ fui --path=~/source/project/Name --ignore-path=Pods find
 
 Running `fui` with `-i` (or `--ignore-path`) can ignore multiple folders when searching for headers or referencing files.
 
-```
+```sh
 fui --path=~/source/project/Name --ignore-path=Pods --ignore-path=Libraries find
 ```
 
 ### Delete All Unused Class Files with Prompt
 
-```
+```sh
 fui --path=~/source/project/Name delete --perform --prompt
 ```
 

data/RELEASING.md

@@ -11,7 +11,7 @@ bundle install
 rake
 ```
 
-Check that the last build succeeded in [Travis CI](https://travis-ci.org/dblock/fui).
+Check that the last build succeeded in [GitHub Actions](https://github.com/dblock/fui/actions).
 
 Change "Next" in [CHANGELOG.md](CHANGELOG.md) to the current date.
 

data/bin/fui

@@ -37,7 +37,13 @@ command :find do |c|
         puts relative_path
       end
     end
-    exit_now! nil, $fui.unused_references.count
+    count = $fui.unused_references.count
+    if count > 0
+      puts "Found #{count} unused #{count == 1 ? 'header' : 'headers'}."
+    else
+      puts 'No unused imports found.'
+    end
+    exit_now! nil, count
   end
 end
 
@@ -57,7 +63,7 @@ command :delete do |c|
       end.each do |k, _v|
         relative_path = Pathname.new(k.path).relative_path_from(root).to_s
         if options[:prompt]
-          print "Remove #{relative_path}(.m) [y/N] "
+          print "Remove #{relative_path}(.m/.mm) [y/N] "
           response = STDIN.getc.upcase
           puts "#{response.chr}\r\n"
           next unless response.chr == 'Y'
@@ -68,16 +74,18 @@ command :delete do |c|
           puts "#{relative_path}#{options[:perform] ? '' : ' (simulation)'}\r\n"
         end
         File.delete(k.path) if options[:perform]
-        impl_path = k.path.gsub(/\.h$/, '.m')
-        next unless File.exist?(impl_path)
+        ['.m', '.mm'].each do |ext|
+          impl_path = k.path.gsub(/\.h$/, ext)
+          next unless File.exist?(impl_path)
 
-        relative_path = Pathname.new(impl_path).relative_path_from(root).to_s
-        if global_options[:verbose]
-          puts "Removing #{relative_path}#{options[:perform] ? '' : ' (simulation)'}\r\n"
-        else
-          puts "#{relative_path}#{options[:perform] ? '' : ' (simulation)'}\r\n"
+          relative_path = Pathname.new(impl_path).relative_path_from(root).to_s
+          if global_options[:verbose]
+            puts "Removing #{relative_path}#{options[:perform] ? '' : ' (simulation)'}\r\n"
+          else
+            puts "#{relative_path}#{options[:perform] ? '' : ' (simulation)'}\r\n"
+          end
+          File.delete(impl_path) if options[:perform]
         end
-        File.delete(impl_path) if options[:perform]
       end
     ensure
       system('stty -raw echo')

data/lib/fui/finder.rb

@@ -84,26 +84,23 @@ module Fui
     end
 
     def process_code(references, path)
-      File.open(path) do |file|
-        yield path if block_given?
-        headers.each do |header|
-          filename_without_extension = File.basename(path, File.extname(path))
-          file_contents = File.read(file)
-          global_import_exists = global_imported(file_contents, header)
-          local_import_exists = local_imported(file_contents, header)
-          references[header] << path if filename_without_extension != header.filename_without_extension && (local_import_exists || global_import_exists)
-        end
+      yield path if block_given?
+      file_contents = File.read(path, encoding: 'binary').encode('UTF-8', invalid: :replace, undef: :replace)
+      headers.each do |header|
+        filename_without_extension = File.basename(path, File.extname(path))
+        global_import_exists = global_imported(file_contents, header)
+        local_import_exists = local_imported(file_contents, header)
+        references[header] << path if filename_without_extension != header.filename_without_extension && (local_import_exists || global_import_exists)
       end
     end
 
     def process_xml(references, path)
-      File.open(path) do |file|
-        yield path if block_given?
-        headers.each do |header|
-          filename_without_extension = File.basename(path, File.extname(path))
-          check_xibs = !options['ignore-xib-files']
-          references[header] << path if (check_xibs || filename_without_extension != header.filename_without_extension) && File.read(file).include?("customClass=\"#{header.filename_without_extension}\"")
-        end
+      yield path if block_given?
+      file_contents = File.read(path, encoding: 'binary').encode('UTF-8', invalid: :replace, undef: :replace)
+      headers.each do |header|
+        filename_without_extension = File.basename(path, File.extname(path))
+        check_xibs = !options['ignore-xib-files']
+        references[header] << path if (check_xibs || filename_without_extension != header.filename_without_extension) && file_contents.include?("customClass=\"#{header.filename_without_extension}\"")
       end
     end
   end

data/lib/fui/project.rb

@@ -22,7 +22,7 @@ module Fui
 
           path_tokens = tokens[1].split('/')
           bridging_header = path_tokens[path_tokens.length - 1]
-          puts "Bridging Header Found: #{bridging_header} in #{project_path}." if verbose
+          puts "Bridging Header Found: #{bridging_header} in #{path}." if verbose
           bridging_headers << bridging_header
         end
         bridging_headers.uniq

data/lib/fui/version.rb

@@ -1,3 +1,3 @@
 module Fui
-  VERSION = '0.5.0'.freeze
+  VERSION = '0.6.0'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fui
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.6.0
 platform: ruby
 authors:
 - Daniel Doubrovkine
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2018-12-20 00:00:00.000000000 Z
+date: 2026-04-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: gli
@@ -24,7 +24,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: 
+description:
 email: dblock@dblock.org
 executables:
 - fui
@@ -46,7 +46,7 @@ homepage: http://github.com/dblock/fui
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -61,9 +61,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: 1.3.6
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.8
-signing_key: 
+rubygems_version: 3.5.16
+signing_key:
 specification_version: 4
 summary: Find unused Objective-C imports.
 test_files: []