kartman-dev 1.0.0-dev.0 → 1.0.0-dev.1
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/package.json +1 -1
- package/dist/claude-config/worktrees/polymorphic-cooking-goose/.claude/.claude-plugin/plugin.json +0 -11
- package/dist/claude-config/worktrees/polymorphic-cooking-goose/.claude/.mcp.json +0 -23
- package/dist/claude-config/worktrees/polymorphic-cooking-goose/.claude/settings.json +0 -13
- package/dist/claude-config/worktrees/polymorphic-cooking-goose/.claude/skills/doc-drift-check/SKILL.md +0 -140
- package/dist/claude-config/worktrees/polymorphic-cooking-goose/.claude/skills/doc-drift-check/references/severity-guide.md +0 -174
- package/dist/claude-config/worktrees/valiant-sparking-hare/.claude/.claude-plugin/plugin.json +0 -11
- package/dist/claude-config/worktrees/valiant-sparking-hare/.claude/.mcp.json +0 -23
- package/dist/claude-config/worktrees/valiant-sparking-hare/.claude/commands/linear-issue.md +0 -59
- package/dist/claude-config/worktrees/valiant-sparking-hare/.claude/commands/linear-project.md +0 -70
- package/dist/claude-config/worktrees/valiant-sparking-hare/.claude/settings.json +0 -13
- package/dist/claude-config/worktrees/valiant-sparking-hare/.claude/skills/doc-drift-check/SKILL.md +0 -140
- package/dist/claude-config/worktrees/valiant-sparking-hare/.claude/skills/doc-drift-check/references/severity-guide.md +0 -174
- package/dist/claude-config/worktrees/valiant-sparking-hare/.github/workflows/claude-code-review.yml +0 -32
- package/dist/claude-config/worktrees/valiant-sparking-hare/.github/workflows/claude.yml +0 -37
- package/dist/claude-config/worktrees/valiant-sparking-hare/.nvmrc +0 -1
- package/dist/claude-config/worktrees/valiant-sparking-hare/.prettierignore +0 -11
package/package.json
CHANGED
package/dist/claude-config/worktrees/polymorphic-cooking-goose/.claude/.claude-plugin/plugin.json
DELETED
|
@@ -1,11 +0,0 @@
|
|
|
1
|
-
{
|
|
2
|
-
"name": "claude-linear-agent",
|
|
3
|
-
"version": "0.1.0",
|
|
4
|
-
"description": "Skills and MCP integrations for the Linear AI agent — Linear, GitHub, Notion, and Context7",
|
|
5
|
-
"author": {
|
|
6
|
-
"name": "Matt Aliev"
|
|
7
|
-
},
|
|
8
|
-
"repository": "https://github.com/mattaliev/claude-linear-agent",
|
|
9
|
-
"license": "MIT",
|
|
10
|
-
"keywords": ["linear", "agent", "mcp", "notion", "github", "context7"]
|
|
11
|
-
}
|
|
@@ -1,23 +0,0 @@
|
|
|
1
|
-
{
|
|
2
|
-
"mcpServers": {
|
|
3
|
-
"linear": {
|
|
4
|
-
"type": "http",
|
|
5
|
-
"url": "https://mcp.linear.app/mcp",
|
|
6
|
-
"headers": {
|
|
7
|
-
"Authorization": "Bearer ${LINEAR_ACCESS_TOKEN}"
|
|
8
|
-
}
|
|
9
|
-
},
|
|
10
|
-
"notion": {
|
|
11
|
-
"type": "http",
|
|
12
|
-
"url": "https://mcp.notion.com/mcp",
|
|
13
|
-
"headers": {
|
|
14
|
-
"Authorization": "Bearer ${NOTION_API_KEY}",
|
|
15
|
-
"Notion-Version": "2025-09-03"
|
|
16
|
-
}
|
|
17
|
-
},
|
|
18
|
-
"context7": {
|
|
19
|
-
"command": "npx",
|
|
20
|
-
"args": ["-y", "@upstash/context7-mcp", "--api-key", "${CONTEXT7_API_KEY}"]
|
|
21
|
-
}
|
|
22
|
-
}
|
|
23
|
-
}
|
|
@@ -1,13 +0,0 @@
|
|
|
1
|
-
{
|
|
2
|
-
"enabledPlugins": {
|
|
3
|
-
"agent-sdk-dev@claude-plugins-official": true,
|
|
4
|
-
"plugin-dev@claude-plugins-official": true,
|
|
5
|
-
"code-simplifier@claude-plugins-official": true,
|
|
6
|
-
"code-review@claude-plugins-official": true,
|
|
7
|
-
"pr-review-toolkit@claude-plugins-official": true
|
|
8
|
-
},
|
|
9
|
-
"env": {
|
|
10
|
-
"ENABLE_TOOL_SEARCH": "true",
|
|
11
|
-
"CLAUDE_CODE_EXPERIMENTAL_AGENT_TEAMS": "1"
|
|
12
|
-
}
|
|
13
|
-
}
|
|
@@ -1,140 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: Doc Drift Check
|
|
3
|
-
description: This skill should be used when the user asks to "check docs for drift", "compare code changes against documentation", "review docs after PR", "find outdated documentation", "check if docs match the code", "verify documentation is up to date", or after completing a pull request review to verify documentation accuracy. Analyzes code changes against project docs, CLAUDE.md, database schema, Notion module pages, and Notion processes to catch mismatches.
|
|
4
|
-
---
|
|
5
|
-
|
|
6
|
-
# Doc Drift Check
|
|
7
|
-
|
|
8
|
-
Analyze code changes from a PR or issue against all project documentation — both in the repository and in Notion — to catch mismatches that would lead to stale docs. Write a severity-categorized comment to the Linear issue only when meaningful drift is found.
|
|
9
|
-
|
|
10
|
-
## When to Use
|
|
11
|
-
|
|
12
|
-
Invoke manually after reviewing a pull request or completing an issue review. Skip for trivial changes (typo fixes, formatting, dependency bumps) where doc drift is impossible.
|
|
13
|
-
|
|
14
|
-
## Workflow
|
|
15
|
-
|
|
16
|
-
### 1. Gather Context
|
|
17
|
-
|
|
18
|
-
Collect three inputs before analysis:
|
|
19
|
-
|
|
20
|
-
**Code changes** — Get the diff for the issue's PR:
|
|
21
|
-
- Use GitHub MCP to read the PR diff
|
|
22
|
-
- Note which files changed and the nature of changes (new files, modified APIs, removed features, config changes, model changes, service changes)
|
|
23
|
-
|
|
24
|
-
**Linear issue** — Read the issue description and metadata:
|
|
25
|
-
- Issue title, description, and acceptance criteria
|
|
26
|
-
- Team identifier (determines which Notion workspace to check):
|
|
27
|
-
- `KPL-` → Platform
|
|
28
|
-
- `KPE-` → Personal
|
|
29
|
-
- `KBU-` → Business
|
|
30
|
-
- Links to Notion pages or external docs in the description
|
|
31
|
-
|
|
32
|
-
**Repository documentation** — Read relevant local docs:
|
|
33
|
-
- `docs/` directory — scan for files related to changed code areas
|
|
34
|
-
- `CLAUDE.md` — project overview, architecture, env vars, structure
|
|
35
|
-
- Database schema files — check if schema changes are reflected
|
|
36
|
-
|
|
37
|
-
### 2. Identify Relevant Notion Documentation
|
|
38
|
-
|
|
39
|
-
Based on the team identifier from the Linear issue, fetch the corresponding Product Development page in Notion. These pages contain module documentation, processes, and feature documents for each team.
|
|
40
|
-
|
|
41
|
-
**Team → Notion Product Development page mapping:**
|
|
42
|
-
|
|
43
|
-
| Team Prefix | Team | Notion Page ID |
|
|
44
|
-
|---|---|---|
|
|
45
|
-
| `KPL-` | Platform | `24c80383e1518012a3c2c410588782d5` |
|
|
46
|
-
| `KPE-` | Personal | `23a80383e15180ae91bae32b407ebaf9` |
|
|
47
|
-
| `KBU-` | Business | `24c80383e15180e19e78cfca4b49fe66` |
|
|
48
|
-
|
|
49
|
-
**Scan the Product Development page** for the appropriate team:
|
|
50
|
-
1. Fetch the page and scan its child pages
|
|
51
|
-
2. Identify module pages that relate to the changed code area
|
|
52
|
-
3. Read those module pages — they describe models, services, and APIs for each module
|
|
53
|
-
|
|
54
|
-
**Check the Processes database** (`252cc951ac514d86ad3b6d9fc689409c`):
|
|
55
|
-
1. The Processes database has a "Модули" relation linking processes to modules
|
|
56
|
-
2. If code changes affect a module, find linked processes
|
|
57
|
-
3. Check if process descriptions still match the implementation
|
|
58
|
-
|
|
59
|
-
### 3. Analyze for Mismatches
|
|
60
|
-
|
|
61
|
-
Compare code changes against each relevant documentation source. Look for:
|
|
62
|
-
|
|
63
|
-
- **Model drift** — Code adds/removes/renames model fields, but module documentation in Notion still describes the old schema
|
|
64
|
-
- **Service drift** — New services added or service logic changed, but module page doesn't list them
|
|
65
|
-
- **API drift** — Endpoints added/changed/removed, but docs still reference old API
|
|
66
|
-
- **Process drift** — Implementation changes how a process works, but the process page in Notion doesn't reflect it
|
|
67
|
-
- **Missing documentation** — New module, feature, or config with no corresponding docs
|
|
68
|
-
- **Stale references** — Docs reference removed code, old file paths, or deprecated models
|
|
69
|
-
- **Issue description mismatch** — What was implemented differs from what the issue describes
|
|
70
|
-
- **Structural drift** — File/directory structure changed but project structure docs not updated
|
|
71
|
-
- **Environment drift** — New env vars, config values, or dependencies not documented
|
|
72
|
-
|
|
73
|
-
For detailed severity classification examples, consult `references/severity-guide.md`.
|
|
74
|
-
|
|
75
|
-
### 4. Decision Gate
|
|
76
|
-
|
|
77
|
-
After analysis, decide whether to comment:
|
|
78
|
-
|
|
79
|
-
- **No mismatches found** → Do nothing. Do not write a "no issues found" comment.
|
|
80
|
-
- **Only trivial mismatches** (e.g., a slightly outdated example that doesn't mislead) → Skip.
|
|
81
|
-
- **Meaningful mismatches** → Proceed to write comment.
|
|
82
|
-
|
|
83
|
-
The bar for commenting: would someone reading the docs be misled or confused by the current state? If yes, comment. If not, skip.
|
|
84
|
-
|
|
85
|
-
### 5. Write Linear Comment
|
|
86
|
-
|
|
87
|
-
Post a comment on the Linear issue using this format:
|
|
88
|
-
|
|
89
|
-
```
|
|
90
|
-
📋 Doc Drift Report
|
|
91
|
-
|
|
92
|
-
[1-2 sentence summary of what was found]
|
|
93
|
-
|
|
94
|
-
🔴 Critical
|
|
95
|
-
- [Notion: Module "Payments" → Models section] — Code adds TransferFee model but module page doesn't list it
|
|
96
|
-
- [File: `CLAUDE.md` env vars table] — New SYNC_API_KEY env var required but not listed
|
|
97
|
-
|
|
98
|
-
🟡 Moderate
|
|
99
|
-
- [Notion: Process "Customer Onboarding"] — Step 3 now calls ProviderService instead of direct API, process diagram outdated
|
|
100
|
-
- [File: `docs/architecture.md`] — New endpoint POST /api/v2/transfers not documented
|
|
101
|
-
|
|
102
|
-
🟢 Light
|
|
103
|
-
- [Notion: Module "Crypto" → Services section] — CryptoSigningService renamed to AccessKeyService in code
|
|
104
|
-
|
|
105
|
-
---
|
|
106
|
-
Suggested actions:
|
|
107
|
-
- Update Notion module page "Payments" with new TransferFee model
|
|
108
|
-
- Add SYNC_API_KEY to CLAUDE.md environment variables table
|
|
109
|
-
- Update process diagram in "Customer Onboarding" process page
|
|
110
|
-
```
|
|
111
|
-
|
|
112
|
-
**Formatting rules:**
|
|
113
|
-
- Use severity emojis: 🔴 Critical, 🟡 Moderate, 🟢 Light
|
|
114
|
-
- Prefix with source: `[Notion: ...]` or `[File: ...]`
|
|
115
|
-
- Include specific page/section references
|
|
116
|
-
- Include a "Suggested actions" section with concrete fixes
|
|
117
|
-
- Omit empty severity categories
|
|
118
|
-
|
|
119
|
-
## Severity Classification Quick Reference
|
|
120
|
-
|
|
121
|
-
| Severity | Description |
|
|
122
|
-
|---|---|
|
|
123
|
-
| 🔴 Critical | Model/schema changes undocumented, API changes, env var changes, breaking behavior, architecture shifts — someone would build against wrong assumptions |
|
|
124
|
-
| 🟡 Moderate | New services undocumented, process flow changes, new features without docs, workflow changes — someone would miss important capabilities |
|
|
125
|
-
| 🟢 Light | Renamed entities, outdated examples, minor wording, service method renames — someone might be briefly confused |
|
|
126
|
-
|
|
127
|
-
For detailed examples and edge cases, see `references/severity-guide.md`.
|
|
128
|
-
|
|
129
|
-
## Tools Used
|
|
130
|
-
|
|
131
|
-
- **GitHub MCP** — read PR diffs and changed files
|
|
132
|
-
- **Linear MCP** — read issue description and team, post comments
|
|
133
|
-
- **Notion MCP** — fetch Product Development pages, module pages, process pages
|
|
134
|
-
- **Read, Grep, Glob** — scan local documentation files in the repository
|
|
135
|
-
|
|
136
|
-
## Additional Resources
|
|
137
|
-
|
|
138
|
-
### Reference Files
|
|
139
|
-
|
|
140
|
-
- **`references/severity-guide.md`** — Detailed severity classification with examples, edge cases, and per-source guidance
|
|
@@ -1,174 +0,0 @@
|
|
|
1
|
-
# Severity Classification Guide
|
|
2
|
-
|
|
3
|
-
Detailed guide for categorizing doc drift findings by severity level. Use this when analyzing mismatches between code changes and documentation.
|
|
4
|
-
|
|
5
|
-
## 🔴 Critical — Would Cause Wrong Assumptions
|
|
6
|
-
|
|
7
|
-
These mismatches would cause someone to build against incorrect assumptions or make wrong decisions.
|
|
8
|
-
|
|
9
|
-
### Model/Schema Changes
|
|
10
|
-
|
|
11
|
-
- Code adds a new model but the Notion module page doesn't list it
|
|
12
|
-
- Code removes or renames a model field, but the module documentation still references the old field
|
|
13
|
-
- Relationship between models changed (e.g., ForeignKey → ManyToMany) but not reflected in docs
|
|
14
|
-
- New required field added to a model but not documented in the module's models section
|
|
15
|
-
|
|
16
|
-
**Example:**
|
|
17
|
-
> Code adds `AccessKey` model to `andgate_crypto` module, but the Crypto module page in Notion under Platform's Product Development doesn't mention it.
|
|
18
|
-
|
|
19
|
-
### API/Endpoint Changes
|
|
20
|
-
|
|
21
|
-
- New endpoint added without documentation
|
|
22
|
-
- Endpoint removed but still listed in docs
|
|
23
|
-
- Request/response format changed but docs show old format
|
|
24
|
-
- Authentication requirements changed
|
|
25
|
-
|
|
26
|
-
### Environment & Configuration
|
|
27
|
-
|
|
28
|
-
- New environment variable required but not in CLAUDE.md or deployment docs
|
|
29
|
-
- Configuration option removed but still documented
|
|
30
|
-
- Default value changed for existing config
|
|
31
|
-
|
|
32
|
-
### Architecture Changes
|
|
33
|
-
|
|
34
|
-
- New module/app created without corresponding Notion documentation
|
|
35
|
-
- Service dependencies changed (module A now depends on module B, but docs don't show this)
|
|
36
|
-
- Database migration adds/removes tables not reflected in schema docs
|
|
37
|
-
|
|
38
|
-
---
|
|
39
|
-
|
|
40
|
-
## 🟡 Moderate — Would Miss Important Capabilities
|
|
41
|
-
|
|
42
|
-
These mismatches would cause someone to miss important features or follow outdated procedures.
|
|
43
|
-
|
|
44
|
-
### Service Changes
|
|
45
|
-
|
|
46
|
-
- New service class added to a module but module page doesn't list it in the services section
|
|
47
|
-
- Service method signature changed (different params) but process docs reference old signature
|
|
48
|
-
- Service now calls a different external provider but process diagrams show old flow
|
|
49
|
-
|
|
50
|
-
**Example:**
|
|
51
|
-
> Code adds `AccessKeyService` to handle crypto signing, but the Crypto module page doesn't list it under Services.
|
|
52
|
-
|
|
53
|
-
### Process Flow Changes
|
|
54
|
-
|
|
55
|
-
- Steps in a business process reordered or modified
|
|
56
|
-
- New step added to a process (e.g., new validation step in onboarding)
|
|
57
|
-
- Conditional logic changed (e.g., different handling for B2B vs B2C)
|
|
58
|
-
- Process now has prerequisites that aren't documented
|
|
59
|
-
|
|
60
|
-
**Example:**
|
|
61
|
-
> Customer onboarding now requires provider-specific KYC before account creation, but the process page still shows a single KYC step.
|
|
62
|
-
|
|
63
|
-
### Feature Documentation Gaps
|
|
64
|
-
|
|
65
|
-
- New feature implemented but no feature document exists
|
|
66
|
-
- Feature behavior changed significantly but feature docs describe old behavior
|
|
67
|
-
- Feature flag added without documentation in relevant places
|
|
68
|
-
|
|
69
|
-
### Workflow Changes
|
|
70
|
-
|
|
71
|
-
- Webhook handling changed but webhook documentation outdated
|
|
72
|
-
- Background task behavior modified but not reflected in process docs
|
|
73
|
-
- Error handling strategy changed for a module
|
|
74
|
-
|
|
75
|
-
---
|
|
76
|
-
|
|
77
|
-
## 🟢 Light — Brief Confusion Only
|
|
78
|
-
|
|
79
|
-
These mismatches would cause someone brief confusion but wouldn't lead to incorrect work.
|
|
80
|
-
|
|
81
|
-
### Naming/Renaming
|
|
82
|
-
|
|
83
|
-
- Service or utility class renamed but docs use old name
|
|
84
|
-
- File moved to different directory but docs reference old path
|
|
85
|
-
- Variable or constant renamed
|
|
86
|
-
|
|
87
|
-
**Example:**
|
|
88
|
-
> `CryptoSigningService` renamed to `AccessKeyService` in code but Notion module page still says `CryptoSigningService`.
|
|
89
|
-
|
|
90
|
-
### Outdated Examples
|
|
91
|
-
|
|
92
|
-
- Code example in docs uses old import path
|
|
93
|
-
- Example shows deprecated API usage but functionally equivalent
|
|
94
|
-
- Test examples reference old fixtures
|
|
95
|
-
|
|
96
|
-
### Minor Wording
|
|
97
|
-
|
|
98
|
-
- Documentation says "will be implemented" for already-implemented features
|
|
99
|
-
- Status markers (TODO, WIP) left in docs for completed work
|
|
100
|
-
- Comments in code reference old Notion page names
|
|
101
|
-
|
|
102
|
-
---
|
|
103
|
-
|
|
104
|
-
## Per-Source Analysis Guide
|
|
105
|
-
|
|
106
|
-
### Notion Module Pages
|
|
107
|
-
|
|
108
|
-
Module pages describe models and services for each module. When checking:
|
|
109
|
-
|
|
110
|
-
1. **Models section** — Compare listed models against actual Django models in the code
|
|
111
|
-
2. **Services section** — Compare listed services against actual service classes
|
|
112
|
-
3. **API section** — Compare listed endpoints against actual URL configs
|
|
113
|
-
4. **Shared models** — Verify cross-module dependencies are still accurate
|
|
114
|
-
5. **Dependency graph** — Check if module interdependencies changed
|
|
115
|
-
|
|
116
|
-
### Notion Processes Database
|
|
117
|
-
|
|
118
|
-
Process pages describe business workflows. When checking:
|
|
119
|
-
|
|
120
|
-
1. **Process steps** — Do the documented steps match the actual code flow?
|
|
121
|
-
2. **Prerequisites** — Are process prerequisites still accurate?
|
|
122
|
-
3. **Provider interactions** — Do provider-specific flows match current implementation?
|
|
123
|
-
4. **Status transitions** — Do documented state machines match code?
|
|
124
|
-
5. **Related modules** — Is the "Модули" relation still correct?
|
|
125
|
-
|
|
126
|
-
### Repository docs/ Directory
|
|
127
|
-
|
|
128
|
-
Local documentation files. When checking:
|
|
129
|
-
|
|
130
|
-
1. **Architecture docs** — Do they reflect current module structure?
|
|
131
|
-
2. **API docs** — Do endpoint descriptions match implementations?
|
|
132
|
-
3. **Setup/deployment docs** — Are instructions still accurate?
|
|
133
|
-
|
|
134
|
-
### CLAUDE.md
|
|
135
|
-
|
|
136
|
-
Project-level documentation. When checking:
|
|
137
|
-
|
|
138
|
-
1. **Environment variables table** — All required vars listed?
|
|
139
|
-
2. **Project structure** — Does the tree match actual directory layout?
|
|
140
|
-
3. **Technology stack** — Any new dependencies or tools?
|
|
141
|
-
4. **Key documentation links** — Do referenced docs still exist?
|
|
142
|
-
|
|
143
|
-
### Database Schema
|
|
144
|
-
|
|
145
|
-
Schema documentation. When checking:
|
|
146
|
-
|
|
147
|
-
1. **Table definitions** — Do they match current migrations?
|
|
148
|
-
2. **Relationships** — Are FK/M2M relations documented correctly?
|
|
149
|
-
3. **Indexes** — Any new indexes not mentioned in schema docs?
|
|
150
|
-
4. **New tables** — Any new migrations creating tables not in schema docs?
|
|
151
|
-
|
|
152
|
-
---
|
|
153
|
-
|
|
154
|
-
## Edge Cases
|
|
155
|
-
|
|
156
|
-
### When NOT to Flag
|
|
157
|
-
|
|
158
|
-
- Code comments that are slightly outdated (not worth a Linear comment)
|
|
159
|
-
- Test file documentation being slightly behind (tests are self-documenting)
|
|
160
|
-
- Third-party library version bumps with no behavior change
|
|
161
|
-
- Formatting-only changes to documentation files
|
|
162
|
-
- Internal refactoring that doesn't change public interfaces or behavior
|
|
163
|
-
|
|
164
|
-
### When to Escalate to Critical
|
|
165
|
-
|
|
166
|
-
Even if a change seems small, escalate if:
|
|
167
|
-
- It affects an external-facing API that developers build against
|
|
168
|
-
- It changes model fields that other services query
|
|
169
|
-
- It modifies environment variables needed for deployment
|
|
170
|
-
- It alters the behavior described in a process that operations follows
|
|
171
|
-
|
|
172
|
-
### Multiple Small Issues
|
|
173
|
-
|
|
174
|
-
If there are many Light issues (5+) concentrated in one documentation area, consider grouping them and elevating to a single Moderate finding with a suggestion to do a documentation review for that area.
|
package/dist/claude-config/worktrees/valiant-sparking-hare/.claude/.claude-plugin/plugin.json
DELETED
|
@@ -1,11 +0,0 @@
|
|
|
1
|
-
{
|
|
2
|
-
"name": "kartman",
|
|
3
|
-
"version": "0.1.0",
|
|
4
|
-
"description": "Skills and MCP integrations for the Linear AI agent — Linear, GitHub, Notion, and Context7",
|
|
5
|
-
"author": {
|
|
6
|
-
"name": "Matt Aliev"
|
|
7
|
-
},
|
|
8
|
-
"repository": "https://github.com/mattaliev/kartman",
|
|
9
|
-
"license": "MIT",
|
|
10
|
-
"keywords": ["linear", "agent", "mcp", "notion", "github", "context7"]
|
|
11
|
-
}
|
|
@@ -1,23 +0,0 @@
|
|
|
1
|
-
{
|
|
2
|
-
"mcpServers": {
|
|
3
|
-
"linear": {
|
|
4
|
-
"type": "http",
|
|
5
|
-
"url": "https://mcp.linear.app/mcp",
|
|
6
|
-
"headers": {
|
|
7
|
-
"Authorization": "Bearer ${LINEAR_ACCESS_TOKEN}"
|
|
8
|
-
}
|
|
9
|
-
},
|
|
10
|
-
"notion": {
|
|
11
|
-
"type": "http",
|
|
12
|
-
"url": "https://mcp.notion.com/mcp",
|
|
13
|
-
"headers": {
|
|
14
|
-
"Authorization": "Bearer ${NOTION_API_KEY}",
|
|
15
|
-
"Notion-Version": "2025-09-03"
|
|
16
|
-
}
|
|
17
|
-
},
|
|
18
|
-
"context7": {
|
|
19
|
-
"command": "npx",
|
|
20
|
-
"args": ["-y", "@upstash/context7-mcp", "--api-key", "${CONTEXT7_API_KEY}"]
|
|
21
|
-
}
|
|
22
|
-
}
|
|
23
|
-
}
|
|
@@ -1,59 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: linear-issue
|
|
3
|
-
description: Load a Linear issue into context. Provide an issue ID or derive it from the current branch name.
|
|
4
|
-
argument-hint: "[ISSUE-ID] [--branch]"
|
|
5
|
-
---
|
|
6
|
-
|
|
7
|
-
# Load Linear Issue
|
|
8
|
-
|
|
9
|
-
Load a Linear issue into context to work on it.
|
|
10
|
-
|
|
11
|
-
**User arguments:** $ARGUMENTS
|
|
12
|
-
|
|
13
|
-
## Step 1: Parse Arguments
|
|
14
|
-
|
|
15
|
-
Parse the arguments string above:
|
|
16
|
-
- If it contains an issue identifier (e.g. `KPE-143`, `KR-1032` — format: 2-4 uppercase letters, dash, number), use that as the issue ID
|
|
17
|
-
- If it contains `--branch`, the user wants a branch created for this issue
|
|
18
|
-
- If no issue identifier is provided, derive it from the current git branch name:
|
|
19
|
-
1. Run `git branch --show-current` to get the current branch
|
|
20
|
-
2. Extract the issue identifier from the branch name. Common patterns:
|
|
21
|
-
- `{prefix}-{TEAM}-{number}-{slug}` → extract `{TEAM}-{number}` (e.g. `MAT-KPL-56-add-auth` → `KPL-56`)
|
|
22
|
-
- `feature/{TEAM}-{number}_{slug}` → extract `{TEAM}-{number}`
|
|
23
|
-
- `{TEAM}-{number}-{slug}` → extract `{TEAM}-{number}`
|
|
24
|
-
3. If no identifier can be extracted, ask the user for the issue ID
|
|
25
|
-
|
|
26
|
-
## Step 2: Fetch the Issue from Linear
|
|
27
|
-
|
|
28
|
-
Use the Linear MCP to fetch the issue details:
|
|
29
|
-
1. Search for the issue by its identifier
|
|
30
|
-
2. Load the full issue including: title, description, state, priority, assignee, labels, comments, and parent issue/project info
|
|
31
|
-
3. If the issue has a project, note the project name and ID for context
|
|
32
|
-
|
|
33
|
-
## Step 3: Display Issue Context
|
|
34
|
-
|
|
35
|
-
Present the issue information clearly:
|
|
36
|
-
- **Identifier and Title**
|
|
37
|
-
- **Status** and **Priority**
|
|
38
|
-
- **Description** (full text)
|
|
39
|
-
- **Labels** and **Assignee**
|
|
40
|
-
- **Project** (if any)
|
|
41
|
-
- **Key comments** (especially any progress checkpoints from previous sessions)
|
|
42
|
-
|
|
43
|
-
## Step 4: Branch Creation (only if --branch was passed)
|
|
44
|
-
|
|
45
|
-
If `--branch` was in the arguments:
|
|
46
|
-
|
|
47
|
-
1. Read the repository's `CLAUDE.md` and `docs/git-conventions.md` (if it exists) to understand branching conventions
|
|
48
|
-
2. Also check recent branches with `git branch -a` to understand the naming pattern used in this repo
|
|
49
|
-
3. Determine the correct branch name following the repo's conventions. If no convention is found, default to: `feature/{issue-identifier}_{short-slug}` where `{short-slug}` is a kebab-case summary of the issue title (keep it concise, 2-4 words)
|
|
50
|
-
4. Check if a branch for this issue already exists: `git branch -a | grep -i {issue-identifier}`
|
|
51
|
-
- If a branch already exists, check it out instead of creating a new one
|
|
52
|
-
- Tell the user you found an existing branch and checked it out
|
|
53
|
-
5. If no existing branch, create and checkout the new branch from the current branch: `git checkout -b {branch-name}`
|
|
54
|
-
|
|
55
|
-
## Important Notes
|
|
56
|
-
|
|
57
|
-
- This command is used across multiple repositories. Always check the current repository's `CLAUDE.md` and recent branches/PRs to follow its specific conventions.
|
|
58
|
-
- If the issue has checkpoint comments from previous sessions, highlight the most recent one — it contains the continuation point.
|
|
59
|
-
- Keep the output concise but complete. The user needs enough context to start working on the issue.
|
package/dist/claude-config/worktrees/valiant-sparking-hare/.claude/commands/linear-project.md
DELETED
|
@@ -1,70 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: linear-project
|
|
3
|
-
description: Load a Linear project into context. Provide a project name or derive it from the current branch's issue.
|
|
4
|
-
argument-hint: "[PROJECT-NAME] [--branch]"
|
|
5
|
-
---
|
|
6
|
-
|
|
7
|
-
# Load Linear Project
|
|
8
|
-
|
|
9
|
-
Load a Linear project into context to work on it.
|
|
10
|
-
|
|
11
|
-
**User arguments:** $ARGUMENTS
|
|
12
|
-
|
|
13
|
-
## Step 1: Parse Arguments
|
|
14
|
-
|
|
15
|
-
Parse the arguments string above:
|
|
16
|
-
- If it contains a project name (any text that isn't `--branch`), use that to search for the project in Linear
|
|
17
|
-
- If it contains `--branch`, the user wants a feature branch created for this project
|
|
18
|
-
- If no project name is provided, derive it from the current branch:
|
|
19
|
-
1. Run `git branch --show-current` to get the current branch
|
|
20
|
-
2. Extract the issue identifier from the branch name. Common patterns:
|
|
21
|
-
- `{prefix}-{TEAM}-{number}-{slug}` → extract `{TEAM}-{number}` (e.g. `MAT-KPL-56-add-auth` → `KPL-56`)
|
|
22
|
-
- `feature/{TEAM}-{number}_{slug}` → extract `{TEAM}-{number}`
|
|
23
|
-
- `{TEAM}-{number}-{slug}` → extract `{TEAM}-{number}`
|
|
24
|
-
3. Use the Linear MCP to fetch that issue and find which project it belongs to
|
|
25
|
-
4. If the issue doesn't belong to a project, or no identifier can be extracted, ask the user for the project name
|
|
26
|
-
|
|
27
|
-
## Step 2: Fetch the Project from Linear
|
|
28
|
-
|
|
29
|
-
Use the Linear MCP to load the full project details:
|
|
30
|
-
1. Project name, description, status, lead, and members
|
|
31
|
-
2. All issues in the project — with their identifiers, titles, statuses, priorities, and assignees
|
|
32
|
-
3. Any project updates or milestones
|
|
33
|
-
|
|
34
|
-
## Step 3: Display Project Context
|
|
35
|
-
|
|
36
|
-
Present the project information clearly:
|
|
37
|
-
|
|
38
|
-
- **Project Name** and **Status**
|
|
39
|
-
- **Description** (full text)
|
|
40
|
-
- **Lead** and **Members**
|
|
41
|
-
- **Progress** — count of issues by status (e.g. 5 Done, 3 In Progress, 8 Todo)
|
|
42
|
-
|
|
43
|
-
### Issues Breakdown
|
|
44
|
-
|
|
45
|
-
List all project issues grouped by status, showing:
|
|
46
|
-
- Identifier, title, priority, assignee
|
|
47
|
-
- Highlight any blocked issues
|
|
48
|
-
|
|
49
|
-
### Recent Updates
|
|
50
|
-
|
|
51
|
-
Show any recent project updates or milestone comments.
|
|
52
|
-
|
|
53
|
-
## Step 4: Branch Creation (only if --branch was passed)
|
|
54
|
-
|
|
55
|
-
If `--branch` was in the arguments:
|
|
56
|
-
|
|
57
|
-
1. Read the repository's `CLAUDE.md` and `docs/git-conventions.md` (if it exists) to understand branching conventions for feature/project branches
|
|
58
|
-
2. Also check recent branches with `git branch -a` to understand the naming pattern used in this repo
|
|
59
|
-
3. Determine the correct branch name following the repo's conventions. If no convention is found, default to: `feature/{short-project-slug}` where `{short-project-slug}` is a kebab-case version of the project name (e.g. "Модуль Payments" → `feature/payments`)
|
|
60
|
-
4. Check if a branch for this project already exists: look for branches that match the project theme in `git branch -a`
|
|
61
|
-
- If a matching branch already exists, check it out instead of creating a new one
|
|
62
|
-
- Tell the user you found an existing branch and checked it out
|
|
63
|
-
5. If no existing branch, create and checkout the new branch from the current branch: `git checkout -b {branch-name}`
|
|
64
|
-
|
|
65
|
-
## Important Notes
|
|
66
|
-
|
|
67
|
-
- This command is used across multiple repositories. Always check the current repository's `CLAUDE.md` and recent branches/PRs to follow its specific conventions.
|
|
68
|
-
- For projects with many issues, organize the display so it's scannable — group by status, sort by priority within groups.
|
|
69
|
-
- If the project has a Notion planning document linked, mention it so the user can reference it.
|
|
70
|
-
- Keep the output concise but complete. The user needs the full project picture to make decisions.
|
|
@@ -1,13 +0,0 @@
|
|
|
1
|
-
{
|
|
2
|
-
"enabledPlugins": {
|
|
3
|
-
"agent-sdk-dev@claude-plugins-official": true,
|
|
4
|
-
"plugin-dev@claude-plugins-official": true,
|
|
5
|
-
"code-simplifier@claude-plugins-official": true,
|
|
6
|
-
"code-review@claude-plugins-official": true,
|
|
7
|
-
"pr-review-toolkit@claude-plugins-official": true
|
|
8
|
-
},
|
|
9
|
-
"env": {
|
|
10
|
-
"ENABLE_TOOL_SEARCH": "true",
|
|
11
|
-
"CLAUDE_CODE_EXPERIMENTAL_AGENT_TEAMS": "1"
|
|
12
|
-
}
|
|
13
|
-
}
|
package/dist/claude-config/worktrees/valiant-sparking-hare/.claude/skills/doc-drift-check/SKILL.md
DELETED
|
@@ -1,140 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: Doc Drift Check
|
|
3
|
-
description: This skill should be used when the user asks to "check docs for drift", "compare code changes against documentation", "review docs after PR", "find outdated documentation", "check if docs match the code", "verify documentation is up to date", or after completing a pull request review to verify documentation accuracy. Analyzes code changes against project docs, CLAUDE.md, database schema, Notion module pages, and Notion processes to catch mismatches.
|
|
4
|
-
---
|
|
5
|
-
|
|
6
|
-
# Doc Drift Check
|
|
7
|
-
|
|
8
|
-
Analyze code changes from a PR or issue against all project documentation — both in the repository and in Notion — to catch mismatches that would lead to stale docs. Write a severity-categorized comment to the Linear issue only when meaningful drift is found.
|
|
9
|
-
|
|
10
|
-
## When to Use
|
|
11
|
-
|
|
12
|
-
Invoke manually after reviewing a pull request or completing an issue review. Skip for trivial changes (typo fixes, formatting, dependency bumps) where doc drift is impossible.
|
|
13
|
-
|
|
14
|
-
## Workflow
|
|
15
|
-
|
|
16
|
-
### 1. Gather Context
|
|
17
|
-
|
|
18
|
-
Collect three inputs before analysis:
|
|
19
|
-
|
|
20
|
-
**Code changes** — Get the diff for the issue's PR:
|
|
21
|
-
- Use GitHub MCP to read the PR diff
|
|
22
|
-
- Note which files changed and the nature of changes (new files, modified APIs, removed features, config changes, model changes, service changes)
|
|
23
|
-
|
|
24
|
-
**Linear issue** — Read the issue description and metadata:
|
|
25
|
-
- Issue title, description, and acceptance criteria
|
|
26
|
-
- Team identifier (determines which Notion workspace to check):
|
|
27
|
-
- `KPL-` → Platform
|
|
28
|
-
- `KPE-` → Personal
|
|
29
|
-
- `KBU-` → Business
|
|
30
|
-
- Links to Notion pages or external docs in the description
|
|
31
|
-
|
|
32
|
-
**Repository documentation** — Read relevant local docs:
|
|
33
|
-
- `docs/` directory — scan for files related to changed code areas
|
|
34
|
-
- `CLAUDE.md` — project overview, architecture, env vars, structure
|
|
35
|
-
- Database schema files — check if schema changes are reflected
|
|
36
|
-
|
|
37
|
-
### 2. Identify Relevant Notion Documentation
|
|
38
|
-
|
|
39
|
-
Based on the team identifier from the Linear issue, fetch the corresponding Product Development page in Notion. These pages contain module documentation, processes, and feature documents for each team.
|
|
40
|
-
|
|
41
|
-
**Team → Notion Product Development page mapping:**
|
|
42
|
-
|
|
43
|
-
| Team Prefix | Team | Notion Page ID |
|
|
44
|
-
|---|---|---|
|
|
45
|
-
| `KPL-` | Platform | `24c80383e1518012a3c2c410588782d5` |
|
|
46
|
-
| `KPE-` | Personal | `23a80383e15180ae91bae32b407ebaf9` |
|
|
47
|
-
| `KBU-` | Business | `24c80383e15180e19e78cfca4b49fe66` |
|
|
48
|
-
|
|
49
|
-
**Scan the Product Development page** for the appropriate team:
|
|
50
|
-
1. Fetch the page and scan its child pages
|
|
51
|
-
2. Identify module pages that relate to the changed code area
|
|
52
|
-
3. Read those module pages — they describe models, services, and APIs for each module
|
|
53
|
-
|
|
54
|
-
**Check the Processes database** (`252cc951ac514d86ad3b6d9fc689409c`):
|
|
55
|
-
1. The Processes database has a "Модули" relation linking processes to modules
|
|
56
|
-
2. If code changes affect a module, find linked processes
|
|
57
|
-
3. Check if process descriptions still match the implementation
|
|
58
|
-
|
|
59
|
-
### 3. Analyze for Mismatches
|
|
60
|
-
|
|
61
|
-
Compare code changes against each relevant documentation source. Look for:
|
|
62
|
-
|
|
63
|
-
- **Model drift** — Code adds/removes/renames model fields, but module documentation in Notion still describes the old schema
|
|
64
|
-
- **Service drift** — New services added or service logic changed, but module page doesn't list them
|
|
65
|
-
- **API drift** — Endpoints added/changed/removed, but docs still reference old API
|
|
66
|
-
- **Process drift** — Implementation changes how a process works, but the process page in Notion doesn't reflect it
|
|
67
|
-
- **Missing documentation** — New module, feature, or config with no corresponding docs
|
|
68
|
-
- **Stale references** — Docs reference removed code, old file paths, or deprecated models
|
|
69
|
-
- **Issue description mismatch** — What was implemented differs from what the issue describes
|
|
70
|
-
- **Structural drift** — File/directory structure changed but project structure docs not updated
|
|
71
|
-
- **Environment drift** — New env vars, config values, or dependencies not documented
|
|
72
|
-
|
|
73
|
-
For detailed severity classification examples, consult `references/severity-guide.md`.
|
|
74
|
-
|
|
75
|
-
### 4. Decision Gate
|
|
76
|
-
|
|
77
|
-
After analysis, decide whether to comment:
|
|
78
|
-
|
|
79
|
-
- **No mismatches found** → Do nothing. Do not write a "no issues found" comment.
|
|
80
|
-
- **Only trivial mismatches** (e.g., a slightly outdated example that doesn't mislead) → Skip.
|
|
81
|
-
- **Meaningful mismatches** → Proceed to write comment.
|
|
82
|
-
|
|
83
|
-
The bar for commenting: would someone reading the docs be misled or confused by the current state? If yes, comment. If not, skip.
|
|
84
|
-
|
|
85
|
-
### 5. Write Linear Comment
|
|
86
|
-
|
|
87
|
-
Post a comment on the Linear issue using this format:
|
|
88
|
-
|
|
89
|
-
```
|
|
90
|
-
📋 Doc Drift Report
|
|
91
|
-
|
|
92
|
-
[1-2 sentence summary of what was found]
|
|
93
|
-
|
|
94
|
-
🔴 Critical
|
|
95
|
-
- [Notion: Module "Payments" → Models section] — Code adds TransferFee model but module page doesn't list it
|
|
96
|
-
- [File: `CLAUDE.md` env vars table] — New SYNC_API_KEY env var required but not listed
|
|
97
|
-
|
|
98
|
-
🟡 Moderate
|
|
99
|
-
- [Notion: Process "Customer Onboarding"] — Step 3 now calls ProviderService instead of direct API, process diagram outdated
|
|
100
|
-
- [File: `docs/architecture.md`] — New endpoint POST /api/v2/transfers not documented
|
|
101
|
-
|
|
102
|
-
🟢 Light
|
|
103
|
-
- [Notion: Module "Crypto" → Services section] — CryptoSigningService renamed to AccessKeyService in code
|
|
104
|
-
|
|
105
|
-
---
|
|
106
|
-
Suggested actions:
|
|
107
|
-
- Update Notion module page "Payments" with new TransferFee model
|
|
108
|
-
- Add SYNC_API_KEY to CLAUDE.md environment variables table
|
|
109
|
-
- Update process diagram in "Customer Onboarding" process page
|
|
110
|
-
```
|
|
111
|
-
|
|
112
|
-
**Formatting rules:**
|
|
113
|
-
- Use severity emojis: 🔴 Critical, 🟡 Moderate, 🟢 Light
|
|
114
|
-
- Prefix with source: `[Notion: ...]` or `[File: ...]`
|
|
115
|
-
- Include specific page/section references
|
|
116
|
-
- Include a "Suggested actions" section with concrete fixes
|
|
117
|
-
- Omit empty severity categories
|
|
118
|
-
|
|
119
|
-
## Severity Classification Quick Reference
|
|
120
|
-
|
|
121
|
-
| Severity | Description |
|
|
122
|
-
|---|---|
|
|
123
|
-
| 🔴 Critical | Model/schema changes undocumented, API changes, env var changes, breaking behavior, architecture shifts — someone would build against wrong assumptions |
|
|
124
|
-
| 🟡 Moderate | New services undocumented, process flow changes, new features without docs, workflow changes — someone would miss important capabilities |
|
|
125
|
-
| 🟢 Light | Renamed entities, outdated examples, minor wording, service method renames — someone might be briefly confused |
|
|
126
|
-
|
|
127
|
-
For detailed examples and edge cases, see `references/severity-guide.md`.
|
|
128
|
-
|
|
129
|
-
## Tools Used
|
|
130
|
-
|
|
131
|
-
- **GitHub MCP** — read PR diffs and changed files
|
|
132
|
-
- **Linear MCP** — read issue description and team, post comments
|
|
133
|
-
- **Notion MCP** — fetch Product Development pages, module pages, process pages
|
|
134
|
-
- **Read, Grep, Glob** — scan local documentation files in the repository
|
|
135
|
-
|
|
136
|
-
## Additional Resources
|
|
137
|
-
|
|
138
|
-
### Reference Files
|
|
139
|
-
|
|
140
|
-
- **`references/severity-guide.md`** — Detailed severity classification with examples, edge cases, and per-source guidance
|
|
@@ -1,174 +0,0 @@
|
|
|
1
|
-
# Severity Classification Guide
|
|
2
|
-
|
|
3
|
-
Detailed guide for categorizing doc drift findings by severity level. Use this when analyzing mismatches between code changes and documentation.
|
|
4
|
-
|
|
5
|
-
## 🔴 Critical — Would Cause Wrong Assumptions
|
|
6
|
-
|
|
7
|
-
These mismatches would cause someone to build against incorrect assumptions or make wrong decisions.
|
|
8
|
-
|
|
9
|
-
### Model/Schema Changes
|
|
10
|
-
|
|
11
|
-
- Code adds a new model but the Notion module page doesn't list it
|
|
12
|
-
- Code removes or renames a model field, but the module documentation still references the old field
|
|
13
|
-
- Relationship between models changed (e.g., ForeignKey → ManyToMany) but not reflected in docs
|
|
14
|
-
- New required field added to a model but not documented in the module's models section
|
|
15
|
-
|
|
16
|
-
**Example:**
|
|
17
|
-
> Code adds `AccessKey` model to `andgate_crypto` module, but the Crypto module page in Notion under Platform's Product Development doesn't mention it.
|
|
18
|
-
|
|
19
|
-
### API/Endpoint Changes
|
|
20
|
-
|
|
21
|
-
- New endpoint added without documentation
|
|
22
|
-
- Endpoint removed but still listed in docs
|
|
23
|
-
- Request/response format changed but docs show old format
|
|
24
|
-
- Authentication requirements changed
|
|
25
|
-
|
|
26
|
-
### Environment & Configuration
|
|
27
|
-
|
|
28
|
-
- New environment variable required but not in CLAUDE.md or deployment docs
|
|
29
|
-
- Configuration option removed but still documented
|
|
30
|
-
- Default value changed for existing config
|
|
31
|
-
|
|
32
|
-
### Architecture Changes
|
|
33
|
-
|
|
34
|
-
- New module/app created without corresponding Notion documentation
|
|
35
|
-
- Service dependencies changed (module A now depends on module B, but docs don't show this)
|
|
36
|
-
- Database migration adds/removes tables not reflected in schema docs
|
|
37
|
-
|
|
38
|
-
---
|
|
39
|
-
|
|
40
|
-
## 🟡 Moderate — Would Miss Important Capabilities
|
|
41
|
-
|
|
42
|
-
These mismatches would cause someone to miss important features or follow outdated procedures.
|
|
43
|
-
|
|
44
|
-
### Service Changes
|
|
45
|
-
|
|
46
|
-
- New service class added to a module but module page doesn't list it in the services section
|
|
47
|
-
- Service method signature changed (different params) but process docs reference old signature
|
|
48
|
-
- Service now calls a different external provider but process diagrams show old flow
|
|
49
|
-
|
|
50
|
-
**Example:**
|
|
51
|
-
> Code adds `AccessKeyService` to handle crypto signing, but the Crypto module page doesn't list it under Services.
|
|
52
|
-
|
|
53
|
-
### Process Flow Changes
|
|
54
|
-
|
|
55
|
-
- Steps in a business process reordered or modified
|
|
56
|
-
- New step added to a process (e.g., new validation step in onboarding)
|
|
57
|
-
- Conditional logic changed (e.g., different handling for B2B vs B2C)
|
|
58
|
-
- Process now has prerequisites that aren't documented
|
|
59
|
-
|
|
60
|
-
**Example:**
|
|
61
|
-
> Customer onboarding now requires provider-specific KYC before account creation, but the process page still shows a single KYC step.
|
|
62
|
-
|
|
63
|
-
### Feature Documentation Gaps
|
|
64
|
-
|
|
65
|
-
- New feature implemented but no feature document exists
|
|
66
|
-
- Feature behavior changed significantly but feature docs describe old behavior
|
|
67
|
-
- Feature flag added without documentation in relevant places
|
|
68
|
-
|
|
69
|
-
### Workflow Changes
|
|
70
|
-
|
|
71
|
-
- Webhook handling changed but webhook documentation outdated
|
|
72
|
-
- Background task behavior modified but not reflected in process docs
|
|
73
|
-
- Error handling strategy changed for a module
|
|
74
|
-
|
|
75
|
-
---
|
|
76
|
-
|
|
77
|
-
## 🟢 Light — Brief Confusion Only
|
|
78
|
-
|
|
79
|
-
These mismatches would cause someone brief confusion but wouldn't lead to incorrect work.
|
|
80
|
-
|
|
81
|
-
### Naming/Renaming
|
|
82
|
-
|
|
83
|
-
- Service or utility class renamed but docs use old name
|
|
84
|
-
- File moved to different directory but docs reference old path
|
|
85
|
-
- Variable or constant renamed
|
|
86
|
-
|
|
87
|
-
**Example:**
|
|
88
|
-
> `CryptoSigningService` renamed to `AccessKeyService` in code but Notion module page still says `CryptoSigningService`.
|
|
89
|
-
|
|
90
|
-
### Outdated Examples
|
|
91
|
-
|
|
92
|
-
- Code example in docs uses old import path
|
|
93
|
-
- Example shows deprecated API usage but functionally equivalent
|
|
94
|
-
- Test examples reference old fixtures
|
|
95
|
-
|
|
96
|
-
### Minor Wording
|
|
97
|
-
|
|
98
|
-
- Documentation says "will be implemented" for already-implemented features
|
|
99
|
-
- Status markers (TODO, WIP) left in docs for completed work
|
|
100
|
-
- Comments in code reference old Notion page names
|
|
101
|
-
|
|
102
|
-
---
|
|
103
|
-
|
|
104
|
-
## Per-Source Analysis Guide
|
|
105
|
-
|
|
106
|
-
### Notion Module Pages
|
|
107
|
-
|
|
108
|
-
Module pages describe models and services for each module. When checking:
|
|
109
|
-
|
|
110
|
-
1. **Models section** — Compare listed models against actual Django models in the code
|
|
111
|
-
2. **Services section** — Compare listed services against actual service classes
|
|
112
|
-
3. **API section** — Compare listed endpoints against actual URL configs
|
|
113
|
-
4. **Shared models** — Verify cross-module dependencies are still accurate
|
|
114
|
-
5. **Dependency graph** — Check if module interdependencies changed
|
|
115
|
-
|
|
116
|
-
### Notion Processes Database
|
|
117
|
-
|
|
118
|
-
Process pages describe business workflows. When checking:
|
|
119
|
-
|
|
120
|
-
1. **Process steps** — Do the documented steps match the actual code flow?
|
|
121
|
-
2. **Prerequisites** — Are process prerequisites still accurate?
|
|
122
|
-
3. **Provider interactions** — Do provider-specific flows match current implementation?
|
|
123
|
-
4. **Status transitions** — Do documented state machines match code?
|
|
124
|
-
5. **Related modules** — Is the "Модули" relation still correct?
|
|
125
|
-
|
|
126
|
-
### Repository docs/ Directory
|
|
127
|
-
|
|
128
|
-
Local documentation files. When checking:
|
|
129
|
-
|
|
130
|
-
1. **Architecture docs** — Do they reflect current module structure?
|
|
131
|
-
2. **API docs** — Do endpoint descriptions match implementations?
|
|
132
|
-
3. **Setup/deployment docs** — Are instructions still accurate?
|
|
133
|
-
|
|
134
|
-
### CLAUDE.md
|
|
135
|
-
|
|
136
|
-
Project-level documentation. When checking:
|
|
137
|
-
|
|
138
|
-
1. **Environment variables table** — All required vars listed?
|
|
139
|
-
2. **Project structure** — Does the tree match actual directory layout?
|
|
140
|
-
3. **Technology stack** — Any new dependencies or tools?
|
|
141
|
-
4. **Key documentation links** — Do referenced docs still exist?
|
|
142
|
-
|
|
143
|
-
### Database Schema
|
|
144
|
-
|
|
145
|
-
Schema documentation. When checking:
|
|
146
|
-
|
|
147
|
-
1. **Table definitions** — Do they match current migrations?
|
|
148
|
-
2. **Relationships** — Are FK/M2M relations documented correctly?
|
|
149
|
-
3. **Indexes** — Any new indexes not mentioned in schema docs?
|
|
150
|
-
4. **New tables** — Any new migrations creating tables not in schema docs?
|
|
151
|
-
|
|
152
|
-
---
|
|
153
|
-
|
|
154
|
-
## Edge Cases
|
|
155
|
-
|
|
156
|
-
### When NOT to Flag
|
|
157
|
-
|
|
158
|
-
- Code comments that are slightly outdated (not worth a Linear comment)
|
|
159
|
-
- Test file documentation being slightly behind (tests are self-documenting)
|
|
160
|
-
- Third-party library version bumps with no behavior change
|
|
161
|
-
- Formatting-only changes to documentation files
|
|
162
|
-
- Internal refactoring that doesn't change public interfaces or behavior
|
|
163
|
-
|
|
164
|
-
### When to Escalate to Critical
|
|
165
|
-
|
|
166
|
-
Even if a change seems small, escalate if:
|
|
167
|
-
- It affects an external-facing API that developers build against
|
|
168
|
-
- It changes model fields that other services query
|
|
169
|
-
- It modifies environment variables needed for deployment
|
|
170
|
-
- It alters the behavior described in a process that operations follows
|
|
171
|
-
|
|
172
|
-
### Multiple Small Issues
|
|
173
|
-
|
|
174
|
-
If there are many Light issues (5+) concentrated in one documentation area, consider grouping them and elevating to a single Moderate finding with a suggestion to do a documentation review for that area.
|
package/dist/claude-config/worktrees/valiant-sparking-hare/.github/workflows/claude-code-review.yml
DELETED
|
@@ -1,32 +0,0 @@
|
|
|
1
|
-
name: Claude Code Review
|
|
2
|
-
|
|
3
|
-
on:
|
|
4
|
-
pull_request:
|
|
5
|
-
types: [opened, synchronize, ready_for_review, reopened]
|
|
6
|
-
|
|
7
|
-
jobs:
|
|
8
|
-
claude-review:
|
|
9
|
-
runs-on: ubuntu-latest
|
|
10
|
-
permissions:
|
|
11
|
-
contents: write
|
|
12
|
-
pull-requests: write
|
|
13
|
-
issues: write
|
|
14
|
-
id-token: write
|
|
15
|
-
|
|
16
|
-
steps:
|
|
17
|
-
- name: Checkout repository
|
|
18
|
-
uses: actions/checkout@v4
|
|
19
|
-
with:
|
|
20
|
-
fetch-depth: 1
|
|
21
|
-
|
|
22
|
-
- name: Run Claude Code Review
|
|
23
|
-
id: claude-review
|
|
24
|
-
uses: anthropics/claude-code-action@v1
|
|
25
|
-
with:
|
|
26
|
-
claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}
|
|
27
|
-
plugin_marketplaces: 'https://github.com/anthropics/claude-code.git'
|
|
28
|
-
plugins: 'code-review@claude-code-plugins'
|
|
29
|
-
prompt: '/code-review:code-review ${{ github.repository }}/pull/${{ github.event.pull_request.number }}'
|
|
30
|
-
# See https://github.com/anthropics/claude-code-action/blob/main/docs/usage.md
|
|
31
|
-
# or https://code.claude.com/docs/en/cli-reference for available options
|
|
32
|
-
|
|
@@ -1,37 +0,0 @@
|
|
|
1
|
-
name: Claude Code
|
|
2
|
-
|
|
3
|
-
on:
|
|
4
|
-
issue_comment:
|
|
5
|
-
types: [created]
|
|
6
|
-
pull_request_review_comment:
|
|
7
|
-
types: [created]
|
|
8
|
-
issues:
|
|
9
|
-
types: [opened, assigned]
|
|
10
|
-
pull_request_review:
|
|
11
|
-
types: [submitted]
|
|
12
|
-
|
|
13
|
-
jobs:
|
|
14
|
-
claude:
|
|
15
|
-
if: |
|
|
16
|
-
(github.event_name == 'issue_comment' && contains(github.event.comment.body, '@claude')) ||
|
|
17
|
-
(github.event_name == 'pull_request_review_comment' && contains(github.event.comment.body, '@claude')) ||
|
|
18
|
-
(github.event_name == 'pull_request_review' && contains(github.event.review.body, '@claude')) ||
|
|
19
|
-
(github.event_name == 'issues' && (contains(github.event.issue.body, '@claude') || contains(github.event.issue.title, '@claude')))
|
|
20
|
-
runs-on: ubuntu-latest
|
|
21
|
-
permissions:
|
|
22
|
-
contents: write
|
|
23
|
-
pull-requests: write
|
|
24
|
-
issues: write
|
|
25
|
-
id-token: write
|
|
26
|
-
actions: read # Required for Claude to read CI results on PRs
|
|
27
|
-
steps:
|
|
28
|
-
- name: Checkout repository
|
|
29
|
-
uses: actions/checkout@v4
|
|
30
|
-
with:
|
|
31
|
-
fetch-depth: 1
|
|
32
|
-
|
|
33
|
-
- name: Run Claude Code
|
|
34
|
-
id: claude
|
|
35
|
-
uses: anthropics/claude-code-action@v1
|
|
36
|
-
with:
|
|
37
|
-
claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}
|
|
@@ -1 +0,0 @@
|
|
|
1
|
-
v22.18.0
|