Update evernote extension

[magnusmoaner]

Description

Screencast

Checklist

• I read the extension guidelines
• I read the documentation about publishing
• I ran npm run build and tested this distribution build in Raycast
• I checked that files in the assets folder are used by the extension itself
• I checked that assets used by the README are placed outside of the metadata folder

[raycastbot]

Thank you for your first contribution! 🎉

🔔 @artpi you might want to have a look.

You can use this guide to learn how to check out the Pull Request locally in order to test it.

Due to our current reduced availability, the initial review may take up to 10-15 business days

[greptile-apps]

PR Summary

This PR adds a new "Search Tasks" command to the Evernote extension, allowing users to search and view open tasks within their Evernote notes using the local database.

• SQL query in /extensions/evernote/src/components/TasksList.tsx is vulnerable to injection attacks - search text should be parameterized
• Debug console.log(data) statement in TasksList.tsx should be removed before production
• Consider using showFailureToast from @raycast/utils in /extensions/evernote/src/search-tasks.tsx error handling
• url and snippet fields are defined in TaskItem interface but not used in the SQL query
• Missing metadata folder with screenshots for the new view command

_{💡 (1/5) You can manually trigger the bot by mentioning @greptileai in a comment!}

_{3 file(s) reviewed, 6 comment(s)
Edit PR Review Bot Settings | Greptile}

[greptile-apps]

style: Remove extra blank line between dependencies

[greptile-apps]

style: Remove debug console.log before production

Suggested change

console.log(data);

[greptile-apps]

logic: useEffect callback should be wrapped in try-catch to handle potential errors from getApplications() and fs operations

Suggested change

	useEffect(() => {
	getApplications().then(async (applications) => {
	const isEvernoteInstalled = applications.find(({ bundleId }) => bundleId === "com.evernote.Evernote");
	if (!isEvernoteInstalled) {
	await popToRoot();
	await showToast({
	style: Toast.Style.Failure,
	title: "Evernote client is not installed.",
	message: "Download",
	primaryAction: {
	title: "Go to https://evernote.com/download",
	onAction: (toast) => {
	open("https://evernote.com/download");
	toast.hide();
	},
	},
	});
	return;
	}
	let baseDir: string | null = null;
	if (evernoteDB) {
	return;
	}
	for (const directory of knownEvernoteDirLocations) {
	if (fs.existsSync(directory)) {
	baseDir = directory;
	break;
	}
	}
	if (!baseDir || !fs.existsSync(baseDir)) {
	await popToRoot();
	await showToast({
	style: Toast.Style.Failure,
	title: "Cannot find Evernote database.",
	message:
	"The database should be in ~/Library/Application Support/Evernote/conduit-storage/https%3A%2F%2Fwww.evernote.com, but can be somewhere else.",
	});
	return;
	}
	const files = fs.readdirSync(baseDir);
	console.log(files);
	const dbFile = files.find((file) => file.endsWith("+RemoteGraph.sql"));
	if (dbFile) {
	setEvernoteDB(path.join(baseDir, dbFile));
	}
	});
	}, [evernoteDB]);
	useEffect(() => {
	getApplications().then(async (applications) => {
	try {
	const isEvernoteInstalled = applications.find(({ bundleId }) => bundleId === "com.evernote.Evernote");
	if (!isEvernoteInstalled) {
	await popToRoot();
	await showToast({
	style: Toast.Style.Failure,
	title: "Evernote client is not installed.",
	message: "Download",
	primaryAction: {
	title: "Go to https://evernote.com/download",
	onAction: (toast) => {
	open("https://evernote.com/download");
	toast.hide();
	},
	},
	});
	return;
	}
	let baseDir: string | null = null;
	if (evernoteDB) {
	return;
	}
	for (const directory of knownEvernoteDirLocations) {
	if (fs.existsSync(directory)) {
	baseDir = directory;
	break;
	}
	}
	if (!baseDir || !fs.existsSync(baseDir)) {
	await popToRoot();
	await showToast({
	style: Toast.Style.Failure,
	title: "Cannot find Evernote database.",
	message:
	"The database should be in ~/Library/Application Support/Evernote/conduit-storage/https%3A%2F%2Fwww.evernote.com, but can be somewhere else.",
	});
	return;
	}
	const files = fs.readdirSync(baseDir);
	console.log(files);
	const dbFile = files.find((file) => file.endsWith("+RemoteGraph.sql"));
	if (dbFile) {
	setEvernoteDB(path.join(baseDir, dbFile));
	}
	} catch (error) {
	await showToast({
	style: Toast.Style.Failure,
	title: "Error accessing Evernote database",
	message: String(error)
	});
	}
	});
	}, [evernoteDB]);

[greptile-apps]

style: console.log statement should be removed before production

Suggested change

	const files = fs.readdirSync(baseDir);
	console.log(files);
	const files = fs.readdirSync(baseDir);

[greptile-apps]

style: should show loading state instead of empty return when evernoteDB is null

[greptile-apps]

logic: missing error handling if no database file is found - should show toast error

[pernielsentikaer]

Hi @magnusmoaner👋

Thanks for your contribution 🔥

Could you look into suggestions and add a CHANGELOG entry?

@artpi do you want to check this?