[DaveHanns]

Summary

apify push at the root of a repository with Actors in subfolders stopped with exit 4, so pushing several Actors meant one invocation per folder. Now, when the directory has no config file of its own but directories below it hold Actors, the command lists them and pushes the chosen ones, one ordinary push per Actor, in sequence. A single Actor is pushed with --dir, several with --all or the prompt. A directory with its own config is pushed as before.

Closes #1499. Part of apify/apify-core#28685. Stacked on #1502 (feat/push-monorepo-preflight).

Changes

• Selection mode starts when none of .actor/actor.json, actor.json, or apify.json is in the directory itself; a root .actorignore or a .actor/ folder with only shared files does not count.
• --all pushes every Actor found below the root, in path order. Without it, a multi-select prompt opens on a TTY outside CI; otherwise the listing goes to stderr with a hint and the command exits 4 before the login check. The hint points to --all or apify push --dir <path> for each Actor, and the listed paths are relative to where the command ran, so each one works as the --dir value.
• No separate flag picks Actors, so --dir is the one way to push a single Actor.
• --all in a directory that is an Actor itself exits 5.
• Discovery skips the names getActorLocalFilePaths always ignores at every depth, does not follow symlinks or descend below a found Actor, and skips directories it cannot read.
• An Actor in a gitignored directory is never pushed implicitly. --all skips it, names it on stderr with the apify push --dir <path> command that pushes it, and exits 4 when every Actor found is ignored. The prompt and the exit-4 listing still show it, labeled [ignored by .gitignore], and a picked one is pushed. Discovery checks each Actor directory with getGitignoreMatcher, the same check the upload uses, run once from the root. Skipped Actors are not entries in the --json results.
• A failure does not stop the pushes after it; the exit code is that of the first failed Actor. A signal during a build wait aborts that build as before and then stops the loop, naming the Actors not pushed. A signal anywhere else ends the process as it does today, after naming the Actors not pushed; Ctrl+C at the Git-source question stops the loop.
• A declined Git-source switch becomes {dir, ok: false, skipped: true} with no exit code and does not fail the run.
• --json prints one document {ok, operation: "push", results: [{dir, ...single push result}], exitCode?} once a selection was made, also an empty one; ok is true when every entry is ok or skipped. On the exit-4 and exit-5 paths before a selection nothing is printed on stdout. The positional actorId applies to a single selected Actor only (exit 5 otherwise).
• Telemetry records pushSelection: {selected, failed}.
• The multi-select prompt is @inquirer/checkbox. Install size: the package adds 36 KB on disk (about 22 KB unpacked); its dependencies (@inquirer/core, ansi, figures, and type) are already installed for the select prompt.

Tests

• Local: the gate, --all in an Actor directory, path order, failures mid-loop, the skipped entry, a signal during and outside a build wait, Ctrl+C at a prompt, the --json contract, the listed paths with --dir, discovery with an unreadable subdirectory and a missing root.
• E2E: the exit-4 listing, from the root and with --dir actors.
• [api]: two single-folder Actors with --all --json yield two SUCCEEDED entries (runs in CI).

Notes for reviewers

• The prompt is the upstream @inquirer/checkbox; the push tests mock it, and a pseudo-terminal run confirmed the listing, the toggle, and an empty selection exiting 0.
• Outside a build wait the loop's listener writes the not-pushed list (and, with --json, {ok: false, operation: "push", results} with the pushes so far) synchronously, removes itself and raises the signal again, so Node's default action ends the process with the same status as today.

🤖 Generated with Claude Code

[apify-factory]

Looks good, nothing blocking.

🏭 Factory review — ℹ️ not auto-approved

This change was reviewed but cannot be auto-approved: a gate failed, so the review cannot end in an approval.

check	result
Static gates	❌ 13/16 — base-branch, max-files, max-lines
Review findings	✅ none
Reviewer escalation	❌ Adds new public CLI flags and a new --json output shape for apify push, changes the shared flag parser, and re-raises interrupt signals to end the process, so a person should confirm the contract and process-lifecycle changes.
Juror 1 — claude-sonnet-5-5, adversarial	⏭️ skipped — a gate failed

Why it was not auto-approved

Failed gates:

• base-branch: base is feat/push-monorepo-preflight, required master
• max-files: 16 changed files, limit 10
• max-lines: 1508 changed lines (+1467/-41), limit 200

The comments above are the review itself; this section only explains the approval decision. Push a change and request @apify-factory again to re-run it, or ask a human to review.

_{Reviewed 824615776 · workflow run}

[apify-factory]

Looks good, nothing blocking.

[l2ysho]

pre-approved 🚀

[l2ysho]

Note: (nonTTY) As user/agent can use both --dir and also --actor-dir it may be confusing. Example from my testing

#	Command	Result
1	push --dir actors/alpha	pushed alpha
2	push --actor-dir actors/alpha	same target, Pushing the Actor in actors/alpha
3	push --dir actors --actor-dir alpha	same target again, printed as alpha
4	push --dir actors/alpha --actor-dir actors/alpha	error, exit 5

So there are at least 3 ways how to do the same push. Maybe guide user/agent to use more specific --dir instead of introducing new --actor-dir is better idea?

[DaveHanns]

Agreed. --actor-dir is gone in 8dff92a. A single Actor is now pushed with --dir, several with --all or the interactive picker, and the non-interactive message points to --dir with paths that work from where the command ran.

[l2ysho]

Suggestion: this is corner case but findActorDirs should do similar gitignore check as getActorLocalFilePaths as now actors in gitignore are pushed

mkdir -p /tmp/f1min/ignored/.actor
cd /tmp/f1min
git init -q .
echo 'ignored/' > .gitignore
echo '{"actorSpecification":1,"name":"ghost","version":"0.0"}' > ignored/.actor/actor.json
git add -A
git commit -qm init
apify push --all < /dev/null