Skip to content

fix: align vitest and @vitest/coverage-v8 versions#160

Open
shazzar00ni wants to merge 14 commits into
mainfrom
opencode/kind-star
Open

fix: align vitest and @vitest/coverage-v8 versions#160
shazzar00ni wants to merge 14 commits into
mainfrom
opencode/kind-star

Conversation

@shazzar00ni

@shazzar00ni shazzar00ni commented Apr 8, 2026

Copy link
Copy Markdown
Owner

Summary

  • Updated vitest to ^4.1.2
  • Updated @vitest/coverage-v8 to ^4.1.2 to match
  • Resolves ERESOLVE dependency conflict causing Vercel build failure
  • Uses --legacy-peer-deps flag to handle eslint peer dependency

Testing

  • Built locally with npm run build - success

Copilot AI review requested due to automatic review settings April 8, 2026 00:41
@vercel

vercel Bot commented Apr 8, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docugen Ready Ready Preview, Comment May 9, 2026 9:27am

@coderabbitai

coderabbitai Bot commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: e66675b6-5f95-4ab5-a828-2ec185914615

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Walkthrough

One line is added to .gitignore to exclude log files matching the pattern *.log from version control, following existing ignore rules for node_modules/ and dist/.

Changes

GitIgnore Configuration

Layer / File(s) Summary
Git Ignore Rules
.gitignore
Added *.log pattern to exclude log files from version control.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minute

Poem

Log files scattered, taking up space,
A simple pattern hides them away with grace,
*.log now sleeps in the ignore file's embrace,
Keeping the repo clean, a tidy place. 📝✨

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title describes fixing vitest and @vitest/coverage-v8 version alignment, but the changeset only adds a .gitignore pattern. This is misleading. Update the PR title to reflect the actual changes, such as 'Add *.log pattern to .gitignore' or clarify why version alignment changes are not reflected in the raw summary.
Description check ⚠️ Warning The description details vitest and @vitest/coverage-v8 version updates and dependency conflict resolution, but the raw summary shows only .gitignore modification. The description appears unrelated to the actual changeset. Ensure the description accurately reflects the actual file changes shown in the raw summary, or verify that the raw summary is complete and includes all modified files.
✅ Passed checks (3 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch opencode/kind-star

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codacy-production

codacy-production Bot commented Apr 8, 2026

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@shazzar00ni shazzar00ni self-assigned this Apr 8, 2026
@shazzar00ni shazzar00ni added the dependencies Pull requests that update a dependency file label Apr 8, 2026

This comment was marked as resolved.

coderabbitai[bot]

This comment was marked as resolved.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

@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 potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

devin-ai-integration[bot]

This comment was marked as resolved.

Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>
devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

@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 found 1 new potential issue.

⚠️ 1 issue in files not directly in the diff

⚠️ Debug log file with local filesystem paths committed to repository (debug-storybook.log:1-187)

debug-storybook.log is a build debug log that was accidentally committed. It exposes local filesystem paths and the developer's username (e.g., /Users/shannonlockett/.local/share/opencode/worktree/...). This file serves no purpose in the repository, leaks developer environment details, and will grow stale. It is not excluded by .gitignore (which only ignores node_modules/ and dist/).

Open in Devin Review

@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 found 1 new potential issue.

Open in Devin Review

Comment thread debug-storybook.log Outdated
@shazzar00ni

Copy link
Copy Markdown
Owner Author

@copilot resolve the merge conflicts in this pull request

Copilot AI and others added 2 commits May 6, 2026 17:16
# Conflicts:
#	debug-storybook.log

Co-authored-by: shazzar00ni <155769623+shazzar00ni@users.noreply.github.com>

Copilot AI commented May 6, 2026

Copy link
Copy Markdown
Contributor

@copilot resolve the merge conflicts in this pull request

Merge conflicts resolved in a774115. The only conflict was debug-storybook.log (deleted in this branch, modified in main) — kept it deleted since *.log is in .gitignore. Also brought in the @typescript-eslint/eslint-plugin ^8.59.2 bump from main. Build verified passing.

@sonarqubecloud

sonarqubecloud Bot commented May 6, 2026

Copy link
Copy Markdown

@github-actions

github-actions Bot commented May 9, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

  • package-lock.json

@sonarqubecloud

sonarqubecloud Bot commented May 9, 2026

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants