cc-ship 0.0.7 → 0.0.8

package/commands/ship/verify.md

@@ -9,11 +9,20 @@ Lance l'agent ship-verifier pour verifier l'implementation d'un scope contre ses
 
 ## Instructions
 
+### Resolution du projet (PREMIERE ETAPE)
+
+1. Lire `cc-ship.json` a la racine du repo
+2. Resoudre le chemin : `{projectsDir}/{currentProject}/`
+3. Si `cc-ship.json` n'existe pas OU `currentProject` est null → ERREUR : "Lance `/ship:init` ou `/ship:next` d'abord pour initialiser un projet."
+4. Utiliser ce chemin partout au lieu de `.ship/`
+
+### Lancement de l'agent
+
 Tu dois lancer l'agent `ship-verifier` en utilisant le tool Task avec les parametres suivants:
 
 ```
 subagent_type: ship-verifier
-prompt: [Le package et/ou scope a verifier]
+prompt: "Le chemin du projet est {projectPath}. [Le package et/ou scope a verifier]"
 ```
 
 ## Prerequis
@@ -22,8 +31,8 @@ Avant de lancer l'agent, verifie que ces fichiers existent :
 
 | Fichier | Obligatoire | Cree par |
 |---------|-------------|----------|
-| `.ship/packages/<nom>/package.md` | Oui | shaper |
-| `.ship/packages/<nom>/verification.md` | Oui | shaper |
+| `{projectPath}/packages/<nom>/package.md` | Oui | shaper |
+| `{projectPath}/packages/<nom>/verification.md` | Oui | shaper |
 
 ### Verification du status
 
@@ -89,17 +98,11 @@ Quand l'agent te retourne une question pour l'utilisateur (verification manuelle
 
 Verifie le scope courant du package actif (detecte automatiquement).
 
-```
-/ship:verify .ship/packages/auth/package.md
-```
-
-Verifie le scope courant du package specifie.
-
 ```
 /ship:verify auth
 ```
 
-Raccourci : verifie le scope courant du package "auth".
+Verifie le scope courant du package "auth".
 
 ## Output
 

package/hooks/check-agent-completion.js

@@ -0,0 +1,64 @@
+#!/usr/bin/env node
+
+/**
+ * Hook: Verify agent completion state (SubagentStop).
+ * Checks that agents left packages in a consistent state.
+ *
+ * - ship-executor: at least one package should have status "executed"
+ * - ship-verifier: no package should still have status "verifying"
+ *
+ * Exit 0 = OK, Exit 2 = inconsistent state (warns the user).
+ */
+
+const { findPackages } = require('./lib/find-packages');
+
+async function main() {
+  // Read stdin (SubagentStop JSON)
+  let input = '';
+  for await (const chunk of process.stdin) {
+    input += chunk;
+  }
+
+  let data;
+  try {
+    data = JSON.parse(input);
+  } catch {
+    process.exit(0);
+  }
+
+  const agentName = data.agent_name || data.subagent_type || '';
+
+  const packages = findPackages();
+  if (!packages || packages.length === 0) {
+    process.exit(0); // No packages to check
+  }
+
+  // Check executor completion
+  if (agentName === 'ship-executor') {
+    const hasExecuted = packages.some(p => p.metadata.status === 'executed');
+    if (!hasExecuted) {
+      process.stderr.write(
+        "L'executor a termine sans mettre a jour le status d'aucun package a \"executed\".\n" +
+        "Verifiez que le frontmatter du package.md a ete mis a jour correctement.\n"
+      );
+      process.exit(2);
+    }
+  }
+
+  // Check verifier completion
+  if (agentName === 'ship-verifier') {
+    const stillVerifying = packages.filter(p => p.metadata.status === 'verifying');
+    if (stillVerifying.length > 0) {
+      const names = stillVerifying.map(p => p.name).join(', ');
+      process.stderr.write(
+        `Le verifier a termine sans conclure pour: ${names}\n` +
+        'Le status est encore "verifying" au lieu de "done" ou "executing".\n'
+      );
+      process.exit(2);
+    }
+  }
+
+  process.exit(0);
+}
+
+main();

package/hooks/lib/config.js

@@ -0,0 +1,34 @@
+const fs = require('fs');
+const path = require('path');
+
+/**
+ * Reads cc-ship.json and resolves the project path.
+ * Returns { projectsDir, currentProject, projectPath } or null if not configured.
+ */
+function getConfig() {
+  const configPath = path.join(process.cwd(), 'cc-ship.json');
+
+  if (!fs.existsSync(configPath)) {
+    return null;
+  }
+
+  let config;
+  try {
+    config = JSON.parse(fs.readFileSync(configPath, 'utf-8'));
+  } catch {
+    return null;
+  }
+
+  const projectsDir = config.projectsDir || '.ship';
+  const currentProject = config.currentProject || null;
+
+  if (!currentProject) {
+    return null;
+  }
+
+  const projectPath = path.join(process.cwd(), projectsDir, currentProject);
+
+  return { projectsDir, currentProject, projectPath };
+}
+
+module.exports = { getConfig };

package/hooks/lib/find-packages.js

@@ -0,0 +1,37 @@
+const fs = require('fs');
+const path = require('path');
+const { getConfig } = require('./config');
+const { parseFrontmatter } = require('./parse-frontmatter');
+
+// Scans {projectPath}/packages/*/package.md and returns parsed metadata.
+// Returns null if config is not available, otherwise array of packages.
+function findPackages() {
+  const config = getConfig();
+  if (!config) return null;
+
+  const packagesDir = path.join(config.projectPath, 'packages');
+  if (!fs.existsSync(packagesDir)) return [];
+
+  const entries = fs.readdirSync(packagesDir, { withFileTypes: true });
+  const packages = [];
+
+  for (const entry of entries) {
+    if (!entry.isDirectory()) continue;
+
+    const packageMdPath = path.join(packagesDir, entry.name, 'package.md');
+    if (!fs.existsSync(packageMdPath)) continue;
+
+    const content = fs.readFileSync(packageMdPath, 'utf-8');
+    const metadata = parseFrontmatter(content) || {};
+
+    packages.push({
+      path: packageMdPath,
+      name: entry.name,
+      metadata,
+    });
+  }
+
+  return packages;
+}
+
+module.exports = { findPackages };

package/hooks/lib/parse-frontmatter.js

@@ -0,0 +1,135 @@
+/**
+ * Minimal YAML-like frontmatter parser (no npm dependencies).
+ * Supports: key: value, key: ~, simple arrays, one level of nested objects.
+ */
+
+/**
+ * Parse frontmatter from markdown content.
+ * Finds the first block between --- delimiters (may appear after a H1).
+ * @param {string} content - The markdown file content
+ * @returns {object|null} Parsed frontmatter object, or null if not found
+ */
+function parseFrontmatter(content) {
+  const lines = content.split('\n');
+
+  let startIdx = -1;
+  let endIdx = -1;
+
+  for (let i = 0; i < lines.length; i++) {
+    if (lines[i].trim() === '---') {
+      if (startIdx === -1) {
+        startIdx = i;
+      } else {
+        endIdx = i;
+        break;
+      }
+    }
+  }
+
+  if (startIdx === -1 || endIdx === -1) {
+    return null;
+  }
+
+  const yamlLines = lines.slice(startIdx + 1, endIdx);
+  return parseYaml(yamlLines);
+}
+
+/**
+ * Minimal YAML parser supporting:
+ * - key: value (strings, numbers, booleans)
+ * - key: ~ (null)
+ * - arrays (- item)
+ * - one level of nested objects
+ */
+function parseYaml(lines) {
+  const result = {};
+  let currentKey = null;
+  let currentArray = null;
+  let currentObject = null;
+  let inNestedObject = false;
+
+  for (const line of lines) {
+    // Skip empty lines and comments
+    if (line.trim() === '' || line.trim().startsWith('#')) {
+      continue;
+    }
+
+    const indent = line.length - line.trimStart().length;
+    const trimmed = line.trim();
+
+    // Array item at indent level 2+ (belongs to current key)
+    if (trimmed.startsWith('- ') && indent >= 2 && currentKey) {
+      if (!currentArray) {
+        currentArray = [];
+      }
+      if (inNestedObject) {
+        inNestedObject = false;
+        currentObject = null;
+      }
+      currentArray.push(parseValue(trimmed.slice(2).trim()));
+      result[currentKey] = currentArray;
+      continue;
+    }
+
+    // Nested object key: value at indent level 2+
+    if (indent >= 2 && trimmed.includes(':') && currentKey && !trimmed.startsWith('- ')) {
+      if (!currentObject) {
+        currentObject = {};
+        inNestedObject = true;
+      }
+      const colonIdx = trimmed.indexOf(':');
+      const nestedKey = trimmed.slice(0, colonIdx).trim();
+      const nestedVal = trimmed.slice(colonIdx + 1).trim();
+      if (nestedVal !== '') {
+        currentObject[nestedKey] = parseValue(nestedVal);
+      }
+      result[currentKey] = currentObject;
+      continue;
+    }
+
+    // Top-level key: value
+    if (indent === 0 && trimmed.includes(':')) {
+      // Flush previous array/object
+      currentArray = null;
+      currentObject = null;
+      inNestedObject = false;
+
+      const colonIdx = trimmed.indexOf(':');
+      const key = trimmed.slice(0, colonIdx).trim();
+      const val = trimmed.slice(colonIdx + 1).trim();
+
+      currentKey = key;
+
+      if (val === '') {
+        // Could be followed by array or nested object — will be set by next lines
+        result[key] = null;
+      } else {
+        result[key] = parseValue(val);
+      }
+    }
+  }
+
+  return result;
+}
+
+/**
+ * Parse a single YAML value.
+ */
+function parseValue(val) {
+  if (val === '~' || val === 'null') return null;
+  if (val === 'true') return true;
+  if (val === 'false') return false;
+  if (/^-?\d+$/.test(val)) return parseInt(val, 10);
+  if (/^-?\d+\.\d+$/.test(val)) return parseFloat(val);
+  // Remove surrounding quotes if present
+  if ((val.startsWith('"') && val.endsWith('"')) || (val.startsWith("'") && val.endsWith("'"))) {
+    return val.slice(1, -1);
+  }
+  // Handle inline arrays like [item1, item2]
+  if (val.startsWith('[') && val.endsWith(']')) {
+    return val.slice(1, -1).split(',').map(s => parseValue(s.trim()));
+  }
+  return val;
+}
+
+module.exports = { parseFrontmatter };

package/hooks/validate-frontmatter.js

@@ -0,0 +1,109 @@
+#!/usr/bin/env node
+
+/**
+ * Hook: Validate frontmatter of package.md files after Write/Edit.
+ * Checks that status and other fields are valid.
+ *
+ * Exit 0 = OK, Exit 2 = validation error (blocks the tool use).
+ */
+
+const fs = require('fs');
+const path = require('path');
+const { getConfig } = require('./lib/config');
+const { parseFrontmatter } = require('./lib/parse-frontmatter');
+
+const VALID_STATUSES = ['pending', 'shaping', 'shaped', 'executing', 'executed', 'verifying', 'done'];
+
+async function main() {
+  // Read stdin (PostToolUse JSON)
+  let input = '';
+  for await (const chunk of process.stdin) {
+    input += chunk;
+  }
+
+  let data;
+  try {
+    data = JSON.parse(input);
+  } catch {
+    process.exit(0); // Can't parse input, skip
+  }
+
+  const filePath = data.tool_input?.file_path || data.tool_input?.path || null;
+  if (!filePath) {
+    process.exit(0);
+  }
+
+  // Check if this is a package.md in the project
+  const config = getConfig();
+  if (!config) {
+    process.exit(0); // No config, skip
+  }
+
+  const packagesDir = path.join(config.projectPath, 'packages');
+  const normalizedFile = path.resolve(filePath);
+  const normalizedPackages = path.resolve(packagesDir);
+
+  // Must be inside {projectPath}/packages/*/package.md
+  if (!normalizedFile.startsWith(normalizedPackages)) {
+    process.exit(0);
+  }
+
+  if (path.basename(normalizedFile) !== 'package.md') {
+    process.exit(0);
+  }
+
+  // Read and parse the file
+  if (!fs.existsSync(normalizedFile)) {
+    process.exit(0);
+  }
+
+  const content = fs.readFileSync(normalizedFile, 'utf-8');
+  const frontmatter = parseFrontmatter(content);
+
+  if (!frontmatter) {
+    process.stderr.write('Frontmatter invalide : aucun bloc --- trouve dans package.md\n');
+    process.exit(2);
+  }
+
+  const errors = [];
+
+  // Validate status
+  if (!frontmatter.status) {
+    errors.push('Le champ "status" est absent du frontmatter');
+  } else if (!VALID_STATUSES.includes(frontmatter.status)) {
+    errors.push(`Status invalide: "${frontmatter.status}". Valeurs acceptees: ${VALID_STATUSES.join(', ')}`);
+  }
+
+  // Validate current_scope (string or null)
+  if (frontmatter.current_scope !== undefined && frontmatter.current_scope !== null && typeof frontmatter.current_scope !== 'string') {
+    errors.push(`current_scope doit etre une string ou null, recu: ${typeof frontmatter.current_scope}`);
+  }
+
+  // Validate scopes_completed (array or absent)
+  if (frontmatter.scopes_completed !== undefined && frontmatter.scopes_completed !== null && !Array.isArray(frontmatter.scopes_completed)) {
+    errors.push('scopes_completed doit etre un tableau');
+  }
+
+  // Validate last_verification (object with date/result/failed_criteria or absent)
+  if (frontmatter.last_verification !== undefined && frontmatter.last_verification !== null) {
+    if (typeof frontmatter.last_verification !== 'object') {
+      errors.push('last_verification doit etre un objet avec date/result/failed_criteria');
+    }
+  }
+
+  // Validate verification_attempts (integer >= 0 or absent)
+  if (frontmatter.verification_attempts !== undefined && frontmatter.verification_attempts !== null) {
+    if (typeof frontmatter.verification_attempts !== 'number' || frontmatter.verification_attempts < 0 || !Number.isInteger(frontmatter.verification_attempts)) {
+      errors.push('verification_attempts doit etre un entier >= 0');
+    }
+  }
+
+  if (errors.length > 0) {
+    process.stderr.write(`Frontmatter invalide dans package.md:\n${errors.map(e => `  - ${e}`).join('\n')}\n`);
+    process.exit(2);
+  }
+
+  process.exit(0);
+}
+
+main();

package/hooks/validate-transition.js

@@ -0,0 +1,136 @@
+#!/usr/bin/env node
+
+/**
+ * Hook: Validate status transitions in package.md files.
+ * Each agent type has its own set of valid transitions.
+ *
+ * Usage: node validate-transition.js --agent=shaper|executor|verifier
+ *
+ * Exit 0 = OK, Exit 2 = invalid transition (blocks the tool use).
+ */
+
+const fs = require('fs');
+const path = require('path');
+const { execSync } = require('child_process');
+const { getConfig } = require('./lib/config');
+const { parseFrontmatter } = require('./lib/parse-frontmatter');
+
+const VALID_TRANSITIONS = {
+  shaper: [
+    [null, 'pending'],
+    [null, 'shaping'],
+    ['pending', 'shaping'],
+    ['shaping', 'shaped'],
+  ],
+  executor: [
+    ['shaped', 'executing'],
+    ['executing', 'executed'],
+  ],
+  verifier: [
+    ['executed', 'verifying'],
+    ['verifying', 'done'],
+    ['verifying', 'executing'],
+  ],
+};
+
+function getAgentArg() {
+  for (const arg of process.argv.slice(2)) {
+    if (arg.startsWith('--agent=')) {
+      return arg.split('=')[1];
+    }
+  }
+  return null;
+}
+
+function getOldStatus(filePath) {
+  try {
+    // Get the file content from HEAD (last committed version)
+    const relativePath = path.relative(process.cwd(), filePath).replace(/\\/g, '/');
+    const oldContent = execSync(`git show HEAD:${relativePath}`, {
+      encoding: 'utf-8',
+      stdio: ['pipe', 'pipe', 'pipe'],
+    });
+    const fm = parseFrontmatter(oldContent);
+    return fm ? fm.status || null : null;
+  } catch {
+    // File is new (not in git yet)
+    return null;
+  }
+}
+
+async function main() {
+  const agent = getAgentArg();
+  if (!agent || !VALID_TRANSITIONS[agent]) {
+    process.exit(0); // Unknown agent, skip
+  }
+
+  // Read stdin (PostToolUse JSON)
+  let input = '';
+  for await (const chunk of process.stdin) {
+    input += chunk;
+  }
+
+  let data;
+  try {
+    data = JSON.parse(input);
+  } catch {
+    process.exit(0);
+  }
+
+  const filePath = data.tool_input?.file_path || data.tool_input?.path || null;
+  if (!filePath) {
+    process.exit(0);
+  }
+
+  // Check if this is a package.md in the project
+  const config = getConfig();
+  if (!config) {
+    process.exit(0);
+  }
+
+  const packagesDir = path.join(config.projectPath, 'packages');
+  const normalizedFile = path.resolve(filePath);
+  const normalizedPackages = path.resolve(packagesDir);
+
+  if (!normalizedFile.startsWith(normalizedPackages)) {
+    process.exit(0);
+  }
+
+  if (path.basename(normalizedFile) !== 'package.md') {
+    process.exit(0);
+  }
+
+  if (!fs.existsSync(normalizedFile)) {
+    process.exit(0);
+  }
+
+  // Get old and new status
+  const oldStatus = getOldStatus(normalizedFile);
+  const content = fs.readFileSync(normalizedFile, 'utf-8');
+  const newFm = parseFrontmatter(content);
+  const newStatus = newFm ? newFm.status || null : null;
+
+  // If status hasn't changed, it's fine
+  if (oldStatus === newStatus) {
+    process.exit(0);
+  }
+
+  // Check if transition is valid
+  const transitions = VALID_TRANSITIONS[agent];
+  const isValid = transitions.some(([from, to]) => from === oldStatus && to === newStatus);
+
+  if (!isValid) {
+    const fromLabel = oldStatus || 'null';
+    const toLabel = newStatus || 'null';
+    const allowed = transitions.map(([f, t]) => `${f || 'null'} → ${t}`).join(', ');
+    process.stderr.write(
+      `Transition de status invalide pour ${agent}: ${fromLabel} → ${toLabel}\n` +
+      `Transitions autorisees: ${allowed}\n`
+    );
+    process.exit(2);
+  }
+
+  process.exit(0);
+}
+
+main();

package/hooks-settings.json

@@ -0,0 +1,26 @@
+{
+  "hooks": {
+    "PostToolUse": [
+      {
+        "matcher": "Write|Edit",
+        "hooks": [
+          {
+            "type": "command",
+            "command": "node .claude/hooks/validate-frontmatter.js"
+          }
+        ]
+      }
+    ],
+    "SubagentStop": [
+      {
+        "matcher": "ship-executor|ship-verifier",
+        "hooks": [
+          {
+            "type": "command",
+            "command": "node .claude/hooks/check-agent-completion.js"
+          }
+        ]
+      }
+    ]
+  }
+}

package/package.json

@@ -1,31 +1,33 @@
-{
-  "name": "cc-ship",
-  "version": "0.0.7",
-  "description": "Architecture 3 couches pour Claude Code: Commands -> Agents -> Skills",
-  "bin": {
-    "cc-ship": "bin/install.js"
-  },
-  "files": [
-    "bin",
-    "commands",
-    "agents",
-    "skills"
-  ],
-  "keywords": [
-    "claude",
-    "claude-code",
-    "ai",
-    "brainstorming",
-    "shaping",
-    "productivity"
-  ],
-  "author": "",
-  "license": "MIT",
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/vdstack/cc-ship"
-  },
-  "engines": {
-    "node": ">=14.0.0"
-  }
-}
+{
+  "name": "cc-ship",
+  "version": "0.0.8",
+  "description": "Architecture 3 couches pour Claude Code: Commands -> Agents -> Skills",
+  "bin": {
+    "cc-ship": "bin/install.js"
+  },
+  "files": [
+    "bin",
+    "commands",
+    "agents",
+    "skills",
+    "hooks",
+    "hooks-settings.json"
+  ],
+  "keywords": [
+    "claude",
+    "claude-code",
+    "ai",
+    "brainstorming",
+    "shaping",
+    "productivity"
+  ],
+  "author": "",
+  "license": "MIT",
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/vdstack/cc-ship"
+  },
+  "engines": {
+    "node": ">=14.0.0"
+  }
+}