Fix Python Symbol Kind Mapping for CTags

Teach the CTags symbol parser to consider the source file extension when mapping
ctags output into internal symbol kinds. This fixes Python methods that are
reported as "member" entries by ctags so they are recognized as methods rather
than being dropped or misclassified.

The parser now derives the file extension from the target file path and passes
it through the symbol-kind mapping path, keeping the behavior otherwise
unchanged for other languages.

* CTags parsing
* Symbol kind mapping

**Generated by CodeLite**

Signed-off-by: Eran Ifrah <eran@codelite.org>

Author: eranif

CodeLite/CTags.cpp

@@ -100,8 +100,10 @@ bool CTags::IsSupportedSymbolLanguage(const wxString& language)
     return std::find(supported.begin(), supported.end(), lang) != supported.end();
 }
 
-std::optional<CTags::SymbolKind>
-CTags::MapSymbolKind(const wxString& kind, const wxString& scope, const wxString& kind_from_ctags)
+std::optional<CTags::SymbolKind> CTags::MapSymbolKind(const wxString& kind,
+                                                      const wxString& scope,
+                                                      const wxString& kind_from_ctags,
+                                                      const wxString& file_ext)
 {
     const wxString k = kind.Lower();
     const wxString c = kind_from_ctags.Lower();
@@ -115,6 +117,9 @@ CTags::MapSymbolKind(const wxString& kind, const wxString& scope, const wxString
         return SymbolKind::kPrototype;
     if (k == "method" || c == "method")
         return SymbolKind::kMethod;
+    if (file_ext == "py" && c == "member")
+        // Python methods are marked as "members"
+        return SymbolKind::kMethod;
     if (k == "function") {
         if (scope.empty() || scope == "<global>")
             return SymbolKind::kGlobalMethod;
@@ -142,7 +147,8 @@ std::optional<wxString> CTags::DoSymbolGenerate(const wxString& file, const wxSt
     return wxString::FromUTF8(result.out);
 }
 
-std::vector<CTags::SymbolInfo> CTags::ParseSymbolOutput(const wxString& content, const wxString& filename)
+std::vector<CTags::SymbolInfo>
+CTags::ParseSymbolOutput(const wxString& content, const wxString& filename, const wxString& file_ext)
 {
     std::vector<SymbolInfo> out;
     wxArrayString lines = ::wxStringTokenize(content, "\n", wxTOKEN_STRTOK);
@@ -163,7 +169,7 @@ std::vector<CTags::SymbolInfo> CTags::ParseSymbolOutput(const wxString& content,
                 scope = wxString::FromUTF8(entry["scope"].get<std::string>());
             }
 
-            auto mapped_kind = MapSymbolKind(kind, scope, kind);
+            auto mapped_kind = MapSymbolKind(kind, scope, kind, file_ext);
             if (!mapped_kind) {
                 continue;
             }
@@ -233,8 +239,9 @@ clStatusOr<std::vector<CTags::SymbolInfo>> CTags::ParseFileSymbols(const wxStrin
     if (!content)
         return std::vector<CTags::SymbolInfo>{};
 
+    wxString file_ext = wxFileName{tmpfile.get() ? tmpfile->GetFullPath() : file}.GetExt();
     clDEBUG() << "Parsing symbols..." << endl;
-    auto symbols = ParseSymbolOutput(*content, file);
+    auto symbols = ParseSymbolOutput(*content, file, file_ext);
     clDEBUG() << "Parsing symbols...done" << endl;
 
     clDEBUG() << "Sorting symbols..." << endl;

CodeLite/CTags.hpp

@@ -226,15 +226,18 @@ class WXDLLIMPEXP_CL CTags
      *
      * @param content wxString The input text containing one JSON object per line.
      * @param filename wxString The file path to assign to each parsed symbol entry.
-     *
+     * @param file_ext wxString the filename extension.
      * @return std::vector<CTags::SymbolInfo> A vector of parsed symbol records, possibly empty if no valid entries are
      * found.
      *
      * @throws None. Any parsing or conversion errors are caught internally and the corresponding line is ignored.
      */
-    static std::vector<SymbolInfo> ParseSymbolOutput(const wxString& content, const wxString& filename);
-    static std::optional<SymbolKind>
-    MapSymbolKind(const wxString& kind, const wxString& scope, const wxString& kind_from_ctags);
+    static std::vector<SymbolInfo>
+    ParseSymbolOutput(const wxString& content, const wxString& filename, const wxString& file_ext);
+    static std::optional<SymbolKind> MapSymbolKind(const wxString& kind,
+                                                   const wxString& scope,
+                                                   const wxString& kind_from_ctags,
+                                                   const wxString& file_ext);
 };
 
 #endif // CTAGSGENERATOR_HPP