Integrate azd-core v0.2.0 and Extract Testing Utilities (#26)

* refactor: rename AZD_SCRIPT_DEBUG to AZD_DEBUG and update related documentation

* chore: update dependencies and add new changelog entries

- Updated astro from 5.16.6 to 5.16.8
- Updated @playwright/test from 1.49.0 to 1.57.0
- Updated @types/node from 22.10.2 to 25.0.5
- Updated tailwindcss from 4.0.0 to 4.1.18
- Updated tsx from 4.19.2 to 4.21.0
- Updated typescript from 5.7.2 to 5.9.3
- Added new release entries for versions 0.2.30 and 0.2.29 in the changelog

* feat: extract testhelpers to azd-core/testutil package

- Created azd-core/testutil package with enhanced functionality.
- Extracted CaptureOutput, FindTestData, TempDir, and Contains helpers.
- Added comprehensive tests targeting ≥85% coverage.
- Updated azd-core README with testutil documentation.

feat: extract cliout package to azd-core for standardized CLI output

- Created azd-core/cliout package with full output formatting functions.
- Maintained Unicode detection, ANSI colors, and JSON mode.
- Added tests targeting ≥80% coverage.
- Updated azd-core README with cliout documentation.

refactor: migrate azd-exec to use azd-core/testutil

- Updated azd-exec to replace internal testhelpers with testutil.
- Deleted internal testhelpers package.
- Verified all tests pass with no regressions.

refactor: migrate azd-app to use azd-core/testutil

- Updated azd-app test infrastructure to utilize testutil.
- Enhanced existing tests with CaptureOutput and Contains.
- Verified all tests pass.

refactor: migrate azd-app output to azd-core/cliout

- Updated azd-app to import cliout from azd-core.
- Deleted internal output package.
- Verified CLI output remains unchanged.

feat: enhance azd-exec CLI output using cliout

- Updated azd-exec commands to use cliout for consistent output formatting.
- Added JSON output mode support.
- Verified all tests pass.

docs: create extension patterns guide in azd-core

- Documented version management, logging, and structure patterns.
- Included examples from azd-exec and azd-app.

chore: update azd-core v0.3.0 release notes

- Documented new packages (testutil, cliout) and integration impact.
- Updated CHANGELOG.md with v0.3.0 entry.

* chore: update azd-core target version to v0.2.0 in extraction specs and tasks

* fix: handle errors when setting output format in version command

* fix: use azd-core v0.2.1 and remove local replace directive for CI

Author: jongio

cli/docs/cli-reference.md

@@ -212,7 +212,7 @@ All azd environment variables are available to your scripts:
 - Variables from `.azure/<env>/.env` file
 
 **azd-exec Specific:**
-- `AZD_SCRIPT_DEBUG` - Set to "true" when `--debug` flag is used
+- `AZD_DEBUG` - Set to "true" when `--debug` flag is used
 - `AZD_NO_PROMPT` - Set to "true" when `--no-prompt` flag is used
 
 ### Azure Key Vault Integration

cli/docs/security-review.md

@@ -89,12 +89,12 @@ absPath, err := filepath.Abs(scriptPath)  // Resolve to absolute path
 - ✅ Environment variables inherited from parent process (`os.Environ()`)
 - ✅ Azd context variables passed through safely
 - ✅ No sensitive data logged by default
-- ✅ Debug mode explicitly requires `AZD_SCRIPT_DEBUG=true`
+- ✅ Debug mode explicitly requires `AZD_DEBUG=true`
 
 **Debug Output** (Optional, user-controlled):
 ```go
 // executor.go
-if os.Getenv("AZD_SCRIPT_DEBUG") == "true" {
+if os.Getenv("AZD_DEBUG") == "true" {
     fmt.Fprintf(os.Stderr, "Executing: %s %s\n", shell, strings.Join(cmd.Args[1:], " "))
     fmt.Fprintf(os.Stderr, "Working directory: %s\n", workingDir)
 }

cli/go.mod

@@ -3,9 +3,9 @@ module github.com/jongio/azd-exec/cli
 go 1.25.5
 
 require (
-	github.com/jongio/azd-core v0.1.1-0.20260109201524-89cce2a55fae
+	github.com/jongio/azd-core v0.2.1
 	github.com/magefile/mage v1.15.0
-	github.com/spf13/cobra v1.8.1
+	github.com/spf13/cobra v1.10.2
 )
 
 require (
@@ -20,9 +20,9 @@ require (
 	github.com/inconshreveable/mousetrap v1.1.0 // indirect
 	github.com/kylelemons/godebug v1.1.0 // indirect
 	github.com/pkg/browser v0.0.0-20240102092130-5ac0b6a4141c // indirect
-	github.com/spf13/pflag v1.0.5 // indirect
+	github.com/spf13/pflag v1.0.10 // indirect
 	golang.org/x/crypto v0.46.0 // indirect
 	golang.org/x/net v0.48.0 // indirect
 	golang.org/x/sys v0.40.0 // indirect
-	golang.org/x/text v0.32.0 // indirect
+	golang.org/x/text v0.33.0 // indirect
 )

cli/go.sum

@@ -14,17 +14,17 @@ github.com/AzureAD/microsoft-authentication-extensions-for-go/cache v0.1.1 h1:WJ
 github.com/AzureAD/microsoft-authentication-extensions-for-go/cache v0.1.1/go.mod h1:tCcJZ0uHAmvjsVYzEFivsRTN00oz5BEsRgQHu5JZ9WE=
 github.com/AzureAD/microsoft-authentication-library-for-go v1.6.0 h1:XRzhVemXdgvJqCH0sFfrBUTnUJSBrBf7++ypk+twtRs=
 github.com/AzureAD/microsoft-authentication-library-for-go v1.6.0/go.mod h1:HKpQxkWaGLJ+D/5H8QRpyQXA1eKjxkFlOMwck5+33Jk=
-github.com/cpuguy83/go-md2man/v2 v2.0.4/go.mod h1:tgQtvFlXSQOSOSIRvRPT7W67SCa46tRHOmNcaadrF8o=
-github.com/davecgh/go-spew v1.1.1 h1:vj9j/u1bqnvCEfJOwUhtlOARqs3+rkHYY13jYWTU97c=
-github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
+github.com/cpuguy83/go-md2man/v2 v2.0.6/go.mod h1:oOW0eioCTA6cOiMLiUPZOpcVxMig6NIQQ7OS05n1F4g=
+github.com/davecgh/go-spew v1.1.2-0.20180830191138-d8f796af33cc h1:U9qPSI2PIWSS1VwoXQT9A3Wy9MM3WgvqSxFWenqJduM=
+github.com/davecgh/go-spew v1.1.2-0.20180830191138-d8f796af33cc/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
 github.com/golang-jwt/jwt/v5 v5.3.0 h1:pv4AsKCKKZuqlgs5sUmn4x8UlGa0kEVt/puTpKx9vvo=
 github.com/golang-jwt/jwt/v5 v5.3.0/go.mod h1:fxCRLWMO43lRc8nhHWY6LGqRcf+1gQWArsqaEUEa5bE=
 github.com/google/uuid v1.6.0 h1:NIvaJDMOsjHA8n1jAhLSgzrAzy1Hgr+hNrb57e+94F0=
 github.com/google/uuid v1.6.0/go.mod h1:TIyPZe4MgqvfeYDBFedMoGGpEw/LqOeaOT+nhxU+yHo=
 github.com/inconshreveable/mousetrap v1.1.0 h1:wN+x4NVGpMsO7ErUn/mUI3vEoE6Jt13X2s0bqwp9tc8=
 github.com/inconshreveable/mousetrap v1.1.0/go.mod h1:vpF70FUmC8bwa3OWnCshd2FqLfsEA9PFc4w1p2J65bw=
-github.com/jongio/azd-core v0.1.1-0.20260109201524-89cce2a55fae h1:yLg4cfkgnNUvL5K7Ytnh2SHW2r9zoys85ESP2t+E/G4=
-github.com/jongio/azd-core v0.1.1-0.20260109201524-89cce2a55fae/go.mod h1:AJ0B6SuTUZkLdEeFpyfEhVRs6exoWzZz5cBL8gVgt/A=
+github.com/jongio/azd-core v0.2.1 h1:0xByH2Q3GOg5yQw3sRbwuOHsPKuDJcXIf3wTZ4foRJA=
+github.com/jongio/azd-core v0.2.1/go.mod h1:qHrhg2tFMs83tYRax0EC0Bl3EdRN1WDOhmQkKcNpIGs=
 github.com/keybase/go-keychain v0.0.1 h1:way+bWYa6lDppZoZcgMbYsvC7GxljxrskdNInRtuthU=
 github.com/keybase/go-keychain v0.0.1/go.mod h1:PdEILRW3i9D8JcdM+FmY6RwkHGnhHxXwkPPMeUgOK1k=
 github.com/kylelemons/godebug v1.1.0 h1:RPNrshWIDI6G2gRW9EHilWtl7Z6Sb1BR0xunSBf0SNc=
@@ -33,24 +33,26 @@ github.com/magefile/mage v1.15.0 h1:BvGheCMAsG3bWUDbZ8AyXXpCNwU9u5CB6sM+HNb9HYg=
 github.com/magefile/mage v1.15.0/go.mod h1:z5UZb/iS3GoOSn0JgWuiw7dxlurVYTu+/jHXqQg881A=
 github.com/pkg/browser v0.0.0-20240102092130-5ac0b6a4141c h1:+mdjkGKdHQG3305AYmdv1U2eRNDiU2ErMBj1gwrq8eQ=
 github.com/pkg/browser v0.0.0-20240102092130-5ac0b6a4141c/go.mod h1:7rwL4CYBLnjLxUqIJNnCWiEdr3bn6IUYi15bNlnbCCU=
-github.com/pmezard/go-difflib v1.0.0 h1:4DBwDE0NGyQoBHbLQYPwSUPoCMWR5BEzIk/f1lZbAQM=
-github.com/pmezard/go-difflib v1.0.0/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=
+github.com/pmezard/go-difflib v1.0.1-0.20181226105442-5d4384ee4fb2 h1:Jamvg5psRIccs7FGNTlIRMkT8wgtp5eCXdBlqhYGL6U=
+github.com/pmezard/go-difflib v1.0.1-0.20181226105442-5d4384ee4fb2/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=
 github.com/russross/blackfriday/v2 v2.1.0/go.mod h1:+Rmxgy9KzJVeS9/2gXHxylqXiyQDYRxCVz55jmeOWTM=
-github.com/spf13/cobra v1.8.1 h1:e5/vxKd/rZsfSJMUX1agtjeTDf+qv1/JdBF8gg5k9ZM=
-github.com/spf13/cobra v1.8.1/go.mod h1:wHxEcudfqmLYa8iTfL+OuZPbBZkmvliBWKIezN3kD9Y=
-github.com/spf13/pflag v1.0.5 h1:iy+VFUOCP1a+8yFto/drg2CJ5u0yRoB7fZw3DKv/JXA=
-github.com/spf13/pflag v1.0.5/go.mod h1:McXfInJRrz4CZXVZOBLb0bTZqETkiAhM9Iw0y3An2Bg=
+github.com/spf13/cobra v1.10.2 h1:DMTTonx5m65Ic0GOoRY2c16WCbHxOOw6xxezuLaBpcU=
+github.com/spf13/cobra v1.10.2/go.mod h1:7C1pvHqHw5A4vrJfjNwvOdzYu0Gml16OCs2GRiTUUS4=
+github.com/spf13/pflag v1.0.9/go.mod h1:McXfInJRrz4CZXVZOBLb0bTZqETkiAhM9Iw0y3An2Bg=
+github.com/spf13/pflag v1.0.10 h1:4EBh2KAYBwaONj6b2Ye1GiHfwjqyROoF4RwYO+vPwFk=
+github.com/spf13/pflag v1.0.10/go.mod h1:McXfInJRrz4CZXVZOBLb0bTZqETkiAhM9Iw0y3An2Bg=
 github.com/stretchr/testify v1.11.1 h1:7s2iGBzp5EwR7/aIZr8ao5+dra3wiQyKjjFuvgVKu7U=
 github.com/stretchr/testify v1.11.1/go.mod h1:wZwfW3scLgRK+23gO65QZefKpKQRnfz6sD981Nm4B6U=
+go.yaml.in/yaml/v3 v3.0.4/go.mod h1:DhzuOOF2ATzADvBadXxruRBLzYTpT36CKvDb3+aBEFg=
 golang.org/x/crypto v0.46.0 h1:cKRW/pmt1pKAfetfu+RCEvjvZkA9RimPbh7bhFjGVBU=
 golang.org/x/crypto v0.46.0/go.mod h1:Evb/oLKmMraqjZ2iQTwDwvCtJkczlDuTmdJXoZVzqU0=
 golang.org/x/net v0.48.0 h1:zyQRTTrjc33Lhh0fBgT/H3oZq9WuvRR5gPC70xpDiQU=
 golang.org/x/net v0.48.0/go.mod h1:+ndRgGjkh8FGtu1w1FGbEC31if4VrNVMuKTgcAAnQRY=
 golang.org/x/sys v0.1.0/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
 golang.org/x/sys v0.40.0 h1:DBZZqJ2Rkml6QMQsZywtnjnnGvHza6BTfYFWY9kjEWQ=
 golang.org/x/sys v0.40.0/go.mod h1:OgkHotnGiDImocRcuBABYBEXf8A9a87e/uXjp9XT3ks=
-golang.org/x/text v0.32.0 h1:ZD01bjUt1FQ9WJ0ClOL5vxgxOI/sVCNgX1YtKwcY0mU=
-golang.org/x/text v0.32.0/go.mod h1:o/rUWzghvpD5TXrTIBuJU77MTaN0ljMWE47kxGJQ7jY=
+golang.org/x/text v0.33.0 h1:B3njUFyqtHDUI5jMn1YIr5B0IE2U0qck04r6d4KPAxE=
+golang.org/x/text v0.33.0/go.mod h1:LuMebE6+rBincTi9+xWTY8TztLzKHc/9C1uBCG27+q8=
 gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0=
 gopkg.in/yaml.v3 v3.0.1 h1:fxVm/GzAzEWqLHuvctI91KS9hhNmmWOoWu0XTYJS7CA=
 gopkg.in/yaml.v3 v3.0.1/go.mod h1:K4uyk7z7BCEPqu6E+C64Yfv1cQ7kz7rIZviUmN+EgEM=

cli/src/cmd/exec/commands/listen.go

@@ -2,6 +2,7 @@
 package commands
 
 import (
+	"github.com/jongio/azd-core/cliout"
 	"github.com/spf13/cobra"
 )
 
@@ -21,6 +22,7 @@ func NewListenCommand() *cobra.Command {
 			// In a full implementation, this would start a gRPC server for azd communication.
 			// For now, this extension operates in "exec mode" without persistent listener,
 			// so returning nil is appropriate.
+			cliout.Info("Listen command is not implemented - azd-exec operates in exec mode")
 			return nil
 		},
 	}

cli/src/cmd/exec/commands/version.go

@@ -4,8 +4,8 @@ package commands
 import (
 	"encoding/json"
 	"fmt"
-	"os"
 
+	"github.com/jongio/azd-core/cliout"
 	"github.com/jongio/azd-exec/cli/src/internal/version"
 	"github.com/spf13/cobra"
 )
@@ -21,22 +21,43 @@ func NewVersionCommand(outputFormat *string) *cobra.Command {
 		Short: "Display the extension version",
 		Long:  `Display the version information for the azd exec extension.`,
 		Run: func(cmd *cobra.Command, args []string) {
-			switch *outputFormat {
-			case "json":
+			// Set output format from flag
+			if *outputFormat == "json" {
+				if err := cliout.SetFormat("json"); err != nil {
+					cliout.Error("Failed to set output format: %v", err)
+					return
+				}
+			} else {
+				if err := cliout.SetFormat("default"); err != nil {
+					cliout.Error("Failed to set output format: %v", err)
+					return
+				}
+			}
+
+			if cliout.IsJSON() {
+				// JSON output mode
 				output := map[string]string{
 					"version": version.Version,
 				}
 				data, err := json.MarshalIndent(output, "", "  ")
 				if err != nil {
-					fmt.Fprintf(os.Stderr, "Error formatting JSON: %v\n", err)
+					cliout.Error("Error formatting JSON: %v", err)
 					return
 				}
 				fmt.Println(string(data))
-			default:
+			} else {
+				// Human-readable output with colors
 				if quiet {
 					fmt.Println(version.Version)
 				} else {
-					fmt.Printf("azd exec version %s\n", version.Version)
+					cliout.Header("azd exec")
+					cliout.Label("Version", version.Version)
+					if version.BuildDate != "unknown" {
+						cliout.Label("Build Date", version.BuildDate)
+					}
+					if version.GitCommit != "unknown" {
+						cliout.Label("Git Commit", version.GitCommit)
+					}
 				}
 			}
 		},

cli/src/cmd/exec/commands/version_integration_test.go

@@ -8,7 +8,7 @@ import (
 	"strings"
 	"testing"
 
-	"github.com/jongio/azd-exec/cli/src/internal/testhelpers"
+	"github.com/jongio/azd-core/testutil"
 	"github.com/jongio/azd-exec/cli/src/internal/version"
 )
 
@@ -25,7 +25,7 @@ func TestVersionCommandIntegration(t *testing.T) {
 		{
 			name:       "Default output",
 			outputFlag: "default",
-			wantText:   "azd exec version",
+			wantText:   "azd exec",
 		},
 		{
 			name:       "JSON output",
@@ -50,7 +50,7 @@ func TestVersionCommandIntegration(t *testing.T) {
 			}
 
 			// Capture output
-			output := testhelpers.CaptureOutput(t, func() error {
+			output := testutil.CaptureOutput(t, func() error {
 				return cmd.Execute()
 			})
 
@@ -82,7 +82,7 @@ func TestVersionCommandIntegration_QuietFlag(t *testing.T) {
 	cmd := NewVersionCommand(&outputFormat)
 	cmd.Flags().Set("quiet", "true")
 
-	output := testhelpers.CaptureOutput(t, func() error {
+	output := testutil.CaptureOutput(t, func() error {
 		return cmd.Execute()
 	})
 
@@ -106,7 +106,7 @@ func TestVersionCommandIntegration_JSONFormat(t *testing.T) {
 	outputFormat := "json"
 	cmd := NewVersionCommand(&outputFormat)
 
-	output := testhelpers.CaptureOutput(t, func() error {
+	output := testutil.CaptureOutput(t, func() error {
 		return cmd.Execute()
 	})
 

cli/src/cmd/exec/main.go

@@ -9,6 +9,7 @@ import (
 	"os"
 	"path/filepath"
 
+	"github.com/jongio/azd-core/cliout"
 	"github.com/jongio/azd-exec/cli/src/cmd/exec/commands"
 	"github.com/jongio/azd-exec/cli/src/internal/executor"
 	"github.com/spf13/cobra"
@@ -48,7 +49,7 @@ var newScriptExecutor = func(config executor.Config) scriptExecutor {
 func main() {
 	rootCmd := newRootCmd()
 	if err := rootCmd.Execute(); err != nil {
-		fmt.Fprintln(os.Stderr, err)
+		cliout.Error("%v", err)
 		os.Exit(1)
 	}
 }
@@ -100,6 +101,13 @@ Examples:
 			return exec.ExecuteInline(cmd.Context(), scriptInput)
 		},
 		PersistentPreRunE: func(cmd *cobra.Command, args []string) error {
+			// Set output format from flag
+			if outputFormat == "json" {
+				if err := cliout.SetFormat("json"); err != nil {
+					return fmt.Errorf("failed to set output format: %w", err)
+				}
+			}
+
 			// Handle working directory change
 			if cwd != "" {
 				if err := os.Chdir(cwd); err != nil {
@@ -109,7 +117,7 @@ Examples:
 
 			// Handle debug mode
 			if debugMode {
-				_ = os.Setenv("AZD_SCRIPT_DEBUG", "true")
+				_ = os.Setenv("AZD_DEBUG", "true")
 			}
 
 			// Handle no-prompt mode

cli/src/cmd/exec/main_test.go

@@ -33,7 +33,7 @@ func TestPersistentPreRunE_SetsEnvAndCwd(t *testing.T) {
 
 	newWd := t.TempDir()
 
-	oldDebug := os.Getenv("AZD_SCRIPT_DEBUG")
+	oldDebug := os.Getenv("AZD_DEBUG")
 	oldNoPrompt := os.Getenv("AZD_NO_PROMPT")
 	oldEnvName := os.Getenv("AZURE_ENV_NAME")
 	oldTraceFile := os.Getenv("AZD_TRACE_LOG_FILE")
@@ -83,8 +83,8 @@ func TestPersistentPreRunE_SetsEnvAndCwd(t *testing.T) {
 		t.Fatalf("expected cwd %q, got %q", newWdNorm, gotWdNorm)
 	}
 
-	if os.Getenv("AZD_SCRIPT_DEBUG") != "true" {
-		t.Fatalf("expected AZD_SCRIPT_DEBUG=true")
+	if os.Getenv("AZD_DEBUG") != "true" {
+		t.Fatalf("expected AZD_DEBUG=true")
 	}
 	if os.Getenv("AZD_NO_PROMPT") != "true" {
 		t.Fatalf("expected AZD_NO_PROMPT=true")
@@ -101,7 +101,7 @@ func TestPersistentPreRunE_SetsEnvAndCwd(t *testing.T) {
 
 	// Restore process state.
 	_ = os.Chdir(oldWd)
-	_ = os.Setenv("AZD_SCRIPT_DEBUG", oldDebug)
+	_ = os.Setenv("AZD_DEBUG", oldDebug)
 	_ = os.Setenv("AZD_NO_PROMPT", oldNoPrompt)
 	_ = os.Setenv("AZURE_ENV_NAME", oldEnvName)
 	_ = os.Setenv("AZD_TRACE_LOG_FILE", oldTraceFile)

cli/src/internal/executor/command_shell_test.go

@@ -5,6 +5,8 @@ import (
 	"path/filepath"
 	"runtime"
 	"testing"
+
+	"github.com/jongio/azd-core/shellutil"
 )
 
 func TestBuildCommand_Bash(t *testing.T) {
@@ -201,7 +203,6 @@ func TestBuildCommand_CaseInsensitiveShellNames(t *testing.T) {
 }
 
 func TestReadShebang_ValidShebang(t *testing.T) {
-	exec := New(Config{})
 	tmpDir := t.TempDir()
 
 	tests := []struct {
@@ -243,7 +244,7 @@ func TestReadShebang_ValidShebang(t *testing.T) {
 				t.Fatal(err)
 			}
 
-			result := exec.readShebang(scriptPath)
+			result := shellutil.ReadShebang(scriptPath)
 			if result != tt.expected {
 				t.Errorf("Expected %q, got %q", tt.expected, result)
 			}
@@ -252,7 +253,6 @@ func TestReadShebang_ValidShebang(t *testing.T) {
 }
 
 func TestReadShebang_NoShebang(t *testing.T) {
-	exec := New(Config{})
 	tmpDir := t.TempDir()
 
 	tests := []struct {
@@ -280,7 +280,7 @@ func TestReadShebang_NoShebang(t *testing.T) {
 				t.Fatal(err)
 			}
 
-			result := exec.readShebang(scriptPath)
+			result := shellutil.ReadShebang(scriptPath)
 			if result != "" {
 				t.Errorf("Expected empty string, got %q", result)
 			}
@@ -289,10 +289,8 @@ func TestReadShebang_NoShebang(t *testing.T) {
 }
 
 func TestReadShebang_FileErrors(t *testing.T) {
-	exec := New(Config{})
-
 	t.Run("Nonexistent file", func(t *testing.T) {
-		result := exec.readShebang("/nonexistent/file.sh")
+		result := shellutil.ReadShebang("/nonexistent/file.sh")
 		if result != "" {
 			t.Errorf("Expected empty string for nonexistent file, got %q", result)
 		}