github-mobile-reader 0.1.2 → 0.1.4

package/dist/action.js

@@ -32,7 +32,9 @@ function isJSXFile(filename) {
   return /\.(jsx|tsx)$/.test(filename);
 }
 function hasJSXContent(lines) {
-  return lines.some((l) => /<[A-Z][A-Za-z]*[\s/>]/.test(l) || /return\s*\(/.test(l));
+  return lines.some(
+    (l) => /<[A-Z][A-Za-z]*[\s/>]/.test(l) || /return\s*\(/.test(l)
+  );
 }
 function isClassNameOnlyLine(line) {
   return /^className=/.test(line.trim());
@@ -40,7 +42,9 @@ function isClassNameOnlyLine(line) {
 function extractClassName(line) {
   const staticMatch = line.match(/className="([^"]*)"/);
   if (staticMatch) return staticMatch[1];
-  const ternaryMatch = line.match(/className=\{[^?]+\?\s*"([^"]*)"\s*:\s*"([^"]*)"\}/);
+  const ternaryMatch = line.match(
+    /className=\{[^?]+\?\s*"([^"]*)"\s*:\s*"([^"]*)"\}/
+  );
   if (ternaryMatch) return `${ternaryMatch[1]} ${ternaryMatch[2]}`;
   const templateMatch = line.match(/className=\{`([^`]*)`\}/);
   if (templateMatch) {
@@ -62,18 +66,21 @@ function parseClassNameChanges(addedLines, removedLines) {
     const cls = extractClassName(line);
     const comp = extractComponentFromLine(line);
     if (!cls) continue;
-    if (!componentMap.has(comp)) componentMap.set(comp, { added: /* @__PURE__ */ new Set(), removed: /* @__PURE__ */ new Set() });
+    if (!componentMap.has(comp))
+      componentMap.set(comp, { added: /* @__PURE__ */ new Set(), removed: /* @__PURE__ */ new Set() });
     cls.split(/\s+/).filter(Boolean).forEach((c) => componentMap.get(comp).added.add(c));
   }
   for (const line of removedLines.filter((l) => /className=/.test(l))) {
     const cls = extractClassName(line);
     const comp = extractComponentFromLine(line);
     if (!cls) continue;
-    if (!componentMap.has(comp)) componentMap.set(comp, { added: /* @__PURE__ */ new Set(), removed: /* @__PURE__ */ new Set() });
+    if (!componentMap.has(comp))
+      componentMap.set(comp, { added: /* @__PURE__ */ new Set(), removed: /* @__PURE__ */ new Set() });
     cls.split(/\s+/).filter(Boolean).forEach((c) => componentMap.get(comp).removed.add(c));
   }
   const changes = [];
   for (const [comp, { added, removed }] of componentMap) {
+    if (comp === "unknown") continue;
     const pureAdded = [...added].filter((c) => !removed.has(c));
     const pureRemoved = [...removed].filter((c) => !added.has(c));
     if (pureAdded.length === 0 && pureRemoved.length === 0) continue;
@@ -86,7 +93,8 @@ function renderStyleChanges(changes) {
   for (const change of changes) {
     lines.push(`**${change.component}**`);
     if (change.added.length > 0) lines.push(`  + ${change.added.join("  ")}`);
-    if (change.removed.length > 0) lines.push(`  - ${change.removed.join("  ")}`);
+    if (change.removed.length > 0)
+      lines.push(`  - ${change.removed.join("  ")}`);
   }
   return lines;
 }
@@ -155,192 +163,76 @@ function parseJSXToFlowTree(lines) {
 }
 function filterDiffLines(diffText) {
   const lines = diffText.split("\n");
-  const added = lines.filter((l) => l.startsWith("+") && !l.startsWith("+++") && l.trim() !== "+").map((l) => l.substring(1));
-  const removed = lines.filter((l) => l.startsWith("-") && !l.startsWith("---") && l.trim() !== "-").map((l) => l.substring(1));
+  const added = lines.filter(
+    (l) => l.startsWith("+") && !l.startsWith("+++") && l.trim() !== "+"
+  ).map((l) => l.substring(1));
+  const removed = lines.filter(
+    (l) => l.startsWith("-") && !l.startsWith("---") && l.trim() !== "-"
+  ).map((l) => l.substring(1));
   return { added, removed };
 }
-function normalizeCode(lines) {
-  return lines.map((line) => {
-    let normalized = line;
-    normalized = normalized.replace(/\/\/.*$/, "");
-    normalized = normalized.replace(/\/\*.*?\*\//, "");
-    normalized = normalized.trim();
-    normalized = normalized.replace(/;$/, "");
-    return normalized;
-  }).filter((line) => line.length > 0);
-}
 function getIndentDepth(line) {
   const match = line.match(/^(\s*)/);
   if (!match) return 0;
   return Math.floor(match[1].length / 2);
 }
-function isChaining(line, prevLine) {
-  if (!prevLine) return false;
-  if (!line.trim().startsWith(".")) return false;
-  if (!prevLine.match(/[)\}]$/)) return false;
-  return true;
-}
-function extractChainMethod(line) {
-  const match = line.match(/\.(\w+)\(/);
-  if (match) return `${match[1]}()`;
-  return line.trim();
-}
-function simplifyCallback(methodCall) {
-  const arrowMatch = methodCall.match(/(\w+)\((\w+)\s*=>\s*(\w+)\.(\w+)\)/);
-  if (arrowMatch) {
-    const [, method, param, , prop] = arrowMatch;
-    return `${method}(${param} \u2192 ${prop})`;
-  }
-  const callbackMatch = methodCall.match(/(\w+)\([^)]+\)/);
-  if (callbackMatch) return `${callbackMatch[1]}(callback)`;
-  return methodCall;
-}
-function isConditional(line) {
-  return /^(if|else|switch)\s*[\(\{]/.test(line.trim());
-}
-function isLoop(line) {
-  return /^(for|while)\s*\(/.test(line.trim());
-}
-function isFunctionDeclaration(line) {
-  const t = line.trim();
-  return (
-    // function foo() / async function foo()
-    /^(async\s+)?function\s+\w+/.test(t) || // const foo = () => / const foo = async () => / const foo = async (x: T) =>
-    /^(const|let|var)\s+\w+\s*=\s*(async\s*)?\(/.test(t) || // const foo = function / const foo = async function
-    /^(const|let|var)\s+\w+\s*=\s*(async\s+)?function/.test(t)
-  );
-}
-function shouldIgnore(line) {
-  const ignorePatterns = [
-    /^import\s+/,
-    /^export\s+/,
-    /^type\s+/,
-    /^interface\s+/,
-    /^console\./,
-    /^return$/,
-    /^throw\s+/
-  ];
-  return ignorePatterns.some((p) => p.test(line.trim()));
-}
-function extractRoot(line) {
-  const assignMatch = line.match(/(?:const|let|var)\s+(\w+)\s*=\s*(\w+)/);
-  if (assignMatch) return assignMatch[2];
-  const callMatch = line.match(/^(\w+)\(/);
-  if (callMatch) return `${callMatch[1]}()`;
-  const methodMatch = line.match(/^(\w+)\./);
-  if (methodMatch) return methodMatch[1];
-  return null;
-}
-function parseToFlowTree(lines) {
-  const roots = [];
-  let currentChain = null;
-  let prevLine = null;
-  let baseDepth = -1;
-  for (let i = 0; i < lines.length; i++) {
-    const line = lines[i];
-    if (shouldIgnore(line)) {
-      prevLine = line;
-      continue;
-    }
-    const depth = getIndentDepth(lines[i]);
-    if (baseDepth === -1) baseDepth = depth;
-    const relativeDepth = depth - baseDepth;
-    if (isChaining(line, prevLine)) {
-      const method = extractChainMethod(line);
-      const simplified = simplifyCallback(method);
-      if (currentChain) {
-        const chainNode = {
-          type: "chain",
-          name: simplified,
-          children: [],
-          depth: relativeDepth,
-          priority: 1 /* CHAINING */
-        };
-        let parent = currentChain;
-        while (parent.children.length > 0 && parent.children[parent.children.length - 1].depth >= relativeDepth) {
-          const last = parent.children[parent.children.length - 1];
-          if (last.children.length > 0) parent = last;
-          else break;
-        }
-        parent.children.push(chainNode);
+function extractChangedSymbols(addedLines, removedLines) {
+  const FUNC_RE = /^(?:export\s+)?(?:async\s+)?function\s+([a-z]\w+)|^(?:export\s+)?(?:const|let|var)\s+([a-z]\w+)\s*=\s*(?:async\s+)?\(?|^(?:export\s+)?(?:const|let|var)\s+([a-z]\w+)\s*=\s*[a-z]\w+\s*[<(]/;
+  const COMPONENT_RE = /^(?:export\s+)?(?:default\s+)?(?:function|const)\s+([A-Z][a-z][A-Za-z0-9]*)/;
+  const extract = (lines) => {
+    const names = /* @__PURE__ */ new Set();
+    for (const line of lines) {
+      const cm = line.match(COMPONENT_RE) || line.match(FUNC_RE);
+      if (cm) {
+        const name = cm[1] || cm[2] || cm[3];
+        if (name) names.add(name);
       }
-      prevLine = line;
-      continue;
-    }
-    if (isFunctionDeclaration(line)) {
-      const funcMatch = line.match(/(?:function|const|let|var)\s+(\w+)/);
-      roots.push({
-        type: "function",
-        name: funcMatch ? `${funcMatch[1]}()` : "function()",
-        children: [],
-        depth: relativeDepth,
-        priority: 4 /* FUNCTION */
-      });
-      currentChain = null;
-      prevLine = line;
-      continue;
     }
-    const root = extractRoot(line);
-    if (root) {
-      currentChain = {
-        type: "root",
-        name: root,
-        children: [],
-        depth: relativeDepth,
-        priority: 1 /* CHAINING */
-      };
-      roots.push(currentChain);
-    } else if (isConditional(line)) {
-      const condMatch = line.match(/(if|else|switch)\s*\(([^)]+)\)/);
-      const condName = condMatch ? `${condMatch[1]} (${condMatch[2]})` : line.trim();
-      roots.push({
-        type: "condition",
-        name: condName,
-        children: [],
-        depth: relativeDepth,
-        priority: 2 /* CONDITIONAL */
-      });
-      currentChain = null;
-    } else if (isLoop(line)) {
-      roots.push({
-        type: "loop",
-        name: "loop",
-        children: [],
-        depth: relativeDepth,
-        priority: 3 /* LOOP */
-      });
-      currentChain = null;
+    return names;
+  };
+  const addedNames = extract(addedLines);
+  const removedNames = extract(removedLines);
+  const results = [];
+  const seen = /* @__PURE__ */ new Set();
+  for (const name of addedNames) {
+    seen.add(name);
+    results.push({
+      name,
+      status: removedNames.has(name) ? "modified" : "added"
+    });
+  }
+  for (const name of removedNames) {
+    if (!seen.has(name)) {
+      results.push({ name, status: "removed" });
     }
-    prevLine = line;
   }
-  return roots;
+  return results;
 }
-function renderFlowTree(nodes, indent = 0) {
+function renderJSXTreeCompact(nodes, maxDepth = 3) {
   const lines = [];
-  const prefix = indent === 0 ? "" : " ".repeat((indent - 1) * 4) + " \u2514\u2500 ";
-  for (const node of nodes) {
-    lines.push(prefix + node.name);
-    if (node.children.length > 0) {
-      lines.push(...renderFlowTree(node.children, indent + 1));
+  function walk(node, depth) {
+    if (depth > maxDepth) return;
+    const indent = "  ".repeat(depth);
+    const hasChildren = node.children.length > 0;
+    lines.push(`${indent}${node.name}${hasChildren ? "" : ""}`);
+    for (const child of node.children) {
+      walk(child, depth + 1);
     }
   }
-  return lines;
+  for (const root of nodes) {
+    walk(root, 0);
+  }
+  return lines.join("\n");
 }
 function generateReaderMarkdown(diffText, meta = {}) {
   const { added, removed } = filterDiffLines(diffText);
   const isJSX = Boolean(
     meta.file && isJSXFile(meta.file) || hasJSXContent(added)
   );
-  const addedForFlow = isJSX ? added.filter((l) => !isClassNameOnlyLine(l)) : added;
-  const normalizedAdded = normalizeCode(addedForFlow);
-  const flowTree = parseToFlowTree(normalizedAdded);
-  const rawCode = addedForFlow.join("\n");
-  const removedForCode = isJSX ? removed.filter((l) => !isClassNameOnlyLine(l)) : removed;
-  const removedCode = removedForCode.join("\n");
+  const changedSymbols = extractChangedSymbols(added, removed);
   const classNameChanges = isJSX ? parseClassNameChanges(added, removed) : [];
   const jsxTree = isJSX ? parseJSXToFlowTree(added) : [];
   const sections = [];
-  const lang = isJSX ? "tsx" : "typescript";
   sections.push("# \u{1F4D6} GitHub Reader View\n");
   sections.push("> Generated by **github-mobile-reader**");
   if (meta.repo) sections.push(`> Repository: ${meta.repo}`);
@@ -348,37 +240,29 @@ function generateReaderMarkdown(diffText, meta = {}) {
   if (meta.commit) sections.push(`> Commit: \`${meta.commit}\``);
   if (meta.file) sections.push(`> File: \`${meta.file}\``);
   sections.push("\n");
-  if (flowTree.length > 0) {
-    sections.push("## \u{1F9E0} Logical Flow\n");
-    sections.push("```");
-    sections.push(...renderFlowTree(flowTree));
-    sections.push("```\n");
+  if (changedSymbols.length > 0) {
+    sections.push("### \uBCC0\uACBD\uB41C \uD568\uC218 / \uCEF4\uD3EC\uB10C\uD2B8\n");
+    const STATUS_ICON = { added: "\u2705", removed: "\u274C", modified: "\u270F\uFE0F" };
+    for (const { name, status } of changedSymbols) {
+      sections.push(`- ${STATUS_ICON[status]} \`${name}()\` \u2014 ${status}`);
+    }
+    sections.push("");
   }
   if (isJSX && jsxTree.length > 0) {
-    sections.push("## \u{1F3A8} JSX Structure\n");
+    sections.push("### \u{1F3A8} JSX Structure\n");
     sections.push("```");
-    sections.push(...renderFlowTree(jsxTree));
+    sections.push(renderJSXTreeCompact(jsxTree));
     sections.push("```\n");
   }
   if (isJSX && classNameChanges.length > 0) {
-    sections.push("## \u{1F485} Style Changes\n");
+    sections.push("### \u{1F485} Style Changes\n");
     sections.push(...renderStyleChanges(classNameChanges));
     sections.push("");
   }
-  if (rawCode.trim()) {
-    sections.push("## \u2705 Added Code\n");
-    sections.push(`\`\`\`${lang}`);
-    sections.push(rawCode);
-    sections.push("```\n");
-  }
-  if (removedCode.trim()) {
-    sections.push("## \u274C Removed Code\n");
-    sections.push(`\`\`\`${lang}`);
-    sections.push(removedCode);
-    sections.push("```\n");
-  }
   sections.push("---");
-  sections.push("\u{1F6E0} Auto-generated by [github-mobile-reader](https://github.com/your-org/github-mobile-reader). Do not edit manually.");
+  sections.push(
+    "\u{1F6E0} Auto-generated by [github-mobile-reader](https://github.com/3rdflr/github-mobile-reader). Do not edit manually."
+  );
   return sections.join("\n");
 }