refactor: standardize logging messages to lowercase (#2750)

Normalize all log messages to lowercase for consistency across the
codebase.

Author: julieqiu

internal/automation/cli.go

@@ -30,13 +30,13 @@ func Run(args []string) error {
 	ctx := context.Background()
 	options, err := parseFlags(args)
 	if err != nil {
-		slog.Error("Error parsing command", slog.Any("err", err))
+		slog.Error("error parsing command", slog.Any("err", err))
 		return err
 	}
 
 	err = runCommandFn(ctx, options.Command, options.ProjectId, options.Push, options.Build, options.ForceRun)
 	if err != nil {
-		slog.Error("Error running command", slog.Any("err", err))
+		slog.Error("error running command", slog.Any("err", err))
 		return err
 	}
 	return nil

internal/automation/trigger.go

@@ -97,7 +97,7 @@ func runCommandWithConfig(ctx context.Context, client CloudBuildClient, ghClient
 	if triggerName == "stage-release" {
 		_, week := dateTime.ISOWeek()
 		if week%2 == 0 && !forceRun {
-			slog.Info("Skipping stage-release on an even week.")
+			slog.Info("skipping stage-release on an even week.")
 			return nil
 		}
 	}
@@ -124,12 +124,12 @@ func runCommandWithConfig(ctx context.Context, client CloudBuildClient, ghClient
 			repositoryOwner := parts[len(parts)-2]
 			prs, err := ghClient.FindMergedPullRequestsWithPendingReleaseLabel(ctx, repositoryOwner, repository.Name)
 			if err != nil {
-				slog.Error("Error finding merged pull requests for publish-release", slog.Any("err", err), slog.String("repository", repository.Name))
+				slog.Error("error finding merged pull requests for publish-release", slog.Any("err", err), slog.String("repository", repository.Name))
 				errs = append(errs, err)
 				continue
 			}
 			if len(prs) == 0 {
-				slog.Info("No pull requests with label 'release:pending' found. Skipping 'publish-release' trigger.", slog.String("repository", repository.Name))
+				slog.Info("no pull requests with label 'release:pending' found. Skipping 'publish-release' trigger.", slog.String("repository", repository.Name))
 				continue
 			} else {
 				substitutions["_PR"] = fmt.Sprintf("%v", prs[0].GetHTMLURL())
@@ -140,7 +140,7 @@ func runCommandWithConfig(ctx context.Context, client CloudBuildClient, ghClient
 		}
 		err = runCloudBuildTriggerByName(ctx, client, projectId, region, triggerName, substitutions)
 		if err != nil {
-			slog.Error("Error triggering cloudbuild", slog.Any("err", err))
+			slog.Error("error triggering cloudbuild", slog.Any("err", err))
 			errs = append(errs, err)
 		}
 	}

internal/config/config.go

@@ -277,15 +277,15 @@ func (c *Config) setupUser() error {
 
 func (c *Config) createWorkRoot() error {
 	if c.WorkRoot != "" {
-		slog.Info("Using specified working directory", "dir", c.WorkRoot)
+		slog.Info("using specified working directory", "dir", c.WorkRoot)
 		return nil
 	}
 	path, err := os.MkdirTemp(tempDir(), "librarian-*")
 	if err != nil {
 		return err
 	}
 
-	slog.Info("Temporary working directory", "dir", path)
+	slog.Info("temporary working directory", "dir", path)
 	c.WorkRoot = path
 	return nil
 }

internal/github/github.go

@@ -188,10 +188,10 @@ func (c *Client) GetRawContent(ctx context.Context, path, ref string) ([]byte, e
 // which must have a GitHub HTTPS URL. We assume a base branch of "main".
 func (c *Client) CreatePullRequest(ctx context.Context, repo *Repository, remoteBranch, baseBranch, title, body string, isDraft bool) (*PullRequestMetadata, error) {
 	if body == "" {
-		slog.Warn("Provided PR body is empty, setting default.")
+		slog.Warn("provided PR body is empty, setting default.")
 		body = "Regenerated all changed APIs. See individual commits for details."
 	}
-	slog.Info("Creating PR", "branch", remoteBranch, "base", baseBranch, "title", title)
+	slog.Info("creating PR", "branch", remoteBranch, "base", baseBranch, "title", title)
 	// The body may be excessively long, only display in debug mode.
 	slog.Debug("with PR body", "body", body)
 	newPR := &github.NewPullRequest{
@@ -207,14 +207,14 @@ func (c *Client) CreatePullRequest(ctx context.Context, repo *Repository, remote
 		return nil, err
 	}
 
-	slog.Info("PR created", "url", pr.GetHTMLURL())
+	slog.Info("pr created", "url", pr.GetHTMLURL())
 	pullRequestMetadata := &PullRequestMetadata{Repo: repo, Number: pr.GetNumber()}
 	return pullRequestMetadata, nil
 }
 
 // GetLabels fetches the labels for an issue.
 func (c *Client) GetLabels(ctx context.Context, number int) ([]string, error) {
-	slog.Info("Getting labels", "number", number)
+	slog.Info("getting labels", "number", number)
 	var allLabels []string
 	opts := &github.ListOptions{
 		PerPage: 100,
@@ -237,14 +237,14 @@ func (c *Client) GetLabels(ctx context.Context, number int) ([]string, error) {
 
 // ReplaceLabels replaces all labels for an issue.
 func (c *Client) ReplaceLabels(ctx context.Context, number int, labels []string) error {
-	slog.Info("Replacing labels", "number", number, "labels", labels)
+	slog.Info("replacing labels", "number", number, "labels", labels)
 	_, _, err := c.Issues.ReplaceLabelsForIssue(ctx, c.repo.Owner, c.repo.Name, number, labels)
 	return err
 }
 
 // AddLabelsToIssue adds labels to an existing issue in a GitHub repository.
 func (c *Client) AddLabelsToIssue(ctx context.Context, repo *Repository, number int, labels []string) error {
-	slog.Info("Labels added to issue", "number", number, "labels", labels)
+	slog.Info("labels added to issue", "number", number, "labels", labels)
 	_, _, err := c.Issues.AddLabelsToIssue(ctx, repo.Owner, repo.Name, number, labels)
 	return err
 }
@@ -351,7 +351,7 @@ func (c *Client) FindMergedPullRequestsWithLabel(ctx context.Context, owner, rep
 // CreateTag creates a lightweight tag in the repository at the given commit SHA.
 // This does NOT create a release, just the tag.
 func (c *Client) CreateTag(ctx context.Context, tagName, commitSHA string) error {
-	slog.Info("Creating tag", "tag", tagName, "commit", commitSHA)
+	slog.Info("creating tag", "tag", tagName, "commit", commitSHA)
 	ref := "refs/tags/" + tagName
 	tagRef := &github.Reference{
 		Ref:    github.Ptr(ref),
@@ -363,7 +363,7 @@ func (c *Client) CreateTag(ctx context.Context, tagName, commitSHA string) error
 
 // ClosePullRequest closes the pull request specified by pull request number.
 func (c *Client) ClosePullRequest(ctx context.Context, number int) error {
-	slog.Info("Closing pull request", slog.Int("number", number))
+	slog.Info("closing pull request", slog.Int("number", number))
 	state := "closed"
 	_, _, err := c.PullRequests.Edit(ctx, c.repo.Owner, c.repo.Name, number, &github.PullRequest{
 		State: &state,

internal/gitrepo/gitrepo.go

@@ -123,7 +123,7 @@ func newRepositoryWithoutUser(opts *RepositoryOptions) (*LocalRepository, error)
 	if !opts.MaybeClone {
 		return open(opts.Dir)
 	}
-	slog.Info("Checking for repository", "dir", opts.Dir)
+	slog.Info("checking for repository", "dir", opts.Dir)
 	_, err := os.Stat(opts.Dir)
 	if err == nil {
 		return open(opts.Dir)
@@ -135,14 +135,14 @@ func newRepositoryWithoutUser(opts *RepositoryOptions) (*LocalRepository, error)
 		if opts.RemoteBranch == "" {
 			return nil, fmt.Errorf("gitrepo: remote branch is required when cloning")
 		}
-		slog.Info("Repository not found, executing clone")
+		slog.Info("repository not found, executing clone")
 		return clone(opts.Dir, opts.RemoteURL, opts.RemoteBranch, opts.CI, opts.Depth)
 	}
 	return nil, fmt.Errorf("failed to check for repository at %q: %w", opts.Dir, err)
 }
 
 func open(dir string) (*LocalRepository, error) {
-	slog.Info("Opening repository", "dir", dir)
+	slog.Info("opening repository", "dir", dir)
 	repo, err := git.PlainOpen(dir)
 	if err != nil {
 		return nil, err
@@ -155,7 +155,7 @@ func open(dir string) (*LocalRepository, error) {
 }
 
 func clone(dir, url, branch, ci string, depth int) (*LocalRepository, error) {
-	slog.Info("Cloning repository", "url", url, "dir", dir)
+	slog.Info("cloning repository", "url", url, "dir", dir)
 	options := &git.CloneOptions{
 		URL:           url,
 		ReferenceName: plumbing.NewBranchReferenceName(branch),
@@ -197,7 +197,7 @@ func (r *LocalRepository) AddAll() error {
 // Commit creates a new commit with the provided message and author
 // information.
 func (r *LocalRepository) Commit(msg string) error {
-	slog.Info("Committing", "message", msg)
+	slog.Info("committing", "message", msg)
 	worktree, err := r.repo.Worktree()
 	if err != nil {
 		return err
@@ -239,7 +239,7 @@ func (r *LocalRepository) IsClean() (bool, error) {
 // ChangedFiles returns a list of files that have been modified, added, or deleted
 // in the working tree, including both staged and unstaged changes.
 func (r *LocalRepository) ChangedFiles() ([]string, error) {
-	slog.Debug("Getting changed files")
+	slog.Debug("getting changed files")
 	worktree, err := r.repo.Worktree()
 	if err != nil {
 		return nil, err
@@ -301,7 +301,7 @@ func (r *LocalRepository) GetCommit(commitHash string) (*Commit, error) {
 
 // GetLatestCommit returns the latest commit of the given path in the repository.
 func (r *LocalRepository) GetLatestCommit(path string) (*Commit, error) {
-	slog.Info("Retrieving the latest commit", "path", path)
+	slog.Info("retrieving the latest commit", "path", path)
 	opt := &git.LogOptions{
 		Order:    git.LogOrderCommitterTime,
 		FileName: &path,
@@ -450,7 +450,7 @@ func getHashForPath(commit *object.Commit, path string) (string, error) {
 
 // ChangedFilesInCommit returns the files changed in the given commit.
 func (r *LocalRepository) ChangedFilesInCommit(commitHash string) ([]string, error) {
-	slog.Debug("Getting changed files in commit", "hash", commitHash)
+	slog.Debug("getting changed files in commit", "hash", commitHash)
 	commit, err := r.repo.CommitObject(plumbing.NewHash(commitHash))
 	if err != nil {
 		return nil, fmt.Errorf("failed to get commit object for hash %s: %w", commitHash, err)
@@ -500,7 +500,7 @@ func (r *LocalRepository) ChangedFilesInCommit(commitHash string) ([]string, err
 // CreateBranchAndCheckout creates a new git branch and checks out the
 // branch in the local git repository.
 func (r *LocalRepository) CreateBranchAndCheckout(name string) error {
-	slog.Info("Creating branch and checking out", "name", name)
+	slog.Info("creating branch and checking out", "name", name)
 	worktree, err := r.repo.Worktree()
 	if err != nil {
 		return err
@@ -516,7 +516,7 @@ func (r *LocalRepository) CreateBranchAndCheckout(name string) error {
 func (r *LocalRepository) Push(branchName string) error {
 	// https://stackoverflow.com/a/75727620
 	refSpec := fmt.Sprintf("+refs/heads/%s:refs/heads/%s", branchName, branchName)
-	slog.Info("Pushing changes", "branch name", branchName, slog.Any("refspec", refSpec))
+	slog.Info("pushing changes", "branch name", branchName, slog.Any("refspec", refSpec))
 	return r.pushRefSpec(refSpec)
 }
 
@@ -527,7 +527,7 @@ func (r *LocalRepository) DeleteBranch(branchName string) error {
 }
 
 func (r *LocalRepository) pushRefSpec(refSpec string) error {
-	slog.Info("Pushing changes", "refSpec", refSpec)
+	slog.Info("pushing changes", "refSpec", refSpec)
 
 	// Check for the configured URI for the `origin` remote.
 	// If there are multiple URLs, the first one is selected.
@@ -558,7 +558,7 @@ func (r *LocalRepository) pushRefSpec(refSpec string) error {
 	}); err != nil {
 		return err
 	}
-	slog.Info("Successfully pushed changes", "refSpec", refSpec)
+	slog.Info("successfully pushed changes", "refSpec", refSpec)
 	return nil
 }
 
@@ -593,7 +593,7 @@ func canUseSSH(remoteURI string) bool {
 // remote repository. The useSSH determines if Basic Auth or SSH is used.
 func (r *LocalRepository) authCreds(useSSH bool) (transport.AuthMethod, error) {
 	if useSSH {
-		slog.Info("Authenticating with SSH")
+		slog.Info("authenticating with SSH")
 		// This is the generic `git` username when cloning via SSH. It is the value
 		// that exists before the URL. e.g. [email protected]:googleapis/librarian.git
 		auth, err := ssh.DefaultAuthBuilder("git")
@@ -602,7 +602,7 @@ func (r *LocalRepository) authCreds(useSSH bool) (transport.AuthMethod, error) {
 		}
 		return auth, nil
 	}
-	slog.Info("Authenticating with basic auth")
+	slog.Info("authenticating with basic auth")
 	return &httpAuth.BasicAuth{
 		// GitHub's authentication needs the username set to a non-empty value, but
 		// it does not need to match the token
@@ -619,7 +619,7 @@ func (r *LocalRepository) authCreds(useSSH bool) (transport.AuthMethod, error) {
 func (r *LocalRepository) Restore(paths []string) error {
 	args := []string{"restore"}
 	args = append(args, paths...)
-	slog.Info("Restoring uncommitted changes", "paths", strings.Join(paths, ","))
+	slog.Info("restoring uncommitted changes", "paths", strings.Join(paths, ","))
 	cmd := exec.Command("git", args...)
 	cmd.Stderr = os.Stderr
 	cmd.Stdout = os.Stdout
@@ -629,7 +629,7 @@ func (r *LocalRepository) Restore(paths []string) error {
 
 // CleanUntracked removes untracked files within the given paths.
 func (r *LocalRepository) CleanUntracked(paths []string) error {
-	slog.Info("Cleaning untracked files", "paths", strings.Join(paths, ","))
+	slog.Info("cleaning untracked files", "paths", strings.Join(paths, ","))
 	worktree, err := r.repo.Worktree()
 	if err != nil {
 		return err

internal/images/images.go

@@ -102,13 +102,13 @@ func (c *ArtifactRegistryClient) FindLatest(ctx context.Context, imageName strin
 	if err != nil {
 		return "", err
 	}
-	slog.Info("Found packages version", "version", version.GetName())
+	slog.Info("found packages version", "version", version.GetName())
 
 	// latest SHA is found as the "subjectDigest" metadata field
 	latestSha := ""
 	for key, field := range version.GetMetadata().GetFields() {
 		if key == "subjectDigest" {
-			slog.Info("Found SHA", "sha", field.GetStringValue())
+			slog.Info("found SHA", "sha", field.GetStringValue())
 			latestSha = field.GetStringValue()
 			break
 		}

internal/librarian/build.go

@@ -35,7 +35,7 @@ func buildSingleLibrary(ctx context.Context, containerClient ContainerClient, st
 		RepoDir:   repo.GetDir(),
 		State:     state,
 	}
-	slog.Info("Performing build for library", "id", libraryState.ID)
+	slog.Info("performing build for library", "id", libraryState.ID)
 	if containerErr := containerClient.Build(ctx, buildRequest); containerErr != nil {
 		if restoreErr := restoreLibrary(libraryState, repo); restoreErr != nil {
 			return errors.Join(containerErr, restoreErr)
@@ -54,6 +54,6 @@ func buildSingleLibrary(ctx context.Context, containerClient ContainerClient, st
 		return responseErr
 	}
 
-	slog.Info("Build succeeds", "id", libraryState.ID)
+	slog.Info("build succeeds", "id", libraryState.ID)
 	return nil
 }