infra: add new-line makefile target

yuluo-yx · yuluo-yx · commit 3a3923e2268a · 2025-10-16T20:52:33.000+08:00
Signed-off-by: yuluo-yx &lt;yuluo08290126@gmail.com&gt;
diff --git a/.gitignore b/.gitignore
@@ -142,4 +142,4 @@ dashboard/backend/dashboard-backend
 dashboard/backend/dashboard-backend.exe
 
 # Keep old HTML backup for reference
-dashboard/frontend/index.html.old
+dashboard/frontend/index.html.old
diff --git a/config/tools_db.json b/config/tools_db.json
@@ -139,4 +139,4 @@
     "category": "productivity",
     "tags": ["calendar", "event", "meeting", "appointment", "schedule"]
   }
-] 
+] 
diff --git a/dashboard/OWNER b/dashboard/OWNER
@@ -1,3 +1,3 @@
 # Dashboard directory Owners
 @JaredforReal
-@Xunzhuo
+@Xunzhuo
diff --git a/deploy/docker-compose/addons/llm-router-dashboard.json b/deploy/docker-compose/addons/llm-router-dashboard.json
@@ -1235,4 +1235,4 @@
   "uid": "llm-router-metrics",
   "version": 14,
   "weekStart": ""
-}
+}
diff --git a/deploy/kubernetes/istio/tools_db.json b/deploy/kubernetes/istio/tools_db.json
@@ -139,4 +139,4 @@
     "category": "productivity",
     "tags": ["calendar", "event", "meeting", "appointment", "schedule"]
   }
-] 
+] 
diff --git a/deploy/kubernetes/tools_db.json b/deploy/kubernetes/tools_db.json
@@ -139,4 +139,4 @@
     "category": "productivity",
     "tags": ["calendar", "event", "meeting", "appointment", "schedule"]
   }
-] 
+] 
diff --git a/deploy/openshift/Dockerfile.llm-katan b/deploy/openshift/Dockerfile.llm-katan
@@ -42,4 +42,4 @@ HEALTHCHECK --interval=30s --timeout=10s --start-period=60s --retries=3 \
     CMD curl -f http://localhost:8000/health || exit 1
 
 # Default command - this will be overridden by deployment args
-CMD ["llm-katan", "--help"]
+CMD ["llm-katan", "--help"]
diff --git a/deploy/openshift/cleanup-openshift.sh b/deploy/openshift/cleanup-openshift.sh
@@ -491,4 +491,4 @@ main() {
 }
 
 # Run main function with all arguments
-main "$@"
+main "$@"
diff --git a/deploy/openshift/config-openshift.yaml b/deploy/openshift/config-openshift.yaml
@@ -205,4 +205,4 @@ observability:
     resource:
       service_name: "vllm-semantic-router"
       service_version: "v0.1.0"
-      deployment_environment: "development"
+      deployment_environment: "development"
diff --git a/deploy/openshift/deploy-to-openshift.sh b/deploy/openshift/deploy-to-openshift.sh
@@ -985,4 +985,4 @@ main() {
 }
 
 # Run main function with all arguments
-main "$@"
+main "$@"
diff --git a/deploy/openshift/deployment.yaml b/deploy/openshift/deployment.yaml
@@ -348,4 +348,4 @@ spec:
           claimName: semantic-router-models
       - name: cache-volume
         persistentVolumeClaim:
-          claimName: semantic-router-cache
+          claimName: semantic-router-cache
diff --git a/deploy/openshift/envoy-openshift.yaml b/deploy/openshift/envoy-openshift.yaml
@@ -193,4 +193,4 @@ admin:
   address:
     socket_address:
       address: "127.0.0.1"
-      port_value: 19000
+      port_value: 19000
diff --git a/deploy/openshift/kustomization.yaml b/deploy/openshift/kustomization.yaml
@@ -33,4 +33,4 @@ commonLabels:
 
 # Add OpenShift-specific annotations
 commonAnnotations:
-  deployment.openshift.io/type: "vllm-semantic-router"
+  deployment.openshift.io/type: "vllm-semantic-router"
diff --git a/deploy/openshift/openwebui/deploy-openwebui-on-openshift.sh b/deploy/openshift/openwebui/deploy-openwebui-on-openshift.sh
@@ -126,4 +126,4 @@ echo "  1. Open https://$ROUTE_URL in your browser"
 echo "  2. Complete initial setup in OpenWebUI"
 echo "  3. The models should be automatically available"
 echo ""
-echo -e "${GREEN}✨ Happy chatting with your models!${NC}"
+echo -e "${GREEN}✨ Happy chatting with your models!${NC}"
diff --git a/deploy/openshift/openwebui/deployment.yaml b/deploy/openshift/openwebui/deployment.yaml
@@ -82,4 +82,4 @@ spec:
         persistentVolumeClaim:
           claimName: openwebui-data
       - name: tmp-volume
-        emptyDir: {}
+        emptyDir: {}
diff --git a/deploy/openshift/openwebui/kustomization.yaml b/deploy/openshift/openwebui/kustomization.yaml
@@ -19,4 +19,4 @@ commonLabels:
   app.kubernetes.io/part-of: semantic-router
 
 # Namespace for all resources
-namespace: vllm-semantic-router-system
+namespace: vllm-semantic-router-system
diff --git a/deploy/openshift/openwebui/pvc.yaml b/deploy/openshift/openwebui/pvc.yaml
@@ -13,4 +13,4 @@ spec:
     requests:
       storage: 2Gi
   # Use default storage class for OpenShift
-  # storageClassName: ""
+  # storageClassName: ""
diff --git a/deploy/openshift/openwebui/route.yaml b/deploy/openshift/openwebui/route.yaml
@@ -19,4 +19,4 @@ spec:
   tls:
     termination: edge
     insecureEdgeTerminationPolicy: Redirect
-  wildcardPolicy: None
+  wildcardPolicy: None
diff --git a/deploy/openshift/openwebui/service.yaml b/deploy/openshift/openwebui/service.yaml
@@ -14,4 +14,4 @@ spec:
     protocol: TCP
     name: http
   selector:
-    app: openwebui
+    app: openwebui
diff --git a/deploy/openshift/openwebui/uninstall-openwebui.sh b/deploy/openshift/openwebui/uninstall-openwebui.sh
@@ -156,4 +156,4 @@ echo ""
 echo -e "${BLUE}💡 To redeploy OpenWebUI:${NC}"
 echo "  ./deploy-openwebui-on-openshift.sh"
 echo ""
-echo -e "${GREEN}✨ Cleanup completed successfully!${NC}"
+echo -e "${GREEN}✨ Cleanup completed successfully!${NC}"
diff --git a/deploy/openshift/template.yaml b/deploy/openshift/template.yaml
@@ -358,4 +358,4 @@ parameters:
 - name: METRICS_HOSTNAME
   description: "Custom hostname for metrics route (leave empty for auto-generated)"
   value: ""
-  required: false
+  required: false
diff --git a/e2e-tests/llm-katan/.gitignore b/e2e-tests/llm-katan/.gitignore
@@ -17,4 +17,4 @@ __pycache__/
 
 # OS files
 .DS_Store
-Thumbs.db
+Thumbs.db
diff --git a/e2e-tests/llm-katan/demo-embed.html b/e2e-tests/llm-katan/demo-embed.html
@@ -58,4 +58,4 @@
 .pause(800)
 .type('<span style="color: #4fc1e9;"># Same tiny model, different API names! 🎯</span>')
 .go();
-</script>
+</script>
diff --git a/e2e-tests/llm-katan/pyproject.toml b/e2e-tests/llm-katan/pyproject.toml
@@ -71,4 +71,4 @@ target-version = ['py38', 'py39', 'py310', 'py311', 'py312']
 
 [tool.isort]
 profile = "black"
-line_length = 100
+line_length = 100
diff --git a/e2e-tests/llm-katan/terminal-demo.html b/e2e-tests/llm-katan/terminal-demo.html
@@ -240,4 +240,4 @@
         }, 8500); // Start after both terminals complete (~8.5 seconds)
     </script>
 </body>
-</html>
+</html>
diff --git a/e2e-tests/start-llm-katan.sh b/e2e-tests/start-llm-katan.sh
@@ -136,4 +136,4 @@ start_servers_foreground() {
 }
 
 # Main execution - always run in foreground mode
-start_servers_foreground
+start_servers_foreground
diff --git a/scripts/quickstart.sh b/scripts/quickstart.sh
@@ -287,4 +287,4 @@ main() {
 trap 'echo; print_color $RED "❌ Setup interrupted!"; exit 1' INT TERM
 
 # Run main function
-main "$@"
+main "$@"
diff --git a/src/OWNER b/src/OWNER
@@ -1,4 +1,4 @@
 # Source code owners
 @rootfs
 @Xunzhuo
-@wangchen615
+@wangchen615
diff --git a/src/training/training_lora/OWNER b/src/training/training_lora/OWNER
@@ -1,2 +1,2 @@
 # lora training owners
-@OneZero-Y
+@OneZero-Y
diff --git a/src/training/training_lora/classifier_model_fine_tuning_lora/go.mod b/src/training/training_lora/classifier_model_fine_tuning_lora/go.mod
@@ -4,4 +4,4 @@ go 1.24.1
 
 replace github.com/vllm-project/semantic-router/candle-binding => ../../../../candle-binding
 
-require github.com/vllm-project/semantic-router/candle-binding v0.0.0-00010101000000-000000000000 
+require github.com/vllm-project/semantic-router/candle-binding v0.0.0-00010101000000-000000000000 
diff --git a/src/training/training_lora/classifier_model_fine_tuning_lora/train_cpu_optimized.sh b/src/training/training_lora/classifier_model_fine_tuning_lora/train_cpu_optimized.sh
@@ -304,4 +304,4 @@ echo ""
 
 # Display final summary
 echo "📊 FINAL CPU TRAINING SUMMARY:"
-cat "$SUMMARY_FILE" 
+cat "$SUMMARY_FILE" 
diff --git a/src/training/training_lora/pii_model_fine_tuning_lora/go.mod b/src/training/training_lora/pii_model_fine_tuning_lora/go.mod
@@ -4,4 +4,4 @@ go 1.24.1
 
 replace github.com/vllm-project/semantic-router/candle-binding => ../../../../candle-binding
 
-require github.com/vllm-project/semantic-router/candle-binding v0.0.0-00010101000000-000000000000 
+require github.com/vllm-project/semantic-router/candle-binding v0.0.0-00010101000000-000000000000 
diff --git a/src/training/training_lora/pii_model_fine_tuning_lora/train_cpu_optimized.sh b/src/training/training_lora/pii_model_fine_tuning_lora/train_cpu_optimized.sh
@@ -304,4 +304,4 @@ echo ""
 
 # Display final summary
 echo "📊 FINAL CPU TRAINING SUMMARY:"
-cat "$SUMMARY_FILE" 
+cat "$SUMMARY_FILE" 
diff --git a/src/training/training_lora/prompt_guard_fine_tuning_lora/go.mod b/src/training/training_lora/prompt_guard_fine_tuning_lora/go.mod
@@ -4,4 +4,4 @@ go 1.24.1
 
 replace github.com/vllm-project/semantic-router/candle-binding => ../../../../candle-binding
 
-require github.com/vllm-project/semantic-router/candle-binding v0.0.0-00010101000000-000000000000 
+require github.com/vllm-project/semantic-router/candle-binding v0.0.0-00010101000000-000000000000 
diff --git a/src/training/training_lora/prompt_guard_fine_tuning_lora/train_cpu_optimized.sh b/src/training/training_lora/prompt_guard_fine_tuning_lora/train_cpu_optimized.sh
@@ -304,4 +304,4 @@ echo ""
 
 # Display final summary
 echo "📊 FINAL CPU TRAINING SUMMARY:"
-cat "$SUMMARY_FILE" 
+cat "$SUMMARY_FILE" 
diff --git a/tools/linter/go/.golangci.yml b/tools/linter/go/.golangci.yml
@@ -163,4 +163,4 @@ formatters:
     paths:
       - examples$
       - zz_generated
-  
+  
diff --git a/tools/linter/newline/config.json b/tools/linter/newline/config.json
@@ -0,0 +1,57 @@
+{
+  "ignore_dirs": [
+    "node_modules",
+    ".idea",
+    ".git",
+    ".vscode",
+    "target",
+    "static",
+    "static_legacy",
+    "frontend",
+    ".husky",
+    "docs",
+    "docker"
+  ],
+  "ignore_suffix": [
+    ".png",
+    ".jpg",
+    ".jpeg",
+    ".gif",
+    ".svg",
+    ".ico",
+    ".webp",
+    ".pdf",
+    ".word",
+    ".docx",
+    ".doc",
+    ".ppt",
+    ".xlsx",
+    ".xls",
+    ".exe",
+    ".pptx",
+    ".jar",
+    ".flac",
+    ".txt",
+    ".wav",
+    ".bib",
+    ".cmd",
+    "mvnw",
+    ".bin",
+    ".pcm",
+    ".ttf",
+    ".mp4",
+    ".PPT",
+    ".jif",
+    ".zip",
+    ".tar",
+    ".gz",
+    ".rar",
+    ".7z",
+    ".tar.gz",
+    ".tar.bz2",
+    ".tar.xz",
+    ".tgz",
+    ".tbz2",
+    ".txz"
+  ]
+}
diff --git a/tools/linter/newline/newline.py b/tools/linter/newline/newline.py
@@ -0,0 +1,92 @@
+import os
+import sys
+import json
+
+
+# Load configuration from JSON file
+def load_config():
+    config_path = os.path.join(os.path.dirname(__file__), 'config.json')
+    try:
+        with open(config_path, 'r', encoding='utf-8') as f:
+            config = json.load(f)
+            return config.get('ignore_dirs', []), config.get('ignore_suffix', [])
+    except FileNotFoundError:
+        print(f"Configuration file not found: {config_path}")
+        sys.exit(1)
+    except json.JSONDecodeError as e:
+        print(f"Error parsing configuration file: {e}")
+        sys.exit(1)
+
+
+# Load configuration
+ignore_dirs, ignore_suffix = load_config()
+
+# Check if the incoming file ends with a blank line
+def check_file(path):
+    try:
+        with open(path, 'rb') as f:
+            f.seek(0, os.SEEK_END)
+            size = f.tell()
+            if size == 0:
+                return None
+            f.seek(-1, os.SEEK_END)
+            if f.read(1) != b'\n':
+                return path
+    except OSError as e:
+        print(f"Cannot check file: {path}: {e}")
+    return None
+
+
+# Accept a list, check if each file ends with a blank line, and if not, write a new line at the end of the file
+def add_newline(file):
+    print("Fixing: " + file)
+    with open(file, 'a') as f:
+        f.write('\n')
+
+
+# Gets all the files in the current directory and returns a list of files
+def get_files():
+    files_to_check = []
+    for root, dirs, files in os.walk('.'):
+        # Ignore the specified directory
+        dirs[:] = [d for d in dirs if d not in ignore_dirs]
+        for file in files:
+            if not any(file.endswith(suffix) for suffix in ignore_suffix):
+                files_to_check.append(os.path.join(root, file))
+    return files_to_check
+
+
+# Run the checks
+def run(check_only=False):
+    files = get_files()
+    files_to_fix = []
+
+    for file in files:
+        result = check_file(file)
+        if result:
+            files_to_fix.append(result)
+
+    if files_to_fix:
+        print("The following files are missing a blank line:")
+        for file in files_to_fix:
+            print(file)
+        if check_only:
+            print("Error: Some files do not end with a blank line.")
+            sys.exit(1)  # Exit with an error code
+        else:
+            for file in files_to_fix:
+                add_newline(file)
+                print(f"Added a line break at the end of {file}.")
+    else:
+        print("All files have ended with a blank line.")
+
+
+if __name__ == "__main__":
+
+    mode = sys.argv[1] if len(sys.argv) > 1 else 'check'
+    if mode == 'check':
+        run(check_only=True)
+    elif mode == 'fix':
+        run(check_only=False)
+    else:
+        print("Invalid mode. Please use 'check' or 'fix'.")
diff --git a/tools/make/linter.mk b/tools/make/linter.mk
diff --git a/tools/make/milvus.mk b/tools/make/milvus.mk
diff --git a/tools/make/openshift.mk b/tools/make/openshift.mk
diff --git a/tools/observability/llm-router-dashboard.json b/tools/observability/llm-router-dashboard.json
diff --git a/website/src/components/ZoomableMermaid/styles.module.css b/website/src/components/ZoomableMermaid/styles.module.css

Original file line number	Diff line number	Diff line change
`@@ -139,4 +139,4 @@`
`139`	`139`	`"category": "productivity",`
`140`	`140`	`"tags": ["calendar", "event", "meeting", "appointment", "schedule"]`
`141`	`141`	`}`
`142`		`-]`
	`142`	`+]`
-Original file line number
+Diff line change
@@ @@ -1,3 +1,3 @@ @@
 # Dashboard directory Owners
 @JaredforReal
 -@Xunzhuo
 +@Xunzhuo
Original file line number	Diff line number	Diff line change
`@@ -491,4 +491,4 @@ main() {`
`491`	`491`	`}`
`492`	`492`
`493`	`493`	`# Run main function with all arguments`
`494`		`-main "$@"`
	`494`	`+main "$@"`
Original file line number	Diff line number	Diff line change
`@@ -985,4 +985,4 @@ main() {`
`985`	`985`	`}`
`986`	`986`
`987`	`987`	`# Run main function with all arguments`
`988`		`-main "$@"`
	`988`	`+main "$@"`