Skip to content

refactor: centralize character truncation 💚#108

Closed
kingrubic wants to merge 1 commit into
agentctxhq:mainfrom
kingrubic:fix/90-truncate-chars
Closed

refactor: centralize character truncation 💚#108
kingrubic wants to merge 1 commit into
agentctxhq:mainfrom
kingrubic:fix/90-truncate-chars

Conversation

@kingrubic

Copy link
Copy Markdown
Contributor

Summary

  • add one dependency-free truncateChars helper
  • route all four character-truncation sites through it
  • fix the pre-compact max+1 off-by-one and cover exact/overflow boundaries

Closes #90

Testing

  • npm run check (31 files, 300 tests passed)

AI assistance

AI tooling assisted with the implementation; I reviewed and understand the submitted changes.

@kingrubic

Copy link
Copy Markdown
Contributor Author

Closing this PR to tidy the backlog after more than 48 hours without maintainer feedback. Please feel free to reopen it if a maintainer finds the contribution useful. Thank you.

@kingrubic kingrubic closed this Jul 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Character-truncation helper (s.slice(0, max-1) + "…") is duplicated in 4 modules

1 participant