Skip packages API call for incomplete content (#226)

* Skip packages API call for incomplete content
* Update manifest checksums

Author: dotNomad

extensions/package-vulnerability-scanner/manifest.json

@@ -20,14 +20,14 @@
 	},
 	"packages": {},
 	"files": {
-		"dist/assets/index-BeB0Q4U9.js": {
-			"checksum": "eec48b7aef372e410f6eddb398ec9b53"
-		},
 		"dist/assets/index-C-y7BWfk.css": {
 			"checksum": "9c79124c79ef1f350c8af2a925a7f4ad"
 		},
+		"dist/assets/index-CIpfqLN8.js": {
+			"checksum": "f39ee65bd9c1a23d4b5babf37e39683f"
+		},
 		"dist/index.html": {
-			"checksum": "a555313182523439d7b4001d392385af"
+			"checksum": "13d888e5f1a3a3c9953e1099082a9d48"
 		},
 		"main.py": {
 			"checksum": "f8385dbd8a8cd24204f1eb6209f8bb30"

extensions/package-vulnerability-scanner/src/components/ContentCard.vue

@@ -15,6 +15,7 @@ const props = defineProps<{
     py_version?: string;
     r_version?: string;
     quarto_version?: string;
+    bundle_id?: string | null;
   };
 }>();
 
@@ -35,6 +36,7 @@ const hasError = computed(
 const packageCount = computed(
   () => packagesStore.contentItems[props.content.guid]?.packages.length || 0,
 );
+const isIncomplete = computed(() => props.content.bundle_id === null);
 
 // Count vulnerable packages in this content item
 function countVulnerablePackages(): number {
@@ -117,6 +119,9 @@ function handleClick() {
       <span v-if="packageCount > 0" class="text-gray-600">
         {{ packageCount }} packages
       </span>
+      <span v-else-if="isIncomplete" class="text-red-700">
+        Incomplete content
+      </span>
       <span v-else-if="hasError" class="text-red-700">
         Error loading packages
       </span>

extensions/package-vulnerability-scanner/src/components/ContentList.vue

@@ -21,6 +21,14 @@ async function fetchPackagesInBatches(batchSize = 3) {
 
     // Fetch packages for this batch in parallel
     const fetchPromises = batch.map(async (content) => {
+      if (content.bundle_id === null) {
+        packagesStore.setPackagesForContent(
+          content.guid,
+          [],
+          new Error("This content has not been fully deployed."),
+        );
+        return;
+      }
       try {
         return await packagesStore.fetchPackagesForContent(content.guid);
       } catch (err) {

extensions/package-vulnerability-scanner/src/components/VulnerabilityChecker.vue

@@ -148,6 +148,11 @@ const filteredPackages = computed((): PackageWithVulnsAndFix[] => {
       return packages.value;
   }
 });
+
+const isIncomplete = computed(() => {
+  const content = contentStore.currentContent;
+  return content ? content.bundle_id === null : false;
+});
 </script>
 
 <template>
@@ -198,7 +203,12 @@ const filteredPackages = computed((): PackageWithVulnsAndFix[] => {
     <!-- Loading state -->
     <LoadingSpinner v-if="isLoading" :message="loadingMessage" size="md" />
 
-    <!-- Error state -->
+    <StatusMessage
+      v-if="isIncomplete"
+      type="error"
+      message="This has not been fully deployed. Try publishing again."
+    />
+
     <StatusMessage
       v-else-if="hasError"
       type="error"

extensions/package-vulnerability-scanner/src/stores/content.ts

@@ -13,6 +13,7 @@ export interface ContentListItem {
   py_version?: string;
   r_version?: string;
   quarto_version?: string;
+  bundle_id?: string | null;
 }
 
 export const useContentStore = defineStore("content", () => {

extensions/package-vulnerability-scanner/src/stores/packages.ts

@@ -25,7 +25,6 @@ export const usePackagesStore = defineStore("packages", () => {
 
   // Fetch packages for a specific content ID
   async function fetchPackagesForContent(contentId: string) {
-    // Initialize or update the content item
     if (!contentItems.value[contentId]) {
       // Initialize content item
       contentItems.value[contentId] = {
@@ -65,6 +64,21 @@ export const usePackagesStore = defineStore("packages", () => {
     }
   }
 
+  function setPackagesForContent(
+    guid: string,
+    packages: Package[],
+    error: Error | null = null,
+  ) {
+    contentItems.value[guid] = {
+      guid,
+      packages,
+      isLoading: false,
+      error,
+      isFetched: true,
+      lastFetchTime: new Date(),
+    };
+  }
+
   return {
     // State
     contentItems,
@@ -73,5 +87,6 @@ export const usePackagesStore = defineStore("packages", () => {
 
     // Actions
     fetchPackagesForContent,
+    setPackagesForContent,
   };
 });