refactor: update audit log calls to ignore errors

Author: ToeiRei

client/bun/client.go

@@ -470,10 +470,7 @@ func (c *BunClient) UpdateAccount(ctx context.Context, id client.AccountId, user
 			return client.Account{}, fmt.Errorf("failed to update hostname: %w", err)
 		}
 	}
-	if m.Username != username {
-		// Note: Store doesn't have UpdateAccountUsername; skip for now or add if needed.
-		// For this phase, we only update hostname via UpdateAccountHostname.
-	}
+	// Username is not persisted through this store path yet.
 
 	return client.Account{
 		Id:           client.AccountId(m.ID),

client/testui/client.go

@@ -130,7 +130,7 @@ func (c *Client) CreatePublicKey(ctx context.Context, key string, comment string
 	publicKey := client.PublicKey{Id: c.publicKeyIdCounter, Algorithm: algorithm, Data: data, Comment: comment, Tags: tags}
 	c.publicKeys[publicKey.Id] = publicKey
 
-	c.writeAuditLog("public_key.create", fmt.Sprintf("%#v", publicKey))
+	_ = c.writeAuditLog("public_key.create", fmt.Sprintf("%#v", publicKey))
 	return publicKey, nil
 }
 
@@ -185,7 +185,7 @@ func (c *Client) UpdateLink(ctx context.Context, id client.LinkId, accountId cli
 		link.ExpiresAt = expiresAt
 		c.links[id] = link
 
-		c.writeAuditLog("link.update", fmt.Sprintf("%#v", link))
+		_ = c.writeAuditLog("link.update", fmt.Sprintf("%#v", link))
 		return link, nil
 	}
 	return client.Link{}, fmt.Errorf("account with id %v not found", id)
@@ -197,7 +197,7 @@ func (c *Client) UpdatePublicKey(ctx context.Context, id client.PublicKeyId, com
 		publicKey.Tags = tags
 		c.publicKeys[id] = publicKey
 
-		c.writeAuditLog("public_key.update", fmt.Sprintf("%#v", publicKey))
+		_ = c.writeAuditLog("public_key.update", fmt.Sprintf("%#v", publicKey))
 		return publicKey, nil
 	}
 	return client.PublicKey{}, fmt.Errorf("public key with id %v not found", id)
@@ -212,7 +212,7 @@ func (c *Client) DeletePublicKeys(ctx context.Context, ids ...client.PublicKeyId
 
 	for _, id := range ids {
 		delete(c.publicKeys, id)
-		c.writeAuditLog("public_key.delete", fmt.Sprintf("%#v", id))
+		_ = c.writeAuditLog("public_key.delete", fmt.Sprintf("%#v", id))
 	}
 
 	return nil
@@ -225,7 +225,7 @@ func (c *Client) CreateAccount(ctx context.Context, username string, host string
 	account := client.Account{Id: c.accountIdCounter, Username: username, Host: host, Port: port, DeployMethod: deploymentMethod, DeploySecret: deploymentSecret, DeployCache: ""}
 	c.accounts[account.Id] = account
 
-	c.writeAuditLog("account.create", fmt.Sprintf("%#v", account))
+	_ = c.writeAuditLog("account.create", fmt.Sprintf("%#v", account))
 	return account, nil
 }
 
@@ -273,7 +273,7 @@ func (c *Client) UpdateAccount(ctx context.Context, id client.AccountId, usernam
 		account.DeploySecret = deploymentSecret
 		c.accounts[id] = account
 
-		c.writeAuditLog("account.update", fmt.Sprintf("%#v", account))
+		_ = c.writeAuditLog("account.update", fmt.Sprintf("%#v", account))
 		return account, nil
 	}
 	return client.Account{}, fmt.Errorf("account with id %v not found", id)
@@ -288,7 +288,7 @@ func (c *Client) DeleteAccounts(ctx context.Context, ids ...client.AccountId) er
 
 	for _, id := range ids {
 		delete(c.accounts, id)
-		c.writeAuditLog("account.delete", fmt.Sprintf("%#v", id))
+		_ = c.writeAuditLog("account.delete", fmt.Sprintf("%#v", id))
 	}
 
 	return nil
@@ -310,7 +310,7 @@ func (c *Client) CreateLink(ctx context.Context, accountId client.AccountId, tag
 	link := client.Link{Id: c.linkIdCounter, AccountId: accountId, TagMatcher: tagMatcher, ExpiresAt: expiresAt}
 	c.links[link.Id] = link
 
-	c.writeAuditLog("link.create", fmt.Sprintf("%#v", link))
+	_ = c.writeAuditLog("link.create", fmt.Sprintf("%#v", link))
 	return link, nil
 }
 
@@ -358,7 +358,7 @@ func (c *Client) DeleteLinks(ctx context.Context, ids ...client.LinkId) error {
 
 	for _, id := range ids {
 		delete(c.links, id)
-		c.writeAuditLog("link.delete", fmt.Sprintf("%#v", id))
+		_ = c.writeAuditLog("link.delete", fmt.Sprintf("%#v", id))
 	}
 
 	return nil
@@ -465,7 +465,7 @@ func (c *Client) DeployAccounts(ctx context.Context, accountIds ...client.Accoun
 			// simulate deploying data to remote
 			c.remoteStates[account.Id] = c.accountDeployCache(account, deployDatas[i])
 
-			c.writeAuditLog("account.deploy", fmt.Sprintf("%#v", account))
+			_ = c.writeAuditLog("account.deploy", fmt.Sprintf("%#v", account))
 
 			// update accounts deploy cache
 			_account := c.accounts[account.Id]
@@ -592,7 +592,7 @@ func (c *Client) VerifyAccounts(ctx context.Context, accountIds ...client.Accoun
 				verifyProgress.Accounts[account.Id].Status = "finished"
 			}
 
-			c.writeAuditLog("account.verify", fmt.Sprintf("%#v", account))
+			_ = c.writeAuditLog("account.verify", fmt.Sprintf("%#v", account))
 
 			verifyProgress.Accounts[account.Id].Progress = 1
 			verifyProgressChan <- verifyProgress