add missing change impl and tests

Author: ecraig12345

.github/workflows/pr.yml

@@ -149,6 +149,7 @@ jobs:
 
       - name: Format
         run: test -z "$(gofmt -l .)"
+        shell: bash
         working-directory: ./go
 
       - name: Lint

go/cmd/beachball/main.go

@@ -54,6 +54,13 @@ func main() {
 	}
 	changeCmd.Flags().StringVarP(&cli.ChangeType, "type", "t", "", "change type (patch, minor, major, etc.)")
 	changeCmd.Flags().StringVarP(&cli.Message, "message", "m", "", "change description")
+	changeCmd.Flags().StringSliceVar(&cli.Package, "package", nil, "specific package(s) to create change files for")
+
+	var noCommitFlag bool
+	changeCmd.Flags().BoolVar(&noCommitFlag, "no-commit", false, "don't commit change files")
+
+	var noFetchFlag bool
+	rootCmd.PersistentFlags().BoolVar(&noFetchFlag, "no-fetch", false, "don't fetch remote branch")
 
 	var allFlag, verboseFlag bool
 	rootCmd.PersistentFlags().BoolVar(&allFlag, "all", false, "include all packages")
@@ -66,6 +73,12 @@ func main() {
 		if rootCmd.PersistentFlags().Changed("verbose") {
 			cli.Verbose = boolPtr(verboseFlag)
 		}
+		if rootCmd.PersistentFlags().Changed("no-fetch") {
+			cli.Fetch = boolPtr(!noFetchFlag)
+		}
+		if changeCmd.Flags().Changed("no-commit") {
+			cli.Commit = boolPtr(!noCommitFlag)
+		}
 	})
 
 	rootCmd.AddCommand(checkCmd, changeCmd)

go/internal/changefile/write_change_files.go

@@ -23,15 +23,16 @@ func WriteChangeFiles(options *types.BeachballOptions, changes []types.ChangeFil
 
 	var filePaths []string
 
-	for _, change := range changes {
+	if options.GroupChanges {
+		// Write all changes to a single grouped file
 		id := uuid.New().String()
-		sanitized := nonAlphanumRe.ReplaceAllString(change.PackageName, "-")
-		filename := fmt.Sprintf("%s-%s.json", sanitized, id)
+		filename := fmt.Sprintf("change-%s.json", id)
 		filePath := filepath.Join(changePath, filename)
 
-		data, err := json.MarshalIndent(change, "", "  ")
+		grouped := types.ChangeInfoMultiple{Changes: changes}
+		data, err := json.MarshalIndent(grouped, "", "  ")
 		if err != nil {
-			return fmt.Errorf("failed to marshal change: %w", err)
+			return fmt.Errorf("failed to marshal grouped changes: %w", err)
 		}
 
 		if err := os.WriteFile(filePath, append(data, '\n'), 0o644); err != nil {
@@ -40,6 +41,25 @@ func WriteChangeFiles(options *types.BeachballOptions, changes []types.ChangeFil
 
 		filePaths = append(filePaths, filePath)
 		fmt.Printf("Wrote change file: %s\n", filename)
+	} else {
+		for _, change := range changes {
+			id := uuid.New().String()
+			sanitized := nonAlphanumRe.ReplaceAllString(change.PackageName, "-")
+			filename := fmt.Sprintf("%s-%s.json", sanitized, id)
+			filePath := filepath.Join(changePath, filename)
+
+			data, err := json.MarshalIndent(change, "", "  ")
+			if err != nil {
+				return fmt.Errorf("failed to marshal change: %w", err)
+			}
+
+			if err := os.WriteFile(filePath, append(data, '\n'), 0o644); err != nil {
+				return fmt.Errorf("failed to write change file: %w", err)
+			}
+
+			filePaths = append(filePaths, filePath)
+			fmt.Printf("Wrote change file: %s\n", filename)
+		}
 	}
 
 	if len(filePaths) > 0 {

go/internal/commands/change.go

@@ -19,7 +19,7 @@ func Change(parsed types.ParsedOptions) error {
 		return err
 	}
 
-	if !result.IsChangeNeeded {
+	if !result.IsChangeNeeded && len(parsed.Options.Package) == 0 {
 		fmt.Println("No changes detected; no change files are needed.")
 		return nil
 	}
@@ -46,8 +46,17 @@ func Change(parsed types.ParsedOptions) error {
 		}
 	}
 
+	changedPackages := result.ChangedPackages
+	if len(changedPackages) == 0 && len(options.Package) > 0 {
+		changedPackages = options.Package
+	}
+
+	if len(changedPackages) == 0 {
+		return nil
+	}
+
 	var changes []types.ChangeFileInfo
-	for _, pkg := range result.ChangedPackages {
+	for _, pkg := range changedPackages {
 		changes = append(changes, types.ChangeFileInfo{
 			Type:                changeType,
 			Comment:             message,