ADFA-4128 (6/11): drop the disconnect binder method and its tests

The runtime only ever disconnected by process death, which
ProxyAppConnections.onDisconnected already handles; the AIDL method went in
the qb-04 followup. Asked for in review on #1718.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Xsc7AMGBVyEMfrwpZX87iC

Author: fryanpan

quickbuild/core/src/main/java/org/appdevforall/cotg/quickbuild/service/deploy/QuickBuildHostService.kt

@@ -144,15 +144,6 @@ class QuickBuildHostService : Service() {
 			deathWatch = null
 		}
 
-		override fun disconnect(packageName: String?) {
-			enforceCaller("disconnect")
-			log.info("Proxy app {} disconnected", packageName)
-			// Unlink too: a stale recipient would otherwise fire on the process's eventual
-			// death and report a disconnect against whatever is registered by then.
-			clearDeathWatch()
-			connections.onDisconnected()
-		}
-
 		/**
 		 * Throws unless the caller is the proxy app the live session accepts.
 		 *

quickbuild/core/src/test/java/org/appdevforall/cotg/quickbuild/service/deploy/QuickBuildHostBinderTest.kt

@@ -47,7 +47,6 @@ class QuickBuildHostBinderTest {
 		assertThrows(SecurityException::class.java) { binder.connect(target, "com.example.quickbuild", 0) }
 		assertThrows(SecurityException::class.java) { binder.reportReloaded(1, 40) }
 		assertThrows(SecurityException::class.java) { binder.reportCrash(1, "boom") }
-		assertThrows(SecurityException::class.java) { binder.disconnect("com.example.quickbuild") }
 		assertThat(connections.target.value).isNull()
 	}
 
@@ -122,16 +121,6 @@ class QuickBuildHostBinderTest {
 		return seen
 	}
 
-	@Test
-	fun `disconnect clears the registered target`() {
-		beginMatchingSession()
-		binder.connect(target, "com.example.quickbuild", 0)
-
-		binder.disconnect("com.example.quickbuild")
-
-		assertThat(connections.target.value).isNull()
-	}
-
 	@Test
 	fun `a death from a superseded binder leaves the live registration alone`() {
 		beginMatchingSession()
@@ -236,18 +225,6 @@ class QuickBuildHostBinderTest {
 		assertThat(connections.target.value).isNull()
 	}
 
-	@Test
-	fun `a graceful disconnect unlinks the death watch`() {
-		beginMatchingSession()
-		val live = WatchableBinder()
-		binder.connect(targetOn(live.binder), "com.example.quickbuild", 0)
-
-		binder.disconnect("com.example.quickbuild")
-
-		// No stale recipient stays linked to fire on the process's eventual death.
-		assertThat(live.watching()).isEmpty()
-	}
-
 	/**
 	 * An [IBinder] that records link/unlink traffic and can script [IBinder.linkToDeath],
 	 * so the death-watch wiring is exercised for real. Only the two death-watch methods