Skip to content

Mirror: docs: clarify slash commands /newtask vs /smol (#5869)#5

Merged
jeremylongshore merged 1 commit into
mainfrom
review/PR-5869
Feb 15, 2026
Merged

Mirror: docs: clarify slash commands /newtask vs /smol (#5869)#5
jeremylongshore merged 1 commit into
mainfrom
review/PR-5869

Conversation

@jeremylongshore
Copy link
Copy Markdown
Owner

@jeremylongshore jeremylongshore commented Feb 14, 2026

Mirror of Kilo-Org#5869

Field Value
Upstream PR #5869
Author @EloiRamos
Category docs
Tier 1
Size 17+/3- lines, 2 files

This PR mirrors the upstream change for multi-AI review analysis.

Original Description

Clarifies the difference between /newtask and /smol slash commands. Adds comparison table to Using Modes page and cross-reference in CLI docs. Fixes Kilo-Org#2160.

Bot Review Tracker

  • CodeRabbit
  • Gemini Code Assist
  • Greptile
  • CodeQL
  • Qodo PR-Agent

Links

Summary by CodeRabbit

Documentation

  • Expanded guidance on switching between AI agent modes with detailed descriptions of /newtask and /smol commands
  • Added comparison table clarifying differences between mode commands
  • Introduced comprehensive slash commands documentation with usage guidance and IDE mode tips

@gemini-code-assist
Copy link
Copy Markdown

Summary of Changes

Hello @jeremylongshore, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request enhances the clarity of the documentation by specifically addressing common confusion between the /newtask and /smol slash commands. It provides a dedicated comparison table and ensures that users accessing CLI-related documentation are directed to this clarification, thereby improving the overall user experience and reducing potential misunderstandings.

Highlights

  • Documentation Clarity: Clarified the distinction between the /newtask and /smol slash commands within the documentation.
  • Comparison Table Added: Introduced a new comparison table on the 'Using Modes' page to detail the purpose and usage of /newtask and /smol.
  • Cross-referencing: Added a cross-reference in the CLI documentation to guide users to the detailed comparison of /newtask and /smol.
Changelog
  • apps/kilocode-docs/pages/code-with-ai/agents/using-modes.md
    • Updated the description of slash commands to explicitly mention /newtask and /smol.
    • Added a new section titled 'Understanding /newtask vs /smol' with a detailed comparison table.
  • apps/kilocode-docs/pages/code-with-ai/platforms/cli.md
    • Introduced a new 'Slash Commands' section.
    • Added a callout tip to direct users to the 'Using Modes' documentation for clarification on /newtask vs /smol.
    • Reorganized the configuration management details, removing specific bullet points for /connect and config files.
Activity
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Feb 14, 2026

📝 Walkthrough

Walkthrough

Documentation files have been expanded to provide clearer guidance on slash commands and agent modes, introducing new comparison tables and dedicated sections to explain command purposes and functionality.

Changes

Cohort / File(s) Summary
Documentation - Slash Commands & Modes
apps/kilocode-docs/pages/code-with-ai/agents/using-modes.md, apps/kilocode-docs/pages/code-with-ai/platforms/cli.md
Added and reorganized documentation for slash commands (/newtask, /smol). Introduced a new comparison table contrasting /newtask (start new task with context) versus /smol (condense context window), and created a dedicated "Slash Commands" section with explanatory text and IDE mode usage tip.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Slash commands now gleam with detail so bright,
Tables and tips guide users just right,
Context and modes, all carefully penned,
Documentation improved from start to end!

🚥 Pre-merge checks | ✅ 2 | ❌ 2
❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Description check ⚠️ Warning The description provides context about the mirrored PR, original author, category, and specific changes made. However, it does not follow the repository's required template structure with Context, Implementation, Screenshots, How to Test, and Get in Touch sections. Restructure the description to follow the template: add Context section explaining WHY the clarification is needed, Implementation section describing HOW the docs were updated, Screenshots showing before/after, How to Test instructions, and Get in Touch information.
Merge Conflict Detection ⚠️ Warning ❌ Merge conflicts detected (28 files):

⚔️ .devcontainer/devcontainer.json (content)
⚔️ .github/dependabot.yml (content)
⚔️ apps/kilocode-docs/components/TopNav.tsx (content)
⚔️ apps/kilocode-docs/lib/nav/customize.ts (content)
⚔️ apps/kilocode-docs/pages/ai-providers/kilocode.md (content)
⚔️ apps/kilocode-docs/pages/ai-providers/ollama.md (content)
⚔️ apps/kilocode-docs/pages/ai-providers/v0.md (content)
⚔️ apps/kilocode-docs/pages/ai-providers/virtual-quota-fallback.md (content)
⚔️ apps/kilocode-docs/pages/automate/extending/shell-integration.md (content)
⚔️ apps/kilocode-docs/pages/code-with-ai/agents/using-modes.md (content)
⚔️ apps/kilocode-docs/pages/code-with-ai/features/browser-use.md (content)
⚔️ apps/kilocode-docs/pages/code-with-ai/platforms/cli.md (content)
⚔️ apps/kilocode-docs/pages/collaborate/adoption-dashboard/overview.md (content)
⚔️ apps/kilocode-docs/pages/collaborate/adoption-dashboard/understanding-your-score.md (content)
⚔️ apps/kilocode-docs/pages/collaborate/enterprise/audit-logs.md (content)
⚔️ apps/kilocode-docs/pages/collaborate/enterprise/model-access-controls.md (content)
⚔️ apps/kilocode-docs/pages/collaborate/enterprise/sso.md (content)
⚔️ apps/kilocode-docs/pages/collaborate/teams/dashboard.md (content)
⚔️ apps/kilocode-docs/pages/collaborate/teams/getting-started.md (content)
⚔️ apps/kilocode-docs/pages/contributing/development-environment.md (content)
⚔️ apps/kilocode-docs/pages/customize/context/codebase-indexing.md (content)
⚔️ apps/kilocode-docs/pages/customize/custom-modes.md (content)
⚔️ apps/kilocode-docs/pages/customize/custom-rules.md (content)
⚔️ apps/kilocode-docs/pages/customize/index.md (content)
⚔️ apps/kilocode-docs/pages/deploy-secure/deploy.md (content)
⚔️ apps/kilocode-docs/pages/getting-started/faq.md (content)
⚔️ apps/kilocode-docs/pages/getting-started/installing.md (content)
⚔️ apps/kilocode-docs/public/globals.css (content)

These conflicts must be resolved before merging into main.
Resolve conflicts locally and push changes to this branch.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: clarifying the difference between /newtask and /smol slash commands in documentation, which aligns with the actual documentation updates in both files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch review/PR-5869
⚔️ Resolve merge conflicts (beta)
  • Auto-commit resolved conflicts to branch review/PR-5869
  • Create stacked PR with resolved conflicts
  • Post resolved changes as copyable diffs in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown

Failed to generate code suggestions for PR

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

The documentation changes provide helpful clarification on the /newtask and /smol commands. However, there are some formatting and structural issues: a header inserted within a numbered list in using-modes.md breaks the list continuity, and the placement of a new section in cli.md disrupts the existing 'Configuration' section and accidentally removes useful information about config file locations.

Comment on lines +31 to +38
### Understanding /newtask vs /smol

Users often confuse `/newtask` and `/smol`. Here's the key difference:

| Command | Purpose | When to Use |
| ---------- | ----------------------------------------------------- | ----------------------------------------------------------------------- |
| `/newtask` | Creates a new task with context from the current task | When you want to start something new while carrying over context |
| `/smol` | Condenses your current context window | When your conversation is getting too long and you want to summarize it |
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

Inserting a level-3 header (###) inside a numbered list breaks the list's continuity in most Markdown parsers. This will cause the subsequent list item (currently step 3) to restart its numbering at '1.' or render incorrectly. To maintain the list structure, you should indent this explanatory section so it remains part of step 2.

Suggested change
### Understanding /newtask vs /smol
Users often confuse `/newtask` and `/smol`. Here's the key difference:
| Command | Purpose | When to Use |
| ---------- | ----------------------------------------------------- | ----------------------------------------------------------------------- |
| `/newtask` | Creates a new task with context from the current task | When you want to start something new while carrying over context |
| `/smol` | Condenses your current context window | When your conversation is getting too long and you want to summarize it |
### Understanding /newtask vs /smol
Users often confuse `/newtask` and `/smol`. Here's the key difference:
| Command | Purpose | When to Use |
| ---------- | ----------------------------------------------------- | ----------------------------------------------------------------------- |
| `/newtask` | Creates a new task with context from the current task | When you want to start something new while carrying over context |
| `/smol` | Condenses your current context window | When your conversation is getting too long and you want to summarize it |

Comment on lines +167 to +173
## Slash Commands

The CLI's interactive mode supports slash commands for common operations. The main commands are documented above in the [Interactive Slash Commands](#interactive-slash-commands) section.

{% callout type="tip" %}
**Confused about /newtask vs /smol in the IDE?** See the [Using Modes](/docs/code-with-ai/agents/using-modes#understanding-newtask-vs-smol) documentation for details.
{% /callout %}
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The new 'Slash Commands' header is placed immediately after the 'Configuration is managed through:' sentence, which breaks the grammatical flow and orphans the subsequent list items. Additionally, the reference to the config file location (~/.config/kilo/config.json) was removed. It is better to keep the configuration list intact and place the new section separately.

Suggested change
## Slash Commands
The CLI's interactive mode supports slash commands for common operations. The main commands are documented above in the [Interactive Slash Commands](#interactive-slash-commands) section.
{% callout type="tip" %}
**Confused about /newtask vs /smol in the IDE?** See the [Using Modes](/docs/code-with-ai/agents/using-modes#understanding-newtask-vs-smol) documentation for details.
{% /callout %}
- `/connect` command for provider setup (interactive)
- Config files directly at `~/.config/kilo/config.json`
## Slash Commands
The CLI's interactive mode supports slash commands for common operations. The main commands are documented above in the [Interactive Slash Commands](#interactive-slash-commands) section.
{% callout type="tip" %}
**Confused about /newtask vs /smol in the IDE?** See the [Using Modes](/docs/code-with-ai/agents/using-modes#understanding-newtask-vs-smol) documentation for details.
{% /callout %}

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@apps/kilocode-docs/pages/code-with-ai/platforms/cli.md`:
- Around line 167-174: The document has an orphaned bullet "kilo auth" that
follows the sentence "Configuration is managed through:" but sits after the new
"## Slash Commands" header; move the `kilo auth` bullet so it appears
immediately after the "Configuration is managed through:" sentence (i.e., before
the "## Slash Commands" section) to make the list structurally correct, keeping
the "## Slash Commands" section and its callout unchanged.
🧹 Nitpick comments (1)
apps/kilocode-docs/pages/code-with-ai/agents/using-modes.md (1)

31-38: Excellent addition with minor style suggestion.

The new subsection effectively clarifies the distinction between /newtask and /smol with a clear comparison table. The content is accurate and addresses the confusion mentioned in the PR objectives.

Minor style suggestion: The "When to Use" column uses "want to" twice (lines 37 and 38). Consider varying the phrasing for smoother reading:

✍️ Optional style refinement
 | Command    | Purpose                                               | When to Use                                                             |
 | ---------- | ----------------------------------------------------- | ----------------------------------------------------------------------- |
-| `/newtask` | Creates a new task with context from the current task | When you want to start something new while carrying over context        |
-| `/smol`    | Condenses your current context window                 | When your conversation is getting too long and you want to summarize it |
+| `/newtask` | Creates a new task with context from the current task | When starting something new while carrying over context        |
+| `/smol`    | Condenses your current context window                 | When your conversation is getting too long and needs summarizing |

Comment on lines +167 to 174
## Slash Commands

The CLI's interactive mode supports slash commands for common operations. The main commands are documented above in the [Interactive Slash Commands](#interactive-slash-commands) section.

{% callout type="tip" %}
**Confused about /newtask vs /smol in the IDE?** See the [Using Modes](/docs/code-with-ai/agents/using-modes#understanding-newtask-vs-smol) documentation for details.
{% /callout %}
- `kilo auth` for credential management
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

Fix document structure: orphaned bullet point.

The new "Slash Commands" section is clear and helpful, but it creates a structural issue. Line 165 says "Configuration is managed through:" which suggests a list should follow, but instead there's a peer-level section header (## Slash Commands), and then a single orphaned bullet point at line 174.

Consider one of these solutions:

  1. Move the kilo auth bullet point before the new Slash Commands section (after line 165)
  2. Convert line 174 to a regular paragraph without the bullet
  3. Update line 165 to not imply a list follows
📝 Proposed fix: Move the bullet point before the new section
 Configuration is managed through:
 
+- `kilo auth` for credential management
+
 ## Slash Commands
 
 The CLI's interactive mode supports slash commands for common operations. The main commands are documented above in the [Interactive Slash Commands](`#interactive-slash-commands`) section.
 
 {% callout type="tip" %}
 **Confused about /newtask vs /smol in the IDE?** See the [Using Modes](/docs/code-with-ai/agents/using-modes#understanding-newtask-vs-smol) documentation for details.
 {% /callout %}
-- `kilo auth` for credential management
🤖 Prompt for AI Agents
In `@apps/kilocode-docs/pages/code-with-ai/platforms/cli.md` around lines 167 -
174, The document has an orphaned bullet "kilo auth" that follows the sentence
"Configuration is managed through:" but sits after the new "## Slash Commands"
header; move the `kilo auth` bullet so it appears immediately after the
"Configuration is managed through:" sentence (i.e., before the "## Slash
Commands" section) to make the list structurally correct, keeping the "## Slash
Commands" section and its callout unchanged.

@jeremylongshore jeremylongshore merged commit 35196a2 into main Feb 15, 2026
16 checks passed
@jeremylongshore jeremylongshore deleted the review/PR-5869 branch February 15, 2026 20:45
@jeremylongshore
Copy link
Copy Markdown
Owner Author

Review: kilocode Kilo-Org#5869

docs: clarify slash commands (/newtask vs /smol) (Kilo-Org#2160) by @EloiRamos
Multi-AI analysis: Fork PR #5 — reviewed by CodeRabbit, Gemini, CodeQL, Qodo

Checklist

Check Result Notes
Correctness PASS /newtask and /smol descriptions match source code
Conventions ISSUE cli.md change breaks existing document structure
Changeset SKIP Docs-only PR, no version bump required
Tests N/A No code changes
i18n N/A Docs site, not UI strings
Types N/A No TypeScript
Security N/A Static documentation
Scope PASS Two files, single concern

Findings

🟡 cli.md: New section breaks existing configuration list

File: apps/kilocode-docs/pages/code-with-ai/platforms/cli.md (lines 167-174)

The new ## Slash Commands header is inserted in the middle of the "Configuration is managed through:" list, which:

  1. Removes two existing list items (/connect command and config file path ~/.config/kilo/config.json)
  2. Orphans the remaining kilo auth bullet point after the new section
  3. Breaks the grammatical flow — the sentence "Configuration is managed through:" now has no list following it

Before:

Configuration is managed through:

- `/connect` command for provider setup (interactive)
- Config files directly at `~/.config/kilo/config.json`
- `kilo auth` for credential management

After (problematic):

Configuration is managed through:

## Slash Commands
...
{% /callout %}
- `kilo auth` for credential management

Suggested fix: Keep the configuration list intact. Place the new Slash Commands section either before or after the Configuration block, and restore the removed list items.

✅ using-modes.md: Clean and accurate

The additions to using-modes.md are well-structured. The comparison table for /newtask vs /smol is clear and the descriptions match the source code in webview-ui/src/utils/slash-commands.ts.

CI Status

Check Result
Build Markdoc Site PASS
compile PASS
check-translations PASS
unit-test PASS
test-extension (ubuntu) PASS
test-extension (windows) PASS
test-webview (ubuntu) PASS
test-webview (windows) PASS
build-cli PASS
test-cli PASS
test-jetbrains PASS
Vercel SKIP (auth required, expected for external contributors)

Code Snippets

# apps/kilocode-docs/pages/code-with-ai/platforms/cli.md (structural issue)

 Configuration is managed through:

-- `/connect` command for provider setup (interactive)
-- Config files directly at `~/.config/kilo/config.json`
+## Slash Commands
+
+The CLI's interactive mode supports slash commands for common operations...
+
+{% callout type="tip" %}
+**Confused about /newtask vs /smol in the IDE?**...
+{% /callout %}
 - `kilo auth` for credential management
# apps/kilocode-docs/pages/code-with-ai/agents/using-modes.md (clean)

-2. **Slash command:** Type `/architect`, `/ask`, `/debug`, or `/code` in the chat input
+2. **Slash command:** Type `/architect`, `/ask`, `/debug`, or `/code` in the chat input to switch modes. Type `/newtask` to create a new task, or `/smol` to condense your context window.

+### Understanding /newtask vs /smol
+
+| Command    | Purpose                                               | When to Use                                      |
+| ---------- | ----------------------------------------------------- | ------------------------------------------------ |
+| `/newtask` | Creates a new task with context from the current task | When you want to start something new...          |
+| `/smol`    | Condenses your current context window                 | When your conversation is getting too long...    |

Verdict

COMMENT - The using-modes.md changes are clean and accurate. However, the cli.md change has a structural issue: it breaks the existing configuration list by inserting a new ## Slash Commands section in the middle of it, removing two configuration items (/connect and config file path) and orphaning kilo auth. This was independently flagged by both CodeRabbit and Gemini on the fork analysis. Recommend restructuring cli.md to keep the configuration list intact while adding the slash commands section separately.

@jeremylongshore
Copy link
Copy Markdown
Owner Author

Review Journal: kilocode Kilo-Org#5869

PR: #5869 |
Title: docs: clarify slash commands (/newtask vs /smol) (Kilo-Org#2160) |
Author: @EloiRamos |
Category: docs | Tier: 1 | Size: 20 lines, 2 files | Confidence: 4/5

Multi-AI analysis: Fork PR #5 — CodeRabbit, Gemini, CodeQL, Qodo


Summary

This PR addresses issue #2160 by clarifying the difference between /newtask and /smol slash commands. The using-modes.md change is clean — a well-structured comparison table that matches the source code. The cli.md change has a structural problem: it breaks an existing configuration list by inserting a new section mid-list, removing two items and orphaning a third. Both AI reviewers independently flagged the same issue.

First Impressions

A docs: prefix with two files suggests a cross-cutting documentation improvement. The linked issue (Kilo-Org#2160) is a reasonable user confusion point — /newtask and /smol sound similar but serve different purposes. Expected a straightforward clarification; found one clean change and one that needs structural revision.

What I Looked At

  1. The PR diff — 2 files: using-modes.md (+9 lines, clean) and cli.md (+5/-2, structural issue)
  2. Issue #2160 — User request to clarify command differences
  3. Source codewebview-ui/src/utils/slash-commands.ts:22-34 to verify command descriptions
  4. Fork PR Mirror: docs: clarify slash commands /newtask vs /smol (#5869) #5 — Bot reviews from CodeRabbit and Gemini
  5. All upstream CI checks — 12/12 pass (Vercel skipped, expected)

Analysis

using-modes.md — Clean

The change extends step 2 in the "Four ways to switch modes" list and adds a subsection with a comparison table:

Command Purpose When to Use
/newtask Creates a new task with context from the current task Starting something new while carrying over context
/smol Condenses your current context window Conversation too long, want to summarize

I verified these descriptions against the source code:

// webview-ui/src/utils/slash-commands.ts
{ name: "newtask", description: "Create a new task with context from the current task" },
{ name: "smol", description: "Condenses your current context window" },

Accurate match. The ### Understanding /newtask vs /smol heading also creates a clean anchor for the cross-reference in the cli.md callout.

cli.md — Structural Break

The new ## Slash Commands section is inserted between line 165 ("Configuration is managed through:") and the configuration list items. This creates three problems:

  1. Removed content: The /connect command reference and config file path (~/.config/kilo/config.json) are deleted
  2. Orphaned bullet: kilo auth now sits alone after a callout block, disconnected from its parent list
  3. Broken grammar: "Configuration is managed through:" introduces nothing

The intent is good — adding a cross-reference to the using-modes.md explanation. But the execution needs restructuring: keep the configuration list intact and place the Slash Commands section in its own location.

Verification

All relevant CI checks pass:

Build Markdoc Site     PASS    (directly relevant - docs build)
check-translations     PASS    (directly relevant - no broken strings)
compile                PASS
test-extension         PASS    (ubuntu + windows)
test-webview           PASS    (ubuntu + windows)
unit-test              PASS
build-cli              PASS
test-cli               PASS
test-jetbrains         PASS
Vercel                 SKIP    (auth required for external contributors)

Diagrams

graph LR
    subgraph "using-modes.md ✅"
        A["Step 2: Slash command"] --> B["Extended text<br>/newtask, /smol mentioned"]
        B --> C["NEW: ### Understanding<br>/newtask vs /smol"]
        C --> D["Comparison table"]
    end

    subgraph "cli.md ⚠️"
        E["'Configuration is<br>managed through:'"] --> F["❌ /connect removed"]
        E --> G["❌ config path removed"]
        E --> H["NEW: ## Slash Commands<br>(breaks list)"]
        H --> I["Tip callout<br>(cross-ref to using-modes)"]
        I --> J["⚠️ Orphaned:<br>'kilo auth'"]
    end

    style C fill:#2d6a4f,stroke:#1b4332,color:#d8f3dc
    style D fill:#2d6a4f,stroke:#1b4332,color:#d8f3dc
    style F fill:#9d0208,stroke:#6a040f,color:#fff
    style G fill:#9d0208,stroke:#6a040f,color:#fff
    style H fill:#e9c46a,stroke:#f4a261,color:#264653
    style J fill:#e9c46a,stroke:#f4a261,color:#264653
Loading

Bot Review Synthesis

Bot Verdict Key Finding Useful?
CodeRabbit Comment Flagged orphaned kilo auth bullet and structural break Yes — same finding as manual review
Gemini Comment Flagged both cli.md structure break AND using-modes.md list nesting Yes — caught both files, one was a false positive
Greptile No response Did not comment on this PR Needs investigation
CodeQL N/A No security findings (docs-only) Expected
Qodo Failed "Failed to generate code suggestions" Config issue persists

Bot consensus: CodeRabbit and Gemini both independently identified the cli.md structural issue. Gemini also flagged the ### Understanding /newtask vs /smol header inside a numbered list in using-modes.md as potentially breaking list continuity — this is a valid concern with strict Markdown parsers, though Markdoc may handle it fine. Two-bot consensus on the main finding increases confidence.

Lessons Learned

1. Cross-cutting docs changes need structural awareness. When adding content to existing documentation, check that you're not splitting an in-progress syntactic structure (list, table, code block). The cli.md change is a classic "insert in the wrong spot" issue.

2. Bot agreement strengthens review confidence. When two independent AI reviewers flag the same issue with different framing (CodeRabbit: "orphaned bullet", Gemini: "breaks grammatical flow"), the finding is almost certainly real. This is the first review where bot consensus directly validated manual analysis.

3. Source code verification catches docs drift. Checking slash command descriptions against slash-commands.ts confirmed accuracy. As the codebase evolves, these descriptions could drift — the source code is the ground truth.

4. All CI green doesn't mean all content is correct. Markdoc built successfully despite the structural issue. The build validates syntax, not document coherence. Human/AI review catches what CI cannot.


Review #2 of 75 | Review methodology: AI PR Review Case Studies | Reviewed with GWI + Claude Code

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.

Document various slash commands

2 participants