feat(ui): add Report Issue, Request Feature, and Retry Feedback

gblanc-1a · claude · gblanc-1a · commit b05bd7d51051 · 2026-02-23T10:15:00.000+01:00
Add Report Issue and Request Feature links in marketplace bundle cards
and TreeView context menu. Opens source repository issues page with
pre-filled bug or feature template.

Add Retry Feedback Submission to context menu for re-submitting
locally saved feedback that failed due to network issues.

Co-Authored-By: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/package.json b/package.json
@@ -439,6 +439,24 @@
         "title": "Rate & Feedback",
         "category": "Prompt Registry",
         "icon": "$(star-full)"
+      },
+      {
+        "command": "promptRegistry.reportIssue",
+        "title": "Report Issue",
+        "category": "Prompt Registry",
+        "icon": "$(bug)"
+      },
+      {
+        "command": "promptRegistry.requestFeature",
+        "title": "Request Feature",
+        "category": "Prompt Registry",
+        "icon": "$(lightbulb)"
+      },
+      {
+        "command": "promptRegistry.retryFeedback",
+        "title": "Retry Feedback Submission",
+        "category": "Prompt Registry",
+        "icon": "$(sync)"
       }
     ],
     "viewsContainers": {
@@ -659,6 +677,21 @@
           "when": "view == promptRegistryExplorer && viewItem =~ /^installed_bundle/",
           "group": "engagement@1"
         },
+        {
+          "command": "promptRegistry.reportIssue",
+          "when": "view == promptRegistryExplorer && viewItem =~ /^installed_bundle/",
+          "group": "engagement@2"
+        },
+        {
+          "command": "promptRegistry.requestFeature",
+          "when": "view == promptRegistryExplorer && viewItem =~ /^installed_bundle/",
+          "group": "engagement@3"
+        },
+        {
+          "command": "promptRegistry.retryFeedback",
+          "when": "view == promptRegistryExplorer && viewItem =~ /^installed_bundle/",
+          "group": "engagement@4"
+        },
         {
           "command": "promptRegistry.moveToRepositoryCommit",
           "when": "view == promptRegistryExplorer && viewItem =~ /^installed_bundle.*_user$/",
diff --git a/src/commands/FeedbackCommands.ts b/src/commands/FeedbackCommands.ts
@@ -85,6 +85,18 @@ export class FeedbackCommands {
             vscode.commands.registerCommand(
                 'promptRegistry.submitFeedback',
                 (item: FeedbackableItem | any) => this.submitFeedback(this.normalizeFeedbackItem(item))
+            ),
+            vscode.commands.registerCommand(
+                'promptRegistry.reportIssue',
+                (item: any) => this.reportIssue(this.normalizeFeedbackItem(item))
+            ),
+            vscode.commands.registerCommand(
+                'promptRegistry.requestFeature',
+                (item: any) => this.requestFeature(this.normalizeFeedbackItem(item))
+            ),
+            vscode.commands.registerCommand(
+                'promptRegistry.retryFeedback',
+                (item: any) => this.retryFeedback(this.normalizeFeedbackItem(item))
             )
         );
 
@@ -354,6 +366,139 @@ export class FeedbackCommands {
     }
 
 
+    /**
+     * Report an issue for a bundle (opens issue tracker with bug template)
+     */
+    async reportIssue(item: FeedbackableItem): Promise<void> {
+        await this.openIssueTrackerWithTemplate(item, 'bug');
+    }
+
+    /**
+     * Request a feature for a bundle (opens issue tracker with feature template)
+     */
+    async requestFeature(item: FeedbackableItem): Promise<void> {
+        await this.openIssueTrackerWithTemplate(item, 'feature');
+    }
+
+    /**
+     * Retry submitting unsynced feedback for a bundle
+     */
+    async retryFeedback(item: FeedbackableItem): Promise<void> {
+        const storage = this.engagementService?.getStorage?.();
+        if (!storage) return;
+
+        const unsynced = await storage.getUnsyncedFeedback();
+        const pending = unsynced.filter(f => f.bundleId === item.resourceId);
+
+        if (pending.length === 0) {
+            vscode.window.showInformationMessage('No pending feedback to retry.');
+            return;
+        }
+
+        for (const entry of pending) {
+            try {
+                if (this.engagementService) {
+                    await this.engagementService.submitFeedback(
+                        entry.resourceType,
+                        entry.bundleId,
+                        entry.comment || `Rated ${entry.rating} stars`,
+                        { rating: entry.rating, hubId: entry.hubId || undefined }
+                    );
+                    await storage.markFeedbackSynced(entry.id);
+                }
+            } catch (error) {
+                this.logger.warn(`Retry failed for ${entry.id}: ${error}`);
+            }
+        }
+
+        const stillUnsynced = (await storage.getUnsyncedFeedback()).filter(f => f.bundleId === item.resourceId);
+        if (stillUnsynced.length === 0) {
+            vscode.window.showInformationMessage('Feedback submitted successfully!');
+        } else {
+            vscode.window.showWarningMessage(`${stillUnsynced.length} feedback(s) still pending. Please try again later.`);
+        }
+    }
+
+    private async openIssueTrackerWithTemplate(
+        item: FeedbackableItem,
+        type: 'bug' | 'feature'
+    ): Promise<void> {
+        try {
+            if (!item.sourceUrl) {
+                vscode.window.showWarningMessage('No source repository URL available for this bundle.');
+                return;
+            }
+
+            let issueUrl = item.sourceUrl;
+            if (issueUrl.endsWith('.git')) {
+                issueUrl = issueUrl.slice(0, -4);
+            }
+
+            const githubMatch = issueUrl.match(/^(https?:\/\/github\.com\/[^/]+\/[^/]+)/);
+            if (githubMatch) {
+                issueUrl = `${githubMatch[1]}/issues/new`;
+            } else if (!issueUrl.includes('/issues')) {
+                issueUrl = `${issueUrl}/issues/new`;
+            }
+
+            const isAwesomeCopilot = item.sourceType === 'awesome-copilot';
+            const itemType = isAwesomeCopilot ? 'Collection' : 'Bundle';
+            const name = item.name || item.resourceId;
+
+            let title: string;
+            let bodyParts: string[];
+
+            if (type === 'bug') {
+                title = `[Bug Report] ${name}`;
+                bodyParts = [
+                    `${itemType} Information`,
+                    `- **${itemType} ID:** ${item.resourceId}`,
+                    ...(item.version ? [`- **Version:** ${item.version}`] : []),
+                    '',
+                    '## Bug Description',
+                    '_Describe the bug clearly and concisely_',
+                    '',
+                    '## Steps to Reproduce',
+                    '1. ',
+                    '2. ',
+                    '3. ',
+                    '',
+                    '## Expected Behavior',
+                    '_What did you expect to happen?_',
+                    '',
+                    '## Actual Behavior',
+                    '_What actually happened?_',
+                    '',
+                    '## Additional Context',
+                    '_Any other information that might be helpful_',
+                ];
+            } else {
+                title = `[Feature Request] ${name}`;
+                bodyParts = [
+                    `${itemType} Information`,
+                    `- **${itemType} ID:** ${item.resourceId}`,
+                    ...(item.version ? [`- **Version:** ${item.version}`] : []),
+                    '',
+                    '## Feature Description',
+                    '_Describe the feature you would like_',
+                    '',
+                    '## Use Case',
+                    '_Why would this feature be useful?_',
+                    '',
+                    '## Additional Context',
+                    '_Any other information or examples_',
+                ];
+            }
+
+            const params = new URLSearchParams({ title, body: bodyParts.join('\n') });
+            const uri = vscode.Uri.parse(`${issueUrl}?${params.toString()}`, true);
+            await vscode.env.openExternal(uri);
+        } catch (error) {
+            this.logger.warn('Could not open issue tracker', error as Error);
+            vscode.window.showWarningMessage('Could not open issue tracker. Please visit the repository manually.');
+        }
+    }
+
     /**
      * Save feedback to the engagement service
      */
diff --git a/src/ui/MarketplaceViewProvider.ts b/src/ui/MarketplaceViewProvider.ts
@@ -26,7 +26,7 @@ import { PendingFeedback } from '../types/pendingFeedback';
  * Message types sent from webview to extension
  */
 interface WebviewMessage {
-    type: 'refresh' | 'install' | 'update' | 'uninstall' | 'openDetails' | 'openPromptFile' | 'installVersion' | 'getVersions' | 'toggleAutoUpdate' | 'openSourceRepository' | 'completeSetup' | 'getFeedbacks' | 'submitFeedback';
+    type: 'refresh' | 'install' | 'update' | 'uninstall' | 'openDetails' | 'openPromptFile' | 'installVersion' | 'getVersions' | 'toggleAutoUpdate' | 'openSourceRepository' | 'completeSetup' | 'getFeedbacks' | 'submitFeedback' | 'reportIssue' | 'requestFeature';
     bundleId?: string;
     installPath?: string;
     filePath?: string;
@@ -553,6 +553,26 @@ export class MarketplaceViewProvider implements vscode.WebviewViewProvider {
                     await this.handleWebviewFeedback(message.bundleId, message.rating, message.comment);
                 }
                 break;
+            case 'reportIssue':
+            case 'requestFeature':
+                if (message.bundleId) {
+                    const command = message.type === 'reportIssue' ? 'promptRegistry.reportIssue' : 'promptRegistry.requestFeature';
+                    const bundles = await this.registryManager.searchBundles({ text: message.bundleId });
+                    const foundBundle = bundles.find(b => b.id === message.bundleId);
+                    if (foundBundle) {
+                        const allSources = await this.registryManager.listSources();
+                        const foundSource = allSources.find(s => s.id === foundBundle.sourceId);
+                        await vscode.commands.executeCommand(command, {
+                            resourceId: foundBundle.id,
+                            resourceType: 'bundle',
+                            name: foundBundle.name,
+                            version: foundBundle.version,
+                            sourceUrl: foundSource?.url,
+                            sourceType: foundSource?.type,
+                        });
+                    }
+                }
+                break;
             default:
                 this.logger.warn(`Unknown message type: ${message.type}`);
         }
diff --git a/src/ui/webview/marketplace/marketplace.js b/src/ui/webview/marketplace/marketplace.js
@@ -516,6 +516,8 @@ function renderBundles() {
                 <button class="btn btn-link" data-action="openSourceRepo" data-bundle-id="${bundle.id}" title="Open Source Repository">
                     <svg width="14" height="14" viewBox="0 0 16 16" fill="currentColor"><path d="M4.5 3A1.5 1.5 0 0 0 3 4.5v7A1.5 1.5 0 0 0 4.5 13h7a1.5 1.5 0 0 0 1.5-1.5v-2a.5.5 0 0 1 1 0v2a2.5 2.5 0 0 1-2.5 2.5h-7A2.5 2.5 0 0 1 2 11.5v-7A2.5 2.5 0 0 1 4.5 2h2a.5.5 0 0 1 0 1h-2zM9 2.5a.5.5 0 0 1 .5-.5h4a.5.5 0 0 1 .5.5v4a.5.5 0 0 1-1 0V3.707l-5.146 5.147a.5.5 0 0 1-.708-.708L12.293 3H9.5a.5.5 0 0 1-.5-.5z"/></svg>
                 </button>
+                <button class="btn btn-link" data-action="reportIssue" data-bundle-id="${bundle.id}" title="Report Issue">Report Issue</button>
+                <button class="btn btn-link" data-action="requestFeature" data-bundle-id="${bundle.id}" title="Request Feature">Request Feature</button>
             </div>
         </div>
     `).join('');
@@ -949,6 +951,12 @@ document.addEventListener('mouseout', (e) => {
                 case 'cancelFeedback':
                     if (bundleId) cancelFeedback(bundleId, actionElement);
                     break;
+                case 'reportIssue':
+                    if (bundleId) vscode.postMessage({ type: 'reportIssue', bundleId });
+                    break;
+                case 'requestFeature':
+                    if (bundleId) vscode.postMessage({ type: 'requestFeature', bundleId });
+                    break;
             }
         }
     });
diff --git a/test/commands/FeedbackCommands.test.ts b/test/commands/FeedbackCommands.test.ts
@@ -232,9 +232,12 @@ suite('FeedbackCommands', () => {
 
             commands.registerCommands(mockContext);
 
-            assert.strictEqual(registerCommandStub.callCount, 2);
+            assert.strictEqual(registerCommandStub.callCount, 5);
             assert.ok(registerCommandStub.calledWith('promptRegistry.feedback'));
             assert.ok(registerCommandStub.calledWith('promptRegistry.submitFeedback'));
+            assert.ok(registerCommandStub.calledWith('promptRegistry.reportIssue'));
+            assert.ok(registerCommandStub.calledWith('promptRegistry.requestFeature'));
+            assert.ok(registerCommandStub.calledWith('promptRegistry.retryFeedback'));
         });
     });
 
