Fix selection tree, add some unit tests (#154)

Co-authored-by: ykiko <ykikoykikoykiko@gmail.com>

Author: aurora0x27

.github/workflows/check-format.yml

@@ -0,0 +1,35 @@
+name: format
+
+on:
+  pull_request:
+  push:
+    branches: [main]
+
+jobs:
+  check:
+    runs-on: ubuntu-latest
+
+    steps:
+    - name: Checkout code
+      uses: actions/checkout@v4
+
+    - name: Install clang-format
+      run: |
+        wget -qO - https://apt.llvm.org/llvm-snapshot.gpg.key | sudo tee /etc/apt/trusted.gpg.d/llvm.asc
+        echo "deb http://apt.llvm.org/noble/ llvm-toolchain-noble-20 main" | sudo tee /etc/apt/sources.list.d/llvm20.list
+        echo "deb-src http://apt.llvm.org/noble/ llvm-toolchain-noble-20 main" | sudo tee -a /etc/apt/sources.list.d/llvm20.list
+        sudo apt-get update && sudo apt-get install -y clang-format-20
+        clang-format-20 --version
+
+    - name: Run clang-format check
+      run: |
+        FILES=$(find ./ -type f | grep -P '^\.\/(src|include)(\/[^\/]+)*\/[^\/]+\.(h|cpp|cc)$')
+
+        UNFORMATTED=$(clang-format-20 --dry-run --Werror $FILES)
+
+        if [ $? -ne 0 ]; then
+          echo "× Some files are not properly formatted. Run clang-format to fix them."
+          exit 1
+        else
+          echo "✓ All files are properly formatted."
+        fi

.pre-commit-config.yaml

@@ -1,4 +1,4 @@
-files: '^(src|include)(/[^/]+)*/[^/]+\.(h|cpp)$'
+files: '^(src|include)(/[^/]+)*/[^/]+\.(h|cpp|cc)$'
 repos:
   - repo: https://github.com/pre-commit/mirrors-clang-format
     rev: "v20.1.0"

include/AST/SourceCode.h

@@ -28,7 +28,6 @@ struct LocalSourceRange {
     }
 };
 
-
 /// Get the content of the file with the given file ID.
 llvm::StringRef getFileContent(const clang::SourceManager& SM, clang::FileID fid);
 

include/Async/Task.h

@@ -290,9 +290,9 @@ class Task {
         promise_base* handle = core;
         while(handle) {
             clice::println("{}:{}:{}",
-                    handle->location.file_name(),
-                    handle->location.line(),
-                    handle->location.function_name());
+                           handle->location.file_name(),
+                           handle->location.line(),
+                           handle->location.function_name());
             handle = handle->continuation;
         }
     }

include/Async/libuv.h

@@ -59,7 +59,8 @@ T* uv_cast(U& u) {
     return reinterpret_cast<T*>(&u);
 }
 
-void uv_check_result(const int result, const std::source_location location = std::source_location::current());
+void uv_check_result(const int result,
+                     const std::source_location location = std::source_location::current());
 
 template <typename T>
 class Task;

include/Compiler/Command.h

@@ -79,7 +79,8 @@ class CompilationDatabase {
     auto load_commands(this Self& self, llvm::StringRef json_content)
         -> std::expected<std::vector<UpdateInfo>, std::string>;
 
-    auto get_command(this Self& self, llvm::StringRef file, bool resource_dir = false) -> LookupInfo;
+    auto get_command(this Self& self, llvm::StringRef file, bool resource_dir = false)
+        -> LookupInfo;
 
 private:
     /// The memory pool to hold all cstring and command list.

include/Test/CTest.h

@@ -80,6 +80,7 @@ struct Tester {
 
     Tester& compile(llvm::StringRef standard = "-std=c++20") {
         auto command = std::format("clang++ {} {} -fms-extensions", standard, src_path);
+
         database.update_command("fake", src_path, command);
         params.arguments = database.get_command(src_path).arguments;
 

src/AST/Selection.cpp

@@ -79,6 +79,11 @@ struct SelectionBuilder {
 
     template <typename Node, typename Callback>
     bool hook(const Node* node, const Callback& callback) {
+
+        if(!node) {
+            return true;
+        }
+
         if constexpr(requires { node->isImplicit(); })
             if(node->isImplicit())
                 return true;

src/Async/Async.cpp

@@ -4,7 +4,6 @@
 
 namespace clice::async {
 
-
 /// The default event loop.
 uv_loop_t* loop = nullptr;
 

src/Async/libuv.cpp

@@ -30,10 +30,7 @@ const std::error_category& category() {
 void uv_check_result(const int result, const std::source_location location) {
     if(result < 0) {
         log::warn("libuv error: {}", uv_strerror(result));
-        log::warn("At {}:{}:{}", 
-                  location.file_name(), 
-                  location.line(),
-                  location.function_name());
+        log::warn("At {}:{}:{}", location.file_name(), location.line(), location.function_name());
     }
 }