Skip to content

fix(test): keep vitest out of .cursor skill drivers - #413

Closed
bubbleptr wants to merge 1 commit into
mainfrom
fix/vitest-exclude-agent-skills
Closed

bubbleptr wants to merge 1 commit into
mainfrom
fix/vitest-exclude-agent-skills

Conversation

@bubbleptr

@bubbleptr bubbleptr commented Sep 30, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • a729a40 added .cursor/skills/verify-pace/drive/changes.spec.ts, a Playwright driver. Vitest collected it and failed (no tests), so bun run test is red on main.
  • Add **/.cursor/** to the vitest test.exclude. It is the only tracked test/spec file under a dot-directory.

Test plan

  • Reproduced on origin/main: bunx vitest run .cursor/skills/verify-pace/drive/changes.spec.ts fails
  • bunx vitest list no longer lists anything under .cursor/
  • bun run test: 162 files / 1917 tests passed
  • bun run typecheck, bun run lint clean

Devin Review

.cursor/skills/verify-pace ships a Playwright driver spec that vitest collected, failing bun run test on main.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

@bubbleptr

Copy link
Copy Markdown
Owner Author

Superseded by 1a26a7b on main, which already excludes .cursor/** from vitest.

@bubbleptr bubbleptr closed this Sep 30, 2026
@bubbleptr
bubbleptr deleted the fix/vitest-exclude-agent-skills branch September 30, 2026 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant