no_comments 0.1.11 → 0.1.14

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c98d97eb96221297ef0ef1b6140abd04ac9a54ce759fdfd46b0708497b223022
-  data.tar.gz: 42a109143ae3b1ab99a176e625a74dbe350d878838eb40b93c5f483f13428f11
+  metadata.gz: 5680480af7ba824c7591147b05fb53fc1368695d834518889d42707c9f98306b
+  data.tar.gz: 562fc50c7bcce1ee7dde8296c984dbde4935de25930c4adffbd3c99f22f5cac5
 SHA512:
-  metadata.gz: 82418111acf7d4de92b39c52cd6bafb32959b1ae024836d53a7be3a2b66257cd13299e4b179f84506012debc71db3c48bcbade705e6df0cb18b7a83fa09cc550
-  data.tar.gz: 01df35d37bc0e4200beb1c15c6b4f2846d2904faa681d20706e218681999e7f4e946ef7df0eab2ede035f415b095d784d06e37ffed15062dc5cbae206e52d510
+  metadata.gz: 7f26162ead8d4299371496f7c949aa0992bca529bc2e5ee63f52a2c159e2e493000255cf2676fd0ae79666c73521061fca2ee76fe5ace79bb33a6d1897f8cb39
+  data.tar.gz: c2205fc1deb3fac280a5b3b3b33bfc1106d68508a9c4b56987d1f7419778d6cba429b7c1f6ed8658d2dce3f3bc92bc37c3a33bc7fd7053960efb5a61106e73c8

data/AGENTS.md

@@ -0,0 +1,48 @@
+# Instructions
+- The user will provide a task.
+- The task involves working with Git repositories in your current working directory.
+- Wait for all terminal commands to be completed (or terminate them) before finishing.
+
+# Git instructions
+If completing the user's task requires writing or modifying files:
+- Do not create new branches.
+- Use git to commit your changes.
+- If pre-commit fails, fix issues and retry.
+- Check git status to confirm your commit. You must leave your worktree in a clean state.
+- Only committed code will be evaluated.
+- Do not modify or amend existing commits.
+
+# AGENTS.md spec
+- Containers often contain AGENTS.md files. These files can appear anywhere in the container's filesystem. Typical locations include `/`, `~`, and in various places inside of Git repos.
+- These files are a way for humans to give you (the agent) instructions or tips for working within the container.
+- Some examples might be: coding conventions, info about how code is organized, or instructions for how to run or test code.
+- AGENTS.md files may provide instructions about PR messages (messages attached to a GitHub Pull Request produced by the agent, describing the PR). These instructions should be respected.
+- Instructions in AGENTS.md files:
+  - The scope of an AGENTS.md file is the entire directory tree rooted at the folder that contains it.
+  - For every file you touch in the final patch, you must obey instructions in any AGENTS.md file whose scope includes that file.
+  - Instructions about code style, structure, naming, etc. apply only to code within the AGENTS.md file's scope, unless the file states otherwise.
+  - More-deeply-nested AGENTS.md files take precedence in the case of conflicting instructions.
+  - Direct system/developer/user instructions (as part of a prompt) take precedence over AGENTS.md instructions.
+- AGENTS.md files need not live only in Git repos. For example, you may find one in your home directory.
+- If the AGENTS.md includes programmatic checks to verify your work, you MUST run all of them and make a best effort to validate that the checks pass AFTER all code changes have been made.
+  - This applies even for changes that appear simple, i.e. documentation. You still must run all of the programmatic checks.
+
+# Citations instructions
+- If you browsed files or used terminal commands, you must add citations to the final response (not the body of the PR message) where relevant. Citations reference file paths and terminal outputs with the following formats:
+  1) `F:<file_path>†L<line_start>(-L<line_end>)?`
+  - File path citations must start with `F:`. `file_path` is the exact file path of the file relative to the root of the repository that contains the relevant text.
+  - `line_start` is the 1-indexed start line number of the relevant output within that file.
+  2) `<chunk_id>†L<line_start>(-L<line_end>)?`
+  - Where chunk_id is the chunk_id of the terminal output, line_start and line_end are the 1-indexed start and end line numbers of the relevant output within that chunk.
+- Line ends are optional, and if not provided, line end is the same as line start, so only 1 line is cited.
+- Ensure that the line numbers are correct, and that the cited file paths or terminal outputs are directly relevant to the word or clause before the citation.
+- Do not cite completely empty lines inside the chunk, only cite lines that have content.
+- Only cite from file paths and terminal outputs, DO NOT cite from previous pr diffs and comments, nor cite git hashes as chunk ids.
+- Use file path citations that reference any code changes, documentation or files, and use terminal citations only for relevant terminal output.
+- Prefer file citations over terminal citations unless the terminal output is directly relevant to the clauses before the citation, i.e. clauses on test results.
+  - For PR creation tasks, use file citations when referring to code changes in the summary section of your final response, and terminal citations in the testing section.
+  - For question-answering tasks, you should only use terminal citations if you need to programmatically verify an answer (i.e. counting lines of code). Otherwise, use file citations.
+
+Add the crucial information that the branch name should be written in english if not please translate it into english.
+
+Before commit run 'bundle exec rspec' 'bundle exec bundler-audit --update' and 'bundle exec rubocop --parallel' and the next fix the problems if appear

data/README.md

@@ -16,8 +16,8 @@ It preserves:
 - Documentation comments (e.g., # @param id)
 
 ## Table of Contents
-1. [When to Use This Gem](#When-to-Use-This-Gem)
-2. [When Not to Use This Gem](#When-Not-to-Use-This-Gem)
+1. [When to Use This Gem](#when-to-use-this-gem)
+2. [When Not to Use This Gem](#when-not-to-use-this-gem)
 3. [Installation](#Installation)
 4. [Usage](#Usage)
    - [Audit Mode](#Audit-Mode)
@@ -29,21 +29,22 @@ It preserves:
 
 ## When to Use This Gem
 
-NoComments is particularly useful in the following scenarios:
+NoComments keeps Ruby code tidy by automatically removing unnecessary comments. It can be integrated into an MCP server to sanitize scripts before deployment, ensuring that the code published on the server is clean and production ready.
 
-- **Auto-Generated or Boilerplate Code:** Helps clean up scaffolding or boilerplate comments, commonly generated in frameworks like Rails.
-- **Codebases with Overused Comments:** Removes redundant comments that restate obvious code.
-- **Continuous Integration/Code Review Workflows:** Automates comment cleanup during CI/CD processes.
-- **Educational Projects:** Streamlines code by removing teaching-related comments after the learning phase.
-- **Maintaining Open Source Projects:** Ensures consistency by removing unnecessary comments in contributions.
+**Example use cases:**
+- **Auto-generated code** – clear scaffolding comments produced by frameworks such as Rails.
+- **Projects with excessive comments** – remove remarks that simply restate obvious code.
+- **CI/CD pipelines** – incorporate NoComments as a step that enforces code cleanliness.
+- **Educational projects** – clean files once learning phases are complete.
+- **Maintaining open source projects** – keep contributions consistent and readable.
 
 ## When Not to Use This Gem
 
-While the gem is powerful, it may not be suitable in these scenarios:
+While NoComments streamlines your code, it is not a replacement for documentation comments or notes that are required for compliance.
 
-- **Code with Valuable Documentation Comments:** Avoid using the gem if your code relies on comments for explaining complex logic or business rules.
-- **Codebases Requiring Compliance:** In regulated industries, where specific comments are required for audits or documentation purposes.
-- **Highly Dynamic Environments:** In fast-changing projects, comments might provide crucial context for recent changes or decisions.
+- **Code with valuable documentation** – when comments explain complex algorithms or important business decisions.
+- **Regulated industries** – if comments are mandatory for audit purposes.
+- **Rapidly changing projects** – when comments capture ongoing discussions or decisions.
 
 ## Installation
 
@@ -77,6 +78,12 @@ NoComments::Remover.clean('path/to/your_file.rb')
 
 # Clean all `.rb` files in a directory
 NoComments::Remover.clean('path/to/your_directory')
+
+# Clean all `.rb` files except selected ones or directories
+NoComments::Remover.clean('path/to/your_directory', exclude: ['skip.rb', 'subdir'])
+# Multiple files or directories can be provided, and blank entries are ignored.
+# Absolute or relative paths work, and directories may have a trailing slash.
+NoComments::Remover.clean('/abs/path/project', exclude: ['/abs/path/project/subdir/'])
 ```
 ### Audit Mode
 Audit mode allows you to preview the comments that would be removed without modifying the files. Use the `audit: true` flag with the `clean` method:
@@ -119,6 +126,13 @@ no_comments -p path/to/file.rb --keep-doc-comments
 no_comments -p path/to/directory --keep-doc-comments
 ```
 
+#### Exclude Paths
+```bash
+no_comments -p path/to/directory --exclude file1.rb,subdir
+# absolute paths and trailing slashes are supported. Blank entries are ignored.
+no_comments -p /project --exclude /project/subdir/
+```
+
 ## Testing
 
 This gem uses RSpec for testing. To run tests:
@@ -127,7 +141,7 @@ This gem uses RSpec for testing. To run tests:
 ```bash
 bundle install
 ```
-2. Run the test suite::
+2. Run the test suite:
 ```bash
 bundle exec rspec
 ```
@@ -159,7 +173,7 @@ Please ensure your code follows the existing style and that all tests pass befor
 This gem is available as open source under the terms of the [MIT License](https://opensource.org/licenses/MIT).
 
 ## TODO
-- **Selective Cleaning:**
+- [x] **Selective Cleaning:**
   - Allow users to clean all files in a directory except for specified ones.
 
 ---

data/exe/no_comments

@@ -19,11 +19,20 @@ OptionParser.new do |opts|
   opts.on("-d", "--keep-doc-comments", "Preserve documentation comments like @param") do
     options[:keep_doc_comments] = true
   end
+
+  opts.on("-e", "--exclude x,y,z", Array, "Files to exclude when cleaning a directory") do |list|
+    options[:exclude] = list
+  end
 end.parse!
 
 if options[:path]
   begin
-    NoComments::Remover.clean(options[:path], audit: options[:audit], keep_doc_comments: options[:keep_doc_comments])
+    NoComments::Remover.clean(
+      options[:path],
+      audit: options[:audit],
+      keep_doc_comments: options[:keep_doc_comments],
+      exclude: options[:exclude] || []
+    )
     if options[:audit]
       puts "Audit completed successfully."
     else

data/lib/no_comments/comment_detector.rb

@@ -5,6 +5,8 @@ module NoComments
     MAGIC_COMMENT_REGEX = /\A#.*\b(?:frozen_string_literal|encoding|coding|warn_indent|fileencoding)\b.*\z/
     TOOL_COMMENT_REGEX = /\A#\s*(?:rubocop|reek|simplecov|coveralls|pry|byebug|noinspection|sorbet|type)\b/
     DOC_COMMENT_REGEX = /\A#\s*@\w+/
+    NODOC_STRING = ":nodoc:"
+    CLASS_OR_MODULE_REGEX = /\A(?:class|module)\b/
     def magic_comment?(stripped_line)
       stripped_line.match?(MAGIC_COMMENT_REGEX)
     end
@@ -13,8 +15,12 @@ module NoComments
       stripped_line.match?(TOOL_COMMENT_REGEX)
     end
 
-    def documentation_comment?(stripped_line)
-      stripped_line.match?(DOC_COMMENT_REGEX)
+    def documentation_comment?(stripped_line, next_line = nil)
+      return true if stripped_line.match?(DOC_COMMENT_REGEX)
+      return true if stripped_line.include?(NODOC_STRING)
+      return true if next_line&.match?(CLASS_OR_MODULE_REGEX)
+
+      false
     end
   end
 end

data/lib/no_comments/content_processor.rb

@@ -20,41 +20,42 @@ module NoComments
 
     def process(content)
       lines = content.lines
-      lines.each do |line|
-        @line_number += 1
+      lines.each_with_index do |line, index|
+        @line_number = index + 1
         stripped_line = line.strip
-        process_line(line, stripped_line)
+        next_line = next_non_comment_line(lines, index)
+        process_line(line, stripped_line, next_line)
       end
       cleaned_content = @result_lines.join("\n")
       cleaned_content += "\n" unless cleaned_content.empty?
       [cleaned_content, @comments]
     end
 
-    def process_line(line, stripped_line)
+    def process_line(line, stripped_line, next_line)
       if @in_multiline_comment
         handle_multiline_comment(stripped_line)
       elsif @in_heredoc
         handle_heredoc(line, stripped_line)
       elsif !@code_started
-        handle_initial_lines(line, stripped_line)
+        handle_initial_lines(line, stripped_line, next_line)
       else
-        handle_regular_line(line, stripped_line)
+        handle_regular_line(line, stripped_line, next_line)
       end
     end
 
-    def handle_initial_lines(line, stripped_line)
+    def handle_initial_lines(line, stripped_line, next_line)
       if stripped_line.empty? || stripped_line.start_with?("#!") || magic_comment?(stripped_line)
         @result_lines << line.rstrip
       elsif stripped_line.start_with?("#")
-        handle_initial_comment_line(line, stripped_line)
+        handle_initial_comment_line(line, stripped_line, next_line)
       else
         @code_started = true
-        handle_regular_line(line, stripped_line)
+        handle_regular_line(line, stripped_line, next_line)
       end
     end
 
-    def handle_initial_comment_line(line, stripped_line)
-      if tool_comment?(stripped_line) || (@keep_doc_comments && documentation_comment?(stripped_line))
+    def handle_initial_comment_line(line, stripped_line, next_line)
+      if tool_comment?(stripped_line) || (@keep_doc_comments && documentation_comment?(stripped_line, next_line&.strip))
         @result_lines << line.rstrip
       else
         @comments << [@line_number, stripped_line]
@@ -73,19 +74,21 @@ module NoComments
       )
     end
 
-    def handle_regular_line(line, stripped_line)
+    # rubocop:disable Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
+    def handle_regular_line(line, stripped_line, next_line)
       if stripped_line == "=begin"
         start_multiline_comment(stripped_line)
       elsif (heredoc_start = detect_heredoc_start(line))
         start_heredoc(line, heredoc_start)
       elsif stripped_line.start_with?("#") &&
             (tool_comment?(stripped_line) ||
-             (@keep_doc_comments && documentation_comment?(stripped_line)))
+             (@keep_doc_comments && documentation_comment?(stripped_line, next_line&.strip)))
         @result_lines << line.rstrip
       else
         process_code_line(line)
       end
     end
+    # rubocop:enable Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
 
     def start_multiline_comment(stripped_line)
       @in_multiline_comment = true
@@ -113,6 +116,19 @@ module NoComments
 
     private
 
+    def next_non_comment_line(lines, index)
+      j = index + 1
+      while j < lines.length
+        line = lines[j]
+        stripped = line.strip
+        return nil if stripped.empty?
+        return line unless stripped.start_with?("#")
+
+        j += 1
+      end
+      nil
+    end
+
     def handle_empty_line(line)
       @result_lines << line.rstrip
     end

data/lib/no_comments/remover.rb

@@ -4,9 +4,14 @@ require "no_comments/version"
 require "no_comments/content_processor"
 module NoComments
   class Remover
-    def self.clean(file_path, audit: false, keep_doc_comments: false)
+    def self.clean(file_path, audit: false, keep_doc_comments: false, exclude: [])
       if File.directory?(file_path)
+        filtered = exclude.reject { |p| p.nil? || p.strip.empty? }
+        excluded = filtered.map { |f| File.expand_path(f, file_path) }
         Dir.glob("#{file_path}/**/*.rb").each do |file|
+          absolute = File.expand_path(file)
+          next if excluded_file?(absolute, excluded)
+
           process_file(file, audit: audit, keep_doc_comments: keep_doc_comments)
         end
       else
@@ -14,6 +19,13 @@ module NoComments
       end
     end
 
+    def self.excluded_file?(absolute, paths)
+      paths.any? do |path|
+        absolute == path ||
+          (File.directory?(path) && absolute.start_with?(File.join(path, "")))
+      end
+    end
+
     def self.process_file(file_path, audit: false, keep_doc_comments: false)
       validate_file_extension(file_path)
       content = File.read(file_path)

data/lib/no_comments/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module NoComments
-  VERSION = "0.1.11"
+  VERSION = "0.1.14"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: no_comments
 version: !ruby/object:Gem::Version
-  version: 0.1.11
+  version: 0.1.14
 platform: ruby
 authors:
 - Justyna
@@ -18,6 +18,7 @@ extra_rdoc_files: []
 files:
 - ".rspec"
 - ".rubocop.yml"
+- AGENTS.md
 - LICENSE
 - README.md
 - Rakefile