The changes are clean and focused. Let me verify if the README.md / AGENTS.md documentation references the macro catalog:

The JSG AGENTS.md code review rule says I should check if README.md needs updating when adding macros. The new JSG_TS_DEFINE_LITERAL is not in the TypeScript Generation macro table in README.md (line 154). This is a documentation gap, though minor.

Let me now assess:

Technical correctness: The macro is correct. It properly creates a static const char[] from the string literal and passes it as a non-type template parameter, matching the pattern of JSG_TS_DEFINE.

Potential issues:

1. The README.md macro table doesn't include the new macro - this is a minor documentation gap per JSG AGENTS.md review rules.
2. No other issues - the approach is sound, the comment explains the rationale, and the whitespace is handled by .trim() downstream.

This is a clean, well-motivated change. The only actionable item is the missing documentation entry.

LGTM

github run