fix(uploads): manual file uploads must ignore provenance stamping #25261
icecrasher321ci.yml
on: pull_request
Detect Version
Migrate Dev DB
/
Apply Database Migrations
Detect Desktop Changes
Matrix: Build AMD64
Matrix: Build ARM64 (GHCR Only)
Test and Build
/
Lint and Test
3m 34s
Test and Build
/
Build App
2m 12s
Check Docs Changes
Matrix: Build Dev ECR
Deploy Trigger.dev (Dev)
Matrix: Create GHCR Manifests
Process Docs
/
Process Documentation Embeddings
Prune Desktop Prereleases
0s
Desktop Release
/
Build, Sign, Notarize
Annotations
6 errors and 6 warnings
|
Test and Build / Lint and Test
Process completed with exit code 1.
|
|
app/api/knowledge/[id]/documents/route.test.ts > Knowledge Base Documents API Route > POST /api/knowledge/[id]/documents - Bulk Documents > should create bulk documents successfully:
apps/sim/app/api/knowledge/[id]/documents/route.test.ts#L432
AssertionError: expected "vi.fn()" to be called with arguments: [ [ { …(4) }, { …(4) } ], …(4) ]
Received:
1st vi.fn() call:
@@ -12,32 +12,9 @@
"filename": "doc2.pdf",
"mimeType": "application/pdf",
},
],
"kb-123",
- Any<String>,
+ "mock-uui",
"user-123",
- [
- {
- "content": {
- "entries": [],
- "status": "exact",
- },
- "filename": {
- "entries": [],
- "status": "exact",
- },
- "tags": [],
- },
- {
- "content": {
- "entries": [],
- "status": "exact",
- },
- "filename": {
- "entries": [],
- "status": "exact",
- },
- "tags": [],
- },
- ],
+ undefined,
]
Number of calls: 1
❯ app/api/knowledge/[id]/documents/route.test.ts:432:48
|
|
app/api/knowledge/[id]/documents/route.test.ts > Knowledge Base Documents API Route > POST /api/knowledge/[id]/documents - Single Document > should create single document successfully:
apps/sim/app/api/knowledge/[id]/documents/route.test.ts#L325
AssertionError: expected "vi.fn()" to be called with arguments: [ { …(4) }, 'kb-123', …(3) ]
Received:
1st vi.fn() call:
@@ -4,19 +4,9 @@
"fileUrl": "https://example.com/test-document.pdf",
"filename": "test-document.pdf",
"mimeType": "application/pdf",
},
"kb-123",
- Any<String>,
+ "mock-uui",
"user-123",
- {
- "content": {
- "entries": [],
- "status": "exact",
- },
- "filename": {
- "entries": [],
- "status": "exact",
- },
- "tags": [],
- },
+ undefined,
]
Number of calls: 1
❯ app/api/knowledge/[id]/documents/route.test.ts:325:47
|
|
app/api/knowledge/secret-provenance.test.ts > knowledge write secret provenance > tracks exact-empty provenance only when an internal write supplies a verified envelope:
apps/sim/app/api/knowledge/secret-provenance.test.ts#L107
AssertionError: expected { success: false, …(1) } to deeply equal { success: true, …(1) }
- Expected
+ Received
{
- "provenances": [
- {
- "entries": [],
- "status": "exact",
+ "response": NextResponse {
+ Symbol(internal response): {
+ "cookies": bound ResponseCookies {
+ "_headers": Headers {},
+ "_parsed": Map {},
+ },
+ "url": undefined,
+ },
},
- ],
- "success": true,
+ "success": false,
}
❯ app/api/knowledge/secret-provenance.test.ts:107:20
|
|
Test and Build / Lint and Test
sim#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
|