Skip to content

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

Merged
jennapederson merged 3 commits into
mainfrom
sync/skills-26185821386
May 26, 2026
Merged

sync: skills update from pinecone-io/skills#25
jennapederson merged 3 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: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

Skill contextualization summary

Adapted the two incoming skills (pinecone-help, pinecone-n8n) to the conventions in CLAUDE.md.

Changes

skills/help/SKILL.md (overwrote existing, since skills/help/ already exists as the plugin-adapted version of help):

  • Merged the new content from skills/pinecone-help/SKILL.md — adds pinecone:full-text-search and pinecone:n8n to the skills table and the "Which skill should I use?" section.
  • Adapted naming: name: pinecone-helppinecone:help; renamed every cross-reference (pinecone-quickstartpinecone:quickstart, etc.) including the previously-stale pinecone-full-text-search line.
  • Added allowed-tools: Skill, Bash, Read (matches existing convention for skills that route to others).
  • Condensed the API-key instructions to drop the .env fallback paragraph (Claude Code inherits the shell env), per CLAUDE.md.

skills/pinecone-help/ directory: deleted via git rm -r after merging its content into skills/help/.

skills/pinecone-n8n/skills/n8n/: renamed via git mv to strip the pinecone- prefix per CLAUDE.md.

skills/n8n/SKILL.md:

  • name: pinecone-n8npinecone:n8n.
  • allowed-tools: WriteWrite, Read (the skill generates JSON workflow files; Read is useful for reviewing existing workflows it discusses in "Step 1").

Intentionally left unchanged

  • All Python scripts under scripts/ — none in these two skills, but per task instructions Python files are off-limits regardless.
  • sourceTag values in skills/n8n/SKILL.md — these stay as n8n:n8n_nodes_pinecone_assistant:pinecone_n8n_skill. The CLAUDE.md claude_code_plugin: source-tag convention applies to the Pinecone Python SDK client (Pinecone() constructor), not to attribution strings embedded in generated n8n workflows that run outside this plugin. The existing tag is the correct n8n-side attribution.
  • npm package names like @pinecone-database/n8n-nodes-pinecone-assistant.* — these are real n8n node package identifiers, not skill cross-references, and must not be renamed.
  • Content/meaning of either skill — no factual changes; only frontmatter, naming, and the .env paragraph condensation.

Recommendations for the reviewer

  • Plugin description in .claude-plugin/plugin.json highlights /quickstart and /query but doesn't mention the newer full-text-search, assistant, n8n, or help skills. Consider refreshing the description and keywords (e.g. adding n8n, workflow automation) to reflect the expanded surface area. I did not touch .claude-plugin/ per task scope.
  • commands/join-discord.md is the only slash command in the plugin; nothing in commands/ overlaps with the new help or n8n skills, so no redundancy to remove there.
  • Existing skills/help/SKILL.md was previously missing pinecone:full-text-search from its tables — this sync corrects that drift. Worth a quick read to confirm the help skill is now an accurate index of everything else in skills/.

@jennapederson jennapederson added the bump:minor Label for release automation when minor changes are made label May 26, 2026
@jennapederson jennapederson merged commit 9af99dc into main May 26, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bump:minor Label for release automation when minor changes are made skill-sync

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants