fix: include postinstall script in package and add restart hint (#16)

The postinstall script was missing from the `files` array so npm never
included it in the published tarball -- the `|| true` silenced the
error. Add `scripts/postinstall.js` to `files`.

Also tell users to restart their terminal after installing Bun, since
the installer only updates shell config files and the current session
won't pick up the new PATH entry.

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: simaonogueira101

bin/cli.mjs

@@ -24,7 +24,7 @@ try {
   console.error(
     "talent-agent requires the Bun runtime.\n\n" +
       "  Install it:  curl -fsSL https://bun.sh/install | bash\n" +
-      "  Learn more:  https://bun.sh\n",
+      "  Then restart your terminal (or run: source ~/.bashrc)\n",
   );
   process.exit(1);
 }

package.json

@@ -9,7 +9,8 @@
   "files": [
     "src",
     "skills",
-    "bin"
+    "bin",
+    "scripts/postinstall.js"
   ],
   "exports": {
     ".": "./src/lib.ts",

scripts/postinstall.js

@@ -26,7 +26,7 @@ console.log("");
 if (!hasBun) {
   console.log("  Bun runtime is required but was not found.");
   console.log("  Install it:  curl -fsSL https://bun.sh/install | bash");
-  console.log("  Learn more:  https://bun.sh");
+  console.log("  Then restart your terminal (or run: source ~/.bashrc)");
   console.log("");
 }
 

test/postinstall.test.ts

@@ -83,5 +83,6 @@ describe("scripts/postinstall.js", () => {
 
     expect(stdout).toContain("Bun runtime is required but was not found");
     expect(stdout).toContain("curl -fsSL https://bun.sh/install | bash");
+    expect(stdout).toContain("restart your terminal");
   });
 });