feat(ai): add configurable custom ACP runtimes - #349
Conversation
|
@spamsch this is the last piece of the acp architecture, I waited on this because of security concerns and the stability of the acp protocol, but now it seems that the v1 version is stable enough. Only if you can, give it a test with with other community acp adapters for popular agents like Pi or Hermes. |
|
Fixed edge case found during testing with not very well done acp adapters. |
Preserve custom ACP continuation identity across transactional history reconciliation.
|
Resolved the conflicts with The visible conflicts were resolved as follows:
Two non-textual integration issues were also fixed:
Custom runtime fork behavior remains intentionally isolated:
Validation completed successfully:
An independent review agent also compared the resolved merge against both parents and found no medium- or high-severity regressions in transactional history storage, custom ACP lifecycle behavior, inline review, or the agent changes panel. |
|
QA, external acp working fine, same with chat history migration. |
Summary
Adds user-configurable local ACP runtimes alongside the built-in provider catalog.
session/resume,session/load, and transcript-only recovery.Security and behavior notes
PATHand isolated environment.Validation
cargo test -p neverwrite-native-backendnpm --prefix apps/desktop test -- --run src/features/ai/store/chatStore.test.tsnpm --prefix apps/desktop run lintnpm --prefix apps/desktop run buildnpm --prefix apps/desktop run electron:sidecar:buildnpm --prefix apps/desktop run electron:ai-runtime:smokegit diff --check