whoops

johnstcn · johnstcn · commit 3360b430a566 · 2025-02-11T15:00:30.000Z
diff --git a/agent/agentcontainers/containers.go b/agent/agentcontainers/containers.go
@@ -144,6 +144,6 @@ type Lister interface {
 // NoopLister is a Lister interface that never returns any containers.
 type NoopLister struct{}
 
-func (*NoopLister) List(_ context.Context) (codersdk.WorkspaceAgentListContainersResponse, error) {
+func (NoopLister) List(_ context.Context) (codersdk.WorkspaceAgentListContainersResponse, error) {
 	return codersdk.WorkspaceAgentListContainersResponse{}, nil
 }
diff --git a/cli/agent.go b/cli/agent.go
@@ -318,9 +318,10 @@ func (r *RootCmd) workspaceAgent() *serpent.Command {
 
 			var containerLister agentcontainers.Lister
 			if !devcontainersEnabled {
+				logger.Info(ctx, "agent devcontainer detection not enabled")
 				containerLister = &agentcontainers.NoopLister{}
 			} else {
-				logger.Info(ctx, "agent devcontainer detection not enabled")
+				logger.Info(ctx, "agent devcontainer detection enabled")
 				containerLister = agentcontainers.NewDocker(execer)
 			}
 

Original file line number	Diff line number	Diff line change
`@@ -144,6 +144,6 @@ type Lister interface {`
`144`	`144`	`// NoopLister is a Lister interface that never returns any containers.`
`145`	`145`	`type NoopLister struct{}`
`146`	`146`
`147`		`-func (*NoopLister) List(_ context.Context) (codersdk.WorkspaceAgentListContainersResponse, error) {`
	`147`	`+func (NoopLister) List(_ context.Context) (codersdk.WorkspaceAgentListContainersResponse, error) {`
`148`	`148`	`return codersdk.WorkspaceAgentListContainersResponse{}, nil`
`149`	`149`	`}`