@geek-fun/serverlessinsight 0.6.12 → 0.6.14

package/dist/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@geek-fun/serverlessinsight",
-    "version": "0.6.12",
+    "version": "0.6.14",
     "description": "Full life cycle cross providers serverless application management for your fast-growing business.",
     "homepage": "https://serverlessinsight.geekfun.club",
     "main": "dist/src/index.js",

package/dist/src/common/hashUtils.js

@@ -3,20 +3,41 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.diffAttributes = exports.attributesEqual = exports.computeFileHash = void 0;
+exports.diffAttributes = exports.attributesEqual = exports.computeDirectoryHash = exports.computeFileHash = void 0;
 const node_crypto_1 = __importDefault(require("node:crypto"));
 const node_fs_1 = __importDefault(require("node:fs"));
-/**
- * Compute SHA-256 hash of a file.
- * Used for tracking external artifacts like function code zip files.
- * @param filePath - Path to the file to hash
- * @returns Hex-encoded SHA-256 hash of the file contents
- */
+const node_path_1 = __importDefault(require("node:path"));
 const computeFileHash = (filePath) => {
     const fileBuffer = node_fs_1.default.readFileSync(filePath);
     return node_crypto_1.default.createHash('sha256').update(fileBuffer).digest('hex');
 };
 exports.computeFileHash = computeFileHash;
+const computeDirectoryHash = (dirPath) => {
+    const files = [];
+    const collectFiles = (currentPath) => {
+        const entries = node_fs_1.default.readdirSync(currentPath, { withFileTypes: true });
+        for (const entry of entries) {
+            const fullPath = node_path_1.default.join(currentPath, entry.name);
+            if (entry.isDirectory()) {
+                collectFiles(fullPath);
+            }
+            else if (entry.isFile()) {
+                files.push(fullPath);
+            }
+        }
+    };
+    collectFiles(dirPath);
+    files.sort();
+    const hash = node_crypto_1.default.createHash('sha256');
+    for (const file of files) {
+        const relativePath = node_path_1.default.relative(dirPath, file).split(node_path_1.default.sep).join('/');
+        hash.update(relativePath);
+        const content = node_fs_1.default.readFileSync(file);
+        hash.update(content);
+    }
+    return hash.digest('hex');
+};
+exports.computeDirectoryHash = computeDirectoryHash;
 /**
  * Deep equality comparison for two values.
  * Handles primitives, objects, arrays, null, and undefined.

package/dist/src/stack/aliyunStack/fc3Planner.js

@@ -5,6 +5,32 @@ const common_1 = require("../../common");
 const aliyunClient_1 = require("../../common/aliyunClient");
 const fc3Types_1 = require("./fc3Types");
 const lang_1 = require("../../lang");
+/**
+ * Provider-managed logConfig fields that are set by the system after creation.
+ * These should not be compared when detecting changes.
+ */
+const PROVIDER_MANAGED_LOG_CONFIG_FIELDS = ['project', 'logstore', 'logBeginRule'];
+/**
+ * Normalize definition for comparison by excluding provider-managed fields.
+ * This prevents false-positive change detection when the system populates
+ * fields like logConfig.project and logConfig.logstore after creation.
+ */
+const normalizeDefinitionForComparison = (definition) => {
+    const { logConfig, ...rest } = definition;
+    if (!logConfig || typeof logConfig !== 'object') {
+        return definition;
+    }
+    const normalizedLogConfig = {};
+    for (const [key, value] of Object.entries(logConfig)) {
+        if (!PROVIDER_MANAGED_LOG_CONFIG_FIELDS.includes(key)) {
+            normalizedLogConfig[key] = value;
+        }
+    }
+    return {
+        ...rest,
+        logConfig: Object.keys(normalizedLogConfig).length > 0 ? normalizedLogConfig : null,
+    };
+};
 const isSecurityGroupId = (value) => value.startsWith('sg-');
 const resolveSecurityGroupId = async (context, securityGroupName, vpcId) => {
     if (isSecurityGroupId(securityGroupName)) {
@@ -34,7 +60,7 @@ const planFunctionDeletion = (logicalId, definition) => ({
     logicalId,
     action: 'delete',
     resourceType: 'ALIYUN_FC3',
-    changes: { before: definition },
+    changes: { before: normalizeDefinitionForComparison(definition) },
 });
 const generateFunctionPlan = async (context, state, functions) => {
     if (!functions || functions.length === 0) {
@@ -69,18 +95,23 @@ const generateFunctionPlan = async (context, state, functions) => {
                     logicalId,
                     action: 'create',
                     resourceType: 'ALIYUN_FC3',
-                    changes: { before: currentState.definition, after: desiredDefinition },
+                    changes: {
+                        before: normalizeDefinitionForComparison(currentState.definition),
+                        after: normalizeDefinitionForComparison(desiredDefinition),
+                    },
                     drifted: true,
                 };
             }
             const currentDefinition = currentState.definition || {};
-            const definitionChanged = !(0, common_1.attributesEqual)(currentDefinition, desiredDefinition);
+            const normalizedCurrent = normalizeDefinitionForComparison(currentDefinition);
+            const normalizedDesired = normalizeDefinitionForComparison(desiredDefinition);
+            const definitionChanged = !(0, common_1.attributesEqual)(normalizedCurrent, normalizedDesired);
             if (definitionChanged) {
                 return {
                     logicalId,
                     action: 'update',
                     resourceType: 'ALIYUN_FC3',
-                    changes: { before: currentDefinition, after: desiredDefinition },
+                    changes: { before: normalizedCurrent, after: normalizedDesired },
                     drifted: true,
                 };
             }
@@ -91,7 +122,10 @@ const generateFunctionPlan = async (context, state, functions) => {
                 logicalId,
                 action: 'create',
                 resourceType: 'ALIYUN_FC3',
-                changes: { before: currentState.definition, after: desiredDefinition },
+                changes: {
+                    before: normalizeDefinitionForComparison(currentState.definition),
+                    after: normalizeDefinitionForComparison(desiredDefinition),
+                },
             };
         }
     }));

package/dist/src/stack/aliyunStack/fc3Resource.js

@@ -358,6 +358,7 @@ const createResource = async (context, fn, state) => {
                 project: dependentResources.logConfig.project,
                 logstore: dependentResources.logConfig.logstore,
                 enableRequestMetrics: true,
+                enableInstanceMetrics: true,
             },
         };
     }
@@ -547,6 +548,7 @@ const updateResource = async (context, fn, state) => {
                 project: logConfig.project,
                 logstore: logConfig.logstore,
                 enableRequestMetrics: true,
+                enableInstanceMetrics: true,
             },
         };
     }

package/dist/src/stack/aliyunStack/ossPlanner.js

@@ -1,6 +1,10 @@
 "use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.generateBucketPlan = void 0;
+const node_path_1 = __importDefault(require("node:path"));
 const aliyunClient_1 = require("../../common/aliyunClient");
 const ossTypes_1 = require("./ossTypes");
 const stateManager_1 = require("../../common/stateManager");
@@ -9,8 +13,12 @@ const planBucketDeletion = (logicalId, definition) => ({
     logicalId,
     action: 'delete',
     resourceType: 'ALIYUN_OSS_BUCKET',
-    changes: { before: definition },
+    changes: { before: normalizeDefinitionForDisplay(definition) },
 });
+const normalizeDefinitionForDisplay = (definition) => {
+    const { domainBound: _domainBound, ...rest } = definition;
+    return rest;
+};
 const generateBucketPlan = async (context, state, buckets) => {
     if (!buckets || buckets.length === 0) {
         const allStates = (0, stateManager_1.getAllResources)(state);
@@ -24,7 +32,17 @@ const generateBucketPlan = async (context, state, buckets) => {
         const logicalId = `buckets.${bucket.key}`;
         const currentState = (0, stateManager_1.getResource)(state, logicalId);
         const config = (0, ossTypes_1.bucketToOssBucketConfig)(bucket);
-        const desiredDefinition = (0, ossTypes_1.extractOssBucketDefinition)(config);
+        const websiteCodeHash = (() => {
+            if (!bucket.website?.code)
+                return undefined;
+            try {
+                return (0, hashUtils_1.computeDirectoryHash)(node_path_1.default.resolve(process.cwd(), bucket.website.code));
+            }
+            catch {
+                return null;
+            }
+        })();
+        const desiredDefinition = (0, ossTypes_1.extractOssBucketDefinition)(config, websiteCodeHash);
         if (!currentState) {
             return {
                 logicalId,
@@ -41,20 +59,25 @@ const generateBucketPlan = async (context, state, buckets) => {
                     logicalId,
                     action: 'create',
                     resourceType: 'ALIYUN_OSS_BUCKET',
-                    changes: { before: currentState.definition, after: desiredDefinition },
+                    changes: {
+                        before: normalizeDefinitionForDisplay(currentState.definition),
+                        after: desiredDefinition,
+                    },
                     drifted: true,
                 };
             }
             const currentDefinition = currentState.definition || {};
-            const { domainBound, ...comparableDefinition } = currentDefinition;
-            const definitionChanged = !(0, hashUtils_1.attributesEqual)(comparableDefinition, desiredDefinition);
+            const normalizedCurrent = normalizeDefinitionForDisplay(currentDefinition);
+            const normalizedDesired = normalizeDefinitionForDisplay(desiredDefinition);
+            const { domainBound } = currentDefinition;
+            const definitionChanged = !(0, hashUtils_1.attributesEqual)(normalizedCurrent, normalizedDesired);
             const domainBindingPending = domainBound === false;
             if (definitionChanged || domainBindingPending) {
                 return {
                     logicalId,
                     action: 'update',
                     resourceType: 'ALIYUN_OSS_BUCKET',
-                    changes: { before: currentDefinition, after: desiredDefinition },
+                    changes: { before: normalizedCurrent, after: normalizedDesired },
                     ...(definitionChanged ? { drifted: true } : {}),
                 };
             }
@@ -65,7 +88,10 @@ const generateBucketPlan = async (context, state, buckets) => {
                 logicalId,
                 action: 'create',
                 resourceType: 'ALIYUN_OSS_BUCKET',
-                changes: { before: currentState.definition, after: desiredDefinition },
+                changes: {
+                    before: normalizeDefinitionForDisplay(currentState.definition),
+                    after: desiredDefinition,
+                },
             };
         }
     }));

package/dist/src/stack/aliyunStack/ossResource.js

@@ -136,11 +136,14 @@ const createBucketResource = async (context, bucket, state) => {
     const sid = (0, common_1.buildSid)('aliyun', 'oss', context.stage, config.bucketName);
     const logicalId = `buckets.${bucket.key}`;
     const instances = [buildOssInstanceFromProvider(bucketInfo, sid)];
+    const websiteCodeHash = bucket.website?.code
+        ? (0, common_1.computeDirectoryHash)(node_path_1.default.resolve(process.cwd(), bucket.website.code))
+        : undefined;
     const partialResourceState = {
         mode: 'managed',
         region: context.region,
         definition: {
-            ...(0, ossTypes_1.extractOssBucketDefinition)(config),
+            ...(0, ossTypes_1.extractOssBucketDefinition)(config, websiteCodeHash),
             ...(bucket.website?.domain != null ? { domainBound: null } : {}),
         },
         instances,
@@ -224,7 +227,7 @@ const createBucketResource = async (context, bucket, state) => {
         mode: 'managed',
         region: context.region,
         definition: {
-            ...(0, ossTypes_1.extractOssBucketDefinition)(config),
+            ...(0, ossTypes_1.extractOssBucketDefinition)(config, websiteCodeHash),
             ...(bucket.website?.domain != null
                 ? { domainBound: cnameInfo?.bucketCnameBound ?? null }
                 : {}),
@@ -260,6 +263,9 @@ const updateBucketResource = async (context, bucket, state) => {
     const sid = (0, common_1.buildSid)('aliyun', 'oss', context.stage, config.bucketName);
     const logicalId = `buckets.${bucket.key}`;
     const instances = [buildOssInstanceFromProvider(bucketInfo, sid)];
+    const websiteCodeHash = bucket.website?.code
+        ? (0, common_1.computeDirectoryHash)(node_path_1.default.resolve(process.cwd(), bucket.website.code))
+        : undefined;
     const existingState = state.resources[logicalId];
     const existingDnsInstances = existingState?.instances?.filter((i) => i.type === types_1.ResourceTypeEnum.ALIYUN_OSS_DNS_CNAME);
     const existingPrimaryDnsInstance = existingDnsInstances?.find((i) => !i.isWwwVariant);
@@ -338,7 +344,7 @@ const updateBucketResource = async (context, bucket, state) => {
         mode: 'managed',
         region: context.region,
         definition: {
-            ...(0, ossTypes_1.extractOssBucketDefinition)(config),
+            ...(0, ossTypes_1.extractOssBucketDefinition)(config, websiteCodeHash),
             ...(bucket.website?.domain != null
                 ? { domainBound: cnameInfo?.bucketCnameBound ?? null }
                 : {}),

package/dist/src/stack/aliyunStack/ossTypes.js

@@ -57,7 +57,7 @@ const bucketToOssBucketConfig = (bucket) => {
     return config;
 };
 exports.bucketToOssBucketConfig = bucketToOssBucketConfig;
-const extractOssBucketDefinition = (config) => {
+const extractOssBucketDefinition = (config, websiteCodeHash) => {
     return {
         bucketName: config.bucketName,
         acl: config.acl ?? null,
@@ -67,6 +67,7 @@ const extractOssBucketDefinition = (config) => {
                 errorDocument: config.websiteConfig.errorDocument ?? null,
             }
             : {},
+        websiteCodeHash: websiteCodeHash ?? null,
         storageClass: config.storageClass ?? null,
         domain: config.domain ?? null,
         wwwBindApex: config.wwwBindApex ?? false,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@geek-fun/serverlessinsight",
-  "version": "0.6.12",
+  "version": "0.6.14",
   "description": "Full life cycle cross providers serverless application management for your fast-growing business.",
   "homepage": "https://serverlessinsight.geekfun.club",
   "main": "dist/src/index.js",