mirror of
https://github.com/Sea-Haven-Industries/open-swe.git
synced 2026-10-01 09:43:14 +00:00
b2b49ed944
* feat: add Slack breakout thread tool
Co-authored-by: open-swe[bot] <open-swe@users.noreply.github.com>
* chore: make fake LLM scripts declarative
Co-authored-by: open-swe[bot] <open-swe@users.noreply.github.com>
* fix: exclude slack_start_new_thread from plan mode
The breakout tool can dispatch a fresh agent run that starts outside the
current plan-mode state, bypassing the approval flow. Add it to
PLAN_MODE_EXCLUDED_TOOLS so it's hidden alongside the other mutating
tools while planning.
---------
Co-authored-by: Ramon Nogueira <270434257+ramon-langchain@users.noreply.github.com>
Co-authored-by: open-swe[bot] <open-swe@users.noreply.github.com>
(cherry picked from commit 747ce4bbe5)
73 lines
3.4 KiB
TypeScript
73 lines
3.4 KiB
TypeScript
import { test, expect } from "@playwright/test";
|
|
|
|
// Full happy path, driven entirely through the mock Slack + GitHub UIs:
|
|
// user asks in Slack -> real agent implements in a local sandbox -> opens a PR
|
|
// on the fake GitHub -> replies with the PR link in the SAME Slack thread.
|
|
// Only the LLM is faked; all agent code runs for real via langgraph dev.
|
|
test.describe("Open SWE full flow", () => {
|
|
test.beforeEach(async ({ page }) => {
|
|
await page.goto("/mock/slack");
|
|
await page.locator("#reset").click();
|
|
await expect(page.locator("#thread")).toContainText("No messages yet");
|
|
});
|
|
|
|
test("Slack request → implements → opens PR → links it back in the thread", async ({ page }) => {
|
|
await page.locator("#text").fill("<@U0BOT> please add a greet() helper and open a PR");
|
|
await page.locator("#send").click();
|
|
|
|
// The user's message lands in the thread.
|
|
await expect(page.locator(".msg").filter({ hasText: "add a greet() helper" })).toBeVisible();
|
|
|
|
// The agent replies in the SAME thread with a link to the PR it opened.
|
|
const reply = page.locator(".msg.bot").filter({ hasText: "Add greet() helper" });
|
|
await expect(reply).toBeVisible();
|
|
const prLink = reply.locator('a[href*="/pull/"]');
|
|
await expect(prLink).toBeVisible();
|
|
|
|
// Follow the link → the PR page shows what GitHub would show.
|
|
await prLink.click();
|
|
await expect(page.locator("#pr-title")).toContainText("Add greet() helper");
|
|
await expect(page.locator("#pr-state")).toHaveText("open");
|
|
await expect(page.locator("#pr-head")).toHaveText("add-greet");
|
|
await expect(page.locator('#pr-files li[data-file="greet.py"]')).toBeVisible();
|
|
|
|
// The PR list view shows it too.
|
|
await page.goto("/mock/github");
|
|
await expect(page.locator('.pr[data-pr="1"]')).toContainText("Add greet() helper");
|
|
await expect(page.locator('.pr[data-pr="1"]')).toContainText("greet.py");
|
|
});
|
|
|
|
test("Slack breakout request starts a new top-level Open SWE thread", async ({ page }) => {
|
|
await page.locator("#text").fill("<@U0BOT> please break out adding a greet() helper into a separate thread");
|
|
await page.locator("#send").click();
|
|
|
|
const breakout = page
|
|
.locator(".msg.bot")
|
|
.filter({ hasText: /Open SWE breakout thread:\* Add greet\(\) helper/ });
|
|
await expect(breakout).toBeVisible({ timeout: 60_000 });
|
|
const breakoutThreadTs = await breakout.getAttribute("data-thread-ts");
|
|
expect(breakoutThreadTs).toBeTruthy();
|
|
|
|
const breakoutThreadMessages = page.locator(`.msg.bot[data-thread-ts="${breakoutThreadTs}"]`);
|
|
await expect(breakoutThreadMessages.locator('a[href*="/agents/"]')).toBeVisible({
|
|
timeout: 60_000,
|
|
});
|
|
await expect(
|
|
page.locator(".msg.bot").filter({ hasText: "I started a separate Open SWE thread" }),
|
|
).toBeVisible({ timeout: 60_000 });
|
|
});
|
|
|
|
test("a message that does not mention the bot produces no run and no PR", async ({ page }) => {
|
|
await page.locator("#mention").uncheck();
|
|
await page.locator("#text").fill("just chatting with the team, nothing for the bot");
|
|
await page.locator("#send").click();
|
|
|
|
await expect(page.locator(".msg").filter({ hasText: "just chatting" })).toBeVisible();
|
|
// No agent activity: give the (non-)run a moment, then assert nothing came back.
|
|
await page.waitForTimeout(3000);
|
|
await expect(page.locator(".msg.bot")).toHaveCount(0);
|
|
|
|
const prs = await (await page.request.get("/mock/github/data")).json();
|
|
expect(prs.length).toBe(0);
|
|
});
|
|
});
|