diff --git a/src/context.h b/src/context.h
index 868f069..d6d094f 100644
--- a/src/context.h
+++ b/src/context.h
@@ -19,7 +19,10 @@
 
 namespace tint {
 
+/// Context object for Tint. Holds various global resources used through
+/// the system.
 struct Context {
+  /// Manager to hold all of the various type objects
   TypeManager* type_mgr = nullptr;
 };
 
diff --git a/src/reader/spirv/parser_impl_test_helper.h b/src/reader/spirv/parser_impl_test_helper.h
index 3e4b8d1..0838b00 100644
--- a/src/reader/spirv/parser_impl_test_helper.h
+++ b/src/reader/spirv/parser_impl_test_helper.h
@@ -17,6 +17,7 @@
 
 #include <memory>
 #include <string>
+#include <vector>
 
 #include "gtest/gtest.h"
 #include "src/context.h"
@@ -26,18 +27,24 @@
 namespace reader {
 namespace spirv {
 
+/// SPIR-V Parser test class
 class SpvParserTest : public testing::Test {
  public:
   SpvParserTest() = default;
   ~SpvParserTest() = default;
 
+  /// Sets up the test helper
   void SetUp() { ctx_.type_mgr = &tm_; }
 
+  /// Tears down the test helper
   void TearDown() {
     impl_ = nullptr;
     ctx_.type_mgr = nullptr;
   }
 
+  /// Retrieves the parser from the helper
+  /// @param input the string to parse
+  /// @returns the parser implementation
   ParserImpl* parser(const std::vector<uint32_t>& input) {
     impl_ = std::make_unique<ParserImpl>(ctx_, input);
     return impl_.get();
@@ -53,4 +60,4 @@
 }  // namespace reader
 }  // namespace tint
 
-#endif  // SRC_READER_WGSL_PARSER_IMPL_TEST_HELPER_H_
+#endif  // SRC_READER_SPIRV_PARSER_IMPL_TEST_HELPER_H_
diff --git a/src/reader/wgsl/parser_impl.cc b/src/reader/wgsl/parser_impl.cc
index eb66cd2..1b71bea 100644
--- a/src/reader/wgsl/parser_impl.cc
+++ b/src/reader/wgsl/parser_impl.cc
@@ -372,7 +372,7 @@
   std::tie(name, type) = variable_ident_decl();
   if (has_error())
     return nullptr;
-  if (name == "" || type == nullptr) {
+  if (name.empty() || type == nullptr) {
     set_error(peek(), "error parsing constant variable identifier");
     return nullptr;
   }
@@ -593,7 +593,7 @@
   std::tie(name, type) = variable_ident_decl();
   if (has_error())
     return nullptr;
-  if (name == "" || type == nullptr) {
+  if (name.empty() || type == nullptr) {
     set_error(peek(), "invalid identifier declaration");
     return nullptr;
   }
@@ -1102,7 +1102,7 @@
   std::tie(name, type) = variable_ident_decl();
   if (has_error())
     return nullptr;
-  if (name == "" || type == nullptr) {
+  if (name.empty() || type == nullptr) {
     set_error(peek(), "invalid identifier declaration");
     return nullptr;
   }
@@ -1283,7 +1283,7 @@
   std::tie(name, type) = variable_ident_decl();
   if (has_error())
     return {};
-  if (name == "" || type == nullptr)
+  if (name.empty() || type == nullptr)
     return {};
 
   for (;;) {
@@ -1300,7 +1300,7 @@
     std::tie(name, type) = variable_ident_decl();
     if (has_error())
       return {};
-    if (name == "" || type == nullptr) {
+    if (name.empty() || type == nullptr) {
       set_error(t, "found , but no variable declaration");
       return {};
     }
@@ -1683,7 +1683,7 @@
     std::tie(name, type) = variable_ident_decl();
     if (has_error())
       return nullptr;
-    if (name == "" || type == nullptr) {
+    if (name.empty() || type == nullptr) {
       set_error(peek(), "unable to parse variable declaration");
       return nullptr;
     }
diff --git a/src/reader/wgsl/parser_impl_test_helper.h b/src/reader/wgsl/parser_impl_test_helper.h
index a0d2033..63f50cb 100644
--- a/src/reader/wgsl/parser_impl_test_helper.h
+++ b/src/reader/wgsl/parser_impl_test_helper.h
@@ -26,23 +26,31 @@
 namespace reader {
 namespace wgsl {
 
+/// WGSL Parser test class
 class ParserImplTest : public testing::Test {
  public:
+  /// Constructor
   ParserImplTest() = default;
   ~ParserImplTest() = default;
 
+  /// Sets up the test helper
   void SetUp() { ctx_.type_mgr = &tm_; }
 
+  /// Tears down the test helper
   void TearDown() {
     impl_ = nullptr;
     ctx_.type_mgr = nullptr;
   }
 
+  /// Retrieves the parser from the helper
+  /// @param str the string to parse
+  /// @returns the parser implementation
   ParserImpl* parser(const std::string& str) {
     impl_ = std::make_unique<ParserImpl>(ctx_, str);
     return impl_.get();
   }
 
+  /// @returns the type manager
   TypeManager* tm() { return &tm_; }
 
  private:
diff --git a/src/writer/spirv/builder.h b/src/writer/spirv/builder.h
index fe56501..b075fa9 100644
--- a/src/writer/spirv/builder.h
+++ b/src/writer/spirv/builder.h
@@ -66,6 +66,7 @@
   }
 
   /// Retrives the id for the given function name
+  /// @param name the function name to search for
   /// @returns the id for the given name or 0 on failure
   uint32_t id_for_func_name(const std::string& name) {
     if (func_name_to_id_.count(name) == 0) {
diff --git a/src/writer/wgsl/generator_impl.h b/src/writer/wgsl/generator_impl.h
index 891073a..e77ee95 100644
--- a/src/writer/wgsl/generator_impl.h
+++ b/src/writer/wgsl/generator_impl.h
@@ -15,8 +15,10 @@
 #ifndef SRC_WRITER_WGSL_GENERATOR_IMPL_H_
 #define SRC_WRITER_WGSL_GENERATOR_IMPL_H_
 
+#include <memory>
 #include <sstream>
 #include <string>
+#include <vector>
 
 #include "src/ast/array_accessor_expression.h"
 #include "src/ast/const_initializer_expression.h"