Skip to content

Sync vendored skills with skills main (f8959e14 / #63) - #23

Merged
davidmytton merged 2 commits into
mainfrom
david/cursor/sync-skills-f8959e14-6a86
Sep 16, 2026
Merged

davidmytton merged 2 commits into
mainfrom
david/cursor/sync-skills-f8959e14-6a86

Conversation

@davidmytton

@davidmytton davidmytton commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Summary

Updates the vendored skill tree to match current arcjet/skills main at f8959e141446b8f021a6b34e00211df9598a8e59 (arcjet/skills#63, merged 2026-09-16).

That SHA is skills main after the published @arcjet/* 1.12.0 / Python arcjet 1.1.0 sync (docs through arcjet-docs#939).

This is a follow-up to #22, which synced d9aadab3. Did not wait for open arcjet/skills#59 (Python ADK + Cloudflare Think) — that PR is still open (mergeable_state dirty) and is not on main.

Files were fetched from GitHub (cloned arcjet/skills at that SHA) rather than rewritten by hand. After copy, this repo’s dprint formatter was applied so CI’s format check passes.

What changed

Canonical copy is plugins/arcjet/skills/ (skills/ is the inbound symlink). Only that tree was updated. No new skill dirs and no new guards_js_* files.

Version stamps and dropped git pins (skills#63)

  • @arcjet/* 1.12.0 / Python arcjet 1.1.0 / Go v1.0.0-rc.2
  • JS Google ADK, TanStack AI, and Claude Managed Agents now ship on npm 1.12.0 — git SHA pins removed
  • Python Claude Agent SDK, Claude Managed Agents, and Strands Agents extras now ship on PyPI 1.1.0 — git SHA pins removed (pip install "arcjet[<extra>]")
  • Adapter floor clarifications: LangChain / CrewAI / OpenAI Agents remain 1.0.0+; Claude Agent SDK / Claude Managed Agents / Strands need 1.1.0
  • Remote Guard policies are MCP-only (list-guard-policies / describe-guard-policy / validate-guard-policy / put-guard-policy); the CLI has no policy commands

Existing files updated in place:

  • arcjet/SKILL.md plus request/guard/MCP references
  • Existing guards_js_*.md adapter files (unchanged: guards_js_claude_agent_sdk.md, guards_js_langgraph.md)
  • Existing Python integrate-arcjet-guard-* skill dirs (langchain-py, crewai, openai-agents-py, claude-agent-sdk-py, claude-managed-agents-py, strands-agents-py)

Skills main still has no JS integrate-arcjet-guard-* dirs. JS adapters stay in arcjet/. HTTP frameworks stay in the request references.

No extra remote-policy teaching beyond what that SHA already has (actor / inputs / policyInput wording is from upstream).

Follow-up on this branch: the Claude Managed Agents worked-example path is the verified singular https://github.com/arcjet/examples/tree/main/examples/claude-managed-agent (plural and arcjet-js example links 404).

What was left alone

Deprecated alias skill directories are not generated from the canonical skill. They are standalone deprecation stubs (add-request-protection, add-guard-protection, protect-route, add-ai-protection) and were left unchanged.

No marketplace catalog or plugin-manifest work. Hosts discover skills by scanning plugins/arcjet/skills/ (./skills/ in the Codex manifest).

Formatter note

dprint table alignment treats unescaped || inside backticks as a column break (same issue as prior sync PRs). The Node version-range cells keep the escapes (\|\|) so the upstream ranges survive formatting:

  • Node >=22.21.0 <23 || >=24.5.0 (requests + guards JS tables)

Test plan

  • dprint check
  • bash scripts/validate.sh
  • Vendored skill dirs match skills main at f8959e14 (arcjet/ + the six integrate-arcjet-guard-* dirs; existing guards_js_*.md files; no JS integrate dirs on skills main)
  • Confirmed Node version-range \|\| escapes survived dprint
  • Confirmed no extra remote-policy teaching beyond that SHA
  • Confirmed alias skill directories unchanged
  • CI lint + validate jobs
Open in Web Open in Cursor 

Vendor-sync plugins/arcjet/skills/ from arcjet/skills main at
f8959e141446b8f021a6b34e00211df9598a8e59 (skills#63). Published
@arcjet/* 1.12.0 / Python arcjet 1.1.0: drop git pins, stamp Go
v1.0.0-rc.2, mark remote policies MCP-only, and clarify adapter
floors. Apply this repo's dprint formatter after the copy.

Co-authored-by: David Mytton <davidmytton@users.noreply.github.com>

@arcjet-review arcjet-review Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Arcjet Review — 🟢 Low Risk

Decision: Checked

Rationale: Documentation-only sync of the vendored plugins/arcjet/skills/ tree to arcjet/skills main at f8959e14. All changes are to markdown files (SKILL.md, references/*.md) plus a CHANGELOG entry. The edits update version stamps (@arcjet/* 1.11.0 → 1.12.0, Python arcjet 1.0.0 → 1.1.0, Go v1.0.0-rc.1v1.0.0-rc.2), drop obsolete git-SHA install pins now that adapters ship on npm/PyPI, and add MCP Guard policy tool descriptions. No code, no config, no secrets, no auth/infra/CI/dependency-manifest changes. Diff is internally consistent and description accurately catalogs the changes.

Summary of Changes

Syncs the vendored plugins/arcjet/skills/ tree with arcjet/skills main at f8959e14: bumps version stamps to @arcjet/* 1.12.0 / Python arcjet 1.1.0 / Go v1.0.0-rc.2, replaces pip install ... @ git+... and @arcjet/guard SHA pins with published-package installs for Google ADK, TanStack AI, Claude Managed Agents, Claude Agent SDK (py), and Strands Agents (py), clarifies that remote actor/inputs support depends on installed adapter types, and documents MCP-only Guard policy tools. Adds a matching CHANGELOG entry. No behavior/code changes.

Notes

The AI assessed this PR as approvable, but the trust level (1) does not allow auto-approval. A human reviewer must approve this PR.

Review: cc6f53fe | Model: anthropic/claude-opus-4-7 | Powered by Arcjet Review

Comment thread plugins/arcjet/skills/arcjet/references/guards_js_claude_managed_agents.md Outdated
The worked-example move to arcjet/examples was intentional, but the
directory on main is claude-managed-agent (singular). The plural path
and the leftover arcjet-js/examples link both 404.

Co-authored-by: David Mytton <davidmytton@users.noreply.github.com>
@davidmytton
davidmytton merged commit 183c079 into main Sep 16, 2026
4 checks passed
@arcjet-review arcjet-review Bot removed the needs review Awaiting human review label Sep 16, 2026
@davidmytton
davidmytton deleted the david/cursor/sync-skills-f8959e14-6a86 branch September 16, 2026 21:52
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