refactor(skills): consolidate git into github, rename init-review to review (LISA-19, LISA-33)

CLAUDE.md

@@ -114,8 +114,8 @@ Skills are invoked with `/skill-name` in Claude Code:
 | `/tasks` | Create, load, or summarize tasks | "tasks", "list tasks", "add task" |
 | `/lisa` | Intelligent assistant for memory and tasks | "lisa", "hey lisa" |
 | `/jira` | Create and manage Jira issues | "jira", "create ticket" |
-| `/git` | GitHub and Git workflow helpers | "create pr", "pr checks" |
-| `/init-review` | Initial codebase review and summary | First session in a repo |
+| `/github` | GitHub/Git workflow helpers, Issues, Projects | "create pr", "github issues", "bump version" |
+| `/review` | Initial codebase review and summary | First session in a repo |
 
 Skills source: `src/project/.lisa/skills/`
 Skills deployed to: `.lisa/skills/`

package.json

@@ -15,12 +15,12 @@
     "./skills/memory": "./dist/lib/skills/memory/memory.js",
     "./skills/tasks": "./dist/lib/skills/tasks/tasks.js",
     "./skills/jira": "./dist/lib/skills/jira/jira.js",
-    "./skills/git/bump-version": "./dist/lib/skills/git/bump-version.js",
+    "./skills/github/bump-version": "./dist/lib/skills/github/bump-version.js",
     "./skills/lisa/storage": "./dist/lib/skills/lisa/storage.js",
     "./skills/lisa/compile-skills": "./dist/lib/skills/lisa/compile-skills.js",
     "./skills/prompt": "./dist/lib/skills/prompt/prompt.js",
-    "./skills/init-review": "./dist/lib/skills/init-review/init-review.js",
-    "./skills/init-review/ai-enrich": "./dist/lib/skills/init-review/ai-enrich.js"
+    "./skills/review": "./dist/lib/skills/review/init-review.js",
+    "./skills/review/ai-enrich": "./dist/lib/skills/review/ai-enrich.js"
   },
   "engines": {
     "node": ">=18.0.0"

scripts/deploy-lisa.js

@@ -441,7 +441,7 @@ async function main() {
   await fs.ensureDir(claudeSkillsDir);
 
   // Create individual symlinks for each Lisa skill (same pattern as OpenCode)
-  const lisaSkills = ['memory', 'tasks', 'lisa', 'git', 'jira', 'init-review', 'prompt'];
+  const lisaSkills = ['memory', 'tasks', 'lisa', 'jira', 'review', 'prompt', 'github', 'pr'];
   for (const skill of lisaSkills) {
     const skillLink = path.join(claudeSkillsDir, skill);
     const skillTarget = path.join(targetLisa, 'skills', skill);
@@ -481,7 +481,7 @@ async function main() {
     const opencodeSkillsDir = path.join(targetOpenCode, 'skills');
     await fs.ensureDir(opencodeSkillsDir);
 
-    const lisaSkills = ['memory', 'tasks', 'lisa', 'git', 'jira', 'init-review', 'prompt'];
+    const lisaSkills = ['memory', 'tasks', 'lisa', 'jira', 'review', 'prompt', 'github', 'pr'];
     for (const skill of lisaSkills) {
       const skillLink = path.join(opencodeSkillsDir, skill);
       const skillTarget = path.join(targetLisa, 'skills', skill);

src/lib/commands/init.ts

@@ -480,6 +480,7 @@ export async function initCommand(opts: IInitOptions, services: ICliServices): P
       if (relativePath.includes('shared') || relativePath.includes('common')) return false;
       if (relativePath.includes('scripts')) return false;
       if (basename === 'SKILL.md' || basename === 'SKILL.local.md') return true;
+      if (basename.endsWith('.txt') || basename.endsWith('.json')) return true;
       if (basename === 'cache' || basename === '.gitkeep') return true;
 
       return fs.statSync(src).isDirectory();
@@ -502,6 +503,10 @@ export async function initCommand(opts: IInitOptions, services: ICliServices): P
   copies.push(services.templateCopier.copy('.lisa/rules/shared/clean-architecture.md', path.join(rulesDir, 'shared', 'clean-architecture.md'), replacements, force));
   copies.push(services.templateCopier.copy('.lisa/rules/shared/code-quality-rules.md', path.join(rulesDir, 'shared', 'code-quality-rules.md'), replacements, force));
   copies.push(services.templateCopier.copy('.lisa/rules/shared/testing-principles.md', path.join(rulesDir, 'shared', 'testing-principles.md'), replacements, force));
+  copies.push(services.templateCopier.copy('.lisa/rules/shared/git-rules.md', path.join(rulesDir, 'shared', 'git-rules.md'), replacements, force));
+  copies.push(services.templateCopier.copy('.lisa/rules/shared/git-rules.local.md', path.join(rulesDir, 'shared', 'git-rules.local.md'), replacements, force));
+  copies.push(services.templateCopier.copy('.lisa/rules/shared/memory-rules.md', path.join(rulesDir, 'shared', 'memory-rules.md'), replacements, force));
+  copies.push(services.templateCopier.copy('.lisa/rules/shared/task-rules.md', path.join(rulesDir, 'shared', 'task-rules.md'), replacements, force));
   copies.push(services.templateCopier.copy('.lisa/rules/typescript/coding-standards.md', path.join(rulesDir, 'typescript', 'coding-standards.md'), replacements, force));
   copies.push(services.templateCopier.copy('.lisa/rules/typescript/testing.md', path.join(rulesDir, 'typescript', 'testing.md'), replacements, force));
   copies.push(services.templateCopier.copy('.lisa/rules/typescript/typescript-config-guide.md', path.join(rulesDir, 'typescript', 'typescript-config-guide.md'), replacements, force));
@@ -522,7 +527,7 @@ export async function initCommand(opts: IInitOptions, services: ICliServices): P
 
     await fs.ensureDir(claudeSkillsDir);
 
-    const lisaSkills = ['memory', 'tasks', 'lisa', 'git', 'jira', 'init-review', 'prompt', 'github'];
+    const lisaSkills = ['memory', 'tasks', 'lisa', 'jira', 'review', 'prompt', 'github', 'pr'];
     for (const skill of lisaSkills) {
       const skillLink = path.join(claudeSkillsDir, skill);
       const skillTarget = path.join(skillsDir, skill);
@@ -583,7 +588,7 @@ export async function initCommand(opts: IInitOptions, services: ICliServices): P
 
     await fs.ensureDir(opencodeSkillsDir);
 
-    const lisaSkills = ['memory', 'tasks', 'lisa', 'git', 'jira', 'init-review', 'prompt'];
+    const lisaSkills = ['memory', 'tasks', 'lisa', 'jira', 'review', 'prompt', 'github', 'pr'];
     for (const skill of lisaSkills) {
       const skillLink = path.join(opencodeSkillsDir, skill);
       const skillTarget = path.join(skillsDir, skill);

src/lib/commands/skills.ts

@@ -2,7 +2,7 @@
  * Skill Passthrough Command Module
  *
  * Simple passthrough commands that delegate to skill scripts:
- * jira, github, prompt, bump-version, init-review, compile-skills
+ * jira, github, prompt, bump-version, review, compile-skills
  */
 
 import type {Command} from 'commander';
@@ -50,18 +50,18 @@ export function registerSkillCommands(program: Command): void {
     .allowUnknownOption()
     .action(async (_opts, cmd) => {
       const args = cmd.args || [];
-      const scriptPath = path.join(__dirname, '..', 'skills', 'git', 'bump-version.js');
+      const scriptPath = path.join(__dirname, '..', 'skills', 'github', 'bump-version.js');
       await spawnAndWait(scriptPath, args);
     });
 
-  // Subcommand: lisa init-review
+  // Subcommand: lisa review
   program
-    .command('init-review')
+    .command('review')
     .description('Run initial codebase review')
     .allowUnknownOption()
     .action(async (_opts, cmd) => {
       const args = cmd.args || [];
-      const scriptPath = path.join(__dirname, '..', 'skills', 'init-review', 'init-review.js');
+      const scriptPath = path.join(__dirname, '..', 'skills', 'review', 'init-review.js');
       await spawnAndWait(scriptPath, args);
     });
 

src/lib/scanner/reviewer.ts

@@ -93,8 +93,8 @@ async function runInitReview(projectPath: string): Promise<{
   // Look for init-review script in the project's .lisa folder
   // or in our dist/project location
   const possiblePaths = [
-    path.join(projectPath, '.lisa', 'skills', 'init-review', 'scripts', 'init-review.js'),
-    path.join(__dirname, '..', '..', 'project', '.lisa', 'skills', 'init-review', 'scripts', 'init-review.js'),
+    path.join(projectPath, '.lisa', 'skills', 'review', 'scripts', 'init-review.js'),
+    path.join(__dirname, '..', '..', 'project', '.lisa', 'skills', 'review', 'scripts', 'init-review.js'),
   ];
 
   let scriptPath: string | null = null;

src/lib/skills/github/bump-version.ts

@@ -0,0 +1,52 @@
+#!/usr/bin/env node
+/**
+ * Version Bump CLI - thin entry point.
+ *
+ * Usage: node bump-version.js [major|minor|patch]
+ * Default: minor
+ *
+ * Set LISA_AUTO_BUMP_VERSION in .lisa/.env to control behavior:
+ *   true          — enabled (default)
+ *   false         — disabled, skip with message
+ *   patch|minor|major — enabled with that bump type as default
+ */
+
+export {};
+
+async function main(): Promise<void> {
+  const { createVersionService } = await import('../shared/services');
+  const { loadEnv } = await import('../shared/utils/env');
+
+  const env = loadEnv();
+  const autoBump = env.raw['LISA_AUTO_BUMP_VERSION'] ?? 'true';
+
+  // Check if version bumping is disabled
+  if (autoBump.toLowerCase() === 'false') {
+    const result = { status: 'skipped', reason: 'LISA_AUTO_BUMP_VERSION is false' };
+    console.log(JSON.stringify(result));
+    console.error('Version bump skipped (LISA_AUTO_BUMP_VERSION=false)');
+    return;
+  }
+
+  const service = createVersionService();
+
+  try {
+    // Resolve bump type: CLI arg > env default > 'minor'
+    const explicitArg = process.argv[2];
+    let defaultType = 'minor';
+    if (['major', 'minor', 'patch'].includes(autoBump.toLowerCase())) {
+      defaultType = autoBump.toLowerCase();
+    }
+
+    const bumpType = service.validateBumpType(explicitArg || defaultType);
+    const result = service.bump(bumpType);
+
+    console.log(JSON.stringify(result));
+    console.error(`Bumped version: ${result.oldVersion} → ${result.newVersion} (${result.bumpType})`);
+  } catch (err: unknown) {
+    console.error(`Error: ${err instanceof Error ? err.message : String(err)}`);
+    process.exit(1);
+  }
+}
+
+main();

src/project/.lisa/.env.template

@@ -25,3 +25,6 @@ NEO4J_DATABASE=neo4j
 # LLM API Keys
 # ANTHROPIC_API_KEY=
 # OPENAI_API_KEY=
+
+# Auto version bump on push (true, false, or default bump type: patch|minor|major)
+# LISA_AUTO_BUMP_VERSION=true