Fix parsing of /** **/ nested block comments

Fixes #747
sql-formatter-org · Jun 20, 2024 · 862f7d3 · 862f7d3
1 parent 7aee250
commit 862f7d3
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/src/lexer/NestedComment.ts b/src/lexer/NestedComment.ts
@@ -2,7 +2,7 @@
 import { RegExpLike } from './TokenizerEngine.js';
 
 const START = /\/\*/uy; // matches: /*
-const MIDDLE = /([^/*]|\*[^/]|\/[^*])+/uy; // matches text NOT containing /* or */
+const ANY_CHAR = /[\s\S]/uy; // matches single character
 const END = /\*\//uy; // matches: */
 
 /**
@@ -31,7 +31,7 @@ export class NestedComment implements RegExpLike {
       } else if ((match = this.matchSection(END, input))) {
         result += match;
         nestLevel--;
-      } else if ((match = this.matchSection(MIDDLE, input))) {
+      } else if ((match = this.matchSection(ANY_CHAR, input))) {
         result += match;
       } else {
         return null;

diff --git a/test/features/comments.ts b/test/features/comments.ts
@@ -239,6 +239,12 @@ export default function supportsComments(format: FormatFn, opts: CommentsConfig
     `);
   });
 
+  // Regression test for #747
+  it('handles block comments with /** and **/ patterns', () => {
+    const sql = `/** This is a block comment **/`;
+    expect(format(sql)).toBe(sql);
+  });
+
   if (opts.hashComments) {
     it('supports # line comment', () => {
       const result = format('SELECT alpha # commment\nFROM beta');