immosquare-cleaner 1.1.0 → 1.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 684fefa6f6061c2cfd2a30c10852463e2a8a0605dc8d4078fe9cb8f136bd3f97
-  data.tar.gz: 9437ae1b6de815f2b97d5dd10f61fb3ad412e6989969f73a10b481b9560efd8b
+  metadata.gz: ce05c959aa1bf3f812d6f676b2864037f59668a97da382b9bd1d7c7b13b6bde6
+  data.tar.gz: f6d6650dbbea335a474030d9808043820de93d25c6829f0423aaea1758a8d034
 SHA512:
-  metadata.gz: d0962e5e3df0bf9493abeed5871e85fa8146787aec41b09ec37a322ccd88fd7d2454d1d08bb97aa03cf153f1e6ff75809b6d7f286fab0bf1f3ad5e4095f15291
-  data.tar.gz: 53315a827d4a2030c1ce6e05bd70c24ded051f9259056fd695261c99e3c3d4e969c89543fb46498e5dfd39ff3a8ec076300352c41860491e020255f8b9edd83f
+  metadata.gz: 7d261eb4d38439c234c8836c5e04f56c3172277ea330722d352288e4b55ae2432d17e14ccb9db350ba6686d2016b9aeebe3c9eec98e61f0f7b85fe686ebf5a45
+  data.tar.gz: 6214f6c06cc2b1155a4272b12fd95d6a0531245b456eb904ac1c04b63a2e4b642745399e26004ddbbdff6301c0d0ba6257618bdc5d041fe00440bafbebe0d1f7

data/bin/immosquare-cleaner

@@ -41,16 +41,18 @@ OptionParser.new do |parser|
 end.parse!
 
 ##============================================================##
-## Hook mode : the edited file comes from the agent's JSON payload.
-## Linters write their reports on stdout, which some agents
-## (Gemini CLI) parse as the hook's JSON answer: everything goes
-## to stderr instead. An unsupported or missing file ends the
-## hook silently with exit 0, so the agent is never blocked.
+## Hook mode : the edited files come from the agent's JSON payload
+## (one file, or several for a Codex patch). Linters write their
+## reports on stdout, which some agents (Gemini CLI, Codex) parse
+## as the hook's JSON answer: everything goes to stderr instead.
+## No supported file ends the hook silently with exit 0, so the
+## agent is never blocked.
 ##============================================================##
 if options[:hook]
-  file_path = ImmosquareCleaner::Hook.file_path($stdin.read)
-  exit(0) if file_path.nil?
+  hook_files = ImmosquareCleaner::Hook.file_paths($stdin.read)
+  exit(0) if hook_files.empty?
   $stdout.reopen($stderr)
+  file_path = hook_files.first
 else
   file_path = ARGV[0]
 end
@@ -95,7 +97,9 @@ end
 ## We can now call the clean method, passing the file absolute path and options
 ##============================================================##
 file_path = File.expand_path(file_path)
-if File.directory?(file_path)
+if options[:hook]
+  hook_files.each {|path| ImmosquareCleaner.clean(path) }
+elsif File.directory?(file_path)
   ##============================================================##
   ## Run from the cleaned directory: the project's
   ## config/initializers/immosquare-cleaner.rb (exclude_files) is

data/lib/immosquare-cleaner/directory_cleaner.rb

@@ -27,7 +27,10 @@ module ImmosquareCleaner
     ## EXCLUDED_PATHS : paths relative to the root — generated or
     ##                  vendored content that repositories commit
     ##                  (db/schema.rb, Tauri schemas...) and test
-    ##                  fixtures that must stay as written
+    ##                  fixtures that must stay as written: YAML
+    ##                  fixtures hold ERB tags Prettier would break,
+    ##                  sample files are compared byte for byte.
+    ##                  Test code itself is cleaned like any source.
     ##============================================================##
     EXCLUDED_DIRS = [
       ".git",
@@ -47,9 +50,10 @@ module ImmosquareCleaner
       "app/assets/images",
       "db",
       "public",
+      "spec/fixtures",
       "src-tauri/gen",
       "src-tauri/permissions/autogenerated",
-      "test"
+      "test/fixtures"
     ].freeze
     EXCLUDED_FILES = [
       "npm-shrinkwrap.json",

data/lib/immosquare-cleaner/hook.rb

@@ -2,36 +2,59 @@ require "json"
 
 module ImmosquareCleaner
   ##============================================================##
-  ## `immosquare-cleaner --hook` : the file to clean comes from the
+  ## `immosquare-cleaner --hook` : the files to clean come from the
   ## JSON an AI agent pipes on stdin after an edit, not from ARGV.
   ##
-  ## Each agent nests the edited path differently:
+  ## Each agent describes the edit differently:
   ## - Claude Code, Gemini CLI : tool_input.file_path
   ## - Kimi Code               : tool_input.path
   ## - Grok                    : toolInput.file_path
   ## - Cursor (afterFileEdit)  : file_path at the top level
-  ## A relative path is resolved against the payload's `cwd`.
+  ## - Codex (apply_patch)     : tool_input.command holds the patch,
+  ##                             one `*** Update File: <path>` (or
+  ##                             Add File / Move to) line per file
+  ## Relative paths are resolved against tool_input.workdir (Codex),
+  ## then the payload's `cwd`.
   ##
   ## A hook must never block the agent: anything that is not a
-  ## supported file (no path, deleted file, directory, LICENSE,
-  ## .env...) is skipped instead of reported.
+  ## supported file (deleted file, directory, symlink, LICENSE,
+  ## .env...) is dropped instead of reported. A symlink is never
+  ## followed, so an edit cannot reformat a file outside the project.
   ##============================================================##
   module Hook
     PAYLOAD_KEYS = ["tool_input", "toolInput"].freeze
     PATH_KEYS    = ["file_path", "path", "target_file"].freeze
+    PATCH_LINE   = /^\*\*\* (?:Add File|Update File|Move to): (.+)$/
 
-    def self.file_path(raw_payload)
+    def self.file_paths(raw_payload)
       payload = JSON.parse(raw_payload.to_s)
-      return if !payload.is_a?(Hash)
+      return [] if !payload.is_a?(Hash)
 
-      sources = PAYLOAD_KEYS.map {|key| payload[key] }.grep(Hash) + [payload]
-      path    = sources.lazy.flat_map {|source| PATH_KEYS.map {|key| source[key] } }.find {|value| value.is_a?(String) && !value.empty? }
-      return if path.nil?
+      sources  = PAYLOAD_KEYS.map {|key| payload[key] }.grep(Hash)
+      cwd      = payload["cwd"].is_a?(String) && !payload["cwd"].empty? ? payload["cwd"] : Dir.pwd
+      workdir  = sources.map {|source| source["workdir"] }.find {|value| value.is_a?(String) && !value.empty? }
+      base_dir = workdir ? File.expand_path(workdir, cwd) : cwd
 
-      path = File.expand_path(path, payload["cwd"].is_a?(String) ? payload["cwd"] : Dir.pwd)
-      path if File.file?(path) && ImmosquareCleaner.supported?(path)
+      paths = patch_paths(sources)
+      paths = [direct_path(sources + [payload])].compact if paths.empty?
+      paths.map {|path| File.expand_path(path, base_dir) }.uniq.select {|path| File.file?(path) && !File.symlink?(path) && ImmosquareCleaner.supported?(path) }
     rescue JSON::ParserError
-      nil
+      []
     end
+
+    def self.direct_path(sources)
+      sources.lazy.flat_map {|source| PATH_KEYS.map {|key| source[key] } }.find {|value| value.is_a?(String) && !value.empty? }
+    end
+
+    ##============================================================##
+    ## Codex sends the patch as a string, older versions as an argv
+    ## array (`["apply_patch", "*** Begin Patch..."]`)
+    ##============================================================##
+    def self.patch_paths(sources)
+      patch = sources.map {|source| source["command"] }.find {|value| value.is_a?(String) || value.is_a?(Array) }
+      Array(patch).grep(String).join("\n").scan(PATCH_LINE).flatten.map(&:strip)
+    end
+
+    private_class_method(:direct_path, :patch_paths)
   end
 end

data/lib/immosquare-cleaner/markdown.rb

@@ -144,6 +144,33 @@ module ImmosquareCleaner
         nil
       end
 
+      ##============================================================##
+      ## Splits a table row on its delimiting pipes. A pipe escaped as
+      ## `\|` is cell content (GFM), not a delimiter: splitting on it
+      ## would add a column to the whole table. A trailing empty field
+      ## only exists when the row ends with a delimiting pipe, which
+      ## closes the row rather than opening an extra cell.
+      ##============================================================##
+      def table_cells(line)
+        cells   = [+""]
+        escaped = false
+        line.strip.delete_prefix("|").each_char do |char|
+          if escaped
+            cells.last << char
+            escaped = false
+          elsif char == "\\"
+            cells.last << char
+            escaped = true
+          elsif char == "|"
+            cells << +""
+          else
+            cells.last << char
+          end
+        end
+        cells.pop if cells.size > 1 && cells.last.empty?
+        cells.map(&:strip)
+      end
+
       ##============================================================##
       ## we want to clean the markdown files to have a uniform style
       ## for the tables.
@@ -161,9 +188,9 @@ module ImmosquareCleaner
           ## above a column of row labels, a value that does not apply).
           ## Rejecting empty cells would shift every following cell one
           ## column to the left and silently file values under the wrong
-          ## header. `split("|", -1)` keeps trailing empty fields.
+          ## header.
           ##============================================================##
-          cells = line.strip.delete_prefix("|").delete_suffix("|").split("|", -1).map(&:strip)
+          cells = table_cells(line)
 
           ##============================================================##
           ## We increase the size of the array if needed

data/lib/immosquare-cleaner/version.rb

@@ -1,3 +1,3 @@
 module ImmosquareCleaner
-  VERSION = "1.1.0".freeze
+  VERSION = "1.2.1".freeze
 end

data/linters/eslint.config.mjs

@@ -134,6 +134,20 @@ export default [
       "no-undef": 0
     }
   },
+  //============================================================//
+  // JSX files: ESLint only lints .js/.mjs/.cjs by default, so a
+  // .jsx file matched by no `files` pattern is skipped with
+  // "File ignored because no matching configuration was supplied".
+  // The common rules above carry no `files` key and apply here too.
+  //============================================================//
+  {
+    files:           ["**/*.jsx"],
+    languageOptions: {
+      parserOptions: {
+        ecmaFeatures: { jsx: true }
+      }
+    }
+  },
   erb.configs.recommended,
   {
     linterOptions: {

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: immosquare-cleaner
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.1
 platform: ruby
 authors:
 - immosquare