docs(feature-research): address #1616 follow-ups (items 1-4)#1654
Merged
Conversation
…maps - Strip Rulesync-support disclaimers from Upstream surface columns across cline, qwencode, zed, factorydroid, copilotcli, augmentcode, pi, junie, roo, rovodev, goose, deepagents, kilo, replit. The Capability Gaps section is the right home for those statements. - Re-verify URLs per SKILL.md Collect step 4: replit mcp downgraded to sentinel (was a Skills tutorial, not MCP); windsurf mcp/commands/skills/hooks swapped to dedicated upstream pages confirmed via curl. - Mirror the hedged hooks phrasing in kilo subagents row so it reads consistently with the existing hooks row (KiloSubagent is registered in subagents-processor.ts). - Codify the third sentinel "Rulesync maps X; verify upstream before expanding behavior" in SKILL.md Collect step 4, and simplify the Capability Gaps caveat to reference the documented sentinel list. Refs dyoshikawa#1616 (items 1-4). Item 5 (sync-check script) is left as future work.
…ismatch note Cursor review flagged that the previous edit dropped a concrete capability-gap signal: upstream uses .kilocodeignore while Rulesync's kilo-ignore.ts maps .kiloignore. Re-add the difference using the third sentinel pattern now documented in SKILL.md.
Owner
|
@saitota Thank you! |
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Refs #1616
Summary
Addresses 4 of 5 follow-ups from #1616 in a single pass. Item 5 (sync-check script) is marked optional in the issue and is skipped here.
; no (README-)supported Rulesync … targetclauses fromOfficial docs/Upstream surfacecolumns across cline, qwencode, zed, factorydroid, copilotcli, augmentcode, pi, junie, roo, rovodev, goose, deepagents, kilo, replit. Rulesync-support deltas belong in the run-time Capability Gaps section per SKILL.md.mcpdowngraded to sentinel (old URL was a Skills tutorial, not MCP). Windsurfmcp/commands/skills/hooksswapped to dedicated Cascade docs.kilo.mdsubagents row so it reads consistently (KiloSubagentis registered insubagents-processor.ts).Rulesync maps X; verify upstream before expanding behaviorin SKILL.md Collect step 4, and simplify the Capability Gaps caveat to reference the documented sentinel list.Post-review touch-up: Cursor flagged that Item 1 dropped a concrete capability-gap signal on
kilo.mdignore. Restored using the newly documented third sentinel, since upstream uses.kilocodeignorewhilesrc/features/ignore/kilo-ignore.tsmaps.kiloignore.References
Test plan
pnpm cicheck