fix(knowledge): read an unfilled keyword candidate's source from its document #32631
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
Matrix: Test and Build / test-shard
Test and Build
/
Lint and Test
2m 48s
Test and Build
/
Build App
3m 20s
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 4 warnings
|
Test and Build / Lint and Test
Process completed with exit code 1.
|
|
Test and Build / PostgreSQL integration (push)
Process completed with exit code 1.
|
|
lib/knowledge/connectors/sync-persistence.postgres.test.ts > persistDocumentAcls in PostgreSQL > writes each document once when one page mixes unchanged and changed ACLs:
apps/sim/lib/knowledge/connectors/sync-persistence.postgres.test.ts#L161
AssertionError: expected [ null, null, null, null ] to deeply equal [ [ 'u:alice@corp.com', …(1) ], …(3) ]
- Expected
+ Received
[
- [
- "u:alice@corp.com",
- "u:bob@corp.com",
- ],
+ null,
+ null,
null,
- [
- "u:alice@corp.com",
- "u:bob@corp.com",
- ],
null,
]
❯ lib/knowledge/connectors/sync-persistence.postgres.test.ts:161:7
|
|
lib/knowledge/connectors/sync-persistence.postgres.test.ts > persistDocumentAcls in PostgreSQL > propagates a changed ACL to every chunk projection row, filled or not:
apps/sim/lib/knowledge/connectors/sync-persistence.postgres.test.ts#L122
AssertionError: expected [ …(4) ] to deeply equal [ …(4) ]
- Expected
+ Received
@@ -4,23 +4,19 @@
"u:bob@corp.com",
],
"id": "kw-moved-filled",
},
{
- "acl": [
- "u:bob@corp.com",
- ],
+ "acl": null,
"id": "kw-moved-unfilled",
},
{
"acl": [
"u:bob@corp.com",
],
"id": "vec-moved-filled",
},
{
- "acl": [
- "u:bob@corp.com",
- ],
+ "acl": null,
"id": "vec-moved-unfilled",
},
]
❯ lib/knowledge/connectors/sync-persistence.postgres.test.ts:122:77
|
|
Test and Build / PostgreSQL integration (migrate)
Process completed with exit code 1.
|
|
lib/knowledge/connectors/sync-persistence.postgres.test.ts > persistDocumentAcls in PostgreSQL > writes each document once when one page mixes unchanged and changed ACLs:
apps/sim/lib/knowledge/connectors/sync-persistence.postgres.test.ts#L161
AssertionError: expected [ null, null, null, null ] to deeply equal [ [ 'u:alice@corp.com', …(1) ], …(3) ]
- Expected
+ Received
[
- [
- "u:alice@corp.com",
- "u:bob@corp.com",
- ],
+ null,
+ null,
null,
- [
- "u:alice@corp.com",
- "u:bob@corp.com",
- ],
null,
]
❯ lib/knowledge/connectors/sync-persistence.postgres.test.ts:161:7
|
|
lib/knowledge/connectors/sync-persistence.postgres.test.ts > persistDocumentAcls in PostgreSQL > propagates a changed ACL to every chunk projection row, filled or not:
apps/sim/lib/knowledge/connectors/sync-persistence.postgres.test.ts#L122
AssertionError: expected [ …(4) ] to deeply equal [ …(4) ]
- Expected
+ Received
@@ -4,23 +4,19 @@
"u:bob@corp.com",
],
"id": "kw-moved-filled",
},
{
- "acl": [
- "u:bob@corp.com",
- ],
+ "acl": null,
"id": "kw-moved-unfilled",
},
{
"acl": [
"u:bob@corp.com",
],
"id": "vec-moved-filled",
},
{
- "acl": [
- "u:bob@corp.com",
- ],
+ "acl": null,
"id": "vec-moved-unfilled",
},
]
❯ lib/knowledge/connectors/sync-persistence.postgres.test.ts:122:77
|
|
Test and Build / PostgreSQL integration (push)
Skipping sticky disk commit due to previous step failures
|
|
Test and Build / PostgreSQL integration (push)
Found 2 failed/cancelled steps in previous workflow steps
|
|
Test and Build / PostgreSQL integration (migrate)
Skipping sticky disk commit due to previous step failures
|
|
Test and Build / PostgreSQL integration (migrate)
Found 2 failed/cancelled steps in previous workflow steps
|