qfai 1.6.2 → 1.6.4

package/assets/init/.github/instructions/code-review.instructions.md

@@ -0,0 +1,63 @@
+---
+applyTo: "**/*"
+excludeAgent: "coding-agent"
+---
+
+# Copilot Code Review Instructions (Repository)
+
+Goal:
+
+- Provide high-quality, line-level PR review comments that improve code health.
+- Base the review on the PR description and the diff against the merge target.
+
+Language:
+
+- Default: Japanese.
+- If the PR description includes `Review Language:` (e.g., `ja`, `en`, `ja+en`), follow it.
+- For multi-language requests, write each comment in all requested languages, Japanese first.
+
+Process:
+
+1. Read the PR description (use the PR template sections) and extract:
+   - Why/background
+   - Business/process position
+   - Intended behavior change
+   - Design decisions and alternatives
+   - Risks and tests
+2. Compare the description with the diff. If missing or inconsistent, leave a single top-level review comment requesting clarification.
+3. Review every changed line and surrounding context. Prefer inline comments for concrete issues.
+
+Comment format:
+
+- Prefix severity: [BLOCKER], [MAJOR], [MINOR], [NIT], or [FYI] (if other labels are used, map critical -> [BLOCKER], moderate -> [MAJOR], nit -> [NIT]).
+- Include: Issue -> Why (impact/risk) -> Suggestion (concrete fix or test).
+- Use respectful, code-focused language and explain reasoning.
+- Provide positive feedback when something is notably well done.
+
+Review checklist (from code review best practices):
+
+- Design: fits existing architecture/patterns; avoid over-engineering.
+- Correctness: edge cases, error handling, input validation, concurrency safety.
+- Security/Privacy: secrets, authZ/authN, data exposure, logging.
+- Performance: N+1, unnecessary full scans, expensive operations, caching.
+- Maintainability: duplication, naming clarity, modularity, responsibility boundaries.
+- Tests: appropriate unit/integration/e2e coverage; tests fail when code is wrong.
+- Docs/UX: README/RELEASE/usage steps are consistent and copy-pasteable; UI changes make sense.
+- Consistency/style: follow existing conventions; style-only nits should be labeled [NIT].
+
+Specific checks:
+
+- If the PR claims "no behavior change," verify the diff matches; otherwise raise [MAJOR].
+- For documentation-only PRs, validate that steps are self-consistent and have no contradicting prerequisites.
+
+Library/CLI compatibility checks:
+
+- If a public function signature, exported type, or CLI flag changes, confirm the PR documents the breaking change.
+- Removing or renaming an export requires a CHANGELOG entry and a major version bump justification.
+
+Constraints:
+
+- Copilot reviews are comments only and do not block merging; be explicit about severity.
+- If a change affects user-facing behavior, call out expected impact and any missing tests or docs.
+
+<!-- qfai:language-rules -->

package/assets/init/.github/instructions/principles.instructions.md

@@ -0,0 +1,110 @@
+---
+applyTo: "**/*"
+excludeAgent: "coding-agent"
+---
+
+# Software Engineering Principles Review
+
+ソフトウェア開発・設計の原則やセオリーに照らして、変更が適切かを検証する。
+原則違反を見つけた場合は、どの原則に違反しているかを明示し、改善案を提示すること。
+
+## 設計の基本原則
+
+### SOLID
+
+- **SRP（単一責任）**: 1つのクラス・関数・モジュールが複数の理由で変更される設計になっていないか。名前から責務が一意に読み取れるか。
+- **OCP（開放閉鎖）**: 既存コードを書き換えずに拡張できる設計か。設定値・依存注入・ストラテジーパターン等で切り替え可能か。
+- **LSP（リスコフの置換）**: 派生型やサブタイプが親の契約を破っていないか。事前条件を厳しくしたり事後条件を弱めたりしていないか。
+- **ISP（インターフェース分離）**: クライアントが使わないメソッドへの依存を強要していないか。インターフェースは小さく役割別に分割されているか。
+- **DIP（依存性逆転）**: 高水準モジュールが低水準の具象に直接依存していないか。モジュール境界では抽象（インターフェース）に依存しているか。
+
+### KISS（Keep It Simple）
+
+- 不必要に複雑な実装になっていないか。標準ライブラリや既存ユーティリティで代替可能な自前実装がないか。
+- 認知負荷を下げる書き方か（深いネスト、過度な抽象化、暗黙の挙動を避ける）。
+
+### YAGNI（You Aren't Gonna Need It）
+
+- 現時点で不要な機能・設定・拡張ポイントを先回りして実装していないか。
+- 「将来必要になるかも」で追加されたコードがないか。
+
+### DRY（Don't Repeat Yourself）
+
+- 同一または類似のロジックが複数箇所に重複していないか。共通化・ユーティリティ化の余地がないか。
+- ただし過度な共通化（shotgun surgery を招く）は避け、Rule of Three を意識する。
+
+## モジュール間関係の原則
+
+### Separation of Concerns（関心の分離）
+
+- ビジネスロジック・UI・データアクセス・インフラなど、異なる関心が1つのモジュールに混在していないか。
+
+### Law of Demeter（最小知識の原則）
+
+- `a.getB().getC().doSomething()` のようなメソッドチェーンで遠いオブジェクトに依存していないか。
+- オブジェクトは直接の協力者のみとやり取りしているか。
+
+### Minimise Coupling / Maximise Cohesion
+
+- モジュール間の結合度が不必要に高くないか。変更が他モジュールに波及しにくい設計か。
+- 1つのモジュール内の要素は同じ責務に向かってまとまっているか（高凝集）。
+
+### Composition over Inheritance（継承より合成）
+
+- 「is-a」関係でないのに継承を使っていないか。振る舞いの組み合わせは合成（委譲・ミックスイン）で実現すべきか。
+
+## 堅牢性・安全性の原則
+
+### Fail Fast（早期失敗）
+
+- 不正な入力や前提条件の違反を関数の冒頭で検出し、即座にエラーを返しているか。
+- 無効な状態が伝播して後続処理で不可解な障害を起こす設計になっていないか。
+
+### Defensive Programming（防御的プログラミング）
+
+- 外部入力・API応答・ユーザー入力に対して適切なバリデーションがあるか。
+- null/undefined の安全な処理、リソース解放（finally/using）が適切か。
+
+### Principle of Least Privilege（最小権限）
+
+- 必要最小限のアクセス権限・スコープで動作しているか。過剰な権限を要求していないか。
+
+### Design by Contract（契約による設計）
+
+- 関数の事前条件（引数の制約）・事後条件（戻り値の保証）・不変条件が明確か。
+
+## 可読性・保守性の原則
+
+### Principle of Least Astonishment（驚き最小の原則）
+
+- API・関数の命名と実際の振る舞いが一致しているか。副作用が名前から予測できるか。
+- 言語やフレームワークのイディオムに従っているか。
+
+### Boy Scout Rule（来た時よりきれいに）
+
+- 変更箇所の周辺に明らかな改善余地（命名・型安全・不要コード）があれば併せて改善しているか。
+
+### Avoid Premature Optimization（早すぎる最適化の回避）
+
+- 計測なしに最適化していないか。可読性を犠牲にした最適化がボトルネック以外に適用されていないか。
+
+## 振る舞い・インターフェースの原則
+
+### Tell, Don't Ask（聞くな、命じよ）
+
+- オブジェクトの内部状態を外部で問い合わせて判断するのではなく、オブジェクトに振る舞いを委ねているか。
+
+### Command Query Separation（CQS）
+
+- 状態を変更するメソッド（コマンド）と値を返すメソッド（クエリ）が混在していないか。
+
+### Encapsulation（カプセル化）
+
+- 内部実装の詳細が不必要に外部に露出していないか。公開APIは必要最小限か。
+
+## 適用上の注意
+
+- 原則同士はトレードオフになることがある（DRY vs YAGNI、KISS vs OCP など）。文脈に応じたバランスを評価すること。
+- 原則違反の指摘には [MAJOR] or [MINOR] の重要度を付け、なぜその原則が重要かを簡潔に説明すること。
+
+<!-- qfai:language-rules -->

package/assets/init/.qfai/assistant/agents/facilitator.md

@@ -10,7 +10,7 @@
 - .qfai/assistant/instructions/\*
 - .qfai/assistant/steering/\*
 - .qfai/specs/spec-\*/09_delta.md (Decision Records; check rejected)
-- Existing discussion records under `.qfai/discuss/`
+- Existing discussion records under `.qfai/discussion/`
 - .qfai/require/require-\*/01_Sources.md
 - .qfai/require/require-\*/03_REQ.md
 - .qfai/require/require-\*/08_OQ.md (input gaps ledger)

package/assets/init/.qfai/assistant/agents/interviewer.md

@@ -9,7 +9,7 @@
 - .qfai/assistant/instructions/\*
 - .qfai/assistant/steering/\*
 - .qfai/specs/spec-\*/09_delta.md (Decision Records; check rejected)
-- Existing discussion records under `.qfai/discuss/`
+- Existing discussion records under `.qfai/discussion/`
 - .qfai/require/require-\*/01_Sources.md
 - .qfai/require/require-\*/03_REQ.md
 - .qfai/require/require-\*/08_OQ.md (input gaps ledger)

package/assets/init/.qfai/assistant/agents/option-explorer.md

@@ -15,7 +15,7 @@
 - .qfai/specs/spec-\*/01_Spec.md
 - .qfai/specs/spec-\*/09_delta.md (draft)
 - .qfai/contracts/\*\*
-- Existing discussion records under `.qfai/discuss/`
+- Existing discussion records under `.qfai/discussion/`
 
 ## Deliverables (MANDATORY)
 

package/assets/init/.qfai/assistant/agents/oq-harvester.md

@@ -16,7 +16,7 @@
 - .qfai/specs/spec-\*/01_Spec.md
 - .qfai/specs/spec-\*/09_delta.md
 - .qfai/contracts/\*\*
-- Existing discussion records under `.qfai/discuss/`
+- Existing discussion records under `.qfai/discussion/`
 
 ## Deliverables (MANDATORY)
 

package/assets/init/.qfai/assistant/agents/requirements-analyst.md

@@ -12,7 +12,7 @@
 - .qfai/require/require-\*/01_Sources.md
 - .qfai/require/require-\*/03_REQ.md
 - .qfai/require/require-\*/08_OQ.md (input gaps ledger)
-- Discussion records under `.qfai/discuss/`
+- Discussion records under `.qfai/discussion/`
 - .qfai/specs/spec-\*/01_Spec.md (if available)
 
 ## Deliverables (MANDATORY)

package/assets/init/.qfai/assistant/agents/researcher.md

@@ -10,7 +10,7 @@
 - .qfai/assistant/steering/\*
 - .qfai/specs/spec-\*/09_delta.md (Decision Records; check rejected)
 - User-provided idea/problem statement
-- Existing discussion records under `.qfai/discuss/`
+- Existing discussion records under `.qfai/discussion/`
 
 ## Deliverables (MANDATORY)
 

package/assets/init/.qfai/assistant/skills/qfai-sdd/SKILL.md

@@ -67,7 +67,7 @@ Then read inputs in this order:
 - P1: `.qfai/assistant/instructions/*`
 - P2: `.qfai/assistant/steering/*`
 - P3: existing `.qfai/specs/<spec-id>/**` (if updating)
-- P4: `.qfai/discuss/**`, `.qfai/require/**`, `.qfai/contracts/**`
+- P4: `.qfai/discussion/**`, `.qfai/require/**`, `.qfai/contracts/**`
 
 ## Sub-agent Delegation (MANDATORY)
 

package/dist/cli/index.cjs

@@ -1522,8 +1522,8 @@ var import_promises7 = require("fs/promises");
 var import_node_path8 = __toESM(require("path"), 1);
 var import_node_url2 = require("url");
 async function resolveToolVersion() {
-  if ("1.6.2".length > 0) {
-    return "1.6.2";
+  if ("1.6.4".length > 0) {
+    return "1.6.4";
   }
   try {
     const packagePath = resolvePackageJsonPath();
@@ -2758,6 +2758,17 @@ async function runInit(options) {
   await ensureRequiredEmptyScaffoldDirs(destQfai, options.dryRun);
   const removedLegacySkills = options.force ? await pruneLegacySkillFiles(destRoot, options.dryRun) : [];
   const removed = [...removedLegacySkills, ...wrappersResult.removed];
+  const expectedInstructionsDir = import_node_path14.default.join(destRoot, ".github", "instructions");
+  const instructionsCreated = wrappersResult.copied.some(
+    (p) => import_node_path14.default.basename(p).endsWith(".instructions.md") && import_node_path14.default.dirname(p) === expectedInstructionsDir
+  );
+  if (instructionsCreated && !options.dryRun) {
+    info("");
+    info("Copilot \u30B3\u30FC\u30C9\u30EC\u30D3\u30E5\u30FC\u7528 instructions \u3092\u4F5C\u6210\u3057\u307E\u3057\u305F\u3002");
+    info("\u6709\u52B9\u5316: PR \u30B3\u30E1\u30F3\u30C8\u3067 '@github-copilot review' \u3092\u5B9F\u884C\u3059\u308B\u304B\u3001");
+    info("GitHub Actions \u30EF\u30FC\u30AF\u30D5\u30ED\u30FC\u3067\u81EA\u52D5\u30EC\u30D3\u30E5\u30FC\u3092\u8A2D\u5B9A\u3057\u3066\u304F\u3060\u3055\u3044\u3002");
+    info("\u53C2\u8003: https://docs.github.com/en/copilot/using-github-copilot/code-review");
+  }
   report(
     [...rootResult.copied, ...qfaiResult.copied, ...skillsResult.copied, ...wrappersResult.copied],
     [
@@ -2841,6 +2852,20 @@ async function exists7(target) {
     return false;
   }
 }
+function isEnoent(err) {
+  return typeof err === "object" && err !== null && err.code === "ENOENT";
+}
+async function pathExists(target) {
+  try {
+    await (0, import_promises12.lstat)(target);
+    return true;
+  } catch (err) {
+    if (isEnoent(err)) {
+      return false;
+    }
+    throw err;
+  }
+}
 async function configureGitSymlinks(destRoot, dryRun) {
   try {
     await execAsync("git rev-parse --git-dir", { cwd: destRoot });
@@ -2905,6 +2930,31 @@ async function syncIntegrationWrappers(assistantAssetsDir, destRoot, options) {
       await (0, import_promises12.writeFile)(copilotDest, buildCopilotInstructions(), "utf-8");
     }
   }
+  const instructionsFiles = ["code-review.instructions.md", "principles.instructions.md"];
+  for (const fileName of instructionsFiles) {
+    const dest = import_node_path14.default.join(destRoot, ".github", "instructions", fileName);
+    const alreadyExists = await pathExists(dest);
+    if (alreadyExists) {
+      skipped.push(dest);
+    } else {
+      copied.push(dest);
+      if (!options.dryRun) {
+        await (0, import_promises12.mkdir)(import_node_path14.default.dirname(dest), { recursive: true });
+        const templateSrc = import_node_path14.default.join(getInitAssetsDir(), ".github", "instructions", fileName);
+        let content;
+        try {
+          content = await (0, import_promises12.readFile)(templateSrc, "utf-8");
+        } catch (err) {
+          const code = typeof err === "object" && err !== null ? err.code : void 0;
+          const detail = err instanceof Error ? err.message : String(err);
+          throw new Error(
+            `instructions \u30C6\u30F3\u30D7\u30EC\u30FC\u30C8\u306E\u8AAD\u307F\u8FBC\u307F\u306B\u5931\u6557\u3057\u307E\u3057\u305F: ${templateSrc} (${code ?? detail})\u3002\u30D1\u30C3\u30B1\u30FC\u30B8\u304C\u6B63\u3057\u304F\u30A4\u30F3\u30B9\u30C8\u30FC\u30EB\u3055\u308C\u3066\u3044\u308B\u304B\u78BA\u8A8D\u3057\u3066\u304F\u3060\u3055\u3044\u3002`
+          );
+        }
+        await (0, import_promises12.writeFile)(dest, content, "utf-8");
+      }
+    }
+  }
   const skillResult = await createSkillSymlinks(destRoot, skills, options);
   copied.push(...skillResult.copied);
   skipped.push(...skillResult.skipped);