syncing model-customization with main (#6428)

* adding docs for the internal test recorder (#6350)

* adding docs for the internal test recorder

* spell

* Remove deprecated `login` and `logout` commands (#6395)

* Initial plan

* Remove deprecated login and logout commands

Co-authored-by: spboyer <7681382+spboyer@users.noreply.github.com>

* Add helpful error messages for removed login/logout commands

Co-authored-by: spboyer <7681382+spboyer@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: spboyer <7681382+spboyer@users.noreply.github.com>

* Add `azd env config` commands for environment-specific configuration management (#6348)

* Initial plan

* Implement env config get, set, and unset commands

Co-authored-by: vhvb1989 <24213737+vhvb1989@users.noreply.github.com>

* Fix error message formatting to match existing patterns

Co-authored-by: vhvb1989 <24213737+vhvb1989@users.noreply.github.com>

* Final update - all implementation and testing complete

Co-authored-by: vhvb1989 <24213737+vhvb1989@users.noreply.github.com>

* Add comprehensive unit tests for env config commands

- Added 6 test functions with 23 test cases total
- Tests cover all three commands: get, set, and unset
- Test cases include: simple values, nested values, objects, error handling, default environment, and multiple operations
- All tests passing with 100% coverage of new functionality

Co-authored-by: vhvb1989 <24213737+vhvb1989@users.noreply.github.com>

* Run gofmt on env_config_test.go to fix formatting

- Adjusted struct field alignment in test cases
- All tests still passing

Co-authored-by: wbreza <6540159+wbreza@users.noreply.github.com>

* Update test snapshots for new env config commands

- Added snapshots for azd env config, get, set, and unset commands
- Updated TestUsage-azd-env.snap to include config subcommand
- Updated TestFigSpec.ts with new command definitions
- All snapshot tests passing

Co-authored-by: JeffreyCA <9157833+JeffreyCA@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: vhvb1989 <24213737+vhvb1989@users.noreply.github.com>
Co-authored-by: wbreza <6540159+wbreza@users.noreply.github.com>
Co-authored-by: JeffreyCA <9157833+JeffreyCA@users.noreply.github.com>

---------

Co-authored-by: Victor Vazquez <vhvb1989@gmail.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: spboyer <7681382+spboyer@users.noreply.github.com>
Co-authored-by: vhvb1989 <24213737+vhvb1989@users.noreply.github.com>
Co-authored-by: wbreza <6540159+wbreza@users.noreply.github.com>
Co-authored-by: JeffreyCA <9157833+JeffreyCA@users.noreply.github.com>

Author: 6 people

cli/azd/.vscode/cspell.yaml

@@ -243,6 +243,12 @@ overrides:
       - userosscache
       - docstates
       - dylib
+  - filename: docs/recording-functional-tests-guide.md
+    words:
+      - httptest
+      - Logf
+      - Getenv
+      - httptest      
 ignorePaths:
   - "**/*_test.go"
   - "**/mock*.go"

cli/azd/cmd/auth_login.go

@@ -196,13 +196,6 @@ func (lf *loginFlags) Bind(local *pflag.FlagSet, global *internal.GlobalCommandO
 	lf.global = global
 }
 
-func newLoginFlags(cmd *cobra.Command, global *internal.GlobalCommandOptions) *loginFlags {
-	flags := &loginFlags{}
-	flags.Bind(cmd.Flags(), global)
-
-	return flags
-}
-
 func newLoginCmd(parent string) *cobra.Command {
 	return &cobra.Command{
 		Use:   "login",
@@ -237,9 +230,6 @@ type loginAction struct {
 	commandRunner     exec.CommandRunner
 }
 
-// it is important to update both newAuthLoginAction and newLoginAction at the same time
-// newAuthLoginAction is the action that is bound to `azd auth login`,
-// and newLoginAction is the action that is bound to `azd login`
 func newAuthLoginAction(
 	formatter output.Formatter,
 	writer io.Writer,
@@ -262,31 +252,6 @@ func newAuthLoginAction(
 	}
 }
 
-// it is important to update both newAuthLoginAction and newLoginAction at the same time
-// newAuthLoginAction is the action that is bound to `azd auth login`,
-// and newLoginAction is the action that is bound to `azd login`
-func newLoginAction(
-	formatter output.Formatter,
-	writer io.Writer,
-	authManager *auth.Manager,
-	accountSubManager *account.SubscriptionsManager,
-	flags *loginFlags,
-	console input.Console,
-	annotations CmdAnnotations,
-	commandRunner exec.CommandRunner,
-) actions.Action {
-	return &loginAction{
-		formatter:         formatter,
-		writer:            writer,
-		console:           console,
-		authManager:       authManager,
-		accountSubManager: accountSubManager,
-		flags:             flags,
-		annotations:       annotations,
-		commandRunner:     commandRunner,
-	}
-}
-
 func (la *loginAction) Run(ctx context.Context) (*actions.ActionResult, error) {
 	if len(la.flags.scopes) == 0 {
 		la.flags.scopes = la.authManager.LoginScopes()

cli/azd/cmd/auto_install.go

@@ -385,6 +385,18 @@ func ExecuteWithAutoInstall(ctx context.Context, rootContainer *ioc.NestedContai
 			log.Panic("failed to resolve console for unknown flags error:", err)
 		}
 
+		// Check for deprecated commands and provide helpful redirection messages
+		if unknownCommand == "login" {
+			console.Message(ctx, "Error: The 'azd login' command has been removed.")
+			console.Message(ctx, "Please use 'azd auth login' instead.")
+			return fmt.Errorf("unknown command 'login'")
+		}
+		if unknownCommand == "logout" {
+			console.Message(ctx, "Error: The 'azd logout' command has been removed.")
+			console.Message(ctx, "Please use 'azd auth logout' instead.")
+			return fmt.Errorf("unknown command 'logout'")
+		}
+
 		// If unknown flags were found before a non-built-in command, return an error with helpful guidance
 		if len(unknownFlags) > 0 {
 			flagsList := strings.Join(unknownFlags, ", ")