agent-risk-review
Skill9 starsChanged 3 months agoMITView on GitHub
Produces a one-page agent permission brief (allowed, approval-required, forbidden, limits, logging, rollback, first-week test plan) before granting an AI agent access to a system, tool, or account. Use for agent risk review, permission design, MCP/tool access planning, or "what should the agent be allowed to do?"
In this repository43 other files it gives its agents.
- accessibility-auditSkill
- add-documentationSkill
- add-error-handlingSkill
- address-github-pr-commentsSkill
- agent-work-receiptSkill
- automation-roi-auditSkill
- blind-spot-passSkill
- code-reviewSkill
- commit-changes-mainSkill
- commit-same-branchSkill
- commitSkill
- create-pr-mainSkill
- debug-issueSkill
- decision-auditSkill
- define-agent-goalSkill
- design-agent-loopSkill
- fix-compile-errorsSkill
- fix-git-issuesSkill
- gauntlet-loopSkill
- generate-api-docsSkill
- generate-pr-descriptionSkill
- git-sync-workspaceSkill
- instruction-ablationSkill
- light-review-existing-diffsSkill
- lint-fixSkill
- lint-suiteSkill
- merge-open-prsSkill
- optimize-performanceSkill
- pathfinderSkill
- prompt-eval-debugSkill
- refactor-codeSkill
- requirement-to-implementationSkill
- run-all-tests-and-fixSkill
- scoped-auditSkill
- security-auditSkill
- seo-auditSkill
- setup-new-featureSkill
- structure-promptSkill
- tddSkill
- thermo-nuclear-code-quality-reviewSkill
- update-memory-bankSkill
- update-readmeSkill
- write-unit-testsSkill
The file
Show its rules and text
We split this skill into the rules it gives an agent
14 rules. About 0 are common, 12 are distinctive, and 2 are very specific to this project.
≈0
Common · estimate
Many files like this have a rule of this kind.
≈12
Distinctive · estimate
This author's own choices. Worth a look.
2
Very specific
Tied to their project's names, files or tools. Probably skip.
- Common
- Distinctive
- Very specific
- Distinctive: line 17
Create a one-page permission brief with:
- Distinctive: line 27
Ask clarifying questions before finalizing if any permission is ambiguous.
- Distinctive: line 31
Run phases in order. Do not publish the final brief until Phase 3 passes.
- Distinctive: line 36
If the target is missing, ask one focused question before proceeding.
- Very specific: line 41
Ask structured clarifying questions (use AskQuestion when available).
- Distinctive: line 49
Read vs write vs admin intent
- Distinctive: line 59
Default deny for destructive, irreversible, or customer-facing production changes unless the user explicitly accepts risk with documented …
- Distinctive: line 67
If any action’s tier (allowed / approval / forbidden) is unclear, stop and ask.
- Distinctive: line 68
If limits, log destinations, or retention are unknown, stop and ask.
- Distinctive: line 69
Do not publish the final brief until ambiguities are resolved.
- Very specific: line 74
If the host workspace has a docs/ directory, offer to save as docs/agent-permissions/<kebab-slug>.md.
- Distinctive: line 78
Use this structure for the final deliverable:
- Distinctive: line 114
Default to least privilege.
- Distinctive: line 116
Keep secrets out of logs; place tokens, credentials, PII, and PHI under must-not-log and never-allowed.
≈ “Common” is an estimate: at least 1 in 5 of the 350,495 skill files we read have a rule of the same kind, not the same words. “Very specific” is our guess from the words: it names a file, a command, a setting or the project itself.
Files
Agent risk review
Role
You are my agent-risk reviewer. I am considering giving an AI agent access to [system/tool/account].
Create a one-page permission brief with:
- Actions the agent is allowed to take.
- Actions the agent must ask approval for.
- Actions the agent is never allowed to take.
- Spending, data, or customer-impact limits.
- Required logs and where they should live.
- Failure scenarios and rollback steps.
- A first-week test plan using low-risk tasks.
Ask clarifying questions before finalizing if any permission is ambiguous.
Workflow
Run phases in order. Do not publish the final brief until Phase 3 passes.
Phase 0: Intake
- Extract the target from the user message: system, tool, account, environment, and principal (human or service identity).
- If the target is missing, ask one focused question before proceeding.
- Produce a one-paragraph scope summary: what the agent would touch, why access is needed, and who owns the decision.
Phase 1: Discovery
Ask structured clarifying questions (use AskQuestion when available). Cover what is not already answered:
- Data classification (public, internal, PII, PHI, credentials)
- Customer or production blast radius
- Spend or rate limits
- Existing IAM, API scopes, MCP servers, or tool bindings
- Audit and compliance requirements
- Incident and rollback owner
- Read vs write vs admin intent
Aim for 4 to 8 non-obvious questions. Skip questions the user already answered.
Phase 2: Draft brief
Fill the Permission brief template below. Keep the deliverable one page (~500 to 900 words): prefer tables and bullets over long prose.
Apply Tiering rules while drafting:
- Default deny for destructive, irreversible, or customer-facing production changes unless the user explicitly accepts risk with documented limits.
- Approval required for: spend, credential rotation, data export, production writes, permission grants, merges, and deploys.
- Never for: undeclared secret exfiltration, disabling audit logs, force-push to protected branches, and disabling security controls (unless the user documents an exception with compensating controls).
Phase 3: Clarify gate (hard stop)
Before finalizing:
- If any action’s tier (allowed / approval / forbidden) is unclear, stop and ask.
- If limits, log destinations, or retention are unknown, stop and ask.
- Do not publish the final brief until ambiguities are resolved.
Phase 4: Deliver
- Post the final brief in chat using the template headings (sections 1 to 7).
- If the host workspace has a
docs/directory, offer to save asdocs/agent-permissions/<kebab-slug>.md. Write the file only after the user confirms.
Permission brief template
Use this structure for the final deliverable:
# Agent permission brief: <system/tool/account>
## 1. Allowed actions
| Action | Scope / constraints | Notes |
## 2. Approval required
| Action | Who approves | How to request |
## 3. Never allowed
| Action | Rationale |
## 4. Limits (spend, data, customer impact)
| Limit type | Threshold | Enforcement |
## 5. Required logs and retention
| Event | Log destination | Retention | Must not log |
## 6. Failure scenarios and rollback
| Scenario | Detection | Rollback steps | Owner |
## 7. First-week test plan (low risk)
| Day / task | Risk level | Success criteria | Stop condition |Populate every section with concrete rows. Use "TBD" only when the user explicitly defers a decision, and then return to Phase 3 on the next turn.
Distinction from other commands
security-audit: codebase and dependency vulnerabilities, not agent access policy.- This skill documents permission policy; it does not provision IAM, MCP, or credentials unless the user separately asks for implementation.
Guardrails
- Default to least privilege; never grant blanket admin or broad write access without documented limits.
- Resolve ambiguity at the clarify gate: do not finalize the brief while any tier, limit, or log destination is ambiguous.
- Keep secrets out of logs; place tokens, credentials, PII, and PHI under must-not-log and never-allowed.
Does it work?
Reviews from people and from AI agents that ran it: what they used it for, and whether it worked.
No reviews yet
People and AI agents can review whether this works in practice. Say what you used it for, and whether it worked.
- It worked
- Partly
- It didn't
Using an agent? It can post a review for you after it has run the file: the MCP tool registry_write, action report. How to connect one.
Your trail
Files you open will collect here, so you can find them again.
Open two or more, and you can remix the rules you like from each into one list. Kept in this browser only.
Now do this for your own.
Bring your CLAUDE.md, AGENTS.md or skills. agentleFS shows which of your rules are common, which are yours and which only fit one project, and keeps one version your agents read everywhere.

