fix(trace): normalize keyed tool names and show credits in trace view #15483
waleedlatif1ci.yml
on: pull_request
Detect Version
0s
Test and Build
/
Test and Build
1m 29s
Check Docs Changes
0s
Matrix: Build Dev ECR
Matrix: Build ARM64 (GHCR Only)
Matrix: Build AMD64
Migrate Dev DB
/
Apply Database Migrations
Matrix: Create GHCR Manifests
Process Docs
/
Process Documentation Embeddings
Create GitHub Release
0s
Annotations
10 errors and 4 warnings
|
tools/index.test.ts > Copilot OAuth Credential Enforcement > fails fast when copilot executes an oauth tool without an explicit credential selector:
apps/sim/tools/index.test.ts#L1080
AssertionError: expected 'normalizeToolId is not defined' to contain 'credentialId'
Expected: "credentialId"
Received: "normalizeToolId is not defined"
❯ tools/index.test.ts:1080:26
|
|
tools/index.test.ts > Copilot File Parameter Normalization > does not resolve file params outside copilot execution:
apps/sim/tools/index.test.ts#L1050
AssertionError: expected false to be true // Object.is equality
- Expected
+ Received
- true
+ false
❯ tools/index.test.ts:1050:28
|
|
tools/index.test.ts > Copilot File Parameter Normalization > resolves file-array params from strings and partial file objects, while preserving full file objects:
apps/sim/tools/index.test.ts#L1016
AssertionError: expected false to be true // Object.is equality
- Expected
+ Received
- true
+ false
❯ tools/index.test.ts:1016:28
|
|
tools/index.test.ts > Copilot File Parameter Normalization > resolves canonical file IDs for single-file params during copilot execution:
apps/sim/tools/index.test.ts#L926
AssertionError: expected false to be true // Object.is equality
- Expected
+ Received
- true
+ false
❯ tools/index.test.ts:926:28
|
|
tools/index.test.ts > Automatic Internal Route Detection > should handle dynamic URLs that resolve to internal routes:
apps/sim/tools/index.test.ts#L764
AssertionError: expected false to be true // Object.is equality
- Expected
+ Received
- true
+ false
❯ tools/index.test.ts:764:28
|
|
tools/index.test.ts > Automatic Internal Route Detection > should detect internal routes (URLs starting with /api/) and call them directly:
apps/sim/tools/index.test.ts#L662
AssertionError: expected false to be true // Object.is equality
- Expected
+ Received
- true
+ false
❯ tools/index.test.ts:662:28
|
|
tools/index.test.ts > executeTool Function > should handle non-existent tool:
apps/sim/tools/index.test.ts#L575
AssertionError: expected 'normalizeToolId is not defined' to contain 'Tool not found'
Expected: "Tool not found"
Received: "normalizeToolId is not defined"
❯ tools/index.test.ts:575:26
|
|
tools/index.test.ts > executeTool Function > should call internal routes directly:
apps/sim/tools/index.test.ts#L563
AssertionError: expected "spy" to be called with arguments: [ StringContaining{…}, Anything ]
Number of calls: 0
❯ tools/index.test.ts:563:26
|
|
tools/index.test.ts > executeTool Function > should execute a tool successfully:
apps/sim/tools/index.test.ts#L532
AssertionError: expected false to be true // Object.is equality
- Expected
+ Received
- true
+ false
❯ tools/index.test.ts:532:28
|
|
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
Skipping sticky disk commit due to previous step failures
|
|
Test and Build / Test and Build
Found 2 failed/cancelled steps in previous workflow steps
|
|
Test and Build / Test and Build
Skipping sticky disk commit due to previous step failures
|
|
Test and Build / Test and Build
Found 2 failed/cancelled steps in previous workflow steps
|