Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion agents.toml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ agents = ["claude"]

[trust]
github_orgs = [ "taskless" ]
github_repos = [ "getsentry/skills", "getsentry/dotagents" ]
github_repos = [ "getsentry/skills", "getsentry/dotagents", "cloudflare/security-audit-skill" ]

[[skills]]
name = "dotagents"
Expand All @@ -16,3 +16,7 @@ source = "getsentry/skills"
[[skills]]
name = "pr-writer"
source = "getsentry/skills"

[[skills]]
name = "security-audit"
source = "cloudflare/security-audit-skill"
7 changes: 4 additions & 3 deletions dotagents.md
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ before you edit one.
**Managed by dotagents.** Declared in `agents.toml`, installed on demand, and
listed in `.agents/.gitignore` (which dotagents writes). Editing one in place
accomplishes nothing, because the next install overwrites it. Today that is
`dotagents`, `code-review`, and `pr-writer`.
`dotagents`, `code-review`, `pr-writer`, and `security-audit`.

**Committed to this repository.** Everything else under `.agents/skills/` is
tracked in git and reviewed like any other source. Two of them started as
Expand Down Expand Up @@ -93,7 +93,7 @@ exclude = [
That installs three skills: `code-simplifier`, `iterate-pr`, and
`worktrees-pnpm`. Without the exclusions the same entry resolves to 14, because
the wildcard reaches every tracked skill. Only tracked ones appear either way,
since the dotagents-managed three are gitignored and never reach a clone.
since the dotagents-managed four are gitignored and never reach a clone.

`exclude` takes literal skill names. A pattern such as `"openspec-*"` is
refused when the config is validated (`skills.0: Invalid input`), because an
Expand Down Expand Up @@ -159,6 +159,7 @@ Credit where it is owed, and a pointer for anyone tracking upstream changes.
| -------------------------- | ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | --------------------------------- |
| `code-review`, `pr-writer` | [getsentry/skills](https://github.com/getsentry/skills) | Managed by dotagents, unmodified |
| `dotagents` | [getsentry/dotagents](https://github.com/getsentry/dotagents) | Managed by dotagents, unmodified |
| `security-audit` | [cloudflare/security-audit-skill](https://github.com/cloudflare/security-audit-skill) (MIT) | Managed by dotagents, unmodified |
| `iterate-pr` | [getsentry/skills](https://github.com/getsentry/skills/tree/main/skills/iterate-pr) (Apache-2.0) | Forked and substantially modified |
| `code-simplifier` | [getsentry/skills](https://github.com/getsentry/skills), which credits [Anthropic's code-simplifier](https://github.com/anthropics/claude-plugins-official/blob/main/plugins/code-simplifier/agents/code-simplifier.md) | Vendored copy |
| `openspec-*` | [openspec](https://github.com/Fission-AI/OpenSpec) (MIT) | Generated by the openspec CLI |
Expand All @@ -178,7 +179,7 @@ if you want none of that.
```toml
[trust]
github_orgs = [ "taskless" ]
github_repos = [ "getsentry/skills", "getsentry/dotagents" ]
github_repos = [ "getsentry/skills", "getsentry/dotagents", "cloudflare/security-audit-skill" ]
```

A source outside that list is refused rather than fetched. Widen it
Expand Down
6 changes: 6 additions & 0 deletions eslint.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,12 @@ export default tseslint.config(
// worth doing, but it is a separate change — the rules that would apply
// are not these ones.
".agents/skills/*/scripts/",
// Third-party skills that dotagents installs (see `agents.toml`). They
// are gitignored and replaced on every install, so there is nothing here
// to fix. Most are prose, but `security-audit` ships zero-dependency
// CommonJS validators at its top level, which the type-aware rules
// cannot parse and which would otherwise fail `pnpm lint` locally.
".agents/skills/security-audit/",
// Taskless rule fixtures. A rule's `tests/` holds inputs written to be
// flagged, and a rule about source comments needs `.ts` fixtures
// specifically — Vale picks its comments-only tier by extension. They are
Expand Down
Loading