From 414a45452c12079294c3b1b590829af499cad7f6 Mon Sep 17 00:00:00 2001
From: He1pa <18012015693@163.com>
Date: Fri, 7 Apr 2023 11:26:56 +0800
Subject: [PATCH] bugfix: Fix CI failure cause by inconsistent order of walk
 files

---
 kclvm/driver/src/lib.rs               | 3 ++-
 kclvm/tools/src/LSP/src/completion.rs | 9 +++++++--
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/kclvm/driver/src/lib.rs b/kclvm/driver/src/lib.rs
index 32ad6a625..d39d30d2c 100644
--- a/kclvm/driver/src/lib.rs
+++ b/kclvm/driver/src/lib.rs
@@ -13,7 +13,7 @@ use kclvm_parser::LoadProgramOptions;
 use kclvm_runtime::PanicInfo;
 use kclvm_utils::path::PathPrefix;
 use std::{
-    fs::read_dir,
+    fs::{read_dir, self},
     io::{self, ErrorKind},
     path::{Path, PathBuf},
 };
@@ -186,5 +186,6 @@ pub fn get_kcl_files<P: AsRef<Path>>(path: P, recursively: bool) -> Result<Vec<S
             }
         }
     }
+    files.sort();
     Ok(files)
 }
diff --git a/kclvm/tools/src/LSP/src/completion.rs b/kclvm/tools/src/LSP/src/completion.rs
index 2c53d0a71..37e7f7a40 100644
--- a/kclvm/tools/src/LSP/src/completion.rs
+++ b/kclvm/tools/src/LSP/src/completion.rs
@@ -8,6 +8,7 @@
 //!  + defitions in pkg
 //!  + system module functions
 
+use std::io;
 use std::{fs, path::Path};
 
 use indexmap::IndexSet;
@@ -74,8 +75,12 @@ fn completion_for_import(
         Path::new(&program.root).join(pkgpath.replace('.', &std::path::MAIN_SEPARATOR.to_string()));
     if real_path.is_dir() {
         if let Ok(entries) = fs::read_dir(real_path) {
-            for entry in entries.flatten() {
-                let path = entry.path();
+            let mut entries = entries
+                .map(|res| res.map(|e| e.path()))
+                .collect::<Result<Vec<_>, io::Error>>()
+                .unwrap();
+            entries.sort();
+            for path in entries {
                 let filename = path.file_name().unwrap().to_str().unwrap().to_string();
                 if path.is_dir() {
                     items.insert(filename);