@pcoliveira90/pdd 0.2.6 → 0.3.1

package/src/core/patch-generator.js

@@ -1,126 +1,125 @@
-import fs from 'fs';
-import path from 'path';
-
-function ensureDir(filePath) {
-  fs.mkdirSync(path.dirname(filePath), { recursive: true });
-}
-
-function writeFile(filePath, content) {
-  ensureDir(filePath);
-  fs.writeFileSync(filePath, content, 'utf-8');
-}
-
-function slugify(value) {
-  return value
-    .toLowerCase()
-    .replace(/[^a-z0-9]+/g, '-')
-    .replace(/^-+|-+$/g, '')
-    .slice(0, 48);
-}
-
-export function generatePatchArtifacts({ issue, baseDir = process.cwd() }) {
-  const timestamp = Date.now();
-  const changeId = `change-${timestamp}-${slugify(issue || 'update')}`;
-  const changeDir = path.join(baseDir, 'changes', changeId);
-
-  const files = [
-    path.join('changes', changeId, 'delta-spec.md'),
-    path.join('changes', changeId, 'patch-plan.md'),
-    path.join('changes', changeId, 'verification-report.md')
-  ];
-
-  writeFile(
-    path.join(changeDir, 'delta-spec.md'),
-    `# Delta Spec
-
-## Change ID
-${changeId}
-
-## Issue
-${issue}
-
-## Type
-bugfix | feature | refactor-safe | hotfix
-
-## Context
-
-## Current Behavior
-
-## Expected Behavior
-
-## Evidence
-
-## Root Cause Hypothesis
-
-## Impacted Areas
-
-## Constraints
-
-## Minimal Safe Delta
-
-## Alternatives Considered
-
-## Acceptance Criteria
-
-## Verification Strategy
-`
-  );
-
-  writeFile(
-    path.join(changeDir, 'patch-plan.md'),
-    `# Patch Plan
-
-## Change ID
-${changeId}
-
-## Issue
-${issue}
-
-## Files to Inspect
-
-## Files to Change
-
-## Execution Steps
-1. Reproduce issue
-2. Confirm root cause
-3. Apply minimal change
-4. Adjust tests
-5. Run validations
-
-## Regression Risks
-
-## Rollback Strategy
-`
-  );
-
-  writeFile(
-    path.join(changeDir, 'verification-report.md'),
-    `# Verification Report
-
-## Change ID
-${changeId}
-
-## Issue
-${issue}
-
-## Reproduction
-
-## Changes Made
-
-## Tests Run
-
-## Manual Validation
-
-## Residual Risks
-
-## Final Status
-pending
-`
-  );
-
-  return {
-    changeId,
-    changeDir,
-    files
-  };
-}
+import fs from 'fs';
+import path from 'path';
+
+function ensureDir(filePath) {
+  fs.mkdirSync(path.dirname(filePath), { recursive: true });
+}
+
+function writeFile(filePath, content) {
+  ensureDir(filePath);
+  fs.writeFileSync(filePath, content, 'utf-8');
+}
+
+function slugify(value) {
+  return value
+    .toLowerCase()
+    .replace(/[^a-z0-9]+/g, '-')
+    .replace(/^-+|-+$/g, '')
+    .slice(0, 48);
+}
+
+export function generatePatchArtifacts({ issue, baseDir = process.cwd(), changeId = null }) {
+  const resolvedChangeId = changeId || `change-${Date.now()}-${slugify(issue || 'update')}`;
+  const changeDir = path.join(baseDir, 'changes', resolvedChangeId);
+
+  const files = [
+    path.join('changes', resolvedChangeId, 'delta-spec.md'),
+    path.join('changes', resolvedChangeId, 'patch-plan.md'),
+    path.join('changes', resolvedChangeId, 'verification-report.md')
+  ];
+
+  writeFile(
+    path.join(changeDir, 'delta-spec.md'),
+    `# Delta Spec
+
+## Change ID
+${resolvedChangeId}
+
+## Issue
+${issue}
+
+## Type
+bugfix | feature | refactor-safe | hotfix
+
+## Context
+
+## Current Behavior
+
+## Expected Behavior
+
+## Evidence
+
+## Root Cause Hypothesis
+
+## Impacted Areas
+
+## Constraints
+
+## Minimal Safe Delta
+
+## Alternatives Considered
+
+## Acceptance Criteria
+
+## Verification Strategy
+`
+  );
+
+  writeFile(
+    path.join(changeDir, 'patch-plan.md'),
+    `# Patch Plan
+
+## Change ID
+${resolvedChangeId}
+
+## Issue
+${issue}
+
+## Files to Inspect
+
+## Files to Change
+
+## Execution Steps
+1. Reproduce issue
+2. Confirm root cause
+3. Apply minimal change
+4. Adjust tests
+5. Run validations
+
+## Regression Risks
+
+## Rollback Strategy
+`
+  );
+
+  writeFile(
+    path.join(changeDir, 'verification-report.md'),
+    `# Verification Report
+
+## Change ID
+${resolvedChangeId}
+
+## Issue
+${issue}
+
+## Reproduction
+
+## Changes Made
+
+## Tests Run
+
+## Manual Validation
+
+## Residual Risks
+
+## Final Status
+pending
+`
+  );
+
+  return {
+    changeId: resolvedChangeId,
+    changeDir,
+    files
+  };
+}

package/src/core/pr-manager.js

@@ -1,21 +1,21 @@
-import fs from 'fs';
-import path from 'path';
-import { execSync } from 'child_process';
-
-function exec(command) {
-  execSync(command, { stdio: 'inherit' });
-}
-
-export async function openPullRequest({ issue, changeId, changeDir }) {
-  const branch = `pdd/${changeId}`;
-  const title = `fix: ${issue}`;
-
-  fs.writeFileSync(path.join(changeDir, 'pr-title.txt'), title);
-  fs.writeFileSync(path.join(changeDir, 'pr-body.md'), issue);
-
-  exec(`git checkout -b ${branch}`);
-  exec('git add .');
-  exec(`git commit -m "${title}"`);
-
-  console.log('PR ready (use IDE to open)');
-}
+import fs from 'fs';
+import path from 'path';
+import { execFileSync } from 'child_process';
+
+function runGit(args, baseDir = process.cwd()) {
+  execFileSync('git', args, { stdio: 'inherit', cwd: baseDir });
+}
+
+export async function openPullRequest({ issue, changeId, changeDir, baseDir = process.cwd() }) {
+  const branch = `pdd/${changeId}`;
+  const title = `fix: ${issue}`;
+
+  fs.writeFileSync(path.join(changeDir, 'pr-title.txt'), title);
+  fs.writeFileSync(path.join(changeDir, 'pr-body.md'), issue);
+
+  runGit(['checkout', '-b', branch], baseDir);
+  runGit(['add', '.'], baseDir);
+  runGit(['commit', '-m', title], baseDir);
+
+  console.log('PR ready (use IDE to open)');
+}