/review
Reviews an inbound agreement against the playbook in ~/.codebuddy/plugins/config/legal-workflows/commercial-legal/practice-profile.md. Identifies the agreement structure from titles, selects the appropriate skill(s), and — if confirm_routing is enabled — checks with the user before proceeding.
Instructions
Load ~/.codebuddy/plugins/config/legal-workflows/commercial-legal/practice-profile.md. If placeholders present, stop and prompt: "Run /commercial-legal:cold-start-interview first — I need to learn your playbook before I can review against it."
Also read ~/.codebuddy/plugins/config/legal-workflows/commercial-legal/practice-profile.md → ## Review preferences → confirm_routing. If the field is missing, treat it as true.
Get the agreement: From file path, Drive link, [CLM ID], or pasted text. If none provided, ask.
Read the document structure — titles first.
Before reading the body, extract:
- The main agreement title (e.g., "Master Services Agreement", "Non-Disclosure Agreement")
- All exhibit, schedule, addendum, and attachment titles (e.g., "Exhibit A — Data Processing Addendum", "Schedule 1 — Subscription Order Form", "Annex B — Service Level Agreement")
This is the routing signal. Do not rely on body keywords alone — a 40-page MSA with "confidential" throughout is not an NDA.
Select the skill(s) based on document structure.
Map each identified document or section to a skill:
| Document / section title contains |
Skill |
| Non-Disclosure, NDA, Confidentiality Agreement (as the main agreement) |
nda-review |
| Master Services Agreement, Professional Services, Statement of Work, Consulting Agreement |
vendor-agreement-review |
| Subscription, SaaS, Cloud Services, Order Form with auto-renewal, Software License with recurring fees |
saas-msa-review (overlay on vendor-agreement-review) |
| Data Processing Addendum, DPA, Data Processing Agreement (as exhibit or standalone) |
note for vendor-agreement-review → data protection section |
| Service Level Agreement, SLA (as exhibit) |
note for saas-msa-review → SLA section |
Multiple skills may apply. Common combinations:
- MSA + DPA exhibit → vendor-agreement-review, with DPA noted
- SaaS subscription + Order Form + SLA exhibit → saas-msa-review (covers all three)
- MSA + Order Form with auto-renewal → vendor-agreement-review + saas-msa-review overlay
When the structure is genuinely ambiguous after reading titles (e.g., a document titled "Agreement" with no exhibits listed), read the first two pages of the body to resolve it — then stop and route.
Confirm routing if enabled.
If confirm_routing is true in ~/.codebuddy/plugins/config/legal-workflows/commercial-legal/practice-profile.md (or field is absent):
I'm going to review this as: [agreement type(s)].
Documents identified:
- [Main agreement title] → [skill]
- [Exhibit A title] → [how it will be handled]
- [Exhibit B title] → [how it will be handled]
Sound right? (yes / no — or tell me what I got wrong)
Wait for confirmation before proceeding. If the user corrects the routing, apply their instruction and proceed.
If confirm_routing is false: proceed silently. Log the routing decision at the top of the review memo so the user can see what was applied.
Run the skill(s). Follow each skill's workflow fully. If multiple skills apply, run them in sequence and integrate the output into a single memo — don't produce separate memos.
Check for escalations: If any issue exceeds the reviewer's authority per the ~/.codebuddy/plugins/config/legal-workflows/commercial-legal/practice-profile.md matrix, invoke escalation-flagger to route and draft the ask.
Offer follow-ups:
- Stakeholder summary for the business owner
- Redline .docx with tracked changes
- [CLM] record creation (if connected)
- Add to renewal register (if auto-renewal found)
Configuring confirm_routing
Add to ~/.codebuddy/plugins/config/legal-workflows/commercial-legal/practice-profile.md → ## Review preferences:
## Review preferences
confirm_routing: true # Set to false to skip routing confirmation and proceed automatically
The cold-start interview should ask about this preference. Default is true — confirmation on. As trust builds, the user can set it to false.
Examples
/commercial-legal:review vendor-msa.pdf
/commercial-legal:review https://drive.google.com/file/d/ABC123
/commercial-legal:review
[paste agreement text]
Output
Full review memo per the skill's format. Routing decision logged at the top. Deviation-by-deviation, specific redline language, named approver. Saved where ~/.codebuddy/plugins/config/legal-workflows/commercial-legal/practice-profile.md → House style says work product goes.
1---2name: review3description: Review a vendor agreement, NDA, or SaaS subscription against your playbook. Identifies the agreement structure from titles, routes to the right review skill (vendor-agreement-review, nda-review, saas-msa-review), and integrates the output into a single memo. Use when the user says "review this contract", "check this MSA", "is this NDA okay", "look at this SaaS agreement", or attaches an inbound agreement for review.4---5
6# /review
7
8Reviews an inbound agreement against the playbook in `~/.codebuddy/plugins/config/legal-workflows/commercial-legal/practice-profile.md`. Identifies the agreement structure from titles, selects the appropriate skill(s), and — if confirm_routing is enabled — checks with the user before proceeding.
9
10## Instructions
11
121. **Load `~/.codebuddy/plugins/config/legal-workflows/commercial-legal/practice-profile.md`.** If placeholders present, stop and prompt: "Run `/commercial-legal:cold-start-interview` first — I need to learn your playbook before I can review against it."
13
14 Also read `~/.codebuddy/plugins/config/legal-workflows/commercial-legal/practice-profile.md` → `## Review preferences` → `confirm_routing`. If the field is missing, treat it as `true`.
15
162. **Get the agreement:** From file path, Drive link, [CLM ID], or pasted text. If none provided, ask.
17
183. **Read the document structure — titles first.**
19
20 Before reading the body, extract:
21 - The main agreement title (e.g., "Master Services Agreement", "Non-Disclosure Agreement")
22 - All exhibit, schedule, addendum, and attachment titles (e.g., "Exhibit A — Data Processing Addendum", "Schedule 1 — Subscription Order Form", "Annex B — Service Level Agreement")
23
24 This is the routing signal. Do not rely on body keywords alone — a 40-page MSA with "confidential" throughout is not an NDA.
25
264. **Select the skill(s) based on document structure.**
27
28 Map each identified document or section to a skill:
29
30 | Document / section title contains | Skill |
31 |---|---|
32 | Non-Disclosure, NDA, Confidentiality Agreement (as the *main* agreement) | **nda-review** |
33 | Master Services Agreement, Professional Services, Statement of Work, Consulting Agreement | **vendor-agreement-review** |
34 | Subscription, SaaS, Cloud Services, Order Form with auto-renewal, Software License with recurring fees | **saas-msa-review** (overlay on vendor-agreement-review) |
35 | Data Processing Addendum, DPA, Data Processing Agreement (as exhibit or standalone) | note for **vendor-agreement-review** → data protection section |
36 | Service Level Agreement, SLA (as exhibit) | note for **saas-msa-review** → SLA section |
37
38 Multiple skills may apply. Common combinations:
39 - MSA + DPA exhibit → vendor-agreement-review, with DPA noted
40 - SaaS subscription + Order Form + SLA exhibit → saas-msa-review (covers all three)
41 - MSA + Order Form with auto-renewal → vendor-agreement-review + saas-msa-review overlay
42
43 When the structure is genuinely ambiguous after reading titles (e.g., a document titled "Agreement" with no exhibits listed), read the first two pages of the body to resolve it — then stop and route.
44
455. **Confirm routing if enabled.**
46
47 If `confirm_routing` is `true` in `~/.codebuddy/plugins/config/legal-workflows/commercial-legal/practice-profile.md` (or field is absent):
48
49 ```
50 I'm going to review this as: [agreement type(s)].
51
52 Documents identified:
53 - [Main agreement title] → [skill]
54 - [Exhibit A title] → [how it will be handled]
55 - [Exhibit B title] → [how it will be handled]
56
57 Sound right? (yes / no — or tell me what I got wrong)
58 ```
59
60 Wait for confirmation before proceeding. If the user corrects the routing, apply their instruction and proceed.
61
62 If `confirm_routing` is `false`: proceed silently. Log the routing decision at the top of the review memo so the user can see what was applied.
63
646. **Run the skill(s).** Follow each skill's workflow fully. If multiple skills apply, run them in sequence and integrate the output into a single memo — don't produce separate memos.
65
667. **Check for escalations:** If any issue exceeds the reviewer's authority per the `~/.codebuddy/plugins/config/legal-workflows/commercial-legal/practice-profile.md` matrix, invoke **escalation-flagger** to route and draft the ask.
67
688. **Offer follow-ups:**
69 - Stakeholder summary for the business owner
70 - Redline .docx with tracked changes
71 - [CLM] record creation (if connected)
72 - Add to renewal register (if auto-renewal found)
73
74## Configuring confirm_routing
75
76Add to `~/.codebuddy/plugins/config/legal-workflows/commercial-legal/practice-profile.md` → `## Review preferences`:
77
78```markdown
79## Review preferences
80
81confirm_routing: true # Set to false to skip routing confirmation and proceed automatically
82```
83
84The cold-start interview should ask about this preference. Default is `true` — confirmation on. As trust builds, the user can set it to `false`.
85
86## Examples
87
88```
89/commercial-legal:review vendor-msa.pdf
90```
91
92```
93/commercial-legal:review https://drive.google.com/file/d/ABC123
94```
95
96```
97/commercial-legal:review
98[paste agreement text]
99```
100
101## Output
102
103Full review memo per the skill's format. Routing decision logged at the top. Deviation-by-deviation, specific redline language, named approver. Saved where `~/.codebuddy/plugins/config/legal-workflows/commercial-legal/practice-profile.md` → House style says work product goes.
ahang1598