Very minor cleanup and comments (#1)

corsonknowles · web-flow · commit f9f28c085b2f · 2025-11-24T10:08:28.000-08:00
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -16,7 +16,7 @@ jobs:
         ruby:
           - '3.4'
           - '3.3'
-          - '3.2'
+          - '3.2' # EOL: 2026-03-31 (expected)
 
     steps:
       - uses: actions/checkout@v4
diff --git a/README.md b/README.md
@@ -7,14 +7,16 @@ Chatwerk provides AI tool integration for the [QueryPackwerk](https://github.com
 
 ## Installation
 
-Install the gem, either add in to your packwerk'd application's Gemfile:
+Install the gem. 
+
+Either add it to your packwerk'd application's Gemfile:
 
 ```ruby
 $ bundle add chatwerk
 $ bundle install
 ```
 
-or install it on its own:
+Or, install it on its own:
 
 ```bash
 $ gem install chatwerk
diff --git a/lib/chatwerk/cli.rb b/lib/chatwerk/cli.rb
@@ -1,4 +1,3 @@
-# typed: false
 # frozen_string_literal: true
 
 require 'thor'
diff --git a/lib/chatwerk/helpers.rb b/lib/chatwerk/helpers.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Chatwerk
   module Helpers
     def chdir(&)
diff --git a/lib/chatwerk/mcp.rb b/lib/chatwerk/mcp.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'mcp'
 
 module Chatwerk
diff --git a/lib/chatwerk/tools/print_env_tool.rb b/lib/chatwerk/tools/print_env_tool.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'mcp'
 require_relative '../helpers'
 
diff --git a/lib/chatwerk/version.rb b/lib/chatwerk/version.rb
@@ -1,4 +1,3 @@
-# typed: false
 # frozen_string_literal: true
 
 module Chatwerk
diff --git a/lib/chatwerk/views.rb b/lib/chatwerk/views.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Chatwerk
   module Views
     autoload :NoPackagesView, 'chatwerk/views/no_packages_view'
diff --git a/spec/chatwerk/api_spec.rb b/spec/chatwerk/api_spec.rb
@@ -1,8 +1,5 @@
-# typed: false
 # frozen_string_literal: true
 
-require 'spec_helper'
-
 RSpec.describe Chatwerk::Api do
   let(:pwd) { '/test/workspace' }
   let(:package_path) { 'packs/test_package' }
diff --git a/spec/chatwerk/error_spec.rb b/spec/chatwerk/error_spec.rb
@@ -1,4 +1,4 @@
-require 'spec_helper'
+# frozen_string_literal: true
 
 RSpec.describe Chatwerk::Error do
   context 'with a Chatwerk::Error' do
diff --git a/spec/chatwerk/helpers_spec.rb b/spec/chatwerk/helpers_spec.rb
@@ -1,4 +1,4 @@
-require 'spec_helper'
+# frozen_string_literal: true
 
 RSpec.describe Chatwerk::Helpers do
   describe '.chdir' do
diff --git a/spec/chatwerk/mcp_tools_spec.rb b/spec/chatwerk/mcp_tools_spec.rb
@@ -1,9 +1,5 @@
-# typed: false
 # frozen_string_literal: true
 
-require 'spec_helper'
-require 'chatwerk/mcp'
-
 RSpec.describe 'MCP Tools' do
   let(:server_context) { double('server_context') }
 
diff --git a/spec/chatwerk/views/no_packages_view_spec.rb b/spec/chatwerk/views/no_packages_view_spec.rb
@@ -1,4 +1,4 @@
-require 'spec_helper'
+# frozen_string_literal: true
 
 RSpec.describe Chatwerk::Views::NoPackagesView do
   context 'when packwerk.yml exists' do
diff --git a/spec/chatwerk/views/no_violations_view_spec.rb b/spec/chatwerk/views/no_violations_view_spec.rb
@@ -1,4 +1,4 @@
-require 'spec_helper'
+# frozen_string_literal: true
 
 RSpec.describe Chatwerk::Views::NoViolationsView do
   it 'renders a message indicating no violations found' do
diff --git a/spec/chatwerk/views/package_view_spec.rb b/spec/chatwerk/views/package_view_spec.rb
@@ -1,4 +1,4 @@
-require 'spec_helper'
+# frozen_string_literal: true
 
 RSpec.describe Chatwerk::Views::PackageView do
   context 'with a valid package' do
diff --git a/spec/chatwerk/views/packages_view_spec.rb b/spec/chatwerk/views/packages_view_spec.rb
@@ -1,4 +1,4 @@
-require 'spec_helper'
+# frozen_string_literal: true
 
 RSpec.describe Chatwerk::Views::PackagesView do
   context 'when packages are found' do
diff --git a/spec/chatwerk/views/violations_details_view_spec.rb b/spec/chatwerk/views/violations_details_view_spec.rb
@@ -1,4 +1,4 @@
-require 'spec_helper'
+# frozen_string_literal: true
 
 RSpec.describe Chatwerk::Views::ViolationsDetailsView do
   context 'with violations having source locations' do
diff --git a/spec/chatwerk/views/violations_list_view_spec.rb b/spec/chatwerk/views/violations_list_view_spec.rb
@@ -1,4 +1,4 @@
-require 'spec_helper'
+# frozen_string_literal: true
 
 RSpec.describe Chatwerk::Views::ViolationsListView do
   context 'with multiple violations' do
diff --git a/spec/chatwerk_spec.rb b/spec/chatwerk_spec.rb
@@ -1,4 +1,3 @@
-# typed: false
 # frozen_string_literal: true
 
 RSpec.describe Chatwerk do
diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb
@@ -1,4 +1,3 @@
-# typed: false
 # frozen_string_literal: true
 
 require 'chatwerk'

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,3 @@`
`1`		`-# typed: false`
`2`	`1`	`# frozen_string_literal: true`
`3`	`2`
`4`	`3`	`require 'thor'`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`module Chatwerk`
`2`	`4`	`module Helpers`
`3`	`5`	`def chdir(&)`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`require 'mcp'`
`2`	`4`
`3`	`5`	`module Chatwerk`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-require 'spec_helper'`
	`1`	`+# frozen_string_literal: true`
`2`	`2`
`3`	`3`	`RSpec.describe Chatwerk::Error do`
`4`	`4`	`context 'with a Chatwerk::Error' do`