Improve Miner handling of excess items

src/main/kotlin/net/shadowfacts/phycon/network/block/terminal/TerminalBlockEntity.kt

@@ -161,12 +161,11 @@ class TerminalBlockEntity: DeviceBlockEntity(PhyBlockEntities.TERMINAL),
 
 	private fun finishPendingRequests() {
 		if (world!!.isClient) return
+		if (pendingRequests.isEmpty()) return
 
-		for (request in pendingRequests) {
-			if (request.isFinishable(counter)) {
-				stackLocateRequestCompleted(request)
-			}
-		}
+		val finishable = pendingRequests.filter { it.isFinishable(counter) }
+		finishable.forEach(::stackLocateRequestCompleted)
+		pendingRequests.removeAll(finishable)
 	}
 
 	fun addObserver() {

src/main/kotlin/net/shadowfacts/phycon/network/component/NetworkStackDispatcher.kt

@@ -72,9 +72,10 @@ interface NetworkStackDispatcher<Insertion: NetworkStackDispatcher.PendingInsert
 
 fun <Self, Insertion: NetworkStackDispatcher.PendingInsertion<Insertion>> Self.finishTimedOutPendingInsertions() where Self: BlockEntity, Self: NetworkStackDispatcher<Insertion> {
 	if (world!!.isClient) return
+	if (pendingInsertions.isEmpty()) return
 
-	pendingInsertions
-		.filter { it.isFinishable(this) }
-		.forEach(::finishInsertion)
+	val finishable = pendingInsertions.filter { it.isFinishable(this) }
+	finishable.forEach(::finishInsertion)
+	pendingInsertions.removeAll(finishable)
 	// todo: if a timed-out insertion can't be finished, we should probably retry after some time (exponential backoff?)
 }