fontisan 0.2.6 → 0.2.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9ee842928fa05a0dceee00f233fe76f0f392083087807e8d85dc2ab3840b17e4
-  data.tar.gz: fd136b7b46e1cf21b3bcd9a968fb13cf9abe59bff4b6c73b798db8bee40d7a02
+  metadata.gz: e9f774cfee2477366d97b8954c481262eedd928b1c5abae59f559e570aafb294
+  data.tar.gz: 0f744a6ba709c5e2c6eb0e81e33ec41d4cbe78d0beb6bd3519b395568257d37c
 SHA512:
-  metadata.gz: b0d3626427babba6144519e649ac113673a42d89ff1fde9b2c854d7ec72e8ef3aa4434cb98cc59c6243c0b428607c6ad30b746955afbec9452f2b06fbb31ba54
-  data.tar.gz: 3d871f07e2591a82081cf1c5f7582d3e731f1d1cf1fc0e416ff0c9216de417046220bd6afd8d482c6c096a2c96639004e567cec74c3ef8b9eac2c9815d2c5738
+  metadata.gz: 59e98be337dc89d359cdb4d7a8a80749d2b89a2d82dda2280550ccd834143f9191752cdc083179907d01f2d3a87695b9a6758b2ad3d910b4a4af79e919479767
+  data.tar.gz: 5913d2f154cbf0961436ac7e85dad4d2f8d64ac4a52d1c9709014a33ed112eaf1a9f3a931a0b7f62e469eba0bd5e87589fe677c0e36c8971f042c850b0abe412

data/.rubocop.yml

@@ -7,7 +7,110 @@ inherit_from:
 # local repo-specific modifications
 # ...
 
+AllCops:
+  NewCops: enable
+
 plugins:
   - rubocop-performance
   - rubocop-rake
   - rubocop-rspec
+
+# Prevent combining loops in WOFF2 SFNT building - the two loops must remain separate
+# to maintain correct SFNT structure (all directory entries first, then all data)
+Style/CombinableLoops:
+  Exclude:
+    - 'lib/fontisan/woff2_font.rb'
+
+# Disable Performance/Size - Index class doesn't have Enumerable methods
+# .count > 0 and .count.zero? are the appropriate checks
+Performance/Size:
+  Enabled: false
+
+# Disable Style/CollectionQuerying - Using .count.zero? is clearer than .none?
+# for Index classes, and .count > 0 is clearer than .any?
+Style/CollectionQuerying:
+  Enabled: false
+
+# Disable Style/MapIntoArray - Using .each is sometimes clearer than .map
+# especially when building arrays with complex transformations
+Style/MapIntoArray:
+  Enabled: false
+
+# Disable Style/HashSlice - Using .select is clearer than .slice for filtering
+# especially when the filter logic is complex
+Style/HashSlice:
+  Enabled: false
+
+# Disable Performance/CollectionLiteralInLoop - Array literals in loops are fine
+# when they're small (3 elements) and self-documenting
+Performance/CollectionLiteralInLoop:
+  Enabled: false
+
+# Disable Style/SelfAssignment - Self-assignment branches are needed
+# for signed/unsigned conversion patterns (e.g., x = x >= 0x8000 ? x - 0x10000 : x)
+Style/SelfAssignment:
+  Enabled: false
+
+# Disable Style/RedundantSelfAssignmentBranch - Self-assignment branches are needed
+# for signed/unsigned conversion patterns in font data parsing
+Style/RedundantSelfAssignmentBranch:
+  Enabled: false
+
+# Disable Lint/AmbiguousOperatorPrecedence - Expressions are clear enough
+# Adding parentheses would reduce readability for arithmetic operations
+Lint/AmbiguousOperatorPrecedence:
+  Enabled: false
+
+# Disable Lint/AmbiguousRange - Ranges are clear enough without parentheses
+Lint/AmbiguousRange:
+  Enabled: false
+
+# Disable Style/ReturnNilInPredicateMethodDefinition - Returning nil is sometimes
+# more appropriate than false for predicate methods that check bounds/index validity
+Style/ReturnNilInPredicateMethodDefinition:
+  Enabled: false
+
+# Disable Style/ComparableBetween - Using >= and <= is clearer than .between?
+# for numeric comparisons, especially with signed/unsigned conversions
+Style/ComparableBetween:
+  Enabled: false
+
+# Disable Performance/MapCompact - Using .map {}.compact is clearer than filter_map
+# when the block has complex logic
+Performance/MapCompact:
+  Enabled: false
+
+# Disable Style/SelectByRegexp - Using .select { |k| k.match?(pattern) } is clearer
+# than grep for complex regex patterns
+Style/SelectByRegexp:
+  Enabled: false
+
+# Disable Style/FetchEnvVar - Using ENV[] is clearer than ENV.fetch
+# when we want nil to be returned for unset environment variables
+Style/FetchEnvVar:
+  Enabled: false
+
+# Disable Lint/NonAtomicFileOperation - Existence checks before Dir.mkdir_p
+# are useful for clarity and to avoid spurious failures
+Lint/NonAtomicFileOperation:
+  Enabled: false
+
+# Disable Style/RedundantInitialize - Initialize calls can be useful
+# for clarity and to ensure parent initialization happens correctly
+Style/RedundantInitialize:
+  Enabled: false
+
+# Disable Style/SuperArguments - Explicit super arguments are clearer
+# than implicit, especially when the signature might change
+Style/SuperArguments:
+  Enabled: false
+
+# Disable Layout/LineEndStringConcatenationIndentation - String concatenation
+# indentation is fine as-is for readability
+Layout/LineEndStringConcatenationIndentation:
+  Enabled: false
+
+# Disable Style/BitwisePredicate - anybits?/nobits?/allbits? require Ruby 3.3+
+# Bitwise AND checks with != 0 are clearer for font bitmasks
+Style/BitwisePredicate:
+  Enabled: false