Simplify scripts that are meant to be executed on multiple packages

[lukepighetti]

If I try to use select-package.dir-exists to scope a command to only packages that contain the ios directory, when I try to run the command it gives me a user prompt instead of running it on all packages like melos exec --dir-exists="ios"

  upgrade:pods:
    select-package:
      dir-exists: 'ios'
    run: |
      rm -rf Pods Podfile.lock
      pod repo update
      flutter precache --ios
      pod install

The user prompt it creates instead of running on all of them automatically

% melos upgrade:pods
Select a package to run the upgrade:pods script:

1) * [Default - Press Enter]
2) example
3) ml_pose_detection
4) timeline_player
5) tonal
6) tonal_ar


? Enter your choice 

[blaugold]

You can skip the prompt with the --no-select option, e.g. melos run upgrade:pods --no-select

[lukepighetti]

That appears to run the default (option 1), which only appears to be a single package. I would like to run it on all packages that meet the dir-exists (and any other filter I may wish to use). My expectation was that it would work like melos exec

[blaugold]

Ah, I see. You need to use melos exec in run:

  upgrade:pods:
    select-package:
      dir-exists: 'ios'
    run: |
      melos exec -- \
      rm -rf Pods Podfile.lock && \
      pod repo update && \
      flutter precache --ios && \
      pod install

It's not the nicest syntax because it needs to be a single command. There is some room for improvement here :)

[lukepighetti]

I would expect that to run melos exec in just the default package. While it does solve my problem, I find this highly confusing. It's not clear to me how the results of select-package are being passed to melos exec

In an ideal world we'd be doing something like this

  upgrade:pods:
    filter:
      dir-exists: 'ios'
    run: |
      rm -rf Pods Podfile.lock
      pod repo update
      flutter precache --ios
      pod install

$ melos upgrade:pods

And then it runs on

example
ml_pose_detection
timeline_player
tonal
tonal_ar