refactor(eslint): ♻️ add eslint-plugin-n to eslint configuration (#528)

<!-- 👋 Hi, thanks for sending a PR to template-typescript-node-package!
💖.
Please fill out all fields below and make sure each item is true and [x]
checked.
Otherwise we may not be able to review your PR. -->

## PR Checklist

- [x] Addresses an existing open issue: fixes #470
- [x] That issue was marked as [`status: accepting
prs`](https://github.com/JoshuaKGoldberg/template-typescript-node-package/issues?q=is%3Aopen+is%3Aissue+label%3A%22status%3A+accepting+prs%22)
- [x] Steps in
[CONTRIBUTING.md](https://github.com/JoshuaKGoldberg/template-typescript-node-package/blob/main/.github/CONTRIBUTING.md)
were taken

## Overview

<!-- Description of what is changed and how the code change does that.
-->
Added eslint-plugin-n & checked for linting errors.

**Note:** Seems perfectionist eslint was not run on `.eslintrc.cjs`, so
I did that which changes the order of the objects.

---------

Co-authored-by: Josh Goldberg <[email protected]>

Author: navin-moorthy

.eslintrc.cjs

@@ -17,6 +17,7 @@ module.exports = {
 	extends: [
 		"eslint:recommended",
 		"plugin:eslint-comments/recommended",
+		"plugin:n/recommended",
 		"plugin:perfectionist/recommended-natural",
 		"plugin:regexp/recommended",
 		"prettier",
@@ -126,6 +127,12 @@ module.exports = {
 		// These off/less-strict-by-default rules work well for this repo and we like them on.
 		"@typescript-eslint/no-unused-vars": ["error", { caughtErrors: "all" }],
 		"import/extensions": ["error", "ignorePackages"],
+		"n/no-missing-import": [
+			"error",
+			{
+				allowModules: ["template-typescript-node-package"],
+			},
+		],
 		"no-only-tests/no-only-tests": "error",
 
 		// These on-by-default rules don't work well for this repo and we like them off.

package.json

@@ -67,6 +67,7 @@
 		"eslint-plugin-jsdoc": "^46.0.0",
 		"eslint-plugin-jsonc": "^2.8.0",
 		"eslint-plugin-markdown": "^3.0.0",
+		"eslint-plugin-n": "^16.0.0",
 		"eslint-plugin-no-only-tests": "^3.1.0",
 		"eslint-plugin-perfectionist": "^1.1.2",
 		"eslint-plugin-regexp": "^1.15.0",

pnpm-lock.yaml

@@ -46,6 +46,7 @@ module.exports = {
 	extends: [
 		"eslint:recommended",
 		"plugin:eslint-comments/recommended",
+		"plugin:n/recommended",
 		"plugin:perfectionist/recommended-natural",
 		"plugin:regexp/recommended",
 		"prettier",
@@ -148,9 +149,13 @@ module.exports = {
 	rules: {
 		// These off/less-strict-by-default rules work well for this repo and we like them on.
 		"@typescript-eslint/no-unused-vars": ["error", { caughtErrors: "all" }],
-		"import/extensions": ["error", "ignorePackages"],${
-			values.unitTests ? `\n"no-only-tests/no-only-tests": "error",` : ""
-		}
+		"import/extensions": ["error", "ignorePackages"],
+		"n/no-missing-import": [
+			"error",
+			{
+				allowModules: ["template-typescript-node-package"],
+			},
+		],${values.unitTests ? `\n"no-only-tests/no-only-tests": "error",` : ""}
 
 		// These on-by-default rules don't work well for this repo and we like them off.
 		"no-case-declarations": "off",

src/hydrate/steps/writing/creation/rootFiles.ts

@@ -25,7 +25,8 @@ export async function getOctokit(
 			console.error();
 			console.error(chalk.red((error as Error).message));
 			console.error();
-			process.exit(0);
+			// eslint-disable-next-line n/no-process-exit
+			process.exit(1);
 		}
 
 		const auth = (await $`gh auth token`).stdout.trim();

src/shared/getOctokit.ts

@@ -1,8 +1,9 @@
 import * as prompts from "@clack/prompts";
 
-export function handleCancel(): never {
+export function handleCancel() {
 	prompts.cancel("Operation cancelled. Exiting - maybe another time? 👋");
-	process.exit(0);
+	// eslint-disable-next-line n/no-process-exit
+	process.exit(1);
 }
 
 export function handlePromptCancel(