feat: Add constant, better error handling, new path for prompts

Author: NathBo470

.github/workflows/ci-cd.yml

@@ -0,0 +1,52 @@
+name: CI/CD Pipeline
+
+on:
+  push:
+    branches:
+      - production-release
+  pull_request:
+    branches:
+      - production-release
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v2
+
+      - name: Set up Node.js
+        uses: actions/setup-node@v2
+        with:
+          node-version: '14'
+
+      - name: Install dependencies
+        run: npm install
+
+      - name: Build the extension
+        run: npm run build
+
+      - name: Package the extension
+        run: npm run package
+
+  deploy:
+    runs-on: ubuntu-latest
+    needs: build
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v2
+
+      - name: Upload VSIX file
+        uses: actions/upload-artifact@v2
+        with:
+          name: vscode-extension
+          path: path/to/your/package.vsix
+
+      - name: Deploy to Marketplace
+        run: |
+          echo "Deploying to the marketplace..."
+          # Add your deployment script or command here
+          # For example, using vsce to publish the extension
+          npx vsce publish --pat ${{ secrets.VSCE_PAT }}

.vscode/launch.json

@@ -11,6 +11,7 @@
             "outFiles": [
                 "${workspaceFolder}/out/**/*.js"
             ],
+            "preLaunchTask": "npm: compile",
         }
     ]
 }

src/configManager.ts

@@ -39,7 +39,7 @@ export class ConfigManager {
     }
 
     get branch(): string {
-        return vscode.workspace.getConfiguration('promptsSync').get('branch', 'master');
+        return vscode.workspace.getConfiguration('promptsSync').get('branch', 'main');
     }
 
     get syncOnStartup(): boolean {
@@ -59,7 +59,7 @@ export class ConfigManager {
     }
 
     get syncInstructions(): boolean {
-        return vscode.workspace.getConfiguration('promptsSync').get('syncInstructions', true);
+        return vscode.workspace.getConfiguration('promptsSync').get('syncInstructions', false);
     }
 
     get syncPrompt(): boolean {

src/constant.ts

@@ -1,4 +1,3 @@
-export const REPO_ROOT_PROMPT_PATH = 'prompts/';
-export const REPO_SYNC_CHAT_MODE_PATH = `${REPO_ROOT_PROMPT_PATH}chatmode/`;
-export const REPO_SYNC_INSTRUCTIONS_PATH = `${REPO_ROOT_PROMPT_PATH}instructions/`;
-export const REPO_SYNC_PROMPT_PATH = `${REPO_ROOT_PROMPT_PATH}prompt/`;
+export const REPO_SYNC_CHAT_MODE_PATH = `chatmode/`;
+export const REPO_SYNC_INSTRUCTIONS_PATH = `instructions/`;
+export const REPO_SYNC_PROMPT_PATH = `prompts/`;

src/syncManager.ts

@@ -148,6 +148,7 @@ export class SyncManager {
         if (this.config.syncPrompt) {
             allowedPaths.push(REPO_SYNC_PROMPT_PATH);
         }
+        console.log('Allowed paths for sync:', allowedPaths);
 
         // If no types are selected, return empty array
         if (allowedPaths.length === 0) {
@@ -159,7 +160,7 @@ export class SyncManager {
             if (item.type !== 'blob') {
                 return false;
             }
-            
+
             return allowedPaths.some(path => item.path.startsWith(path)) &&
                    (item.path.endsWith('.md') || item.path.endsWith('.txt'));
         });
@@ -172,17 +173,28 @@ export class SyncManager {
         let itemsUpdated = 0;
 
         for (const file of files) {
+            this.logger.debug(`Syncing file: ${file.path}`);
+            let content = null;
+
+            try {
+                content = await this.github.getFileContent(owner, repo, file.path, this.config.branch);
+            } catch (error) {
+                // An error occured will retrieving file content, Return here
+                this.logger.warn(`Failed to fetch content for ${file.path}: ${error}`);
+                this.notifications.showSyncError(`Failed to fetch content for ${file.path}: ${error}. Make sure that the correct is set. Current branch: ${this.config.branch}`);
+                return itemsUpdated;
+            }
+
             try {
-                this.logger.debug(`Syncing file: ${file.path}`);
-                
-                // Get file content from GitHub
-                const content = await this.github.getFileContent(owner, repo, file.path, this.config.branch);
-                
                 // Flatten the structure - extract just the filename and place directly in prompts directory
                 const fileName = this.fileSystem.getBasename(file.path);
                 const localPath = this.fileSystem.joinPath(promptsDir, fileName);
 
                 // Check if file needs updating
+                if(!content) {
+                    this.logger.warn(`No content retrieved for ${file.path}, skipping`);
+                    continue;
+                }
                 const needsUpdate = await this.shouldUpdateFile(localPath, content);
 
                 if (needsUpdate) {
@@ -221,7 +233,7 @@ export class SyncManager {
 
                 // Filter relevant files
                 const relevantFiles = this.filterRelevantFiles(tree.tree);
-
+                
                 if(relevantFiles.length === 0) {
                     this.logger.warn(`No relevant files found to sync in ${repoUrl} based on current settings`);
                     const promptLocation = `${REPO_SYNC_CHAT_MODE_PATH}, ${REPO_SYNC_INSTRUCTIONS_PATH}, ${REPO_SYNC_PROMPT_PATH}`;

src/utils/github.ts

@@ -88,7 +88,7 @@ export class GitHubApiManager {
         if (!session) {
             throw new Error('GitHub authentication required');
         }
-
+        console.log(`Fetching file content from ${owner}/${repo}/${path} on branch ${branch}`);
         const response = await fetch(`${this.baseUrl}/repos/${owner}/${repo}/contents/${path}?ref=${branch}`, {
             headers: {
                 'Authorization': `Bearer ${session.accessToken}`,