Skip to content

Commit 858306c

Browse files
committed
pnpm ci fix, fix evals unnecessary changes
1 parent 5759024 commit 858306c

File tree

4 files changed

+16
-11
lines changed

4 files changed

+16
-11
lines changed

.github/workflows/ci.yml

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -19,19 +19,24 @@ jobs:
1919
uses: actions/setup-node@v4
2020
with:
2121
node-version: "22"
22-
cache: "npm"
22+
cache: "pnpm"
23+
24+
- name: Install pnpm
25+
uses: pnpm/action-setup@v4
26+
with:
27+
version: 8
2328

2429
- name: Install dependencies
25-
run: npm ci
30+
run: pnpm install --frozen-lockfile
2631

2732
- name: Run linting
28-
run: npm run lint
33+
run: pnpm lint
2934

3035
- name: Check formatting
31-
run: npm run format
36+
run: pnpm format
3237

3338
- name: Build project
34-
run: npm run build
39+
run: pnpm build
3540

3641
- name: Run evaluation tests
3742
env:
@@ -40,4 +45,4 @@ jobs:
4045
ANTHROPIC_API_KEY: ${{ secrets.ANTHROPIC_API_KEY }}
4146
GEMINI_API_KEY: ${{ secrets.GEMINI_API_KEY }}
4247
run: |
43-
npm run evals
48+
pnpm evals

evals/mcp-eval-minimal.config.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
},
2424
{
2525
"user": "Close the browser",
26-
"expectedState": "session closed"
26+
"expectedState": "closed successfully via Stagehand"
2727
}
2828
],
2929
"expectTools": [
@@ -42,7 +42,7 @@
4242
},
4343
{
4444
"user": "Close the session",
45-
"expectedState": "session closed"
45+
"expectedState": "closed successfully via Stagehand"
4646
}
4747
],
4848
"expectTools": [

src/tools/multiSession.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -227,7 +227,7 @@ export const closeSessionTool = defineTool({
227227
content: [
228228
{
229229
type: "text",
230-
text: `closed session ${sessionId}`,
230+
text: `Closed session ${sessionId}`,
231231
},
232232
],
233233
}),

src/tools/session.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@ async function handleCreateSession(
108108
content: [
109109
{
110110
type: "text",
111-
text: `Browserbase Live Session View URL: https://www.browserbase.com/sessions/${session.sessionId}\nBrowserbase Live Debugger URL: ${debugUrl} \n session created`,
111+
text: `Browserbase Live Session View URL: https://www.browserbase.com/sessions/${session.sessionId}`,
112112
},
113113
{
114114
type: "text",
@@ -224,7 +224,7 @@ async function handleCloseSession(context: Context): Promise<ToolResult> {
224224
}
225225

226226
if (stagehandClosedSuccessfully) {
227-
let successMessage = `Browserbase session (${previousSessionId || "default"}) closed successfully via Stagehand. Context reset to default. session closed`;
227+
let successMessage = `Browserbase session (${previousSessionId || "default"}) closed successfully via Stagehand. Context reset to default.`;
228228
if (browserbaseSessionId && previousSessionId !== defaultSessionId) {
229229
successMessage += ` View replay at https://www.browserbase.com/sessions/${browserbaseSessionId}`;
230230
}

0 commit comments

Comments
 (0)