Add linode provider updates + general cleanup/refactor

Author: theognis1002

AGENTS.md

@@ -320,14 +320,18 @@ When adding framework detection:
 
 ### Provider Registry System
 
-Lightfold uses a provider registry pattern that makes adding new cloud providers trivial. All providers implement the `Provider` interface and auto-register themselves at init time.
+Lightfold uses a provider registry pattern that makes adding new cloud providers straightforward. All providers implement the `Provider` interface and auto-register themselves at init time.
 
 **Key Components:**
 1. **Provider Interface** (`pkg/providers/provider.go`): Defines standard methods all providers must implement
 2. **Provider Registry** (`pkg/providers/registry.go`): Central factory for creating provider instances
 3. **Provider Implementations**: Self-contained packages (e.g., `pkg/providers/digitalocean/`, `pkg/providers/hetzner/`)
 
 **Adding a New Provider:**
+
+See **[docs/ADDING_NEW_PROVIDERS.md](docs/ADDING_NEW_PROVIDERS.md)** for a complete step-by-step integration guide.
+
+Quick overview:
 ```go
 // 1. Create pkg/providers/newprovider/client.go
 package newprovider
@@ -345,7 +349,11 @@ func (c *Client) DisplayName() string { return "New Provider" }
 // ... implement remaining interface methods
 ```
 
-That's it! The provider is now available throughout the application with zero changes to orchestrator, CLI, or config layers.
+**Critical Integration Points** (see guide for details):
+- Import provider in `pkg/deploy/orchestrator.go` (blank import)
+- Add config struct to `pkg/config/config.go`
+- Add switch cases to 5 key functions (orchestrator + utilities)
+- Test provisioning, IP recovery, and multi-app flows
 
 ### Deployment Flow Architecture
 

cmd/deploy.go

@@ -315,7 +315,7 @@ Examples:
 			fmt.Printf("%s %s\n", deploySuccessStyle.Render("✓"), deployMutedStyle.Render("Configuring environment variables..."))
 		}
 
-		if err := executor.DeployWithHealthCheck(releasePath, 5, 3*time.Second); err != nil {
+		if err := executor.DeployWithHealthCheck(releasePath, target.Port, 5, 3*time.Second); err != nil {
 			state.MarkPushFailed(targetName, fmt.Sprintf("deployment failed: %v", err))
 			fmt.Fprintf(os.Stderr, "Error during deployment: %v\n", err)
 			os.Exit(1)
@@ -349,13 +349,26 @@ Examples:
 			fmt.Sprintf("%s %s", deployMutedStyle.Render("Release:"), deployValueStyle.Render(releaseTimestamp)),
 		}
 
-		// Add port information
+		// Check if this is a multi-app deployment
+		serverState, serverStateErr := state.GetServerState(sshProviderCfg.GetIP())
+		isMultiApp := serverStateErr == nil && len(serverState.DeployedApps) > 1
+
+		// Add port and access information
 		if target.Port > 0 {
 			if target.Domain != nil && target.Domain.Domain != "" {
-				successLines = append(successLines, fmt.Sprintf("%s %s (proxied via nginx)", deployMutedStyle.Render("Port:"), deployValueStyle.Render(fmt.Sprintf("%d", target.Port))))
+				// Has custom domain - nginx proxies to app port
+				// Don't show port for single-app with domain
+				if isMultiApp {
+					successLines = append(successLines, fmt.Sprintf("%s %s (proxied via nginx)", deployMutedStyle.Render("Port:"), deployValueStyle.Render(fmt.Sprintf("%d", target.Port))))
+				}
+			} else if !isMultiApp {
+				// Single-app without domain - nginx proxies port 80 to app port
+				// Don't show internal port, just the access URL
+				successLines = append(successLines, fmt.Sprintf("%s %s", deployMutedStyle.Render("Access:"), deployValueStyle.Render(fmt.Sprintf("http://%s", sshProviderCfg.GetIP()))))
 			} else {
+				// Multi-app without domain - needs domain or direct port access
 				successLines = append(successLines, fmt.Sprintf("%s %s", deployMutedStyle.Render("Port:"), deployValueStyle.Render(fmt.Sprintf("%d", target.Port))))
-				successLines = append(successLines, fmt.Sprintf("%s %s", deployMutedStyle.Render("Access:"), deployValueStyle.Render(fmt.Sprintf("http://%s:%d", sshProviderCfg.GetIP(), target.Port))))
+				successLines = append(successLines, fmt.Sprintf("%s %s", deployMutedStyle.Render("Access:"), deployValueStyle.Render(fmt.Sprintf("http://%s:%d (direct port access)", sshProviderCfg.GetIP(), target.Port))))
 			}
 		}
 

cmd/push.go

@@ -267,7 +267,7 @@ Examples:
 			fmt.Printf("%s %s\n", pushSuccessStyle.Render("✓"), pushMutedStyle.Render("Configuring environment variables..."))
 		}
 
-		if err := executor.DeployWithHealthCheck(releasePath, 5, 3*time.Second); err != nil {
+		if err := executor.DeployWithHealthCheck(releasePath, target.Port, 5, 3*time.Second); err != nil {
 			state.MarkPushFailed(targetNameResolved, fmt.Sprintf("deployment failed: %v", err))
 			fmt.Fprintf(os.Stderr, "Error during deployment: %v\n", err)
 			os.Exit(1)

cmd/root.go

@@ -13,7 +13,7 @@ import (
 	"github.com/spf13/cobra"
 )
 
-const Version = "0.1.2"
+const Version = "0.1.3"
 
 var (
 	jsonOutput      bool

cmd/ui/sequential/model.go

@@ -559,7 +559,7 @@ func (m FlowModel) renderHelp() string {
 func (m FlowModel) renderCompleted() string {
 	mutedStyle := lipgloss.NewStyle().Foreground(lipgloss.Color("245"))
 
-	var content string
+	var lines []string
 
 	for i, step := range m.Steps {
 		if stepState, exists := m.StepStates[i]; exists {
@@ -593,17 +593,25 @@ func (m FlowModel) renderCompleted() string {
 			}
 
 			if value != "" {
-				content += fmt.Sprintf("%s %s\n", label, value)
+				lines = append(lines, fmt.Sprintf("%s %s", label, value))
 			}
 		}
 	}
 
+	content := ""
+	for i, line := range lines {
+		content += line
+		if i < len(lines)-1 {
+			content += "\n"
+		}
+	}
+
 	mutedBox := lipgloss.NewStyle().
 		Border(lipgloss.RoundedBorder()).
 		BorderForeground(lipgloss.Color("245")).
-		Padding(0, 1)
+		Padding(0, 1, 0, 1)
 
-	return "\n" + mutedBox.Render(mutedStyle.Render(content)) + "\n"
+	return "\n" + mutedBox.Render(mutedStyle.Render(content))
 }
 
 func (m FlowModel) GetResults() map[string]string {

cmd/utils/provider_recovery.go

@@ -85,6 +85,14 @@ func updateProviderConfigWithIP(target *config.TargetConfig, providerName, ip, s
 		vultrConfig.IP = ip
 		vultrConfig.InstanceID = serverID
 		return target.SetProviderConfig("vultr", vultrConfig)
+	case "linode":
+		linodeConfig, err := target.GetLinodeConfig()
+		if err != nil {
+			return err
+		}
+		linodeConfig.IP = ip
+		linodeConfig.InstanceID = serverID
+		return target.SetProviderConfig("linode", linodeConfig)
 	default:
 		return fmt.Errorf("unsupported provider: %s", providerName)
 	}

cmd/utils/server_setup.go

@@ -97,6 +97,15 @@ func SetupTargetWithExistingServer(target *config.TargetConfig, serverIP string,
 				Provisioned: false,
 			}
 			target.SetProviderConfig("vultr", vultrConfig)
+		case "linode":
+			linodeConfig := &config.LinodeConfig{
+				IP:          serverIP,
+				InstanceID:  serverState.ServerID,
+				SSHKey:      sshKey,
+				Username:    "deploy",
+				Provisioned: false,
+			}
+			target.SetProviderConfig("linode", linodeConfig)
 		case "byos":
 			// For BYOS, use the SSH key we found
 			doConfig := &config.DigitalOceanConfig{