refactor: Add multi-lang support and remove unnecessary comments

Applied coding guidelines:
1. Added multi-lang support for all log messages in event, function, and bucket resources
2. Removed unnecessary comments throughout the codebase
3. Refactored dnsOperations.ts to use functional approach with const arrow functions

Changes:
- Added lang keys to en.ts and zh-CN.ts for API Gateway, Functions, and Buckets
- Updated apigwResource.ts to use lang.__() for all logger calls
- Updated fc3Resource.ts to use lang.__() for all logger calls
- Updated ossResource.ts to use lang.__() for all logger calls
- Refactored dnsOperations.ts from function returning object to const arrow functions with proper composition
- Removed unnecessary comments that explained obvious code behavior

All messages now support English and Chinese localization.

Co-authored-by: Blankll <28639911+Blankll@users.noreply.github.com>

Author: Copilot

src/common/aliyunClient/dnsOperations.ts

@@ -3,9 +3,6 @@ import * as dns from '@alicloud/alidns20150109';
 
 type DnsSdkClient = DnsClient;
 
-/**
- * DNS record configuration
- */
 export type DnsRecordConfig = {
   domainName: string;
   rr: string;
@@ -14,9 +11,6 @@ export type DnsRecordConfig = {
   ttl?: number;
 };
 
-/**
- * DNS record info
- */
 export type DnsRecordInfo = {
   recordId?: string;
   domainName?: string;
@@ -27,76 +21,78 @@ export type DnsRecordInfo = {
   status?: string;
 };
 
-/**
- * Create DNS operations
- */
-export function createDnsOperations(dnsClient: DnsSdkClient) {
-  const operations = {
-    /**
-     * Add a DNS record
-     */
-    addDomainRecord: async (config: DnsRecordConfig): Promise<string> => {
-      const request = new dns.AddDomainRecordRequest({
-        domainName: config.domainName,
-        RR: config.rr,
-        type: config.type,
-        value: config.value,
-        TTL: config.ttl || 600,
-      });
+export type DnsOperations = {
+  addDomainRecord: (config: DnsRecordConfig) => Promise<string>;
+  deleteDomainRecord: (recordId: string) => Promise<void>;
+  describeDomainRecords: (domainName: string, rrKeyWord?: string) => Promise<Array<DnsRecordInfo>>;
+  checkDomainRecordExists: (domainName: string, rr: string) => Promise<boolean>;
+};
+
+const addDomainRecord = (dnsClient: DnsSdkClient) => async (
+  config: DnsRecordConfig,
+): Promise<string> => {
+  const request = new dns.AddDomainRecordRequest({
+    domainName: config.domainName,
+    RR: config.rr,
+    type: config.type,
+    value: config.value,
+    TTL: config.ttl || 600,
+  });
+
+  const response = await dnsClient.addDomainRecord(request);
+  return response.body?.recordId || '';
+};
 
-      const response = await dnsClient.addDomainRecord(request);
-      return response.body?.recordId || '';
-    },
+const deleteDomainRecord = (dnsClient: DnsSdkClient) => async (recordId: string): Promise<void> => {
+  const request = new dns.DeleteDomainRecordRequest({
+    recordId,
+  });
 
-    /**
-     * Delete a DNS record
-     */
-    deleteDomainRecord: async (recordId: string): Promise<void> => {
-      const request = new dns.DeleteDomainRecordRequest({
-        recordId,
-      });
+  await dnsClient.deleteDomainRecord(request);
+};
 
-      await dnsClient.deleteDomainRecord(request);
-    },
+const describeDomainRecords = (dnsClient: DnsSdkClient) => async (
+  domainName: string,
+  rrKeyWord?: string,
+): Promise<Array<DnsRecordInfo>> => {
+  const request = new dns.DescribeDomainRecordsRequest({
+    domainName,
+    RRKeyWord: rrKeyWord,
+  });
 
-    /**
-     * Describe domain records
-     */
-    describeDomainRecords: async (
-      domainName: string,
-      rrKeyWord?: string,
-    ): Promise<Array<DnsRecordInfo>> => {
-      const request = new dns.DescribeDomainRecordsRequest({
-        domainName,
-        RRKeyWord: rrKeyWord,
-      });
+  const response = await dnsClient.describeDomainRecords(request);
+  const records = response.body?.domainRecords?.record || [];
 
-      const response = await dnsClient.describeDomainRecords(request);
-      const records = response.body?.domainRecords?.record || [];
+  return records.map((record: dns.DescribeDomainRecordsResponseBodyDomainRecordsRecord) => ({
+    recordId: record.recordId,
+    domainName: record.domainName,
+    rr: record.RR,
+    type: record.type,
+    value: record.value,
+    ttl: record.TTL,
+    status: record.status,
+  }));
+};
 
-      return records.map((record: dns.DescribeDomainRecordsResponseBodyDomainRecordsRecord) => ({
-        recordId: record.recordId,
-        domainName: record.domainName,
-        rr: record.RR,
-        type: record.type,
-        value: record.value,
-        ttl: record.TTL,
-        status: record.status,
-      }));
-    },
+const checkDomainRecordExists = (
+  describeFn: (domainName: string, rrKeyWord?: string) => Promise<Array<DnsRecordInfo>>,
+) => async (domainName: string, rr: string): Promise<boolean> => {
+  try {
+    const records = await describeFn(domainName, rr);
+    return records.some((record) => record.rr === rr);
+  } catch {
+    return false;
+  }
+};
 
-    /**
-     * Check if a DNS record exists
-     */
-    checkDomainRecordExists: async (domainName: string, rr: string): Promise<boolean> => {
-      try {
-        const records = await operations.describeDomainRecords(domainName, rr);
-        return records.some((record) => record.rr === rr);
-      } catch {
-        return false;
-      }
-    },
+export const createDnsOperations = (dnsClient: DnsSdkClient): DnsOperations => {
+  const describe = describeDomainRecords(dnsClient);
+  
+  return {
+    addDomainRecord: addDomainRecord(dnsClient),
+    deleteDomainRecord: deleteDomainRecord(dnsClient),
+    describeDomainRecords: describe,
+    checkDomainRecordExists: checkDomainRecordExists(describe),
   };
+};
 
-  return operations;
-}

src/lang/en.ts

@@ -214,4 +214,20 @@ export const en = {
   LOCK_TIME_AGO_MINUTES: '{{minutes}} minutes ago',
   LOCK_TIME_AGO_ONE_HOUR: '1 hour ago',
   LOCK_TIME_AGO_HOURS: '{{hours}} hours ago',
+
+  // API Gateway messages
+  APIGW_GROUP_FOUND_REUSING: 'Found existing API Group: {{groupName}}, will reuse it',
+  APIGW_DOMAIN_BINDING_FAILED: 'Failed to bind custom domain: {{error}}',
+  APIGW_GROUP_APIS_CREATED_DOMAIN_FAILED:
+    'API Gateway group and APIs created successfully, but domain binding failed',
+  APIGW_STATE_SAVED_RETRY: 'State has been saved. You can retry or fix domain configuration',
+
+  // Function resource messages
+  FC3_DEPENDENT_RESOURCES_TRACKED:
+    'Dependent resources (SLS, RAM, Security Group, NAS) are tracked in state',
+  FC3_CAN_RETRY_DEPLOYMENT:
+    'You can retry deployment - the system will reuse existing dependent resources',
+
+  // Bucket resource messages
+  OSS_BUCKET_TRACKED_CAN_RETRY: 'Bucket is tracked in state, you can retry deployment to upload files',
 };

src/lang/zh-CN.ts

@@ -176,4 +176,17 @@ export const zhCN = {
   PARTIAL_FAILURE_NEXT_STEPS:
     '后续步骤: 1) 查看上面的错误, 2) 修复配置问题, 3) 再次运行 deploy 继续。',
   STATE_PERSISTED_AFTER_OPERATION: '{{action}} {{resourceId}} 后状态已持久化',
+
+  // API Gateway messages
+  APIGW_GROUP_FOUND_REUSING: '找到现有 API 分组: {{groupName}}，将复用它',
+  APIGW_DOMAIN_BINDING_FAILED: '绑定自定义域名失败: {{error}}',
+  APIGW_GROUP_APIS_CREATED_DOMAIN_FAILED: 'API 网关分组和 API 创建成功，但域名绑定失败',
+  APIGW_STATE_SAVED_RETRY: '状态已保存。您可以重试或修复域名配置',
+
+  // Function resource messages
+  FC3_DEPENDENT_RESOURCES_TRACKED: '依赖资源（SLS、RAM、安全组、NAS）已在状态中跟踪',
+  FC3_CAN_RETRY_DEPLOYMENT: '您可以重试部署 - 系统将复用现有的依赖资源',
+
+  // Bucket resource messages
+  OSS_BUCKET_TRACKED_CAN_RETRY: '存储桶已在状态中跟踪，您可以重试部署以上传文件',
 };