js-code-detector 0.0.22 → 0.0.23

package/dist/cjs/util/report_util/createMdByJson.js

@@ -43,7 +43,8 @@ function reportItemToMd(report) {
 ${filesDependsOnMe.slice(0, 1).map((files) => files.map((file) => `- ${file}`)).flat().join("\n")}` : "",
     undefinedIdentifiers.length > 0 ? `### 未定义的变量
 > ${undefinedIdentifiers.map((e) => `**${e}**`).join(", ")}` : "",
-    // dangerIdentifiers.length > 0 ? `### 重点检查使用的变量\n> ${dangerIdentifiers.join(', ')}` : '',
+    // todo 拿到具体的节点 对应的 文本
+    // dangerIdentifiers.length > 0 ? `### 请确认以下删除是安全的\n${dangerIdentifiers.map(e => `> ${e}`).join('\n')}` : '',
     blockReports.length > 0 ? `### 对比分析 共${blockReports.length}处` : "",
     ...blockReports.map(blockReportToMd)
   ].filter(Boolean).join("\n\n");
@@ -65,16 +66,20 @@ ${diff_txt.join("\n")}
 function blockReportInfoItemToMd(info) {
   const {
     causeBy,
-    effects,
+    effectsUpstream,
+    effectsDownstream,
     occupations
   } = info;
   return [
-    effects.length > 0 ? `#### ${causeBy}
-- 影响：
-${effects.map((e) => `> ${e}`).join("\n")}` : "- 无影响",
+    effectsUpstream.length > 0 ? `#### ${causeBy}
+- 上游影响：
+${effectsUpstream.map((e) => `> ${e}`).join("\n")}` : "",
     occupations.length > 0 ? `#### ${causeBy}
 - 使用：
-${occupations.map((e) => `> ${e}`).join("\n")}` : "- 无"
+${occupations.map((e) => `> ${e}`).join("\n")}` : "",
+    effectsDownstream.length > 0 ? `#### ${causeBy}
+- 下游影响：
+${effectsDownstream.map((e) => `> ${e}`).join("\n")}` : ""
   ].filter(Boolean).join("\n\n");
 }
 // Annotate the CommonJS export names for ESM import in node:

package/dist/cjs/util/report_util/file_identifier_detect.d.ts

@@ -1,2 +1 @@
 export declare function extractUndefinedIdentifiers(filePath: string, absPathPrefix: string): string[];
-export declare function fileIdentifierDetect(filePath: string, absPathPrefix: string): string[];

package/dist/cjs/util/report_util/file_identifier_detect.js

@@ -29,8 +29,7 @@ var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: tru
 // src/util/report_util/file_identifier_detect.ts
 var file_identifier_detect_exports = {};
 __export(file_identifier_detect_exports, {
-  extractUndefinedIdentifiers: () => extractUndefinedIdentifiers,
-  fileIdentifierDetect: () => fileIdentifierDetect
+  extractUndefinedIdentifiers: () => extractUndefinedIdentifiers
 });
 module.exports = __toCommonJS(file_identifier_detect_exports);
 var import_getAstKitByFilePath = __toESM(require("../ast_util/getAstKitByFilePath"));
@@ -45,17 +44,7 @@ function extractUndefinedIdentifiers(filePath, absPathPrefix) {
   const ids = [...dependenceIds].filter((id) => import_AstUtil.default.isUntrackedId(id));
   return ids.map((id) => import_AstUtil.default.getShortNodeMsg(id));
 }
-function fileIdentifierDetect(filePath, absPathPrefix) {
-  const { mapUuidToNode } = (0, import_getAstKitByFilePath.default)(filePath, absPathPrefix);
-  const programNode = mapUuidToNode.get("Program");
-  if (!programNode) {
-    return [];
-  }
-  const { effectIds } = programNode._util;
-  return [...effectIds].map((id) => import_AstUtil.default.getShortNodeMsg(id));
-}
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {
-  extractUndefinedIdentifiers,
-  fileIdentifierDetect
+  extractUndefinedIdentifiers
 });

package/dist/cjs/util/report_util.d.ts

@@ -25,18 +25,19 @@ type Arg = {
     absPathPrefix: string;
 };
 export declare function createDetectReport(arg: Arg): {
+    dangerIdentifiers: string[];
     blockReports: {
         diff_txt: string[];
         infos: {
             causeBy: string;
-            effects: string[];
+            effectsUpstream: string[];
             occupations: string[];
+            effectsDownstream: string[];
         }[];
     }[];
-    type: "delete" | "modify" | "add";
+    type: "modify" | "add" | "delete";
     filesDependsOnMe: string[][];
     undefinedIdentifiers: string[];
-    dangerIdentifiers: string[];
     filePath: string;
 }[];
 export {};

package/dist/cjs/util/report_util.js

@@ -58,7 +58,6 @@ function createDetectReport(arg) {
       reports.push(reportItem);
     }
     reportItem.undefinedIdentifiers = (0, import_file_identifier_detect.extractUndefinedIdentifiers)((0, import_path.join)(absPathPrefix, filePath), absPathPrefix);
-    reportItem.dangerIdentifiers = (0, import_file_identifier_detect.fileIdentifierDetect)((0, import_path.join)(absPathPrefix, filePath), absPathPrefix);
     if (type === "modify") {
       (0, import_diffBlockDetect.diffBlockDetect)(item, index, { reportItem, absPathPrefix });
     }
@@ -67,10 +66,11 @@ function createDetectReport(arg) {
     const { _fileAddedNodesPaths, _fileRemovedNodesPaths, filePath, blockReports, ...reportProperties } = reportItem;
     (0, import_diffBlockDetect.reportItemDetect)(reportItem, absPathPrefix);
     const fileAddedNodesPaths = reportItem._fileAddedNodesPaths.map((item) => item.nodePath);
-    const fileRemovedNodesPaths = reportItem._fileRemovedNodesPaths.map((item) => item.nodePath);
+    const fileRemovedNodesPaths = reportItem._fileRemovedNodesPaths.map((item) => import_AstUtil.default.getShortNodeMsg(item.node, true));
     return {
       filePath,
       ...reportProperties,
+      dangerIdentifiers: fileRemovedNodesPaths,
       blockReports: blockReports.map((blockReport) => {
         const { diff_txt } = blockReport;
         const addNodeAndPaths = blockReport.addNodeAndPaths.filter((e) => fileAddedNodesPaths.includes(e.nodePath));
@@ -79,16 +79,17 @@ function createDetectReport(arg) {
           const { effectIds, occupation, holdingIdType } = node._util;
           return {
             causeBy: import_AstUtil.default.getShortNodeMsg(node, true),
-            effects: holdingIdType ? [...occupation].map((occupationId) => {
+            effectsUpstream: [...effectIds].map((e) => import_AstUtil.default.getShortNodeMsg(e, true)),
+            occupations: [...occupation].map((e) => import_AstUtil.default.getShortNodeMsg(e, true)),
+            effectsDownstream: holdingIdType ? [...occupation].map((occupationId) => {
               const ans = import_AstUtil.default.getAncestorsFromBirth(occupationId, node);
               return import_AstUtil.default.getNearestImpactedNode(ans.reverse());
-            }).filter(Boolean).map((e) => import_AstUtil.default.getShortNodeMsg(e, true)) : [...effectIds].map((e) => import_AstUtil.default.getShortNodeMsg(e, true)),
-            occupations: [...occupation].map((e) => import_AstUtil.default.getShortNodeMsg(e, true))
+            }).filter(Boolean).map((e) => import_AstUtil.default.getShortNodeMsg(e, true)) : []
           };
         });
         return {
           diff_txt,
-          infos: infosList.filter((e) => e.effects.length > 0 || e.occupations.length > 0)
+          infos: infosList.filter((e) => e.effectsUpstream.length > 0 || e.occupations.length > 0 || e.effectsDownstream.length > 0)
         };
       }).filter((e) => e.infos.length > 0)
     };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "js-code-detector",
-  "version": "0.0.22",
+  "version": "0.0.23",
   "description": "",
   "main": "dist/cjs/index.js",
   "types": "dist/cjs/index.d.ts",