gm-skill 0.1.1 → 0.1.2

package/test-e2e.js

@@ -0,0 +1,117 @@
+const path = require('path');
+const fs = require('fs');
+
+console.log('=== GM-SKILL END-TO-END TEST ===\n');
+
+const skillsDir = path.join(__dirname, 'skills');
+const skills = ['gm', 'planning', 'gm-execute', 'gm-emit', 'gm-complete', 'update-docs'];
+
+console.log('1. Checking skill structure...\n');
+let structureOk = true;
+
+skills.forEach(skill => {
+  const skillPath = path.join(skillsDir, skill);
+  const indexPath = path.join(skillPath, 'index.js');
+  const skillMdPath = path.join(skillPath, 'SKILL.md');
+
+  const hasIndex = fs.existsSync(indexPath);
+  const hasMd = fs.existsSync(skillMdPath);
+
+  const status = hasIndex && hasMd ? '✓' : '✗';
+  console.log(`  ${status} ${skill}`);
+  console.log(`      index.js: ${hasIndex ? '✓' : '✗'} SKILL.md: ${hasMd ? '✓' : '✗'}`);
+
+  if (!hasIndex || !hasMd) {
+    structureOk = false;
+  }
+});
+
+if (!structureOk) {
+  console.log('\n✗ FAILED: Some skills are missing index.js or SKILL.md');
+  process.exit(1);
+}
+
+console.log('\n2. Testing skill exports...\n');
+
+let exportsOk = true;
+skills.forEach(skill => {
+  try {
+    const skillPath = path.join(skillsDir, skill, 'index.js');
+    const skillModule = require(skillPath);
+
+    if (typeof skillModule !== 'function') {
+      console.log(`  ✗ ${skill}: module.exports is not a function`);
+      exportsOk = false;
+    } else {
+      console.log(`  ✓ ${skill}: exports async function`);
+    }
+  } catch (err) {
+    console.log(`  ✗ ${skill}: failed to require - ${err.message}`);
+    exportsOk = false;
+  }
+});
+
+if (!exportsOk) {
+  console.log('\n✗ FAILED: Some skill modules are not properly exported');
+  process.exit(1);
+}
+
+console.log('\n3. Testing orchestration chain (async)...\n');
+
+async function testChain() {
+  try {
+    const gmModule = require(path.join(skillsDir, 'gm', 'index.js'));
+
+    const result = await gmModule({
+      request: 'test task for gm-skill implementation',
+      taskId: 'test-' + Date.now(),
+    });
+
+    if (!result) {
+      console.log('  ✗ gm returned null');
+      return false;
+    }
+
+    if (!result.phase) {
+      console.log('  ✗ result missing phase');
+      return false;
+    }
+
+    if (!result.context) {
+      console.log('  ✗ result missing context');
+      return false;
+    }
+
+    console.log(`  ✓ gm returned valid JSON`);
+    console.log(`    - phase: ${result.phase}`);
+    console.log(`    - nextSkill: ${result.nextSkill || 'null'}`);
+
+    if (result.context.phases && result.context.phases.length > 0) {
+      console.log(`    - chain: ${result.context.phases.map(p => p.skill).join(' → ')}`);
+    }
+
+    if (result.error) {
+      console.log(`    - error: ${result.error}`);
+      return false;
+    }
+
+    return true;
+  } catch (err) {
+    console.log(`  ✗ Chain failed: ${err.message}`);
+    console.log(`    ${err.stack.split('\n')[1]}`);
+    return false;
+  }
+}
+
+testChain().then(success => {
+  if (success) {
+    console.log('\n✓ ALL TESTS PASSED');
+    process.exit(0);
+  } else {
+    console.log('\n✗ TESTS FAILED');
+    process.exit(1);
+  }
+}).catch(err => {
+  console.error('\n✗ Fatal error:', err.message);
+  process.exit(1);
+});

package/test-unified.js

@@ -0,0 +1,24 @@
+const { prepare } = require('./index.js');
+
+async function test() {
+  console.log('Testing gm-skill prepare()...');
+  try {
+    const state = await prepare({ sessionId: 'test-session-' + Date.now() });
+    console.log('Prepare successful!');
+    console.log('Session ID:', state.sessionId);
+    console.log('CWD:', state.cwd);
+    console.log('Snapshot Git:', JSON.stringify(state.snapshot.git, null, 2));
+    console.log('Snapshot Tasks count:', state.snapshot.tasks.length);
+    
+    if (state.snapshot.error) {
+      console.warn('Snapshot error:', state.snapshot.error);
+    }
+    
+    process.exit(0);
+  } catch (e) {
+    console.error('Prepare failed:', e);
+    process.exit(1);
+  }
+}
+
+test();

package/update-docs.SKILL.md

@@ -0,0 +1,66 @@
+---
+name: update-docs
+description: UPDATE-DOCS phase. Refresh README.md, AGENTS.md, and docs/index.html to reflect changes made this session. Commits and pushes doc updates. Terminal phase — declares COMPLETE.
+---
+
+# GM UPDATE-DOCS
+
+Entry: feature verified, committed, pushed. Exit: docs match disk, committed, pushed → COMPLETE. Unknown architecture change → `planning`.
+
+Every claim in docs is verifiable against disk. Phase names match frontmatter, platform names match `platforms/`, file paths exist, constraint counts are accurate. An unverifiable section is removed, not speculated.
+
+## Sequence
+
+What changed — run directly via Bash:
+
+```
+git log -5 --oneline
+git diff HEAD~1 --stat
+```
+
+Read current docs via Read tool, or via a nodejs spool file (`in/nodejs/<N>.js`):
+
+```
+const fs = require('fs');
+['README.md', 'AGENTS.md', 'docs/index.html', 'gm-starter/agents/gm.md'].forEach(f => {
+  try { console.log(`=== ${f} ===\n` + fs.readFileSync(f, 'utf8')); }
+  catch(e) { console.log(`MISSING: ${f}`); }
+});
+```
+
+Write changed sections only:
+
+- **README.md** — platform count, skill tree diagram, quick-start commands
+- **AGENTS.md** — via `Agent(subagent_type='gm:memorize', model='haiku', run_in_background=true, prompt='## CONTEXT TO MEMORIZE\n<learnings>')`. Never inline-edit.
+- **docs/index.html** — `PHASES` array, platform lists, state machine diagram
+- **gm-starter/agents/gm.md** — skill chain line if new skills added
+
+Verify from disk (Read tool, or a nodejs spool file):
+
+```
+const content = require('fs').readFileSync('/abs/path/file.md', 'utf8');
+console.log(content.includes('expectedString'), content.length);
+```
+
+Commit and push directly via Bash:
+
+```
+git add README.md docs/index.html gm-starter/agents/gm.md
+git commit -m "docs: update documentation to reflect session changes"
+git push -u origin HEAD
+```
+
+## Exit: browser cleanup
+
+After docs push succeeds, close any browser sessions spawned during this or prior skill phases. Write a nodejs spool file calling rs-exec:
+
+```javascript
+const sessionId = process.env.CLAUDE_SESSION_ID;
+if (!sessionId) return;
+const rs = require('rs-exec');
+try {
+  rs.client().close_sessions_for(sessionId).catch(() => {});
+} catch (e) {}
+```
+
+Best-effort: session context or rs-exec unavailable → skip gracefully. No error thrown.