@artefact-ci/sdd 0.2.0 → 0.2.1

package/README.md

@@ -49,7 +49,7 @@ L'installeur enregistre la version de la méthode installée et sa source dans `
 
 ```json
 {
-  "version": "0.2.0",
+  "version": "0.2.1",
   "source": "release"
 }
 ```
@@ -64,4 +64,4 @@ Un projet installé avant ce fichier (phase 4, avec `--source`) a déjà les com
 
 ## Ce que `update` touche
 
-Il réinstalle l'extension et le preset, puis régénère les fichiers d'agent et réapplique les hooks. Sans `--source`, il les prend dans les zips de la release de sa version : `extension add --from <url du zip> --force` et `preset update artefact-sdd --from <url du zip>`. Avec `--source`, il les prend dans le clone en `--dev` : `extension add --dev … --force` et `preset update artefact-sdd --dev …` (soit un retrait puis un ajout). Si un téléchargement échoue (réseau coupé, version sans release), l'installeur s'arrête en citant l'URL du zip : relancer la même commande une fois le réseau rétabli. `installed_at` change dans les registres de `.specify/` : c'est attendu. Il ne touche jamais `stack.yml` ni la constitution, et s'arrête si `.specify/` est absent.
+Il réinstalle l'extension et le preset, puis régénère les fichiers d'agent et réapplique les hooks. Quand `.specify/integration.json` liste plusieurs outils, il lance `specify integration use` pour chaque outil autre que celui par défaut, puis pour l'outil par défaut : Spec Kit n'enregistre les commandes de l'extension et du preset que pour l'outil sélectionné. L'outil par défaut ne change pas, même si un `use` échoue, et les scripts de `.specify/scripts/` gardent leurs droits. Sans `--source`, il les prend dans les zips de la release de sa version : `extension add --from <url du zip> --force` et `preset update artefact-sdd --from <url du zip>`. Avec `--source`, il les prend dans le clone en `--dev` : `extension add --dev … --force` et `preset update artefact-sdd --dev …` (soit un retrait puis un ajout). Si un téléchargement échoue (réseau coupé, version sans release), l'installeur s'arrête en citant l'URL du zip : relancer la même commande une fois le réseau rétabli. `installed_at` change dans les registres de `.specify/` : c'est attendu. Il ne touche jamais `stack.yml` ni la constitution, et s'arrête si `.specify/` est absent.

package/dist/agents.mjs

@@ -24,6 +24,14 @@ export function readIntegrations(root) {
         return json.installed_integrations;
     return json.integration ? [json.integration] : [];
 }
+/** Intégration par défaut de `.specify/integration.json` (`default_integration`, sinon `integration`) ; null si aucune. */
+export function readDefaultIntegration(root) {
+    const file = join(root, '.specify', 'integration.json');
+    if (!existsSync(file))
+        return null;
+    const json = JSON.parse(readFileSync(file, 'utf8'));
+    return json.default_integration ?? json.integration ?? null;
+}
 function body(spec) {
     const lines = [
         `Lis \`${INSTRUCTIONS_DIR}/${spec.name}.md\` et applique-le à la lettre : méthode, grille et format du rapport.`,

package/dist/install.mjs

@@ -1,6 +1,6 @@
-import { existsSync, readdirSync, readFileSync } from 'node:fs';
+import { chmodSync, existsSync, readdirSync, readFileSync, statSync } from 'node:fs';
 import { basename, join } from 'node:path';
-import { agentFiles, readAgentSpecs, readIntegrations, skippedAgents } from "./agents.mjs";
+import { agentFiles, readAgentSpecs, readDefaultIntegration, readIntegrations, skippedAgents } from "./agents.mjs";
 import { compareVersions, readInstalled, writeInstalled } from "./artefact.mjs";
 import { baseConstitution, isBlankConstitution } from "./constitution.mjs";
 import { applyGitHooks } from "./hooks.mjs";
@@ -62,6 +62,62 @@ async function branches(o) {
     const defaultBranch = head?.startsWith('refs/remotes/origin/') ? head.slice('refs/remotes/origin/'.length) : undefined;
     return { names: [...new Set(refs)], ...(defaultBranch ? { defaultBranch } : {}) };
 }
+const SCRIPTS = '.specify/scripts';
+/** Droits de chaque fichier sous `.specify/scripts/` (chemin absolu → mode). */
+function scriptModes(root) {
+    const modes = new Map();
+    const walk = (dir) => {
+        if (!existsSync(dir))
+            return;
+        for (const e of readdirSync(dir, { withFileTypes: true })) {
+            const p = join(dir, e.name);
+            if (e.isDirectory())
+                walk(p);
+            else if (e.isFile())
+                modes.set(p, statSync(p).mode & 0o777);
+        }
+    };
+    walk(join(root, SCRIPTS));
+    return modes;
+}
+/**
+ * Spec Kit 1.3.2 n'enregistre les commandes d'une extension ou d'un preset que pour l'intégration
+ * par défaut ; une autre ne les reçoit qu'à `specify integration use`. On sélectionne donc chaque
+ * intégration secondaire, puis on revient toujours à celle par défaut, même après un échec.
+ * `use` réécrit les scripts partagés sans leur droit d'exécution : on remet les droits d'avant.
+ */
+async function registerSecondary(o, integrations) {
+    const fallback = readDefaultIntegration(o.root);
+    if (fallback === null || !integrations.includes(fallback))
+        return [];
+    const others = integrations.filter((i) => i !== fallback);
+    if (others.length === 0)
+        return [];
+    const modes = scriptModes(o.root);
+    let failed;
+    for (const integration of others) {
+        try {
+            await specify(o.runner, o.root, ['integration', 'use', integration]);
+        }
+        catch (e) {
+            failed = { integration, error: e };
+            break;
+        }
+    }
+    try {
+        await specify(o.runner, o.root, ['integration', 'use', fallback]);
+    }
+    finally {
+        for (const [file, mode] of modes) {
+            if (existsSync(file) && (statSync(file).mode & 0o777) !== mode)
+                chmodSync(file, mode);
+        }
+    }
+    if (failed) {
+        throw new SetupError(`impossible d'enregistrer les commandes de la méthode pour ${failed.integration} : ${failed.error.message} ; ${fallback} reste l'outil par défaut, relance update`);
+    }
+    return others;
+}
 async function isDirty(o) {
     return ((await git(o, ['status', '--porcelain'])) ?? '').trim() !== '';
 }
@@ -101,6 +157,8 @@ async function common(o, mode) {
     catch (e) {
         throw new SetupError(`.specify/integration.json illisible : ${e.message}`);
     }
+    // Seulement quand un composant vient d'être (ré)installé : un second init ne lance rien.
+    const registeredFor = installed.length > 0 ? await registerSecondary(o, integrations) : [];
     const specs = readAgentSpecs(root);
     const agents = [];
     const agentsSkipped = [];
@@ -127,6 +185,7 @@ async function common(o, mode) {
         agents,
         agentsSkipped,
         noAgentGenerator,
+        registeredFor,
         legacy: legacyAgents(root),
     };
 }
@@ -141,7 +200,8 @@ export async function init(o) {
     }
     const noneInstalled = !registered(root, 'presets', 'artefact-sdd') && !registered(root, 'extensions', 'artefact-product');
     const dirty = await isDirty(o);
-    if (!existsSync(join(root, '.specify'))) {
+    const hadSpecify = existsSync(join(root, '.specify'));
+    if (!hadSpecify) {
         const args = ['init', '--here', '--script', 'sh', '--non-interactive', '--ignore-agent-tools'];
         args.push('--integration', o.integration ?? 'claude');
         // Spec Kit 1.3.2 compte `.git` comme contenu : sans --force, il refuse un dossier non vide.
@@ -169,7 +229,22 @@ export async function init(o) {
         writeIfChanged(constFile, baseConstitution(readFileSync(template, 'utf8'), basename(root)));
         constitution = 'written';
     }
-    return { mode: 'init', ...done, stack, constitution, dirty, version: { installed: record ? current : null, previous, downgrade: false } };
+    // Spec Kit n'installe une intégration qu'à son init : sur un projet déjà initialisé, --integration ne fait rien.
+    let integrationIgnored;
+    if (hadSpecify && o.integration !== undefined) {
+        const installed = readIntegrations(root);
+        if (!installed.includes(o.integration))
+            integrationIgnored = { requested: o.integration, installed };
+    }
+    return {
+        mode: 'init',
+        ...done,
+        stack,
+        constitution,
+        dirty,
+        ...(integrationIgnored ? { integrationIgnored } : {}),
+        version: { installed: record ? current : null, previous, downgrade: false }
+    };
 }
 export async function update(o) {
     if (!(await hasUv(o.runner)))

package/dist/report.mjs

@@ -1,3 +1,4 @@
+import { SPECIFY_VERSION } from "./specify.mjs";
 const LEGACY_STATE = {
     unchanged: 'identique au boilerplate, tu peux le supprimer',
     /** Variante quand des fichiers citent encore docs/agents/. */
@@ -17,6 +18,9 @@ export function formatReport(r) {
     }
     if (r.skipped.length > 0)
         out.push(`  Déjà installé : ${list(r.skipped)}`);
+    if (r.registeredFor.length > 0) {
+        out.push(`  Commandes de la méthode enregistrées aussi pour : ${list(r.registeredFor)} (outil par défaut inchangé)`);
+    }
     const { installed, previous, downgrade } = r.version;
     if (installed === null) {
         out.push('  Version installée inconnue : lance update pour aligner tous les composants');
@@ -50,7 +54,7 @@ export function formatReport(r) {
             out.push(`  ${name} : pas de générateur de fichiers d'agent, la relecture se fait en contexte neuf`);
         }
         for (const { integration, names } of r.agentsSkipped) {
-            out.push(`  ${integration}, ${list(names)} : pas de fichier d'agent pour cet outil : la commande fera la relecture en nouvelle session`);
+            out.push(`  ${integration} : pas de fichier d'agent ${list(names)} pour cet outil : la commande fera ce travail en nouvelle session`);
         }
     }
     if (r.stack) {
@@ -74,6 +78,12 @@ export function formatReport(r) {
             out.push(`  Citent encore docs/agents/ : ${list(r.legacy.mentions)}`);
         }
     }
+    if (r.integrationIgnored) {
+        const { requested, installed } = r.integrationIgnored;
+        const current = installed.length > 0 ? list(installed) : 'aucune intégration';
+        const specify = `uvx --from specify-cli==${SPECIFY_VERSION} specify integration`;
+        out.push('', `Attention : --integration ${requested} ignoré : Spec Kit était déjà installé ici avec ${current}.`, `  pour garder ${current} et ajouter ${requested} : ${specify} install ${requested} --script sh --force`, `  pour remplacer ${current} par ${requested} : ${specify} switch ${requested}`, "  puis : npx @artefact-ci/sdd update (voir « Changer ou ajouter un outil » dans le README de speckit-artefact)");
+    }
     if (r.dirty) {
         out.push('', "Attention : le dépôt avait des changements non commités avant l'installation ; relis le diff avant de commiter.");
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@artefact-ci/sdd",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "Installeur de la méthode Artefact CI pour Spec Kit (preset artefact-sdd, extension artefact-product)",
   "license": "MIT",
   "repository": {