Format and simplify intergration test (#245)

Author: 16bit-ykiko

.dockerignore

@@ -1,2 +1,2 @@
 build/
-.cache
+.cache

.clangd

@@ -1,2 +1,2 @@
 Diagnostics:
-  UnusedIncludes: None
+  UnusedIncludes: None

.github/workflows/check-format.yml

@@ -4,15 +4,21 @@ on:
   push:
     branches: [main]
     paths:
-      - .clang-format
+      - ".clang-format"
+      - ".pre-commit-config.yaml"
+      - "**/*.py"
+      - "pyproject.toml"
       - "include/**"
       - "src/**"
       - "tests/**"
 
   pull_request:
     branches: [main]
     paths:
-      - .clang-format
+      - ".clang-format"
+      - ".pre-commit-config.yaml"
+      - "**/*.py"
+      - "pyproject.toml"
       - "include/**"
       - "src/**"
       - "tests/**"
@@ -25,23 +31,34 @@ jobs:
       - name: Checkout code
         uses: actions/checkout@v4
 
-      - name: Install clang-format
+      - name: Install uv
+        uses: astral-sh/setup-uv@v6
+
+      - name: Run check
+        id: precommit
         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
+          uv venv
+          uv pip install pre-commit
+          uv run pre-commit run --all-files
+        continue-on-error: true
 
-      - name: Run clang-format check
+      - name: Show diff on failure
+        if: steps.precommit.outcome == 'failure'
         run: |
-          FILES=$(find ./ -type f | grep -P '^\.\/(src|include)(\/[^\/]+)*\/[^\/]+\.(h|cpp|cc)$')
+          echo "❌ pre-commit found issues that it auto-fixed."
+          echo "   Please run 'pre-commit run --all-files' locally,"
+          echo "   commit the changes, and push again."
+          echo ""
+          echo "👇 The required changes are shown below:"
+          git diff
 
-          UNFORMATTED=$(clang-format-20 --dry-run --Werror $FILES)
+      - name: Fail the job if pre-commit failed
+        if: steps.precommit.outcome == 'failure'
+        run: |
+          echo "pre-commit checks failed."
+          exit 1
 
-          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
+      - name: Auto correct
+        uses: huacnlee/autocorrect-action@main
+        with:
+          args: --lint ./docs

.github/workflows/cmake.yml

@@ -34,7 +34,7 @@ jobs:
         with:
           buckets: main
           apps: ninja
-      
+
       - name: Setup llvm & libstdc++ & cmake & ninja
         if: matrix.os == 'ubuntu-24.04'
         run: |

.gitignore

@@ -51,4 +51,4 @@ tests/unit/Local/
 
 **/node_modules
 docs/.vitepress/dist
-docs/.vitepress/cache
+docs/.vitepress/cache

.pre-commit-config.yaml

@@ -1,6 +1,24 @@
+# .pre-commit-config.yaml
+
 repos:
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v6.0.0
+    hooks:
+      - id: trailing-whitespace
+      - id: end-of-file-fixer
+      - id: check-yaml
+      - id: check-added-large-files
+      - id: check-merge-conflict
+
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: v0.12.12
+    hooks:
+      - id: ruff
+        args: [--fix]
+      - id: ruff-format
+
   - repo: https://github.com/pre-commit/mirrors-clang-format
-    rev: "v20.1.0"
+    rev: v21.1.0
     hooks:
       - id: clang-format
-        files: '^(src|include)(/[^/]+)*/[^/]+\.(h|cpp|cc)$'
+        types_or: [c++, c]

CMakeLists.txt

@@ -35,7 +35,7 @@ endif()
 
 message(STATUS "Fetching tomlplusplus...")
 FetchContent_Declare(
-    tomlplusplus   
+    tomlplusplus
     GIT_REPOSITORY https://github.com/marzer/tomlplusplus.git
 )
 
@@ -87,9 +87,9 @@ endif()
 file(GLOB_RECURSE CLICE_SOURCES CONFIGURE_DEPENDS
     "${CMAKE_SOURCE_DIR}/src/AST/*.cpp"
     "${CMAKE_SOURCE_DIR}/src/Async/*.cpp"
-    "${CMAKE_SOURCE_DIR}/src/Basic/*.cpp" 
-    "${CMAKE_SOURCE_DIR}/src/Compiler/*.cpp" 
-    "${CMAKE_SOURCE_DIR}/src/Index/*.cpp" 
+    "${CMAKE_SOURCE_DIR}/src/Basic/*.cpp"
+    "${CMAKE_SOURCE_DIR}/src/Compiler/*.cpp"
+    "${CMAKE_SOURCE_DIR}/src/Index/*.cpp"
     "${CMAKE_SOURCE_DIR}/src/Feature/*.cpp"
     "${CMAKE_SOURCE_DIR}/src/Server/*.cpp"
     "${CMAKE_SOURCE_DIR}/src/Support/*.cpp"

Dockerfile

@@ -10,7 +10,7 @@ RUN curl https://apt.llvm.org/llvm-snapshot.gpg.key | tee /usr/share/keyrings/ll
     echo "deb [signed-by=/usr/share/keyrings/llvm-snapshot.gpg.key] http://apt.llvm.org/trixie/ llvm-toolchain-trixie main" >> /etc/apt/sources.list && \
     echo "deb [signed-by=/usr/share/keyrings/llvm-snapshot.gpg.key] http://apt.llvm.org/trixie/ llvm-toolchain-trixie-20 main" >> /etc/apt/sources.list && \
     apt-get update && apt-get install -y \
-    clang-20 lld-20 
+    clang-20 lld-20
 RUN ln -s /usr/bin/lld-20 /usr/bin/lld
 
 # Adds source code
@@ -28,7 +28,7 @@ RUN cmake -B build -G Ninja -DCMAKE_C_COMPILER=clang-20 -DCMAKE_CXX_COMPILER=cla
 RUN cmake --build build -j && \
     cmake --install build --prefix=/opt/clice
 
-FROM debian:13 
+FROM debian:13
 
 COPY --from=builder /opt/clice /opt/clice/
 COPY LICENSE /opt/clice/LICENSE

README.md

@@ -35,4 +35,4 @@ Download the latest `clice` binary from the [releases page](https://github.com/c
 
 ## Documentation
 
-To learn more about building, installing, and configuring clice, or to dive deep into its features and architecture, please visit our official documentation at [**clice.io**](https://clice.io/).
+To learn more about building, installing, and configuring clice, or to dive deep into its features and architecture, please visit our official documentation at [**clice.io**](https://clice.io/).

cmake/github.cmake

@@ -5,25 +5,25 @@ function(github_lookup_tag_commit REPO_OWNER REPO_NAME TAG_NAME OUTPUT_VAR)
     set(GITHUB_API_URL "https://api.github.com/repos/${REPO_OWNER}/${REPO_NAME}/git/ref/tags/${TAG_NAME}")
 
     message(STATUS "Fetching tag info from: ${GITHUB_API_URL}")
-    
+
     # Download tag info directly into a string
     download_to_string(${GITHUB_API_URL} TAG_JSON DOWNLOAD_RESULT)
     if(NOT DOWNLOAD_RESULT EQUAL "0")
         message(WARNING "Failed to fetch tag info. Result: ${DOWNLOAD_RESULT}")
         set(${OUTPUT_VAR} "NOTFOUND" PARENT_SCOPE)
         return()
     endif()
-    
+
     # Parse the JSON to get object type and SHA
     parse_json_field_from_string("${TAG_JSON}" object "type" OBJECT_TYPE)
     parse_json_field_from_string("${TAG_JSON}" object "sha" OBJECT_SHA)
-    
+
     if(OBJECT_TYPE STREQUAL "NOTFOUND" OR OBJECT_SHA STREQUAL "NOTFOUND")
         message(WARNING "Could not find object type or SHA in the JSON response.")
         set(${OUTPUT_VAR} "NOTFOUND" PARENT_SCOPE)
         return()
     endif()
-    
+
     set(COMMIT_SHA "NOTFOUND")
 
     if(OBJECT_TYPE STREQUAL "commit")
@@ -34,24 +34,24 @@ function(github_lookup_tag_commit REPO_OWNER REPO_NAME TAG_NAME OUTPUT_VAR)
         set(TAG_API_URL "https://api.github.com/repos/${REPO_OWNER}/${REPO_NAME}/git/tags/${OBJECT_SHA}")
 
         message(STATUS "Fetching annotated tag info from: ${TAG_API_URL}")
-        
+
         # Download annotated tag info directly into a string
         download_to_string(${TAG_API_URL} ANNOTATED_TAG_JSON TAG_DOWNLOAD_RESULT)
         if(NOT TAG_DOWNLOAD_RESULT EQUAL "0")
             message(WARNING "Failed to fetch annotated tag info. Result: ${TAG_DOWNLOAD_RESULT}")
             set(${OUTPUT_VAR} "NOTFOUND" PARENT_SCOPE)
             return()
         endif()
-        
+
         # Parse the annotated tag JSON to get the commit SHA
         parse_json_field_from_string("${ANNOTATED_TAG_JSON}" object "sha" COMMIT_SHA_FROM_TAG)
-        
+
         if(NOT COMMIT_SHA_FROM_TAG STREQUAL "NOTFOUND")
             set(COMMIT_SHA ${COMMIT_SHA_FROM_TAG})
         endif()
     else()
         message(WARNING "Unknown object type: ${OBJECT_TYPE}")
     endif()
-    
+
     set(${OUTPUT_VAR} ${COMMIT_SHA} PARENT_SCOPE)
-endfunction()
+endfunction()