Merge branch 'fix402' into completion

Author: xuhuanzy

crates/emmylua_code_analysis/src/compilation/analyzer/flow/build_flow_tree.rs

@@ -67,7 +67,7 @@ fn build_name_expr_flow(
 ) -> Option<()> {
     let parent = name_expr.get_parent::<LuaAst>()?;
     let mut is_assign = false;
-    match parent {
+    match &parent {
         LuaAst::LuaIndexExpr(_) | LuaAst::LuaCallExpr(_) | LuaAst::LuaFuncStat(_) => return None,
         LuaAst::LuaAssignStat(assign_stat) => {
             let eq_pos = assign_stat
@@ -87,7 +87,20 @@ fn build_name_expr_flow(
     let mut ref_id: Option<VarRefId> = None;
     if let Some(local_refs) = db.get_reference_index().get_local_reference(&file_id) {
         if let Some(decl_id) = local_refs.get_decl_id(&name_expr.get_range()) {
-            ref_id = Some(VarRefId::DeclId(decl_id.clone()));
+            if let Some(decl) = db.get_decl_index().get_decl(&decl_id) {
+                // 处理`self`作为参数传入的特殊情况
+                if decl.is_param()
+                    && name_expr
+                        .get_name_text()
+                        .map_or(false, |name| name == "self")
+                {
+                    ref_id = Some(VarRefId::Name(SmolStr::new("self")));
+                } else {
+                    ref_id = Some(VarRefId::DeclId(decl_id.clone()));
+                }
+            } else {
+                ref_id = Some(VarRefId::DeclId(decl_id.clone()));
+            }
         }
     }
 

crates/emmylua_code_analysis/src/diagnostic/test/need_check_nil_test.rs

@@ -33,4 +33,23 @@ mod test {
         "#
         ));
     }
+
+    #[test]
+    fn test_issue_402() {
+        let mut ws = VirtualWorkspace::new();
+        assert!(ws.check_code_for(
+            DiagnosticCode::NeedCheckNil,
+            r#"
+            ---@class A
+            local a = {}
+
+            ---@param self table?
+            function a.new(self)
+                if self then
+                    self.a = 1
+                end
+            end
+        "#
+        ));
+    }
 }