feat: implement extension build and test commands

- Add 'efctl env extension build' to compile Move contracts
- Add 'efctl env extension test' to run Move contract tests
- Extract common environment preparation logic in pkg/builder
- Fix Move compilation errors in turret_aggressor_first extension
- Update E2E tests and project TODO.md

Author: Scetrov

TODO.md

@@ -12,9 +12,9 @@ Feature ideas and improvements for `efctl`.
 
 ## Builder Flow
 
-- [ ] **`efctl env extension test [contract-path]`** — Run `sui move test` for a Move contract inside the container, so developers don't need to `shell` in or use `env run`.
-- [ ] **`efctl env extension build [contract-path]`** — Compile a Move contract without publishing, catching errors earlier.
-- [ ] **`efctl env extension list`** — List published extensions with their package IDs, config IDs, and status from the `.env`.
+- [x] **`efctl env extension test [contract-path]`** — Run `sui move test` for a Move contract inside the container, so developers don't need to `shell` in or use `env run`.
+- [x] **`efctl env extension build [contract-path]`** — Compile a Move contract without publishing, catching errors earlier.
+- [x] **`efctl env extension list`** — List published extensions with their package IDs, config IDs, and status from the `.env`.
 
 ## GraphQL & Chain Interaction
 
@@ -24,14 +24,14 @@ Feature ideas and improvements for `efctl`.
 
 ## Developer Experience
 
-- [ ] **`efctl env init`** — Scaffold a new project directory with a starter `efctl.yaml`, Move contract template, and directory structure, reducing manual boilerplate for new builders.
-- [ ] **`efctl doctor`** — Comprehensive diagnostic that checks prerequisites, port conflicts, Docker daemon health, Sui client config, disk space, and version compatibility — then outputs a shareable report.
+- [x] **`efctl env init`** — Scaffold a new project directory with a starter `efctl.yaml`, Move contract template, and directory structure, reducing manual boilerplate for new builders. (Implemented as `efctl init`).
+- [x] **`efctl doctor`** — Comprehensive diagnostic that checks prerequisites, port conflicts, Docker daemon health, Sui client config, disk space, and version compatibility — then outputs a shareable report.
 - [x] **`efctl completion`** — Shell completion generation (bash/zsh/fish/powershell) via Cobra's built-in `GenBashCompletion` etc.
 
 ## Deployment & Networking
 
 - [ ] **Testnet/Devnet deployment flow** — Currently `env up` is localnet-only. A `--network testnet` flag (or `efctl env deploy --network testnet`) would bridge the gap for staging.
-- [ ] **`efctl sui faucet [address]`** — Request test tokens from the localnet or testnet faucet without leaving the CLI.
+- [x] **`efctl sui faucet [address]`** — Request test tokens from the localnet or testnet faucet without leaving the CLI. (Implemented as `efctl env faucet`).
 
 ## Operational
 

cmd/extension_build.go

@@ -0,0 +1,54 @@
+package cmd
+
+import (
+	"os"
+
+	"efctl/pkg/builder"
+	"efctl/pkg/container"
+	"efctl/pkg/ui"
+	"efctl/pkg/validate"
+	"github.com/spf13/cobra"
+)
+
+var extensionBuildCmd = &cobra.Command{
+	Use:   "build [extension-path]",
+	Short: "Compile a Move contract without publishing",
+	Long:  `Compiles the specified extension contract (path relative to /workspace) inside the container, catching errors earlier.`,
+	Args:  cobra.ExactArgs(1),
+	Run: func(cmd *cobra.Command, args []string) {
+		extensionPath := args[0]
+		if err := validate.Network(envNetwork); err != nil {
+			ui.Error.Println(err.Error())
+			os.Exit(1)
+		}
+
+		c, err := container.NewClient()
+		if err != nil {
+			ui.Error.Println("Failed to create container client: " + err.Error())
+			os.Exit(1)
+		}
+
+		candidate, err := builder.GetCandidate(workspacePath, extensionPath)
+		if err != nil {
+			ui.Error.Printf("Error: extension %q not found.\n\n", extensionPath)
+			closest := builder.FindClosestMatch(workspacePath, extensionPath)
+			if len(closest) > 0 {
+				ui.Info.Println("Did you mean:")
+				for _, match := range closest {
+					ui.Info.Printf("  - %s\n", match)
+				}
+			}
+			os.Exit(1)
+		}
+
+		if err := builder.BuildExtension(c, workspacePath, envNetwork, candidate); err != nil {
+			ui.Error.Println("Build failed: " + err.Error())
+			os.Exit(1)
+		}
+	},
+}
+
+func init() {
+	extensionBuildCmd.Flags().StringVarP(&envNetwork, "network", "n", "localnet", "The network to build for (localnet or testnet)")
+	extensionCmd.AddCommand(extensionBuildCmd)
+}

cmd/extension_test_cli.go

@@ -0,0 +1,54 @@
+package cmd
+
+import (
+	"os"
+
+	"efctl/pkg/builder"
+	"efctl/pkg/container"
+	"efctl/pkg/ui"
+	"efctl/pkg/validate"
+	"github.com/spf13/cobra"
+)
+
+var extensionTestCmd = &cobra.Command{
+	Use:   "test [extension-path]",
+	Short: "Run sui move test for a Move contract",
+	Long:  `Runs 'sui move test' for the specified extension contract (path relative to /workspace) inside the container.`,
+	Args:  cobra.ExactArgs(1),
+	Run: func(cmd *cobra.Command, args []string) {
+		extensionPath := args[0]
+		if err := validate.Network(envNetwork); err != nil {
+			ui.Error.Println(err.Error())
+			os.Exit(1)
+		}
+
+		c, err := container.NewClient()
+		if err != nil {
+			ui.Error.Println("Failed to create container client: " + err.Error())
+			os.Exit(1)
+		}
+
+		candidate, err := builder.GetCandidate(workspacePath, extensionPath)
+		if err != nil {
+			ui.Error.Printf("Error: extension %q not found.\n\n", extensionPath)
+			closest := builder.FindClosestMatch(workspacePath, extensionPath)
+			if len(closest) > 0 {
+				ui.Info.Println("Did you mean:")
+				for _, match := range closest {
+					ui.Info.Printf("  - %s\n", match)
+				}
+			}
+			os.Exit(1)
+		}
+
+		if err := builder.TestExtension(c, workspacePath, envNetwork, candidate); err != nil {
+			ui.Error.Println("Tests failed: " + err.Error())
+			os.Exit(1)
+		}
+	},
+}
+
+func init() {
+	extensionTestCmd.Flags().StringVarP(&envNetwork, "network", "n", "localnet", "The network to test for (localnet or testnet)")
+	extensionCmd.AddCommand(extensionTestCmd)
+}

docs/efctl_env_extension.md

@@ -24,6 +24,8 @@ The extension command groups operations defined in the EVE Frontier Builder Flow
 ### SEE ALSO
 
 * [efctl env](efctl_env.md)	 - Manage the local Sui development environment
+* [efctl env extension build](efctl_env_extension_build.md)	 - Compile a Move contract without publishing
 * [efctl env extension list](efctl_env_extension_list.md)	 - List all available extensions in the workspace
 * [efctl env extension publish](efctl_env_extension_publish.md)	 - Publish a custom extension contract
+* [efctl env extension test](efctl_env_extension_test.md)	 - Run sui move test for a Move contract
 

docs/efctl_env_extension_build.md

@@ -0,0 +1,32 @@
+## efctl env extension build
+
+Compile a Move contract without publishing
+
+### Synopsis
+
+Compiles the specified extension contract (path relative to /workspace) inside the container, catching errors earlier.
+
+```
+efctl env extension build [extension-path] [flags]
+```
+
+### Options
+
+```
+  -h, --help             help for build
+  -n, --network string   The network to build for (localnet or testnet) (default "localnet")
+```
+
+### Options inherited from parent commands
+
+```
+      --config-file string   Path to the efctl.yaml or efctl.yml configuration file (default "efctl.yaml")
+      --debug                Enable verbose debug logging
+      --no-progress          Disable the progress spinner for cleaner CI output
+  -w, --workspace string     Path to the workspace directory (default ".")
+```
+
+### SEE ALSO
+
+* [efctl env extension](efctl_env_extension.md)	 - Manage the builder-scaffold extension flow
+

docs/efctl_env_extension_test.md

@@ -0,0 +1,32 @@
+## efctl env extension test
+
+Run sui move test for a Move contract
+
+### Synopsis
+
+Runs 'sui move test' for the specified extension contract (path relative to /workspace) inside the container.
+
+```
+efctl env extension test [extension-path] [flags]
+```
+
+### Options
+
+```
+  -h, --help             help for test
+  -n, --network string   The network to test for (localnet or testnet) (default "localnet")
+```
+
+### Options inherited from parent commands
+
+```
+      --config-file string   Path to the efctl.yaml or efctl.yml configuration file (default "efctl.yaml")
+      --debug                Enable verbose debug logging
+      --no-progress          Disable the progress spinner for cleaner CI output
+  -w, --workspace string     Path to the workspace directory (default ".")
+```
+
+### SEE ALSO
+
+* [efctl env extension](efctl_env_extension.md)	 - Manage the builder-scaffold extension flow
+

pkg/builder/build.go

@@ -0,0 +1,38 @@
+package builder
+
+import (
+	"context"
+	"fmt"
+	"strings"
+
+	"efctl/pkg/container"
+	"efctl/pkg/ui"
+)
+
+// BuildExtension compiles the Move contract inside the container without publishing.
+func BuildExtension(c container.ContainerClient, workspace string, network string, candidate PublishCandidate) error {
+	if err := PrepareExtensionEnv(c, workspace, network); err != nil {
+		return err
+	}
+
+	ui.Info.Printf("Building extension contract from %s...\n", candidate.HostPath)
+	ui.Info.Printf("Executing build inside container at %s...\n", candidate.ContainerPath)
+
+	buildCmd := fmt.Sprintf("cd %s && sui move build --build-env testnet", candidate.ContainerPath)
+
+	ui.Warn.Println("Build logging will be piped below:")
+
+	output, err := c.ExecCapture(context.Background(), container.ContainerSuiPlayground, []string{"/bin/bash", "-c", buildCmd})
+	if output != "" {
+		fmt.Print(output)
+	}
+	if err != nil {
+		if strings.Contains(output, "Build error") || strings.Contains(output, "Compilation error") {
+			return fmt.Errorf("build failed due to compilation errors")
+		}
+		return fmt.Errorf("build command failed: %w", err)
+	}
+
+	ui.Success.Println("Extension contract built successfully.")
+	return nil
+}

pkg/builder/publish.go

@@ -44,9 +44,8 @@ type PublishCandidate struct {
 
 const worldDependencyMarker = "world = {"
 
-// PublishExtension publishes the custom extension to the smart assembly testnet
-// and updates the builder-scaffold/.env with the extracted package IDs.
-func PublishExtension(c container.ContainerClient, workspace string, network string, candidate PublishCandidate) error {
+// PrepareExtensionEnv initializes the environment, repairs it if mismatched, and cleans stale files.
+func PrepareExtensionEnv(c container.ContainerClient, workspace string, network string) error {
 	// Automatically initialize/sync the builder-scaffold environment with world artifacts
 	if err := InitExtensionEnv(workspace, network); err != nil {
 		return fmt.Errorf("failed to initialize extension environment: %w", err)
@@ -56,6 +55,22 @@ func PublishExtension(c container.ContainerClient, workspace string, network str
 		return err
 	}
 
+	// Clean stale Move.lock files before building/testing/publishing to avoid framework drift issues
+	setup.CleanStaleMoveLocks(workspace)
+	if err := setup.PatchBuilderExampleMoveTomls(workspace); err != nil {
+		return err
+	}
+
+	return nil
+}
+
+// PublishExtension publishes the custom extension to the smart assembly testnet
+// and updates the builder-scaffold/.env with the extracted package IDs.
+func PublishExtension(c container.ContainerClient, workspace string, network string, candidate PublishCandidate) error {
+	if err := PrepareExtensionEnv(c, workspace, network); err != nil {
+		return err
+	}
+
 	ui.Info.Printf("Publishing extension contract from %s...\n", candidate.HostPath)
 
 	ui.Info.Printf("Executing publish inside container at %s...\n", candidate.ContainerPath)
@@ -65,12 +80,6 @@ func PublishExtension(c container.ContainerClient, workspace string, network str
 		return err
 	}
 
-	// Clean stale Move.lock files before publishing to avoid framework drift issues
-	setup.CleanStaleMoveLocks(workspace)
-	if err := setup.PatchBuilderExampleMoveTomls(workspace); err != nil {
-		return err
-	}
-
 	ui.Warn.Println("Publish logging will be piped below:")
 
 	output, err := c.ExecCapture(context.Background(), container.ContainerSuiPlayground, []string{"/bin/bash", "-c", publishCmd})

pkg/builder/test.go

@@ -0,0 +1,38 @@
+package builder
+
+import (
+	"context"
+	"fmt"
+	"strings"
+
+	"efctl/pkg/container"
+	"efctl/pkg/ui"
+)
+
+// TestExtension runs sui move test for the Move contract inside the container.
+func TestExtension(c container.ContainerClient, workspace string, network string, candidate PublishCandidate) error {
+	if err := PrepareExtensionEnv(c, workspace, network); err != nil {
+		return err
+	}
+
+	ui.Info.Printf("Testing extension contract from %s...\n", candidate.HostPath)
+	ui.Info.Printf("Executing test inside container at %s...\n", candidate.ContainerPath)
+
+	testCmd := fmt.Sprintf("cd %s && sui move test --build-env testnet", candidate.ContainerPath)
+
+	ui.Warn.Println("Test logging will be piped below:")
+
+	output, err := c.ExecCapture(context.Background(), container.ContainerSuiPlayground, []string{"/bin/bash", "-c", testCmd})
+	if output != "" {
+		fmt.Print(output)
+	}
+	if err != nil {
+		if strings.Contains(output, "Test failures") {
+			return fmt.Errorf("tests failed")
+		}
+		return fmt.Errorf("test command failed: %w", err)
+	}
+
+	ui.Success.Println("Extension contract tests passed.")
+	return nil
+}