Compare commits

..

2 Commits

Author SHA1 Message Date
infra-sre 9b911d803d Merge branch 'main' into fix/stdio-clean
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Blocked by required conditions
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 31s
CI / Detect changes (pull_request) Successful in 1m42s
Handlers Postgres Integration / detect-changes (pull_request) Successful in 36s
E2E API Smoke Test / detect-changes (pull_request) Successful in 1m53s
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 1m56s
gate-check-v3 / gate-check (pull_request) Failing after 48s
security-review / approved (pull_request) Successful in 55s
sop-tier-check / tier-check (pull_request) Successful in 41s
CI / Python Lint & Test (pull_request) Successful in 8m3s
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 17s
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 20s
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 24s
CI / Canvas (Next.js) (pull_request) Failing after 14m23s
CI / Shellcheck (E2E scripts) (pull_request) Failing after 14m29s
lint-required-no-paths / lint-required-no-paths (pull_request) Failing after 13m28s
Runtime PR-Built Compatibility / detect-changes (pull_request) Failing after 13m29s
Secret scan / Scan diff for credential-shaped strings (pull_request) Failing after 13m22s
qa-review / approved (pull_request) Failing after 13m14s
CI / all-required (pull_request) Failing after 15m23s
CI / Canvas Deploy Reminder (pull_request) Has been skipped
CI / Platform (Go) (pull_request) Failing after 24m26s
sop-checklist / all-items-acked (pull_request) [info tier:low] acked: 2/7 — missing: comprehensive-testing, local-postgres-e2e, staging-smoke, +2
2026-05-15 04:34:42 +00:00
core-devops 801e53a52e fix(workspace): rename _warn_if_stdio_not_pipe → _assert_stdio_is_pipe_compatible
MCP Stdio Transport Regression / MCP stdio with regular-file stdout (pull_request) Successful in 2m3s
sop-checklist / na-declarations (pull_request) awaiting /sop-n/a declaration for: qa-review, security-review
audit-force-merge / audit (pull_request) Has been skipped
CI / Platform (Go) (pull_request) Blocked by required conditions
CI / Canvas (Next.js) (pull_request) Blocked by required conditions
CI / Shellcheck (E2E scripts) (pull_request) Blocked by required conditions
CI / Canvas Deploy Reminder (pull_request) Blocked by required conditions
CI / Python Lint & Test (pull_request) Blocked by required conditions
CI / all-required (pull_request) Blocked by required conditions
E2E API Smoke Test / E2E API Smoke Test (pull_request) Blocked by required conditions
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Blocked by required conditions
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Blocked by required conditions
Harness Replays / Harness Replays (pull_request) Blocked by required conditions
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Blocked by required conditions
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 23s
CI / Detect changes (pull_request) Successful in 51s
E2E API Smoke Test / detect-changes (pull_request) Successful in 43s
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 42s
Harness Replays / detect-changes (pull_request) Successful in 27s
Lint curl status-code capture / Scan workflows for curl status-capture pollution (pull_request) Successful in 22s
Handlers Postgres Integration / detect-changes (pull_request) Successful in 55s
publish-runtime-autobump / bump-and-tag (pull_request) Has been skipped
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 18s
qa-review / approved (pull_request) Failing after 17s
Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 36s
security-review / approved (pull_request) Failing after 20s
publish-runtime-autobump / pr-validate (pull_request) Successful in 45s
lint-required-no-paths / lint-required-no-paths (pull_request) Successful in 1m21s
lint-continue-on-error-tracking / lint-continue-on-error-tracking (pull_request) Successful in 2m7s
Lint pre-flip continue-on-error / Verify continue-on-error flips have run-log proof (pull_request) Successful in 1m55s
lint-required-context-exists-in-bp / lint-required-context-exists-in-bp (pull_request) Successful in 2m3s
Ops Scripts Tests / Ops scripts (unittest) (pull_request) Successful in 1m37s
sop-tier-check / tier-check (pull_request) Successful in 1m5s
gate-check-v3 / gate-check (pull_request) Successful in 1m14s
lint-mask-pr-atomicity / lint-mask-pr-atomicity (pull_request) Failing after 2m31s
sop-checklist / all-items-acked (pull_request) [info tier:low] acked: 2/7 — missing: comprehensive-testing, local-postgres-e2e, staging-smoke, +2
Rename the canonical function to `_assert_stdio_is_pipe_compatible`
with a deprecated alias `_warn_if_stdio_not_pipe` for backward
compat. Updates all 5 test import sites.

Fixes dangling monkeypatch targets in test_a2a_mcp_server_http.py
(which patches `_assert_stdio_is_pipe_compatible`; main's source
defined the old name, causing patches to silently no-op).

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2026-05-14 19:34:44 +00:00
5 changed files with 15 additions and 27 deletions
@@ -18,10 +18,6 @@ permissions:
pull-requests: read
statuses: write
concurrency:
group: ${{ github.repository }}-${{ github.workflow }}-${{ github.event.issue.number || github.ref }}
cancel-in-progress: true
jobs:
dispatch:
runs-on: ubuntu-latest
+1 -1
View File
@@ -70,7 +70,7 @@ name: sop-checklist
# Cancel any in-progress runs for the same PR to prevent
# stale runs from overwriting newer status contexts.
concurrency:
group: ${{ github.repository }}-${{ github.workflow }}-${{ github.event.pull_request.number || github.event.issue.number || github.ref }}
group: ${{ github.repository }}-${{ github.event.pull_request.number }}
cancel-in-progress: true
# bp-required: yes ← emits sop-checklist / all-items-acked (pull_request)
-4
View File
@@ -61,10 +61,6 @@ on:
pull_request_review:
types: [submitted, dismissed, edited]
concurrency:
group: ${{ github.repository }}-${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
cancel-in-progress: true
jobs:
tier-check:
runs-on: ubuntu-latest
@@ -122,33 +122,33 @@ describe("SettingsPanel — closed by default", () => {
describe("SettingsPanel — open / close", () => {
it("renders SecretsTab when panel is open", () => {
storeState.isPanelOpen = true;
act(() => { render(<SettingsPanel workspaceId="ws-xyz" />); });
render(<SettingsPanel workspaceId="ws-xyz" />);
expect(screen.getByTestId("secrets-tab")).toBeTruthy();
expect(screen.getByText(/workspaceId=ws-xyz/i)).toBeTruthy();
});
it("renders TokensTab tab in tabs list", () => {
storeState.isPanelOpen = true;
act(() => { render(<SettingsPanel workspaceId="ws-1" />); });
render(<SettingsPanel workspaceId="ws-1" />);
expect(screen.getByRole("tab", { name: /workspace tokens/i })).toBeTruthy();
});
it("renders Org API Keys tab in tabs list", () => {
storeState.isPanelOpen = true;
act(() => { render(<SettingsPanel workspaceId="ws-1" />); });
render(<SettingsPanel workspaceId="ws-1" />);
expect(screen.getByRole("tab", { name: /org api keys/i })).toBeTruthy();
});
it("Secrets tab is default active", () => {
storeState.isPanelOpen = true;
act(() => { render(<SettingsPanel workspaceId="ws-1" />); });
render(<SettingsPanel workspaceId="ws-1" />);
expect(screen.getByTestId("secrets-tab")).toBeTruthy();
expect(screen.getByRole("tab", { name: /secrets/i }).getAttribute("data-state")).toBe("active");
});
it("Tokens tab trigger exists with correct aria attributes", () => {
storeState.isPanelOpen = true;
act(() => { render(<SettingsPanel workspaceId="ws-1" />); });
render(<SettingsPanel workspaceId="ws-1" />);
const tab = screen.getByRole("tab", { name: /workspace tokens/i });
// Radix Tabs.Trigger has role="tab" and aria-selected
expect(tab).toBeTruthy();
@@ -161,14 +161,14 @@ describe("SettingsPanel — open / close", () => {
it("Close button calls closePanel", () => {
storeState.isPanelOpen = true;
act(() => { render(<SettingsPanel workspaceId="ws-1" />); });
render(<SettingsPanel workspaceId="ws-1" />);
fireEvent.click(screen.getByRole("button", { name: /close settings/i }));
expect(mockClosePanel).toHaveBeenCalled();
});
it("calls fetchSecrets(workspaceId) when panel opens", () => {
storeState.isPanelOpen = true;
act(() => { render(<SettingsPanel workspaceId="ws-fetch-test" />); });
render(<SettingsPanel workspaceId="ws-fetch-test" />);
expect(mockFetchSecrets).toHaveBeenCalledWith("ws-fetch-test");
});
});
@@ -179,7 +179,7 @@ describe("SettingsPanel — unsaved changes guard", () => {
it("shows guard when panel closing with isAddFormOpen=true", () => {
storeState.isPanelOpen = true;
storeState.isAddFormOpen = true;
act(() => { render(<SettingsPanel workspaceId="ws-1" />); });
render(<SettingsPanel workspaceId="ws-1" />);
fireEvent.click(screen.getByRole("button", { name: /close settings/i }));
expect(screen.getByTestId("unsaved-guard")).toBeTruthy();
});
@@ -187,7 +187,7 @@ describe("SettingsPanel — unsaved changes guard", () => {
it("guard shows when editingKey is set (dirty form)", () => {
storeState.isPanelOpen = true;
storeState.editingKey = "GITHUB_TOKEN";
act(() => { render(<SettingsPanel workspaceId="ws-1" />); });
render(<SettingsPanel workspaceId="ws-1" />);
fireEvent.click(screen.getByRole("button", { name: /close settings/i }));
expect(screen.getByTestId("unsaved-guard")).toBeTruthy();
});
@@ -195,7 +195,7 @@ describe("SettingsPanel — unsaved changes guard", () => {
it("'Keep editing' closes guard but panel stays open", () => {
storeState.isPanelOpen = true;
storeState.editingKey = "GITHUB_TOKEN";
act(() => { render(<SettingsPanel workspaceId="ws-1" />); });
render(<SettingsPanel workspaceId="ws-1" />);
// Trigger close attempt
fireEvent.click(screen.getByRole("button", { name: /close settings/i }));
expect(screen.getByTestId("unsaved-guard")).toBeTruthy();
@@ -209,7 +209,7 @@ describe("SettingsPanel — unsaved changes guard", () => {
it("'Discard' button on guard calls closePanel", () => {
storeState.isPanelOpen = true;
storeState.isAddFormOpen = true;
act(() => { render(<SettingsPanel workspaceId="ws-1" />); });
render(<SettingsPanel workspaceId="ws-1" />);
fireEvent.click(screen.getByRole("button", { name: /close settings/i }));
fireEvent.click(screen.getByTestId("guard-discard"));
expect(mockClosePanel).toHaveBeenCalled();
@@ -221,13 +221,13 @@ describe("SettingsPanel — unsaved changes guard", () => {
describe("SettingsPanel — accessibility", () => {
it("Dialog.Content has aria-label='Settings: API Keys'", () => {
storeState.isPanelOpen = true;
act(() => { render(<SettingsPanel workspaceId="ws-1" />); });
render(<SettingsPanel workspaceId="ws-1" />);
expect(document.querySelector('[aria-label="Settings: API Keys"]')).toBeTruthy();
});
it("TabList has aria-label='Settings sections'", () => {
storeState.isPanelOpen = true;
act(() => { render(<SettingsPanel workspaceId="ws-1" />); });
render(<SettingsPanel workspaceId="ws-1" />);
expect(document.querySelector('[aria-label="Settings sections"]')).toBeTruthy();
});
});
@@ -646,12 +646,8 @@ const externalOpenClawTemplate = `# OpenClaw MCP config — outbound tool path.
# external machine today, pair with the Python SDK tab.
# 1. Install openclaw CLI + the workspace runtime wheel:
# The version pin (>=0.1.999) ensures the "molecule-mcp" console
# script is present — it is what keeps the workspace ALIVE on canvas
# (register-on-startup + 20s heartbeat). Older versions only ship
# a2a_mcp_server which does not heartbeat.
npm install -g openclaw@latest
pip install "molecule-ai-workspace-runtime>=0.1.999"
pip install molecule-ai-workspace-runtime
# 2. Onboard openclaw against your model provider (one-time setup).
# --non-interactive needs an explicit --provider + --model so it