feat: Unkey-Next.js - Google Gemini Plant Identifier template [SIDE QUEST] #2607
Conversation
Khaan25
requested review from
perkinsjr,
chronark,
mcstepp and
MichaelUnkey
as code owners
|
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
📝 Walkthrough📝 WalkthroughWalkthroughThe pull request updates the document Changes
Assessment against linked issues
Possibly related issues
Possibly related PRs
Suggested labels
Suggested reviewers
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
@Khaan25 is attempting to deploy a commit to the Unkey Team on Vercel. A member of the Team first needs to authorize it. |
|
Thank you for following the naming conventions for pull request titles! 🙏 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Outside diff range and nitpick comments (1)
oss.gg/7_create_a_template.md (1)
43-43: Update framework name to use official spelling.
The framework name should be written as "Next.js" instead of "Nextjs" to maintain consistency with official branding.
Apply this change:
-» 29-October-2024 by Khaan25 [Google Gemini Plant Identifier Nextjs API with Unkey](https://github.com/Khaan25/ai-plant-detector)
+» 29-October-2024 by Khaan25 [Google Gemini Plant Identifier Next.js API with Unkey](https://github.com/Khaan25/ai-plant-detector)🧰 Tools
🪛 LanguageTool
[uncategorized] ~43-~43: The official spelling of this programming framework is “Next.js”.
Context: ...Khaan25 [Google Gemini Plant Identifier Nextjs API with Unkey](https://github.com/Khaa...
(NODE_JS)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (1)
- oss.gg/7_create_a_template.md (1 hunks)
🧰 Additional context used
🪛 LanguageTool
oss.gg/7_create_a_template.md
[uncategorized] ~43-~43: The official spelling of this programming framework is “Next.js”.
Context: ...Khaan25 [Google Gemini Plant Identifier Nextjs API with Unkey](https://github.com/Khaa...
(NODE_JS)
| » 29-October-2024 by Khaan25 [Google Gemini Plant Identifier Nextjs API with Unkey](https://github.com/Khaan25/ai-plant-detector) | ||
|
|
There was a problem hiding this comment.
💡 Codebase verification
Repository does not meet side quest requirements
The verification reveals several issues with the submitted repository:
- Missing LICENSE file which is a required component
- README.md contains only the default Next.js template content without any project-specific documentation
- No evidence of Google Gemini or Unkey integration as claimed in the title
- Repository appears to be a basic Next.js starter template without the promised functionality
🔗 Analysis chain
Verify repository meets side quest requirements.
Let's verify that the linked repository meets all the requirements specified in the side quest.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Verify the repository meets the side quest requirements
# Test 1: Check if repository exists and is public
echo "Checking repository accessibility..."
gh repo view Khaan25/ai-plant-detector
# Test 2: Check for README.md and LICENSE
echo "Checking for required files..."
gh api repos/Khaan25/ai-plant-detector/contents/README.md
gh api repos/Khaan25/ai-plant-detector/contents/LICENSE
# Test 3: Check if this is a unique template
echo "Checking for duplicate templates..."
rg -i "plant.*identifier|plant.*detect"
Length of output: 5076
🧰 Tools
🪛 LanguageTool
[uncategorized] ~43-~43: The official spelling of this programming framework is “Next.js”.
Context: ...Khaan25 [Google Gemini Plant Identifier Nextjs API with Unkey](https://github.com/Khaa...
(NODE_JS)
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (1)
oss.gg/7_create_a_template.md (1)
69-70: Use correct framework naming: "Next.js" instead of "Nextjs"
Please update the framework name to follow the official spelling:
-» 29-October-2024 by Khaan25 [Google Gemini Plant Identifier Nextjs API with Unkey](https://github.com/Khaan25/ai-plant-detector)
+» 29-October-2024 by Khaan25 [Google Gemini Plant Identifier Next.js API with Unkey](https://github.com/Khaan25/ai-plant-detector)🧰 Tools
🪛 LanguageTool
[uncategorized] ~69-~69: The official spelling of this programming framework is “Next.js”.
Context: ...Khaan25 [Google Gemini Plant Identifier Nextjs API with Unkey](https://github.com/Khaa...
(NODE_JS)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (1)
- oss.gg/7_create_a_template.md (2 hunks)
🧰 Additional context used
🪛 Markdownlint
oss.gg/7_create_a_template.md
49-49: null
Spaces inside link text
(MD039, no-space-in-links)
🪛 LanguageTool
oss.gg/7_create_a_template.md
[uncategorized] ~69-~69: The official spelling of this programming framework is “Next.js”.
Context: ...Khaan25 [Google Gemini Plant Identifier Nextjs API with Unkey](https://github.com/Khaa...
(NODE_JS)
🔇 Additional comments (1)
oss.gg/7_create_a_template.md (1)
69-70: Repository does not meet side quest requirements
The previously identified issues with the repository remain unaddressed. Please refer to the earlier review comment for details about the missing LICENSE file, insufficient README documentation, and lack of integration evidence.
Let's verify if the issues have been addressed:
#!/bin/bash
# Description: Verify repository requirements
# Check for required files and content
gh api repos/Khaan25/ai-plant-detector/contents/LICENSE || echo "LICENSE still missing"
gh api repos/Khaan25/ai-plant-detector/contents/README.md | jq -r '.content' | base64 -d | grep -i "gemini\|unkey" || echo "README still lacks integration details"
# Check for integration evidence
rg -l "gemini|unkey" || echo "No evidence of claimed integrations found"🧰 Tools
🪛 LanguageTool
[uncategorized] ~69-~69: The official spelling of this programming framework is “Next.js”.
Context: ...Khaan25 [Google Gemini Plant Identifier Nextjs API with Unkey](https://github.com/Khaa...
(NODE_JS)
|
please license it under MIT |
|
Done |
|
@chronark sorry to tag you again but today's last day so I'm trying to gather points |
|
@Khaan25,here is another NextJS template created by @harshsbhat. https://www.unkey.com/templates/cost-ratelimit It's key idea is to showcase how you can ratelimit your AI powered application based on estimated cost. I'd love to hear how yours is novel, apart from using another popular AI service? |
|
Let @chronark decide because we've to create a template on how we can use unkey with particular framework. In my case, Rate limiting AI powered Plant Identifier app in nextjs. We're protecting the API route, if someone wants to build similar AI project, this starter kit may help him out. |
|
Hey @Khaan25 , I am not the maintainer of this repo. Not sure about what's unique and how Andreas is planning to determine that. But I am pretty sure you need to add a repo-specific |
|
Yes, thank you for that. I've updated the readme |
What does this PR do?
Fixes #2150
Created a Google Gemini Plant Identifier app with Next.js 15 and protected the endpoint using Unkey. It uses Redis to store the the response returned from Gemini. We've a limit of 3 per 10 minutes. We've loading, error state, response output, image preview.
Link to the repo:
https://github.com/Khaan25/ai-plant-detector
Live link:
https://ai-plant-detector-zia-unkey.vercel.app/
Screenshots
Summary by CodeRabbit
New Features
Documentation