feat(catalog): add Nexus skill bundle v1.0 (DRAFT — operator review)#10
feat(catalog): add Nexus skill bundle v1.0 (DRAFT — operator review)#10kelliott-cloud wants to merge 1 commit into
Conversation
Adds 230 nexus-* skills (catalog name nexus-skills@1.0.0) — one per Nexus backend route surface — generated from kelliott-cloud/Nexus-10.0-A. Each SKILL.md is agentskills.io-conformant: YAML frontmatter (name, description, license, metadata, compatibility) + Markdown body with endpoint table. Names follow the spec (a-z0-9-, ≤64 chars, parent dir matches name). Bundle facts: - skills added: 230 - endpoints covered: 2120 - files skipped at source: 22 (admin / health / internal / webhooks / websocket / widget loader / dispatch push) - auth contract per skill: bearer_token, env NEXUS_API_TOKEN, scope nexus.api, base URL via NEXUS_BASE_URL - flagged families (metadata.flags): streaming SSE, multipart upload NEXUS_CATALOG.yaml at repo root is the catalog index. NEXUS_CATALOG.md documents the contribution and the operator review notes. Upstream PR: kelliott-cloud/Nexus-10.0-A#785 DRAFT: operator (Kevin) to review and decide promotion strategy: accept all 230, move under skills/nexus/ subdir (requires install-skills update), or curate a subset. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
|
Codex review: needs real behavior proof before merge. Reviewed June 10, 2026, 4:41 PM ET / 20:41 UTC. Summary Reproducibility: not applicable. This PR proposes an optional generated catalog rather than reporting broken existing behavior. The manifest and installer problems are directly source-verifiable against the submitted commit and current main. Review metrics: 4 noteworthy metrics.
Merge readiness Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch. Rank-up moves:
Proof guidance:
Risk before merge
Maintainer options:
Next step before merge
Security Review findings
Review detailsBest possible solution: Keep the generated Nexus catalog with its source generator and tests, or first establish a maintainer-approved opt-in catalog boundary here; any later contribution should be curated, use valid repository paths, provide genuinely operational instructions, exclude or explicitly govern privileged routes, and include live installation and invocation proof. Do we have a high-confidence way to reproduce the issue? Not applicable; this PR proposes an optional generated catalog rather than reporting broken existing behavior. The manifest and installer problems are directly source-verifiable against the submitted commit and current main. Is this the best way to solve the issue? No; directly adding the complete generated service API mirror to the default shared-skill directory is not the narrowest maintainable solution. An external source-owned catalog or an explicitly opt-in, curated distribution would better preserve repository scope and installation compatibility. Full review comments:
Overall correctness: patch is incorrect AGENTS.md: found and applied where relevant. Codex review notes: reasoning high; reviewed against 3446a70a26fc. Label changesLabel changes:
Label justifications:
Evidence reviewedSecurity concerns:
What I checked:
Likely related people:
What the crustacean ranks mean
Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics. How this review workflow works
|
Summary
Adds the
nexus-skills@1.0.0catalog: 230 agentskills.io-conformantskills (one per Nexus backend route surface) that let OpenJarvis users do
+ "jarvis skill install nexus:" +for any user-callable Nexus endpoint.+ "nexus-*/SKILL.md" +under+ "skills/" +.+ "bearer_token" +, env+ "NEXUS_API_TOKEN" +, scope+ "nexus.api" +,base URL via
+ "NEXUS_BASE_URL" +(declared in skill+ "metadata" +).+ "metadata.flags" +): streaming SSE, multipart upload.Upstream PR (source of truth)
the Nexus repo. This PR is the published mirror.
How it was generated
Generated deterministically by
+ "scripts/generate_agentskills.py" +in the Nexusrepo, which reads
+ "backend/routes/*.py" +and emits SKILL.md per surface.Contract tests in the Nexus repo verify spec conformance and round-trip
idempotence on every CI run, so this catalog stays in lockstep with the API
surface.
Files at a glance
+ "NEXUS_CATALOG.yaml" +— catalog index with every skill, source file, endpoint count, and flags.+ "NEXUS_CATALOG.md" +— contribution overview + operator review notes.+ "skills/nexus-*/SKILL.md" +— 230 individual skills.Operator review notes
Promotion options (operator's call):
+ "scripts/install-skills" +with no args installs every skill by default, which may not be what catalog
subscribers want.
+ "skills/nexus/" +subdirectory — requires a small update to+ "scripts/install-skills" +to recurse one level.+ "skills/" +(e.g. top 20 most user-relevant surfaces)and keep the rest catalog-only via
+ "NEXUS_CATALOG.yaml" +for discovery.+ "scripts/validate-skills" +should pass.Test plan
+ "scripts/validate-skills" +passes against the 230 new skills.🤖 Generated with Claude Code