-
-
Notifications
You must be signed in to change notification settings - Fork 141
docs(feature-research/kilo): mirror hooks phrasing for subagents row #1646
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -9,7 +9,7 @@ | |
| | `ignore` | `https://kilo.ai/docs/customize/context/kilocodeignore` | `.kilocodeignore` access-control surface; Rulesync currently maps `.kiloignore` | | ||
| | `mcp` | `https://kilo.ai/docs/automate/mcp/using-in-kilo-code` | `kilo.jsonc`, `.kilo/kilo.jsonc`, global `~/.config/kilo/kilo.jsonc`, MCP tool permissions | | ||
| | `commands` | `https://kilo.ai/docs/customize/custom-modes` | Mode and workflow customization surfaces; Rulesync maps command files | | ||
| | `subagents` | `https://kilo.ai/docs/customize/custom-subagents` | Custom modes/agents; no README-supported Rulesync Kilo subagents target | | ||
| | `subagents` | `https://kilo.ai/docs/customize/custom-subagents` | Adapter exists in source, but no README-supported Rulesync Kilo subagents target | | ||
|
Owner
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This drops the leading "Custom modes/agents;" prefix that main keeps. The hooks row omits an upstream-surface phrase because it has no dedicated upstream surface ("No dedicated upstream hooks surface in map"), but the subagents row does point at an upstream surface (custom-subagents) realized as custom modes/agents. So aligning the two phrasings verbatim removes meaningful upstream info. Better to keep the prefix. |
||
| | `skills` | `https://kilo.ai/docs/customize/skills` | `.kilocode/skills`, `~/.kilocode/skills`, mode-specific `skills-{mode}` directories | | ||
| | `hooks` | No dedicated upstream hooks surface in map | Adapter exists in source, but no README-supported Rulesync Kilo hooks target | | ||
| | `permissions` | `https://kilo.ai/docs/getting-started/settings/auto-approving-actions` | `kilo.jsonc` permission values, `allow`/`ask`/`deny`, built-in and MCP tool permission keys | | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The wording change matches the pattern already established by the hooks row — "Adapter exists in source, but no README-supported Rulesync Kilo ... target". No concerns with the content itself.