Skip to content

feat: add careers tool#67

Open
jhamon wants to merge 8 commits intomainfrom
feat/careers-tool
Open

feat: add careers tool#67
jhamon wants to merge 8 commits intomainfrom
feat/careers-tool

Conversation

@jhamon
Copy link
Copy Markdown
Contributor

@jhamon jhamon commented Mar 5, 2026

Summary

Adds a new careers MCP tool that helps users discover open roles at Pinecone. When invoked, it opens the Pinecone careers page in the user's default browser and returns the URL as a text response — a lightweight but delightful way to surface hiring opportunities directly from within an AI assistant.

Screenshot 2026-03-05 at 4 28 35 PM

Changes

  • src/tools/jobs/careers.ts — new tool implementation that opens https://www.pinecone.io/careers/#open-roles via the OS default browser
  • src/server.ts — imports and registers the new careers tool on startup

Test Plan

  • Tool appears in the MCP tool list after server starts
  • Invoking the tool opens the careers page in the default browser
  • Tool returns a well-formed text response with the careers URL
  • Works on macOS (open), Linux (xdg-open), and Windows (start)

Note

Medium Risk
Introduces a new tool that makes outbound network requests and invokes OS-specific browser launch commands (execFile), which could affect runtime environments (especially headless/CI) despite best-effort error handling.

Overview
Adds a new MCP tool careers and registers it during server startup (src/server.ts). The tool opens https://www.pinecone.io/careers/#open-roles in the default browser and returns a text response listing open roles pulled from Ashby, with optional team/keyword filtering and a 20-result cap.

Adds test coverage for role formatting, filtering/sorting, truncation messaging, and API-failure fallback (src/tools/jobs/careers.test.ts).

Updates package-lock.json with version bumps for several dependencies (notably hono, @hono/node-server, rollup, ajv, minimatch, and qs).

Written by Cursor Bugbot for commit 3590196. This will update automatically on new commits. Configure here.

Adds a new MCP tool that surfaces Pinecone's open roles page.
When invoked, it opens https://www.pinecone.io/careers/#open-roles
in the user's default browser and returns the URL as a text response.
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Autofix Details

Bugbot Autofix prepared a fix for the issue found in the latest run.

  • ✅ Fixed: Windows start command treats quoted URL as title
    • Added empty title parameter to Windows start command so URL is opened in browser instead of being treated as window title.

Create PR

Or push these changes by commenting:

@cursor push 5d8a6f47fa
Preview (5d8a6f47fa)
diff --git a/src/tools/jobs/careers.ts b/src/tools/jobs/careers.ts
--- a/src/tools/jobs/careers.ts
+++ b/src/tools/jobs/careers.ts
@@ -2,11 +2,10 @@
 import {platform} from 'os';
 import {McpServer} from '@modelcontextprotocol/sdk/server/mcp.js';
 
-const CAREERS_URL =
-  'https://www.pinecone.io/careers/#open-roles';
+const CAREERS_URL = 'https://www.pinecone.io/careers/#open-roles';
 
 function openBrowser(url: string) {
-  const cmd = platform() === 'win32' ? 'start' : platform() === 'darwin' ? 'open' : 'xdg-open';
+  const cmd = platform() === 'win32' ? 'start ""' : platform() === 'darwin' ? 'open' : 'xdg-open';
   exec(`${cmd} "${url}"`);
 }
This Bugbot Autofix run was free. To enable autofix for future PRs, go to the Cursor dashboard.

Comment thread src/tools/jobs/careers.ts Outdated
jhamon added 5 commits March 5, 2026 16:23
Instead of just returning a URL, the careers tool now fetches current
open roles from Pinecone's Ashby job board at call-time, groups them
by team, and returns formatted listings with direct links to each posting.
Falls back to the careers page URL if the API is unavailable.
Adds optional 'team' and 'keyword' input parameters so the AI can
narrow listings when there are many open roles. Both filters are
case-insensitive partial matches and can be combined. Results include
a note about which filters were applied, and a friendly no-results
message when nothing matches.
Comment thread src/tools/jobs/careers.ts Outdated
Comment thread src/tools/jobs/careers.ts
Comment thread src/tools/jobs/careers.ts Outdated
- Use execFile instead of exec in openBrowser to eliminate shell
  injection risk; on Windows, route through cmd.exe /c start
- Fix false truncation: check pre-slice count against MAX_LISTINGS
  so exactly-20-result sets are not wrongly marked as truncated
- Fix plural grammar: use "1 open role" vs "N open roles"
- Run npm audit fix to resolve pre-existing high/moderate severity
  dependency vulnerabilities (hono, ajv, minimatch)
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Comment thread src/tools/jobs/careers.ts
Attach a no-op callback to execFile calls so Node does not emit an
unhandled 'error' event when the browser launcher is unavailable
(e.g. headless Linux MCP servers). Also fix subject-verb agreement
when exactly one job matches ("Here is 1 open role").
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants