skiv 0.0.2 → 0.0.5

package/README.md

@@ -3,7 +3,7 @@
 ## Installation
 
 ```shell
-npm i -g skiv
+$ npm i -g skiv
 ```
 
 ## Usage
@@ -13,15 +13,18 @@ npm i -g skiv
 cd to your project directory
 
 ```shell
-npx skiv init
+$ npx skiv init
 ```
 
 generate `.skiv` directory
 
 edit `.skiv/config.yaml` and `.skiv/roles/*`
 
+add `.skiv` to your gitignore
+
 ### Start
 
 ```shell
-npx skiv start
+$ tmux
+$ npx skiv start
 ```

package/bin/skiv.ts

@@ -0,0 +1,3 @@
+#!/usr/bin/env tsx
+
+import "../skiv/cli"

package/package.json

@@ -1,12 +1,16 @@
 {
   "name": "skiv",
-  "version": "0.0.2",
+  "version": "0.0.5",
   "publishConfig": {
     "access": "public"
   },
   "description": "claude code orchestration tool",
   "license": "MIT",
   "author": "",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/masashimachida/skiv.git"
+  },
   "type": "commonjs",
   "main": "dist/esm/index.js",
   "scripts": {
@@ -22,14 +26,13 @@
     "skeleton"
   ],
   "bin": {
-    "skiv": "bin/skiv.js"
+    "skiv": "bin/skiv.ts"
   },
   "devDependencies": {
     "@types/better-sqlite3": "^7.6.13",
     "@types/fs-extra": "^11.0.4",
     "@types/js-yaml": "^4.0.9",
     "@types/node": "^25.2.0",
-    "tsx": "^4.21.0",
     "typescript": "^5.9.3"
   },
   "dependencies": {
@@ -37,6 +40,8 @@
     "commander": "^14.0.3",
     "fs-extra": "^11.3.3",
     "js-yaml": "^4.1.1",
-    "kysely": "^0.28.11"
+    "kysely": "^0.28.11",
+    "simple-git": "^3.30.0",
+    "tsx": "^4.21.0"
   }
 }

package/skeleton/config.yaml

@@ -1,6 +1,6 @@
 model: sonnet
 members:
-  - name: Corder001
+  - name: CORDER
     role: corder
-  - name: Reviewer001
+  - name: REVIEWER
     role: reviewer

package/skeleton/roles/corder/CLAUDE.md

@@ -9,58 +9,40 @@
 
 ### 2. 前提条件
 - ブランチ名: `issues/issue-<issue.id>`
-- 作業パス： `repo/issues/issue-<issue.id>`
+- 作業パス： `worktree`
 
-### 3. 作業環境の準備
-Issueが存在する場合、以下の手順で作業ブランチを作成します。
-- Git Worktreeを作成: `git worktree add <作業パス> -b <ブランチ名> || git worktree add <作業パス> <ブランチ名>`
-- **作業ディレクトリへ移動: `cd <作業パス>`**
-
-### 4. 実装
+### 3. 実装
 Issueのタイトルや仕様を確認し、実装を行います。
 - **実装前に既存の関連コードを読み、プロジェクトのコーディングスタイルを確認してください。**
+- ファイルの更新は作業パス以外行わないこと。
 - 変更は最小限に留めること。
 - 関連のないコードをリファクタリングしないこと。
 
-### 5. コミット
-変更完了後、以下の形式でコミットします。
-- メッセージ: `<type>: issue #<issue.id> <issue.title>`
-    - <type>は新機能なら `feat`, 修正なら `fix` を使用。
-
-### 6. worktree削除
-- コマンド: `git worktree remove <ブランチ名>`
-
-### 7. レビュー準備
-最後にIssueを更新して完了報告をします。
-`npx skiv issue update_status <issue.id> ready_for_review`
-
-
-## 利用可能なツール
-
-Issue Tool:
-npx skiv issue
-
-Git:
-git 
-
-### 使用可能なコマンド
-
-Issue Tool:
-- ready_for_review
-
-Git:
-- worktree 
-- add
-- commit
-- diff
+### コミット
+コミットなどの作業はやらないこと。
+
+## アウトプット形式
+からなず以下のjson形式で出力してください。
+出力をこのままパースするので、json以外の出力をしないでください。
+「```json」などの修飾もしないでください。
+絶対に以下のJSON以外を出力しないでください。
+
+正常時：
+{
+"result": "FINISH",
+"branch": "<ブランチ名>",
+"issue_title": "<issue名>"
+}
+
+異常時：
+{
+"result": "ERROR",
+"branch": "<ブランチ名>",
+"issue_title": "<issue名>",
+"reason": "<完了しなかった理由>"
+}
 
 ## 禁止事項
 - 新しいIssueの作成
 - 他タスクのレビュー
 - 割り当てられたタスク範囲外のファイル編集
-
-## 行動ルール
-1. タスクを取得する
-2. 実装する
-3. コミットする
-4. レビュー準備完了としてマークする

package/skeleton/roles/corder/custom.ts

@@ -1,39 +1,21 @@
-import {Issue} from "../../../../skiv/src/db/schema";
-import Member from "../../../src/Member";
+import {Issue} from "skiv/skiv/db/schema"
+import AbstractWorker from "../../worker"
 
-export default class CustomMember extends Member {
+export default class Custom extends AbstractWorker {
 
   public async before(): Promise<boolean> {
-    const assignedIssue = await this.issueService.getAssignedIssue(this.NAME, 'open')
+    this.logger.debug('before()')
+    const assignedIssue = await this.issueService.getNextIssue('open', 'in_progress', this.NAME)
     if (assignedIssue) {
-
-      console.log(`[NEW ISSUE]
-Title: ${assignedIssue.title}
-Description: ${assignedIssue.description}
-`)
-
-      this.createPrompt(assignedIssue)
-      return true
-    }
-
-    const assign = await this.issueService.assignNextIssue(this.NAME)
-    if (assign) {
-
-      console.log(`[NEW ISSUE]
-Title: ${assign.title}
-Description: ${assign.description}
-`)
-
-      this.createPrompt(assign)
+      this.logger.info(`found issue: #${assignedIssue.id} ${assignedIssue.title}`)
+      await this.setup(assignedIssue)
       return true
     }
 
-    console.log('no issue')
-
     return false
   }
 
-  private createPrompt(issue: Issue) {
+  protected createPrompt(issue: Issue): void {
 
     const comments = issue.comments?.map(comment => {
       return `- id: ${comment.id}
@@ -59,4 +41,39 @@ ${comments}
 
 このあとはCLAUDE.mdに沿って処理をしてください。`
   }
+
+  public async after(response: string): Promise<void> {
+    this.logger.debug('after()')
+
+    if (!this.ISSUE_ID) {
+      throw new Error('ISSUE_ID is not set')
+    }
+    const res = this.parseResponse<{
+      result: "FINISH" | "ERROR",
+      branch: string,
+      issue_title: string,
+      reason?: string
+    }>(response)
+
+    if (res.result === "FINISH") {
+
+      try {
+        await this.git
+          .add('.')
+          .commit(`feature: issue #${this.ISSUE_ID} ${res.issue_title}`)
+        await this.issueService.updateStatus(this.ISSUE_ID, 'ready_for_review')
+      } catch (e) {
+        this.logger.error(e as string)
+        process.exit(1)
+      } finally {
+        await this.cleanupWorktree()
+      }
+
+    } else {
+      this.logger.error(res)
+      await this.issueService.updateStatus(this.ISSUE_ID, 'error')
+      await this.cleanupWorktree()
+      process.exit(1)
+    }
+  }
 }

package/skeleton/roles/pm/.claude/settings.local.json

@@ -0,0 +1,13 @@
+{
+  "permissions": {
+    "allow": [
+      "Bash(npx skiv issue *)",
+      "Bash(git *)"
+    ],
+    "deny": [
+      "Bash(curl:*)",
+      "Read(./.env)",
+      "Read(./.env.*)"
+    ]
+  }
+}

package/skeleton/roles/pm/CLAUDE.md

@@ -0,0 +1,56 @@
+# ROLE: Project Manager
+
+あなたは指定されたブランチをmainブランチにマージをします。
+適宜、仕様書(spec.md)を参照します。
+
+## ワークフロー
+
+### 1. マージする Issue の取得
+プロンプトで渡された Issue を参照してください。
+
+### 2. 前提条件
+- ブランチ名: `issues/issue-<issue.id>`
+
+### 3. マージ
+該当のブランチをmainへマージします。
+コンフリクトした場合は適宜修正をしてコンフリクトを解消してください。
+修正の判断がつかない場合は無理に解消しようとしないでください。
+
+## アウトプット形式
+からなず以下のjson形式で出力してください。
+出力をこのままパースするので、json以外の出力をしないでください。
+「```json」などの修飾もしないでください。
+絶対に以下のJSON以外を出力しないでください。
+
+マージOK時：
+{
+  "result": "MERGED",
+  "branch": "<ブランチ名>",
+  "issue_title": "<issue名>",
+  "reason": "<何かコメントがあれば>"
+}
+
+コンフリクトが解消できない時：
+{
+  "result": "CONFLICT",
+  "branch": "<ブランチ名>",
+  "issue_title": "<issue名>",
+  "reason": "<コンフリクトが解消できなかった理由>"
+}
+
+異常時：
+{
+  "result": "ERROR",
+  "branch": "<ブランチ名>",
+  "issue_title": "<issue名>",
+  "reason": "<完了しなかった理由>"
+}
+
+### 実施可能なこと
+- 差分の確認
+- コンフリクトの解消
+- gitの操作
+
+## 禁止事項
+- Issueの作成
+- Issueのアサイン

package/skeleton/roles/pm/custom.ts

@@ -0,0 +1,75 @@
+import {Issue} from "skiv/skiv/db/schema"
+import AbstractWorker from "../../worker"
+
+export default class CustomMember extends AbstractWorker {
+
+  public async before(): Promise<boolean> {
+    this.logger.debug('before()')
+    const issue = await this.issueService.getNextIssue('request_for_merge', 'in_merge_process', this.NAME)
+    if (issue) {
+      this.logger.info(`found issue: #${issue.id} ${issue.title}`)
+      await this.setup(issue)
+      return true
+    }
+
+    return false
+  }
+
+  protected createPrompt(issue: Issue) {
+
+    const comments = issue.comments?.map(comment => {
+      return `- id: ${comment.id}
+  by: ${comment.by}
+  message: ${comment.message}
+`
+    }).join("\n") || ''
+
+    this.PROMPT = `あなたの名前は \`${this.NAME}\` です。
+あなたがレビューするタスクは以下の通りです。
+
+id:
+${issue.id}
+
+タスク名:
+${issue.title}
+
+概要:
+${issue.description}
+
+コメント:
+${comments}
+
+このあとはCLAUDE.mdに沿って処理をしてください。`
+  }
+
+  public async after(response: string): Promise<void> {
+    this.logger.debug('after()')
+
+    if (!this.ISSUE_ID) {
+      throw new Error('ISSUE_ID is not set')
+    }
+
+    const res = this.parseResponse<{
+      result: "MERGED" | "ERROR",
+      branch: string,
+      issue_title: string,
+      reason?: string
+    }>(response)
+
+    if (res.result === "MERGED") {
+
+      await this.issueService.updateStatus(this.ISSUE_ID, 'done')
+      if (res.reason) {
+        await this.issueService.comment(this.ISSUE_ID, this.NAME, res.reason)
+      }
+    } else {
+      this.logger.error(res)
+      await this.issueService.updateStatus(this.ISSUE_ID, 'error')
+      await this.issueService.comment(this.ISSUE_ID, this.NAME, res.reason || "(no reason)")
+      await this.cleanupWorktree()
+      process.exit(1)
+    }
+
+    await this.cleanupWorktree()
+  }
+}

package/skeleton/roles/reviewer/CLAUDE.md

@@ -9,50 +9,50 @@
 
 ### 2. 前提条件
 - ブランチ名: `issues/issue-<issue.id>`
-- 作業パス： `repo/issues/issue-<issue.id>`
+- 作業パス： `worktree`
 
-### 3. 作業環境の準備
-Issue が存在する場合、以下の手順で作業ブランチを作成します。
-- Git Worktree を作成: `git worktree add <作業パス> -b <ブランチ名> || git worktree add <作業パス> <ブランチ名>`
-- **作業ディレクトリへ移動: `cd <作業パス>`**
-
-### 4. レビュー
+### 3. レビュー
 Issue のタイトルや仕様を確認し、以下の観点から実装が適切に行われているか確認します。
 - コーディングスタイルに沿っているか
 - テストは充実しているか
 - 変更は最小限に留まっているか
 - Issue に関連のない変更をしていないか
 
-### 5. worktree削除
-ブランチを削除します
-- コマンド: `git worktree remove <ブランチ名>`
-
-### 6. マージリクエスト / 却下
-コードに問題がなければ Issue をマージリクエストの状態にします。
-- コマンド: `npx skiv issue update_status <issue.id> request_for_merge`
-
-問題があれば指摘事項をコメントして Issue を再び assign に戻します。
-- コマンド: 
-  - `npx skiv issue comment <issue.id> <YOUR_NAME> <message>`
-  - `npx skiv issue assign <issue.id> <YOUR_NAME>`
-
-## 利用可能なツール
-
-Issue Tool:
-npx skiv issue
-
-Git:
-git diff
+## アウトプット形式
+からなず以下のjson形式で出力してください。
+出力をこのままパースするので、json以外の出力をしないでください。
+「```json」などの修飾もしないでください。
+絶対に以下のJSON以外を出力しないでください。
+
+レビューOK時：
+{
+  "result": "APPROVED",
+  "branch": "<ブランチ名>",
+  "issue_title": "<issue名>",
+  "reason": "<何かコメントがあれば>"
+}
+
+レビューNG時：
+{
+  "result": "REJECT",
+  "branch": "<ブランチ名>",
+  "issue_title": "<issue名>",
+  "reason": "<却下の理由/修正点>"
+}
+
+異常時：
+{
+  "result": "ERROR",
+  "branch": "<ブランチ名>",
+  "issue_title": "<issue名>",
+  "reason": "<完了しなかった理由>"
+}
 
 ### 実施可能なこと
 - 差分の確認
 - レビューコメントの追加
 - 完了としてマーク
 
-### 例
-npx skiv issue comment 12 Reviewer "Refactor this function"
-npx skiv issue done 12
-
 ## 禁止事項
 - 実装コードの作成
 - Issueの作成

package/skeleton/roles/reviewer/custom.ts

@@ -1,31 +1,21 @@
-import {Issue} from "../../../../skiv/src/db/schema";
-import Member from "../../../src/Member";
+import {Issue} from "skiv/skiv/db/schema"
+import AbstractWorker from "../../worker"
 
-export default class CustomMember extends Member {
+export default class CustomMember extends AbstractWorker {
 
   public async before(): Promise<boolean> {
-    const readyForReviewIssue = await this.issueService.getReviewIssue()
-
-    if (!readyForReviewIssue) {
-      console.log('no issue for review')
-      return false
+    this.logger.debug('before()')
+    const readyForReviewIssue = await this.issueService.getNextIssue('ready_for_review', 'reviewing', this.NAME)
+    if (readyForReviewIssue) {
+      this.logger.info(`found issue: #${readyForReviewIssue.id} ${readyForReviewIssue.title}`)
+      await this.setup(readyForReviewIssue)
+      return true
     }
 
-    // if (readyForReviewIssue.status !== 'ready_for_review') {
-    //   console.log('issue is not ready for review')
-    //   return false
-    // }
-
-    console.log(`[NEW ISSUE]
-Title: ${readyForReviewIssue.title}
-Description: ${readyForReviewIssue.description}
-`)
-
-    this.createPrompt(readyForReviewIssue)
-    return true
+    return false
   }
 
-  private createPrompt(issue: Issue) {
+  protected createPrompt(issue: Issue) {
 
     const comments = issue.comments?.map(comment => {
       return `- id: ${comment.id}
@@ -51,4 +41,42 @@ ${comments}
 
 このあとはCLAUDE.mdに沿って処理をしてください。`
   }
+
+  public async after(response: string): Promise<void> {
+    this.logger.debug('after()')
+
+    if (!this.ISSUE_ID) {
+      throw new Error('ISSUE_ID is not set')
+    }
+
+    const res = this.parseResponse<{
+      result: "APPROVED" | "REJECT" | "ERROR",
+      branch: string,
+      issue_title: string,
+      reason?: string
+    }>(response)
+
+    if (res.result === "APPROVED") {
+
+      await this.issueService.updateStatus(this.ISSUE_ID, 'request_for_merge')
+      if (res.reason) {
+        await this.issueService.comment(this.ISSUE_ID, this.NAME, res.reason)
+      }
+
+    } else if (res.result === "REJECT") {
+
+      await this.issueService.updateStatus(this.ISSUE_ID, 'open')
+      await this.issueService.updateAssignee(this.ISSUE_ID, null)
+      await this.issueService.comment(this.ISSUE_ID, this.NAME, res.reason || "(no reason)")
+
+    } else {
+      this.logger.error(res)
+      await this.issueService.updateStatus(this.ISSUE_ID, 'error')
+      await this.issueService.comment(this.ISSUE_ID, this.NAME, res.reason || "(no reason)")
+      await this.cleanupWorktree()
+      process.exit(1)
+    }
+
+    await this.cleanupWorktree()
+  }
 }

package/skeleton/roles/se/CLAUDE.md

@@ -1,6 +1,7 @@
 # ROLE: SE (System Engineer Agent)
 
-あなたの責任範囲は計画立案とタスク分解のみです。
+あなたの責任範囲は計画立案とタスク分解のみです
+ただし仕様書(spec.md)の編集だけは行えます。
 
 ## 利用可能なツール
 
@@ -15,7 +16,7 @@ npx skiv issue
 ### 例
 
 Issueの作成:
-npx skiv issue create "Implement auth API" high design.md#auth
+npx skiv issue create "fizzbuzz.tsの作成" high "1から順に数を数え上げ、3の倍数で「Fizz」、5の倍数で「Buzz」、15の倍数（3と5の公倍数）で「FizzBuzz」と表示する関数を作成する"
 
 優先度は次のいずれかである必要があります:
 - low
@@ -25,8 +26,7 @@ npx skiv issue create "Implement auth API" high design.md#auth
 他の値が必要な場合は、最も近いものを選択してください。  
 新しい優先度レベルを作成してはいけません。
 
-指示コメントの追加:
-npx skiv issue comment 12 SE "Use JWT auth"
+説明(作業指示)を必ず記述してください。
 
 ## 禁止事項
 - ソースコードの編集
@@ -36,6 +36,12 @@ npx skiv issue comment 12 SE "Use JWT auth"
 
 ## 行動ルール
 - 要件を小さなタスクに分解すること
-  - ただし実装とテストは同一のタスクにすること
+  - タスクを分割する際は、以下の制約を厳守してください。
+    - 粒度：1つのエンドポイント、1つのUIパーツ、1つのロジック関数
+    - 禁止：『〜の構築』『〜の実装』といった大きな単位
+    - 推奨：『xxxをyyyに追加してzzzできるようにする』という具体的な記述
+- 1つのタスクは、シニアエンジニアが 30分以内に実装・テスト完了できる規模 であること
+- 実装とテストは同一のタスクにすること
 - 各タスクは単独で実装可能でなければならない
 - 常に仕様書の該当セクションを参照すること
+- 必要に応じて仕様書を修正すること

package/skeleton/worker.ts

@@ -0,0 +1,29 @@
+import {Issue} from "skiv/skiv/db/schema";
+import Worker from "skiv/skiv/Worker";
+
+export default abstract class AbstractWorker extends Worker {
+
+  protected ISSUE_ID: number | null = null
+
+  protected async setup(issue: Issue) {
+    this.ISSUE_ID = issue.id
+    await this.cleanupWorktree()
+    await this.setupWorktree(`issues/issue-${issue.id}`)
+    this.createPrompt(issue)
+  }
+
+  protected abstract createPrompt(issue: Issue): void
+
+  protected parseResponse<T>(response: string): T {
+    const matches = response.match(/\{[\s\S]*\}/)
+    const jsonStr = matches ? matches[0] : response
+
+    const parsed = JSON.parse(jsonStr) as T
+
+    this.logger.debug('----------------------')
+    this.logger.debug(parsed as unknown as object)
+    this.logger.debug('----------------------')
+
+    return parsed
+  }
+}

package/bin/________________________skiv.ts

@@ -1,3 +0,0 @@
-#!/usr/bin/env tsx
-
-import "../src/cli";

package/bin/skiv.js

@@ -1,3 +0,0 @@
-#!/usr/bin/env node
-
-require('../dist/cjs/cli.js')