Skip to content

test(handlers): add unit test suite for Helicone custom property header tagging - #5812

Open
gcoinstash-cmd wants to merge 1 commit into
Helicone:mainfrom
gcoinstash-cmd:test/provider-model-header-tagging
Open

test(handlers): add unit test suite for Helicone custom property header tagging#5812
gcoinstash-cmd wants to merge 1 commit into
Helicone:mainfrom
gcoinstash-cmd:test/provider-model-header-tagging

Conversation

@gcoinstash-cmd

Copy link
Copy Markdown

Summary

Adds focused unit tests for Helicone custom property header extraction (helicone-property-*):

  • Case-insensitive prefix extraction
  • Non-property header exclusion
  • Value whitespace trimming

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your trial has ended. Reactivate Greptile to resume code reviews.

@vercel

vercel Bot commented Sep 13, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

3 Skipped Deployments
Project Deployment Actions Updated
helicone Skipped Skipped Sep 13, 2026 2:40am UTC
helicone-bifrost Skipped Skipped Sep 13, 2026 2:40am UTC
helicone-eu Skipped Skipped Sep 13, 2026 2:40am UTC

Request Review

@vercel
vercel Bot temporarily deployed to Preview – helicone-bifrost September 13, 2026 02:40 Inactive
@vercel
vercel Bot temporarily deployed to Preview – helicone-eu September 13, 2026 02:40 Inactive
@vercel
vercel Bot temporarily deployed to Preview – helicone September 13, 2026 02:40 Inactive
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.

1 participant