scriptoriumadmin/Roo-Code
1
0
Fork 0
mirror of https://github.com/RooVetGit/Roo-Code.git synced 2026-08-28 05:27:24 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

add sourceURL option to yaml

Browse source
This commit is contained in:
Smartsheet-JB-Brown 2025-04-10 14:42:11 -07:00
parent 655899673b
commit fd48ffcfe9
7 changed files with 60 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
package-manager-template/README.md
View file

@ -52,6 +52,7 @@ author: "Author Name"
version: "1.0.0"
lastUpdated: "2025-04-08"
tags: ["tag1", "tag2"]
sourceUrl: "https://github.com/username/repo" # Optional URL for the "view source" button
```
## Testing

3
package-manager-template/mcp-servers/file-analyzer/metadata.yml
View file

@ -4,4 +4,5 @@ type: "mcp-server"
author: "Roo Team"
version: "1.0.0"
lastUpdated: "2025-04-08"
tags: ["file-analyzer", "code-quality", "security", "performance"]
tags: ["file-analyzer", "code-quality", "security", "performance"]
sourceUrl: "https://github.com/roo-team/file-analyzer-server"

3
package-manager-template/roles/developer-role/metadata.yml
View file

@ -4,4 +4,5 @@ type: "role"
author: "Roo Team"
version: "1.0.0"
lastUpdated: "2025-04-08"
tags: ["developer", "full-stack", "web", "database", "api"]
tags: ["developer", "full-stack", "web", "database", "api"]
sourceUrl: "https://github.com/roo-team/developer-resources"

3
package-manager-template/storage-systems/github-storage/metadata.yml
View file

@ -4,4 +4,5 @@ type: "storage"
author: "Roo Team"
version: "1.0.0"
lastUpdated: "2025-04-08"
tags: ["storage", "github", "git", "repository"]
tags: ["storage", "github", "git", "repository"]
sourceUrl: "https://github.com/roo-team/github-storage-system"

4
src/services/package-manager/GitFetcher.ts
View file

@ -276,6 +276,7 @@ export class GitFetcher {
const type = metadataContent.match(/type:\s*["']?([^"'\n]+)["']?/)?.[1] || dirType.type;
const author = metadataContent.match(/author:\s*["']?([^"'\n]+)["']?/)?.[1];
const version = metadataContent.match(/version:\s*["']?([^"'\n]+)["']?/)?.[1];
const sourceUrl = metadataContent.match(/sourceUrl:\s*["']?([^"'\n]+)["']?/)?.[1];
// Parse tags if present
const tagsMatch = metadataContent.match(/tags:\s*\[(.*?)\]/);
@ -291,7 +292,8 @@ export class GitFetcher {
repoUrl,
author,
tags,
version
version,
sourceUrl
};
items.push(item);

1
src/services/package-manager/types.ts
View file

@ -13,6 +13,7 @@ export interface PackageManagerItem {
lastUpdated?: string;
stars?: number;
downloads?: number;
sourceUrl?: string; // Optional URL to use for the "view source" button
}
/**

60
webview-ui/src/components/package-manager/PackageManagerView.tsx
View file

@ -38,9 +38,13 @@ const PackageManagerView = ({ onDone }: PackageManagerViewProps) => {
// Track if we're currently fetching items to prevent duplicate requests
const [isFetching, setIsFetching] = useState(false);
// Track if the fetch was manually triggered by a refresh button
const isManualRefresh = useRef(false);
// Use a ref to track if we've already fetched items
const hasInitialFetch = useRef(false);
// Track the last sources we fetched to avoid duplicate fetches
const lastSourcesKey = useRef<string | null>(null);
// Fetch function without debounce for immediate execution
const fetchPackageManagerItems = useCallback(() => {
@ -88,10 +92,19 @@ const PackageManagerView = ({ onDone }: PackageManagerViewProps) => {
itemsLength: items.length
});
// Only fetch if packageManagerSources changes and we're not already fetching
if (packageManagerSources && hasInitialFetch.current && !isFetching) {
console.log("DEBUG: Calling fetchPackageManagerItems due to sources change");
fetchPackageManagerItems();
// Only fetch if packageManagerSources changes, we're not already fetching, and this isn't the initial render
if (packageManagerSources && hasInitialFetch.current && !isFetching && packageManagerSources.length > 0) {
// Generate a key based on the current sources
const sourcesKey = JSON.stringify(packageManagerSources.map(s => s.url));
// Only fetch if the sources have changed and it's not a manual refresh
if (sourcesKey !== lastSourcesKey.current && !isManualRefresh.current) {
console.log("DEBUG: Calling fetchPackageManagerItems due to sources change");
lastSourcesKey.current = sourcesKey;
fetchPackageManagerItems();
} else {
console.log("DEBUG: Skipping fetch because sources haven't changed or manual refresh is in progress");
}
}
}, [packageManagerSources, fetchPackageManagerItems, isFetching]);
@ -147,12 +160,23 @@ const PackageManagerView = ({ onDone }: PackageManagerViewProps) => {
// Force a new array reference to ensure React detects the change
setItems([...receivedItems]);
setIsFetching(false);
console.log("DEBUG: States updated - items:", receivedItems.length, "isFetching: false");
// Update the fetching state in a separate call to avoid triggering another fetch
setTimeout(() => {
setIsFetching(false);
isManualRefresh.current = false; // Reset the manual refresh flag
console.log("DEBUG: States updated - items:", receivedItems.length, "isFetching: false, isManualRefresh: false");
}, 0);
} else {
console.log("DEBUG: Received empty items array");
setItems([]);
setIsFetching(false);
// Update the fetching state in a separate call to avoid triggering another fetch
setTimeout(() => {
setIsFetching(false);
isManualRefresh.current = false; // Reset the manual refresh flag
console.log("DEBUG: States updated - items: 0, isFetching: false, isManualRefresh: false");
}, 0);
}
}
}
@ -317,6 +341,7 @@ const PackageManagerView = ({ onDone }: PackageManagerViewProps) => {
<p>No package manager items found</p>
<Button
onClick={() => {
isManualRefresh.current = true;
vscode.postMessage({
type: "fetchPackageManagerItems",
forceRefresh: true
@ -336,6 +361,7 @@ const PackageManagerView = ({ onDone }: PackageManagerViewProps) => {
</p>
<Button
onClick={() => {
isManualRefresh.current = true;
vscode.postMessage({
type: "fetchPackageManagerItems",
forceRefresh: true
@ -374,6 +400,16 @@ const PackageManagerView = ({ onDone }: PackageManagerViewProps) => {
const PackageManagerItemCard = ({ item }: { item: PackageManagerItem }) => {
const { t } = useAppTranslation();
// Helper function to validate URL
const isValidUrl = (urlString: string): boolean => {
try {
new URL(urlString);
return true;
} catch (e) {
return false;
}
};
const getTypeLabel = (type: string) => {
switch (type) {
case "role":
@ -401,12 +437,14 @@ const PackageManagerItemCard = ({ item }: { item: PackageManagerItem }) => {
};
const handleOpenUrl = () => {
console.log(`PackageManagerItemCard: Opening URL: ${item.url}`);
// Use sourceUrl if it exists and is a valid URL, otherwise fall back to url
const urlToOpen = item.sourceUrl && isValidUrl(item.sourceUrl) ? item.sourceUrl : item.url;
console.log(`PackageManagerItemCard: Opening URL: ${urlToOpen}`);
vscode.postMessage({
type: "openExternal",
url: item.url
url: urlToOpen
});
console.log(`PackageManagerItemCard: Sent openExternal message with URL: ${item.url}`);
console.log(`PackageManagerItemCard: Sent openExternal message with URL: ${urlToOpen}`);
};
return (
@ -470,7 +508,7 @@ const PackageManagerItemCard = ({ item }: { item: PackageManagerItem }) => {
<Button onClick={handleOpenUrl}>
<span className="codicon codicon-link-external mr-2"></span>
View on GitHub
{item.sourceUrl && isValidUrl(item.sourceUrl) ? "View Source" : "View on GitHub"}
</Button>
</div>
</div>