feat: Add vehicle-service-advisor kit - #325
Open
saivarun1410 wants to merge 6 commits into
Open
Conversation
Contributor
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI (base), Organization UI (inherited) Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (7)
WalkthroughChangesVehicle Service Advisor
Suggested reviewers: 🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (3 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 |
Contributor
:robot_face: AgentKit Structural ValidationNew Contributions Detected
Check Results
🎉 All checks passed! This contribution follows the AgentKit structure. |
saivarun1410
marked this pull request as ready for review
August 6, 2026 05:10
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 5
🤖 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/vehicle-service-advisor/apps/features/assessment/components/assessment-form.tsx`:
- Around line 118-120: Update the “Preview sample report” button in the
assessment form to set its disabled state from isLoading, preventing
onShowSample while the assessment submission is pending; preserve the existing
button label and click behavior when loading is false.
In
`@kits/vehicle-service-advisor/apps/features/assessment/hooks/use-assessment.ts`:
- Around line 21-28: Update the assessment submission flow around assessVehicle
so setIsLoading(false) executes in a finally block even when the server action
rejects. Preserve the existing success handling, report assignment, and error
fallback for resolved unsuccessful results, while ensuring rejected requests
also surface through the existing error state as appropriate.
In
`@kits/vehicle-service-advisor/apps/features/assessment/types/sample-report.ts`:
- Around line 4-48: Update the sample report’s stopDriving field to true and
revise the safetyMessage, ownerActions, and inspectionPlan guidance to prohibit
continued driving and recommend towing or immediate professional inspection.
Keep the overheating evidence and diagnostic details unchanged while ensuring
the preview consistently communicates a no-drive response.
In `@kits/vehicle-service-advisor/apps/package.json`:
- Around line 12-17: Update the app package configuration around the
dependencies block to declare the required UI stack: Tailwind CSS v4 tooling,
react-hook-form, lucide-react, shadcn/ui components, and next.config.mjs, while
preserving the existing Next.js, React, Zod, and lamatic dependencies. Add the
corresponding missing configuration files, or reference the repository-level
source that provides them.
In
`@kits/vehicle-service-advisor/prompts/vehicle-service-advisor_llmnode-201_system_0.md`:
- Around line 11-17: Update the JSON response schema in the system prompt to
define stop_driving as urgency-dependent: require true when urgency is stop_now
and false for urgent, soon, or monitor. Keep the existing response shape and
valid urgency values unchanged.
🪄 Autofix
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 Plus
Run ID: d48701d6-9451-446f-a7e3-b61743118e0f
⛔ Files ignored due to path filters (1)
kits/vehicle-service-advisor/apps/package-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (30)
kits/vehicle-service-advisor/.env.examplekits/vehicle-service-advisor/.gitignorekits/vehicle-service-advisor/README.mdkits/vehicle-service-advisor/agent.mdkits/vehicle-service-advisor/apps/.env.examplekits/vehicle-service-advisor/apps/.gitignorekits/vehicle-service-advisor/apps/README.mdkits/vehicle-service-advisor/apps/actions/orchestrate.tskits/vehicle-service-advisor/apps/app/globals.csskits/vehicle-service-advisor/apps/app/layout.tsxkits/vehicle-service-advisor/apps/app/page.tsxkits/vehicle-service-advisor/apps/eslint.config.mjskits/vehicle-service-advisor/apps/features/assessment/components/assessment-form.tsxkits/vehicle-service-advisor/apps/features/assessment/components/assessment-report.tsxkits/vehicle-service-advisor/apps/features/assessment/hooks/use-assessment.tskits/vehicle-service-advisor/apps/features/assessment/types/assessment.tskits/vehicle-service-advisor/apps/features/assessment/types/sample-report.tskits/vehicle-service-advisor/apps/features/assessment/validation/assessment-schema.tskits/vehicle-service-advisor/apps/lib/lamatic-client.tskits/vehicle-service-advisor/apps/lib/server-logger.tskits/vehicle-service-advisor/apps/next-env.d.tskits/vehicle-service-advisor/apps/next.config.tskits/vehicle-service-advisor/apps/package.jsonkits/vehicle-service-advisor/apps/tsconfig.jsonkits/vehicle-service-advisor/constitutions/default.mdkits/vehicle-service-advisor/flows/vehicle-service-advisor.tskits/vehicle-service-advisor/lamatic.config.tskits/vehicle-service-advisor/model-configs/vehicle-service-advisor_llmnode-201_generative-model-name.tskits/vehicle-service-advisor/prompts/vehicle-service-advisor_llmnode-201_system_0.mdkits/vehicle-service-advisor/prompts/vehicle-service-advisor_llmnode-201_user_1.md
Contributor
|
Hi @saivarun1410! 👋 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! 🙏 |
Contributor
|
Note GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer. |
Contributor
There was a problem hiding this comment.
Actionable comments posted: 6
🤖 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/vehicle-service-advisor/.gitignore`:
- Around line 4-5: Update the .gitignore environment-file patterns to ignore all
.env variants using .env*, then explicitly re-include only .env.example so
template configuration remains trackable.
In `@kits/vehicle-service-advisor/apps/.env.example`:
- Around line 1-4: Reorder the environment variable entries alphabetically in
both kits/vehicle-service-advisor/apps/.env.example lines 1-4 and
kits/vehicle-service-advisor/.env.example lines 1-4, preserving all existing
keys and empty values.
In `@kits/vehicle-service-advisor/apps/.gitignore`:
- Around line 3-4: Update the environment-file rules in the app .gitignore to
ignore all .env* variants, including Next.js development and production files,
while explicitly re-including .env.example so the template remains trackable.
In `@kits/vehicle-service-advisor/apps/components/ui/button.tsx`:
- Around line 32-48: Update the public Button component to use the app’s
established React 19-compatible ref-forwarding approach, such as
React.forwardRef or the exported useForwardRef utility. Explicitly accept and
pass the ref to the rendered component, ensuring native button refs work when
asChild is false while preserving Slot behavior when asChild is true.
In
`@kits/vehicle-service-advisor/apps/features/assessment/components/assessment-report.tsx`:
- Around line 93-95: Replace the native button in the assessment report with the
shared Button component imported from "`@/components/ui/button`". Preserve the
onClear handler, button type, label, and className="secondary-button", and set
variant="outline".
In `@kits/vehicle-service-advisor/apps/package.json`:
- Line 10: Make clean-checkout typechecking deterministic: in
kits/vehicle-service-advisor/apps/package.json at line 10, update the typecheck
script to run Next.js type generation before tsc; in
kits/vehicle-service-advisor/apps/.gitignore at line 6, add next-env.d.ts; and
remove the tracked generated file
kits/vehicle-service-advisor/apps/next-env.d.ts.
🪄 Autofix
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 Plus
Run ID: 504a08c5-1dda-4944-a169-2085cd124eb2
⛔ Files ignored due to path filters (1)
kits/vehicle-service-advisor/apps/package-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (34)
kits/vehicle-service-advisor/.env.examplekits/vehicle-service-advisor/.gitignorekits/vehicle-service-advisor/README.mdkits/vehicle-service-advisor/agent.mdkits/vehicle-service-advisor/apps/.env.examplekits/vehicle-service-advisor/apps/.gitignorekits/vehicle-service-advisor/apps/README.mdkits/vehicle-service-advisor/apps/actions/orchestrate.tskits/vehicle-service-advisor/apps/app/globals.csskits/vehicle-service-advisor/apps/app/layout.tsxkits/vehicle-service-advisor/apps/app/page.tsxkits/vehicle-service-advisor/apps/components.jsonkits/vehicle-service-advisor/apps/components/ui/button.tsxkits/vehicle-service-advisor/apps/eslint.config.mjskits/vehicle-service-advisor/apps/features/assessment/components/assessment-form.tsxkits/vehicle-service-advisor/apps/features/assessment/components/assessment-report.tsxkits/vehicle-service-advisor/apps/features/assessment/hooks/use-assessment.tskits/vehicle-service-advisor/apps/features/assessment/types/assessment.tskits/vehicle-service-advisor/apps/features/assessment/types/sample-report.tskits/vehicle-service-advisor/apps/features/assessment/validation/assessment-schema.tskits/vehicle-service-advisor/apps/lib/lamatic-client.tskits/vehicle-service-advisor/apps/lib/server-logger.tskits/vehicle-service-advisor/apps/lib/utils.tskits/vehicle-service-advisor/apps/next-env.d.tskits/vehicle-service-advisor/apps/next.config.mjskits/vehicle-service-advisor/apps/package.jsonkits/vehicle-service-advisor/apps/postcss.config.mjskits/vehicle-service-advisor/apps/tsconfig.jsonkits/vehicle-service-advisor/constitutions/default.mdkits/vehicle-service-advisor/flows/vehicle-service-advisor.tskits/vehicle-service-advisor/lamatic.config.tskits/vehicle-service-advisor/model-configs/vehicle-service-advisor_llmnode-201_generative-model-name.tskits/vehicle-service-advisor/prompts/vehicle-service-advisor_llmnode-201_system_0.mdkits/vehicle-service-advisor/prompts/vehicle-service-advisor_llmnode-201_user_1.md
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
What changed
Adds a complete Vehicle Service Advisor kit under
kits/vehicle-service-advisor/:Live demo
https://agent-kit-smoky.vercel.app
Why
Vehicle owners often describe warning lights, noises, smells, and drivability issues imprecisely. This kit turns those observations into a focused pre-inspection brief while explicitly avoiding false diagnostic certainty and unsafe repair advice.
The deployment follow-up removes a custom
outputFileTracingRootoverride that caused Vercel's Next.js monorepo packager to resolve the application as a nonexistent top-level/appsdirectory.User impact
Owners get clear urgency guidance and a better workshop handoff. Technicians receive a concise, evidence-based symptom summary instead of an unstructured description.
Validation
npm run lintnpm run typechecknpm run buildStop drivingandDo not drivewith no browser runtime errorsNotes
The Lamatic provider credential, project API key, and deployed flow ID are intentionally excluded. Import the flow into Studio, select a provider credential, deploy it, and populate
apps/.env.localfrom the provided example.kits/vehicle-service-advisor/.README.mdandagent.md.VEHICLE_SERVICE_ADVISOR_FLOW_ID.report.SAMPLE_REPORTfixture.Buttoncomponent.