skillsets 0.4.6 → 0.4.7

package/dist/commands/submit.js

@@ -204,13 +204,25 @@ export async function submit() {
         if (pushResult.status !== 0) {
             throw new Error(pushResult.stderr || 'Failed to push to fork');
         }
-        // Create PR
-        spinner.text = 'Creating pull request...';
-        const prTitle = isUpdate
-            ? `Update ${skillsetId} to v${skillset.version}`
-            : `Add ${skillsetId}`;
-        const prBody = isUpdate
-            ? `## Skillset Update
+        // Check if PR already exists for this branch (force push already updated it)
+        spinner.text = 'Checking for existing pull request...';
+        const existingPr = spawnSync('gh', ['pr', 'list', '--repo', REGISTRY_REPO, '--head', `${username}:${branchName}`, '--json', 'url', '--jq', '.[0].url', '--state', 'open', '--limit', '1'], {
+            cwd: tempDir,
+            encoding: 'utf-8',
+        });
+        let prUrl;
+        if (existingPr.status === 0 && existingPr.stdout.trim()) {
+            // PR exists — force push already updated the branch, nothing else to do
+            prUrl = existingPr.stdout.trim();
+        }
+        else {
+            // Create new PR
+            spinner.text = 'Creating pull request...';
+            const prTitle = isUpdate
+                ? `Update ${skillsetId} to v${skillset.version}`
+                : `Add ${skillsetId}`;
+            const prBody = isUpdate
+                ? `## Skillset Update
 
 **Skillset:** ${skillsetId}
 **Version:** ${existingVersion} → ${skillset.version}
@@ -230,7 +242,7 @@ _Describe what changed in this version._
 ---
 Submitted via \`npx skillsets submit\`
 `
-            : `## New Skillset Submission
+                : `## New Skillset Submission
 
 **Skillset:** ${skillsetId}
 **Version:** ${skillset.version}
@@ -251,36 +263,20 @@ _Add any additional context for reviewers here._
 ---
 Submitted via \`npx skillsets submit\`
 `;
-        // Check if PR already exists for this branch
-        const existingPr = spawnSync('gh', ['pr', 'list', '--repo', REGISTRY_REPO, '--head', `${username}:${branchName}`, '--json', 'url', '--jq', '.[0].url', '--state', 'open', '--limit', '1'], {
-            cwd: tempDir,
-            encoding: 'utf-8',
-        });
-        let prResult;
-        if (existingPr.status === 0 && existingPr.stdout.trim()) {
-            // PR exists — force push already updated it, just edit title/body
-            prResult = spawnSync('gh', ['pr', 'edit', '--repo', REGISTRY_REPO, existingPr.stdout.trim(), '--title', prTitle, '--body', prBody], {
+            const prResult = spawnSync('gh', ['pr', 'create', '--repo', REGISTRY_REPO, '--head', `${username}:${branchName}`, '--title', prTitle, '--body', prBody], {
                 cwd: tempDir,
                 encoding: 'utf-8',
             });
-            prResult.stdout = existingPr.stdout;
-        }
-        else {
-            // Create new PR
-            prResult = spawnSync('gh', ['pr', 'create', '--repo', REGISTRY_REPO, '--head', `${username}:${branchName}`, '--title', prTitle, '--body', prBody], {
-                cwd: tempDir,
-                encoding: 'utf-8',
-            });
-        }
-        if (prResult.status !== 0) {
-            throw new Error(prResult.stderr || 'Failed to create PR');
+            if (prResult.status !== 0) {
+                throw new Error(prResult.stderr || 'Failed to create PR');
+            }
+            prUrl = (prResult.stdout || '').trim();
         }
         // Cleanup
         spinner.text = 'Cleaning up...';
         rmSync(tempDir, { recursive: true, force: true });
-        spinner.succeed('Pull request created');
-        // Extract PR URL from result
-        const prUrl = (prResult.stdout || '').trim();
+        const prExisted = existingPr.status === 0 && existingPr.stdout.trim();
+        spinner.succeed(prExisted ? 'Pull request updated' : 'Pull request created');
         console.log(chalk.green(`\n✓ ${isUpdate ? 'Update' : 'Submission'} complete!\n`));
         console.log(`  Skillset: ${chalk.bold(skillsetId)}`);
         if (isUpdate) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skillsets",
-  "version": "0.4.6",
+  "version": "0.4.7",
   "description": "CLI tool for discovering and installing verified skillsets",
   "type": "module",
   "bin": {