devmethod-ai 0.2.0 → 0.3.1

package/examples/review/review.json

@@ -0,0 +1,199 @@
+{
+  "format": 1,
+  "id": "DEMO-REVIEW",
+  "title": "Review de la bibliothèque",
+  "project": "Atelier de lecture",
+  "mission": "Première sauvegarde",
+  "tickets": [
+    {
+      "id": "DEMO-12",
+      "title": "PR de l’interface (lien de démonstration)",
+      "url": "https://github.com/montassarkhalloufi/DevMethod/pull/19"
+    }
+  ],
+  "date": "2026-09-13",
+  "scope": [
+    "Démonstration fictive de consultation, sans audit du projet illustré."
+  ],
+  "exclusions": [
+    "Aucun résultat réel de sécurité ou de performance."
+  ],
+  "revision": {
+    "commit": "demo-revision-1",
+    "dirty": []
+  },
+  "technologies": [
+    {
+      "name": "HTML / DOM",
+      "version": "Living Standard",
+      "detectedFrom": "Fixture authored for the viewer"
+    }
+  ],
+  "sources": [
+    {
+      "id": "WAI",
+      "title": "Messages de statut accessibles",
+      "kind": "documentation",
+      "publisher": "W3C WAI",
+      "technology": "Web accessibility",
+      "version": "WCAG 2.2",
+      "url": "https://www.w3.org/WAI/WCAG22/Understanding/status-messages.html",
+      "consultedAt": null,
+      "access": "unverified",
+      "usage": "Référence proposée pour cette démonstration, non consultée dans une review réelle.",
+      "compatibility": "À vérifier pour le parcours cible.",
+      "provenance": "Domaine w3.org ; cette fixture ne constitue pas une vérification de provenance."
+    }
+  ],
+  "checks": [
+    {
+      "id": "C-01",
+      "title": "Annonce après sauvegarde",
+      "domain": "Accessibilité",
+      "kind": "manual",
+      "status": "failed",
+      "result": "Exemple fictif : le changement de statut ne serait pas annoncé.",
+      "reason": null,
+      "evidenceIds": [
+        "E-01"
+      ],
+      "revision": "demo-revision-1",
+      "targets": [
+        "save-status"
+      ]
+    },
+    {
+      "id": "C-02",
+      "title": "Conservation du titre",
+      "domain": "Données",
+      "kind": "automated",
+      "status": "passed",
+      "result": "Exemple fictif : la valeur enregistrée correspond à la saisie.",
+      "reason": null,
+      "evidenceIds": [
+        "E-02"
+      ],
+      "revision": "demo-revision-1",
+      "targets": [
+        "save-title"
+      ]
+    },
+    {
+      "id": "C-03",
+      "title": "Temps de réponse sur liste longue",
+      "domain": "Performance",
+      "kind": "automated",
+      "status": "not-run",
+      "result": "Aucune mesure disponible dans cette fixture.",
+      "reason": "Jeu de données représentatif indisponible.",
+      "evidenceIds": [],
+      "revision": "demo-revision-1",
+      "targets": [
+        "list-view"
+      ]
+    }
+  ],
+  "findings": [
+    {
+      "id": "R-01",
+      "title": "Le retour de sauvegarde reste silencieux",
+      "domain": "Accessibilité",
+      "severity": "moderate",
+      "severityReason": "Exemple fictif : le lecteur pourrait ignorer que son action a réussi.",
+      "confidence": "confirmed",
+      "resolution": "open",
+      "location": {
+        "path": "save-status (composant fictif)",
+        "line": null,
+        "component": "Confirmation de sauvegarde"
+      },
+      "trigger": "Sauvegarder un titre en utilisant un lecteur d’écran.",
+      "expected": "Le succès est annoncé sans déplacer le focus.",
+      "observed": "Dans ce scénario fictif, seul le texte visuel change.",
+      "impact": "Le lecteur ne sait pas si son titre a été conservé.",
+      "reproduction": [
+        "Scénario illustratif, non exécuté.",
+        "Saisir un titre, activer Sauvegarder et écouter le retour."
+      ],
+      "evidenceIds": [
+        "E-01"
+      ],
+      "correction": "Exposer le message de succès comme statut accessible, puis vérifier son annonce.",
+      "tradeoffs": "Éviter les annonces répétées ; conserver le focus dans le parcours.",
+      "sourceIds": [
+        "WAI"
+      ],
+      "ticketIds": [
+        "DEMO-12"
+      ],
+      "verification": "Inspecter avec un lecteur d’écran réel et vérifier le maintien du focus.",
+      "resolutionEvidenceIds": [],
+      "targets": [
+        "save-status"
+      ]
+    },
+    {
+      "id": "R-02",
+      "title": "Une grande liste pourrait ralentir la recherche",
+      "domain": "Performance",
+      "severity": "minor",
+      "severityReason": "Impact supposé limité aux collections volumineuses.",
+      "confidence": "suspected",
+      "resolution": "open",
+      "location": {
+        "path": "list-view (composant fictif)",
+        "line": null,
+        "component": "Liste de titres"
+      },
+      "trigger": "Rechercher dans une collection volumineuse.",
+      "expected": "La saisie reste réactive.",
+      "observed": "Aucune mesure ; risque hypothétique seulement.",
+      "impact": "Un délai pourrait gêner la saisie ; cela reste à vérifier.",
+      "reproduction": [
+        "Préparer un jeu représentatif puis mesurer ; non exécuté."
+      ],
+      "evidenceIds": [],
+      "correction": "Mesurer avant de décider d’une optimisation.",
+      "tradeoffs": "Éviter une complexité non justifiée par des mesures.",
+      "sourceIds": [],
+      "ticketIds": [],
+      "verification": "Mesurer latence et réactivité sur le matériel cible.",
+      "resolutionEvidenceIds": [],
+      "targets": [
+        "list-view"
+      ]
+    }
+  ],
+  "evidence": [
+    {
+      "id": "E-01",
+      "title": "Flux du retour utilisateur — schéma explicatif",
+      "kind": "diagram",
+      "content": "Saisie du titre → Sauvegarder → Titre conservé → Message visuel\n                                      ↳ Annonce vocale manquante (hypothèse de fixture)\nSchéma explicatif fictif, pas une capture ni un journal d’exécution.",
+      "url": null,
+      "image": null
+    },
+    {
+      "id": "E-02",
+      "title": "Résultat illustratif de conservation",
+      "kind": "log",
+      "content": "DÉMONSTRATION — journal rédigé, test non exécuté.\nEntrée « Un titre » → valeur conservée « Un titre ».",
+      "url": null,
+      "image": null
+    }
+  ],
+  "limits": [
+    "DÉMONSTRATION : données fictives, sans validation d’un produit réel.",
+    "Le contrôle de performance n’a pas été exécuté."
+  ],
+  "policy": {
+    "blockingSeverities": [
+      "critical",
+      "major",
+      "moderate"
+    ],
+    "requireAllChecks": true,
+    "rationale": "Politique illustrative : corriger les problèmes confirmés modérés ou plus graves avant intégration."
+  },
+  "summary": "Démonstration de l’interface. Les constats et contrôles ci-dessous sont fictifs."
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "devmethod-ai",
-  "version": "0.2.0",
+  "version": "0.3.1",
   "description": "From idea to delivery with your AI coding agents",
   "license": "MIT",
   "type": "module",
@@ -28,7 +28,7 @@
     "url": "git+https://github.com/montassarkhalloufi/DevMethod.git"
   },
   "scripts": {
-    "build": "tsc",
+    "build": "tsc && node scripts/build-review.mjs",
     "test": "npm run build && node --test tests/*.test.mjs",
     "test:fixtures": "node --test tests/evaluation.test.mjs",
     "check:docs": "node scripts/check-docs.mjs",

package/scripts/build-review.mjs

@@ -0,0 +1,8 @@
+// The two browser modules have no runtime dependencies. Keep this bundle explicit.
+import fs from 'node:fs';
+const model = fs.readFileSync('dist/review-model.js', 'utf8').replace(/\r\n/g, '\n');
+const app = fs.readFileSync('dist/review-app.js', 'utf8').replace(/\r\n/g, '\n');
+if (/^import /m.test(model) || (app.match(/^import /gm) || []).length !== 1 || !app.includes("from './review-model.js'")) throw new Error('Review browser dependencies changed; inspect bundling.');
+const code = `(function () {\n'use strict';\n${model.replace(/^export /gm, '')}\n${app.replace(/^import .*;\n/m, '')}\n})();\n`;
+fs.writeFileSync('dist/review-browser.js', code);
+fs.copyFileSync('src/review-ui.css', 'dist/review-ui.css');

package/scripts/media/review-extension/extend.py

@@ -0,0 +1,100 @@
+"""Insert real review screenshots into the existing 0.3.0 film; never rebuild its story.
+Requires ffmpeg, ffprobe, macOS say/Thomas and Pillow. Run from repository root.
+"""
+import json, math, pathlib, subprocess, re, hashlib
+from PIL import Image, ImageDraw, ImageFont
+ROOT=pathlib.Path.cwd(); TEMP=pathlib.Path('/private/tmp/devmethod-review-film'); TEMP.mkdir(exist_ok=True)
+MEDIA=ROOT/'docs/media/visual-chain'; EXT=ROOT/'docs/media/review-extension'
+MOVIE='docs/media/visual-chain/devmethod-du-besoin-au-produit-4k.fr.mp4'
+CUT=171.52
+
+def run(args):
+    subprocess.run(args,check=True,stdout=subprocess.DEVNULL,stderr=open(TEMP/'encode.log','a'))
+def duration(file):
+    return float(subprocess.check_output(['ffprobe','-v','error','-show_entries','format=duration','-of','csv=p=0',str(file)]))
+def font(size,serif=False,mono=False):
+    return ImageFont.truetype('/System/Library/Fonts/Menlo.ttc' if mono else '/System/Library/Fonts/Supplemental/'+('Georgia.ttf' if serif else 'Arial.ttf'),size)
+def wrap(draw,text,f,width):
+    lines=[]; line=''
+    for word in text.split():
+        candidate=(line+' '+word).strip()
+        if draw.textlength(candidate,font=f)>width and line: lines.append(line);line=word
+        else:line=candidate
+    return lines+[line]
+def frame(i,s):
+    im=Image.new('RGB',(3840,2160),'#0e1d21'); d=ImageDraw.Draw(im)
+    d.text((160,76),'DevMethod / Review',font=font(40),fill='#b0c6bf')
+    d.text((3020,76),f'COMPLÉMENT  {i+1:02} / 06',font=font(32),fill='#b0c6bf')
+    d.text((160,210),s['title'],font=font(110,True),fill='#f6f0e5')
+    d.rounded_rectangle((160,440,1220,1850),radius=28,fill='#183137',outline='#45625f',width=2)
+    y=510
+    d.text((210,y),'DEPUIS LE PACKAGE NPM',font=font(30),fill='#e9ae82'); y+=95
+    for line in s['command'].split('\n'):
+        d.text((210,y),line,font=font(34,mono=True),fill='#f1b98e');y+=58
+    y+=90
+    for line in wrap(d,s['explanation'],font(49),950):
+        d.text((210,y),line,font=font(49),fill='#ecf1e8');y+=68
+    y+=75
+    for line in wrap(d,s['note'],font(33),950):
+        d.text((210,y),line,font=font(33),fill='#b5d0c2');y+=48
+    shot=Image.open(EXT/'captures'/s['capture']).convert('RGB');scale=min(2360/shot.width,1470/shot.height);shot=shot.resize((round(shot.width*scale),round(shot.height*scale)),Image.Resampling.LANCZOS)
+    # Screenshot pixels are preserved apart from proportional display scaling.
+    im.paste(shot,(1320+(2360-shot.width)//2,460+(1470-shot.height)//2))
+    d.text((160,2040),'Captures réelles de l’interface · Données de démonstration fictives · Montage commenté',font=font(33),fill='#b0c6bf')
+    im.save(TEMP/f'frame-{i}.png')
+
+scenes=[
+ dict(title='La review devient consultable',capture='findings.jpg',command='npx devmethod-ai@0.3.1 '+chr(92)+'\n  review --demo '+chr(92)+'\n  --output review.html --open',explanation='Une interface locale pour relier constats, preuves et corrections.',note='HTML autonome hors ligne. La commande présente des résultats enregistrés ; elle ne réalise pas la review.',voice='Dev Method ajoute maintenant une interface de review. Voici sa démonstration fictive, distincte des résultats réels du pilote Lisière. Cette commande ouvre un rapport local.'),
+ dict(title='Retrouver le constat utile',capture='filtered.jpg',command='Constats → Rechercher\n« silencieux »',explanation='Rechercher, filtrer et sélectionner sans confondre gravité et confiance.',note='Un constat confirmé et un risque à vérifier restent distincts. Les compteurs décrivent toute la review.',voice='La recherche isole un constat. Les filtres distinguent domaine, gravité, confiance et résolution. Un risque à vérifier reste une hypothèse, pas un défaut confirmé.'),
+ dict(title='Comprendre, puis corriger',capture='correction.jpg',command='Preuve → Correction\n→ Vérification attendue',explanation='Du scénario observé à la correction et à sa nouvelle vérification.',note='Le ticket complète le constat. Fermer le panneau ne résout jamais le problème.',voice='Chaque constat relie scénario, preuve, impact et correction. La vérification attendue et le ticket restent accessibles. Masquer un résultat ne le marque pas comme résolu.'),
+ dict(title='Voir la couverture réelle',capture='coverage.jpg',command='Couverture\nRéussi · Échec · Non exécuté',explanation='Les contrôles ont leur propre statut, indépendant du nombre de constats.',note='Une zone non inspectée ne devient pas verte. La révision et les limites restent visibles.',voice='La couverture sépare les contrôles réussis, en échec et non exécutés. L’absence de constat ne prouve pas que tout a été vérifié.'),
+ dict(title='Vérifier les références',capture='sources.jpg',command='Sources\nVersion → Provenance → Usage',explanation='Une référence connue n’est pas automatiquement une source consultée.',note='La démonstration signale sa référence non vérifiée. Une vraie review consigne sa consultation et ses limites.',voice='Les sources indiquent technologie, version, provenance et usage. Ici, la référence est clairement non vérifiée : cette démonstration ne prétend pas l’avoir consultée.'),
+ dict(title='Exporter et reprendre',capture='export.jpg',command='Exporter le rapport\nHTML · Markdown · JSON',explanation='Une source structurée commune pour l’interface et les rapports.',note='Conserver la révision, les preuves et les limites. Le HTML exporté conserve aussi l’état de consultation.',voice='Exportez la même review en HTML, Markdown ou JSON. Les résultats partagent une source commune. Le rapport conserve la révision, les preuves et les limites pour la reprise.'),
+]
+source=TEMP/'original-0.3.0.mp4'
+if not source.exists(): source.write_bytes(subprocess.check_output(['git','show',f'v0.3.0:{MOVIE}']))
+# Existing footage and narration are retained. Only the six-second ending is moved.
+enc=['-c:v','libx264','-profile:v','high','-level:v','5.1','-pix_fmt','yuv420p','-r','25','-threads','4','-preset','veryfast','-crf','20','-c:a','aac','-ar','48000','-ac','1','-b:a','192k','-video_track_timescale','12800']
+for i,s in enumerate(scenes):
+    frame(i,s)
+    (TEMP/f'voice-{i}.txt').write_text(s['voice'])
+    run(['say','-v','Thomas','-r','165','-f',str(TEMP/f'voice-{i}.txt'),'-o',str(TEMP/f'voice-{i}.aiff')])
+    s['duration']=math.ceil((duration(TEMP/f'voice-{i}.aiff')+0.8)*25)/25
+    clip=TEMP/f'clip-{i}.mp4'
+    fingerprint=hashlib.sha256((TEMP/f'frame-{i}.png').read_bytes()+(TEMP/f'voice-{i}.aiff').read_bytes()+json.dumps(enc).encode()).hexdigest()
+    cache=TEMP/f'clip-{i}.sha256'
+    if not clip.exists() or not cache.exists() or cache.read_text()!=fingerprint:
+        run(['ffmpeg','-y','-loop','1','-framerate','25','-i',str(TEMP/f'frame-{i}.png'),'-i',str(TEMP/f'voice-{i}.aiff'),'-vf',f"fade=t=in:st=0:d=0.3,fade=t=out:st={s['duration']-0.3}:d=0.3",'-af','apad','-t',str(s['duration']),*enc,str(clip)])
+        cache.write_text(fingerprint)
+    print('Review scene encoded',i+1,flush=True)
+# Encode only the splice tracks once, leaving the original scene design and timing intact.
+if not (TEMP/'before.mp4').exists():
+    run(['ffmpeg','-y','-i',str(source),'-t',str(CUT),'-map','0:v:0','-map','0:a:0',*enc,str(TEMP/'before.mp4')])
+if not (TEMP/'after.mp4').exists():
+    run(['ffmpeg','-y','-ss',str(CUT),'-i',str(source),'-map','0:v:0','-map','0:a:0',*enc,str(TEMP/'after.mp4')])
+clips=[TEMP/'before.mp4']+[TEMP/f'clip-{i}.mp4' for i in range(len(scenes))]+[TEMP/'after.mp4']
+(TEMP/'concat.txt').write_text('\n'.join(f"file '{p}'" for p in clips))
+# Subtitles follow measured segment durations, not estimated spoken word timing.
+def seconds(st):
+    h,m,sec=st.replace(',','.').split(':');return int(h)*3600+int(m)*60+float(sec)
+def stamp(sec):
+    n=round(sec*1000);return f'{n//3600000:02}:{n//60000%60:02}:{n//1000%60:02},{n%1000:03}'
+original_srt=subprocess.check_output(['git','show','v0.3.0:docs/media/visual-chain/devmethod-du-besoin-au-produit.fr.srt']).decode()
+added=sum(duration(p) for p in clips[1:-1]); prefix=duration(clips[0]); items=[]
+for block in original_srt.strip().split('\n\n'):
+    lines=block.splitlines();a,b=map(seconds,lines[1].split(' --> ')); shift=added+(prefix-CUT) if a>=171.5 else 0
+    items.append((a+shift,b+shift,'\n'.join(lines[2:])))
+cursor=prefix
+for i,s in enumerate(scenes):
+    s['start']=cursor
+    parts=re.split(r'(?<=[.!?])\s+',s['voice']); spoken=duration(TEMP/f'voice-{i}.aiff');t=cursor
+    for part in parts:
+        length=spoken*len(part)/sum(map(len,parts));items.append((t,t+length,part));t+=length
+    cursor+=duration(TEMP/f'clip-{i}.mp4')
+items.sort(); srt=MEDIA/'devmethod-du-besoin-au-produit.fr.srt'
+srt.write_text('\n\n'.join(f'{i+1}\n{stamp(a)} --> {stamp(b)}\n{text}' for i,(a,b,text) in enumerate(items))+'\n')
+run(['ffmpeg','-y','-f','concat','-safe','0','-i',str(TEMP/'concat.txt'),'-i',str(srt),'-map','0:v:0','-map','0:a:0','-map','1:0','-c:v','copy','-c:a','copy','-c:s','mov_text','-metadata:s:a:0','language=fra','-metadata:s:s:0','language=fra','-movflags','+faststart',str(ROOT/MOVIE)])
+manifest=dict(baseTag='v0.3.0',baseSHA256=hashlib.sha256(source.read_bytes()).hexdigest(),insertionSeconds=CUT,baseDuration=duration(source),duration=duration(ROOT/MOVIE),method='Original footage and narration retained; review screenshot chapter inserted before original ending. Re-encoded splice tracks; no original scene regenerated.',scenes=scenes)
+(EXT/'scenes.json').write_text(json.dumps(manifest,ensure_ascii=False,indent=2)+'\n')
+run(['ffmpeg','-y','-ss',str(prefix+3),'-i',str(ROOT/MOVIE),'-frames:v','1','-vf','scale=1920:-1',str(MEDIA/'video-preview.jpg')])
+print('Final duration',manifest['duration'],flush=True)

package/scripts/package-smoke.mjs

@@ -15,8 +15,21 @@ try {
   run('tar', ['-xzf', '-'], root, 0, fs.readFileSync(path.resolve(archive)));
   const pkg = path.join(root, 'package'); const cli = path.join(pkg, 'dist/cli.js');
   const call = (args, expected = 0) => run(process.execPath, [cli, ...args], root, expected);
-  assert.equal(JSON.parse(fs.readFileSync(path.join(pkg, 'package.json'))).version, '0.2.0');
+  assert.equal(JSON.parse(fs.readFileSync(path.join(pkg, 'package.json'))).version, '0.3.1');
   run(process.execPath, ['scripts/check-docs.mjs'], pkg);
+  assert.match(call(['--help']), /Markdown PLAN\/tickets and legacy missions/);
+  for (const resource of ['project-foundation/references/exploration.md', 'project-foundation/references/delivery-planning.md', 'project-foundation/assets/EXISTANT.md', 'project-foundation/assets/OPPORTUNITES.md', 'project-foundation/assets/CADRAGE.md', 'project-foundation/assets/REGLES.md', 'scoped-delivery/assets/PLAN.md', 'scoped-delivery/assets/TICKET.md', 'scoped-delivery/assets/REPRISE.md', 'scoped-delivery/assets/MISSION.md', 'scoped-delivery/assets/REVIEW.md', 'scoped-delivery/references/review-workflow.md']) {
+    assert.ok(fs.statSync(path.join(pkg, '.agents/skills', resource)).size > 0, resource);
+  }
+  assert.ok(fs.existsSync(path.join(pkg, 'examples/mission-dialogue/docs/missions/first-save/tickets/SAVE-1.md')));
+  for (const file of ['review.js', 'review-cli.js', 'review-model.js', 'review-browser.js', 'review-ui.css']) assert.ok(fs.statSync(path.join(pkg, 'dist', file)).size > 0, file);
+  const reviewRoot = path.join(root, 'review-output');
+  const reviewResult = JSON.parse(call(['review', '--demo', '--dest', reviewRoot, '--output', 'review.html', '--markdown', 'REVIEW.md', '--json']));
+  assert.equal(reviewResult.status, 'corrections');
+  assert.ok(fs.readFileSync(path.join(reviewRoot, 'review.html'), 'utf8').includes('Content-Security-Policy'));
+  assert.ok(fs.readFileSync(path.join(reviewRoot, 'REVIEW.md'), 'utf8').includes('R-01'));
+  call(['review', '--demo', '--dest', reviewRoot, '--output', 'review.html'], 2);
+
   for (const testFile of [...fs.readdirSync(path.join(pkg, 'examples/pocket-tasks/tests')).filter(f => f.endsWith('.test.mjs')).map(f => `examples/pocket-tasks/tests/${f}`), 'evaluation/greenfield/acceptance.test.mjs', 'evaluation/greenfield/security.test.mjs']) {
     run(process.execPath, ['--test', testFile], pkg);
   }
@@ -54,11 +67,13 @@ try {
     fs.appendFileSync(customized, '\nLocal workflow policy.\n'); fs.appendFileSync(profile, '\nFilled project context.\n');
     const before = [fs.readFileSync(customized), fs.readFileSync(profile)];
     const preview = JSON.parse(call(['update-preview', '--dest', adopted, '--json']));
-    assert.equal(preview.entries.find(e => e.path === '.agents/skills/project-foundation/SKILL.md').classification, 'conflict');
-    assert.equal(preview.provenance, 'unknown');
+    const skillPath = '.agents/skills/project-foundation/SKILL.md';
+    const upstreamChanged = !fs.readFileSync(path.join(legacy, skillPath)).equals(fs.readFileSync(path.join(pkg, skillPath)));
+    assert.equal(preview.entries.find(e => e.path === skillPath).classification, upstreamChanged ? 'conflict' : 'customized');
+    assert.equal(preview.provenance, JSON.parse(fs.readFileSync(path.join(adopted, 'kit-manifest.json'))).provenance ? 'recorded' : 'unknown');
     call(['init', '--tool', 'codex', '--dest', adopted], 2);
     assert.deepEqual([fs.readFileSync(customized), fs.readFileSync(profile)], before);
-    console.log('Actual legacy tarball: local/upstream conflict detected and filled profile/custom skill preserved.');
+    console.log(`Actual legacy tarball: ${upstreamChanged ? 'local/upstream conflict' : 'unchanged upstream with local customization'} detected; filled profile/custom skill preserved.`);
   }
-  console.log('Packed 0.2.0: three host installs, subset, customization preservation, mission/context/staleness/planning and documentation links passed. No native host execution.');
+  console.log('Packed 0.3.1: three host installs, subset, customization preservation, mission/context/staleness/planning and documentation links passed. No native host execution.');
 } finally { fs.rmSync(root, { recursive: true, force: true }); }