direct: Remove unnecessary error wrapping (#4157)

denik · web-flow · commit c6fe5e8b1acc · 2025-12-18T18:10:06.000+01:00
We already have "reading config" prefix at caller side.

Noticed this error message:
```
  Error: reading config: reading config: unsupported resource type: quality_monitors
```
diff --git a/bundle/direct/bundle_plan.go b/bundle/direct/bundle_plan.go
@@ -599,7 +599,7 @@ func (b *DeploymentBundle) makePlan(ctx context.Context, configRoot *config.Root
 				},
 			)
 			if err != nil {
-				return nil, fmt.Errorf("reading config: %w", err)
+				return nil, err
 			}
 		}
 	}
@@ -612,7 +612,7 @@ func (b *DeploymentBundle) makePlan(ctx context.Context, configRoot *config.Root
 		prefix := "cannot plan " + node
 		inputConfig, err := configRoot.GetResourceConfig(node)
 		if err != nil {
-			return nil, fmt.Errorf("%s: reading config: %s", prefix, err)
+			return nil, err
 		}
 
 		adapter, err := b.getAdapterForKey(node)

Original file line number	Diff line number	Diff line change
`@@ -599,7 +599,7 @@ func (b DeploymentBundle) makePlan(ctx context.Context, configRoot config.Root`
`599`	`599`	`},`
`600`	`600`	`)`
`601`	`601`	`if err != nil {`
`602`		`- return nil, fmt.Errorf("reading config: %w", err)`
	`602`	`+ return nil, err`
`603`	`603`	`}`
`604`	`604`	`}`
`605`	`605`	`}`
`@@ -612,7 +612,7 @@ func (b DeploymentBundle) makePlan(ctx context.Context, configRoot config.Root`
`612`	`612`	`prefix := "cannot plan " + node`
`613`	`613`	`inputConfig, err := configRoot.GetResourceConfig(node)`
`614`	`614`	`if err != nil {`
`615`		`- return nil, fmt.Errorf("%s: reading config: %s", prefix, err)`
	`615`	`+ return nil, err`
`616`	`616`	`}`
`617`	`617`
`618`	`618`	`adapter, err := b.getAdapterForKey(node)`