Update string interpolation highlighting

kiendang · kiendang · commit 19068130bf87 · 2020-01-27T12:46:47.000+08:00
Add scopes `meta.template.expression.scala` and `meta.embedded.line.scala`
Eg. In `s"${val a = 4; foo(a)}"`, `${val a = 4; foo(a)}` will have `meta.template.expression.scala` while `val a = 4; foo(a)` will have `meta.embedded.line.scala`.
Earlier `val a = 4; foo(a)` was under `punctuation.definition.template-expression`. This makes things without a scope like `; foo(a)` highlighted differently inside string interpolation (they will have the same highlighting as `${` and `}`) vs outside, which is not desireable.
diff --git a/src/typescript/Scala.tmLanguage.ts b/src/typescript/Scala.tmLanguage.ts
@@ -327,6 +327,7 @@ export const scalaTmLanguage: TmLanguage = {
           match: "\\$\\$"
         },
         {
+          name: "meta.template.expression.scala",
           match: `(\\$)(${simpleInterpolatedVariable})`,
           captures: {
             "1": {
@@ -335,7 +336,7 @@ export const scalaTmLanguage: TmLanguage = {
           }
         },
         {
-            name: "punctuation.definition.template-expression.scala",
+            name: "meta.template.expression.scala",
             begin: "\\$\\{",
             beginCaptures: { "0": { name: "punctuation.definition.template-expression.begin.scala" } },
             end: "\\}",
@@ -344,7 +345,8 @@ export const scalaTmLanguage: TmLanguage = {
                 {
                     include: "#code"
                 }
-            ]
+            ],
+            contentName: "meta.embedded.line.scala"
         }
       ]
     },
diff --git a/syntaxes/Scala.tmLanguage.json b/syntaxes/Scala.tmLanguage.json
diff --git a/tests/snap/lexical.test.scala.snap b/tests/snap/lexical.test.scala.snap
@@ -279,90 +279,90 @@
 #^^ source.scala
 #  ^ source.scala keyword.interpolation.scala
 #   ^ source.scala string.quoted.double.interpolated.scala punctuation.definition.string.begin.scala
-#    ^ source.scala punctuation.definition.template-expression.begin.scala
-#     ^ source.scala
+#    ^ source.scala meta.template.expression.scala punctuation.definition.template-expression.begin.scala
+#     ^ source.scala meta.template.expression.scala
 #      ^ source.scala string.quoted.double.interpolated.scala
 #       ^ source.scala string.quoted.double.interpolated.scala
 #        ^ source.scala string.quoted.double.interpolated.scala
 #         ^ source.scala string.quoted.double.interpolated.scala
 #          ^ source.scala string.quoted.double.interpolated.scala
 #           ^ source.scala string.quoted.double.interpolated.scala
 #            ^ source.scala string.quoted.double.interpolated.scala
-#             ^^ source.scala punctuation.definition.template-expression.scala punctuation.definition.template-expression.begin.scala
-#               ^^^ source.scala punctuation.definition.template-expression.scala keyword.declaration.stable.scala
-#                  ^ source.scala punctuation.definition.template-expression.scala
-#                   ^ source.scala punctuation.definition.template-expression.scala variable.other.declaration.scala
-#                    ^ source.scala punctuation.definition.template-expression.scala
-#                     ^ source.scala punctuation.definition.template-expression.scala keyword.operator.comparison.scala
-#                      ^^ source.scala punctuation.definition.template-expression.scala
-#                        ^ source.scala punctuation.definition.template-expression.scala punctuation.definition.template-expression.end.scala
+#             ^^ source.scala meta.template.expression.scala punctuation.definition.template-expression.begin.scala
+#               ^^^ source.scala meta.template.expression.scala meta.embedded.line.scala keyword.declaration.stable.scala
+#                  ^ source.scala meta.template.expression.scala meta.embedded.line.scala
+#                   ^ source.scala meta.template.expression.scala meta.embedded.line.scala variable.other.declaration.scala
+#                    ^ source.scala meta.template.expression.scala meta.embedded.line.scala
+#                     ^ source.scala meta.template.expression.scala meta.embedded.line.scala keyword.operator.comparison.scala
+#                      ^^ source.scala meta.template.expression.scala meta.embedded.line.scala
+#                        ^ source.scala meta.template.expression.scala punctuation.definition.template-expression.end.scala
 #                         ^ source.scala string.quoted.double.interpolated.scala punctuation.definition.string.end.scala
 >  custom"$x plain ${val x = y}"
 #^^ source.scala
 #  ^^^^^^ source.scala keyword.interpolation.scala
 #        ^ source.scala string.quoted.double.interpolated.scala punctuation.definition.string.begin.scala
-#         ^ source.scala punctuation.definition.template-expression.begin.scala
-#          ^ source.scala
+#         ^ source.scala meta.template.expression.scala punctuation.definition.template-expression.begin.scala
+#          ^ source.scala meta.template.expression.scala
 #           ^ source.scala string.quoted.double.interpolated.scala
 #            ^ source.scala string.quoted.double.interpolated.scala
 #             ^ source.scala string.quoted.double.interpolated.scala
 #              ^ source.scala string.quoted.double.interpolated.scala
 #               ^ source.scala string.quoted.double.interpolated.scala
 #                ^ source.scala string.quoted.double.interpolated.scala
 #                 ^ source.scala string.quoted.double.interpolated.scala
-#                  ^^ source.scala punctuation.definition.template-expression.scala punctuation.definition.template-expression.begin.scala
-#                    ^^^ source.scala punctuation.definition.template-expression.scala keyword.declaration.stable.scala
-#                       ^ source.scala punctuation.definition.template-expression.scala
-#                        ^ source.scala punctuation.definition.template-expression.scala variable.other.declaration.scala
-#                         ^ source.scala punctuation.definition.template-expression.scala
-#                          ^ source.scala punctuation.definition.template-expression.scala keyword.operator.comparison.scala
-#                           ^^ source.scala punctuation.definition.template-expression.scala
-#                             ^ source.scala punctuation.definition.template-expression.scala punctuation.definition.template-expression.end.scala
+#                  ^^ source.scala meta.template.expression.scala punctuation.definition.template-expression.begin.scala
+#                    ^^^ source.scala meta.template.expression.scala meta.embedded.line.scala keyword.declaration.stable.scala
+#                       ^ source.scala meta.template.expression.scala meta.embedded.line.scala
+#                        ^ source.scala meta.template.expression.scala meta.embedded.line.scala variable.other.declaration.scala
+#                         ^ source.scala meta.template.expression.scala meta.embedded.line.scala
+#                          ^ source.scala meta.template.expression.scala meta.embedded.line.scala keyword.operator.comparison.scala
+#                           ^^ source.scala meta.template.expression.scala meta.embedded.line.scala
+#                             ^ source.scala meta.template.expression.scala punctuation.definition.template-expression.end.scala
 #                              ^ source.scala string.quoted.double.interpolated.scala punctuation.definition.string.end.scala
 >
 >  s"""$x plain ${val x = y}"""
 #^^ source.scala
 #  ^ source.scala keyword.interpolation.scala
 #   ^^^ source.scala string.quoted.triple.interpolated.scala punctuation.definition.string.begin.scala
-#      ^ source.scala punctuation.definition.template-expression.begin.scala
-#       ^ source.scala
+#      ^ source.scala meta.template.expression.scala punctuation.definition.template-expression.begin.scala
+#       ^ source.scala meta.template.expression.scala
 #        ^ source.scala string.quoted.triple.interpolated.scala
 #         ^ source.scala string.quoted.triple.interpolated.scala
 #          ^ source.scala string.quoted.triple.interpolated.scala
 #           ^ source.scala string.quoted.triple.interpolated.scala
 #            ^ source.scala string.quoted.triple.interpolated.scala
 #             ^ source.scala string.quoted.triple.interpolated.scala
 #              ^ source.scala string.quoted.triple.interpolated.scala
-#               ^^ source.scala punctuation.definition.template-expression.scala punctuation.definition.template-expression.begin.scala
-#                 ^^^ source.scala punctuation.definition.template-expression.scala keyword.declaration.stable.scala
-#                    ^ source.scala punctuation.definition.template-expression.scala
-#                     ^ source.scala punctuation.definition.template-expression.scala variable.other.declaration.scala
-#                      ^ source.scala punctuation.definition.template-expression.scala
-#                       ^ source.scala punctuation.definition.template-expression.scala keyword.operator.comparison.scala
-#                        ^^ source.scala punctuation.definition.template-expression.scala
-#                          ^ source.scala punctuation.definition.template-expression.scala punctuation.definition.template-expression.end.scala
+#               ^^ source.scala meta.template.expression.scala punctuation.definition.template-expression.begin.scala
+#                 ^^^ source.scala meta.template.expression.scala meta.embedded.line.scala keyword.declaration.stable.scala
+#                    ^ source.scala meta.template.expression.scala meta.embedded.line.scala
+#                     ^ source.scala meta.template.expression.scala meta.embedded.line.scala variable.other.declaration.scala
+#                      ^ source.scala meta.template.expression.scala meta.embedded.line.scala
+#                       ^ source.scala meta.template.expression.scala meta.embedded.line.scala keyword.operator.comparison.scala
+#                        ^^ source.scala meta.template.expression.scala meta.embedded.line.scala
+#                          ^ source.scala meta.template.expression.scala punctuation.definition.template-expression.end.scala
 #                           ^^^ source.scala string.quoted.triple.interpolated.scala punctuation.definition.string.end.scala
 >  custom"""$x plain ${val x = y}"""
 #^^ source.scala
 #  ^^^^^^ source.scala keyword.interpolation.scala
 #        ^^^ source.scala string.quoted.triple.interpolated.scala punctuation.definition.string.begin.scala
-#           ^ source.scala punctuation.definition.template-expression.begin.scala
-#            ^ source.scala
+#           ^ source.scala meta.template.expression.scala punctuation.definition.template-expression.begin.scala
+#            ^ source.scala meta.template.expression.scala
 #             ^ source.scala string.quoted.triple.interpolated.scala
 #              ^ source.scala string.quoted.triple.interpolated.scala
 #               ^ source.scala string.quoted.triple.interpolated.scala
 #                ^ source.scala string.quoted.triple.interpolated.scala
 #                 ^ source.scala string.quoted.triple.interpolated.scala
 #                  ^ source.scala string.quoted.triple.interpolated.scala
 #                   ^ source.scala string.quoted.triple.interpolated.scala
-#                    ^^ source.scala punctuation.definition.template-expression.scala punctuation.definition.template-expression.begin.scala
-#                      ^^^ source.scala punctuation.definition.template-expression.scala keyword.declaration.stable.scala
-#                         ^ source.scala punctuation.definition.template-expression.scala
-#                          ^ source.scala punctuation.definition.template-expression.scala variable.other.declaration.scala
-#                           ^ source.scala punctuation.definition.template-expression.scala
-#                            ^ source.scala punctuation.definition.template-expression.scala keyword.operator.comparison.scala
-#                             ^^ source.scala punctuation.definition.template-expression.scala
-#                               ^ source.scala punctuation.definition.template-expression.scala punctuation.definition.template-expression.end.scala
+#                    ^^ source.scala meta.template.expression.scala punctuation.definition.template-expression.begin.scala
+#                      ^^^ source.scala meta.template.expression.scala meta.embedded.line.scala keyword.declaration.stable.scala
+#                         ^ source.scala meta.template.expression.scala meta.embedded.line.scala
+#                          ^ source.scala meta.template.expression.scala meta.embedded.line.scala variable.other.declaration.scala
+#                           ^ source.scala meta.template.expression.scala meta.embedded.line.scala
+#                            ^ source.scala meta.template.expression.scala meta.embedded.line.scala keyword.operator.comparison.scala
+#                             ^^ source.scala meta.template.expression.scala meta.embedded.line.scala
+#                               ^ source.scala meta.template.expression.scala punctuation.definition.template-expression.end.scala
 #                                ^^^ source.scala string.quoted.triple.interpolated.scala punctuation.definition.string.end.scala
 >}
 #^ source.scala punctuation.section.block.end.scala
diff --git a/tests/unit/#71.test.scala b/tests/unit/#71.test.scala
@@ -2,12 +2,13 @@
 
    s"1 + 2 = ${ 1 + { val x = 2; x } }."
 // ^ keyword.interpolation.scala
+//           ^^^^^^^^^^^^^^^^^^^^^^^^^ meta.template.expression.scala
 //  ^ punctuation.definition.string.begin.scala
 //   ^^^^^^^^ string.quoted.double.interpolated.scala
 //           ^^ punctuation.definition.template-expression.begin.scala
 //                  ^ punctuation.section.block.begin.scala
 //                                 ^ punctuation.section.block.end.scala
-//             ^^^^^^^^^^^^^^^^^^^^^^ punctuation.definition.template-expression.scala
+//             ^^^^^^^^^^^^^^^^^^^^^^ meta.embedded.line.scala
 //                                   ^ punctuation.definition.template-expression.end.scala
 //                                    ^ string.quoted.double.interpolated.scala
 //                                     ^ punctuation.definition.string.end.scala
@@ -16,18 +17,17 @@
 // ^ keyword.interpolation.scala
 //  ^^^ punctuation.definition.string.begin.scala
 //     ^^^^^^^^ string.quoted.triple.interpolated.scala
-//             ^^ punctuation.definition.template-expression.begin.scala
+//             ^^ meta.template.expression.scala punctuation.definition.template-expression.begin.scala
      def add(x: Int, y: Int) = {
-//   ^^^^^^^^^^^^^^^^^^^^^^^^^^^ punctuation.definition.template-expression.scala
+//   ^^^^^^^^^^^^^^^^^^^^^^^^^^^ meta.template.expression.scala meta.embedded.line.scala
 //                             ^ punctuation.section.block.begin.scala
        x + y
-//     ^^^^^ punctuation.definition.template-expression.scala
+//     ^^^^^ meta.template.expression.scala meta.embedded.line.scala
      }
-//   ^ punctuation.section.block.end.scala
-//   ^ punctuation.definition.template-expression.scala
+//   ^ meta.template.expression.scala meta.embedded.line.scala punctuation.section.block.end.scala
      add(1, 2)
-//   ^^^^^^^^^ punctuation.definition.template-expression.scala
+//   ^^^^^^^^^ meta.template.expression.scala meta.embedded.line.scala
    }."""
-// ^ punctuation.definition.template-expression.end.scala
+// ^ meta.template.expression.scala punctuation.definition.template-expression.end.scala
 //  ^ string.quoted.triple.interpolated.scala
 //   ^^^ punctuation.definition.string.end.scala
diff --git a/tests/unit/#72.test.scala b/tests/unit/#72.test.scala
@@ -2,25 +2,42 @@
 
    s"$first$second"
 // ^ keyword.interpolation.scala
+//   ^^^^^^^^^^^^^ meta.template.expression.scala
 //  ^ punctuation.definition.string.begin.scala
 //   ^ punctuation.definition.template-expression.begin.scala
 //         ^ punctuation.definition.template-expression.begin.scala
 //                ^ punctuation.definition.string.end.scala
 
    s"$safeTagMarker${mtch.matched}$safeTagMarker"
 // ^ keyword.interpolation.scala
+//   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ meta.template.expression.scala
 //  ^ punctuation.definition.string.begin.scala
 //   ^ punctuation.definition.template-expression.begin.scala
 //                 ^^ punctuation.definition.template-expression.begin.scala
-//                   ^^^^^^^^^^^^ punctuation.definition.template-expression.scala
+//                   ^^^^^^^^^^^^ meta.embedded.line.scala
 //                               ^ punctuation.definition.template-expression.end.scala
 //                                ^ punctuation.definition.template-expression.begin.scala
 //                                              ^ punctuation.definition.string.end.scala
 
    s"${x$}"
 // ^ keyword.interpolation.scala
+//   ^^^^^ meta.template.expression.scala
 //  ^ punctuation.definition.string.begin.scala
 //   ^^ punctuation.definition.template-expression.begin.scala
-//     ^^ punctuation.definition.template-expression.scala
+//     ^^ meta.embedded.line.scala
 //       ^ punctuation.definition.template-expression.end.scala
 //        ^ punctuation.definition.string.end.scala
+
+   val a = 4; foo(a)
+// ^^^ keyword.declaration.stable.scala
+//     ^ variable.other.declaration.scala
+//       ^ keyword.operator.comparison.scala
+//         ^ constant.numeric.scala
+   s"$safeTagMarker${val a = 4; foo(a)}$safeTagMarker"
+//                 ^^^^^^^^^^^^^^^^^^^^ meta.template.expression.scala
+//                   ^^^^^^^^^^^^^^^^^ meta.embedded.line.scala
+//                   ^^^^^^^^^^^^^^^^^ - punctuation.definition.template-expression
+//                   ^^^ keyword.declaration.stable.scala
+//                       ^ variable.other.declaration.scala
+//                         ^ keyword.operator.comparison.scala
+//                           ^ constant.numeric.scala
diff --git a/tests/unit/lexical.test.scala b/tests/unit/lexical.test.scala

Original file line number	Diff line number	Diff line change
`@@ -327,6 +327,7 @@ export const scalaTmLanguage: TmLanguage = {`
`327`	`327`	`match: "\\$\\$"`
`328`	`328`	`},`
`329`	`329`	`{`
	`330`	`+ name: "meta.template.expression.scala",`
`330`	`331`	match: `(\\$)(${simpleInterpolatedVariable})`,
`331`	`332`	`captures: {`
`332`	`333`	`"1": {`
`@@ -335,7 +336,7 @@ export const scalaTmLanguage: TmLanguage = {`
`335`	`336`	`}`
`336`	`337`	`},`
`337`	`338`	`{`
`338`		`- name: "punctuation.definition.template-expression.scala",`
	`339`	`+ name: "meta.template.expression.scala",`
`339`	`340`	`begin: "\\$\\{",`
`340`	`341`	`beginCaptures: { "0": { name: "punctuation.definition.template-expression.begin.scala" } },`
`341`	`342`	`end: "\\}",`
`@@ -344,7 +345,8 @@ export const scalaTmLanguage: TmLanguage = {`
`344`	`345`	`{`
`345`	`346`	`include: "#code"`
`346`	`347`	`}`
`347`		`- ]`
	`348`	`+ ],`
	`349`	`+ contentName: "meta.embedded.line.scala"`
`348`	`350`	`}`
`349`	`351`	`]`
`350`	`352`	`},`