fix: deduplicate legal-ops plugin — remove Anthropic-colliding surfaces#7
Closed
mjunaidca wants to merge 2 commits into
Closed
fix: deduplicate legal-ops plugin — remove Anthropic-colliding surfaces#7mjunaidca wants to merge 2 commits into
mjunaidca wants to merge 2 commits into
Conversation
… and commands Remove duplicate skills (jurisdiction-contract-review, jurisdiction-nda-triage) that were weaker copies of Anthropic's Layer 1 review-contract and triage-nda. Remove colliding commands (/review-contract, /triage-nda, /vendor-check) that shadowed Anthropic's namespace. Add two new Layer 2 wrapper commands: - /contract-intake: orchestrated intake with jurisdiction routing + SLA tracking - /compliance-calendar: obligation tracking, renewal calendar, deadline management Additional changes: - Remove all plugin-commands from SKILL.md files (commands/ is sole owner) - Update router to hand off contract review/NDA triage to Anthropic Layer 1 - Split evals into Panaversity-local and integration (Anthropic handoff) buckets - Soften /brief vs /legal-brief from "zero overlap" to "different entrypoints" - Align all SKILL.md frontmatter with agentskills.io spec (non-standard fields → metadata) - Update hooks.json, CLAUDE.md, plugin.json (→ v2.0.0), README, exercises, workflow recipes - Add TESTING.md with pre-merge smoke test protocol Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The inline validation steps had hardcoded thresholds (5+ product cases) that didn't match the new split: 2 Panaversity-local + 3 integration. Replace redundant inline checks with single `uv run evals/run-evals.py` which validates all 4 golden files with correct ownership enforcement. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
5 tasks
Collaborator
Author
|
Superseded by #8 (v3.0.0 redesign — 1 agent + 6 skills, correct agent format per official docs, CI infra kept). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
jurisdiction-contract-review,jurisdiction-nda-triage) that were weaker copies of Anthropic's Layer 1review-contractandtriage-nda/review-contract,/triage-nda,/vendor-check) that shadowed Anthropic's namespace/contract-intake(orchestrated intake + jurisdiction routing) and/compliance-calendar(obligation tracking + deadline management)plugin-commandsfrom SKILL.md files — command ownership lives exclusively incommands/metadata)Design Principle
Anthropic owns base primitives (
/review-contract,/triage-nda,/vendor-check,/brief). Panaversity owns higher-level wrappers that add jurisdiction routing, orchestration, and unique domain skills — with clearly different names.Pre-Merge Smoke Tests (see TESTING.md)
/review-contractwith English law → UCTA overlay fires/contract-intakewrapper → hands off to Anthropic + preserves overlay/compliance-calendarstandalone → RED/YELLOW/GREEN dashboard/legal-briefdomain routing → routes to ip-protection skillFiles Changed
Deleted (5): 2 duplicate skills, 3 colliding commands
Created (4): 2 wrapper commands, 2 integration eval golden files, TESTING.md
Edited (23): router, 6 SKILL.md files (spec compliance + plugin-commands removal), hooks, CLAUDE.md, plugin.json, README, 5 exercises, 2 workflow recipes, 3 eval files
Companion PR
Book repo (Ch 22 alignment): https://github.com/panaversity/agentfactory/pull/855
🤖 Generated with Claude Code