Melhora o desempenho da RegEx que evita a criação de conteúdos invisíveis

[aprendendofelipe]

Mudanças realizadas

Substituído o agrupamento com operador | entre parênteses (\s|\p{C}|...) por colchetes [\s\p{C}...] para representar o conjunto de caracteres que devem ser removidos do início e final das strings.

Obs. Troquei a ordem de alguns caracteres porque, com a remoção do operador |, algumas sequências de caracteres entravam na regra ESLint no-misleading-character-class.

Resolve #1141 (comment)

Tipo de mudança

• Correção de bug

Checklist:

• As modificações não geram novos logs de erro ou aviso (warning).
• Os testes antigos estão passando localmente.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
tabnews	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Nov 29, 2024 4:26pm

[Rafatcb]

Show! Está bem melhor.

Como esses padrões se repetem algumas vezes (até em arquivos diferentes), acha interessante ter um arquivo exportando-os como constantes?

Edit: Acha válido criar um teste que não passaria por timeout (ou demoraria muito), mas agora passa normalmente? Isso poderia chamar a atenção de um eventual regresso do comportamento no futuro.

[aprendendofelipe]

Em produção! 🚀

Como esses padrões se repetem algumas vezes (até em arquivos diferentes), acha interessante ter um arquivo exportando-os como constantes?

Cheguei a pensar em criar uma função customizada pra passar pro Joi, mas tem pequenas diferenças em cada esquema, então achei que não valia a pena.

Acha válido criar um teste que não passaria por timeout (ou demoraria muito), mas agora passa normalmente? Isso poderia chamar a atenção de um eventual regresso do comportamento no futuro.

Não vale a pena, pois seria um teste muito instável, com o resultado dependendo muito do ambiente de execução.