Fixes IS CONNECTED concatenate bug in formatter (#401)

neo4j · Mar 11, 2025 · 224b9d4 · 224b9d4
1 parent 8faf81a
commit 224b9d4
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/packages/language-support/src/formatting/formatting.ts b/packages/language-support/src/formatting/formatting.ts
@@ -768,8 +768,8 @@ export class TreePrintVisitor extends CypherCmdParserVisitor<void> {
   ) => {
     this.visitIfNotNull(ctx.variable());
     this.visitIfNotNull(ctx.labelExpression());
-    if (ctx instanceof RelationshipPatternContext) {
-      this.visitIfNotNull(ctx.pathLength());
+    if (ctx instanceof RelationshipPatternContext && ctx.pathLength()) {
+      this.visit(ctx.pathLength());
       this.concatenate();
     }
     this.visitIfNotNull(ctx.properties());

diff --git a/packages/language-support/src/tests/formatting/edgecases.test.ts b/packages/language-support/src/tests/formatting/edgecases.test.ts
@@ -617,4 +617,11 @@ FOREACH (item IN items |
 RETURN n`;
     verifyFormatting(query, expected);
   });
+
+  test('relation with IS CONNECTED should not concatenate to ISCONNECTED', () => {
+    const query = `MATCH (n)-[IS CONNECTED]->(m)
+RETURN n, m`;
+    const expected = query;
+    verifyFormatting(query, expected);
+  });
 });