@firestartr/cli 2.1.0-snapshot → 2.1.0-snapshot-1

package/build/index.js

@@ -367438,12 +367438,12 @@ class RepoGithubDecanter extends GithubDecanter {
             });
         }
         const directWriters = this.data.teamsAndMembers.directMembers
-            .filter((member) => member.role === 'push')
+            .filter((member) => member.role === 'write')
             .map((member) => {
             return `user-imported-ref:${member.name}`;
         });
         const outsideWriters = this.data.teamsAndMembers.outsideMembers
-            .filter((member) => member.role === 'push')
+            .filter((member) => member.role === 'write')
             .map((member) => {
             return `collaborator:${member.name.toLowerCase()}`;
         });
@@ -367462,12 +367462,12 @@ class RepoGithubDecanter extends GithubDecanter {
             });
         }
         const directReaders = this.data.teamsAndMembers.directMembers
-            .filter((member) => member.role === 'pull')
+            .filter((member) => member.role === 'read')
             .map((member) => {
             return `user-imported-ref:${member.name}`;
         });
         const outsideReaders = this.data.teamsAndMembers.outsideMembers
-            .filter((member) => member.role === 'pull')
+            .filter((member) => member.role === 'read')
             .map((member) => {
             return `collaborator:${member.name.toLowerCase()}`;
         });
@@ -367502,7 +367502,11 @@ class RepoGithubDecanter extends GithubDecanter {
             };
         });
         this.data['teamsAndMembers']['teams'] = teams;
-        this.data['teamsAndMembers']['directMembers'] = directMembers;
+        // GitHub's `affiliation=direct` includes outside collaborators, so we must
+        // exclude them from directMembers to avoid tagging them as user-imported-ref.
+        const outsideMemberNames = new Set(outsideMembers.map((m) => m.name.toLowerCase()));
+        const filteredDirectMembers = directMembers.filter((m) => !outsideMemberNames.has(m.name.toLowerCase()));
+        this.data['teamsAndMembers']['directMembers'] = filteredDirectMembers;
         this.data['teamsAndMembers']['outsideMembers'] = outsideMembers;
     }
     async __gatherPages() {
@@ -367923,7 +367927,7 @@ async function reimportGithubGitopsRepository(org, crsPath, configPath, generate
             }
             cr.metadata.annotations[reImportAnnotation] = 'true';
             cr.metadata.annotations[importAnnotation] = 'true';
-            // we set an annotion to trigger the update
+            // we set an annotation to trigger the update
             cr.metadata.annotations[reconcileAtAnnotation] = new Date()
                 .toISOString()
                 .replace(/\.\d{3}Z$/, 'Z'); // exact format: 2026-04-19T20:45:00Z
@@ -367935,10 +367939,10 @@ async function reimportGithubGitopsRepository(org, crsPath, configPath, generate
     // we need to search for dependencies if proceeds
     if (!collection.IS_SKIP_SET(generatedFilters, 'gh-repo')) {
         importer_src_logger.info(`Searching for dependencies to annotate with ${reImportAnnotation} and ${importAnnotation}`);
-        await searchForDependencies(org, crsPath, configPath, generatedFilters, crsWithDependenciesToAnnotate, reImportAnnotation, importAnnotation);
+        await searchForDependencies(org, crsPath, configPath, generatedFilters, crsWithDependenciesToAnnotate, reImportAnnotation, importAnnotation, reconcileAtAnnotation);
     }
 }
-async function searchForDependencies(org, crsPath, configPath, generatedFilters, crsWithDependenciesToAnnotate, reImportAnnotation, importAnnotation) {
+async function searchForDependencies(org, crsPath, configPath, generatedFilters, crsWithDependenciesToAnnotate, reImportAnnotation, importAnnotation, reconcileAtAnnotation) {
     // we need to perform a search in all the crsPath
     await searchCRs(crsPath, async (cr, filePath) => {
         importer_src_logger.debug(`${JSON.stringify(cr)}`);
@@ -367956,6 +367960,10 @@ async function searchForDependencies(org, crsPath, configPath, generatedFilters,
                     }
                     cr.metadata.annotations[reImportAnnotation] = 'true';
                     cr.metadata.annotations[importAnnotation] = 'true';
+                    // we set an annotation to trigger the update
+                    cr.metadata.annotations[reconcileAtAnnotation] = new Date()
+                        .toISOString()
+                        .replace(/\.\d{3}Z$/, 'Z'); // exact format: 2026-04-19T20:45:00Z
                     // we write the cr back to the file system
                     const newContent = catalog_common.io.toYaml(cr);
                     await promises_default().writeFile(filePath, newContent, 'utf-8');
@@ -369061,7 +369069,7 @@ function shouldForceReconcileByAnnotation(cr) {
         operator_src_logger.warn(`Timestamp is invalid for the annotation ${cr.kind}/${cr.metadata?.name || 'unknown'}: ${annotationValue}`);
         return false;
     }
-    // Buscamos el timestamp más reciente de TODAS las conditions
+    // Find the most recent timestamp across all conditions
     let latestConditionTime = 0;
     for (const condition of cr.status?.conditions || []) {
         const timeStr = condition.lastUpdateTime || condition.lastTransitionTime;
@@ -369073,7 +369081,7 @@ function shouldForceReconcileByAnnotation(cr) {
         }
     }
     const shouldForce = annotationTime > latestConditionTime;
-    operator_src_logger.debug(shouldForce, `Evaluating force-reconcile for ${cr.metadata.name}: annotation time (${new Date(annotationTime).toISOString()}) vs latest condition time (${new Date(latestConditionTime).toISOString()})`);
+    operator_src_logger.debug(`Evaluating force-reconcile for ${cr.metadata.name}: annotation time (${new Date(annotationTime).toISOString()}) vs latest condition time (${new Date(latestConditionTime).toISOString()})`, { metadata: { shouldForce } });
     if (shouldForce) {
         operator_src_logger.info(`force-reconcile activated for ${cr.metadata.name} → annotation (${annotationValue}) is newer than latest condition time (${new Date(latestConditionTime).toISOString()})`);
     }
@@ -370757,6 +370765,19 @@ function getQueueMetrics() {
         },
     };
 }
+// we need to assign weight to the deletion operation
+// to avoid blockades
+// https://github.com/prefapp/gitops-k8s/issues/1864
+// ghrepo feat | grss -> ghrepo -> ghgroup -> membership
+const DELETION_WEIGHTS = {
+    FirestartrGithubRepositoryFeature: 5,
+    FirestartrGithubRepositorySecretsSection: 4,
+    FirestartrGithubRepository: 3,
+    FirestartrGithubGroup: 2,
+    FirestartrGithubMembership: 1,
+    FirestartrTerraformWorkspace: 1,
+    FirestartrGithubOrgWebhook: 1,
+};
 // Do the kinds need different weights for the operations?
 // We need to discuss this with the team in this issue: https://github.com/prefapp/gitops-k8s/issues/524
 // const KIND_WEIGHTS: any = {
@@ -370826,7 +370847,17 @@ function sortQueue(queue) {
             // If weightA is larger, return -1 (a comes before b)
             return weightB - weightA;
         }
-        // --- SECONDARY SORT: By upsertTime (Ascending) ---
+        // in case of deletions we need to
+        // establish a secondary sort by kind
+        if (wa.operation === 'MARKED_TO_DELETION' &&
+            wb.operation === 'MARKED_TO_DELETION') {
+            const deletionWeightA = DELETION_WEIGHTS[wa.item.kind] || 0;
+            const deletionWeightB = DELETION_WEIGHTS[wb.item.kind] || 0;
+            if (deletionWeightA !== deletionWeightB) {
+                return deletionWeightB - deletionWeightA;
+            }
+        }
+        // --- 3rd SORT: By upsertTime (Ascending) ---
         // If weights are equal, sort by the oldest upsertTime (ascending)
         return wa.upsertTime - wb.upsertTime;
     });
@@ -379455,7 +379486,7 @@ const crs_analyzerSubcommand = {
 };
 
 ;// CONCATENATED MODULE: ./package.json
-const package_namespaceObject = JSON.parse('{"i8":"2.1.0-snapshot"}');
+const package_namespaceObject = JSON.parse('{"i8":"2.1.0-snapshot-1"}');
 ;// CONCATENATED MODULE: ../../package.json
 const package_namespaceObject_1 = {"i8":"2.0.0"};
 ;// CONCATENATED MODULE: ./src/subcommands/index.ts

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@firestartr/cli",
-  "version": "2.1.0-snapshot",
+  "version": "2.1.0-snapshot-1",
   "private": false,
   "description": "Commandline tool",
   "main": "build/main.js",