fix(change-detection): move change detection logic to client-side to prevent unnecessary API calls, consolidate utils #7584
waleedlatif1ci.yml
on: pull_request
Detect Version
Test and Build
/
Test and Build
42s
Check Docs Changes
Matrix: Build AMD64
Matrix: Build ARM64 (GHCR Only)
Process Docs
/
Process Documentation Embeddings
Matrix: Create GHCR Manifests
Annotations
6 errors and 1 warning
|
Test and Build / Test and Build
Process completed with exit code 1.
|
|
lib/workflows/comparison/normalize.test.ts > Workflow Normalization Utilities > sanitizeTools > should preserve all other fields:
apps/sim/lib/workflows/comparison/normalize.test.ts#L339
AssertionError: expected { id: 'tool1', …(5) } to deeply equal { id: 'tool1', …(4) }
- Expected
+ Received
@@ -1,7 +1,8 @@
{
"id": "tool1",
+ "isExpanded": true,
"name": "Complex Tool",
"nested": {
"deep": {
"value": 123,
},
❯ lib/workflows/comparison/normalize.test.ts:339:25
|
|
lib/workflows/comparison/normalize.test.ts > Workflow Normalization Utilities > sanitizeTools > should remove isExpanded field from tools:
apps/sim/lib/workflows/comparison/normalize.test.ts#L319
AssertionError: expected [ { id: 'tool1', …(2) }, …(2) ] to deeply equal [ …(3) ]
- Expected
+ Received
@@ -1,12 +1,14 @@
[
{
"id": "tool1",
+ "isExpanded": true,
"name": "Search",
},
{
"id": "tool2",
+ "isExpanded": false,
"name": "Calculator",
},
{
"id": "tool3",
"name": "Weather",
❯ lib/workflows/comparison/normalize.test.ts:319:22
|
|
lib/workflows/comparison/compare.test.ts > hasWorkflowChanged > Tool Input Scenarios > should not detect change when tool isExpanded toggles:
apps/sim/lib/workflows/comparison/compare.test.ts#L1287
AssertionError: expected true to be false // Object.is equality
- Expected
+ Received
- false
+ true
❯ lib/workflows/comparison/compare.test.ts:1287:63
|
|
lib/workflows/comparison/compare.test.ts > hasWorkflowChanged > Tools SubBlock Special Handling > should ignore isExpanded field in tools:
apps/sim/lib/workflows/comparison/compare.test.ts#L493
AssertionError: expected true to be false // Object.is equality
- Expected
+ Received
- false
+ true
❯ lib/workflows/comparison/compare.test.ts:493:50
|
|
Test and Build / Test and Build
sim#test: command (/home/runner/_work/sim/sim/apps/sim) /home/runner/.bun/bin/bun run test exited (1)
|
|
Test and Build / Test and Build
Failed to restore: "/usr/bin/tar" failed with error: The process '/usr/bin/tar' failed with exit code 2
|