Skip to content

sync: skills update from pinecone-io/skills#11

Open
arjunpatel7 wants to merge 2 commits into
mainfrom
sync/skills-26185821386
Open

sync: skills update from pinecone-io/skills#11
arjunpatel7 wants to merge 2 commits into
mainfrom
sync/skills-26185821386

Conversation

@arjunpatel7
Copy link
Copy Markdown
Collaborator

Automated skill sync from pinecone-io/skills — run #18.

Changed files:

skills/pinecone-help/SKILL.md skills/pinecone-n8n/SKILL.md 

The contextualization workflow will run automatically on this branch to adapt these skills for this IDE. Review both the raw sync and Claude's adaptations before merging.

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions
Copy link
Copy Markdown

Contextualize summary

Automated pass to adapt synced skills for the pinecone-io/pinecone-cursor-plugin Cursor plugin conventions.

Files changed

skills/pinecone-help/SKILL.md

  • Restored Cursor-specific API key setup: .env at workspace root with MCP envFile, plus export for terminal scripts (replaces generic IDE wording from upstream sync).
  • Added how to invoke skills in Cursor Agent chat: /pinecone-<skill-name>.
  • Left upstream content intact: pinecone-n8n table entry, expanded pinecone-full-text-search description, and routing guidance for FTS and n8n.

skills/pinecone-n8n/SKILL.md

  • Removed allowed-tools: Write from frontmatter (Claude Code–specific; not used by other plugin skills).
  • Added /pinecone-n8n invocation note at the top.
  • Left all workflow JSON, n8n node configuration, debugging tables, and sourceTag values unchanged (n8n telemetry tags, not repo attribution).

Intentionally left unchanged

  • No directory renames — both changed skills already use the pinecone- prefix.
  • No .py files under skills/ (none in this PR diff).
  • No edits outside skills/ (per instructions: .cursor-plugin/, mcp.json, rules, commands untouched).
  • Full n8n skill body (~690 lines): workflow templates, node package names, and best-practice checklists kept as synced from upstream.
  • pinecone-query/SKILL.md still references /pinecone-query as a command name (pre-existing; not part of this PR diff).

Submission checklist

Check Status
.cursor-plugin/plugin.json valid JSON Pass
Plugin name lowercase kebab-case (pinecone) Pass
Plugin description present Pass
All 9 skills have name + description frontmatter Pass
Skill name matches folder for all skills Pass
Rules/commands frontmatter N/A (no rules/ or commands/ dirs)
Manifest paths relative Pass (logo: assets/logo.svg)
node scripts/validate-plugin.mjs Pass

No blocking checklist violations found.

Recommendations for manual review

  1. plugin.json description — Consider mentioning n8n workflow support and /pinecone-n8n so marketplace users discover the new skill (not changed here per scope).
  2. Overlap with existing skillspinecone-n8n Assistant-path guidance overlaps with pinecone-assistant (document Q&A, citations). That is intentional (n8n vs CLI/SDK); reviewers may want cross-links in pinecone-help only (already lists both).
  3. pinecone-query wording — Still says "/pinecone-query command"; in Cursor this is skill invocation — optional consistency pass in a follow-up.
  4. Upstream sync — This PR only adds/updates pinecone-help and pinecone-n8n; if the base skills repo gained FTS reference doc changes, a separate sync PR may be needed for pinecone-full-text-search/references/*.md.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant