Skip to content

chore(dx): recommend Vitest extension and document Antares-only setup#165

Open
rmojica-godaddy wants to merge 2 commits intomainfrom
rmb/add-vitest-extension
Open

chore(dx): recommend Vitest extension and document Antares-only setup#165
rmojica-godaddy wants to merge 2 commits intomainfrom
rmb/add-vitest-extension

Conversation

@rmojica-godaddy
Copy link
Copy Markdown
Contributor

Summary

Make it easy for engineers to run and debug @godaddy/antares component tests directly from VS Code or Cursor using the Vitest test explorer, without picking up the slower visual regression project by accident.
Today contributors have to either:

  • Run nx run @godaddy/antares:test from the terminal, losing the inline test UI, or
  • Configure the Vitest editor extension manually and remember which projects exist (SSR, Browser, Visual).
    This PR documents the recommended setup, recommends the official Vitest extension at the workspace level, and gives the Visual project a distinct Chromium instance name so it’s identifiable in the editor’s test tree.

Changes Made

  • .vscode/extensions.json — Add vitest.explorer to workspace recommendations so contributors get prompted to install the Vitest extension when they open the repo.
  • README.md — New "Vitest in VS Code or Cursor" section under Development with the exact vitest.rootConfig and vitest.cliArguments snippet to point the extension at @godaddy/antares and limit it to the SSR and Browser projects.
  • configs/vitest.config.mts — Name the Visual project's Chromium instance Visual (Chromium) so it is distinguishable from the Browser project's Chromium instance in the Vitest editor UI.

Copilot AI review requested due to automatic review settings April 24, 2026 16:11
@rmojica-godaddy rmojica-godaddy requested a review from a team as a code owner April 24, 2026 16:11
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 24, 2026

⚠️ No Changeset found

Latest commit: 6cfc23e

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR improves the contributor experience for running @godaddy/antares component tests in VS Code/Cursor by recommending the Vitest test explorer extension, documenting the suggested editor configuration, and making the Visual Vitest project easier to distinguish in the test tree.

Changes:

  • Recommend the official vitest.explorer extension at the workspace level.
  • Document the recommended vitest.rootConfig and vitest.cliArguments settings for focusing on Antares SSR + Browser projects.
  • Rename the Visual project’s Chromium instance to Visual (Chromium) in the shared Vitest config.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
configs/vitest.config.mts Adds a named Chromium instance for the Visual test project for clearer editor test tree display.
README.md Adds “Vitest in VS Code or Cursor” setup documentation for the Vitest extension.
.vscode/extensions.json Recommends installing vitest.explorer alongside Biome.

Comment thread README.md Outdated
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 24, 2026

📚 Storybook Preview

Preview URL: https://godaddy.github.io/antares/pr-165/

Preview will be automatically deleted when this PR is closed.

@github-actions
Copy link
Copy Markdown
Contributor

Docs Site Preview

Preview URL: https://godaddy.github.io/antares/pr-165-site/

Preview will be automatically deleted when this PR is closed.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Comment thread README.md
Comment on lines +132 to +137
```json
{
"vitest.rootConfig": "packages/@godaddy/antares/vitest.config.ts",
"vitest.cliArguments": "--project=SSR --project=Browser"
}
```
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.

we have a /.vscode/settings.json workspace file, can we add these settings there directly?

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.

3 participants