fix(mothership): explain withheld code and workflow results and bound workflow block logs #33713
waleedlatif1ci.yml
on: pull_request
Detect Version
Detect Desktop Changes
Prepare Trigger.dev
Matrix: Build AMD64
Matrix: Build Dev ECR
Matrix: Build ARM64 (GHCR Only)
Matrix: Test and Build / oauth-postgres
Test and Build
/
Lint and Test
5m 25s
Test and Build
/
Build App
4m 10s
Check Docs Changes
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
7 errors and 8 warnings
|
Test and Build / Lint and Test
Process completed with exit code 1.
|
|
providers/runtime-context.test.ts > provider runtime context > keeps a raw thrown error separate from the omitted model error:
apps/sim/providers/runtime-context.test.ts#L896
AssertionError: expected { success: false, …(2) } to deeply equal { success: false, output: {}, …(1) }
- Expected
+ Received
{
"error": "Tool execution settled, but its result could not be returned safely. Do not retry a mutation automatically.",
- "output": {},
+ "output": {
+ "resultWithheld": true,
+ "withheldReason": "Secret provenance for this call's inputs could not be verified, so its output could contain a secret value.",
+ },
"success": false,
}
❯ providers/runtime-context.test.ts:896:37
|
|
providers/runtime-context.test.ts > provider runtime context > retains child provenance for raw traces when model projection fails:
apps/sim/providers/runtime-context.test.ts#L870
AssertionError: expected { success: true, output: { …(2) } } to deeply equal { success: true, output: {} }
- Expected
+ Received
{
- "output": {},
+ "output": {
+ "resultWithheld": true,
+ "withheldReason": "The result could not be checked for secret values, usually because it is too large. Request a narrower result.",
+ },
"success": true,
}
❯ providers/runtime-context.test.ts:870:37
|
|
providers/runtime-context.test.ts > provider runtime context > structurally omits an incomplete failed model result and marks provenance incomplete:
apps/sim/providers/runtime-context.test.ts#L841
AssertionError: expected { success: false, …(2) } to deeply equal { success: false, output: {}, …(1) }
- Expected
+ Received
{
"error": "Tool execution settled, but its result could not be returned safely. Do not retry a mutation automatically.",
- "output": {},
+ "output": {
+ "resultWithheld": true,
+ "withheldReason": "Secret provenance for this call's inputs could not be verified, so its output could contain a secret value.",
+ },
"success": false,
}
❯ providers/runtime-context.test.ts:841:20
|
|
providers/runtime-context.test.ts > provider runtime context > omits an incomplete model result and marks parent provenance incomplete:
apps/sim/providers/runtime-context.test.ts#L818
AssertionError: expected { success: true, output: { …(2) } } to deeply equal { success: true, output: {} }
- Expected
+ Received
{
- "output": {},
+ "output": {
+ "resultWithheld": true,
+ "withheldReason": "Secret provenance for this call's inputs could not be verified, so its output could contain a secret value.",
+ },
"success": true,
}
❯ providers/runtime-context.test.ts:818:20
|
|
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
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
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
Found 2 failed/cancelled steps in previous workflow steps
|
|
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/
|
|
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/
|
Artifacts
Produced during runtime
| Name | Size | Digest | |
|---|---|---|---|
|
integration-reports-migrate
|
67.5 KB |
sha256:5bdabd1a551b75ded4568dde6a55727c048bb827753861201c0244cdb6a74cf1
|
|
|
integration-reports-push
|
68.1 KB |
sha256:05f32296444a4b49144cc426163f7b73f91fa0d32509a35da0bdf000643c3b5f
|
|