fix(mothership): answer a retried task wake whose turn already ran #33805
ci.yml
on: pull_request
Detect Version
0s
Detect Desktop Changes
Prepare Trigger.dev
0s
Matrix: Build AMD64
Matrix: Build Dev ECR
Matrix: Build ARM64 (GHCR Only)
Matrix: Test and Build / oauth-postgres
Test and Build
/
Lint and Test
7m 56s
Test and Build
/
Build App
4m 17s
Check Docs Changes
0s
Migrate Dev DB
/
Apply Database Migrations
Matrix: Create GHCR Manifests
Process Docs
/
Process Documentation Embeddings
Promote Trigger.dev
0s
Prune Desktop Prereleases
0s
Matrix: Attest Images
Desktop Release
/
Build, Sign, Notarize
Annotations
5 errors and 11 warnings
|
Test and Build / Lint and Test
Process completed with exit code 1.
|
|
lib/mothership/tasks/application/tasks.test.ts > worker HTTP contracts > returns accepted only after reserving the wake:
apps/sim/lib/mothership/tasks/application/tasks.test.ts#L296
AssertionError: expected 500 to be 202 // Object.is equality
- Expected
+ Received
- 202
+ 500
❯ lib/mothership/tasks/application/tasks.test.ts:296:29
|
|
lib/mothership/tasks/application/tasks.test.ts > wake admission > reserves the exact retry identity before acknowledging:
apps/sim/lib/mothership/tasks/application/prepare-wake.ts#L47
Error: [vitest] No "getLocalChatStreamLease" export is defined on the "@/lib/mothership/request/session/abort" 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("@/lib/mothership/request/session/abort"), async (importOriginal) => {
const actual = await importOriginal()
return {
...actual,
// your mocked methods
}
})
❯ Object.execute lib/mothership/tasks/application/prepare-wake.ts:47:19
❯ lib/core/application/authorized-workspace-use-case.ts:218:24
|
|
Test and Build / Lint and Test
@sim/app#test: command (/home/runner/_work/sim/sim/apps/sim) /home/runner/.bun/bin/bun run test exited (1)
|
|
Test and Build / Lint and Test
Process completed with exit code 1.
|
|
Test and Build / Lint and Test
Skipping sticky disk commit due to previous step failures
|
|
Test and Build / Lint and Test
- Step: unknown (failed)
|
|
Test and Build / Lint and Test
Found 2 failed/cancelled steps in previous workflow steps
|
|
Test and Build / Lint and Test
Skipping sticky disk commit due to previous step failures
|
|
Test and Build / Lint and Test
- Step: unknown (failed)
|
|
Test and Build / Lint and Test
Found 2 failed/cancelled steps in previous workflow steps
|
|
Test and Build / Lint and Test
Skipping sticky disk commit due to previous step failures
|
|
Test and Build / Lint and Test
- Step: unknown (failed)
|
|
Test and Build / Lint and Test
Found 2 failed/cancelled steps in previous workflow steps
|
|
Test and Build / PostgreSQL integration (push)
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
Test and Build / PostgreSQL integration (migrate)
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
Artifacts
Produced during runtime
| Name | Size | Digest | |
|---|---|---|---|
|
integration-reports-migrate
|
72 KB |
sha256:1b4845d2b244cffa0f521eb0aa73d0951bf6228cbf06243a45327b4719ed02a9
|
|
|
integration-reports-push
|
73.1 KB |
sha256:b2ff136fb9ef51bd5d1be857dc72285dcd975fb91829413994681dec5ea7dd5e
|
|