Skip to content

docs(jetbrains): split agent guidance into skills - #14619

Merged
kirillk merged 1 commit into
mainfrom
clever-urchin
Sep 28, 2026
Merged

kirillk merged 1 commit into
mainfrom
clever-urchin

Conversation

@kirillk

@kirillk kirillk commented Sep 27, 2026

Copy link
Copy Markdown
Contributor

Issue

No linked issue. This is an internal contributor-guidance and command organization change.

Context

packages/kilo-jetbrains/AGENTS.md was 844 lines and loaded in full for every JetBrains plugin task, even when most of its detailed UI, session, architecture, or development guidance was irrelevant. This made the always-on agent context unnecessarily large.

Keep only hard restrictions, reuse references, and the retained Swing component model in always-loaded context. Move detailed guidance into focused, on-demand skills, and make the user-initiated JetBrains release workflow an explicit command.

Implementation

Split the detailed guidance into jetbrains-ui, jetbrains-session, jetbrains-arch, and jetbrains-dev skills. The slim package AGENTS.md now indexes those skills while retaining non-negotiable restrictions and shared-code references.

Convert release-jetbrains from a skill to /release-jetbrains while preserving its helper script paths because CI and CLI pin tooling call them directly. Remove completed release setup/implementation documents and update contributor references.

Screenshots / Video

N/A. No visual or runtime product changes.

before after
N/A N/A

How to Test

Manual/local verification

  • Agent: ran bun run script/check-md-table-padding.ts; all 372 tracked Markdown files passed.
  • Agent: confirmed packages/kilo-jetbrains/AGENTS.md is 89 lines, down from 844.
  • Agent: audited every original AGENTS heading against the four new skills; all sections remain represented.
  • Agent: confirmed .kilo/skills/release-jetbrains/script/set-pin.ts still exists and script/publish.ts still references that path.
  • Push hook: root package typecheck passed for 29 packages.
  • Push hook: JetBrains ./gradlew typecheck passed.

Reviewer test steps

  1. Read packages/kilo-jetbrains/AGENTS.md and confirm it retains the hard restrictions, shared-code index, component model, and skill/command index.
  2. Compare its removed sections with the corresponding jetbrains-ui, jetbrains-session, jetbrains-arch, and jetbrains-dev skills.
  3. Confirm .kilo/command/release-jetbrains.md references the unchanged helper scripts under .kilo/skills/release-jetbrains/script/.
  4. Run bun run script/check-md-table-padding.ts.

Blocked checks and substitute verification

  • None. All relevant documentation checks and push-hook typechecks completed successfully.

Checklist

  • Issue linked above, or exception explained
  • Tests/verification described
  • Screenshots/video included for visual changes, or marked N/A
  • Changeset considered for user-facing changes (not needed; internal contributor guidance only)
  • I personally reviewed the diff and can explain the changes, including any AI-assisted work.

Get in Touch

GitHub: @kirillk

Keep hard restrictions and shared-code references always loaded while moving detailed UI, session, architecture, and development guidance into on-demand skills. Convert the release workflow to an explicit command and remove completed release planning documents.
@kilo-code-bot

kilo-code-bot Bot commented Sep 27, 2026 •

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Documentation-only refactor: packages/kilo-jetbrains/AGENTS.md is slimmed to always-on restrictions plus a skill/command index, detailed guidance moves into the jetbrains-ui, jetbrains-session, jetbrains-arch, and jetbrains-dev skills, and release-jetbrains becomes the /release-jetbrains command. Verified that all moved sections are preserved, every command/script path still resolves (.kilo/skills/release-jetbrains/script/* remains and is referenced unchanged by script/publish.ts and jetbrains-cli-pin), no stale release-jetbrains skill references remain, and the removed RELEASE_TODO.md/bundled-release-plan.md have no dangling references.

Files Reviewed (12 files)
  • .kilo/command/release-jetbrains.md
  • .kilo/skills/jetbrains-arch/SKILL.md
  • .kilo/skills/jetbrains-cli-pin/SKILL.md
  • .kilo/skills/jetbrains-dev/SKILL.md
  • .kilo/skills/jetbrains-session/SKILL.md
  • .kilo/skills/jetbrains-ui/SKILL.md
  • AGENTS.md
  • packages/kilo-docs/pages/contributing/architecture/jetbrains-plugin.md
  • packages/kilo-jetbrains/AGENTS.md
  • packages/kilo-jetbrains/RELEASE_TODO.md (deleted)
  • packages/kilo-jetbrains/RELEASING.md
  • packages/kilo-jetbrains/docs/bundled-release-plan.md (deleted)

Reviewed by deepseek-v4.1-flash · Input: 0 · Output: 0 · Cached: 0

Review guidance: REVIEW.md from base branch main

@kirillk
kirillk merged commit e5f9149 into main Sep 28, 2026
36 checks passed
@kirillk
kirillk deleted the clever-urchin branch September 28, 2026 19:38
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.

2 participants