Skip to content

Conversation

@mattpocock
Copy link
Owner

No description provided.

@changeset-bot
Copy link

changeset-bot bot commented Oct 19, 2025

🦋 Changeset detected

Latest commit: 7f73ec0

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

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

@vercel
Copy link

vercel bot commented Oct 19, 2025

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

1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
evalite Ignored Ignored Dec 3, 2025 1:01pm

@pkg-pr-new
Copy link

pkg-pr-new bot commented Oct 19, 2025

Open in StackBlitz

npm i https://pkg.pr.new/mattpocock/evalite@240

commit: 7f73ec0

@mattpocock
Copy link
Owner Author

@claude Remove the BACKEND_ONLY_CONSTANT import that's causing the error in CI

@claude
Copy link

claude bot commented Oct 19, 2025

Claude finished @mattpocock's task —— View job


Task completed

  • Check CI status to understand the error
  • Read modified files to locate the problematic import
  • Remove the BACKEND_ONLY_CONSTANT imports
  • Commit and push the fix

Fixed the CI error:

The error was caused by imports from evalite/backend-only-constants that referenced exports that no longer exist:

  1. test-utils.ts: Removed unused DB_LOCATION import
  2. files.test.ts: Replaced FILES_LOCATION import with a local constant "./node_modules/.evalite/files"

The changes have been committed and pushed to the v1 branch. CI should now pass!


Job Run | Branch: v1

mattpocock and others added 11 commits November 10, 2025 17:39
- Remove unused DB_LOCATION import from test-utils.ts
- Replace FILES_LOCATION import with local constant in files.test.ts

Co-authored-by: Matt Pocock <[email protected]>
- Add dotenv as a dependency
- Create env-setup-file module that imports dotenv/config
- Export env-setup-file as 'evalite/env-setup-file'
- Automatically prepend env-setup-file to setupFiles array
- Update documentation to reflect automatic .env loading
- Update example config to remove manual dotenv setup

Fixes #234

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-authored-by: Matt Pocock <[email protected]>
… precedence

- Add loadVitestSetupFiles() to load setupFiles from vitest.config.ts
- Merge setupFiles from both configs with evalite.config.ts taking precedence
- Add tests for vitest.config.ts setupFiles support and precedence
- setupFiles execution order: env-setup-file -> vitest -> evalite

Co-authored-by: Matt Pocock <[email protected]>
mattpocock and others added 24 commits November 15, 2025 09:53
* Add .editorconfig file

* Return vitest instance when returning with !shouldKeepRunning

This fixes the TS errors.

* Introduce ESLint and add typecheck npm script

- Include ESLint 9 as root dependency
- Set up ESLint to lint the whole repo
- Extend the root config and add a few package-specific plugins for Evalite UI
- Add a consistent `typecheck` npm script for type checking across the repo

Use can now use `pnpm lint` in root and UI app and `pnpm typecheck` anywhere in the repo.
Use `pnpm lint --fix` to attempt to fix the issues.

* Add missing break in switch case

* Fix CI

---------

Co-authored-by: Matt Pocock <[email protected]>
mattpocock and others added 5 commits November 28, 2025 20:14
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
…er incorrectly reports success.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
…eshold success from overriding failures.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
* feat: add watchFiles support for Evalite watch mode

* refactor: rename watchFiles option to forceRerunTriggers

* Fixed errors and made the forceReruntriggers work how it does in vitest

* Changeset

---------

Co-authored-by: Matt Pocock <[email protected]>
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.

5 participants