docs(readme): restructure the support matrix into a presence overview - #1710
Conversation
88cc256 to
0b4dee7
Compare
dyoshikawa
left a comment
There was a problem hiding this comment.
Thanks for taking this on — the dense matrix really was hard to scan, and the presence-overview idea reads a lot better. The Kilo subagents/hooks correction is a genuine find too.
A couple of things to sort before this can merge. The branch is based on an older main, so as-is it reverts the Antigravity 2.0 split (antigravity-ide / antigravity-cli) and drops the deprecation markers — a rebase should clear that up. The bigger open question is AGENTS.md and AgentsSkills: the description says they moved into an Open Standards section, but that table isn't in the diff, so right now they just disappear. And since you're correcting Kilo, docs/reference/supported-tools.md needs the same edit to stay in sync.
On the design call itself, dropping the project/global/simulated nuance from the README in favor of the reference link is reasonable, but that's a maintainer judgement — I'd let dyoshikawa weigh in on losing the at-a-glance mode info. Left a few inline notes on the specific spots.
| | Takt | ✅ | | | ✅ | ✅ | ✅ | | | | ||
| | Qwen Code | ✅ | ✅ | | | | | | ✅ | | ||
| | Kiro | ✅ | ✅ | ✅ | ✅ | ✅ | ✅ | ✅ | ✅ | | ||
| | Google Antigravity | ✅ | | | ✅ | | ✅ | | | |
There was a problem hiding this comment.
This branch looks like it's based on an older main. Since the merge base, main split Antigravity into antigravity-ide and antigravity-cli (Antigravity 2.0), and both support mcp and hooks. Collapsing them back into a single "Google Antigravity" row with mcp/hooks blank drops two rows and reads as a regression against current main. The ⚠️ deprecation markers on Gemini CLI and Antigravity also disappear here. Could you rebase on latest main and re-derive the table from the current one?
| - 🔧: Supports MCP tool config (`enabledTools`/`disabledTools`) | ||
| The table below shows whether each tool supports a given feature (✅ = supported, blank = not supported). For each tool's `--targets` value and mode details (project / global / simulated), see the [Supported Tools reference](https://rulesync.dyoshikawa.com/reference/supported-tools). | ||
|
|
||
| | Tool | rules | ignore | mcp | commands | subagents | skills | hooks | permissions | |
There was a problem hiding this comment.
The description mentions splitting AI Coding Tools from Open Standards (AGENTS.md, Agent Skills), but I don't see an Open Standards table in the diff — the agentsmd and agentsskills rows are just removed. Both are still valid --targets (src/types/tool-targets.ts) and present in the docs reference table, so they vanish from the README entirely. Could you add the Open Standards table you mentioned, or keep these two in the matrix?
| | Factory Droid | ✅ | | ✅ | ✅ | ✅ | ✅ | ✅ | | | ||
| | OpenCode | ✅ | | ✅ | ✅ | ✅ | ✅ | ✅ | ✅ | | ||
| | Cline | ✅ | ✅ | ✅ | ✅ | | ✅ | | ✅ | | ||
| | Kilo Code | ✅ | ✅ | ✅ | ✅ | ✅ | ✅ | ✅ | ✅ | |
There was a problem hiding this comment.
Marking Kilo subagents/hooks as ✅ is correct — KiloSubagent and KiloHooks are both registered, so the README was genuinely lagging here. Nice catch. One follow-up: docs/reference/supported-tools.md still shows these blank, and the project guidelines ask for README and docs to stay in sync, so it'd be good to update that file in the same PR.
0b4dee7 to
641f602
Compare
|
Thanks for the review! Addressed all three:
|
Split the dense matrix into AI Coding Tools / Open Standards presence tables (✅ / blank), and link to the Supported Tools reference for --targets values and project/global/simulated mode details.
641f602 to
02e796f
Compare
Resolve README.md conflict by keeping the presence-overview restructuring. The overview already reflects the latest data; Kilo Code (subagents/hooks) and Kiro (hooks) markers match the processors and the merged reference table. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
@saitota Thank you! |
Summary
The single dense matrix was hard to read, so I tried restructuring it. Sharing as a proposal — please judge whether it's worth merging.
--targetsvalues and mode details (project / global / simulated) to the Supported Tools reference, which already has that table.⚠️Deprecated markers and the Deprecation notes section as-is.Note
The matrix was missing 3 entries the code already supports. Synced both
README.mdanddocs/reference/supported-tools.mdto the implementation:kilo× subagents/hooks,kiro× hooks.Test plan
src/features/*/(29 targets incl. open standards)oxfmt --check/cspell