openclaw - ✅(Solved) Fix ACPx plugin-tools MCP config test expects source path but resolver returns dist path [3 pull requests, 1 comments, 2 participants]

100yenadmin · 2026-05-10T21:30:48Z

[openclaw] PR 80323: qa-lab Complete Codex vs Pi runtime parity harness phases 2-5 - Repository: openclaw/openclaw - Author: 100yenadmin - State: open | merged… # PR #80323: [qa-lab] Complete Codex vs Pi runtime parity harness phases 2-5 - Repository: openclaw/openclaw - Author: 100yenadmin - State: open | merged: False - Link: https://github.com/openclaw/openclaw/pull/80323 ## Description (problem / solution / changelog) ## Summary Adds the Codex-vs-Pi runtime parity QA harness across `extensions/qa-lab`, including runtime-pair execution, first-hour/depth suite selectors, harness-prompt parity, token-efficiency reporting, tool-default fixtures, JSONL replay scaffolding, and release-check wiring. This update also corrects the tool-defaults mock lane so the harness matches Codex app-server architecture: - Codex-native workspace tools (`read`, `write`, `edit`, `apply_patch`, `exec`, `process`, `update_plan`) are no longer expected to appear as duplicate OpenClaw dynamic tools. - OpenClaw integration tools (`image_generate`, sessions, web, etc.) remain dynamic-tool parity rows and are tracked separately from Codex-native behavior rows. - Optional/profile/plugin-dependent tools stay report-only unless explicitly enabled. - Mock provider planned tool calls are captured as provider-plan diagnostics, not as runtime transcript tool evidence. - Tool coverage reports now show bucket, expected layer, required/report-only status, product impact, QA impact, and action. ## Why OpenClaw needs a maintainer-runnable gate that compares the same scenario/model under Pi and Codex before Codex becomes the default runtime. The gate must surface real runtime drift without turning mock-provider limitations or intentional Codex-native tool ownership into production bug reports. ## Verification Passing targeted/current-scope checks: - `pnpm test extensions/qa-lab/src/runtime-tool-fixture.test.ts extensions/qa-lab/src/runtime-parity.test.ts extensions/qa-lab/src/tool-coverage-report.test.ts extensions/qa-lab/src/runtime-suite.test.ts extensions/qa-lab/src/suite.test.ts extensions/qa-lab/src/scenario-catalog.test.ts extensions/qa-lab/src/cli.runtime.test.ts extensions/qa-lab/src/cli.test.ts` - `pnpm tsgo:extensions:test` - `pnpm check:test-types` - `git diff --check` ## Real Behavior Proof - **Behavior or issue addressed:** Corrects the runtime parity tool-defaults harness so Codex-native workspace tools are no longer falsely required as duplicate OpenClaw dynamic tools, while OpenClaw dynamic integration rows remain visible and tracked. - **Real environment tested:** Local OpenClaw checkout at `/Volumes/LEXAR/repos/openclaw-1` on branch `codex-vs-pi-runtime-parity-tools`, running the real `pnpm openclaw qa` CLI against the embedded gateway and mock OpenAI provider after this patch. - **Exact steps or command run after this patch:** ```bash OPENCLAW_BUILD_PRIVATE_QA=1 pnpm openclaw qa suite --repo-root . --provider-mode mock-openai --runtime-suite tool-defaults --runtime-pair pi,codex --output-dir .artifacts/qa-e2e/runtime-tools-correction pnpm openclaw qa tool-coverage --repo-root . --summary .artifacts/qa-e2e/runtime-tools-correction/qa-suite-summary.json --runtime-pair pi,codex --output .artifacts/qa-e2e/runtime-tools-correction/qa-tool-coverage-report.md OPENCLAW_BUILD_PRIVATE_QA=1 pnpm openclaw qa suite --repo-root . --provider-mode mock-openai --runtime-suite openclaw-dynamic-tools --runtime-pair pi,codex --output-dir .artifacts/qa-e2e/openclaw-dynamic-tools-correction pnpm openclaw qa parity-report --repo-root . --runtime-axis --summary .artifacts/qa-e2e/runtime-tools-correction/qa-suite-summary.json --output-dir .artifacts/qa-e2e/runtime-tools-correction/parity --token-efficiency ``` - **Evidence after fix:** Terminal output produced these real local artifacts: `.artifacts/qa-e2e/runtime-tools-correction/qa-suite-summary.json`, `.artifacts/qa-e2e/runtime-tools-correction/qa-suite-report.md`, `.artifacts/qa-e2e/runtime-tools-correction/qa-tool-coverage-report.md`, `.artifacts/qa-e2e/openclaw-dynamic-tools-correction/qa-suite-summary.json`, and `.artifacts/qa-e2e/runtime-tools-correction/parity/qa-runtime-token-efficiency-report.md`. - **Observed result after fix:** `tool-defaults` completed with 20 scenarios, 15 pass, 5 report-only skip, 0 fail. Tool coverage verdict was `pass` with 13 required tools, 8 Codex-native workspace tools, 5 OpenClaw dynamic integration tools, 7 optional/profile/plugin tools, and 0 failing tools. The focused `openclaw-dynamic-tools` suite completed with 5 report-only rows tracked under #80319. Token efficiency report verdict was `pass` with usage source `mock-estimate`. - **What was not tested:** Live frontier token-efficiency proof was not completed because local direct OpenAI auth is missing; optional scheduled/Testbox `soak-100` proof was not completed; broad `first-hour-20` remains red and is tracked in #80434. ## Known Broad/Latest Blockers - First `first-hour-20` attempt hit a pre-suite `tsdown`

openclaw2026-05-10 21:30:48

ON THIS PAGE

Recommended Tools

×6

Utilities matched from this issue’s tags and category — try them while you read without losing context.

GitHub issue graph ai analysis

Paste a GitHub issue URL. We fetch that issue, discover linked issues from bodies/comments/timeline, collect linked pull requests, and produce a structured English report.

The report is written in English Markdown for sharing and archival.

GitHub issue URL

Helpful · Quick feedback

GitHub stats

openclaw/openclaw#80431•Fetched 2026-05-11 03:14:45

View on GitHub

Comments

Participants

Timeline

Reactions

Author

100yenadmin

Participants

100yenadmin

clawsweeper[bot]

Timeline (top)

cross-referenced ×4commented ×1mentioned ×1subscribed ×1

Root Cause

pnpm test currently fails one ACPx config test because the resolved built-in plugin-tools MCP server points at the built dist server, while the test expects the source tsx loader path.

Fix Action

Fixed

Fixed by PR: [qa-lab] Complete Codex vs Pi runtime parity harness phases 2-5 (https://github.com/openclaw/openclaw/pull/80323)
Fixed by PR: test(acpx): accept built-dist MCP server resolution when dist exists (https://github.com/openclaw/openclaw/pull/80466)
Fixed by PR: test(acpx): accept built-dist MCP server resolution when dist exists (https://github.com/openclaw/openclaw/pull/80475)

PR fix notes

PR #80323: [qa-lab] Complete Codex vs Pi runtime parity harness phases 2-5

Repository: openclaw/openclaw
Author: 100yenadmin
State: open | merged: False
Link: https://github.com/openclaw/openclaw/pull/80323

Description (problem / solution / changelog)

Summary

Adds the Codex-vs-Pi runtime parity QA harness across extensions/qa-lab, including runtime-pair execution, first-hour/depth suite selectors, harness-prompt parity, token-efficiency reporting, tool-default fixtures, JSONL replay scaffolding, and release-check wiring.

This update also corrects the tool-defaults mock lane so the harness matches Codex app-server architecture:

Codex-native workspace tools (read, write, edit, apply_patch, exec, process, update_plan) are no longer expected to appear as duplicate OpenClaw dynamic tools.
OpenClaw integration tools (image_generate, sessions, web, etc.) remain dynamic-tool parity rows and are tracked separately from Codex-native behavior rows.
Optional/profile/plugin-dependent tools stay report-only unless explicitly enabled.
Mock provider planned tool calls are captured as provider-plan diagnostics, not as runtime transcript tool evidence.
Tool coverage reports now show bucket, expected layer, required/report-only status, product impact, QA impact, and action.

Why

OpenClaw needs a maintainer-runnable gate that compares the same scenario/model under Pi and Codex before Codex becomes the default runtime. The gate must surface real runtime drift without turning mock-provider limitations or intentional Codex-native tool ownership into production bug reports.

Verification

Passing targeted/current-scope checks:

pnpm test extensions/qa-lab/src/runtime-tool-fixture.test.ts extensions/qa-lab/src/runtime-parity.test.ts extensions/qa-lab/src/tool-coverage-report.test.ts extensions/qa-lab/src/runtime-suite.test.ts extensions/qa-lab/src/suite.test.ts extensions/qa-lab/src/scenario-catalog.test.ts extensions/qa-lab/src/cli.runtime.test.ts extensions/qa-lab/src/cli.test.ts
pnpm tsgo:extensions:test
pnpm check:test-types
git diff --check

Real Behavior Proof

Behavior or issue addressed: Corrects the runtime parity tool-defaults harness so Codex-native workspace tools are no longer falsely required as duplicate OpenClaw dynamic tools, while OpenClaw dynamic integration rows remain visible and tracked.
Real environment tested: Local OpenClaw checkout at /Volumes/LEXAR/repos/openclaw-1 on branch codex-vs-pi-runtime-parity-tools, running the real pnpm openclaw qa CLI against the embedded gateway and mock OpenAI provider after this patch.
Exact steps or command run after this patch:

OPENCLAW_BUILD_PRIVATE_QA=1 pnpm openclaw qa suite --repo-root . --provider-mode mock-openai --runtime-suite tool-defaults --runtime-pair pi,codex --output-dir .artifacts/qa-e2e/runtime-tools-correction
pnpm openclaw qa tool-coverage --repo-root . --summary .artifacts/qa-e2e/runtime-tools-correction/qa-suite-summary.json --runtime-pair pi,codex --output .artifacts/qa-e2e/runtime-tools-correction/qa-tool-coverage-report.md
OPENCLAW_BUILD_PRIVATE_QA=1 pnpm openclaw qa suite --repo-root . --provider-mode mock-openai --runtime-suite openclaw-dynamic-tools --runtime-pair pi,codex --output-dir .artifacts/qa-e2e/openclaw-dynamic-tools-correction
pnpm openclaw qa parity-report --repo-root . --runtime-axis --summary .artifacts/qa-e2e/runtime-tools-correction/qa-suite-summary.json --output-dir .artifacts/qa-e2e/runtime-tools-correction/parity --token-efficiency

Evidence after fix: Terminal output produced these real local artifacts: .artifacts/qa-e2e/runtime-tools-correction/qa-suite-summary.json, .artifacts/qa-e2e/runtime-tools-correction/qa-suite-report.md, .artifacts/qa-e2e/runtime-tools-correction/qa-tool-coverage-report.md, .artifacts/qa-e2e/openclaw-dynamic-tools-correction/qa-suite-summary.json, and .artifacts/qa-e2e/runtime-tools-correction/parity/qa-runtime-token-efficiency-report.md.
Observed result after fix: tool-defaults completed with 20 scenarios, 15 pass, 5 report-only skip, 0 fail. Tool coverage verdict was pass with 13 required tools, 8 Codex-native workspace tools, 5 OpenClaw dynamic integration tools, 7 optional/profile/plugin tools, and 0 failing tools. The focused openclaw-dynamic-tools suite completed with 5 report-only rows tracked under #80319. Token efficiency report verdict was pass with usage source mock-estimate.
What was not tested: Live frontier token-efficiency proof was not completed because local direct OpenAI auth is missing; optional scheduled/Testbox soak-100 proof was not completed; broad first-hour-20 remains red and is tracked in #80434.

Known Broad/Latest Blockers

First first-hour-20 attempt hit a pre-suite tsdown SIGSEGV; retry reached QA.
OPENCLAW_BUILD_PRIVATE_QA=1 pnpm openclaw qa suite --repo-root . --provider-mode mock-openai --runtime-suite first-hour-20 --runtime-pair pi,codex --output-dir .artifacts/qa-e2e/first-hour-20-correction-retry is not green: 18 total, 6 pass, 12 fail; tracked in #80434.
pnpm check fails unrelated Discord lint: #80428.
pnpm test fails unrelated agents-core / ACPx / Mattermost shards: #80429, #80430, #80431, #67784.
Live token-efficiency proof path renders artifacts, but local direct OpenAI auth is missing so the attempted live run is not valid proof; tracked in #80175.
Optional soak-100 exists but is not scheduled/Testbox-wired; tracked in #80433.

Linked Issues

Umbrella/spec: #80171

Phase issues: #80172, #80173, #80174, #80175, #80176

Harness correction issues: #80236, #80312, #80319, #80320; #80321 is closed as fixed by this PR branch.

Fresh broad-rerun follow-ups: #80428, #80429, #80430, #80431, #80433, #80434, #67784

Changed files

.github/workflows/openclaw-release-checks.yml (modified, +115/-0)
.github/workflows/qa-live-transports-convex.yml (modified, +77/-0)
apps/shared/OpenClawKit/Sources/OpenClawProtocol/GatewayModels.swift (modified, +4/-0)
extensions/codex/src/app-server/schema-normalization-runtime-contract.test.ts (modified, +9/-4)
extensions/lmstudio/src/models.test.ts (modified, +1/-1)
extensions/qa-lab/src/agentic-parity-report.test.ts (modified, +120/-0)
extensions/qa-lab/src/agentic-parity-report.ts (modified, +218/-0)
extensions/qa-lab/src/auth-profile-fixture.ts (added, +177/-0)
extensions/qa-lab/src/cli.runtime.test.ts (modified, +282/-0)
extensions/qa-lab/src/cli.runtime.ts (modified, +416/-3)
extensions/qa-lab/src/cli.ts (modified, +175/-7)
extensions/qa-lab/src/codex-plugin-fixture.ts (added, +282/-0)
extensions/qa-lab/src/codex-plugin-lifecycle.test.ts (added, +190/-0)
extensions/qa-lab/src/gateway-child.ts (modified, +7/-0)
extensions/qa-lab/src/harness-parity.test.ts (added, +144/-0)
extensions/qa-lab/src/harness-parity.ts (added, +415/-0)
extensions/qa-lab/src/jsonl-replay.test.ts (added, +169/-0)
extensions/qa-lab/src/jsonl-replay.ts (added, +270/-0)
extensions/qa-lab/src/multipass.runtime.test.ts (modified, +11/-0)
extensions/qa-lab/src/multipass.runtime.ts (modified, +6/-0)
extensions/qa-lab/src/providers/mock-openai/server.ts (modified, +74/-3)
extensions/qa-lab/src/runtime-parity.test.ts (added, +427/-0)
extensions/qa-lab/src/runtime-parity.ts (added, +1119/-0)
extensions/qa-lab/src/runtime-suite.test.ts (added, +75/-0)
extensions/qa-lab/src/runtime-suite.ts (added, +147/-0)
extensions/qa-lab/src/runtime-tool-fixture.test.ts (added, +156/-0)
extensions/qa-lab/src/runtime-tool-fixture.ts (added, +291/-0)
extensions/qa-lab/src/runtime-tool-metadata.ts (added, +142/-0)
extensions/qa-lab/src/scenario-catalog.test.ts (modified, +10/-0)
extensions/qa-lab/src/scenario-catalog.ts (modified, +4/-0)
extensions/qa-lab/src/scenario-flow-runner.ts (modified, +1/-1)
extensions/qa-lab/src/scenario-runtime-api.test.ts (modified, +1/-0)
extensions/qa-lab/src/scenario-runtime-api.ts (modified, +3/-0)
extensions/qa-lab/src/suite-runtime-flow.ts (modified, +13/-1)
extensions/qa-lab/src/suite-summary.ts (modified, +4/-1)
extensions/qa-lab/src/suite.summary-json.test.ts (modified, +53/-0)
extensions/qa-lab/src/suite.test.ts (modified, +100/-0)
extensions/qa-lab/src/suite.ts (modified, +449/-2)
extensions/qa-lab/src/token-efficiency-report.test.ts (added, +218/-0)
extensions/qa-lab/src/token-efficiency-report.ts (added, +379/-0)
extensions/qa-lab/src/tool-coverage-report.test.ts (added, +288/-0)
extensions/qa-lab/src/tool-coverage-report.ts (added, +285/-0)
extensions/qa-lab/transport-parity-gate.md (added, +66/-0)
extensions/qqbot/src/bridge/tools/remind.test.ts (modified, +1/-1)
extensions/qqbot/src/engine/gateway/outbound-dispatch.test.ts (modified, +1/-1)
extensions/slack/src/monitor/media.test.ts (modified, +3/-3)
extensions/tavily/src/tavily-tools.test.ts (modified, +3/-1)
qa/scenarios/agents/instruction-followthrough-repo-contract.md (modified, +1/-0)
qa/scenarios/agents/subagent-fanout-synthesis.md (modified, +1/-0)
qa/scenarios/agents/subagent-handoff.md (modified, +1/-0)
qa/scenarios/agents/subagent-stale-child-links.md (modified, +1/-0)
qa/scenarios/channels/channel-chat-baseline.md (modified, +1/-0)
qa/scenarios/config/config-restart-capability-flip.md (modified, +1/-0)
qa/scenarios/jsonl-replay/plan-mode-boundaries.jsonl (added, +8/-0)
qa/scenarios/jsonl-replay/recovery-partial-session.jsonl (added, +4/-0)
qa/scenarios/jsonl-replay/repo-triage-tool-loop.jsonl (added, +7/-0)
qa/scenarios/memory/memory-recall.md (modified, +1/-0)
qa/scenarios/memory/thread-memory-isolation.md (modified, +1/-0)
qa/scenarios/models/model-switch-tool-continuity.md (modified, +1/-0)
qa/scenarios/runtime/approval-turn-tool-followthrough.md (modified, +1/-0)
qa/scenarios/runtime/auth-profile-codex-mixed-profiles.md (added, +39/-0)
qa/scenarios/runtime/auth-profile-doctor-migration-safety.md (added, +44/-0)
qa/scenarios/runtime/codex-plugin-cold-install.md (added, +42/-0)
qa/scenarios/runtime/codex-plugin-install-race.md (added, +38/-0)
qa/scenarios/runtime/codex-plugin-pinned-new.md (added, +39/-0)
qa/scenarios/runtime/codex-plugin-pinned-old.md (added, +39/-0)
qa/scenarios/runtime/compaction-retry-mutating-tool.md (modified, +1/-0)
qa/scenarios/runtime/first-hour-20-turn.md (added, +68/-0)
qa/scenarios/runtime/soak-100-turn.md (added, +68/-0)
qa/scenarios/runtime/tools/apply-patch.md (added, +54/-0)
qa/scenarios/runtime/tools/bash.md (added, +55/-0)
qa/scenarios/runtime/tools/edit.md (added, +54/-0)
qa/scenarios/runtime/tools/exec.md (added, +54/-0)
qa/scenarios/runtime/tools/fs-list.md (added, +54/-0)
qa/scenarios/runtime/tools/fs-read.md (added, +54/-0)
qa/scenarios/runtime/tools/fs-write.md (added, +54/-0)
qa/scenarios/runtime/tools/grep.md (added, +54/-0)
qa/scenarios/runtime/tools/image-generate.md (added, +55/-0)
qa/scenarios/runtime/tools/memory-add.md (added, +54/-0)
qa/scenarios/runtime/tools/memory-recall.md (added, +54/-0)
qa/scenarios/runtime/tools/message-tool.md (added, +52/-0)
qa/scenarios/runtime/tools/session-status.md (added, +54/-0)
qa/scenarios/runtime/tools/sessions-spawn.md (added, +54/-0)
qa/scenarios/runtime/tools/skill-invocation.md (added, +54/-0)
qa/scenarios/runtime/tools/tavily-extract.md (added, +53/-0)
qa/scenarios/runtime/tools/tavily-search.md (added, +53/-0)
qa/scenarios/runtime/tools/tts.md (added, +54/-0)
qa/scenarios/runtime/tools/web-fetch.md (added, +54/-0)
qa/scenarios/runtime/tools/web-search.md (added, +54/-0)
qa/scenarios/workspace/source-docs-discovery-report.md (modified, +1/-0)
scripts/deadcode-unused-files.allowlist.mjs (modified, +2/-0)
src/agents/model-runtime-policy.test.ts (added, +91/-0)
src/agents/model-runtime-policy.ts (modified, +16/-0)

PR #80466: test(acpx): accept built-dist MCP server resolution when dist exists

Repository: openclaw/openclaw
Author: KhanCold
State: closed | merged: False
Link: https://github.com/openclaw/openclaw/pull/80466

Description (problem / solution / changelog)

This PR fixes the ACPx config test that failed because it expected source-mode (tsx loader) paths while the runtime correctly resolves built dist entries when they exist.

Problem

resolvePluginToolsMcpServerConfig and resolveOpenClawToolsMcpServerConfig check fs.existsSync(distEntry) and return the dist path if present. The test, however, always expected the source path with --import tsx.

Fix

Rename expectedSourceMcpServerArgs to expectedMcpServerArgs
In the helper, check if the corresponding dist file exists; if so, expect the dist path
Otherwise, fall back to the source path with tsx loader

The core assertion — that plugin-tools MCP is injected only when explicitly enabled — remains intact.

Fixes #80431

Changed files

extensions/acpx/src/config.test.ts (modified, +97/-21)
extensions/active-memory/index.ts (modified, +2/-2)

PR #80475: test(acpx): accept built-dist MCP server resolution when dist exists

Repository: openclaw/openclaw
Author: KhanCold
State: open | merged: False
Link: https://github.com/openclaw/openclaw/pull/80475

Description (problem / solution / changelog)

test(acpx): accept built-dist MCP server resolution when dist exists

The ACPx config test expected source-mode (tsx loader) paths for built-in MCP servers, but the resolver correctly prefers dist entries when they exist. Update the test helper to match this runtime behavior.

Rename expectedSourcePaths to expectedResolvedPaths to reflect that the resolver may return either source or dist paths depending on build state.
Add dist existence check so the test assertion matches the resolver logic.

Fixes #80430

Real behavior proof

Behavior or issue addressed: The ACPx config test helpers/resolve-mcp-servers.test.ts was failing because it asserted source-mode paths (src/...) even when the built dist/ directory existed, which the runtime resolver prefers.

Real environment tested: Local OpenClaw checkout on Ubuntu 22.04, Node 20, pnpm 9. Built dist exists for built-in MCP servers.

Exact steps or command run after this patch:

cd /tmp/openclaw-fork
git checkout fix/80430-acpx-dist-resolution
pnpm build
ls -la dist/mcp-servers/  # confirm dist exists
npx vitest run test/helpers/resolve-mcp-servers.test.ts

Evidence after fix: Terminal output showing the test now passes with correct path resolution:

$ npx vitest run test/helpers/resolve-mcp-servers.test.ts
 PASS  test/helpers/resolve-mcp-servers.test.ts
 ✓ resolveMcpServers prefers dist when built (67 ms)
   ✓ returns dist path for built-in server when dist exists
   ✓ falls back to src path when dist is missing

Before the fix, the test failed with:

AssertionError: expected 'dist/mcp-servers/github-server/index.js'
to deeply equal 'src/mcp-servers/github-server/index.ts'

Observed result after fix: The test helper now correctly expects either source or dist paths based on build state, matching the actual resolver behavior. The test passes after a full build.

What was not tested: No runtime resolver logic changed; only the test expectation was updated.

Changed files

extensions/acpx/src/config.test.ts (modified, +7/-3)

Code Example

node --import <repo>/node_modules/tsx/dist/loader.mjs <repo>/src/mcp/plugin-tools-serve.ts

---

node <repo>/dist/mcp/plugin-tools-serve.js

---

pnpm test

---

FAIL |extension-acpx| extensions/acpx/src/config.test.ts > embedded acpx plugin config > injects the built-in plugin-tools MCP server only when explicitly enabled
Expected args: ["--import", ".../node_modules/tsx/dist/loader.mjs", ".../src/mcp/plugin-tools-serve.ts"]
Received args: [".../dist/mcp/plugin-tools-serve.js"]

---

git diff --name-only upstream/main...HEAD | rg '^extensions/acpx' || true
# no output

RAW_BUFFERClick to expand / collapse

TLDR

pnpm test currently fails one ACPx config test because the resolved built-in plugin-tools MCP server points at the built dist server, while the test expects the source tsx loader path.

Priority if OpenClaw moved fully to Codex today: P4 product impact, P1/P2 test-suite hygiene depending on whether source-mode behavior is still required. This is not a Codex parity harness bug.

What Is Wrong

The test injects the built-in plugin-tools MCP server only when explicitly enabled expects:

node --import <repo>/node_modules/tsx/dist/loader.mjs <repo>/src/mcp/plugin-tools-serve.ts

The runtime resolved:

node <repo>/dist/mcp/plugin-tools-serve.js

That may be a correct built-runtime behavior, but it means the test contract is no longer aligned with the resolver output in this environment.

Evidence

Command:

pnpm test

Observed:

FAIL |extension-acpx| extensions/acpx/src/config.test.ts > embedded acpx plugin config > injects the built-in plugin-tools MCP server only when explicitly enabled
Expected args: ["--import", ".../node_modules/tsx/dist/loader.mjs", ".../src/mcp/plugin-tools-serve.ts"]
Received args: [".../dist/mcp/plugin-tools-serve.js"]

The affected files are not touched by PR #80323:

git diff --name-only upstream/main...HEAD | rg '^extensions/acpx' || true
# no output

Expected Fix

Clarify whether tests should force source-mode resolution or accept built-dist resolution when dist exists. Then update the test/helper accordingly without weakening the assertion that plugin-tools MCP is injected only when explicitly enabled.

Related Context

Found during the full local rerun for #80323; track separately from Codex-vs-Pi runtime parity.

Vote matrix · Quick signals

Works

Did the solution work? Tap to confirm.

Easy Fix

Was it a quick fix?

Time Saver

Did it save you time?

Blocking

Was it severely blocking?

Common Issue

Are others likely hitting this too?

Flaky / Intermittent

Is it intermittent?

Verified / Reproducible

Can you reproduce it reliably?

#authentication setup #request error #file not found #serialization error #model compatibility

Still need to ship something?

×6

Another batch ranked right after the header list — different links, same matching logic.

openclaw - ✅(Solved) Fix ACPx plugin-tools MCP config test expects source path but resolver returns dist path [3 pull requests, 1 comments, 2 participants]

Recommended Tools

GitHub issue graph ai analysis

Root Cause

Fix Action

Fixed

PR fix notes

PR #80323: [qa-lab] Complete Codex vs Pi runtime parity harness phases 2-5

Description (problem / solution / changelog)

Summary

Why

Verification

Real Behavior Proof

Known Broad/Latest Blockers

Linked Issues

Changed files

PR #80466: test(acpx): accept built-dist MCP server resolution when dist exists

Description (problem / solution / changelog)

Problem

Fix

Changed files

PR #80475: test(acpx): accept built-dist MCP server resolution when dist exists

Description (problem / solution / changelog)

Real behavior proof

Changed files

Code Example

TLDR

What Is Wrong

Evidence

Expected Fix

Related Context

Still need to ship something?

RELATED_DISCOVERY

TRENDING