[build] Expand clang-tidy coverage

Author: fhanau

.clang-tidy

@@ -16,11 +16,13 @@ Checks: >
   bugprone-return-const-ref-from-parameter,
   bugprone-suspicious-*,
   -bugprone-suspicious-semicolon,
+  bugprone-switch-missing-default-case,
   bugprone-undefined-memory-manipulation,
   bugprone-unhandled-self-assignment,
   bugprone-unused-raii,
   bugprone-use-after-move,
   cppcoreguidelines-c-copy-assignment-signature,
+  cppcoreguidelines-interfaces-global-init,
   cppcoreguidelines-misleading-capture-default-by-value,
   cppcoreguidelines-noexcept-destructor,
   cppcoreguidelines-prefer-member-initializer,
@@ -32,6 +34,7 @@ Checks: >
   misc-unused-alias-decls,
   misc-unused-using-decls,
   modernize-avoid-variadic-functions,
+  modernize-loop-convert,
   modernize-macro-to-enum,
   modernize-redundant-void-arg,
   modernize-type-traits,
@@ -72,6 +75,7 @@ Checks: >
 # modernize-use-override
 # modernize-use-ranges
 # readability-avoid-return-with-void-value
+# readability-convert-member-functions-to-static
 # readability-redundant-smartptr-get
 # readability-use-anyofallof
 

src/workerd/api/analytics-engine-impl.h

@@ -20,6 +20,7 @@ void setDoubles(Message msg, kj::ArrayPtr<double> arr, kj::StringPtr errorPrefix
 
   uint index = 1;
   for (auto& item: arr) {
+    // NOLINTNEXTLINE(bugprone-switch-missing-default-case)
     switch (index) {
       case 1:
         msg.setDouble1(item);
@@ -108,6 +109,7 @@ void setBlobs(Message msg,
       sizeSum += value.size();
       JSG_REQUIRE(sizeSum <= MAX_CUMULATIVE_BYTES_IN_BLOBS, TypeError, errorPrefix,
           "Cumulative size of blobs exceeds ", MAX_CUMULATIVE_BYTES_IN_BLOBS, " bytes).");
+      // NOLINTNEXTLINE(bugprone-switch-missing-default-case)
       switch (index) {
         case 1:
           msg.setBlob1(value);

src/workerd/api/crypto/aes.c++

@@ -475,8 +475,9 @@ class AesCtrKey final: public AesKeyBase {
         return *EVP_aes_192_ctr();
       case 32:
         return *EVP_aes_256_ctr();
+      default:
+        KJ_FAIL_ASSERT("CryptoKey has invalid data length");
     }
-    KJ_FAIL_ASSERT("CryptoKey has invalid data length");
   }
 
   jsg::BufferSource encryptOrDecrypt(jsg::Lock& js,

src/workerd/api/crypto/ec.c++

@@ -1120,9 +1120,9 @@ kj::OneOf<jsg::Ref<CryptoKey>, CryptoKeyPair> EdDsaKey::generateKey(jsg::Lock& j
     case NID_X25519:
       return generateKeyImpl<X25519_PUBLIC_VALUE_LEN, X25519_keypair>(js, normalizedName, nid,
           privateKeyUsages, publicKeyUsages, extractablePrivateKey, "X25519"_kj);
+    default:
+      KJ_FAIL_REQUIRE("ED ", normalizedName, " unimplemented", nid);
   }
-
-  KJ_FAIL_REQUIRE("ED ", normalizedName, " unimplemented", nid);
 }
 
 }  // namespace

src/workerd/io/trace.c++

@@ -1156,8 +1156,8 @@ kj::Maybe<kj::Array<kj::String>> getScriptTagsFromReader(const rpc::Trace::Onset
   if (reader.hasScriptTags()) {
     auto tags = reader.getScriptTags();
     kj::Vector<kj::String> scriptTags(tags.size());
-    for (size_t i = 0; i < tags.size(); i++) {
-      scriptTags.add(kj::str(tags[i]));
+    for (auto tag: tags) {
+      scriptTags.add(kj::str(tag));
     }
     return kj::Maybe(scriptTags.releaseAsArray());
   }

src/workerd/jsg/url-test.c++

@@ -1215,8 +1215,8 @@ KJ_TEST("Special scheme URLS") {
     kj::str("file:///example"),
   };
 
-  for (auto n = 0; n < kj::size(tests); n++) {
-    KJ_ASSERT_NONNULL(Url::tryParse(tests[n].asPtr()));
+  for (const auto& test: tests) {
+    KJ_ASSERT_NONNULL(Url::tryParse(test.asPtr()));
   }
 }
 

src/workerd/jsg/value-test.c++

@@ -1219,8 +1219,9 @@ struct ExceptionContext: public ContextGlobalObject {
         return JSG_KJ_EXCEPTION(FAILED, TypeError, "boom");
       case 2:
         return JSG_KJ_EXCEPTION(FAILED, DOMAbortError, "boom");
+      default:
+        KJ_UNREACHABLE;
     }
-    KJ_UNREACHABLE;
   }
 
   JSG_RESOURCE_TYPE(ExceptionContext) {

src/workerd/tests/bench-api-headers.c++

@@ -92,8 +92,7 @@ BENCHMARK_F(ApiHeaders, set_append)(benchmark::State& state) {
       for (size_t i = 0; i < 1000; ++i) {
         auto headers = js.alloc<api::Headers>();
         // Set common headers with various representative lengths
-        for (int n = 0; n < 13; n++) {
-          auto& h = kHeaders[n];
+        for (auto& h: kHeaders) {
           if (h.append) {
             headers->append(env.js, kj::str(h.name), kj::str(h.value));
           } else {