Repository navigation
chore: updating git-kb .claude files - #36
mateodelnorte wants to merge 1 commit into
Conversation
WalkthroughAdded seven new KB command documentation files that define structured workflows for knowledge base operations: creating documents, starting tasks, logging progress, reviewing criteria, searching, closing tasks, and end-of-session handoff. Also updated terminology from "database-first" to "distributed knowledge base" in existing documentation. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Greptile SummaryAdds comprehensive GitKB command documentation and updates terminology from "database-first" to "distributed" knowledge base. Seven new command files provide detailed workflows for task lifecycle management (create, start, progress, review, close), search, and session handoff. Confidence Score: 5/5
|
| Filename | Overview |
|---|---|
| .claude/commands/kb-close.md | New command added for closing tasks with verification - comprehensive workflow for task completion |
| .claude/commands/kb-create.md | New command for creating KB documents with auto-incrementing slugs and context-aware content generation |
| .claude/commands/kb-handoff.md | New command for session handoff - ensures all work-in-progress is committed and context is fresh |
| .claude/commands/kb-progress.md | New command for logging progress on tasks - quick datestamped updates without full edit cycle |
| .claude/commands/kb-review.md | New command for reviewing tasks against acceptance criteria and codebase state |
| .claude/commands/kb-search.md | New command for searching the knowledge base with formatted results and suggested actions |
| .claude/commands/kb-start.md | New command for starting work on tasks - loads context, sets status, and prepares workspace |
| .claude/skills/gitkb/SKILL.md | Updated GitKB description from "database-first" to "distributed" knowledge base |
| .kb/AGENTS.md | Updated GitKB description from "database-first" to "distributed" in two locations |
Last reviewed commit: 2144603
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.claude/commands/kb-handoff.md:
- Around line 74-89: Add a language specifier to the fenced code block starting
with the triple backticks that contains "## Session Handoff" (e.g., change the
opening ``` to ```text or ```md) so the code block passes MD040; update the
opening fence only and leave the block contents (the headings and list items
such as "### What Was Done", "### Current State", and "### For Next Session")
unchanged.
- Around line 1-17: The file lacks a top-level heading after the YAML
frontmatter (MD041); open .claude/commands/kb-handoff.md and add an H1
immediately after the closing frontmatter marker (the trailing '---') — e.g., a
concise heading like "KB Handoff" or "End-of-session Handoff" — so the document
contains a top-level heading directly following the frontmatter to satisfy
markdownlint.
| --- | ||
| allowed-tools: | ||
| - mcp__gitkb__kb_context | ||
| - mcp__gitkb__kb_show | ||
| - mcp__gitkb__kb_list | ||
| - mcp__gitkb__kb_checkout | ||
| - mcp__gitkb__kb_status | ||
| - mcp__gitkb__kb_diff | ||
| - mcp__gitkb__kb_commit | ||
| - mcp__gitkb__kb_set | ||
| - Bash(git kb:*) | ||
| - Read | ||
| - Edit | ||
| description: End-of-session handoff — update context, log progress, commit pending changes | ||
| --- | ||
|
|
||
| Prepare for session end or agent handoff. Ensures all work-in-progress is captured and context documents are fresh for the next session. |
There was a problem hiding this comment.
🧹 Nitpick | 🔵 Trivial
Add a top-level heading after frontmatter (MD041).
Markdownlint flags the lack of an H1. If frontmatter is required, add a heading immediately after it (or explicitly configure MD041 to ignore frontmatter-only files).
📝 Suggested fix
---
allowed-tools:
- mcp__gitkb__kb_context
@@
description: End-of-session handoff — update context, log progress, commit pending changes
---
+ # KB Handoff
+
Prepare for session end or agent handoff. Ensures all work-in-progress is captured and context documents are fresh for the next session.🧰 Tools
🪛 markdownlint-cli2 (0.21.0)
[warning] 17-17: First line in a file should be a top-level heading
(MD041, first-line-heading, first-line-h1)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In @.claude/commands/kb-handoff.md around lines 1 - 17, The file lacks a
top-level heading after the YAML frontmatter (MD041); open
.claude/commands/kb-handoff.md and add an H1 immediately after the closing
frontmatter marker (the trailing '---') — e.g., a concise heading like "KB
Handoff" or "End-of-session Handoff" — so the document contains a top-level
heading directly following the frontmatter to satisfy markdownlint.
| ``` | ||
| ## Session Handoff | ||
|
|
||
| ### What Was Done | ||
| - [Summary of work completed] | ||
|
|
||
| ### Current State | ||
| - Active tasks: [list with status] | ||
| - Pending work: [what's next] | ||
| - Blockers: [if any] | ||
|
|
||
| ### For Next Session | ||
| - Start with: /kb-context to load project state | ||
| - Then: /kb-start <suggested-task> to resume work | ||
| - Watch out for: [any gotchas or context the next session needs] | ||
| ``` |
There was a problem hiding this comment.
Specify a language for the fenced block (MD040).
The code block lacks a language specifier.
📝 Suggested fix
-```
+```text
## Session Handoff
### What Was Done
@@
- Start with: /kb-context to load project state
- Then: /kb-start <suggested-task> to resume work
- Watch out for: [any gotchas or context the next session needs]</details>
<!-- suggestion_start -->
<details>
<summary>📝 Committable suggestion</summary>
> ‼️ **IMPORTANT**
> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
```suggestion
🧰 Tools
🪛 markdownlint-cli2 (0.21.0)
[warning] 74-74: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In @.claude/commands/kb-handoff.md around lines 74 - 89, Add a language
specifier to the fenced code block starting with the triple backticks that
contains "## Session Handoff" (e.g., change the opening ``` to ```text or ```md)
so the code block passes MD040; update the opening fence only and leave the
block contents (the headings and list items such as "### What Was Done", "###
Current State", and "### For Next Session") unchanged.
chore: updating git-kb .claude files
Summary by CodeRabbit
New Features
Documentation