dispenser/dispenserclient.go

@@ -301,14 +301,16 @@ func (c *Client) DispenserStart(ctx context.Context) (string, error) {
 	for {
 		time.Sleep(delay * 2)
 		if time.Now().After(deadline) {
-			return stockStatus, nil
+			return stockStatus, fmt.Errorf("[%s] timeout waiting keycard at encoder position", funcName)
 		}
 
-		status, _ = c.do(ctx, cmdStatus)
+		status, err = c.do(ctx, cmdStatus)
+		if err != nil {
+			return stockStatus, fmt.Errorf("[%s] re-check status: %w", funcName, err)
+		}
 
 		stockStatus = stockTake(status)
 		c.setStock(status)
-		logStatus(status)
 
 		// error states first
 		if isCardWellEmpty(status) {
@@ -353,14 +355,17 @@ func (c *Client) DispenserFinal(ctx context.Context) (string, error) {
 			return stockStatus, nil
 		}
 
-		status, _ = c.do(ctx, cmdStatus)
+		status, err = c.do(ctx, cmdStatus)
+		if err != nil {
+			return stockStatus, nil
+		}
 
 		stockStatus = stockTake(status)
 		c.setStock(status)
 		logStatus(status)
 
 		if isCardWellEmpty(status) {
-			return stockStatus, nil
+			return stockStatus, fmt.Errorf(stockStatus)
 		}
 
 		if isAtEncoderPosition(status) {

main.go

@@ -29,7 +29,7 @@ import (
 )
 
 const (
-	buildVersion     = "1.2.5"
+	buildVersion     = "1.2.4"
 	serviceName      = "hardlink"
 	pollingFrequency = 8 * time.Second
 )

release notes.md

@@ -2,9 +2,6 @@
 
 builtVersion is a const in main.go
 
-#### 1.2.5 - 20 March 2026
-removed early return on error when checking dispenser status in the start and final loops.
-
 #### 1.2.4 - 18 March 2026
 added check if keycard at the encoder position before trying to encode key