Skip to content

fix(vibenet): split acronym boundaries in humanizeKey - #70

Open
omerbek wants to merge 1 commit into
base:mainfrom
omerbek:fix/humanize-key-acronyms
Open

fix(vibenet): split acronym boundaries in humanizeKey#70
omerbek wants to merge 1 commit into
base:mainfrom
omerbek:fix/humanize-key-acronyms

Conversation

@omerbek

@omerbek omerbek commented Aug 16, 2026

Copy link
Copy Markdown

Summary

  • split acronym-to-word boundaries before the existing camelCase pass in humanizeKey
  • add regression tests for USDC/API/WSTETH-style keys
  • keep existing fallback behavior for ordinary camelCase and separator-delimited keys

Fixes #69.

Testing

  • npx vitest run app/vibenet/library/format.test.ts
  • npm run typecheck
  • npm run lint
  • git diff --check

Note: local install reports existing npm audit findings and Node engine warnings because this machine runs Node 21.6.1 while the repo declares Node 24.x. Lint passed with existing img-element warnings outside the touched files.

@vercel

vercel Bot commented Aug 16, 2026

Copy link
Copy Markdown

@omerbek is attempting to deploy a commit to the Coinbase Team on Vercel.

A member of the Team first needs to authorize it.

@cb-heimdall

Copy link
Copy Markdown
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/2
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 1
Sum 2

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.

humanizeKey glues acronyms to the next word in Vibenet fallback labels

2 participants