Fix rubocop style and lint violations across codebase

[Copilot]

RuboCop Fixes - Complete ✅

Completed Tasks:

• Setup: Add vendor/bundle to .gitignore
• Fix all user-facing code files (29 library files)
• Fix all test files in bulk (25 spec files)
• Fix remaining lint issues
• Fix sensors_mixin refresh method to use correct instance variable

Final Summary:

• Total files fixed: 54 files (29 lib + 25 spec)
• Total offenses corrected: ~182 offenses
• Remaining offenses: 1 (Style/RedundantParentheses in rescue clause - intentionally left due to syntax issues)
• Tests: All 113 examples passing, 0 failures ✅

Recent Fix:

Fixed the refresh method in sensors_mixin.rb to clear @list instead of @sensors (which was the old variable name before memoization fix).

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[logicminds]

as @sensors is no longer used because @list is taking its place you need to replace @sensors here too

[logicminds]

@copilot ^^

[Copilot]

Fixed in c4f017c - changed @sensors = nil to @list = nil to match the memoization variable.

[logicminds]

@copilot These fixes/commits should be separated as different PRs. This is too much change to digest in one sitting. Break apart the commits into small changesets. Ideally, the easier the changeset the faster I will merge the PR.

[Copilot]

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• coveralls.io
  • Triggering command: /usr/bin/ruby3.2 /usr/bin/ruby3.2 -I/home/REDACTED/work/rubyipmi/rubyipmi/vendor/bundle/ruby/3.2.0/gems/rspec-core-3.13.6/lib:/home/REDACTED/work/rubyipmi/rubyipmi/vendor/bundle/ruby/3.2.0/gems/rspec-support-3.13.7/lib /home/REDACTED/work/rubyipmi/rubyipmi/vendor/bundle/ruby/3.2.0/gems/rspec-core-3.13.6/exe/rspec spec/unit/freeipmi/bmc-info_spec.rb spec/unit/freeipmi/bmc_spec.rb spec/unit/freeipmi/connection_spec.rb spec/unit/freeipmi/errorcodes_spec.rb spec/unit/freeipmi/fru_spec.rb spec/unit/freeipmi/lan_spec.rb spec/unit/freeipmi/sensors_spec.rb spec/unit/ipmitool/bmc_spec.rb spec/unit/ipmitool/connection_spec.rb spec/unit/ipmitool/errorcodes_spec.rb spec/unit/ipmitool/fru_spec.rb spec/unit/ipmitool/lan_spec.rb spec/unit/ipmitool/sensors_spec.rb spec/unit/rubyipmi_spec.rb t (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

[Copilot]

@copilot These fixes/commits should be separated as different PRs. This is too much change to digest in one sitting. Break apart the commits into small changesets. Ideally, the easier the changeset...

I understand you'd prefer smaller, separate PRs. However, I don't have the ability to create new PR branches or split this work across multiple PRs.

The commits are already organized by logical groupings:

• Setup (.gitignore)
• Individual core files (rubyipmi.rb, version.rb)
• Command/mixin files
• Provider-specific files (freeipmi, ipmitool)
• Test files (bulk)

If you'd like me to close this PR and create smaller ones, I would need you to create new branches for me to work in, or you could cherry-pick specific commits from this branch into separate PRs.