Skip to content

fix(test): resolve defineCodeJudge fixture import for worktree compatibility#450

Merged
christso merged 1 commit intomainfrom
fix/449-test-fixture-worktree
Mar 7, 2026
Merged

fix(test): resolve defineCodeJudge fixture import for worktree compatibility#450
christso merged 1 commit intomainfrom
fix/449-test-fixture-worktree

Conversation

@christso
Copy link
Copy Markdown
Collaborator

@christso christso commented Mar 7, 2026

Summary

  • Use relative import path in test-define-judge.ts fixture instead of @agentv/eval workspace package, which fails to resolve in git worktrees where node_modules symlinks may not be set up

Note: The tilde range test fix (issue item #1) was already addressed in commit 90561eb.

Test plan

  • All 37 evaluator tests pass
  • All 9 version-check tests pass
  • Pre-push hooks pass (build, typecheck, lint, test)

Closes #449

🤖 Generated with Claude Code

…e compatibility

The fixture imported from '@agentv/eval' which relies on workspace
symlinks in node_modules. In git worktrees, these symlinks may not be
set up, causing the test to fail with an unresolved import. Using a
relative path to the eval package source avoids workspace resolution.

Closes #449

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying agentv with  Cloudflare Pages  Cloudflare Pages

Latest commit: 7c4b11d
Status: ✅  Deploy successful!
Preview URL: https://be084caa.agentv.pages.dev
Branch Preview URL: https://fix-449-test-fixture-worktre.agentv.pages.dev

View logs

@christso christso merged commit 17e2bd7 into main Mar 7, 2026
1 check passed
@christso christso deleted the fix/449-test-fixture-worktree branch March 7, 2026 00:18
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.

fix: pre-existing test failures in worktrees (version-check tilde range, CodeEvaluator fixture)

1 participant