Skip to content

fix(kilo-subagent): align read defaults#1635

Merged
dyoshikawa merged 1 commit into
dyoshikawa:mainfrom
sirmacik:fix/kilo-subagent-default-followups
May 14, 2026
Merged

fix(kilo-subagent): align read defaults#1635
dyoshikawa merged 1 commit into
dyoshikawa:mainfrom
sirmacik:fix/kilo-subagent-default-followups

Conversation

@sirmacik
Copy link
Copy Markdown
Contributor

Summary

Draft follow-up for #1634. This addresses the Kilo subagent default-mode cleanup from the maintainer-scrap issue.

Changes

  • Give Kilo its own subagent frontmatter schema with mode defaulting to all.
  • Route generated Kilo frontmatter through that schema so the default is centralized.
  • Align fromFile and deletion placeholders with the same default.
  • Strengthen the explicit kilo.mode: all test so it also proves other Kilo-specific fields survive.
  • Document Kilo's default and the kilo.mode: subagent opt-out in file-format docs and synced skill docs.
  • Update Kilo docs links from kilocode.ai to kilo.ai.

Fixes #1634.

Verification

  • pnpm test src/features/subagents/kilo-subagent.test.ts
  • pnpm test:e2e src/e2e/e2e-subagents.spec.ts
  • pnpm cicheck

Review notes

Opened as draft to signal ownership of the maintainer-scrap follow-up while keeping review pressure low. Local regression-gate review and PAL review were run; the only follow-up identified is a broader future cleanup to separate per-target subagent schemas from the shared OpenCode-style base constructor.

@github-actions
Copy link
Copy Markdown
Contributor

Thank you for your contribution! Unfortunately, you currently have 3 open PRs (including this one), which exceeds the limit of 2 for external contributors.

Please wait for an existing PR to be reviewed/merged, or close one before opening a new one. See CONTRIBUTING.md for details.

@sirmacik sirmacik marked this pull request as ready for review May 13, 2026 10:05
@dyoshikawa dyoshikawa merged commit 04ab4cf into dyoshikawa:main May 14, 2026
7 checks passed
@dyoshikawa
Copy link
Copy Markdown
Owner

@sirmacik Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PR #1632 follow-ups: kilo subagent default mode change

2 participants