Apply suggestions from code review

donbarbos · srittau · web-flow · commit 039da9dc4364 · 2026-01-26T16:54:56.000Z
add comments to regex

Co-authored-by: Sebastian Rittau &lt;sebastian.rittau@zfutura.de&gt;
diff --git a/lib/ts_utils/utils.py b/lib/ts_utils/utils.py
@@ -30,17 +30,17 @@ def colored(text: str, color: str | None = None, **kwargs: Any) -> str:  # type:
 
 _REMOVE_COMMENT_RE = re.compile(
     r"""
-    (\"(?:\\.|[^\\\"])*?\")
+    (\"(?:\\.|[^\\\"])*?\")  # matches literal strings
     |
-    (\/\*.*?\*\/ | \/\/[^\r\n]*?(?:[\r\n]))
+    (\/\*.*?\*\/ | \/\/[^\r\n]*?(?:[\r\n]))  # matches single- and multi-line comments
     """,
     re.DOTALL | re.VERBOSE,
 )
 _REMOVE_TRAILING_COMMA_RE = re.compile(
     r"""
-    (\"(?:\\.|[^\\\"])*?\")
+    (\"(?:\\.|[^\\\"])*?\")  # matches literal strings
     |
-    ,\s*([\]}])
+    ,\s*([\]}])  # matches commas before '}' or ']'
     """,
     re.DOTALL | re.VERBOSE,
 )

Original file line number	Diff line number	Diff line change
`@@ -30,17 +30,17 @@ def colored(text: str, color: str \| None = None, **kwargs: Any) -> str: # type:`
`30`	`30`
`31`	`31`	`_REMOVE_COMMENT_RE = re.compile(`
`32`	`32`	`r"""`
`33`		`- (\"(?:\\.\|[^\\\"])*?\")`
	`33`	`+ (\"(?:\\.\|[^\\\"])*?\") # matches literal strings`
`34`	`34`	`\|`
`35`		`- (\/\.?\\/ \| \/\/[^\r\n]?(?:[\r\n]))`
	`35`	`+ (\/\.?\\/ \| \/\/[^\r\n]?(?:[\r\n])) # matches single- and multi-line comments`
`36`	`36`	`""",`
`37`	`37`	`re.DOTALL \| re.VERBOSE,`
`38`	`38`	`)`
`39`	`39`	`_REMOVE_TRAILING_COMMA_RE = re.compile(`
`40`	`40`	`r"""`
`41`		`- (\"(?:\\.\|[^\\\"])*?\")`
	`41`	`+ (\"(?:\\.\|[^\\\"])*?\") # matches literal strings`
`42`	`42`	`\|`
`43`		`- ,\s*([\]}])`
	`43`	`+ ,\s*([\]}]) # matches commas before '}' or ']'`
`44`	`44`	`""",`
`45`	`45`	`re.DOTALL \| re.VERBOSE,`
`46`	`46`	`)`