@firestartr/cli 1.54.0-snapshot-0 → 1.54.0-snapshot-1

package/build/index.js

@@ -356807,6 +356807,10 @@ class SyncerInitializer extends InitializerPatches {
             const provider = helperCTX(ctx).provider;
             return claim.providers[provider].sync || {};
         }
+        function policyInfo(ctx) {
+            const provider = helperCTX(ctx).provider;
+            return claim.providers[provider].policy;
+        }
         return [
             {
                 validate(cr) {
@@ -356825,33 +356829,38 @@ class SyncerInitializer extends InitializerPatches {
                         if (!PERIOD_VALIDATOR.test(cr.metadata.annotations['firestartr.dev/sync-period'])) {
                             throw `${this.identify()}: period incorrect '${cr.metadata.annotations['firestartr.dev/sync-period']}' for ${cr.kind}/${cr.metadata.name}`;
                         }
-                        return true;
                     }
                     else if (helperHasSyncSchedule(cr)) {
                         if (!catalog_common.cron.isValidCron(cr.metadata.annotations[SYNC_SCHED_ANNOTATION])) {
                             throw `${this.identify()}: sync-schedule: cron incorrect '${cr.metadata.annotations[SYNC_SCHED_ANNOTATION]}' for ${cr.kind}/${cr.metadata.name}`;
                         }
                     }
-                    else {
-                        return true;
-                    }
+                    return true;
                 },
                 apply(cr) {
+                    cr.metadata.annotations = cr.metadata.annotations || {};
+                    // Apply general policy annotation
+                    const policy = policyInfo(this);
+                    // Default to 'apply' for GitHub resources when no policy is specified
+                    const provider = helperCTX(this).provider;
+                    const defaultPolicy = provider === 'github' ? 'apply' : undefined;
+                    if (policy) {
+                        cr.metadata.annotations['firestartr.dev/policy'] = policy;
+                    }
+                    else if (defaultPolicy) {
+                        cr.metadata.annotations['firestartr.dev/policy'] = defaultPolicy;
+                    }
                     if (syncInfo(this).enabled) {
-                        cr.metadata.annotations = cr.metadata.annotations || {};
                         cr.metadata.annotations['firestartr.dev/sync-enabled'] = 'true';
                         if (syncInfo(this).period) {
-                            cr.metadata.annotations = cr.metadata.annotations || {};
                             cr.metadata.annotations['firestartr.dev/sync-period'] =
                                 syncInfo(this).period;
                         }
                         if (syncInfo(this).policy) {
-                            cr.metadata.annotations = cr.metadata.annotations || {};
                             cr.metadata.annotations['firestartr.dev/sync-policy'] =
                                 syncInfo(this).policy;
                         }
                         if (syncInfo(this).schedule) {
-                            cr.metadata.annotations = cr.metadata.annotations || {};
                             cr.metadata.annotations[SYNC_SCHED_ANNOTATION] =
                                 syncInfo(this).schedule;
                             cr.metadata.annotations[SYNC_SCHED_TIMEZONE_ANNOTATION] =
@@ -357572,6 +357581,76 @@ const external_node_child_process_namespaceObject = __WEBPACK_EXTERNAL_createReq
             },
             additionalProperties: false,
         },
+        PolicyType: {
+            $id: 'firestartr.dev://common/PolicyType',
+            type: 'string',
+            description: 'Policy for resource management',
+            enum: [
+                'apply',
+                'create-only',
+                'create-update-only',
+                'full-control',
+                'observe',
+                'observe-only',
+            ],
+        },
+    },
+});
+
+;// CONCATENATED MODULE: ../cdk8s_renderer/src/claims/base/schemas/sync-config.schema.ts
+/* harmony default export */ const sync_config_schema = ({
+    $id: 'SyncConfig',
+    definitions: {
+        SyncConfig: {
+            $id: 'firestartr.dev://common/SyncConfig',
+            type: 'object',
+            description: 'Sync configuration for resources',
+            properties: {
+                enabled: {
+                    type: 'boolean',
+                    description: 'Enable periodic sync operations',
+                },
+                period: {
+                    type: 'string',
+                    pattern: '^[0-9]+[smhd]$',
+                    description: 'Sync period (e.g., 1h, 30m, 5s). Must be enabled without schedule.',
+                },
+                schedule: {
+                    type: 'string',
+                    description: 'Cron schedule for sync operations. Must be enabled without period.',
+                },
+                schedule_timezone: {
+                    type: 'string',
+                    description: 'Timezone for cron schedule (e.g., UTC, America/New_York)',
+                },
+                policy: {
+                    type: 'string',
+                    description: 'Policy for sync operations (apply or observe)',
+                },
+            },
+            additionalProperties: false,
+            required: ['enabled'],
+            oneOf: [
+                {
+                    required: ['period'],
+                },
+                {
+                    required: ['schedule'],
+                },
+                {
+                    not: {
+                        anyOf: [
+                            {
+                                required: ['period'],
+                            },
+                            {
+                                required: ['schedule'],
+                            },
+                        ],
+                    },
+                },
+            ],
+        },
     },
 });
 
@@ -357932,6 +358011,9 @@ const external_node_child_process_namespaceObject = __WEBPACK_EXTERNAL_createReq
                 {
                     type: 'object',
                     properties: {
+                        policy: {
+                            $ref: 'firestartr.dev://common/PolicyType',
+                        },
                         privacy: {
                             type: 'string',
                             enum: ['closed', 'secret'],
@@ -357942,6 +358024,9 @@ const external_node_child_process_namespaceObject = __WEBPACK_EXTERNAL_createReq
                         org: {
                             type: 'string',
                         },
+                        sync: {
+                            $ref: 'firestartr.dev://common/SyncConfig',
+                        },
                     },
                     required: ['org', 'privacy'],
                 },
@@ -357967,6 +358052,9 @@ const external_node_child_process_namespaceObject = __WEBPACK_EXTERNAL_createReq
                 {
                     type: 'object',
                     properties: {
+                        policy: {
+                            $ref: 'firestartr.dev://common/PolicyType',
+                        },
                         role: {
                             type: 'string',
                             enum: ['admin', 'member'],
@@ -357974,6 +358062,9 @@ const external_node_child_process_namespaceObject = __WEBPACK_EXTERNAL_createReq
                         org: {
                             type: 'string',
                         },
+                        sync: {
+                            $ref: 'firestartr.dev://common/SyncConfig',
+                        },
                     },
                     required: ['org', 'role'],
                 },
@@ -357998,6 +358089,9 @@ const external_node_child_process_namespaceObject = __WEBPACK_EXTERNAL_createReq
                 {
                     type: 'object',
                     properties: {
+                        policy: {
+                            $ref: 'firestartr.dev://common/PolicyType',
+                        },
                         org: {
                             type: 'string',
                             description: 'The github organization name',
@@ -358006,6 +358100,9 @@ const external_node_child_process_namespaceObject = __WEBPACK_EXTERNAL_createReq
                             type: 'string',
                             enum: ['private', 'public', 'internal'],
                         },
+                        sync: {
+                            $ref: 'firestartr.dev://common/SyncConfig',
+                        },
                         features: {
                             type: 'array',
                             items: {
@@ -358041,6 +358138,9 @@ const external_node_child_process_namespaceObject = __WEBPACK_EXTERNAL_createReq
                 {
                     type: 'object',
                     properties: {
+                        policy: {
+                            $ref: 'firestartr.dev://common/PolicyType',
+                        },
                         orgName: {
                             type: 'string',
                             description: 'Organization name on GitHub',
@@ -358074,6 +358174,9 @@ const external_node_child_process_namespaceObject = __WEBPACK_EXTERNAL_createReq
                             },
                             required: ['url', 'contentType', 'events', 'secretRef'],
                         },
+                        sync: {
+                            $ref: 'firestartr.dev://common/SyncConfig',
+                        },
                     },
                     required: ['orgName', 'webhook'],
                 },
@@ -358245,15 +358348,7 @@ const GithubSchemas = [
                     type: 'object',
                     properties: {
                         policy: {
-                            type: 'string',
-                            enum: [
-                                'apply',
-                                'create-only',
-                                'create-update-only',
-                                'full-control',
-                                'observe',
-                                'observe-only',
-                            ],
+                            $ref: 'firestartr.dev://common/PolicyType',
                         },
                         name: {
                             type: 'string',
@@ -358263,47 +358358,7 @@ const GithubSchemas = [
                             enum: ['remote', 'inline', 'Remote', 'Inline'],
                         },
                         sync: {
-                            type: 'object',
-                            properties: {
-                                enabled: {
-                                    type: 'boolean',
-                                },
-                                period: {
-                                    type: 'string',
-                                    pattern: '^[0-9]+[smhd]$',
-                                },
-                                schedule: {
-                                    type: 'string',
-                                },
-                                schedule_timezone: {
-                                    type: 'string',
-                                },
-                                policy: {
-                                    type: 'string',
-                                },
-                            },
-                            additionalProperties: false,
-                            required: ['enabled'],
-                            oneOf: [
-                                {
-                                    required: ['period'],
-                                },
-                                {
-                                    required: ['schedule'],
-                                },
-                                {
-                                    not: {
-                                        anyOf: [
-                                            {
-                                                required: ['period'],
-                                            },
-                                            {
-                                                required: ['schedule'],
-                                            },
-                                        ],
-                                    },
-                                },
-                            ],
+                            $ref: 'firestartr.dev://common/SyncConfig',
                         },
                         valuesSchema: {
                             type: 'string',
@@ -358604,10 +358659,12 @@ const SecretsSchemas = [external_secrets_schema];
 
 
 
+
 const schemas = {
     root: root_schema,
     schemas: [
         common_meta_schema,
+        sync_config_schema,
         group_schema,
         user_schema,
         component_schema,
@@ -359560,7 +359617,6 @@ var ajv_default = /*#__PURE__*/__nccwpck_require__.n(dist_ajv);
 
 
 
-
 function validate_validate(featurePath) {
     __validateDirStructure(featurePath);
     const configData = __validateFeatureConfig(featurePath);
@@ -359575,17 +359631,6 @@ function __validateDirStructure(featurePath) {
     if (!isFile(external_path_.join(featurePath, 'config.yaml'))) {
         throw `Feature: ${featurePath}/config.yaml not found or not a file`;
     }
-    if (!isDirectory(external_path_.join(featurePath, 'templates/docs'))) {
-        throw `Feature: ${featurePath}templates/docs not a directory`;
-    }
-    if (!__validateDocsHasReadmes(featurePath)) {
-        throw `Feature: ${featurePath}: should have at least one .md file in the templates/docs`;
-    }
-}
-function __validateDocsHasReadmes(featurePath) {
-    const dirEntries = external_fs_default().readdirSync(external_path_.join(featurePath, 'templates/docs'));
-    return (dirEntries.some((entryFile) => !isDirectory(external_path_.join(featurePath, '/templates/docs', entryFile)) &&
-        entryFile.endsWith('.md')) !== undefined);
 }
 function __validateFeatureConfig(featurePath) {
     try {
@@ -360641,7 +360686,6 @@ function buildExpectedOutput(config, renderDir) {
         localPath: external_node_path_.join(renderDir, f.dest),
         repoPath: f.dest,
         userManaged: f.user_managed,
-        targetBranch: f.target_branch ?? '',
     }));
     return {
         files,
@@ -364105,8 +364149,12 @@ class FeatureRepoChart extends BaseGithubChart {
         const annotations = this.getAnnotationsFromRepo(this.get('repoCr'), [
             'claim-ref',
             'revision',
+            'policy',
+            'sync-policy',
             'sync-enabled',
             'sync-period',
+            'sync-schedule',
+            'sync-schedule-timezone',
         ]);
         cr.metadata.annotations = {
             ...cr.metadata.annotations,
@@ -364181,8 +364229,12 @@ class RepoSecretsSectionChart extends BaseGithubChart {
         const annotations = this.getAnnotationsFromRepo(this.get('repoCr'), [
             'claim-ref',
             'revision',
+            'policy',
+            'sync-policy',
             'sync-enabled',
             'sync-period',
+            'sync-schedule',
+            'sync-schedule-timezone',
         ]);
         cr.metadata.annotations = {
             ...cr.metadata.annotations,
@@ -370480,10 +370532,15 @@ class Resource {
         this.set('operation', operation);
         this.set('deps', deps);
     }
-    async run() {
+    async run(options) {
         await this.preprocess();
         await this.synth();
-        await this.runTerraform();
+        if (options?.planOnly) {
+            await this.runTerraformPlanOnly();
+        }
+        else {
+            await this.runTerraform();
+        }
         await this.postprocess();
         if (this.logStream) {
             this.logStream.end();
@@ -370508,6 +370565,13 @@ class Resource {
     log(msg) {
         this.logFn(msg);
     }
+    async runTerraformPlanOnly() {
+        await this.onTFStreaming();
+        let output = '';
+        output += await terraformInit(this.get('main_artifact'), this.logStream);
+        output += await terraformPlan(this.get('main_artifact'), this.logStream);
+        this.set('output', output);
+    }
     async runTerraform() {
         await this.onTFStreaming();
         let output = '';
@@ -370895,7 +370959,7 @@ async function runProvisioner(data, opts) {
     if ('logStreamCallbacksTF' in opts) {
         resource.setTFStreamLogs(opts['logStreamCallbacksTF']);
     }
-    await resource.run();
+    await resource.run({ planOnly: opts.planOnly });
     return resource;
 }
 function createInstanceOf(entity, op, deps) {
@@ -371177,6 +371241,52 @@ function helperCreateCheckRunName(cmd, item) {
     return `${item.kind} - ${cmd}`;
 }
 
+;// CONCATENATED MODULE: ../operator/src/utils/index.ts
+const secretRegex = /\$\{\{ secrets\.(.*?) \}\}/g;
+function replaceConfigSecrets(config, secrets) {
+    for (const key in config) {
+        if (typeof config[key] === 'object' && config[key] !== null) {
+            // If the property is an object, call this function recursively
+            replaceConfigSecrets(config[key], secrets);
+        }
+        else if (typeof config[key] === 'string') {
+            // If the property is a string and its value is equal to secrets.something,
+            // replace the value with the value of the 'something' key in the secrets object
+            config[key] = config[key].replace(secretRegex, (_, group1) => {
+                if (!secrets[group1]) {
+                    throw new Error(`Secret ${group1} not found in secrets`);
+                }
+                return secrets[group1];
+            });
+        }
+    }
+    return config;
+}
+function replaceInlineSecrets(inline, secrets) {
+    if (typeof inline !== 'string' || !inline)
+        return inline;
+    let result = inline;
+    result = result.replace(secretRegex, (_, group1) => {
+        if (!secrets[group1]) {
+            throw new Error(`Secret ${group1} not found in secrets`);
+        }
+        return secrets[group1];
+    });
+    return result;
+}
+/**
+ * Retrieves a policy annotation value from a custom resource
+ * @param item - The CR to get the policy from
+ * @param annotation - The annotation key to retrieve
+ * @returns The policy value, or undefined if not set
+ */
+function getPolicy(item, annotation) {
+    const policy = item.metadata.annotations && item.metadata.annotations[annotation];
+    if (policy)
+        return policy;
+    return undefined;
+}
+
 ;// CONCATENATED MODULE: ../operator/src/utils/operationErrorMessages.ts
 const APPLY_DEFAULT_ERROR_MESSAGE = 'An error occurred while executing the Terraform apply operation.';
 const DESTROY_DEFAULT_ERROR_MESSAGE = 'An error occurred while executing the Terraform destroy operation.';
@@ -371192,9 +371302,17 @@ const PLAN_DEFAULT_ERROR_MESSAGE = 'An error occurred while executing the Terraf
 
 
 
+const cdktf_LAST_STATE_PR_ANNOTATION = 'firestartr.dev/last-state-pr';
+
 function processOperation(item, op, handler) {
     operator_src_logger.info(`Processing operation ${op} on ${item.kind}/${item.metadata?.name}`);
     try {
+        const policy = getPolicy(item, 'firestartr.dev/policy');
+        // If general policy is observe/observe-only, route to observe mode instead of apply
+        if (!policy || policy === 'observe' || policy === 'observe-only') {
+            operator_src_logger.info(`Policy is '${policy || 'not set (default)'}', routing to observe mode`);
+            return cdktf_observe(item, op, handler);
+        }
         switch (op) {
             case OperationType.UPDATED:
                 return updated(item, op, handler);
@@ -371219,6 +371337,11 @@ function processOperation(item, op, handler) {
         throw e;
     }
 }
+async function* cdktf_observe(item, op, handler) {
+    for await (const transition of doPlan(item, op, handler)) {
+        yield transition;
+    }
+}
 async function* created(item, op, handler) {
     for await (const transition of doApply(item, op, handler)) {
         yield transition;
@@ -371254,8 +371377,18 @@ async function* sync(item, op, handler) {
         status: 'False',
         message: 'Synth CDKTF',
     };
-    for await (const transition of doApply(item, op, handler)) {
-        yield transition;
+    const syncPolicy = getPolicy(item, 'firestartr.dev/sync-policy');
+    if (syncPolicy === 'apply') {
+        operator_src_logger.info(`SYNC OPERATION: applying item ${item.metadata.name} with sync-policy=${syncPolicy}`);
+        for await (const transition of doApply(item, op, handler)) {
+            yield transition;
+        }
+    }
+    else {
+        operator_src_logger.info(`SYNC OPERATION: planning item ${item.metadata.name} with sync-policy=${syncPolicy || 'default (observe)'}`);
+        for await (const transition of doPlan(item, op, handler)) {
+            yield transition;
+        }
     }
     yield {
         item,
@@ -371295,15 +371428,14 @@ async function* markedToDeletion(item, op, handler) {
             message: 'Destroying process started',
         };
         const deps = await handler.resolveReferences();
-        const annotation = 'firestartr.dev/last-state-pr';
-        const statePr = item?.metadata?.annotations?.[annotation];
+        const statePr = item?.metadata?.annotations?.[cdktf_LAST_STATE_PR_ANNOTATION];
         const hasStatePr = typeof statePr === 'string' && statePr.trim().length > 0;
         if (!hasStatePr) {
             operator_src_logger.warn(`CR ${item?.kind ?? 'UnknownKind'}/${item?.metadata?.name ?? 'unknown'} ` +
-                `has no "${annotation}" annotation; skipping GitHub Check Runs (synth, terraform apply).`);
+                `has no "${cdktf_LAST_STATE_PR_ANNOTATION}" annotation; skipping GitHub Check Runs (synth, terraform apply).`);
         }
         else {
-            operator_src_logger.debug(`CR ${item.kind}/${item.metadata.name} uses "${annotation}" = ${statePr}`);
+            operator_src_logger.debug(`CR ${item.kind}/${item.metadata.name} uses "${cdktf_LAST_STATE_PR_ANNOTATION}" = ${statePr}`);
         }
         const destroyOutput = await provisioner.runProvisioner({
             mainCr: item,
@@ -371338,7 +371470,7 @@ async function* markedToDeletion(item, op, handler) {
         };
         await handler.finalize(handler.pluralKind, item.metadata.namespace, item, 'firestartr.dev/finalizer');
         await handler.writeTerraformOutputInTfResult(item, output);
-        if (item.metadata.annotations['firestartr.dev/last-state-pr'] || false) {
+        if (item.metadata.annotations[cdktf_LAST_STATE_PR_ANNOTATION] || false) {
             await addDestroyCommitStatus(item, 'success', 'Destroy operation completed', `Terraform Destroy ${item.metadata.name}`);
         }
         void handler.success();
@@ -371425,15 +371557,14 @@ async function* doApply(item, op, handler) {
         }
         const deps = await handler.resolveReferences();
         operator_src_logger.info(`Item ${item.metadata.name} has the following dependencies: ${deps}`);
-        const annotation = 'firestartr.dev/last-state-pr';
-        const statePr = item?.metadata?.annotations?.[annotation];
+        const statePr = item?.metadata?.annotations?.[cdktf_LAST_STATE_PR_ANNOTATION];
         const hasStatePr = typeof statePr === 'string' && statePr.trim().length > 0;
         if (!hasStatePr) {
             operator_src_logger.warn(`CR ${item?.kind ?? 'UnknownKind'}/${item?.metadata?.name ?? 'unknown'} ` +
-                `has no "${annotation}" annotation; skipping GitHub Check Runs (synth, terraform apply).`);
+                `has no "${cdktf_LAST_STATE_PR_ANNOTATION}" annotation; skipping GitHub Check Runs (synth, terraform apply).`);
         }
         else {
-            operator_src_logger.debug(`CR ${item.kind}/${item.metadata.name} uses "${annotation}" = ${statePr}`);
+            operator_src_logger.debug(`CR ${item.kind}/${item.metadata.name} uses "${cdktf_LAST_STATE_PR_ANNOTATION}" = ${statePr}`);
         }
         const applyOutput = await provisioner.runProvisioner({
             mainCr: item,
@@ -371534,6 +371665,220 @@ async function* doApply(item, op, handler) {
         }
     }
 }
+async function* doPlan(item, op, handler) {
+    let checkRunCtl;
+    try {
+        cleanTerraformState();
+        yield {
+            item,
+            reason: op,
+            type: 'PLANNING',
+            status: 'True',
+            message: 'Planning process started',
+        };
+        const deps = await handler.resolveReferences();
+        const statePr = item?.metadata?.annotations?.[cdktf_LAST_STATE_PR_ANNOTATION];
+        const hasStatePr = typeof statePr === 'string' && statePr.trim().length > 0;
+        if (!hasStatePr) {
+            operator_src_logger.warn(`CR ${item?.kind ?? 'UnknownKind'}/${item?.metadata?.name ?? 'unknown'} ` +
+                `has no "${cdktf_LAST_STATE_PR_ANNOTATION}" annotation; skipping GitHub Check Runs for plan.`);
+        }
+        else {
+            operator_src_logger.debug(`CR ${item.kind}/${item.metadata.name} uses "${cdktf_LAST_STATE_PR_ANNOTATION}" = ${statePr}`);
+            await addPlanStatusCheck(statePr, 'CDKTF plan in progress...');
+        }
+        // Run provisioner in plan-only mode
+        const planResult = await provisioner.runProvisioner({ mainCr: item, deps }, {
+            planOnly: true,
+            delete: 'deletionTimestamp' in item.metadata,
+            ...(hasStatePr
+                ? {
+                    logStreamCallbacksCDKTF: {
+                        prepare: async () => {
+                            checkRunCtl = await GHCheckRun('synth', item);
+                            return checkRunCtl;
+                        },
+                    },
+                    logStreamCallbacksTF: {
+                        prepare: async () => {
+                            checkRunCtl = await GHCheckRun('plan', item);
+                            return checkRunCtl;
+                        },
+                    },
+                }
+                : {}),
+        });
+        const planOutput = planResult?.output || '';
+        // Parse terraform plan output to detect changes
+        // Handles multiple Terraform output formats and versions
+        const hasChanges = detectPlanChanges(planOutput);
+        if (hasChanges) {
+            yield {
+                item,
+                reason: op,
+                type: 'OUT_OF_SYNC',
+                status: 'True',
+                message: 'Plan has changes',
+            };
+            yield {
+                item,
+                reason: op,
+                type: 'PROVISIONED',
+                status: 'False',
+                message: 'Plan has changes',
+            };
+        }
+        else {
+            yield {
+                item,
+                reason: op,
+                type: 'OUT_OF_SYNC',
+                status: 'False',
+                message: 'Plan has no changes',
+            };
+            yield {
+                item,
+                reason: op,
+                type: 'PROVISIONED',
+                status: 'True',
+                message: 'Plan has no changes',
+            };
+        }
+        // Store plan details for later reference
+        yield {
+            item,
+            reason: op,
+            type: 'LAST_PLAN_DETAILS',
+            status: 'Unknown',
+            message: planOutput,
+        };
+        yield {
+            item,
+            reason: op,
+            type: 'PLANNING',
+            status: 'False',
+            message: 'Planning process finished',
+        };
+        if (hasStatePr) {
+            await addPlanStatusCheck(statePr, hasChanges ? 'Plan has changes' : 'Plan has no changes', 'completed');
+        }
+    }
+    catch (e) {
+        operator_src_logger.error(`CDKTF plan failed: ${e}`);
+        if (checkRunCtl) {
+            checkRunCtl.fnOnError(e);
+        }
+        yield {
+            item,
+            reason: op,
+            type: 'ERROR',
+            status: 'True',
+            message: PLAN_DEFAULT_ERROR_MESSAGE,
+        };
+        yield {
+            item,
+            reason: op,
+            type: 'PLANNING',
+            status: 'False',
+            message: PLAN_DEFAULT_ERROR_MESSAGE,
+        };
+        yield {
+            item,
+            reason: op,
+            type: 'PROVISIONED',
+            status: 'False',
+            message: PLAN_DEFAULT_ERROR_MESSAGE,
+        };
+        const statePr = item?.metadata?.annotations?.[cdktf_LAST_STATE_PR_ANNOTATION];
+        if (statePr) {
+            const summaryText = tryCreateErrorSummary('CDKTF Plan failed', e);
+            await addPlanStatusCheck(statePr, summaryText, 'completed', true);
+        }
+        await handler.writeTerraformOutputInTfResult(item, e);
+        void handler.error();
+    }
+}
+/**
+ * Detects if a Terraform plan output contains changes
+ * Handles multiple Terraform versions and output formats
+ * @param planOutput - The text output from terraform plan
+ * @returns true if changes are detected, false otherwise
+ */
+function detectPlanChanges(planOutput) {
+    if (!planOutput || planOutput.trim().length === 0) {
+        return false;
+    }
+    // Pattern 1: "Plan: X to add, Y to change, Z to destroy"
+    // Matches: "Plan: 1 to add, 0 to change, 0 to destroy"
+    const planPattern = /plan:\s*(\d+)\s+to\s+add,\s*(\d+)\s+to\s+change,\s*(\d+)\s+to\s+destroy/i;
+    const planMatch = planOutput.match(planPattern);
+    if (planMatch) {
+        const add = Number(planMatch[1]);
+        const change = Number(planMatch[2]);
+        const destroy = Number(planMatch[3]);
+        if (add > 0 || change > 0 || destroy > 0) {
+            return true;
+        }
+        // Explicitly found "Plan:" with 0/0/0 - no changes
+        return false;
+    }
+    // Pattern 2: Individual change indicators
+    // Handles variations like "1 to add", "2 to change", "3 to destroy"
+    const hasAdditions = /\d+\s+to\s+add/i.test(planOutput);
+    const hasChanges = /\d+\s+to\s+change/i.test(planOutput);
+    const hasDestructions = /\d+\s+to\s+destroy/i.test(planOutput);
+    const hasImports = /\d+\s+to\s+import/i.test(planOutput);
+    if (hasAdditions || hasChanges || hasDestructions || hasImports) {
+        return true;
+    }
+    // Pattern 3: Resource-level change indicators
+    // Matches: "# resource will be created", "# resource will be updated", etc.
+    const resourceChangePatterns = [
+        /will\s+be\s+(created|destroyed|updated|replaced)/i,
+        /must\s+be\s+(created|destroyed|updated|replaced)/i,
+        /#.*\s+(create|destroy|update|replace)/i,
+    ];
+    for (const pattern of resourceChangePatterns) {
+        if (pattern.test(planOutput)) {
+            return true;
+        }
+    }
+    // Pattern 4: Action symbols in plan output
+    // Terraform uses symbols like +, -, ~, -/+ to indicate changes
+    const actionSymbols = [
+        /^\s*[+]\s+/m,
+        /^\s*[-]\s+/m,
+        /^\s*[~]\s+/m,
+        /^\s*[-][/][+]\s+/m, // Replace
+    ];
+    for (const pattern of actionSymbols) {
+        if (pattern.test(planOutput)) {
+            return true;
+        }
+    }
+    // Pattern 5: No changes messages (inverse check)
+    const noChangesPatterns = [
+        /no\s+changes/i,
+        /infrastructure\s+is\s+up[-\s]to[-\s]date/i,
+        /your\s+infrastructure\s+matches\s+the\s+configuration/i,
+        /0\s+to\s+add,\s*0\s+to\s+change,\s*0\s+to\s+destroy/i,
+    ];
+    for (const pattern of noChangesPatterns) {
+        if (pattern.test(planOutput)) {
+            return false;
+        }
+    }
+    // If we find "Plan:" keyword but couldn't parse it, log a warning and err on the side of caution
+    if (planOutput.toLowerCase().includes('plan:')) {
+        operator_src_logger.warn('Found "Plan:" in output but could not parse change counts. ' +
+            'Raw output excerpt (first 500 chars):\n' +
+            planOutput.slice(0, 500) +
+            '\nErring on the side of caution and assuming changes.');
+        return true;
+    }
+    // Default: assume no changes if we can't detect any
+    return false;
+}
 function cleanTerraformState() {
     external_fs_.rmSync('/library/packages/provisioner/cdktf.out', {
         recursive: true,
@@ -372533,40 +372878,6 @@ function tf_checkrun_helperCreateCheckRunName(cmd) {
     return `TFWorkspace - ${cmd}`;
 }
 
-;// CONCATENATED MODULE: ../operator/src/utils/index.ts
-const secretRegex = /\$\{\{ secrets\.(.*?) \}\}/g;
-function replaceConfigSecrets(config, secrets) {
-    for (const key in config) {
-        if (typeof config[key] === 'object' && config[key] !== null) {
-            // If the property is an object, call this function recursively
-            replaceConfigSecrets(config[key], secrets);
-        }
-        else if (typeof config[key] === 'string') {
-            // If the property is a string and its value is equal to secrets.something,
-            // replace the value with the value of the 'something' key in the secrets object
-            config[key] = config[key].replace(secretRegex, (_, group1) => {
-                if (!secrets[group1]) {
-                    throw new Error(`Secret ${group1} not found in secrets`);
-                }
-                return secrets[group1];
-            });
-        }
-    }
-    return config;
-}
-function replaceInlineSecrets(inline, secrets) {
-    if (typeof inline !== 'string' || !inline)
-        return inline;
-    let result = inline;
-    result = result.replace(secretRegex, (_, group1) => {
-        if (!secrets[group1]) {
-            throw new Error(`Secret ${group1} not found in secrets`);
-        }
-        return secrets[group1];
-    });
-    return result;
-}
-
 ;// CONCATENATED MODULE: ../operator/src/tfworkspaces/process-operation.ts
 
 
@@ -372764,11 +373075,6 @@ async function* process_operation_renamed(item, op, handler) {
         yield transition;
     }
 }
-function getPolicy(item, annotation) {
-    const policy = item.metadata.annotations && item.metadata.annotations[annotation];
-    if (policy)
-        return policy;
-}
 async function* process_operation_updated(item, op, handler) {
     for await (const transition of process_operation_doApply(item, op, handler)) {
         yield transition;
@@ -373425,7 +373731,7 @@ const processOperationPlan_TF_PROJECTS_PATH = '/tmp/tfworkspaces';
 function processOperationPlan(item, op, handler) {
     try {
         processOperationPlan_clearLocalTfProjects();
-        const policy = processOperationPlan_getPolicy(item);
+        const policy = getPolicy(item, 'firestartr.dev/policy');
         if (policy === 'observe' || policy === 'apply') {
             return processOperationPlan_plan(item, op, handler);
         }
@@ -373884,12 +374190,6 @@ function processOperationPlan_getErrorOutputMessage(cr, key, ref) {
         throw new Error(`❌ Source ${cr.spec.source} not supported`);
     }
 }
-function processOperationPlan_getPolicy(item) {
-    const policy = item.metadata.annotations &&
-        item.metadata.annotations['firestartr.dev/policy'];
-    if (policy)
-        return policy;
-}
 
 ;// CONCATENATED MODULE: ../operator/src/ctx.ts
 class Ctx {