fix: re-implement the auto-generate-object-id flag

The v4 API client doesn't have an option for this;
it's always true. This is the same behavior as the API.
This commit implements this check in the CLI.

Author: kai687

pkg/cmd/objects/import/import.go

@@ -22,12 +22,13 @@ type ImportOptions struct {
 	SearchClient func() (*search.APIClient, error)
 	Index        string
 
-	Scanner   *bufio.Scanner
-	BatchSize int
-	Wait      bool
+	Scanner       *bufio.Scanner
+	BatchSize     int
+	AutoObjectIDs bool
+	Wait          bool
 }
 
-// NewImportCmd creates and returns an import command for indice object
+// NewImportCmd creates and returns an import command for records
 func NewImportCmd(f *cmdutil.Factory) *cobra.Command {
 	opts := &ImportOptions{
 		IO:           f.IOStreams,
@@ -76,6 +77,8 @@ func NewImportCmd(f *cmdutil.Factory) *cobra.Command {
 	_ = cmd.MarkFlagRequired("file")
 
 	cmd.Flags().IntVarP(&opts.BatchSize, "batch-size", "b", 1000, "Specify the upload batch size")
+	cmd.Flags().
+		BoolVarP(&opts.AutoObjectIDs, "auto-generate-object-id-if-not-exist", "a", false, "Auto-generate objectIDs if they don't exist")
 	cmd.Flags().BoolVarP(&opts.Wait, "wait", "w", false, "wait for the operation to complete")
 	return cmd
 }
@@ -100,6 +103,15 @@ func runImportCmd(opts *ImportOptions) error {
 			err := fmt.Errorf("failed to parse JSON object on line %d: %s", count, err)
 			return err
 		}
+		// The API client doesn't support this flag since v4
+		// The API always automatically generates objectIDs
+		// if you pass in records without them
+		// Implement it here to avoid breaking changes
+		if !opts.AutoObjectIDs {
+			if _, ok := record["objectID"]; !ok {
+				return fmt.Errorf("missing objectID on line %d", count)
+			}
+		}
 		records = append(records, record)
 		count++
 	}

pkg/cmd/objects/import/import_test.go

@@ -37,6 +37,18 @@ func Test_runImportCmd(t *testing.T) {
 			cli:     fmt.Sprintf("foo -F '%s'", tmpFile),
 			wantOut: "✓ Successfully imported 1 objects to foo in",
 		},
+		{
+			name:    "missing objectID",
+			cli:     "foo -F -",
+			stdin:   `{"attribute": "foo"}`,
+			wantErr: "missing objectID on line 0",
+		},
+		{
+			name:    "with auto-generated objectID",
+			cli:     "foo --auto-generate-object-id-if-not-exist -F -",
+			stdin:   `{"attribute": "foo"}`,
+			wantOut: "✓ Successfully imported 1 objects to foo in",
+		},
 		{
 			name:    "from stdin with invalid JSON",
 			cli:     "foo -F -",