Hey! I was taking a look at why the tests were failing in CI across the recent PRs and tracked it down.
It looks like jest.config.js has a moduleNameMapper mapping ^tasknotes-nlp-core$ to <rootDir>/../tasknotes-nlp-core/src/index.ts. Since GitHub Actions only checks out the main repository, it crashes when Jest looks for the sibling directory.
I didn't want to just open a PR removing that line since it looks like you rely on it for side-by-side local development across the two repos. Let me know if you have a preferred way you'd like to handle this (e.g., using an environment variable for CI, or moving local overrides to a jest.local.config.js ignored by git). Happy to write the PR for it once I know your preferred approach!
Hey! I was taking a look at why the tests were failing in CI across the recent PRs and tracked it down.
It looks like
jest.config.jshas amoduleNameMappermapping^tasknotes-nlp-core$to<rootDir>/../tasknotes-nlp-core/src/index.ts. Since GitHub Actions only checks out the main repository, it crashes when Jest looks for the sibling directory.I didn't want to just open a PR removing that line since it looks like you rely on it for side-by-side local development across the two repos. Let me know if you have a preferred way you'd like to handle this (e.g., using an environment variable for CI, or moving local overrides to a
jest.local.config.jsignored by git). Happy to write the PR for it once I know your preferred approach!