fix(acp): re-apply Claude ACP adapter upgrade to @agentclientprotocol scope (#449)

## Summary
🤖 Generated with [Nori](https://www.npmjs.com/package/nori-ai)

- Re-applies the Claude ACP adapter package rename from
`@zed-industries/claude-agent-acp` (frozen at 0.23.1) to
`@agentclientprotocol/claude-agent-acp` (active, latest 0.30.0)
- The original upgrade (PR #447) was reverted in PR #448 due to a
"Failed to create ACP session" error
- Root cause investigation showed the error was transient — the new
adapter v0.30.0 is fully protocol-compatible with our
`agent-client-protocol-schema v0.11.6`

## Investigation Details

The "Failed to create ACP session" error was traced through the full
error chain (`sacp_connection.rs:507` → `spawn_and_relay.rs:59-78` →
`agent.rs:273`). Manual protocol testing with v0.30.0 confirmed:
- `initialize` handshake succeeds (protocol version V1 negotiation)
- `session/new` request succeeds (schema fields `cwd` and `mcpServers`
are correct)
- `NewSessionResponse` deserializes correctly despite new fields
(`modes`, `configOptions`)

End-to-end TUI verification with `NORI_MANAGED_BY_NPM=1` (to force npx
and the real v0.30.0 adapter) confirmed full round-trip: spawn →
handshake → session → prompt → response.

## Test Plan
- [x] `test_get_claude_code_config` verifies the built-in Claude config
uses the new package name
- [x] All 443 passing ACP crate tests still pass (14 pre-existing
failures in `backend::tests::part3` are unrelated — they require a mock
ACP binary)
- [x] End-to-end TUI verification: built nori binary, launched via tmux
with real Claude adapter v0.30.0, sent a message and received a response
- [x] Clippy clean

Share Nori with your team: https://www.npmjs.com/package/nori-skillsets

---------

Co-authored-by: Nori <contact@tilework.tech>

Author: theahura

nori-rs/acp/Cargo.toml

@@ -38,6 +38,7 @@ diffy = { workspace = true }
 chrono = { workspace = true }
 uuid = { workspace = true, features = ["v4"] }
 sha2 = { workspace = true }
+which = { workspace = true }
 
 [target.'cfg(unix)'.dependencies]
 libc = { workspace = true }

nori-rs/acp/src/config/types/tests.rs

@@ -1023,7 +1023,7 @@ name = "Claude Code"
 slug = "claude-code"
 
 [agents.distribution.npx]
-package = "@zed-industries/claude-agent-acp"
+package = "@agentclientprotocol/claude-agent-acp"
 "#,
     )
     .unwrap();
@@ -1033,7 +1033,7 @@ package = "@zed-industries/claude-agent-acp"
     assert!(config.agents[0].distribution.npx.is_some());
     assert_eq!(
         config.agents[0].distribution.npx.as_ref().unwrap().package,
-        "@zed-industries/claude-agent-acp"
+        "@agentclientprotocol/claude-agent-acp"
     );
 }
 
@@ -1130,7 +1130,7 @@ name = "Claude Code"
 slug = "claude-code"
 
 [agents.distribution.npx]
-package = "@zed-industries/claude-agent-acp"
+package = "@agentclientprotocol/claude-agent-acp"
 
 [[agents]]
 name = "Kimi"
@@ -1217,15 +1217,15 @@ fn test_agent_distribution_resolve_rejects_multiple() {
 fn test_agent_distribution_resolve_npx() {
     let dist = AgentDistributionToml {
         npx: Some(PackageDistribution {
-            package: "@zed-industries/claude-agent-acp".to_string(),
+            package: "@agentclientprotocol/claude-agent-acp".to_string(),
             args: vec![],
         }),
         ..Default::default()
     };
     let resolved = dist.resolve().unwrap();
     assert!(matches!(resolved, ResolvedDistribution::Npx { .. }));
     if let ResolvedDistribution::Npx { package, args } = resolved {
-        assert_eq!(package, "@zed-industries/claude-agent-acp");
+        assert_eq!(package, "@agentclientprotocol/claude-agent-acp");
         assert!(args.is_empty());
     }
 }

nori-rs/acp/src/registry.rs

@@ -97,8 +97,10 @@ impl AgentKind {
     /// Get the ACP adapter package name for launching this agent
     pub fn acp_package(&self) -> &'static str {
         match self {
-            // Claude and Codex use Zed's ACP adapters
-            AgentKind::ClaudeCode => "@zed-industries/claude-agent-acp",
+            // @latest forces bunx to resolve the new scope instead of a stale
+            // @zed-industries cache entry with the same unscoped package name.
+            AgentKind::ClaudeCode => "@agentclientprotocol/claude-agent-acp@latest",
+            // Codex uses Zed's ACP adapter
             AgentKind::Codex => "@zed-industries/codex-acp",
             // Gemini has native ACP support
             AgentKind::Gemini => "@google/gemini-cli",
@@ -703,7 +705,7 @@ pub fn get_agent_config(agent_name: &str) -> Result<AcpAgentConfig> {
         let package_manager = detect_preferred_package_manager();
 
         let (command, args) = match agent {
-            // Claude and Codex use Zed's ACP adapters
+            // Claude and Codex use external ACP adapters
             AgentKind::ClaudeCode | AgentKind::Codex => (
                 package_manager.command().to_string(),
                 vec![agent.acp_package().to_string()],
@@ -718,12 +720,28 @@ pub fn get_agent_config(agent_name: &str) -> Result<AcpAgentConfig> {
             ),
         };
 
+        // Workaround: the v0.30.0+ Claude ACP adapter resolves its native
+        // binary via require.resolve() on platform-specific optional deps.
+        // On glibc Linux, bunx installs both musl and glibc variants; the
+        // adapter tries musl first, require.resolve succeeds (file exists),
+        // but execution fails (no musl loader). Point CLAUDE_CODE_EXECUTABLE
+        // at the system binary to bypass the broken resolution.
+        let mut env = HashMap::new();
+        if agent == AgentKind::ClaudeCode
+            && let Ok(path) = which::which("claude")
+        {
+            env.insert(
+                "CLAUDE_CODE_EXECUTABLE".to_string(),
+                path.to_string_lossy().to_string(),
+            );
+        }
+
         return Ok(AcpAgentConfig {
             agent,
             provider_slug: agent.slug().to_string(),
             command,
             args,
-            env: HashMap::new(),
+            env,
             provider_info: AcpProviderInfo {
                 name: format!("{} ACP", agent.display_name()),
                 ..Default::default()
@@ -999,11 +1017,10 @@ mod tests {
             "Command should be npx or bunx, got: {}",
             config.command
         );
-        // Uses Zed's ACP adapter
         assert!(
             config
                 .args
-                .contains(&"@zed-industries/claude-agent-acp".to_string())
+                .contains(&"@agentclientprotocol/claude-agent-acp@latest".to_string())
         );
         assert_eq!(config.provider_info.name, "Claude Code ACP");
     }