fix(worker): clean up orphaned zoekt .tmp shard files after every index attempt #706
brendan-kellamAnnotations
8 errors and 1 warning
|
test
Process completed with exit code 1.
|
|
src/repoIndexManager.test.ts > RepoIndexManager > latestIndexingJobStatus Updates > sets latestIndexingJobStatus to IN_PROGRESS when job starts:
packages/backend/src/repoIndexManager.ts#L527
Error: [vitest] No "cleanupTempShards" export is defined on the "./zoekt.js" mock. Did you forget to return it from "vi.mock"?
If you need to partially mock a module, you can use "importOriginal" helper inside:
vi.mock(import("./zoekt.js"), async (importOriginal) => {
const actual = await importOriginal()
return {
...actual,
// your mocked methods
}
})
❯ RepoIndexManager.indexRepository src/repoIndexManager.ts:527:19
❯ RepoIndexManager.runJob src/repoIndexManager.ts:333:31
❯ src/repoIndexManager.ts:264:24
|
|
src/repoIndexManager.test.ts > RepoIndexManager > Job Processing - Success > de-duplicates the default branch before truncating matching branches:
packages/backend/src/repoIndexManager.ts#L527
Error: [vitest] No "cleanupTempShards" export is defined on the "./zoekt.js" mock. Did you forget to return it from "vi.mock"?
If you need to partially mock a module, you can use "importOriginal" helper inside:
vi.mock(import("./zoekt.js"), async (importOriginal) => {
const actual = await importOriginal()
return {
...actual,
// your mocked methods
}
})
❯ RepoIndexManager.indexRepository src/repoIndexManager.ts:527:19
❯ RepoIndexManager.runJob src/repoIndexManager.ts:333:31
❯ src/repoIndexManager.ts:264:24
|
|
src/repoIndexManager.test.ts > RepoIndexManager > Job Processing - Success > keeps default branch and truncates to the first 63 matching tags:
packages/backend/src/repoIndexManager.ts#L527
Error: [vitest] No "cleanupTempShards" export is defined on the "./zoekt.js" mock. Did you forget to return it from "vi.mock"?
If you need to partially mock a module, you can use "importOriginal" helper inside:
vi.mock(import("./zoekt.js"), async (importOriginal) => {
const actual = await importOriginal()
return {
...actual,
// your mocked methods
}
})
❯ RepoIndexManager.indexRepository src/repoIndexManager.ts:527:19
❯ RepoIndexManager.runJob src/repoIndexManager.ts:333:31
❯ src/repoIndexManager.ts:264:24
|
|
src/repoIndexManager.test.ts > RepoIndexManager > Job Processing - Success > invokes zoekt-git-index with correct arguments:
packages/backend/src/repoIndexManager.ts#L527
Error: [vitest] No "cleanupTempShards" export is defined on the "./zoekt.js" mock. Did you forget to return it from "vi.mock"?
If you need to partially mock a module, you can use "importOriginal" helper inside:
vi.mock(import("./zoekt.js"), async (importOriginal) => {
const actual = await importOriginal()
return {
...actual,
// your mocked methods
}
})
❯ RepoIndexManager.indexRepository src/repoIndexManager.ts:527:19
❯ RepoIndexManager.runJob src/repoIndexManager.ts:333:31
❯ src/repoIndexManager.ts:264:24
|
|
src/repoIndexManager.test.ts > RepoIndexManager > Job Processing - Success > fetches existing repository when directory exists:
packages/backend/src/repoIndexManager.ts#L527
Error: [vitest] No "cleanupTempShards" export is defined on the "./zoekt.js" mock. Did you forget to return it from "vi.mock"?
If you need to partially mock a module, you can use "importOriginal" helper inside:
vi.mock(import("./zoekt.js"), async (importOriginal) => {
const actual = await importOriginal()
return {
...actual,
// your mocked methods
}
})
❯ RepoIndexManager.indexRepository src/repoIndexManager.ts:527:19
❯ RepoIndexManager.runJob src/repoIndexManager.ts:333:31
❯ src/repoIndexManager.ts:264:24
|
|
src/repoIndexManager.test.ts > RepoIndexManager > Job Processing - Success > deletes directory and performs fresh clone when path exists but is not a valid git repo root:
packages/backend/src/repoIndexManager.ts#L527
Error: [vitest] No "cleanupTempShards" export is defined on the "./zoekt.js" mock. Did you forget to return it from "vi.mock"?
If you need to partially mock a module, you can use "importOriginal" helper inside:
vi.mock(import("./zoekt.js"), async (importOriginal) => {
const actual = await importOriginal()
return {
...actual,
// your mocked methods
}
})
❯ RepoIndexManager.indexRepository src/repoIndexManager.ts:527:19
❯ RepoIndexManager.runJob src/repoIndexManager.ts:333:31
❯ src/repoIndexManager.ts:264:24
|
|
src/repoIndexManager.test.ts > RepoIndexManager > Job Processing - Success > clones new repository when directory does not exist:
packages/backend/src/repoIndexManager.ts#L527
Error: [vitest] No "cleanupTempShards" export is defined on the "./zoekt.js" mock. Did you forget to return it from "vi.mock"?
If you need to partially mock a module, you can use "importOriginal" helper inside:
vi.mock(import("./zoekt.js"), async (importOriginal) => {
const actual = await importOriginal()
return {
...actual,
// your mocked methods
}
})
❯ RepoIndexManager.indexRepository src/repoIndexManager.ts:527:19
❯ RepoIndexManager.runJob src/repoIndexManager.ts:333:31
❯ src/repoIndexManager.ts:264:24
|
|
test
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/checkout@v4, actions/setup-node@v4. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|