feat: Add receipt-budget-tracker kit (agentkit-challenge) - #217
feat: Add receipt-budget-tracker kit (agentkit-challenge)#217Palash-oss wants to merge 6 commits into
Conversation
|
Warning Review limit reached
Next review available in: 58 minutes You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Repository UI (base), Organization UI (inherited) Review profile: ASSERTIVE Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (7)
📒 Files selected for processing (32)
WalkthroughChangesReceipt Budget Tracker
Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
:robot_face: AgentKit Structural ValidationNew Contributions Detected
Check Results
🎉 All checks passed! This contribution follows the AgentKit structure. |
|
Failure recorded at 2026-07-11T07:49:58Z UTC. If this PR is not fixed within 4 weeks it will be automatically closed. |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 18
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@kits/receipt-budget-tracker/agent.md`:
- Line 3: Replace the TODO in the kit-level agent.md with complete documentation
covering the agent’s identity, purpose and capabilities, receipt-processing
flow, operating guardrails, and Lamatic integration reference. Ensure the
document satisfies the required agent overview and explains the supplied flow
and dashboard behavior without leaving the placeholder.
In `@kits/receipt-budget-tracker/apps/.gitignore`:
- Around line 33-35: Update the .gitignore env-file rules so real .env files
remain ignored while .env.example is explicitly unignored, allowing the required
configuration template to be committed.
In `@kits/receipt-budget-tracker/apps/actions/orchestrate.ts`:
- Line 101: Remove or gate the debug console logs in the receipt orchestration
flow, including the OCR output near extractedText and the full Lamatic response
and checkStatus result. Use the existing debug flag if available, ensuring
production execution never logs receipt contents, full API responses, or status
payloads.
- Around line 155-190: The polling loop in the orchestration flow can exceed
serverless execution limits. Update the retry behavior around isCompleted,
retries, and maxRetries to complete within the supported function timeout by
reducing the maximum polling duration, or replace server-side polling with a
client-side/asynchronous status-check pattern; preserve successful and failed
status handling.
- Around line 118-123: Remove the hardcoded fallback Lamatic URL and project ID
from the fetch request in the orchestration flow. Require the endpoint and
project ID from their environment/configuration values, fail clearly when they
are missing, and replace the repeated default project ID usages near the later
request logic without introducing committed infrastructure values.
- Line 3: Remove the unused lamatic import from orchestrate and import
../../lamatic.config directly. Use lamatic.config’s step definitions to obtain
the flow ID instead of hardcoding or bypassing the configured value, while
preserving the existing raw fetch API calls.
- Around line 145-149: Update the executeWorkflow mutation selection used by
orchestrateWorkflow to request requestId alongside status and result. Preserve
the existing async detection in the rawResponse.requestId branch so long-running
workflows enter polling correctly.
In `@kits/receipt-budget-tracker/apps/CLAUDE.md`:
- Line 1: Add a top-level Markdown heading before the existing `@AGENTS.md`
include in CLAUDE.md, preserving the include unchanged and ensuring the file
satisfies markdownlint MD041.
In `@kits/receipt-budget-tracker/apps/components/Dashboard.tsx`:
- Around line 182-566: Update the Dashboard component to use the kit’s shadcn/ui
primitives instead of raw Tailwind-styled controls: replace dashboard panels
with Card, interactive demo/delete controls with Button, category/status pills
with Badge, and the budget bar with Progress. Initialize shadcn/ui if the shared
components are unavailable, while preserving the existing layout, styling
intent, handlers, and displayed behavior.
- Around line 339-384: Update the upload-zone div surrounding fileInputRef and
triggerFileSelect to expose an appropriate interactive role, make it keyboard
focusable, and handle Enter and Space key presses by invoking triggerFileSelect.
Preserve the existing click, drag-and-drop, and disabled-upload behavior.
- Line 67: Update processFile to validate file.size before sending the file to
the server action, rejecting files larger than the advertised 5MB limit and
providing user-facing feedback through the existing error-handling path.
Preserve the current upload flow for files within the limit.
- Around line 67-100: Update processFile to prepend each new receipt via a
functional setHistory updater, ensuring concurrent uploads preserve all items
instead of using the captured history closure. Move localStorage synchronization
into a useEffect that watches history, and update handleDelete to use the same
functional state-update pattern rather than reading stale history.
In `@kits/receipt-budget-tracker/apps/README.md`:
- Line 1: Add a top-level Markdown heading at the beginning of the app README,
before the existing create-next-app prose, using a descriptive title such as
“Receipt Budget Tracker App” to satisfy the document-heading requirement.
In `@kits/receipt-budget-tracker/apps/test-lamatic.js`:
- Around line 2-4: Remove the hardcoded credentials from the constants in
test-lamatic.js and load the API key, project ID, and flow ID from environment
variables with clear missing-value validation. Revoke and rotate the exposed
Lamatic API key, then prevent this test file or equivalent credential-bearing
files from being committed by updating the repository ignore configuration or
relocating the file.
In `@kits/receipt-budget-tracker/constitutions/default.md`:
- Around line 3-4: Update the template or generator that produces the
constitution content, ensuring every Markdown section heading is followed by a
blank line; regenerate default.md so all headings, including Identity and the
sections identified by the additional ranges, comply with MD022 without manually
patching only the checked-in output.
In `@kits/receipt-budget-tracker/flows/receipt-budget-tracker.ts`:
- Line 83: Update the InstructorLLMNode schema to expose category and items as
structured top-level fields instead of packing them into the string data field,
then update orchestrate.ts to consume those fields directly and remove the
fragile JSON.parse(rawData) path while preserving date handling.
In
`@kits/receipt-budget-tracker/prompts/receipt-budget-tracker_instructor-llmnode-575_system_0.md`:
- Line 1: Update the receipt extraction schema used by the
receipt-budget-tracker flow to include every field requested by the
prompt—vendor, transaction date, line items with prices, total amount, and
budget category—using the exact dashboard field names. Update downstream
normalization to retain and propagate all of these fields instead of limiting
output to vendor, data, and total.
In `@kits/receipt-budget-tracker/README.md`:
- Around line 3-4: Replace the placeholder TODOs in the kit README with a
complete human-readable guide covering the kit’s purpose, prerequisites,
required Lamatic environment variables, installation commands, local startup
steps, and usage examples.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI (base), Organization UI (inherited)
Review profile: ASSERTIVE
Plan: Pro
Run ID: 09678611-e0cc-4a89-bde9-610a6973e7d6
⛔ Files ignored due to path filters (7)
kits/receipt-budget-tracker/apps/app/favicon.icois excluded by!**/*.icokits/receipt-budget-tracker/apps/package-lock.jsonis excluded by!**/package-lock.jsonkits/receipt-budget-tracker/apps/public/file.svgis excluded by!**/*.svgkits/receipt-budget-tracker/apps/public/globe.svgis excluded by!**/*.svgkits/receipt-budget-tracker/apps/public/next.svgis excluded by!**/*.svgkits/receipt-budget-tracker/apps/public/vercel.svgis excluded by!**/*.svgkits/receipt-budget-tracker/apps/public/window.svgis excluded by!**/*.svg
📒 Files selected for processing (25)
kits/receipt-budget-tracker/.gitignorekits/receipt-budget-tracker/README.mdkits/receipt-budget-tracker/agent.mdkits/receipt-budget-tracker/apps/.gitignorekits/receipt-budget-tracker/apps/AGENTS.mdkits/receipt-budget-tracker/apps/CLAUDE.mdkits/receipt-budget-tracker/apps/README.mdkits/receipt-budget-tracker/apps/actions/orchestrate.tskits/receipt-budget-tracker/apps/app/globals.csskits/receipt-budget-tracker/apps/app/layout.tsxkits/receipt-budget-tracker/apps/app/page.tsxkits/receipt-budget-tracker/apps/components/Dashboard.tsxkits/receipt-budget-tracker/apps/eslint.config.mjskits/receipt-budget-tracker/apps/lib/lamatic-client.tskits/receipt-budget-tracker/apps/next.config.tskits/receipt-budget-tracker/apps/package.jsonkits/receipt-budget-tracker/apps/postcss.config.mjskits/receipt-budget-tracker/apps/test-lamatic.jskits/receipt-budget-tracker/apps/tsconfig.jsonkits/receipt-budget-tracker/constitutions/default.mdkits/receipt-budget-tracker/flows/receipt-budget-tracker.tskits/receipt-budget-tracker/lamatic.config.tskits/receipt-budget-tracker/model-configs/receipt-budget-tracker_instructor-llmnode-575_generative-model-name.tskits/receipt-budget-tracker/prompts/receipt-budget-tracker_instructor-llmnode-575_system_0.mdkits/receipt-budget-tracker/prompts/receipt-budget-tracker_instructor-llmnode-575_user_1.md
| ## Identity | ||
| You are an AI assistant built on Lamatic.ai. |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Clear the constitution lint warnings at the source.
Add blank lines after each section heading to resolve the reported MD022 violations. Since default.md is generated/template-driven, fix the template or generator rather than patching only this checked-in file.
Based on learnings, treat this constitution as templated output and correct the source-level formatting.
Also applies to: 6-7, 11-12, 15-16
🧰 Tools
🪛 markdownlint-cli2 (0.22.1)
[warning] 3-3: Headings should be surrounded by blank lines
Expected: 1; Actual: 0; Below
(MD022, blanks-around-headings)
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@kits/receipt-budget-tracker/constitutions/default.md` around lines 3 - 4,
Update the template or generator that produces the constitution content,
ensuring every Markdown section heading is followed by a blank line; regenerate
default.md so all headings, including Identity and the sections identified by
the additional ranges, comply with MD022 without manually patching only the
checked-in output.
Sources: Learnings, Linters/SAST tools
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@kits/receipt-budget-tracker/.env.example`:
- Line 10: Set GEMINI_API_KEY to an empty value in the environment template so
orchestrate.ts does not treat the default configuration as enabled. Preserve the
existing key name and ensure users must provide a real credential before any
outbound Gemini request can occur.
In `@kits/receipt-budget-tracker/apps/.env.example`:
- Around line 2-4: Reorder the Lamatic environment variables so LAMATIC_API_URL
appears before LAMATIC_PROJECT_ID, while preserving all variable names and
values unchanged.
- Line 10: Leave GEMINI_API_KEY empty in the environment template instead of
assigning a truthy placeholder, so copying the template does not enable Gemini
OCR or transmit receipts by default.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI (base), Organization UI (inherited)
Review profile: ASSERTIVE
Plan: Pro
Run ID: ed02e31d-f989-4b65-84a5-9a6b5211b4bb
📒 Files selected for processing (2)
kits/receipt-budget-tracker/.env.examplekits/receipt-budget-tracker/apps/.env.example
|
Hi @Palash-oss! 👋 Before this PR can be reviewed by maintainers, please resolve all comments and requested changes from the CodeRabbit automated review. Steps to follow:
This helps keep the review process efficient for everyone. Thank you! 🙏 |
There was a problem hiding this comment.
Actionable comments posted: 3
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (2)
kits/receipt-budget-tracker/apps/components/Dashboard.tsx (1)
42-60: 🎯 Functional Correctness | 🔵 Trivial | ⚡ Quick winUnnecessary decoy delay in the extraction sequence.
Wrapping
setHistory/setSelectedItem/setIsLoadedinsetTimeout(..., 0)inside a mount effect adds an extra macrotask with no evident benefit here — it's a client effect firing after the initial paint already, so there's no hydration mismatch to dodge. It only stretches the "no receipts" flash a touch longer before the real history renders.🎯 Proposed fix: drop the artificial delay
useEffect(() => { const saved = localStorage.getItem("receipt_history"); if (saved) { try { const parsed = JSON.parse(saved) as ReceiptItem[]; - setTimeout(() => { - setHistory(parsed); - if (parsed.length > 0) { - setSelectedItem(parsed[0]); - } - }, 0); + setHistory(parsed); + if (parsed.length > 0) { + setSelectedItem(parsed[0]); + } } catch (err) { console.error("Failed to parse receipt history from local storage:", err); } } - setTimeout(() => { - setIsLoaded(true); - }, 0); + setIsLoaded(true); }, []);🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@kits/receipt-budget-tracker/apps/components/Dashboard.tsx` around lines 42 - 60, Remove the artificial setTimeout wrappers from the mount useEffect in Dashboard.tsx. Apply setHistory, conditionally setSelectedItem, and setIsLoaded directly during the effect while preserving the existing parsing and error-handling behavior.kits/receipt-budget-tracker/apps/actions/orchestrate.ts (1)
152-167: 🩺 Stability & Availability | 🟠 Major | ⚡ Quick winMission: let
requestIdbypass the early return.
If Lamatic returns an asyncexecuteWorkflowresponse withrequestIdbut no immediateresult, this guard exits before polling ever starts. Check forrequestIdfirst, and only fail fast when both fields are missing.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@kits/receipt-budget-tracker/apps/actions/orchestrate.ts` around lines 152 - 167, Update the executeWorkflowResponse validation before the polling logic so a response containing requestId is accepted even when result is absent. Only return the invalid-response error when both executeWorkflowResponse.result and the request ID sources used by the requestId calculation are missing, allowing the existing polling flow to run.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@kits/receipt-budget-tracker/agent.md`:
- Line 31: Update the OCR Pre-processing description in agent.md to say
“server-side OCR” instead of “client-side OCR,” preserving the rest of the
documented behavior.
In `@kits/receipt-budget-tracker/apps/components/ui/badge.tsx`:
- Around line 8-24: Update the Badge component to use React.forwardRef,
accepting a ref parameter and forwarding it to the rendered div while preserving
the existing className, variant handling, and props behavior. Ensure the
exported component retains appropriate ref and element typing.
In `@kits/receipt-budget-tracker/apps/components/ui/card.tsx`:
- Around line 31-40: Update the ref generic in CardTitle to use
HTMLHeadingElement, matching the rendered h3 element and its existing
HTMLAttributes type; leave the component behavior and props unchanged.
---
Outside diff comments:
In `@kits/receipt-budget-tracker/apps/actions/orchestrate.ts`:
- Around line 152-167: Update the executeWorkflowResponse validation before the
polling logic so a response containing requestId is accepted even when result is
absent. Only return the invalid-response error when both
executeWorkflowResponse.result and the request ID sources used by the requestId
calculation are missing, allowing the existing polling flow to run.
In `@kits/receipt-budget-tracker/apps/components/Dashboard.tsx`:
- Around line 42-60: Remove the artificial setTimeout wrappers from the mount
useEffect in Dashboard.tsx. Apply setHistory, conditionally setSelectedItem, and
setIsLoaded directly during the effect while preserving the existing parsing and
error-handling behavior.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI (base), Organization UI (inherited)
Review profile: ASSERTIVE
Plan: Pro
Run ID: 376d9835-b273-4479-991e-766440fe7737
📒 Files selected for processing (18)
kits/receipt-budget-tracker/.env.examplekits/receipt-budget-tracker/README.mdkits/receipt-budget-tracker/agent.mdkits/receipt-budget-tracker/apps/.env.examplekits/receipt-budget-tracker/apps/.gitignorekits/receipt-budget-tracker/apps/CLAUDE.mdkits/receipt-budget-tracker/apps/README.mdkits/receipt-budget-tracker/apps/actions/orchestrate.tskits/receipt-budget-tracker/apps/components/Dashboard.tsxkits/receipt-budget-tracker/apps/components/ui/badge.tsxkits/receipt-budget-tracker/apps/components/ui/button.tsxkits/receipt-budget-tracker/apps/components/ui/card.tsxkits/receipt-budget-tracker/apps/components/ui/progress.tsxkits/receipt-budget-tracker/apps/lib/utils.tskits/receipt-budget-tracker/apps/test-lamatic.jskits/receipt-budget-tracker/constitutions/default.mdkits/receipt-budget-tracker/flows/receipt-budget-tracker.tskits/receipt-budget-tracker/prompts/receipt-budget-tracker_instructor-llmnode-575_system_0.md
|
@coderabbitai review |
✅ Action performedReview finished.
|
|
@coderabbitai review |
✅ Action performedReview finished.
|
|
@coderabbitai review |
✅ Action performedReview finished.
|
|
@coderabbitai review |
✅ Action performedReview finished.
|
|
@coderabbitai review |
✅ Action performedReview finished.
|
|
@coderabbitai review |
✅ Action performedReview finished.
|
|
@coderabbitai review |
✅ Action performedReview finished.
|
|
@coderabbitai review |
✅ Action performedReview finished.
|
|
@coderabbitai review |
✅ Action performedReview finished.
|
|
@coderabbitai review |
|
|
@coderabbitai review |
|
2 participants
PR Checklist
1. Select Contribution Type
kits/<category>/<kit-name>/)bundles/<bundle-name>/)templates/<template-name>/)2. General Requirements
kebab-caseand matches the flow IDREADME.md(purpose, setup, usage)3. File Structure (Check what applies)
config.jsonpresent with valid metadata (name, description, tags, steps, author, env keys)flows/<flow-name>/(where applicable) include:config.json(Lamatic flow export)inputs.jsonmeta.jsonREADME.md.env.examplewith placeholder values only (kits only)config.jsonnode graphs (changes via Lamatic Studio export)4. Validation
npm install && npm run devworks locally (kits: UI runs; bundles/templates: flows are valid)[kit] Add <name> for <use case>)kits/receipt-budget-tracker/.gitignore(ignores.lamatic/,node_modules/,.env,.env.local)kits/receipt-budget-tracker/README.mdkits/receipt-budget-tracker/agent.mdkits/receipt-budget-tracker/apps/.gitignorekits/receipt-budget-tracker/apps/AGENTS.mdkits/receipt-budget-tracker/apps/CLAUDE.mdkits/receipt-budget-tracker/apps/README.mdkits/receipt-budget-tracker/apps/actions/orchestrate.ts(server action to run/poll Lamatic workflow; optional Gemini OCR; normalizes/extracts receipt data with simulation fallbacks)kits/receipt-budget-tracker/apps/app/globals.csskits/receipt-budget-tracker/apps/app/layout.tsxkits/receipt-budget-tracker/apps/app/page.tsxkits/receipt-budget-tracker/apps/components/Dashboard.tsx(receipt upload/history UI, budget tracking/progress, demo receipts, delete/simulated-mode handling)kits/receipt-budget-tracker/apps/components/ui/badge.tsxkits/receipt-budget-tracker/apps/components/ui/button.tsxkits/receipt-budget-tracker/apps/components/ui/card.tsxkits/receipt-budget-tracker/apps/components/ui/progress.tsxkits/receipt-budget-tracker/apps/eslint.config.mjskits/receipt-budget-tracker/apps/lib/lamatic-client.tskits/receipt-budget-tracker/apps/lib/utils.ts(cnTailwind class merge helper)kits/receipt-budget-tracker/apps/next.config.tskits/receipt-budget-tracker/apps/package.jsonkits/receipt-budget-tracker/apps/postcss.config.mjskits/receipt-budget-tracker/apps/test-lamatic.jskits/receipt-budget-tracker/apps/tsconfig.jsonkits/receipt-budget-tracker/constitutions/default.mdkits/receipt-budget-tracker/flows/receipt-budget-tracker.ts(flow definition)kits/receipt-budget-tracker/lamatic.config.tskits/receipt-budget-tracker/model-configs/receipt-budget-tracker_instructor-llmnode-575_generative-model-name.tskits/receipt-budget-tracker/prompts/receipt-budget-tracker_instructor-llmnode-575_system_0.mdkits/receipt-budget-tracker/prompts/receipt-budget-tracker_instructor-llmnode-575_user_1.mdkits/receipt-budget-tracker/.env.examplekits/receipt-budget-tracker/apps/.env.exampleFlow (
kits/receipt-budget-tracker/flows/receipt-budget-tracker.ts)triggerNode(triggerNode_1→webhookTriggerNode)dynamicNode(InstructorLLMNode_575), configured to produce strict JSON withvendor,date,total,category, anditems[](name,price) using the Instructor LLM model + system/user promptsaddNode(plus-node-addNode_728406) as a post-processing steptriggerNode_1→InstructorLLMNode_575InstructorLLMNode_575→plus-node-addNode_728406