mcp-osp-prompt 1.0.0 → 1.0.2

package/platform-utils.js

@@ -3,7 +3,7 @@
  * Shared between config.js and fetcher.js to avoid code duplication
  */
 
-// Platform detection from RESOURCE_PATH (or legacy PROMPT_PATH)
+// Platform detection from PROMPT_PATH
 export function detectPlatformFromPath(promptPath) {
   if (!promptPath) {
     return null;
@@ -29,15 +29,14 @@ export function detectPlatformFromPath(promptPath) {
 }
 
 // Parse remote path information with universal support for complex URLs
-// 🟢 GREEN: Task 1.1 - 支持子目录参数，用于多目录资源拉取
-export function parseRemotePath(promptPath, subDirectory = '') {
+export function parseRemotePath(promptPath) {
   // 🟢 GREEN: Task 1.1 - 添加null/undefined检查和详细错误信息
   if (!promptPath) {
-    throw new Error('RESOURCE_PATH cannot be null or empty. Please set a valid remote URL or local path.');
+    throw new Error('PROMPT_PATH cannot be null or empty. Please set a valid remote URL or local path.');
   }
   
   if (typeof promptPath !== 'string') {
-    throw new Error(`RESOURCE_PATH must be a string, received: ${typeof promptPath}`); // Note: parameter name 'promptPath' kept for compatibility
+    throw new Error(`PROMPT_PATH must be a string, received: ${typeof promptPath}`);
   }
   
   const cleanPath = promptPath.replace(/^https?:\/\//, '');
@@ -74,27 +73,27 @@ export function parseRemotePath(promptPath, subDirectory = '') {
         }
       }
       
-      return appendSubDirectory({
+      return {
         platform: 'github',
         project,
         repo: project, // Alias for backward compatibility
         branch: branch || 'main',
         path: path || '', // File/directory path
         apiUrl: `https://api.github.com/repos/${project}/contents/${path || ''}`
-      }, subDirectory);
+      };
     }
     
     // Match simple GitHub project URLs
     const simpleMatch = cleanPath.match(/github\.com\/([^\/]+\/[^\/]+)/);
     if (simpleMatch) {
-      return appendSubDirectory({
+      return {
         platform: 'github',
         project: simpleMatch[1],
         repo: simpleMatch[1],
         branch: 'main',
         path: '',
         apiUrl: `https://api.github.com/repos/${simpleMatch[1]}/contents`
-      }, subDirectory);
+      };
     }
   }
   
@@ -104,72 +103,46 @@ export function parseRemotePath(promptPath, subDirectory = '') {
     const treeMatch = cleanPath.match(/gitlab\.com\/(.*?)\/-\/tree\/([^\/]+)\/?(.*)/);
     if (treeMatch) {
       const [, project, branch, path] = treeMatch;
-      return appendSubDirectory({
+      return {
         platform: 'gitlab',
         project,
         group: project, // Alias for backward compatibility
         branch,
         path: path || '', // File/directory path
         apiUrl: `https://gitlab.com/api/v4/projects/${encodeURIComponent(project)}/repository/tree?path=${path || ''}&ref=${branch}&recursive=true&per_page=100`
-      }, subDirectory);
+      };
     }
     
     // Match simple GitLab project URLs
     const simpleMatch = cleanPath.match(/gitlab\.com\/(.+?)(?:\/|$)/);
     if (simpleMatch) {
       const project = simpleMatch[1];
-      return appendSubDirectory({
+      return {
         platform: 'gitlab',
         project,
         group: project,
         branch: 'master',
         path: '',
         apiUrl: `https://gitlab.com/api/v4/projects/${encodeURIComponent(project)}/repository/tree?ref=master&recursive=true&per_page=100`
-      }, subDirectory);
+      };
     }
   }
   
   // Handle simple format like "user/repo" (defaults to GitHub)
   if (promptPath.match(/^[\w-]+\/[\w-]+$/)) {
-    return appendSubDirectory({
+    return {
       platform: 'github',
       project: promptPath,
       repo: promptPath,
       branch: 'main',
       path: '',
       apiUrl: `https://api.github.com/repos/${promptPath}/contents`
-    }, subDirectory);
+    };
   }
   
   return null;
 }
 
-// 🟢 GREEN: Task 1.1 - 辅助函数：追加子目录到路径信息
-function appendSubDirectory(pathInfo, subDirectory) {
-  if (!subDirectory || !pathInfo) {
-    return pathInfo;
-  }
-  
-  // 追加子目录到path
-  const newPath = pathInfo.path ? `${pathInfo.path}/${subDirectory}` : subDirectory;
-  
-  // 根据平台更新apiUrl
-  let newApiUrl = pathInfo.apiUrl;
-  if (pathInfo.platform === 'github') {
-    // GitHub: https://api.github.com/repos/{project}/contents/{path}
-    newApiUrl = `https://api.github.com/repos/${pathInfo.project}/contents/${newPath}`;
-  } else if (pathInfo.platform === 'gitlab') {
-    // GitLab: https://gitlab.com/api/v4/projects/{project}/repository/tree?path={path}&ref={branch}&recursive=true&per_page=100
-    newApiUrl = `https://gitlab.com/api/v4/projects/${encodeURIComponent(pathInfo.project)}/repository/tree?path=${newPath}&ref=${pathInfo.branch}&recursive=true&per_page=100`;
-  }
-  
-  return {
-    ...pathInfo,
-    path: newPath,
-    apiUrl: newApiUrl
-  };
-}
-
 // Create authentication headers based on platform
 export function createAuthHeaders(platform, token) {
   if (!token) {
@@ -191,12 +164,12 @@ export function createAuthHeaders(platform, token) {
 }
 
 // Validate configuration for given platform
-export function validatePlatformConfiguration(resourcePath, token) {
-  if (!resourcePath) {
-    throw new Error('RESOURCE_PATH is required. Set it to a local directory path, GitHub repo (user/repo), or full URL.');
+export function validatePlatformConfiguration(promptPath, token) {
+  if (!promptPath) {
+    throw new Error('PROMPT_PATH is required. Set it to a local directory path, GitHub repo (user/repo), or full URL.');
   }
   
-  const platform = detectPlatformFromPath(resourcePath);
+  const platform = detectPlatformFromPath(promptPath);
   
   if ((platform === 'github' || platform === 'gitlab') && !token) {
     throw new Error('GIT_TOKEN is required for remote repositories. Please set your GitHub or GitLab access token.');