Fix test paths and refresh lockfile#9
Conversation
There was a problem hiding this comment.
Review
Test path fixes (test/unit.mjs, test/e2e-battle-test.mjs) — correct. Using fileURLToPath instead of .pathname is the right fix for paths with spaces.
package-lock.json dep removal — expected and correct. The current package.json on main only lists @iarna/toml and supermemory as runtime deps. @modelcontextprotocol/sdk and zod are already gone from package.json — the lockfile was stale from a prior version. Regenerating it produces exactly this result. Running npm install yourself would give the same diff.
Version mismatch — one issue: the lockfile in this PR sets version 1.0.5 but package.json on main is already at 1.0.6. The branch was generated against a slightly older commit. Should be rebased onto main + npm install re-run before merging so the versions are consistent.
|
@williamclay8 thank you for your contribution |
|
LGTM |
|
@MaheshtheDev pls merge this |
Summary
Verification
Direct push to upstream was blocked for williamclay8, so this PR comes from the fork.