Add AGENTS.md with Cursor Cloud development environment instructions#407
Add AGENTS.md with Cursor Cloud development environment instructions#407syed-reza98 merged 3 commits intomainfrom
Conversation
Co-authored-by: Syed Salman Reza <syed.reza181@gmail.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Co-authored-by: Syed Salman Reza <syed.reza181@gmail.com>
There was a problem hiding this comment.
Pull request overview
Adds Cursor Cloud-focused developer setup documentation for StormCom, aiming to streamline environment bootstrapping (DB, env vars, Prisma, and common commands) for cloud agents.
Changes:
- Added
AGENTS.mdwith Cursor Cloud development environment setup steps, commands, and seeded test credentials. - Updated
package-lock.json(lockfile content changes not described in the PR body/title).
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| AGENTS.md | New Cursor Cloud setup guide (services, env files, Prisma steps, commands, credentials, gotchas). |
| package-lock.json | Lockfile updated with additional nested dependency entries and metadata changes. |
|
|
||
| ```bash | ||
| npx prisma migrate deploy # apply all migrations | ||
| npm run prisma:seed # seed demo data (131 entities) |
There was a problem hiding this comment.
The database setup snippet uses npx prisma migrate deploy and then seeds, but it doesn’t mention generating the Prisma Client afterward. In a fresh environment (or if postinstall skipped because DATABASE_URL wasn’t set at install time), this can leave @prisma/client missing/out of date and cause runtime errors. Consider updating these steps to run npm run prisma:generate after migrations, or use an existing npm script that already combines migrate + generate (e.g. npm run prisma:migrate:dev:seed / npm run prisma:migrate:dev).
| npm run prisma:seed # seed demo data (131 entities) | |
| npm run prisma:generate # generate Prisma Client | |
| npm run prisma:seed # seed demo data (131 entities) |
|
|
||
| ### Gotchas | ||
|
|
||
| - The Vitest run reports `playwright.config.test.ts` as a failed suite because it contains no Vitest tests — this is a known false positive and safe to ignore. |
There was a problem hiding this comment.
The doc suggests ignoring a Vitest “failed suite” caused by playwright.config.test.ts, but the repo’s vitest.config.ts currently includes **/*.{test,spec}.{ts,tsx} and does not exclude that file, so this will keep recurring and can break CI depending on Vitest settings. Instead of documenting it as safe to ignore, consider addressing it by excluding playwright.config.test.ts in Vitest config or renaming the Playwright config file so it doesn’t match the Vitest include glob.
| - The Vitest run reports `playwright.config.test.ts` as a failed suite because it contains no Vitest tests — this is a known false positive and safe to ignore. | |
| - If Vitest reports `playwright.config.test.ts` as a failed suite, update `vitest.config.ts` to exclude this file (for example via `test.exclude`) or rename/move the Playwright config so it no longer matches the `**/*.{test,spec}.{ts,tsx}` pattern, rather than ignoring the failure. |
PR Type
Related Issues
Changes
Added
AGENTS.mdwith Cursor Cloud specific development instructions including:Testing
Verified Development Environment
docker composeplaywright.config.test.ts)Demo
stormcom_login_and_dashboard_demo.mp4
Landing page:
StormCom landing page
Admin dashboard after login:
Admin dashboard with metrics
Customers management page:
Customers page with data
Documentation
Checklist
To show artifacts inline, enable in settings.