Merge pull request #8 from lupodevelop/fix/1.2.1-grapheme-wrap-repeat

Fix/1.2.1 grapheme wrap repeat

Author: lupodevelop

CHANGELOG.md

@@ -10,6 +10,20 @@ All notable changes to this project are documented in this file.
 - Optimized `count/3` internals to avoid repeated `list.length` calls inside recursive loops, improving performance on long strings.
 
 
+Contributed by: Daniele (`lupodevelop`)
+
+## [1.2.1] - 2026-01-02
+### Fixed
+- Made `repeat_str/2` iterative to avoid deep recursion and improve performance on large repetition counts.
+- Fixed `wrap_at/2` to measure word lengths using grapheme clusters to avoid splitting emoji and other multi-codepoint graphemes.
+- Replaced several `string.length(part) == 0` checks with `string.is_empty(part)` for clarity.
+
+### Tests
+- Added `wrap_at_emoji_grapheme_test` to verify grapheme-aware wrapping behavior.
+
+### Style
+- Ran `gleam format` to normalize formatting.
+
 Contributed by: Daniele (`lupodevelop`)
 
 ## [1.1.1] - 2025-11-30

gleam.toml

@@ -1,5 +1,5 @@
 name = "str"
-version = "1.2.0"
+version = "1.2.1"
 
 # Project metadata (fill or replace placeholders before publishing)
 description = "Unicode-aware string utilities for Gleam: grapheme-safe operations, pragmatic ASCII transliteration, and slug generation."

src/str/core.gleam

@@ -179,7 +179,7 @@ pub fn is_blank(text: String) -> Bool {
 fn repeat_str(s: String, n: Int) -> String {
   case n <= 0 {
     True -> ""
-    False -> s <> repeat_str(s, n - 1)
+    False -> list.fold(list.range(1, n), "", fn(acc, _) { acc <> s })
   }
 }
 
@@ -693,7 +693,7 @@ fn wrap_words(
   case remaining {
     [] -> acc
     [word, ..rest] -> {
-      let word_len = string.length(word)
+      let word_len = string.to_graphemes(word) |> list.length
       case line_len == 0 {
         True ->
           // First word on line

src/str/extra.gleam

@@ -341,7 +341,7 @@ pub fn to_snake_case(s: String) -> String {
 pub fn to_camel_case(s: String) -> String {
   let parts = string.split(slugify(s), "-")
   list.fold(parts, "", fn(acc, part) {
-    case string.length(part) == 0 {
+    case string.is_empty(part) {
       True -> acc
       False ->
         case acc == "" {
@@ -364,7 +364,7 @@ pub fn to_camel_case(s: String) -> String {
 pub fn to_pascal_case(s: String) -> String {
   let parts = string.split(slugify(s), "-")
   list.fold(parts, "", fn(acc, part) {
-    case string.length(part) == 0 {
+    case string.is_empty(part) {
       True -> acc
       False ->
         acc
@@ -385,7 +385,7 @@ pub fn to_title_case(s: String) -> String {
   let parts = string.split(slugify(s), "-")
   let capitalized =
     list.map(parts, fn(part) {
-      case string.length(part) == 0 {
+      case string.is_empty(part) {
         True -> part
         False ->
           string.uppercase(string.slice(part, 0, 1))

test/str_core_test.gleam

@@ -388,6 +388,13 @@ pub fn wrap_at_zero_width_test() {
   assert core.wrap_at("hello", 0) == "hello"
 }
 
+pub fn wrap_at_emoji_grapheme_test() {
+  // Ensure grapheme-aware wrapping treats emoji as single units
+  let s = "a 👨‍👩‍👧‍👦 b"
+  // Width 2 should force a newline between "a" and the emoji
+  assert string.contains(core.wrap_at(s, 2), "\n")
+}
+
 pub fn ellipsis_basic_test() {
   let result = core.ellipsis("Hello World", 8)
   assert string.ends_with(result, "…")