refactor: decouple flag state, use int64 for key IDs, add body assertions

- Decouple use-automations flag by introducing per-command backing variables
  (bulkUpdateUseAutomations, bulkCreateUseAutomations) to prevent flag state
  leaking across commands
- Change bulkDeleteKeyIds from []int to []int64 with Int64SliceVar, removing
  the manual conversion loop for consistency with single-key operations
- Add testBody assertions to TestKeyBulkUpdate and TestKeyBulkCreate to
  validate outbound request payloads

Author: lechuckcaptain

cmd/key.go

@@ -21,9 +21,16 @@ var (
 
 	useAutomations bool
 
-	bulkDeleteKeyIds []int
-	bulkUpdateKeys   string
-	bulkCreateKeys   string
+	// bulk-delete
+	bulkDeleteKeyIds []int64
+
+	// bulk-update
+	bulkUpdateKeys           string
+	bulkUpdateUseAutomations bool
+
+	// bulk-create
+	bulkCreateKeys           string
+	bulkCreateUseAutomations bool
 )
 
 // keyCmd represents the key command
@@ -137,12 +144,7 @@ var keyBulkDeleteCmd = &cobra.Command{
 	Short: "Delete multiple keys",
 	Long:  "Deletes multiple keys from the project. Requires Manage keys admin right.",
 	RunE: func(*cobra.Command, []string) error {
-		var keyIDs []int64
-		for _, id := range bulkDeleteKeyIds {
-			keyIDs = append(keyIDs, int64(id))
-		}
-
-		resp, err := Api.Keys().BulkDelete(projectId, keyIDs)
+		resp, err := Api.Keys().BulkDelete(projectId, bulkDeleteKeyIds)
 		if err != nil {
 			return err
 		}
@@ -163,7 +165,7 @@ var keyBulkUpdateCmd = &cobra.Command{
 		resp, err := Api.Keys().BulkUpdate(
 			projectId,
 			keys,
-			lokalise.WithAutomations(useAutomations),
+			lokalise.WithAutomations(bulkUpdateUseAutomations),
 		)
 		if err != nil {
 			return err
@@ -185,7 +187,7 @@ var keyBulkCreateCmd = &cobra.Command{
 		resp, err := Api.Keys().Create(
 			projectId,
 			keys,
-			lokalise.WithAutomations(useAutomations),
+			lokalise.WithAutomations(bulkCreateUseAutomations),
 		)
 		if err != nil {
 			return err
@@ -263,7 +265,7 @@ func init() {
 	flagKeyId(keyDeleteCmd)
 
 	// Bulk delete
-	keyBulkDeleteCmd.Flags().IntSliceVar(&bulkDeleteKeyIds, "key-ids", []int{},
+	keyBulkDeleteCmd.Flags().Int64SliceVar(&bulkDeleteKeyIds, "key-ids", []int64{},
 		"Comma-separated list of key IDs to delete (required).")
 	_ = keyBulkDeleteCmd.MarkFlagRequired("key-ids")
 
@@ -272,15 +274,15 @@ func init() {
 	fs.StringVar(&bulkUpdateKeys, "keys", "",
 		"JSON array of key objects to update. Each object must contain key_id and fields to update (required).")
 	_ = keyBulkUpdateCmd.MarkFlagRequired("keys")
-	fs.BoolVar(&useAutomations, "use-automations", true,
+	fs.BoolVar(&bulkUpdateUseAutomations, "use-automations", true,
 		"Whether to run automations on the updated key translations.")
 
 	// Bulk create
 	fs = keyBulkCreateCmd.Flags()
 	fs.StringVar(&bulkCreateKeys, "keys", "",
 		"JSON array of key objects to create. Each object should contain key_name, platforms, and other fields (required).")
 	_ = keyBulkCreateCmd.MarkFlagRequired("keys")
-	fs.BoolVar(&useAutomations, "use-automations", true,
+	fs.BoolVar(&bulkCreateUseAutomations, "use-automations", true,
 		"Whether to run automations on the new key translations.")
 }
 

cmd/key_test.go

@@ -55,6 +55,11 @@ func TestKeyBulkUpdate(t *testing.T) {
 			testMethod(t, r, "PUT")
 			testHeader(t, r, "X-Api-Token", testApiToken)
 
+			data := `{"keys":[{"key_id":123,"description":"Updated description","tags":["updated"]}],"use_automations":true}`
+			req := new(bytes.Buffer)
+			_ = json.Compact(req, []byte(data))
+			testBody(t, r, req.String())
+
 			_, _ = fmt.Fprint(w, `{
 				"project_id": "`+testProjectID+`",
 				"keys": [
@@ -101,6 +106,11 @@ func TestKeyBulkCreate(t *testing.T) {
 			testMethod(t, r, "POST")
 			testHeader(t, r, "X-Api-Token", testApiToken)
 
+			data := `{"keys":[{"key_name":"welcome","platforms":["web"],"tags":[]},{"key_name":"goodbye","platforms":["ios","android"],"tags":["v2"]}],"use_automations":true}`
+			req := new(bytes.Buffer)
+			_ = json.Compact(req, []byte(data))
+			testBody(t, r, req.String())
+
 			_, _ = fmt.Fprint(w, `{
 				"project_id": "`+testProjectID+`",
 				"keys": [