Using the "comment current line" and "run" keyboard shortcuts in the SQL editor leads to invisible syntax errors

[BernieSumption]

Bug report

Describe the bug

A clear and concise description of what the bug is.

To Reproduce

1. Go to the SQL editor and enter this code:

select
  1 as a,
  2 as b;

2. highlight as on the second line and hit ⌘-/, the code becomes:

select
  -- 1 as a,
  2 as b;

Which is valid SQL.

3. hit ⌘-Enter to submit the query

Error message: Failed to validate sql query: syntax error at or near "as"

It would seem that the editor can get itself into a state where the SQL visible to the user is not the same SQL that gets executed on ⌘-Enter. Moving the cursor or clicking anywhere in the editor is sufficient to fix this state:

4. Press an arrow key to move the cursor, then hit ⌘-Enter again to submit the query.

The query is now successful.

Expected behavior

The executed SQL should always be the same as the visible SQL.

Screenshots

System information

• OS: macOS
• Browser: tested on Safari and Chrome
• this happens both on the current app.supabase.com and latest version of the supabase npm package

Additional context

As you can see in the above screenshot, highlighting the as word leads to two highlight markers on the screen, one over the letters and one a couple of lines below. I'm not sure if this is connected or a different issue.

[BernieSumption]

Note, you can achieve the same result by, in step 2 instead of hitting the "comment line" shortcut, you delete the word then ⌘-Z to undo it before submitting the query.

[BernieSumption]

Not sure why this was converted from an issue to a discussion, but I can confirm that the issue - #12086 - is now fixed.