github-mobile-reader 0.1.0 → 0.1.2

package/dist/action.js

@@ -28,6 +28,131 @@ var path = __toESM(require("path"));
 var import_child_process = require("child_process");
 
 // src/parser.ts
+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));
+}
+function isClassNameOnlyLine(line) {
+  return /^className=/.test(line.trim());
+}
+function extractClassName(line) {
+  const staticMatch = line.match(/className="([^"]*)"/);
+  if (staticMatch) return staticMatch[1];
+  const ternaryMatch = line.match(/className=\{[^?]+\?\s*"([^"]*)"\s*:\s*"([^"]*)"\}/);
+  if (ternaryMatch) return `${ternaryMatch[1]} ${ternaryMatch[2]}`;
+  const templateMatch = line.match(/className=\{`([^`]*)`\}/);
+  if (templateMatch) {
+    const raw = templateMatch[1];
+    const literals = raw.replace(/\$\{[^}]*\}/g, " ").trim();
+    const exprStrings = [...raw.matchAll(/"([^"]*)"/g)].map((m) => m[1]);
+    return [literals, ...exprStrings].filter(Boolean).join(" ");
+  }
+  return null;
+}
+function extractComponentFromLine(line) {
+  const tagMatch = line.match(/<([A-Za-z][A-Za-z0-9.]*)/);
+  if (tagMatch) return tagMatch[1];
+  return "unknown";
+}
+function parseClassNameChanges(addedLines, removedLines) {
+  const componentMap = /* @__PURE__ */ new Map();
+  for (const line of addedLines.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() });
+    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() });
+    cls.split(/\s+/).filter(Boolean).forEach((c) => componentMap.get(comp).removed.add(c));
+  }
+  const changes = [];
+  for (const [comp, { added, removed }] of componentMap) {
+    const pureAdded = [...added].filter((c) => !removed.has(c));
+    const pureRemoved = [...removed].filter((c) => !added.has(c));
+    if (pureAdded.length === 0 && pureRemoved.length === 0) continue;
+    changes.push({ component: comp, added: pureAdded, removed: pureRemoved });
+  }
+  return changes;
+}
+function renderStyleChanges(changes) {
+  const lines = [];
+  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("  ")}`);
+  }
+  return lines;
+}
+function isJSXElement(line) {
+  const t = line.trim();
+  return /^<[A-Za-z]/.test(t) || /^<\/[A-Za-z]/.test(t);
+}
+function isJSXClosing(line) {
+  return /^<\/[A-Za-z]/.test(line.trim());
+}
+function isJSXSelfClosing(line) {
+  return /\/>[\s]*$/.test(line.trim());
+}
+function extractJSXComponentName(line) {
+  const trimmed = line.trim();
+  const closingMatch = trimmed.match(/^<\/([A-Za-z][A-Za-z0-9.]*)/);
+  if (closingMatch) return `/${closingMatch[1]}`;
+  const nameMatch = trimmed.match(/^<([A-Za-z][A-Za-z0-9.]*)/);
+  if (!nameMatch) return trimmed;
+  const name = nameMatch[1];
+  const eventProps = [];
+  for (const m of trimmed.matchAll(/\b(on[A-Z]\w+)=/g)) {
+    eventProps.push(m[1]);
+  }
+  return eventProps.length > 0 ? `${name}(${eventProps.join(", ")})` : name;
+}
+function shouldIgnoreJSX(line) {
+  const t = line.trim();
+  return isClassNameOnlyLine(t) || /^style=/.test(t) || /^aria-/.test(t) || /^data-/.test(t) || /^strokeLinecap=/.test(t) || /^strokeLinejoin=/.test(t) || /^strokeWidth=/.test(t) || /^viewBox=/.test(t) || /^fill=/.test(t) || /^stroke=/.test(t) || /^d="/.test(t) || t === "{" || t === "}" || t === "(" || t === ")" || t === "<>" || t === "</>" || /^\{\/\*/.test(t);
+}
+function parseJSXToFlowTree(lines) {
+  const roots = [];
+  const stack = [];
+  for (const line of lines) {
+    if (!isJSXElement(line)) continue;
+    if (shouldIgnoreJSX(line)) continue;
+    const depth = getIndentDepth(line);
+    if (isJSXClosing(line)) {
+      while (stack.length > 0 && stack[stack.length - 1].depth >= depth) {
+        stack.pop();
+      }
+      continue;
+    }
+    const name = extractJSXComponentName(line);
+    const selfClosing = isJSXSelfClosing(line);
+    const node = {
+      type: "call",
+      name,
+      children: [],
+      depth,
+      priority: 5 /* OTHER */
+    };
+    while (stack.length > 0 && stack[stack.length - 1].depth >= depth) {
+      stack.pop();
+    }
+    if (stack.length === 0) {
+      roots.push(node);
+    } else {
+      stack[stack.length - 1].node.children.push(node);
+    }
+    if (!selfClosing) {
+      stack.push({ node, depth });
+    }
+  }
+  return roots;
+}
 function filterDiffLines(diffText) {
   const lines = diffText.split("\n");
   const added = lines.filter((l) => l.startsWith("+") && !l.startsWith("+++") && l.trim() !== "+").map((l) => l.substring(1));
@@ -37,10 +162,10 @@ function filterDiffLines(diffText) {
 function normalizeCode(lines) {
   return lines.map((line) => {
     let normalized = line;
-    normalized = normalized.replace(/;$/, "");
     normalized = normalized.replace(/\/\/.*$/, "");
     normalized = normalized.replace(/\/\*.*?\*\//, "");
     normalized = normalized.trim();
+    normalized = normalized.replace(/;$/, "");
     return normalized;
   }).filter((line) => line.length > 0);
 }
@@ -77,7 +202,13 @@ function isLoop(line) {
   return /^(for|while)\s*\(/.test(line.trim());
 }
 function isFunctionDeclaration(line) {
-  return /^(function|const|let|var)\s+\w+\s*=?\s*(async\s*)?\(/.test(line.trim()) || /^(async\s+)?function\s+\w+/.test(line.trim());
+  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 = [
@@ -136,6 +267,19 @@ function parseToFlowTree(lines) {
       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 = {
@@ -166,16 +310,6 @@ function parseToFlowTree(lines) {
         priority: 3 /* LOOP */
       });
       currentChain = null;
-    } else 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;
   }
@@ -192,19 +326,21 @@ function renderFlowTree(nodes, indent = 0) {
   }
   return lines;
 }
-function parseDiffToLogicalFlow(diffText) {
+function generateReaderMarkdown(diffText, meta = {}) {
   const { added, removed } = filterDiffLines(diffText);
-  const normalizedAdded = normalizeCode(added);
+  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);
-  return {
-    root: flowTree,
-    rawCode: added.join("\n"),
-    removedCode: removed.join("\n")
-  };
-}
-function generateReaderMarkdown(diffText, meta = {}) {
-  const result = parseDiffToLogicalFlow(diffText);
+  const rawCode = addedForFlow.join("\n");
+  const removedForCode = isJSX ? removed.filter((l) => !isClassNameOnlyLine(l)) : removed;
+  const removedCode = removedForCode.join("\n");
+  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}`);
@@ -212,22 +348,33 @@ function generateReaderMarkdown(diffText, meta = {}) {
   if (meta.commit) sections.push(`> Commit: \`${meta.commit}\``);
   if (meta.file) sections.push(`> File: \`${meta.file}\``);
   sections.push("\n");
-  if (result.root.length > 0) {
+  if (flowTree.length > 0) {
     sections.push("## \u{1F9E0} Logical Flow\n");
     sections.push("```");
-    sections.push(...renderFlowTree(result.root));
+    sections.push(...renderFlowTree(flowTree));
     sections.push("```\n");
   }
-  if (result.rawCode.trim()) {
+  if (isJSX && jsxTree.length > 0) {
+    sections.push("## \u{1F3A8} JSX Structure\n");
+    sections.push("```");
+    sections.push(...renderFlowTree(jsxTree));
+    sections.push("```\n");
+  }
+  if (isJSX && classNameChanges.length > 0) {
+    sections.push("## \u{1F485} Style Changes\n");
+    sections.push(...renderStyleChanges(classNameChanges));
+    sections.push("");
+  }
+  if (rawCode.trim()) {
     sections.push("## \u2705 Added Code\n");
-    sections.push("```typescript");
-    sections.push(result.rawCode);
+    sections.push(`\`\`\`${lang}`);
+    sections.push(rawCode);
     sections.push("```\n");
   }
-  if (result.removedCode.trim()) {
+  if (removedCode.trim()) {
     sections.push("## \u274C Removed Code\n");
-    sections.push("```typescript");
-    sections.push(result.removedCode);
+    sections.push(`\`\`\`${lang}`);
+    sections.push(removedCode);
     sections.push("```\n");
   }
   sections.push("---");