Sunrun case study #5277
Open
Sunrun case study #5277
+63
−0
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
Contributor
|
Claude finished @juliettech13's task —— View job PR Review CompleteTodo List:
Score: 3/10 - Critical issues prevent mergeTwo critical issues must be fixed before merge: 1. 🚨 Future Customer Date -
|
Contributor
Greptile OverviewGreptile SummaryThis PR adds a new Sunrun case study to the marketing site, showcasing how Sunrun uses Helicone for caching, observability, and alerting in their AI operations. However, the PR contains two critical issues that need to be addressed:
The case study content itself is well-written and properly structured with customer quotes, feature highlights, and results. The integration into the case studies list is correct. Confidence Score: 1/5
Important Files ChangedFile Analysis
Sequence DiagramsequenceDiagram
participant Dev as Developer
participant Bifrost as Bifrost (Marketing)
participant FileSystem as File System
participant User as Website Visitor
Dev->>FileSystem: Create metadata.json
Dev->>FileSystem: Create src.mdx content
Dev->>FileSystem: Add sunrun.webp logo
Dev->>Bifrost: Update CaseStudies.tsx
Note over Bifrost: Add "sunrun" to case studies array
Dev->>FileSystem: Modify provider-helpers.ts
Note over FileSystem: Unintended: revert kimi-k2 mapping
User->>Bifrost: Visit /customers page
Bifrost->>FileSystem: Load case studies metadata
FileSystem-->>Bifrost: Return all case study data
Bifrost->>Bifrost: Format customerSince dates
Bifrost-->>User: Display case studies grid
Note over User: Sees Sunrun with incorrect<br/>future customer date
|
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
| "claude-3.5-sonnet-20240620": "claude-3.5-sonnet-v2", | ||
| "deepseek-r1": "deepseek-reasoner", | ||
| "kimi-k2": "kimi-k2-0905", | ||
| "kimi-k2": "kimi-k2-0711", |
Contributor
There was a problem hiding this comment.
logic: Unrelated change: reverts PR #5249 ("kimi-k2 go to latest") from Nov 14. This changes kimi-k2 mapping from kimi-k2-0905 back to kimi-k2-0711, which appears to be an accidental revert unrelated to the Sunrun case study.
Suggested change
| "kimi-k2": "kimi-k2-0711", | |
| "kimi-k2": "kimi-k2-0905", |
Prompt To Fix With AI
This is a comment left during a code review.
Path: packages/cost/models/provider-helpers.ts
Line: 346:346
Comment:
**logic:** Unrelated change: reverts PR #5249 ("kimi-k2 go to latest") from Nov 14. This changes `kimi-k2` mapping from `kimi-k2-0905` back to `kimi-k2-0711`, which appears to be an accidental revert unrelated to the Sunrun case study.
```suggestion
"kimi-k2": "kimi-k2-0905",
```
How can I resolve this? If you propose a fix, please make it concise.
colegottdank
approved these changes
Nov 22, 2025
juliettech13
force-pushed
the
sunrun-case-study
branch
from
217adf8 to
7e8e5cd
Compare
juliettech13
force-pushed
the
sunrun-case-study
branch
from
7e8e5cd to
b57b44c
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Ticket
[SAL-1179] - Sunrun Case Study
Component/Service
What part of Helicone does this affect?
Type of Change
Testing
Technical Considerations
Dependencies
Deployment Notes
Context
Why are you making this change?
Adding Sunrun case study for customer visibility.
Screenshots / Demos