style(ruff): enable B912 check and add strict=False to map calls

[schwehr]

Code Review & Suggestions for Improvement

1. Adopt strict=True for Equal-Length Vector Operations:
   • In boolean operations such as __xor__, __and__, __or__ (and their in-place equivalents), self.vector and bv2.vector are expected to have the same number of words after padding/resizing.
   • Changing strict=False to strict=True for these operations would catch any silent truncations or unexpected array size mismatches earlier as an explicit ValueError.
2. Replace map(...) with List/Generator Comprehensions or Vectorized Word Operations:
   • In methods like shift_left_by_one and shift_right_by_one, repeated construction of temporary lists ([1] * size, [63] * size) and multiple map() calls create unnecessary memory allocations. A single list comprehension or direct
     word loop across self.vector would be clearer and faster.
3. Enable B905 (zip-without-explicit-strict):
   • Similarly to B912, B905 is currently in the ignore list in pyproject.toml:79. Enabling B905 and adding explicit strict= arguments to zip() calls would further improve safety and consistency across the codebase.

[Copilot]

Copilot review overview

🟡 Changes recommended

A remaining map() call will fail the enabled B912 lint check, and boolean operations should use strict=True.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1 · 1

Open (2)

• Add strict argument to map call before enabling B912 · New
• Update rotation zip calls before enabling B905 · New

What changed in this PR

Updates Ruff configuration to enable B912 and adds explicit strict=False arguments to map() calls.

Changes:

• Removes B912 from Ruff’s ignore list.
• Adds explicit map strictness to BitVector operations.
• Leaves B905 ignored and one map() call without strict=.

File	Summary
pyproject.toml	Enables B912 linting; B905 remains ignored.
BitVector/​BitVector.py	Updates map calls, but leaves one B912 violation and uses non-strict boolean operations.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.