fix(pull): handle private actors correctly (#865)

Author: vladfrangu

src/commands/actors/pull.ts

@@ -85,10 +85,17 @@ export class ActorsPullCommand extends ApifyCommand<typeof ActorsPullCommand> {
 			throw new Error(`Cannot find Actor with ID/name '${actorId}' in your account.`);
 		}
 
-		if (!actor) throw new Error(`Cannot find Actor with ID/name '${actorId}' in your account.`);
+		if (!actor) {
+			throw new Error(`Cannot find Actor with ID/name '${actorId}' in your account.`);
+		}
 
 		const { name, versions } = actor;
 
+		// @ts-expect-error TODO: fix apify-client types
+		if (actor.isSourceCodeHidden && !actor.versions.length) {
+			throw new Error(`You cannot pull source code of this Actor because you do not have permission to do so.`);
+		}
+
 		let correctVersion = null;
 		if (this.flags.version) {
 			correctVersion = versions.find((version) => version.versionNumber === this.flags.version);

test/api/commands/pull.test.ts

@@ -217,4 +217,12 @@ describe('[api] apify pull', () => {
 
 		expect(exists).to.be.eql(true);
 	});
+
+	it('should fail if actor is private', async () => {
+		await testRunCommand(ActorsPullCommand, { args_actorId: 'apify/website-content-crawler' });
+
+		expect(lastErrorMessage()).toMatch(
+			/You cannot pull source code of this Actor because you do not have permission to do so./i,
+		);
+	});
 });