feat(monorepo): prefix output with package name

[wemeetagain]

In the monorepo commands (exec, releaseRc, and run), prefix each line of output from each subpackage with the name of the package where the command is running.
This can be disabled by providing the --noPrefix cli arg.

• prerequisite for Enable parallel script runs #1375
  • running scripts in parallel results in interleaved lines of output. So each line of output needs to be associated in some way with the package where it's running.

[wemeetagain]

https://github.com/ipfs/aegir/actions/runs/6425141074/job/17447167829#step:5:19
Seems unrelated to this PR??

[achingbrain]

I've opened bbc/typescript-docs-verifier#30

Quick fix: #1380

[achingbrain]

This looks interesting, it should probably default to prefixing scripts run in monorepo projects and not prefixing tasks run in regular projects?

It could use some tests to verify this too.

[wemeetagain]

Ah, so run, exec are monorepo-only, but releaseRc can be used in monorepo or non-monorepo.

[wemeetagain]

It could use some tests to verify [that the prefix isn't added in non-monorepo contexts] too.

run and exec are monorepo-only.
releaseRc is implemented in monorepo and non-monorepo functions separately: releaseMonorepoRcs and releaseRc.

Not sure that a test is needed for this.

[wemeetagain]

@achingbrain ready for re-review

[achingbrain]

Ok, I added some tests because I got bored of testing things manually like an animal - it looks like the output for aegir run is getting double-prefixed. It may be something to do with the child process inheriting stdin/out, needs a bit of investigation.

[achingbrain]

Relevant: sindresorhus/execa#121

[achingbrain]

I stole the way lerna does it, using strong-long-transformer.

[wemeetagain]

👍

[wemeetagain]

This fails on my local machine

exec
    1) should prefix monorepo output with the project name


  50 passing (1m)
  3 pending
  1 failing

  1) exec
       should prefix monorepo output with the project name:

      AssertionError: expected '\na-workspace-project: > ls\na-worksp…' to equal '\nanother-workspace-project: > ls\nan…'
      + expected - actual

       
      +another-workspace-project: > ls
      +another-workspace-project: package.json
      +another-workspace-project: src
      +another-workspace-project: tsconfig.json
      +another-workspace-project: typedoc.json
      +
       a-workspace-project: > ls
       a-workspace-project: package.json
       a-workspace-project: src
       a-workspace-project: tsconfig.json
      -a-workspace-project: typedoc.json
      -
      -another-workspace-project: > ls
      -another-workspace-project: package.json
      -another-workspace-project: src
      -another-workspace-project: tsconfig.json
      -another-workspace-project: typedoc.json
      +a-workspace-project: typedoc.json
      
      at Context.<anonymous> (file:///home/cayman/Code/aegir/test/exec.js:32:30)
      at processTicksAndRejections (node:internal/process/task_queues:95:5)

[achingbrain]

Hmm, weird that it works in CI?

[wemeetagain]

It seems that the glob code emits directories in a different order depending on... ?.

Maybe we can add a package dependency in the monorepo test fixture to ensure the ordering.

[wemeetagain]

ready for re-review

stale

[wemeetagain]

👍