mirror of https://github.com/openclaw/openclaw.git synced 2026-04-14 18:51:04 +00:00

Files

Onur Solmaz 154a7edb7c refactor: consume acpx runtime library (#61495 )

* refactor: consume acpx runtime library

* refactor: remove duplicated acpx runtime files

* fix: update acpx runtime dependency

* fix: preserve acp runtime error codes

* fix: migrate legacy acpx session files

* fix: update acpx runtime dependency

* fix: import Dirent from node fs

* ACPX: repin shared runtime engine

* ACPX: repin runtime semantics fixes

* ACPX: repin runtime contract cleanup

* Extensions: repin ACPX after layout refactor

* ACPX: drop legacy session migration

* ACPX: drop direct ACP SDK dependency

* Discord ACP: stop duplicate direct fallback replies

* ACP: rename delivered text visibility hook

* ACPX: pin extension to 0.5.0

* Deps: drop stale ACPX build-script allowlist

* ACPX: add local development guidance

* ACPX: document temporary pnpm exception flow

* SDK: preserve legacy ACP visibility hook

* ACP: keep reset commands on local path

* ACP: make in-place reset start fresh session

* ACP: recover broken bindings on fresh reset

* ACP: defer fresh reset marker until close succeeds

* ACP: reset bound sessions fresh again

* Discord: ensure ACP bindings before /new

* ACP: recover missing persistent sessions

2026-04-06 15:51:08 +02:00

2.9 KiB

Raw Blame History

ACPX Extension Notes

This file applies to work under extensions/acpx/.

Purpose

The bundled ACPX extension is a thin OpenClaw wrapper around the published acpx package. Keep reusable ACP runtime logic in openclaw/acpx, not in this extension.

Default Version Policy

extensions/acpx/package.json should point at a published npm release by default.
Do not leave the extension pinned to a temporary GitHub commit or local checkout once the ACPX release exists.
Do not leave temporary pnpm build-script allowlist exceptions behind after switching back to a published ACPX package.

Unreleased ACPX Development Flow

Use this flow when OpenClaw needs unreleased ACPX changes before the ACPX version is published.

Make the ACPX code change in the openclaw/acpx repo first.
In OpenClaw, temporarily point extensions/acpx/package.json at the ACPX GitHub commit you need.
If pnpm blocks ACPX lifecycle/build scripts for that temporary GitHub-sourced package, temporarily add acpx to onlyBuiltDependencies in both package.json and pnpm-workspace.yaml.
Refresh the root workspace lock:
- pnpm install --lockfile-only --filter ./extensions/acpx
Refresh the extension-local npm lock for install metadata:
- cd extensions/acpx && npm install --package-lock-only --ignore-scripts
Rebuild OpenClaw and restart the gateway before doing live ACP validation.
Once ACPX is released, switch extensions/acpx/package.json back to the published npm version and refresh the same lockfiles again.
Remove any temporary acpx build-script allowlist entries that were only needed for the GitHub-sourced development pin.

Lockfile Notes

pnpm-lock.yaml is the tracked workspace lockfile and must match the ACPX version referenced by extensions/acpx/package.json.
extensions/acpx/package-lock.json is useful local install metadata for the bundled plugin package.
If extensions/acpx/package-lock.json is gitignored in this repo state, regenerating it is still useful for local verification, but it will not appear in git status.

Local Runtime Validation

When ACPX integration changes here, prefer this sequence:

pnpm install --filter ./extensions/acpx
pnpm test:extension acpx
pnpm build
Restart the local gateway if ACP runtime behavior or bundled plugin wiring changed.
If the change affects direct ACP behavior in chat, run a real ACP smoke after restart.

Direct ACPX Binary Policy

Prefer the plugin-local ACPX binary under extensions/acpx/node_modules/.bin/acpx.
Do not rely on a globally installed acpx binary for OpenClaw ACP validation.
If the plugin-local ACPX binary is missing or on the wrong version, reinstall it from the version pinned in extensions/acpx/package.json.

Boundary Rule

If a change feels like shared ACP runtime behavior instead of OpenClaw-specific glue, move it to openclaw/acpx and consume it from here instead of re-implementing it inside extensions/acpx.

2.9 KiB Raw Blame History