Skip to content

Commit e45355d

Browse files
patnikoCopilot
andcommitted
fix: add Windows glob ordering variant to snapshot
On Windows, glob returns files in alphabetical order (app.ts before index.ts) while macOS/Linux return them differently. Add a second conversation variant to handle both orderings. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 0f4fc97 commit e45355d

1 file changed

Lines changed: 32 additions & 0 deletions

File tree

test/snapshots/builtin_tools/should_find_files_by_pattern.yaml

Lines changed: 32 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,3 +53,35 @@ conversations:
5353
Found 2 TypeScript files:
5454
- `src/app.ts`
5555
- `src/index.ts`
56+
# Windows returns files in alphabetical order
57+
- messages:
58+
- role: system
59+
content: ${system}
60+
- role: user
61+
content: Find all .ts files in this directory (recursively). List the filenames you found.
62+
- role: assistant
63+
content: I'll search for all TypeScript files recursively in the current directory.
64+
tool_calls:
65+
- id: toolcall_0
66+
type: function
67+
function:
68+
name: report_intent
69+
arguments: '{"intent":"Finding TypeScript files"}'
70+
- id: toolcall_1
71+
type: function
72+
function:
73+
name: glob
74+
arguments: '{"pattern":"**/*.ts"}'
75+
- role: tool
76+
tool_call_id: toolcall_0
77+
content: Intent logged
78+
- role: tool
79+
tool_call_id: toolcall_1
80+
content: |-
81+
./src/app.ts
82+
./src/index.ts
83+
- role: assistant
84+
content: |-
85+
Found 2 TypeScript files:
86+
- `src/app.ts`
87+
- `src/index.ts`

0 commit comments

Comments
 (0)