fix(protols): Remove node.parent.unwrap() or add safety (#121)

I can't seem to find the excact cause of this but having this not fail
is better than crashing

fixes #116

Signed-off-by: mohammadkhan <mohammadkhan@digitalocean.com>

Author: asharkhan3101

.devcontainer/Dockerfile

@@ -3,6 +3,7 @@ ARG BASE_IMAGE=mcr.microsoft.com/devcontainers/base:trixie
 FROM $BASE_IMAGE
 
 LABEL org.opencontainers.image.source=https://github.com/coder3101/protols
+LABEL org.opencontainers.image.description="Development container for protols"
 
 RUN apt-get update && export DEBIAN_FRONTEND=noninteractive \
     && apt-get install -y --no-install-recommends \

src/parser/docsymbol.rs

@@ -59,6 +59,9 @@ impl ParsedTree {
                 let name = node.utf8_text(content).unwrap();
                 let kind = NodeKind::to_symbolkind(&node);
                 let detail = self.find_preceding_comments(node.id(), content);
+
+                // Safety: Userdefined nodes usually have a parent as
+                // the document itself.
                 let message = node.parent().unwrap();
 
                 // https://github.com/rust-lang/rust/issues/102777

src/parser/rename.rs

@@ -10,7 +10,9 @@ impl ParsedTree {
         self.get_node_at_position(pos)
             .filter(NodeKind::is_identifier)
             .and_then(|n| {
-                if n.parent().is_some() && NodeKind::is_userdefined(&n.parent().unwrap()) {
+                if let Some(parent) = n.parent()
+                    && NodeKind::is_userdefined(&parent)
+                {
                     Some(Range {
                         start: ts_to_lsp_position(&n.start_position()),
                         end: ts_to_lsp_position(&n.end_position()),

src/parser/tree.rs

@@ -136,11 +136,11 @@ impl ParsedTree {
 
     pub fn get_user_defined_node<'a>(&'a self, pos: &Position) -> Option<Node<'a>> {
         self.get_node_at_position(pos)
-            .map(|n| {
+            .and_then(|n| {
                 if NodeKind::is_actionable(&n) {
-                    n
+                    Some(n)
                 } else {
-                    n.parent().unwrap()
+                    n.parent()
                 }
             })
             .filter(NodeKind::is_actionable)