chore(librarian): rename source_commit_hash to commit_hash (googleapis#13165)

See googleapis/librarian#2224

Fixes googleapis/librarian#2596

Author: parthea

internal/librariangen/release/release.go

@@ -152,12 +152,12 @@ func updateChangelog(cfg *Config, lib *request.Library, t time.Time) error {
 		for _, subj := range subjects {
 			change := subjectsMap[subj]
 			var commitLink string
-			if change.SourceCommitHash != "" {
-				shortHash := change.SourceCommitHash
+			if change.CommitHash != "" {
+				shortHash := change.CommitHash
 				if len(shortHash) > 7 {
 					shortHash = shortHash[:7]
 				}
-				commitURL := fmt.Sprintf("https://github.com/googleapis/google-cloud-go/commit/%s", change.SourceCommitHash)
+				commitURL := fmt.Sprintf("https://github.com/googleapis/google-cloud-go/commit/%s", change.CommitHash)
 				commitLink = fmt.Sprintf("([%s](%s))", shortHash, commitURL)
 			}
 

internal/librariangen/release/release_test.go

@@ -128,9 +128,9 @@ func TestInit(t *testing.T) {
 					"source_roots": ["secretmanager"],
 					"apis": [{"path": "google/cloud/secretmanager/v1"}],
 					"changes": [
-						{"type": "feat", "subject": "another feature", "source_commit_hash": "zxcvbn098765"},
-						{"type": "fix", "subject": "correct typo in documentation", "source_commit_hash": "123456abcdef"},
-						{"type": "feat", "subject": "add new GetSecret API", "source_commit_hash": "abcdef123456"}
+						{"type": "feat", "subject": "another feature", "commit_hash": "zxcvbn098765"},
+						{"type": "fix", "subject": "correct typo in documentation", "commit_hash": "123456abcdef"},
+						{"type": "feat", "subject": "add new GetSecret API", "commit_hash": "abcdef123456"}
 					],
 					"tag_format": "{id}/v{version}"
 				}]
@@ -169,9 +169,9 @@ func TestInit(t *testing.T) {
 					"id": "wholerepo", "version": "1.16.0", "release_triggered": true,
 					"source_roots": ["."],
 					"changes": [
-						{"type": "feat", "subject": "another feature", "source_commit_hash": "zxcvbn098765"},
-						{"type": "fix", "subject": "correct typo in documentation", "source_commit_hash": "123456abcdef"},
-						{"type": "feat", "subject": "add new GetSecret API", "source_commit_hash": "abcdef123456"}
+						{"type": "feat", "subject": "another feature", "commit_hash": "zxcvbn098765"},
+						{"type": "fix", "subject": "correct typo in documentation", "commit_hash": "123456abcdef"},
+						{"type": "feat", "subject": "add new GetSecret API", "commit_hash": "abcdef123456"}
 					],
 					"tag_format": "v{version}"
 				}]
@@ -191,9 +191,9 @@ func TestInit(t *testing.T) {
 					"id": "root-module", "version": "1.16.0", "release_triggered": true,
 					"source_roots": ["civil", "rpcreplay", "httpreplay"],
 					"changes": [
-						{"type": "feat", "subject": "another feature", "source_commit_hash": "zxcvbn098765"},
-						{"type": "fix", "subject": "correct typo in documentation", "source_commit_hash": "123456abcdef"},
-						{"type": "feat", "subject": "add new GetSecret API", "source_commit_hash": "abcdef123456"}
+						{"type": "feat", "subject": "another feature", "commit_hash": "zxcvbn098765"},
+						{"type": "fix", "subject": "correct typo in documentation", "commit_hash": "123456abcdef"},
+						{"type": "feat", "subject": "add new GetSecret API", "commit_hash": "abcdef123456"}
 					],
 					"tag_format": "v{version}"
 				}]
@@ -213,7 +213,7 @@ func TestInit(t *testing.T) {
 					"id": "xyz", "version": "1.16.0", "release_triggered": true,
 					"source_roots": ["xyz"],
 					"changes": [
-						{"type": "feat", "subject": "another feature", "source_commit_hash": "zxcvbn098765"}
+						{"type": "feat", "subject": "another feature", "commit_hash": "zxcvbn098765"}
 					],
 					"tag_format": "custom-{id}-v{version}"
 				}]

internal/librariangen/request/request.go

@@ -53,11 +53,11 @@ type API struct {
 
 // Change represents a single commit change for a library.
 type Change struct {
-	Type             string `json:"type"`
-	Subject          string `json:"subject"`
-	Body             string `json:"body"`
-	PiperCLNumber    string `json:"piper_cl_number"`
-	SourceCommitHash string `json:"source_commit_hash"`
+	Type          string `json:"type"`
+	Subject       string `json:"subject"`
+	Body          string `json:"body"`
+	PiperCLNumber string `json:"piper_cl_number"`
+	CommitHash    string `json:"commit_hash"`
 }
 
 // ParseLibrary reads a file from the given path and unmarshals

internal/librariangen/testdata/release-init/librarian/release-init-request.json

@@ -9,14 +9,14 @@
           "subject": "add new GetSecret API",
           "body": "This adds the ability to retrieve a secret's metadata.",
           "piper_cl_number": "123456789",
-          "source_commit_hash": "a1b2c3d4e5f6a1b2c3d4e5f6a1b2c3d4e5f6a1b2"
+          "commit_hash": "a1b2c3d4e5f6a1b2c3d4e5f6a1b2c3d4e5f6a1b2"
         },
         {
           "type": "fix",
           "subject": "correct typo in documentation",
           "body": "Fixed a minor typo in the client library documentation.",
           "piper_cl_number": "987654321",
-          "source_commit_hash": "f6e5d4c3b2a1f6e5d4c3b2a1f6e5d4c3b2a1f6e5"
+          "commit_hash": "f6e5d4c3b2a1f6e5d4c3b2a1f6e5d4c3b2a1f6e5"
         }
       ],
       "apis": [

internal/stategen/state.go

@@ -100,7 +100,7 @@ type Change struct {
 	// The Changelist number in piper associated with this change.
 	ClNum string `yaml:"piper_cl_number" json:"piper_cl_number"`
 	// The commit hash in the source repository associated with this change.
-	CommitHash string `yaml:"source_commit_hash" json:"source_commit_hash"`
+	CommitHash string `yaml:"commit_hash" json:"commit_hash"`
 }
 
 // GitHubBranch is the representation of a repository branch as returned by the GitHub