The method shouldn't say non-methods/nulls can have method bodies, knowing what to do in the edge case is the caller's responsibility

Author: GrahamTheCoder

CodeConverter/CSharp/DeclarationNodeVisitor.cs

@@ -655,7 +655,7 @@ public override async Task<CSharpSyntaxNode> VisitMethodBlock(VBSyntax.MethodBlo
         var methodBlock = await node.SubOrFunctionStatement.AcceptAsync<BaseMethodDeclarationSyntax>(TriviaConvertingDeclarationVisitor, SourceTriviaMapKind.SubNodesOnly);
 
         var declaredSymbol = ModelExtensions.GetDeclaredSymbol(_semanticModel, node);
-        if (!declaredSymbol.CanHaveMethodBody()) {
+        if (declaredSymbol?.CanHaveMethodBody() == false) {
             return methodBlock;
         }
         var csReturnVariableOrNull = CommonConversions.GetRetVariableNameOrNull(node);
@@ -786,7 +786,8 @@ public override async Task<CSharpSyntaxNode> VisitMethodStatement(VBSyntax.Metho
             null
         );
 
-        return hasBody && declaredSymbol.CanHaveMethodBody() ? decl : decl.WithSemicolonToken(SemicolonToken);
+        bool canHaveMethodBody = declaredSymbol?.CanHaveMethodBody() != false;
+        return hasBody && canHaveMethodBody ? decl : decl.WithSemicolonToken(SemicolonToken);
     }
 
     public override async Task<CSharpSyntaxNode> VisitEventBlock(VBSyntax.EventBlockSyntax node)

CodeConverter/Util/ISymbolExtensions.cs

@@ -75,7 +75,7 @@ public static bool IsPartialMethodImplementation(this ISymbol declaredSymbol)
 
     public static bool CanHaveMethodBody(this ISymbol declaredSymbol)
     {
-        return !(declaredSymbol is IMethodSymbol ms) || (!ms.IsPartialDefinition && (ms.PartialImplementationPart == null && !ms.IsExtern));
+        return declaredSymbol is IMethodSymbol ms && (!ms.IsPartialDefinition && (ms.PartialImplementationPart == null && !ms.IsExtern));
     }
 
     public static bool IsPartialMethodDefinition(this ISymbol declaredSymbol)