feat: support looping over map variables

Author: pd93

docs/docs/experiments/any_variables.mdx

@@ -86,8 +86,8 @@ tasks:
         cmd: echo {{.ITEM}}
 ```
 
-Because this experiment adds support for array variables, the `for` keyword has
-been updated to support looping over arrays directly:
+Because this experiment adds support for "collection-type" variables, the `for`
+keyword has been updated to support looping over arrays directly:
 
 ```yaml
 version: 3
@@ -102,6 +102,28 @@ tasks:
         cmd: echo {{.ITEM}}
 ```
 
+This also works for maps. However, remember that maps are unordered, so the
+order in which the items are looped over is random:
+
+```yaml
+version: 3
+
+tasks:
+  foo:
+    vars:
+      MAP:
+        KEY_1:
+          SUBKEY: sub_value_1
+        KEY_2:
+          SUBKEY: sub_value_2
+        KEY_3:
+          SUBKEY: sub_value_3
+    cmds:
+      - for:
+          var: MAP
+        cmd: echo {{.ITEM.SUBKEY}}
+```
+
 String splitting is still supported and remember that for simple cases, you have
 always been able to loop over an array without using variables at all:
 

testdata/vars/any/Taskfile.yml

@@ -9,6 +9,8 @@ tasks:
     - task: string-array
     - task: for-string
     - task: for-int
+    - task: for-map
+    - task: for-multi-layer-map
 
   dynamic:
     vars:
@@ -78,3 +80,27 @@ tasks:
           var: LIST
         cmd: echo {{add .ITEM 100}}
 
+  for-map:
+    vars:
+      MAP:
+        KEY_1: value_1
+        KEY_2: value_2
+        KEY_3: value_3
+    cmds:
+      - for:
+          var: MAP
+        cmd: echo {{.ITEM}}
+
+  for-multi-layer-map:
+    vars:
+      MAP:
+        KEY_1:
+          SUBKEY: sub_value_1
+        KEY_2:
+          SUBKEY: sub_value_2
+        KEY_3:
+          SUBKEY: sub_value_3
+    cmds:
+      - for:
+          var: MAP
+        cmd: echo {{.ITEM.SUBKEY}}

variables.go

@@ -6,6 +6,7 @@ import (
 	"strings"
 
 	"github.com/joho/godotenv"
+	"golang.org/x/exp/maps"
 
 	"github.com/go-task/task/v3/errors"
 	"github.com/go-task/task/v3/internal/execext"
@@ -170,10 +171,7 @@ func (e *Executor) compiledTask(call taskfile.Call, evaluateShVars bool) (*taskf
 							case []any:
 								list = value
 							case map[string]any:
-								return &taskfile.Task{}, errors.TaskfileInvalidError{
-									URI: origTask.Location.Taskfile,
-									Err: errors.New("sh is not supported with the 'Any Variables' experiment enabled.\nSee https://taskfile.dev/experiments/any-variables for more information."),
-								}
+								list = maps.Values(value)
 							default:
 								return nil, errors.TaskfileInvalidError{
 									URI: origTask.Location.Taskfile,