try env.containerid

Signed-off-by: sirutBuasai <sirutbuasai27@outlook.com>

Author: sirutBuasai

.github/actions/container-cleanup/action.yaml

@@ -1,12 +1,17 @@
 name: Container Cleanup
-description: Remove container and clean up image caches.
+description: Remove container via container ID and clean up image caches.
+
+inputs:
+  container_id:
+    description: Container ID to be removed
+    required: true
 
 runs:
   using: composite
   steps:
     - name: Cleanup container and images
       shell: bash
       run: |
-        docker rm -f ${CONTAINER_ID} || true
+        docker rm -f ${{ container_id }} || true
         docker image prune -a --force --filter "until=24h"
         docker system df

.github/workflows/pr-sglang.yml

@@ -139,3 +139,12 @@ jobs:
       - name: Cleanup container and images
         if: always()
         uses: ./.github/actions/container-cleanup
+        with:
+          container_id: ${{ env.CONTAINER_ID }}
+
+      - name: Cleanup container and images failsafe
+        if: always()
+        run: |
+          docker rm -f ${CONTAINER_ID} || true
+          docker image prune -a --force --filter "until=24h"
+          docker system df

.pre-commit-config.yaml

@@ -16,13 +16,13 @@ repos:
     rev: v6.0.0
     hooks:
   # ------------------------------- 🌳 Git Tools ------------------------------- #
-    - id: check-merge-conflict
-      name: "🌳 git · Detect conflict markers"
-    - id: forbid-new-submodules
-      name: "🌳 git · Prevent submodule creation"
-    - id: no-commit-to-branch
-      name: "🌳 git · Protect main branches"
-      args: ["--branch", "main", "--branch", "master"]
+      - id: check-merge-conflict
+        name: "🌳 git · Detect conflict markers"
+      - id: forbid-new-submodules
+        name: "🌳 git · Prevent submodule creation"
+      - id: no-commit-to-branch
+        name: "🌳 git · Protect main branches"
+        args: ["--branch", "main", "--branch", "master"]
   # ---------------------------- 📁 Filesystem Tools --------------------------- #
       - id: check-executables-have-shebangs
         name: "📁 filesystem/⚙️ exec · Verify shebang presence"
@@ -37,16 +37,16 @@ repos:
       - id: trailing-whitespace
         name: "📁 filesystem/📝 file · Validate empty whitespaces"
   # ----------------------------- 🔒 Security Tools ---------------------------- #
-    - id: check-json
-      name: "📁 filesystem/🔧 config · Validate JSON file"
-    - id: check-yaml
-      name: "📁 filesystem/🔧 config · Validate YAML file"
-    - id: detect-aws-credentials
-      name: "🔒 security · Detect aws credentials"
-      args:
-        - --allow-missing-credentials
-    - id: detect-private-key
-      name: "🔒 security · Detect private keys"
+      - id: check-json
+        name: "📁 filesystem/🔧 config · Validate JSON file"
+      - id: check-yaml
+        name: "📁 filesystem/🔧 config · Validate YAML file"
+      - id: detect-aws-credentials
+        name: "🔒 security · Detect aws credentials"
+        args:
+          - --allow-missing-credentials
+      - id: detect-private-key
+        name: "🔒 security · Detect private keys"
 
 
   - repo: https://github.com/gitleaks/gitleaks