Generated document `@throws` section missing `}`

[typed-sigterm]

Describe the bug

I found that generated @throws document section of @std/async.deadline is missing }:

Throws
DOMException & { name: "TimeoutError"
DOMException & { name: "AbortError"
AbortSignal["reason"]
If the optional signal is aborted with a custom reason before resolving or timing out.

Steps to Reproduce

https://jsr.io/@std/async@1.0.11/doc/deadline/~/deadline#throws

Expected behavior

-DOMException & { name: "TimeoutError"
-DOMException & { name: "AbortError"
+DOMException & { name: "TimeoutError" }
+DOMException & { name: "AbortError" }

This may not be the reason for @std/async. Since I'm not familiar with the architecture behind it, I'm only submitting it here.

Environment

N/A

[WWRS]

Changing a certain regex in deno_doc from ...\{([^}]+)\}... to ...\{(.+)\}... (allowing } between the braces) does fix this.

deno_doc/src/js_doc.rs

Line 27 in aef1632

static ref JS_DOC_TAG_WITH_MAYBE_TYPE_AND_MAYBE_VALUE_RE: Regex = Regex::new(r"(?s)^\s*@(returns?|throws|exception)(?:\s+\{([^}]+)\})?(?:\s+(.+))?").unwrap();

Unfortunately, that's not a permanent solution, since the performance of .+ has the potential to be much worse than [^}]+. Also it probably breaks if there are any more }s later in the string. I suspect the best solution would be to count braces, but that seems like a relatively large investment.

As a note, it's not just the } that's missing, it's everything after the }. Seen:

DOMException & { name: "TimeoutError"
DOMException & { name: "AbortError"

Expected:

DOMException & { name: "TimeoutError" }
If the provided duration runs out before resolving.
DOMException & { name: "AbortError" }