chore(all): prefix env with LIBRARIAN_ (#2749)

Update environment variable to prefix with `LIBRARIAN_`.

Fixes #2716

Author: JoeWang1127

doc/testing.md

@@ -39,11 +39,11 @@ tests are **NOT** run automatically as they create pull requests and branches.
 Usage:
 
 ```bash
-TEST_GITHUB_TOKEN=<a personal access token> \
-  TEST_GITHUB_REPO=<URL of GitHub repo> \
+LIBRARIAN_TEST_GITHUB_TOKEN=<a personal access token> \
+  LIBRARIAN_TEST_GITHUB_REPO=<URL of GitHub repo> \
   go test ./...
 ```
 
-Note: `TEST_GITHUB_TOKEN` must have write access to `TEST_GITHUB_REPO`.
+Note: `LIBRARIAN_TEST_GITHUB_TOKEN` must have write access to `LIBRARIAN_TEST_GITHUB_REPO`.
 
 Note: These tests are skipped unless these environment variables are set.

internal/config/config.go

@@ -57,7 +57,7 @@ const (
 	LibrarianStateFile = "state.yaml"
 	// LibrarianConfigFile is the name of the language-repository config file.
 	LibrarianConfigFile = "config.yaml"
-	// LibrarianGithubToken is the name of the env var used to store the github token.
+	// LibrarianGithubToken is the name of the env var used to store the GitHub token.
 	LibrarianGithubToken = "LIBRARIAN_GITHUB_TOKEN"
 )
 
@@ -68,7 +68,7 @@ var (
 )
 
 var (
-	// pullRequestRegexp is regular expression that describes a uri of a pull request.
+	// pullRequestRegexp is regular expression that describes an uri of a pull request.
 	pullRequestRegexp = regexp.MustCompile(`^https://github\.com/([a-zA-Z0-9-._]+)/([a-zA-Z0-9-._]+)/pull/([0-9]+)$`)
 )
 
@@ -78,7 +78,7 @@ var (
 type Config struct {
 	// API is the path to the API to be configured or generated,
 	// relative to the root of the googleapis repository. It is a directory
-	// name as far as (and including) the version (v1, v2, v1alpha etc). It
+	// name as far as (and including) the version (v1, v2, v1alpha etc.). It
 	// is expected to contain a service config YAML file.
 	// Example: "google/cloud/functions/v2"
 	//

internal/container/java/build-docker-and-test.sh

@@ -22,7 +22,7 @@ echo "Building Docker image..."
 docker build -t "${IMAGE_NAME}" "${SCRIPT_DIR}"
 
 echo "Running version check..."
-output=$(docker run --rm -e GOOGLE_SDK_JAVA_LOGGING_LEVEL=quiet "${IMAGE_NAME}" --version)
+output=$(docker run --rm -e LIBRARIAN_GOOGLE_SDK_JAVA_LOGGING_LEVEL=quiet "${IMAGE_NAME}" --version)
 
 if [[ ! "$output" =~ ^[0-9]+\.[0-9]+\.[0-9]+$ ]]; then
   echo "Version format is incorrect. Got: $output"

internal/container/java/main.go

@@ -32,7 +32,7 @@ func main() {
 }
 
 func runCLI(args []string) int {
-	logLevel := parseLogLevel(os.Getenv("GOOGLE_SDK_JAVA_LOGGING_LEVEL"))
+	logLevel := parseLogLevel(os.Getenv("LIBRARIAN_GOOGLE_SDK_JAVA_LOGGING_LEVEL"))
 	slog.SetDefault(slog.New(slog.NewTextHandler(os.Stdout, &slog.HandlerOptions{
 		Level: logLevel,
 	})))

system_test.go

@@ -31,12 +31,12 @@ import (
 	"github.com/googleapis/librarian/internal/images"
 )
 
-var testToken = os.Getenv("TEST_GITHUB_TOKEN")
-var githubAction = os.Getenv("GITHUB_ACTION")
+var testToken = os.Getenv("LIBRARIAN_TEST_GITHUB_TOKEN")
+var githubAction = os.Getenv("LIBRARIAN_GITHUB_ACTION")
 
 func TestGetRawContentSystem(t *testing.T) {
 	if testToken == "" {
-		t.Skip("TEST_GITHUB_TOKEN not set, skipping GitHub integration test")
+		t.Skip("LIBRARIAN_TEST_GITHUB_TOKEN not set, skipping GitHub integration test")
 	}
 	repoName := "https://github.com/googleapis/librarian"
 
@@ -114,11 +114,11 @@ func TestPullRequestSystem(t *testing.T) {
 	// Fetch the pull request
 	// Close the pull request
 	if testToken == "" {
-		t.Skip("TEST_GITHUB_TOKEN not set, skipping GitHub integration test")
+		t.Skip("LIBRARIAN_TEST_GITHUB_TOKEN not set, skipping GitHub integration test")
 	}
-	testRepoName := os.Getenv("TEST_GITHUB_REPO")
+	testRepoName := os.Getenv("LIBRARIAN_TEST_GITHUB_REPO")
 	if testRepoName == "" {
-		t.Skip("TEST_GITHUB_REPO not set, skipping GitHub integration test")
+		t.Skip("LIBRARIAN_TEST_GITHUB_REPO not set, skipping GitHub integration test")
 	}
 
 	// Clone a repo
@@ -298,7 +298,7 @@ func TestPullRequestSystem(t *testing.T) {
 
 func TestFindMergedPullRequest(t *testing.T) {
 	if testToken == "" {
-		t.Skip("TEST_GITHUB_TOKEN not set, skipping GitHub integration test")
+		t.Skip("LIBRARIAN_TEST_GITHUB_TOKEN not set, skipping GitHub integration test")
 	}
 	repoName := "https://github.com/googleapis/librarian"
 	repo, err := github.ParseRemote(repoName)
@@ -364,11 +364,11 @@ func TestFindMergedPullRequest(t *testing.T) {
 
 func TestCreateTag(t *testing.T) {
 	if testToken == "" {
-		t.Skip("TEST_GITHUB_TOKEN not set, skipping GitHub integration test")
+		t.Skip("LIBRARIAN_TEST_GITHUB_TOKEN not set, skipping GitHub integration test")
 	}
-	testRepoName := os.Getenv("TEST_GITHUB_REPO")
+	testRepoName := os.Getenv("LIBRARIAN_TEST_GITHUB_REPO")
 	if testRepoName == "" {
-		t.Skip("TEST_GITHUB_REPO not set, skipping GitHub integration test")
+		t.Skip("LIBRARIAN_TEST_GITHUB_REPO not set, skipping GitHub integration test")
 	}
 	repo, err := github.ParseRemote(testRepoName)
 	if err != nil {
@@ -404,11 +404,11 @@ func TestCreateTag(t *testing.T) {
 
 func TestCreateRelease(t *testing.T) {
 	if testToken == "" {
-		t.Skip("TEST_GITHUB_TOKEN not set, skipping GitHub integration test")
+		t.Skip("LIBRARIAN_TEST_GITHUB_TOKEN not set, skipping GitHub integration test")
 	}
-	testRepoName := os.Getenv("TEST_GITHUB_REPO")
+	testRepoName := os.Getenv("LIBRARIAN_TEST_GITHUB_REPO")
 	if testRepoName == "" {
-		t.Skip("TEST_GITHUB_REPO not set, skipping GitHub integration test")
+		t.Skip("LIBRARIAN_TEST_GITHUB_REPO not set, skipping GitHub integration test")
 	}
 	repo, err := github.ParseRemote(testRepoName)
 	if err != nil {