Alfy eats errors from Run Script action

[ErikMinekus]

Alfred's workflow debugger only shows output from stderr. These lines redirect all output from stderr to alfy.error(), which then uses console.log() to print it to stdout:

loudRejection(alfy.error);
process.on('uncaughtException', alfy.error);
hookStd.stderr(alfy.error);

This means that any errors that occur inside a Run Script action are never displayed in the workflow debugger.

---

IssueHunt Summary

Backers (Total: $20.00)

• issuehunt ($20.00)

Submitted pull Requests

• #112 Show stderr in debugger

---

Become a backer now!

Or submit a pull request to get the deposits!

Tips

• Checkout the Issuehunt explorer to discover more funded issues.
• Need some help from other developers? Add your repositories on IssueHunt to raise funds.

---

IssueHunt has been backed by the following sponsors. Become a sponsor

[sindresorhus]

Yes, we show the error in the Alfy itself instead:

alfy/index.js

Lines 70 to 96 in 0dafe8f

	alfy.error = error => {
	const stack = cleanStack(error.stack || error);
	const copy = `
	\`\`\`
	${stack}
	\`\`\`
	-
	${alfy.meta.name} ${alfy.meta.version}
	Alfred ${alfy.alfred.version}
	${process.platform} ${os.release()}
	`.trim();
	alfy.output([{
	title: error.stack ? `${error.name}: ${error.message}` : error,
	subtitle: 'Press ⌘L to see the full error and ⌘C to copy it.',
	valid: false,
	text: {
	copy,
	largetype: stack
	},
	icon: {
	path: exports.icon.error
	}
	}]);
	};

But you're right, we should also enable stderr to go through so it's also shown in the debugger.

[IssueHuntBot]

@IssueHunt has funded $20.00 to this issue.

---

• Submit pull request via IssueHunt to receive this reward.
• Want to contribute? Chip in to this issue via IssueHunt.
• Checkout the IssueHunt Issue Explorer to see more funded issues.
• Need help from developers? Add your repository on IssueHunt to raise funds.

[LitoMore]

I think we don't need hook-std here. Remove the hook-std and add console.error to alfy.error will fix this issue.

[sindresorhus]

If anyone wants to work on this, see the initial attempt and feedback in #112.