pi-hashline-edit-pro 7.0.0 → 7.1.0

package/src/outline/vendor/parsers/manager.ts

@@ -356,7 +356,6 @@ function extractJSChild(
           type: value.type === "arrow_function" ? "arrow_function" : "function",
           startLine: sl,
           endLine: el,
-          detail: extractFnDetail(value, source),
         });
       } else {
         const [sl, el] = nodeRange(d);
@@ -410,13 +409,7 @@ function extractJSChild(
 
   const [sl, el] = nodeRange(child);
 
-  // For functions/methods, try to get the parameters as detail
-  let detail: string | undefined;
-  if (type === "function_declaration" || type === "function_expression" || type === "method_definition" || type === "arrow_function") {
-    detail = extractFnDetail(child, source);
-  }
-
-  return { name, type: symbolType, startLine: sl, endLine: el, detail };
+  return { name, type: symbolType, startLine: sl, endLine: el };
 }
 
 const TEST_CALLEES = new Set(["describe", "test", "it", "suite", "context", "specify"]);
@@ -455,17 +448,6 @@ function extractTestCalls(call: Node, source: string): SymbolInfo[] {
   return results;
 }
 
-function extractFnDetail(
-  node: Node,
-  source: string,
-): string | undefined {
-  const params = node.childForFieldName("parameters");
-  if (params) {
-    return source.slice(params.startIndex, params.endIndex);
-  }
-  return undefined;
-}
-
 function findChildByType(
   node: Node,
   types: string[],
@@ -508,11 +490,7 @@ function extractPython(
     } else if (child.type === "function_definition") {
       const name = fieldChildText(child, "name", source) || "<anonymous>";
       const [sl, el] = nodeRange(child);
-      const params = child.childForFieldName("parameters");
-      const detail = params
-        ? source.slice(params.startIndex, params.endIndex)
-        : undefined;
-      symbol = { name, type: "function", startLine: sl, endLine: el, detail };
+      symbol = { name, type: "function", startLine: sl, endLine: el };
     } else if (child.type === "decorated_definition") {
       // Unwrap to the inner definition
       const def = child.namedChild(child.namedChildCount - 1);
@@ -543,11 +521,7 @@ function extractPythonSingle(
   if (child.type === "function_definition") {
     const name = fieldChildText(child, "name", source) || "<anonymous>";
     const [sl, el] = nodeRange(child);
-    const params = child.childForFieldName("parameters");
-    const detail = params
-      ? source.slice(params.startIndex, params.endIndex)
-      : undefined;
-    return { name, type: "function", startLine: sl, endLine: el, detail };
+    return { name, type: "function", startLine: sl, endLine: el };
   }
   return null;
 }
@@ -677,16 +651,11 @@ function extractGoFields(
     const name = fieldChildText(child, "name", source);
     if (!name) continue;
     const [sl, el] = nodeRange(child);
-    const typeNode = child.childForFieldName("type");
-    const typeText = typeNode
-      ? source.slice(typeNode.startIndex, typeNode.endIndex)
-      : "";
     results.push({
       name,
       type: "field",
       startLine: sl,
       endLine: el,
-      detail: typeText,
     });
   }
 
@@ -714,11 +683,7 @@ function extractRust(
     if (child.type === "function_item") {
       const name = fieldChildText(child, "name", source) || "<anonymous>";
       const [sl, el] = nodeRange(child);
-      const params = child.childForFieldName("parameters");
-      const detail = params
-        ? source.slice(params.startIndex, params.endIndex)
-        : undefined;
-      symbol = { name, type: "function", startLine: sl, endLine: el, detail };
+      symbol = { name, type: "function", startLine: sl, endLine: el };
     } else if (child.type === "struct_item") {
       const name = fieldChildText(child, "name", source) || "<anonymous>";
       const [sl, el] = nodeRange(child);
@@ -784,11 +749,7 @@ function extractRustImplBody(
     if (child.type === "function_item") {
       const name = fieldChildText(child, "name", source) || "<anonymous>";
       const [sl, el] = nodeRange(child);
-      const params = child.childForFieldName("parameters");
-      const detail = params
-        ? source.slice(params.startIndex, params.endIndex)
-        : undefined;
-      symbol = { name, type: "method", startLine: sl, endLine: el, detail };
+      symbol = { name, type: "method", startLine: sl, endLine: el };
     } else if (child.type === "const_item") {
       const name = fieldChildText(child, "name", source) || "<anonymous>";
       const [sl, el] = nodeRange(child);
@@ -827,20 +788,7 @@ function extractSolidity(
       const name = fieldChildText(child, "name", source) || "<anonymous>";
       const [sl, el] = nodeRange(child);
 
-      // Collect inheritance detail
-      const inherits: string[] = [];
-      for (let j = 0; j < child.namedChildCount; j++) {
-        const c = child.namedChild(j);
-        if (c && c.type === "inheritance_specifier") {
-          const ancestor = c.childForFieldName("ancestor");
-          if (ancestor) {
-            inherits.push(source.slice(ancestor.startIndex, ancestor.endIndex));
-          }
-        }
-      }
-      const detail = inherits.length > 0 ? `is ${inherits.join(", ")}` : undefined;
-
-      symbol = { name, type: "contract", startLine: sl, endLine: el, detail };
+      symbol = { name, type: "contract", startLine: sl, endLine: el };
 
       // Recurse into contract body
       const body = child.childForFieldName("body");
@@ -851,19 +799,7 @@ function extractSolidity(
       const name = fieldChildText(child, "name", source) || "<anonymous>";
       const [sl, el] = nodeRange(child);
 
-      const inherits: string[] = [];
-      for (let j = 0; j < child.namedChildCount; j++) {
-        const c = child.namedChild(j);
-        if (c && c.type === "inheritance_specifier") {
-          const ancestor = c.childForFieldName("ancestor");
-          if (ancestor) {
-            inherits.push(source.slice(ancestor.startIndex, ancestor.endIndex));
-          }
-        }
-      }
-      const detail = inherits.length > 0 ? `is ${inherits.join(", ")}` : undefined;
-
-      symbol = { name, type: "interface", startLine: sl, endLine: el, detail };
+      symbol = { name, type: "interface", startLine: sl, endLine: el };
 
       const body = child.childForFieldName("body");
       if (body && depth < MAX_DEPTH) {
@@ -914,17 +850,12 @@ function extractSolidityBody(
           if (!m) continue;
           if (m.type === "struct_member") {
             const memberName = fieldChildText(m, "name", source) || "";
-            const memberType = m.childForFieldName("type");
-            const typeStr = memberType
-              ? source.slice(memberType.startIndex, memberType.endIndex)
-              : "";
             const [msl, mel] = nodeRange(m);
             symbol.children.push({
               name: memberName,
               type: "field",
               startLine: msl,
               endLine: mel,
-              detail: typeStr,
             });
           }
         }
@@ -953,22 +884,6 @@ function extractSolidityBody(
   return results;
 }
 
-/** Collect all unnamed `parameter`/`event_parameter`/`error_parameter` named children and return their source ranges joined. */
-function solidityParamsText(
-  node: Node,
-  source: string,
-  paramType: string = "parameter",
-): string | undefined {
-  const parts: string[] = [];
-  for (let j = 0; j < node.namedChildCount; j++) {
-    const c = node.namedChild(j);
-    if (c && c.type === paramType) {
-      parts.push(source.slice(c.startIndex, c.endIndex));
-    }
-  }
-  return parts.length > 0 ? `(${parts.join(", ")})` : undefined;
-}
-
 function extractSolidityItem(
   child: Node,
   source: string,
@@ -978,71 +893,18 @@ function extractSolidityItem(
   if (type === "function_definition") {
     const name = fieldChildText(child, "name", source) || "<anonymous>";
     const [sl, el] = nodeRange(child);
-    const params = solidityParamsText(child, source, "parameter");
-
-    // Check for return type
-    let returnType: string | undefined;
-    for (let j = 0; j < child.namedChildCount; j++) {
-      const c = child.namedChild(j);
-      if (c && c.type === "return_type_definition") {
-        returnType = source.slice(c.startIndex, c.endIndex);
-        break;
-      }
-    }
-
-    // Collect qualifiers: visibility, state_mutability, virtual, override, modifiers
-    const qualifiers: string[] = [];
-    for (let j = 0; j < child.namedChildCount; j++) {
-      const c = child.namedChild(j);
-      if (!c) continue;
-      if (c.type === "visibility" || c.type === "state_mutability") {
-        qualifiers.push(source.slice(c.startIndex, c.endIndex));
-      } else if (c.type === "virtual") {
-        qualifiers.push("virtual");
-      } else if (c.type === "override_specifier") {
-        qualifiers.push(source.slice(c.startIndex, c.endIndex));
-      } else if (c.type === "modifier_invocation") {
-        qualifiers.push(source.slice(c.startIndex, c.endIndex));
-      }
-    }
-
-    const parts: string[] = [];
-    if (params) parts.push(params);
-    if (qualifiers.length > 0) parts.push(qualifiers.join(" "));
-    if (returnType) parts.push(returnType);
-    const detail = parts.length > 0 ? parts.join(" ") : undefined;
-
-    return { name, type: "function", startLine: sl, endLine: el, detail };
+    return { name, type: "function", startLine: sl, endLine: el };
   }
 
   if (type === "constructor_definition") {
     const [sl, el] = nodeRange(child);
-    const params = solidityParamsText(child, source, "parameter");
-
-    const qualifiers: string[] = [];
-    for (let j = 0; j < child.namedChildCount; j++) {
-      const c = child.namedChild(j);
-      if (!c) continue;
-      if (c.type === "visibility" || c.type === "state_mutability") {
-        qualifiers.push(source.slice(c.startIndex, c.endIndex));
-      } else if (c.type === "modifier_invocation") {
-        qualifiers.push(source.slice(c.startIndex, c.endIndex));
-      }
-    }
-
-    const parts: string[] = [];
-    if (params) parts.push(params);
-    if (qualifiers.length > 0) parts.push(qualifiers.join(" "));
-    const detail = parts.length > 0 ? parts.join(" ") : undefined;
-
-    return { name: "constructor", type: "constructor", startLine: sl, endLine: el, detail };
+    return { name: "constructor", type: "constructor", startLine: sl, endLine: el };
   }
 
   if (type === "modifier_definition") {
     const name = fieldChildText(child, "name", source) || "<anonymous>";
     const [sl, el] = nodeRange(child);
-    const detail = solidityParamsText(child, source, "parameter");
-    return { name, type: "modifier", startLine: sl, endLine: el, detail };
+    return { name, type: "modifier", startLine: sl, endLine: el };
   }
 
   if (type === "event_definition") {
@@ -1054,21 +916,7 @@ function extractSolidityItem(
   if (type === "error_declaration") {
     const name = childText(child, "identifier", source) || "<anonymous>";
     const [sl, el] = nodeRange(child);
-    // Collect error parameters for detail
-    const params: string[] = [];
-    for (let j = 0; j < child.namedChildCount; j++) {
-      const c = child.namedChild(j);
-      if (c && c.type === "error_parameter") {
-        const typeName = c.childForFieldName("type");
-        if (typeName) {
-          const typeText = source.slice(typeName.startIndex, typeName.endIndex);
-          const paramName = fieldChildText(c, "name", source);
-          params.push(paramName ? `${typeText} ${paramName}` : typeText);
-        }
-      }
-    }
-    const detail = params.length > 0 ? `(${params.join(", ")})` : undefined;
-    return { name, type: "error", startLine: sl, endLine: el, detail };
+    return { name, type: "error", startLine: sl, endLine: el };
   }
 
   if (type === "struct_declaration") {
@@ -1096,36 +944,19 @@ function extractSolidityItem(
       }
     }
 
-    const qualifiers: string[] = [];
-    for (let j = 0; j < child.namedChildCount; j++) {
-      const c = child.namedChild(j);
-      if (!c) continue;
-      if (c.type === "visibility" || c.type === "state_mutability") {
-        qualifiers.push(source.slice(c.startIndex, c.endIndex));
-      } else if (c.type === "virtual") {
-        qualifiers.push("virtual");
-      } else if (c.type === "override_specifier") {
-        qualifiers.push(source.slice(c.startIndex, c.endIndex));
-      } else if (c.type === "modifier_invocation") {
-        qualifiers.push(source.slice(c.startIndex, c.endIndex));
-      }
-    }
 
     return {
       name: symbolType,
       type: symbolType,
       startLine: sl,
       endLine: el,
-      detail: qualifiers.length > 0 ? qualifiers.join(" ") : undefined,
     };
   }
 
   if (type === "state_variable_declaration") {
     const name = fieldChildText(child, "name", source) || "<anonymous>";
     const [sl, el] = nodeRange(child);
-    const varType = child.childForFieldName("type");
-    const detail = varType ? source.slice(varType.startIndex, varType.endIndex) : undefined;
-    return { name, type: "variable", startLine: sl, endLine: el, detail };
+    return { name, type: "variable", startLine: sl, endLine: el };
   }
 
   return null;
@@ -1542,7 +1373,6 @@ function extractMarkdown(source: string): SymbolInfo[] {
         startLine,
         endLine: lines.length, // Will be updated later
         children: [],
-        detail: `#`.repeat(level),
       };
       
       // Find parent: pop stack until we find a heading with lower level
@@ -1589,7 +1419,6 @@ function extractMarkdown(source: string): SymbolInfo[] {
       type: "code_block",
       startLine,
       endLine,
-      detail: language,
     };
     
     // Find the heading that contains this code block

package/src/outline/vendor/parsers/php.ts

@@ -8,34 +8,23 @@ const PHP_TYPE_LABELS: Record<string, string> = {
   enum_declaration: "enum",
 };
 
-const MODIFIER_TYPES = new Set([
-  "visibility_modifier",
-  "static_modifier",
-  "abstract_modifier",
-  "final_modifier",
-  "readonly_modifier",
-]);
-
 function textOf(node: Node, source: string): string {
   return source.slice(node.startIndex, node.endIndex);
 }
 
-function rangeOf(node: Node): [number, number] {
-  return [node.startPosition.row + 1, node.endPosition.row + 1];
-}
+const PHP_SKIPPED_TYPES = new Set(["attribute_list", "comment"]);
 
-function detailOf(parts: Array<string | undefined>): string | undefined {
-  const kept = parts.filter((part): part is string => part !== undefined && part.length > 0);
-  return kept.length > 0 ? kept.join(" ") : undefined;
+function firstDeclaredTokenRow(node: Node): number | undefined {
+  for (let index = 0; index < node.childCount; index++) {
+    const child = node.child(index);
+    if (child === null || PHP_SKIPPED_TYPES.has(child.type)) continue;
+    return child.startPosition.row + 1;
+  }
+  return undefined;
 }
 
-function modifiersOf(node: Node, source: string): string[] {
-  const modifiers: string[] = [];
-  for (let i = 0; i < node.namedChildCount; i++) {
-    const child = node.namedChild(i);
-    if (child && MODIFIER_TYPES.has(child.type)) modifiers.push(textOf(child, source));
-  }
-  return modifiers;
+function rangeOf(node: Node): [number, number] {
+  return [firstDeclaredTokenRow(node) ?? node.startPosition.row + 1, node.endPosition.row + 1];
 }
 
 function findChild(node: Node, type: string): Node | undefined {
@@ -46,37 +35,11 @@ function findChild(node: Node, type: string): Node | undefined {
   return undefined;
 }
 
-function nameFieldsOf(node: Node, source: string): string[] {
-  const names: string[] = [];
-  for (let i = 0; i < node.namedChildCount; i++) {
-    const child = node.namedChild(i);
-    if (child !== null && child.type === "name") names.push(textOf(child, source));
-  }
-  return names;
-}
-
 function nameOf(node: Node, source: string): string {
   const name = findChild(node, "name");
   return name === undefined ? "<unknown>" : textOf(name, source);
 }
 
-function paramsOf(node: Node, source: string): string | undefined {
-  const params = node.childForFieldName("parameters");
-  return params === null ? undefined : textOf(params, source);
-}
-
-function returnOf(node: Node, source: string): string | undefined {
-  const returned = node.childForFieldName("return_type");
-  return returned === null ? undefined : `: ${textOf(returned, source)}`;
-}
-
-function signatureOf(node: Node, source: string): string | undefined {
-  const params = paramsOf(node, source);
-  const returned = returnOf(node, source);
-  if (params === undefined) return returned;
-  return returned === undefined ? params : `${params}${returned}`;
-}
-
 function extractMethod(node: Node, source: string): SymbolInfo {
   const [startLine, endLine] = rangeOf(node);
   return {
@@ -84,7 +47,6 @@ function extractMethod(node: Node, source: string): SymbolInfo {
     type: "method",
     startLine,
     endLine,
-    detail: detailOf([...modifiersOf(node, source), signatureOf(node, source)]),
   };
 }
 
@@ -95,58 +57,52 @@ function extractFunction(node: Node, source: string): SymbolInfo {
     type: "function",
     startLine,
     endLine,
-    detail: detailOf([signatureOf(node, source)]),
   };
 }
 
 function extractProperties(node: Node, source: string): SymbolInfo[] {
-  const modifiers = modifiersOf(node, source);
-  const typeNode = node.childForFieldName("type");
-  const propertyType = typeNode === null ? undefined : textOf(typeNode, source);
   const results: SymbolInfo[] = [];
+  const declarationStart = firstDeclaredTokenRow(node) ?? node.startPosition.row + 1;
   for (let i = 0; i < node.namedChildCount; i++) {
     const element = node.namedChild(i);
     if (element === null || element.type !== "property_element") continue;
-    const [startLine, endLine] = rangeOf(element);
+    const [elementStart, endLine] = rangeOf(element);
+    const startLine = results.length === 0 ? Math.min(declarationStart, elementStart) : elementStart;
     results.push({
       name: element.childForFieldName("name")?.text ?? "<unknown>",
       type: "property",
       startLine,
       endLine,
-      detail: detailOf([...modifiers, propertyType]),
     });
   }
   return results;
 }
 
 function extractConstants(node: Node, source: string): SymbolInfo[] {
-  const modifiers = modifiersOf(node, source);
   const results: SymbolInfo[] = [];
+  const declarationStart = firstDeclaredTokenRow(node) ?? node.startPosition.row + 1;
   for (let i = 0; i < node.namedChildCount; i++) {
     const element = node.namedChild(i);
     if (element === null || element.type !== "const_element") continue;
-    const value = element.namedChild(element.namedChildCount - 1);
-    const [startLine, endLine] = rangeOf(element);
+    const [elementStart, endLine] = rangeOf(element);
+    const startLine = results.length === 0 ? Math.min(declarationStart, elementStart) : elementStart;
     results.push({
       name: nameOf(element, source),
       type: "const",
       startLine,
       endLine,
-      detail: detailOf([...modifiers, value === null ? undefined : `= ${textOf(value, source)}`]),
     });
   }
   return results;
 }
 
 function extractEnumCase(node: Node, source: string): SymbolInfo {
-  const value = node.childForFieldName("value");
   const [startLine, endLine] = rangeOf(node);
   return {
     name: node.childForFieldName("name")?.text ?? "<unknown>",
     type: "case",
     startLine,
     endLine,
-    detail: detailOf([value === null ? undefined : `= ${textOf(value, source)}`]),
   };
 }
 
@@ -176,32 +132,12 @@ function extractMembers(body: Node, source: string): SymbolInfo[] {
 }
 
 function extractTypeDeclaration(node: Node, source: string, depth: number, maxDepth: number): SymbolInfo {
-  const detailParts: string[] = [...modifiersOf(node, source)];
-  const base = findChild(node, "base_clause");
-  if (base !== undefined) {
-    const baseNames = nameFieldsOf(base, source);
-    if (node.type === "interface_declaration") {
-      if (baseNames.length > 0) detailParts.push(`extends ${baseNames.join(", ")}`);
-    } else if (baseNames.length > 0 && baseNames[0] !== undefined) {
-      detailParts.push(`extends ${baseNames[0]}`);
-    }
-  }
-  if (node.type === "enum_declaration") {
-    const backing = findChild(node, "primitive_type");
-    if (backing !== undefined) detailParts.push(`: ${textOf(backing, source)}`);
-  }
-  const implemented = findChild(node, "class_interface_clause");
-  if (implemented !== undefined) {
-    const implementedNames = nameFieldsOf(implemented, source);
-    if (implementedNames.length > 0) detailParts.push(`implements ${implementedNames.join(", ")}`);
-  }
   const [startLine, endLine] = rangeOf(node);
   const symbol: SymbolInfo = {
     name: node.childForFieldName("name")?.text ?? "<unknown>",
     type: PHP_TYPE_LABELS[node.type] ?? "class",
     startLine,
     endLine,
-    detail: detailOf(detailParts),
   };
   if (depth < maxDepth) {
     const body = findChild(node, "declaration_list") ?? findChild(node, "enum_declaration_list");
@@ -227,7 +163,6 @@ function extractAssignedFunction(statement: Node, source: string): SymbolInfo |
       type: value.type === "arrow_function" ? "arrow_function" : "function",
       startLine,
       endLine,
-      detail: detailOf([signatureOf(value, source)]),
     };
   }
   return null;

package/src/outline/vendor/parsers/sass.ts

@@ -96,18 +96,12 @@ function extractNamedBlock(
   label: string,
 ): SymbolInfo | null {
   let name = "<unknown>";
-  let detail: string | undefined;
 
   for (let i = 0; i < child.namedChildCount; i++) {
     const c = child.namedChild(i);
     if (!c) continue;
     if (c.type === "name") {
       name = source.slice(c.startIndex, c.endIndex);
-    } else if (c.type === "parameters") {
-      detail =
-        "(" +
-        source.slice(c.startIndex, c.endIndex).replace(/^\(|\)$/g, "") +
-        ")";
     }
   }
 
@@ -117,7 +111,6 @@ function extractNamedBlock(
     type: label,
     startLine: sl,
     endLine: el,
-    detail,
   };
 
   if (depth < MAX_DEPTH) {
@@ -293,15 +286,12 @@ function extractBlockChildren(
 
 function extractInclude(child: Node, source: string): SymbolInfo | null {
   let name: string | undefined;
-  let detail: string | undefined;
 
   for (let i = 0; i < child.namedChildCount; i++) {
     const c = child.namedChild(i);
     if (!c) continue;
     if (c.type === "mixin_name") {
       name = source.slice(c.startIndex, c.endIndex);
-    } else if (c.type === "arguments") {
-      detail = source.slice(c.startIndex, c.endIndex);
     }
   }
   if (!name) return null;
@@ -311,7 +301,6 @@ function extractInclude(child: Node, source: string): SymbolInfo | null {
     type: "include",
     startLine: child.startPosition.row + 1,
     endLine: child.endPosition.row + 1,
-    detail,
   };
 }
 

package/src/outline/vendor/parsers/scss.ts

@@ -116,15 +116,12 @@ function extractNamedBlock(
   nameField: string,
 ): SymbolInfo | null {
   let name = "<unknown>";
-  let detail: string | undefined;
 
   for (let i = 0; i < child.namedChildCount; i++) {
     const c = child.namedChild(i);
     if (!c) continue;
     if (c.type === nameField) {
       name = source.slice(c.startIndex, c.endIndex);
-    } else if (c.type === "parameters") {
-      detail = "(" + source.slice(c.startIndex, c.endIndex).replace(/^\(|\)$/g, "") + ")";
     }
   }
 
@@ -134,7 +131,6 @@ function extractNamedBlock(
     type: label,
     startLine: sl,
     endLine: el,
-    detail,
   };
 
   if (depth < MAX_DEPTH) {
@@ -345,15 +341,12 @@ function extractBlockChildren(
  */
 function extractInclude(child: Node, source: string): SymbolInfo | null {
   let name: string | undefined;
-  let detail: string | undefined;
 
   for (let i = 0; i < child.namedChildCount; i++) {
     const c = child.namedChild(i);
     if (!c) continue;
     if (c.type === "identifier") {
       name = source.slice(c.startIndex, c.endIndex);
-    } else if (c.type === "arguments") {
-      detail = source.slice(c.startIndex, c.endIndex);
     }
   }
   if (!name) return null;
@@ -363,7 +356,6 @@ function extractInclude(child: Node, source: string): SymbolInfo | null {
     type: "include",
     startLine: child.startPosition.row + 1,
     endLine: child.endPosition.row + 1,
-    detail,
   };
 }
 

package/src/outline/vendor/types.ts

@@ -5,7 +5,6 @@ export interface SymbolInfo {
   startLine: number; // 1-indexed
   endLine: number;
   children?: SymbolInfo[];
-  detail?: string; // optional extra info (e.g. parameters, visibility)
 }
 
 /** Result of parsing a source file. */