diff --git a/es2panda/binder/variable.h b/es2panda/binder/variable.h
index 65eb6a31fe7613a95b58902450076887c6641f61..6ddcf184f224552658d4550034701a6f2b183792 100644
--- a/es2panda/binder/variable.h
+++ b/es2panda/binder/variable.h
@@ -49,7 +49,7 @@ public:
     NO_COPY_SEMANTIC(Variable);
     NO_MOVE_SEMANTIC(Variable);
 
-    VariableType virtual Type() const = 0;
+    virtual VariableType Type() const = 0;
 
 #define DECLARE_CHECKS_CASTS(variableType, className)     \
     bool Is##className() const                            \
diff --git a/es2panda/compiler/core/regAllocator.cpp b/es2panda/compiler/core/regAllocator.cpp
index c6d9ac35373c50ca9b24a8ff07b9f18e970bc78d..871178f6fd8287c42b194e3c0aba4eff2c6f3a13 100644
--- a/es2panda/compiler/core/regAllocator.cpp
+++ b/es2panda/compiler/core/regAllocator.cpp
@@ -148,8 +148,8 @@ void RegAllocator::AdjustInsRegWhenHasSpill()
     pg_->SetInsns(newInsns);
 }
 
-void RegAllocator::AdjustInsSpill(Span<VReg *> &registers, IRNode *ins, ArenaList<IRNode *> &newInsns,
-                                  std::vector<OperandKind> &regsKind)
+void RegAllocator::AdjustInsSpill(const Span<VReg *> &registers, IRNode *ins, ArenaList<IRNode *> &newInsns,
+                                  const std::vector<OperandKind> &regsKind)
 {
     ASSERT(spillIndex_ == 0);
     ASSERT(!regsKind.empty());
diff --git a/es2panda/compiler/core/regAllocator.h b/es2panda/compiler/core/regAllocator.h
index dd0f440c5f2ac206d9009db5b6e653a07a63fe90..984bab01e72848c35800ddaed13021a02329776e 100644
--- a/es2panda/compiler/core/regAllocator.h
+++ b/es2panda/compiler/core/regAllocator.h
@@ -121,8 +121,8 @@ private:
     void Run(IRNode *ins);
     void Run(IRNode *ins, size_t argCount);
     void Run(IRNode *ins, int64_t typeIndex);
-    void AdjustInsSpill(Span<VReg *> &registers, IRNode *ins, ArenaList<IRNode *> &newInsns,
-                        std::vector<OperandKind> &regsKind);
+    void AdjustInsSpill(const Span<VReg *> &registers, IRNode *ins, ArenaList<IRNode *> &newInsns,
+                        const std::vector<OperandKind> &regsKind);
     void AdjustRangeInsSpill(Span<VReg *> &registers, IRNode *ins, ArenaList<IRNode *> &newInsns);
 
     template <typename T, typename... Args>
diff --git a/es2panda/es2panda.cpp b/es2panda/es2panda.cpp
index 7c429743342c221a3f1dc00d63ee1b99e01e5578..59a7576a157ea7c0b9c9b7b345ee033a6a5bb1e5 100644
--- a/es2panda/es2panda.cpp
+++ b/es2panda/es2panda.cpp
@@ -24,7 +24,6 @@
 #include <parser/transformer/transformer.h>
 #include <typescript/checker.h>
 #include <util/helpers.h>
-#include <util/hotfix.h>
 
 #include <libpandabase/utils/hash.h>
 
@@ -74,14 +73,7 @@ panda::pandasm::Program *Compiler::Compile(const SourceFile &input, const Compil
         return createJsonContentProgram(src, rname);
     }
 
-    bool needDumpSymbolFile = !options.hotfixOptions.dumpSymbolTable.empty();
-    bool needGeneratePatch = options.hotfixOptions.generatePatch && !options.hotfixOptions.symbolTable.empty();
-    util::Hotfix *hotfixHelper = nullptr;
-    if (symbolTable && (needDumpSymbolFile || needGeneratePatch)) {
-        hotfixHelper = new util::Hotfix(needDumpSymbolFile, needGeneratePatch, input.recordName, symbolTable);
-        parser_->AddHotfixHelper(hotfixHelper);
-        compiler_->AddHotfixHelper(hotfixHelper);
-    }
+    auto *hotfixHelper = InitHotfixHelper(input, options, symbolTable);
 
     try {
         auto ast = parser_->Parse(fname, src, rname, kind);
@@ -110,22 +102,37 @@ panda::pandasm::Program *Compiler::Compile(const SourceFile &input, const Compil
                                           sourcefile : options.debugInfoSourceFile;
         auto *prog = compiler_->Compile(&ast, options, debugInfoSourceFile, pkgName);
 
-        if (hotfixHelper) {
-            delete hotfixHelper;
-            hotfixHelper = nullptr;
-        }
+        CleanHotfixHelper(hotfixHelper);
         return prog;
     } catch (const class Error &e) {
         error_ = e;
 
-        if (hotfixHelper) {
-            delete hotfixHelper;
-            hotfixHelper = nullptr;
-        }
+        CleanHotfixHelper(hotfixHelper);
         return nullptr;
     }
 }
 
+util::Hotfix *Compiler::InitHotfixHelper(const SourceFile &input, const CompilerOptions &options, util::SymbolTable *symbolTable)
+{
+    bool needDumpSymbolFile = !options.hotfixOptions.dumpSymbolTable.empty();
+    bool needGeneratePatch = options.hotfixOptions.generatePatch && !options.hotfixOptions.symbolTable.empty();
+    util::Hotfix *hotfixHelper = nullptr;
+    if (symbolTable && (needDumpSymbolFile || needGeneratePatch)) {
+        hotfixHelper = new util::Hotfix(needDumpSymbolFile, needGeneratePatch, input.recordName, symbolTable);
+        parser_->AddHotfixHelper(hotfixHelper);
+        compiler_->AddHotfixHelper(hotfixHelper);
+    }
+    return hotfixHelper;
+}
+
+void Compiler::CleanHotfixHelper(const util::Hotfix *hotfixHelper)
+{
+    if (hotfixHelper) {
+        delete hotfixHelper;
+        hotfixHelper = nullptr;
+    }
+}
+
 void Compiler::DumpAsm(const panda::pandasm::Program *prog)
 {
     compiler::CompilerImpl::DumpAsm(prog);
diff --git a/es2panda/es2panda.h b/es2panda/es2panda.h
index 8ac8084008a2c8b4016fef985cb682f46c537d8b..614bb3f2afa3403af39338d7d0356ebe8f08631f 100644
--- a/es2panda/es2panda.h
+++ b/es2panda/es2panda.h
@@ -18,6 +18,7 @@
 
 #include <macros.h>
 #include <mem/arena_allocator.h>
+#include <util/hotfix.h>
 #include <util/programCache.h>
 #include <util/symbolTable.h>
 
@@ -197,6 +198,9 @@ public:
     }
 
 private:
+    util::Hotfix *InitHotfixHelper(const SourceFile &input, const CompilerOptions &options, util::SymbolTable *symbolTable);
+    static void CleanHotfixHelper(const util::Hotfix *hotfixHelper);
+    
     parser::ParserImpl *parser_;
     compiler::CompilerImpl *compiler_;
     std::unique_ptr<parser::Transformer> transformer_ {nullptr};
diff --git a/es2panda/ir/expressions/literals/bigIntLiteral.cpp b/es2panda/ir/expressions/literals/bigIntLiteral.cpp
index 0a274fa6a227063a2e92c215839d7d85c1282f9a..47251cae62baa0140f520d192632c84824e05fe8 100644
--- a/es2panda/ir/expressions/literals/bigIntLiteral.cpp
+++ b/es2panda/ir/expressions/literals/bigIntLiteral.cpp
@@ -30,7 +30,7 @@ void BigIntLiteral::Dump(ir::AstDumper *dumper) const
 
 void BigIntLiteral::Compile(compiler::PandaGen *pg) const
 {
-    util::StringView bigIntValue = src_.Substr(0, src_.Length()-1);
+    util::StringView bigIntValue = src_.Substr(0, src_.Length() - 1);
     pg->LoadAccumulatorBigInt(this, bigIntValue);
 }
 
diff --git a/es2panda/parser/parserImpl.cpp b/es2panda/parser/parserImpl.cpp
index cce58af4b983c4e3aec671a5e25308b0e737af5b..a245d55f623237c7552bbaa1efe1f2d3e0ac0393 100644
--- a/es2panda/parser/parserImpl.cpp
+++ b/es2panda/parser/parserImpl.cpp
@@ -866,7 +866,7 @@ bool ParserImpl::IsTSNamedTupleMember()
     return isNamedMember;
 }
 
-void ParserImpl::HandleRestType(ir::AstNodeType elementType, bool *hasRestType)
+void ParserImpl::HandleRestType(ir::AstNodeType elementType, bool *hasRestType) const
 {
     if (elementType ==  ir::AstNodeType::TS_ARRAY_TYPE && *hasRestType) {
         ThrowSyntaxError("A rest element cannot follow another rest element");
@@ -878,7 +878,6 @@ ir::Expression *ParserImpl::ParseTsTupleElement(ir::TSTupleKind *kind, bool *see
 {
     lexer::SourcePosition startPos = lexer_->GetToken().Start();
     ir::Expression *element = nullptr;
-    bool isOptional = false;
     bool isRestType = false;
     TypeAnnotationParsingOptions options = TypeAnnotationParsingOptions::THROW_ERROR;
 
@@ -897,6 +896,7 @@ ir::Expression *ParserImpl::ParseTsTupleElement(ir::TSTupleKind *kind, bool *see
         elementIdent->SetRange(lexer_->GetToken().Loc());
         lexer_->NextToken();  // eat identifier
 
+        bool isOptional = false;
         if (lexer_->GetToken().Type() == lexer::TokenType::PUNCTUATOR_QUESTION_MARK) {
             lexer_->NextToken();  // eat '?'
             isOptional = true;
@@ -939,7 +939,6 @@ ir::Expression *ParserImpl::ParseTsTupleElement(ir::TSTupleKind *kind, bool *see
             element = AllocNode<ir::TSOptionalType>(std::move(element));
             element->SetRange({elementStartPos, lexer_->GetToken().End()});
             lexer_->NextToken();  // eat '?'
-            isOptional = true;
             *seenOptional = true;
         } else if (*seenOptional && !isRestType) {
             ThrowSyntaxError("A required element cannot follow an optional element");
@@ -3101,6 +3100,7 @@ ArenaVector<ir::Expression *> ParserImpl::ParseFunctionParams(bool isDeclare,
         if (context_.Status() & ParserStatus::IN_METHOD_DEFINITION) {
             auto decorators = ParseDecorators();
             if (!decorators.empty()) {
+                ASSERT(paramDecorators != nullptr);
                 paramDecorators->push_back({index, std::move(decorators)});
             }
         }
@@ -3520,7 +3520,7 @@ ir::Expression *ParserImpl::ParseFunctionParameter(bool isDeclare)
     return functionParameter;
 }
 
-void ParserImpl::ValidateLvalueAssignmentTarget(ir::Expression *node)
+void ParserImpl::ValidateLvalueAssignmentTarget(ir::Expression *node) const
 {
     switch (node->Type()) {
         case ir::AstNodeType::IDENTIFIER: {
diff --git a/es2panda/parser/parserImpl.h b/es2panda/parser/parserImpl.h
index 03288da0b31ebfcdbfbbbbc901e470cf016ace28..cd12f818eb074aad483505785dba4c48508b877b 100644
--- a/es2panda/parser/parserImpl.h
+++ b/es2panda/parser/parserImpl.h
@@ -261,7 +261,7 @@ private:
     ir::Expression *ParseTsQualifiedReference(ir::Expression *typeName);
     ir::Expression *ParseTsTypeReferenceOrQuery(bool parseQuery = false);
     bool IsTSNamedTupleMember();
-    void HandleRestType(ir::AstNodeType elementType, bool *hasRestType);
+    void HandleRestType(ir::AstNodeType elementType, bool *hasRestType) const;
     ir::Expression *ParseTsTupleElement(ir::TSTupleKind *kind, bool *seenOptional, bool *hasRestType);
     ir::TSTupleType *ParseTsTupleType();
     ir::TSImportType *ParseTsImportType(const lexer::SourcePosition &startLoc, bool isTypeof = false);
@@ -404,7 +404,7 @@ private:
     ir::AstNode *ParseImportDefaultSpecifier(ArenaVector<ir::AstNode *> *specifiers);
     ir::AstNode *ParseImportSpecifiers(ArenaVector<ir::AstNode *> *specifiers);
     void ValidateAssignmentTarget(ExpressionParseFlags flags, ir::Expression *node);
-    void ValidateLvalueAssignmentTarget(ir::Expression *node);
+    void ValidateLvalueAssignmentTarget(ir::Expression *node) const;
     void ValidateArrowParameterBindings(const ir::Expression *node);
 
     ir::ExportDefaultDeclaration *ParseExportDefaultDeclaration(const lexer::SourcePosition &startLoc,
diff --git a/es2panda/parser/transformer/transformer.h b/es2panda/parser/transformer/transformer.h
index 9e90beb7f2c6b3ca624c5c6faaaa368012f17941..69ab3c21f7f7dbf4a70b5b97bd67b8212c788611 100644
--- a/es2panda/parser/transformer/transformer.h
+++ b/es2panda/parser/transformer/transformer.h
@@ -115,7 +115,7 @@ private:
                                                                    ir::Expression *init = nullptr,
                                                                    bool needBinding = true);
     ir::CallExpression *CreateCallExpressionForTsModule(ir::TSModuleDeclaration *node,
-                                                        util::StringView paramName,
+                                                        util::StringView name,
                                                         bool isExport = false);
     ir::Expression *CreateTsModuleParam(util::StringView paramName, bool isExport);
     ir::ExpressionStatement *CreateTsModuleAssignment(util::StringView name);
diff --git a/test262/run_sunspider.py b/test262/run_sunspider.py
index 77cdf53747a428e6b260701949c0a3a438517c01..d3acda909b1916dbdc373df0c3bc149b520c41a3 100755
--- a/test262/run_sunspider.py
+++ b/test262/run_sunspider.py
@@ -227,19 +227,20 @@ class ArkProgram():
 
         self.arch_root = self.args.ark_arch_root
 
-    def check_compile_mod(self, dependency):
-        with open(dependency, 'r', encoding='utf-8') as f:
-            context_file = f.read()
+    def check_compile_mode(self, file):
+        with open(file, 'r', encoding='utf-8') as check_file:
+            content_file = check_file.read()
             module_pattern = '((?:export|import)\s+(?:{[\s\S]+}|\*))|'
             module_pattern += '(export\s+(?:let|const|var|function|class|default))|'
             module_pattern += '(import\s+[\'\"].+[\'\"])'
-            module_mode_list = re.findall(module_pattern, context_file)
+            module_mode_list = re.findall(module_pattern, content_file)
 
-        for module_mode in list(set(module_mode_list)):
-            if len(module_mode[0]) != 0 or len(module_mode[1]) != 0 or \
-                len(module_mode[2]) != 0:
-                return True
-        if "flags: [module]" in context_file or "/language/module-code/" in self.js_file:
+            for module_mode in list(set(module_mode_list)):
+                if len(module_mode[0]) != 0 or len(module_mode[1]) != 0 or \
+                    len(module_mode[2]) != 0:
+                    return True
+
+        if "flags: [module]" in content_file or "/language/module-code/" in self.js_file:
             return True
 
         return False
@@ -260,7 +261,7 @@ class ArkProgram():
         frontend_tool = self.ark_frontend_binary
         merge_abc_binary = self.args.merge_abc_binary
         merge_abc_mode = self.merge_abc_mode
-        compile_as_module = self.check_compile_mod(dependency)
+        compile_as_module = self.check_compile_mode(dependency)
 
         if self.ark_frontend == ARK_FRONTEND_LIST[0]:
             if merge_abc_mode != "0":
@@ -297,9 +298,8 @@ class ArkProgram():
             file_dir = os.path.split(self.js_file)[0]
             is_apart_abc_existed = os.path.exists(file_dir + "/" + output_abc)
             dependency_file_prefix = os.path.basename(dependency)[:-3]
-            dependency_bin_file = file_dir + "/" + \
-                                    ".".join([dependency_file_prefix,
-                                    PROTO_BIN_SUFFIX])
+            dependency_bin_file = '%s/%s.%s' % (file_dir,
+                                                dependency_file_prefix, PROTO_BIN_SUFFIX)
             cmd_args = [merge_abc_binary, '--input', dependency_bin_file,
                         '--suffix', PROTO_BIN_SUFFIX, '--outputFilePath',
                         file_dir, '--output', output_abc]
@@ -320,9 +320,8 @@ class ArkProgram():
 
             for dependency in list(set(dependencies)):
                 dependency_file_prefix = os.path.basename(dependency)[:-3]
-                dependency_bin_file = file_dir + "/" + \
-                                      ".".join([dependency_file_prefix,
-                                      PROTO_BIN_SUFFIX])
+                dependency_bin_file = '%s/%s.%s' % (file_dir,
+                                                    dependency_file_prefix, PROTO_BIN_SUFFIX)
                 # test262 report syntax error cases
                 if not os.path.exists(dependency_bin_file):
                     generate_merged_abc = False
@@ -372,13 +371,13 @@ class ArkProgram():
         if (file_name in self.module_list or file_name in self.dynamicImport_list):
             search_dir = os.path.dirname(js_file.replace(BASE_OUT_DIR, DATA_DIR))
             dependencies = collect_module_dependencies(js_file, search_dir, [])
-            compile_as_module = self.check_compile_mod(js_file)
+            compile_as_module = self.check_compile_mode(js_file)
             if (self.ark_frontend == ARK_FRONTEND_LIST[1]):
                 if list(set(dependencies)):
                     for dependency in list(set(dependencies)):
                         dependency_file = os.path.basename(dependency)
                         dependency_name = os.path.splitext(dependency_file)[0]
-                        out_dependency_pre = file_dir + "/" + dependency_name
+                        out_dependency_pre = '%s/%s' % (file_dir, dependency_name)
                         out_dependency_proto = f"{out_dependency_pre}.protoBin"
                         is_dependency_proto_existed = os.path.exists(out_dependency_proto)
                         if not is_dependency_proto_existed:
@@ -476,7 +475,7 @@ class ArkProgram():
             print_command(cmd_args)
 
     def execute_aot(self):
-        unForceGc = False
+        unforce_gc = False
         os.environ["LD_LIBRARY_PATH"] = self.libs_dir
         file_name_pre = os.path.splitext(self.js_file)[0]
         cmd_args = []
@@ -498,9 +497,9 @@ class ArkProgram():
                         f'{file_name_pre}.abc']
         elif self.arch == ARK_ARCH_LIST[0]:
             if file_name_pre in FORCE_GC_SKIP_TESTS:
-                unForceGc = True
+                unforce_gc = True
             asm_arg1 = "--enable-force-gc=true"
-            if unForceGc:
+            if unforce_gc:
                 asm_arg1 = "--enable-force-gc=false"
             cmd_args = [self.ark_tool, ICU_PATH, asm_arg1,
                         f'--aot-file={file_name_pre}',
@@ -514,7 +513,7 @@ class ArkProgram():
         return retcode
 
     def execute(self):
-        unForceGc = False
+        unforce_gc = False
         if platform.system() == "Windows" :
             #add env path for cmd/powershell execute
             libs_dir = self.libs_dir.replace(":", ";")
@@ -542,9 +541,9 @@ class ArkProgram():
                         f'{file_name_pre}.abc']
         elif self.arch == ARK_ARCH_LIST[0]:
             if file_name_pre in FORCE_GC_SKIP_TESTS:
-                unForceGc = True
+                unforce_gc = True
             asm_arg1 = "--enable-force-gc=true"
-            if unForceGc:
+            if unforce_gc:
                 asm_arg1 = "--enable-force-gc=false"
             cmd_args = [self.ark_tool, ICU_PATH, asm_arg1,
                         f'{file_name_pre}.abc']
diff --git a/test262/run_test262.py b/test262/run_test262.py
index 39569b3c9f59c6cee4ddabaca9007056f1dd02ab..58dfc7d03c2e5ec62ee94a64cc24f226798e60ed 100755
--- a/test262/run_test262.py
+++ b/test262/run_test262.py
@@ -318,8 +318,8 @@ class TestPrepare():
 
         for file_name in file_names:
             with open(file_name, 'r', encoding='utf-8') as file:
-                content = file.read()
-                if esid in content:
+                file_content = file.read()
+                if esid in file_content:
                     files.append(file_name.split(origin_dir)[1])
         return files
 
diff --git a/ts2panda/ts2abc/ts2abc.cpp b/ts2panda/ts2abc/ts2abc.cpp
index 81b63489360a886f089a206584bbfd4c0695d2d3..340d8a1b513806255e0a89fac164dc7c75049cb6 100644
--- a/ts2panda/ts2abc/ts2abc.cpp
+++ b/ts2panda/ts2abc/ts2abc.cpp
@@ -1099,7 +1099,8 @@ static void ParseInputJsonFileContent(const Json::Value &rootValue, panda::panda
         auto inputJsonFileContentField = panda::pandasm::Field(LANG_EXT);
         inputJsonFileContentField.name = "jsonFileContent";
         inputJsonFileContentField.type = panda::pandasm::Type("u32", 0);
-        inputJsonFileContentField.metadata->SetValue(panda::pandasm::ScalarValue::Create<panda::pandasm::Value::Type::STRING>(
+        inputJsonFileContentField.metadata->SetValue(
+            panda::pandasm::ScalarValue::Create<panda::pandasm::Value::Type::STRING>(
             static_cast<std::string_view>(rootValue["ijfc"].asString())));
         rec.field_list.emplace_back(std::move(inputJsonFileContentField));
     }
@@ -1490,7 +1491,7 @@ static bool EmitProgram(const std::string &output, int optLevel, std::string opt
     return true;
 }
 
-static bool EmitAndRestoreProgram(panda::pandasm::Program &prog, panda::ts2abc::Options options)
+static bool EmitAndRestoreProgram(panda::pandasm::Program &prog, const panda::ts2abc::Options &options)
 {
     if (!EmitProgram(g_outputFileName, options.GetOptLevelArg(), options.GetOptLogLevelArg(), prog)) {
         std::cerr << "fail to emit porgram " << g_outputFileName << " in HandleBuffer" << std::endl;
@@ -1502,7 +1503,7 @@ static bool EmitAndRestoreProgram(panda::pandasm::Program &prog, panda::ts2abc::
 }
 
 static bool HandleBuffer(const int &ret, char *buff, std::string &data, panda::pandasm::Program &prog,
-                         panda::ts2abc::Options options)
+                         const panda::ts2abc::Options &options)
 {
     uint32_t startPos = 0;
     if (options.IsMultiProgramsPipe() && ((buff[0] == '*' && data.back() != '#') ||
@@ -1576,7 +1577,7 @@ static bool ReadFromPipe(panda::pandasm::Program &prog, panda::ts2abc::Options o
     return true;
 }
 
-bool GenerateProgramsFromPipe(panda::ts2abc::Options options)
+bool GenerateProgramsFromPipe(const panda::ts2abc::Options &options)
 {
     panda::pandasm::Program prog = panda::pandasm::Program();
     prog.lang = panda::pandasm::extensions::Language::ECMASCRIPT;
diff --git a/ts2panda/ts2abc/ts2abc.h b/ts2panda/ts2abc/ts2abc.h
index e1d9bf08e61c2d1fe59bbce33486f987b82efe0d..c64d63fb98ad015a393a7c73b486b64ffeba699e 100644
--- a/ts2panda/ts2abc/ts2abc.h
+++ b/ts2panda/ts2abc/ts2abc.h
@@ -53,7 +53,7 @@ enum class OptLevel {
 
 bool HandleJsonFile(const std::string &input, std::string &data);
 bool GenerateProgram(const std::string &data, const std::string &output, panda::ts2abc::Options options);
-bool GenerateProgramsFromPipe(panda::ts2abc::Options options);
+bool GenerateProgramsFromPipe(const panda::ts2abc::Options &options);
 bool CompileNpmEntries(const std::string &input, const std::string &output);
 bool GetDebugLog();
 void ParseLogEnable(const Json::Value &rootValue);