Merge pull request #22 from PureStackStudio/normalization-perf

Improve normalization performance and fix double entity decoding.

Author: koculu

package.json

@@ -1,7 +1,7 @@
 {
   "private": true,
   "name": "purestack-repo",
-  "version": "1.1.7",
+  "version": "1.1.8",
   "packageManager": "yarn@4.18.1",
   "type": "module",
   "sideEffects": false,

packages/purestack/package.json

@@ -1,6 +1,6 @@
 {
   "name": "purestack",
-  "version": "1.1.7",
+  "version": "1.1.8",
   "packageManager": "yarn@4.9.2",
   "module": "./dist/purestack.mjs",
   "types": "./dist/purestack.d.mts",

packages/purestack/src/index.ts

@@ -1,3 +1,3 @@
-export const version: string = '1.1.7'
+export const version: string = '1.1.8'
 
 export * from '@purestack/ts-ssg'

packages/ts-common/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@purestack/ts-common",
-  "version": "1.1.7",
+  "version": "1.1.8",
   "packageManager": "yarn@4.9.2",
   "module": "./dist/ts-common.mjs",
   "types": "./dist/ts-common.d.mts",

packages/ts-components/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@purestack/ts-components",
-  "version": "1.1.7",
+  "version": "1.1.8",
   "packageManager": "yarn@4.9.2",
   "module": "./dist/ts-components.mjs",
   "types": "./dist/ts-components.d.mts",

packages/ts-components/src/standard/composer/emailHtmlPolicy.test.ts

@@ -0,0 +1,30 @@
+import { describe, expect, it } from 'vitest'
+import { sanitizeEmailStyleValue } from './emailHtmlPolicy'
+
+describe('sanitizeEmailStyleValue', () => {
+  it.each([
+    [' red ', 'red'],
+    ['red!important', 'red'],
+    [' red \t!IMPORTANT \n', 'red'],
+    ['red !important extra', 'red !important extra'],
+    ['red ! important', 'red ! important'],
+    ['!important', undefined],
+  ])('normalizes %j to %j', (value, expected) => {
+    expect(sanitizeEmailStyleValue('color', value)).toBe(expected)
+  })
+
+  it('rejects oversized values containing long internal whitespace', () => {
+    const value = `red${' '.repeat(200_000)}x`
+    expect(sanitizeEmailStyleValue('color', value)).toBeUndefined()
+  })
+
+  it('applies the length limit after removing the important suffix', () => {
+    const value = 'a'.repeat(512)
+    expect(sanitizeEmailStyleValue('font-family', `${value} !important`)).toBe(
+      value,
+    )
+    expect(
+      sanitizeEmailStyleValue('font-family', `${value}a !important`),
+    ).toBeUndefined()
+  })
+})

packages/ts-components/src/standard/composer/emailHtmlPolicy.ts

@@ -432,7 +432,10 @@ export function normalizeEmailContentId(value: string) {
 }
 
 function stripImportant(value: string) {
-  return value.replace(/\s*!important\s*$/i, '').trim()
+  const trimmed = value.trim()
+  return trimmed.slice(-10).toLowerCase() === '!important'
+    ? trimmed.slice(0, -10).trimEnd()
+    : trimmed
 }
 
 function isSafeCssValue(value: string) {

packages/ts-css/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@purestack/ts-css",
-  "version": "1.1.7",
+  "version": "1.1.8",
   "packageManager": "yarn@4.9.2",
   "module": "./dist/ts-css.mjs",
   "types": "./dist/ts-css.d.mts",

packages/ts-css/src/hyphenize.test.ts

@@ -32,6 +32,14 @@ describe('hyphenizeCss', () => {
     expect(hyphenizeCss('ID')).toBe('id') // all uppercase, no hyphen
   })
 
+  it('handles long uppercase runs with and without an acronym boundary', () => {
+    const acronym = 'A'.repeat(200_000)
+    expect(hyphenizeCss(acronym)).toBe(acronym.toLowerCase())
+    expect(hyphenizeCss(`${acronym}Value`)).toBe(
+      `${acronym.toLowerCase()}-value`,
+    )
+  })
+
   it('preserves existing kebab-case / hyphens', () => {
     expect(hyphenizeCss('font-size')).toBe('font-size')
     expect(hyphenizeCss('-moz-appearance')).toBe('-moz-appearance')

packages/ts-css/src/hyphenize.ts

@@ -36,7 +36,7 @@ export function hyphenizeCss(prop: string): string {
   // 1. Split acronym boundaries like "XMLHttp" → "XML-Http"
   // 2. Then split camelCase boundaries like "fooBar" → "foo-Bar"
   const withSeparators = withOpera
-    .replace(/([A-Z]+)([A-Z][a-z])/g, '$1-$2')
+    .replace(/([A-Z])([A-Z][a-z])/g, '$1-$2')
     .replace(/([a-z0-9])([A-Z])/g, '$1-$2')
 
   return withSeparators.toLowerCase()