@ohos-cpf/3rdloop 0.0.2 → 0.0.3
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +244 -244
- package/bin/3rdloop.mjs +46 -46
- package/lib/cli.js +781 -781
- package/lib/config-cmd.js +451 -451
- package/lib/config.js +318 -318
- package/lib/doctor.js +113 -113
- package/lib/exit-codes.js +30 -30
- package/lib/opencode.js +155 -155
- package/lib/orch.js +426 -426
- package/lib/queries.js +263 -263
- package/lib/step.js +325 -325
- package/lib/ui.js +222 -222
- package/lib/update.js +279 -279
- package/lib/workflow.js +308 -308
- package/lib/workflows.js +131 -131
- package/package.json +50 -50
- package/vendor/Server/Agent/SkillSelector/README.md +343 -343
- package/vendor/Server/Agent/SkillSelector/core/Agent.js +207 -207
- package/vendor/Server/Agent/SkillSelector/index.js +14 -14
- package/vendor/Server/Agent/SkillSelector/llm/llmClient.js +313 -313
- package/vendor/Server/Agent/SkillSelector/llm/prompts.js +161 -161
- package/vendor/Server/Agent/SkillSelector/llm/reactSteps.js +144 -144
- package/vendor/Server/Agent/SkillSelector/matcher/fuzzyMatcher.js +433 -433
- package/vendor/Server/Agent/SkillSelector/scanner/skillScanner.js +382 -382
- package/vendor/Server/Agent/SkillSelector/utils/actionWords.js +59 -59
- package/vendor/Server/Brain/AnalysisStrategies.js +422 -422
- package/vendor/Server/Brain/Brain.js +686 -686
- package/vendor/Server/Brain/DecompositionStrategies.js +621 -621
- package/vendor/Server/Brain/PlanPersistence.js +196 -196
- package/vendor/Server/Brain/SkillExecutor.js +251 -251
- package/vendor/Server/Brain/index.js +11 -11
- package/vendor/Server/CLI/cli.js +275 -275
- package/vendor/Server/CLI/deveco-code/index.js +71 -71
- package/vendor/Server/CLI/deveco-code.js +5 -5
- package/vendor/Server/CLI/opencode/index.js +817 -818
- package/vendor/Server/CLI/opencode.js +5 -5
- package/vendor/Server/DbUse/MessageArchive.js +61 -61
- package/vendor/Server/DbUse/StorageManager.js +107 -107
- package/vendor/Server/DbUse/adapters/BaseAdapter.js +94 -94
- package/vendor/Server/DbUse/adapters/FileAdapter.js +138 -138
- package/vendor/Server/DbUse/adapters/JsonAdapter.js +138 -138
- package/vendor/Server/DbUse/adapters/JsonlAdapter.js +114 -114
- package/vendor/Server/DbUse/adapters/SqliteAdapter.js +157 -157
- package/vendor/Server/DbUse/index.js +160 -160
- package/vendor/Server/DbUse/stores/SessionStore.js +279 -279
- package/vendor/Server/DbUse/stores/TaskExecutionStore.js +133 -133
- package/vendor/Server/DbUse/stores/TaskRegistryStore.js +262 -262
- package/vendor/Server/DbUse/stores/TaskStepStore.js +279 -279
- package/vendor/Server/FlexRunner/FlexRunner.js +1759 -1759
- package/vendor/Server/FlexRunner/index.js +12 -12
- package/vendor/Server/Knowledge/KnowledgeImporter.js +523 -523
- package/vendor/Server/Knowledge/KnowledgeManager.js +341 -341
- package/vendor/Server/Knowledge/index.js +33 -33
- package/vendor/Server/LoopEngine/LoopEngine.js +1133 -1133
- package/vendor/Server/LoopEngine/index.js +11 -11
- package/vendor/Server/Orchestrator/Orchestrator.js +0 -3
- package/vendor/Server/Orchestrator/StepNode.js +312 -312
- package/vendor/Server/Orchestrator/index.js +13 -13
- package/vendor/Server/Routes/controllers/BrainController.js +241 -241
- package/vendor/Server/Routes/controllers/FlexRunnerController.js +1125 -1125
- package/vendor/Server/Routes/controllers/HealthController.js +44 -44
- package/vendor/Server/Routes/controllers/KnowledgeImportController.js +355 -355
- package/vendor/Server/Routes/controllers/LoopEngineController.js +788 -788
- package/vendor/Server/Routes/controllers/StatsController.js +57 -57
- package/vendor/Server/Routes/controllers/StreamController.js +67 -67
- package/vendor/Server/Routes/library/logger.js +179 -179
- package/vendor/Server/Routes/library/zipArchive.js +172 -172
- package/vendor/Server/Routes/router/Router.js +204 -204
- package/vendor/Server/Routes/routes/brain.js +47 -47
- package/vendor/Server/Routes/routes/flexrunner.js +72 -72
- package/vendor/Server/Routes/routes/health.js +28 -28
- package/vendor/Server/Routes/routes/index.js +57 -57
- package/vendor/Server/Routes/routes/knowledge.js +55 -55
- package/vendor/Server/Routes/routes/loopengine.js +58 -58
- package/vendor/Server/Routes/routes/orchestrator.js +61 -61
- package/vendor/Server/Routes/routes/registry.js +97 -97
- package/vendor/Server/Routes/routes/stats.js +23 -23
- package/vendor/Server/Routes/routes/stream.js +30 -30
- package/vendor/Server/Routes/server.js +203 -203
- package/vendor/Server/Routes/sse/SseHub.js +308 -308
- package/vendor/Server/Skills/acceptance-check/SKILL.md +191 -191
- package/vendor/Server/Skills/acceptance-check/references/PHASE1_ARTIFACT_COLLECTION.md +52 -52
- package/vendor/Server/Skills/acceptance-check/references/PHASE2_CRITERIA_EVALUATION.md +77 -77
- package/vendor/Server/Skills/acceptance-check/references/PHASE2_EVIDENCE_VERIFICATION.md +70 -70
- package/vendor/Server/Skills/arkts-code-check/SKILL.md +320 -320
- package/vendor/Server/Skills/arkts-code-check/references/CODE_CHECK_GUIDE.md +210 -210
- package/vendor/Server/Skills/arkts-code-check/references/FILE_TYPE_GUIDE.md +28 -28
- package/vendor/Server/Skills/arkts-code-check/references/PHASE1_INFO_COLLECTION.md +233 -233
- package/vendor/Server/Skills/arkts-code-check/references/REPORT_TEMPLATE.md +219 -219
- package/vendor/Server/Skills/arkts-library-compile-validate/SKILL.md +328 -328
- package/vendor/Server/Skills/arkts-library-compile-validate/scripts/build.ps1 +93 -93
- package/vendor/Server/Skills/arkts-library-compile-validate/scripts/code-linter.cjs +432 -432
- package/vendor/Server/Skills/arkts-library-compile-validate/scripts/migrate.cjs +851 -851
- package/vendor/Server/Skills/arkts-library-compile-validate/scripts/ohpm-switch.cjs +788 -788
- package/vendor/Server/Skills/arkts-library-demo-code/SKILL.md +291 -291
- package/vendor/Server/Skills/arkts-library-demo-code/references/API_SIGNATURE_CHECKS.md +68 -68
- package/vendor/Server/Skills/arkts-library-demo-code/references/BUILD_VALIDATION.md +184 -184
- package/vendor/Server/Skills/arkts-library-demo-code/references/CODE_STANDARDS.md +337 -337
- package/vendor/Server/Skills/arkts-library-demo-code/references/EXAMPLES.md +400 -400
- package/vendor/Server/Skills/arkts-library-demo-code/references/SYSTEM_API_LOOKUP.md +98 -98
- package/vendor/Server/Skills/arkts-library-demo-code/references/TEMPLATE.md +544 -544
- package/vendor/Server/Skills/arkts-library-demo-code/scripts/build.ps1 +93 -93
- package/vendor/Server/Skills/arkts-library-demo-code/scripts/code-linter.cjs +432 -432
- package/vendor/Server/Skills/arkts-library-demo-code/scripts/migrate.cjs +851 -851
- package/vendor/Server/Skills/arkts-library-demo-code/scripts/ohpm-switch.cjs +788 -788
- package/vendor/Server/Skills/arkts-library-demo-coverage/SKILL.md +174 -174
- package/vendor/Server/Skills/arkts-library-demo-scenario/SKILL.md +321 -321
- package/vendor/Server/Skills/arkts-library-demo-scenario/references/EXAMPLES.md +100 -100
- package/vendor/Server/Skills/arkts-library-demo-scenario/references/TEMPLATE.md +185 -185
- package/vendor/Server/Skills/arkts-library-documentation-quality-check/SKILL.md +233 -233
- package/vendor/Server/Skills/arkts-library-documentation-quality-check/assets/REPORT_TEMPLATE.md +102 -102
- package/vendor/Server/Skills/arkts-library-documentation-quality-check/references/CHECK_DIMENSIONS.md +89 -89
- package/vendor/Server/Skills/arkts-library-documentation-quality-check/references/COMPLIANCE_CHECKLIST.md +62 -62
- package/vendor/Server/Skills/arkts-library-documentation-quality-check/references/KB_QUERY_GUIDE.md +107 -107
- package/vendor/Server/Skills/arkts-library-documentation-quality-check/references/MCP_TOOL_DISCOVERY.md +124 -124
- package/vendor/Server/Skills/arkts-library-documentation-quality-check/references/REPORT_TEMPLATE.md +90 -90
- package/vendor/Server/Skills/arkts-library-documentation-quality-check/references/SCORING_FORMULA.md +57 -57
- package/vendor/Server/Skills/arkts-library-documentation-quality-check/scripts/validate-skill.cjs +259 -259
- package/vendor/Server/Skills/arkts-library-interface-spec/SKILL.md +97 -97
- package/vendor/Server/Skills/arkts-library-memory-leak-check/SKILL.md +391 -391
- package/vendor/Server/Skills/arkts-library-source-clone/SKILL.md +113 -113
- package/vendor/Server/Skills/arkts-library-stability-check/SKILL.md +312 -312
- package/vendor/Server/Skills/arkts-library-stability-check/assets/REPORT_TEMPLATE.md +154 -154
- package/vendor/Server/Skills/arkts-library-stability-check/references/CHECKLIST.md +165 -165
- package/vendor/Server/Skills/arkts-library-stability-check/references/CODE_EXAMPLES.md +308 -308
- package/vendor/Server/Skills/arkts-library-stability-check/references/SCORING_RULES.md +90 -90
- package/vendor/Server/Skills/arkts-library-stability-check/scripts/validate-skill.cjs +282 -282
- package/vendor/Server/Skills/arkts-library-test-coverage-check/SKILL.md +360 -360
- package/vendor/Server/Skills/arkts-library-test-coverage-check/assets/REPORT_TEMPLATE.md +145 -145
- package/vendor/Server/Skills/arkts-library-test-coverage-check/references/CHECKLIST.md +252 -252
- package/vendor/Server/Skills/arkts-library-test-coverage-check/references/MCP_TOOL_USAGE.md +114 -114
- package/vendor/Server/Skills/arkts-library-test-coverage-check/scripts/validate-skill.cjs +262 -262
- package/vendor/Server/Skills/arkts-library-xts-code/SKILL.md +455 -455
- package/vendor/Server/Skills/arkts-library-xts-code/assets/report-template.md +60 -60
- package/vendor/Server/Skills/arkts-library-xts-code/references/ACCESS_MODIFIER_PRECHECK.md +76 -76
- package/vendor/Server/Skills/arkts-library-xts-code/references/BUILD_VERIFICATION.md +200 -200
- package/vendor/Server/Skills/arkts-library-xts-code/references/CODE_STANDARDS.md +518 -518
- package/vendor/Server/Skills/arkts-library-xts-code/references/SUBAGENT_PROMPT_TEMPLATE.md +276 -276
- package/vendor/Server/Skills/arkts-library-xts-code/references/SYSTEM_API_LOOKUP.md +92 -92
- package/vendor/Server/Skills/arkts-library-xts-code/references/XTS_TEMPLATE.md +767 -767
- package/vendor/Server/Skills/arkts-library-xts-code/scripts/build.ps1 +93 -93
- package/vendor/Server/Skills/arkts-library-xts-code/scripts/code-linter.cjs +432 -432
- package/vendor/Server/Skills/arkts-library-xts-code/scripts/migrate.cjs +851 -851
- package/vendor/Server/Skills/arkts-library-xts-code/scripts/ohpm-switch.cjs +788 -788
- package/vendor/Server/Skills/arkts-library-xts-code/scripts/precheck-access-modifiers.cjs +221 -221
- package/vendor/Server/Skills/arkts-library-xts-code-fix/SKILL.md +228 -228
- package/vendor/Server/Skills/arkts-library-xts-coverage/SKILL.md +97 -97
- package/vendor/Server/Skills/arkts-library-xts-doc/SKILL.md +194 -194
- package/vendor/Server/Skills/arkts-library-xts-doc/references/EXAMPLES.md +575 -575
- package/vendor/Server/Skills/arkts-library-xts-doc/references/TEMPLATE.md +574 -574
- package/vendor/Server/Skills/arkts-library-xts-doc/references/TESTABILITY_GUIDE.md +182 -182
- package/vendor/Server/Skills/arkts-library-xts-execution/SKILL.md +490 -490
- package/vendor/Server/Skills/arkts-library-xts-execution/references/error-fixing-examples.md +217 -217
- package/vendor/Server/Skills/arkts-library-xts-execution/scripts/build.ps1 +93 -93
- package/vendor/Server/Skills/arkts-library-xts-execution/scripts/code-linter.cjs +432 -432
- package/vendor/Server/Skills/arkts-library-xts-execution/scripts/migrate.cjs +851 -851
- package/vendor/Server/Skills/arkts-library-xts-execution/scripts/ohpm-switch.cjs +788 -788
- package/vendor/Server/Skills/flutter-code-check/SKILL.md +299 -299
- package/vendor/Server/Skills/flutter-code-check/assets/OUTPUT_TEMPLATE.md +311 -311
- package/vendor/Server/Skills/flutter-code-check/references/CHECKLIST.md +190 -190
- package/vendor/Server/Skills/flutter-code-check/references/PLATFORMS.md +133 -133
- package/vendor/Server/Skills/flutter-code-check/scripts/validate-skill.cjs +259 -259
- package/vendor/Server/Skills/flutter-interface/README.md +75 -75
- package/vendor/Server/Skills/flutter-interface/SKILL.md +291 -291
- package/vendor/Server/Skills/flutter-interface/references/EXAMPLES.md +336 -336
- package/vendor/Server/Skills/flutter-interface/references/GIT_CLONE_GUIDE.md +190 -190
- package/vendor/Server/Skills/flutter-interface/references/SCRIPT_REFERENCE.md +258 -258
- package/vendor/Server/Skills/flutter-interface/references/TEMPLATE.md +254 -254
- package/vendor/Server/Skills/flutter-library-consistency-check/SKILL.md +517 -517
- package/vendor/Server/Skills/flutter-library-consistency-check/references/CHECKLIST.md +381 -381
- package/vendor/Server/Skills/flutter-library-consistency-check/references/FUNCTION_COMPARISON_GUIDE.md +341 -341
- package/vendor/Server/Skills/flutter-library-consistency-check/references/PLATFORM_LIMITATION_GUIDE.md +71 -71
- package/vendor/Server/Skills/flutter-library-documentation-quality-check/SKILL.md +518 -518
- package/vendor/Server/Skills/flutter-library-documentation-quality-check/references/CHECKLIST.md +406 -406
- package/vendor/Server/Skills/flutter-library-memory-leak-check/SKILL.md +652 -652
- package/vendor/Server/Skills/flutter-library-memory-leak-check/references/CHECKLIST.md +396 -396
- package/vendor/Server/Skills/flutter-library-stability-check/SKILL.md +744 -744
- package/vendor/Server/Skills/flutter-library-stability-check/references/CHECKLIST.md +720 -720
- package/vendor/Server/Skills/flutter-library-test-coverage-check/SKILL.md +414 -414
- package/vendor/Server/Skills/flutter-library-test-coverage-check/assets/REPORT_TEMPLATE.md +150 -150
- package/vendor/Server/Skills/flutter-library-test-coverage-check/references/CHECKLIST.md +304 -304
- package/vendor/Server/Skills/flutter-library-test-coverage-check/references/SCORING_RULES.md +265 -265
- package/vendor/Server/Skills/flutter-library-test-coverage-check/references/SCRIPT_USAGE.md +286 -286
- package/vendor/Server/Skills/knowledge-import/SKILL.md +383 -383
- package/vendor/Server/Skills/knowledge-import/references/MCP_TOOL_USAGE.md +165 -165
- package/vendor/Server/Skills/knowledge-import/references/SKILL_LOCATOR_GUIDE.md +91 -91
- package/vendor/Server/Skills/knowledge-import/references/SKILL_OPTIMIZATION_GUIDE.md +127 -127
- package/vendor/Server/Skills/ohos-lib-pr-push/SKILL.md +453 -301
- package/vendor/Server/Skills/ohos-lib-pr-push/assets/PR_CHECKLIST.md +151 -0
- package/vendor/Server/Skills/ohos-lib-pr-push/assets/PR_ISSUE_TEMPLATE.md +257 -105
- package/vendor/Server/Skills/ohos-lib-pr-push/references/GITCODE_API.md +191 -127
- package/vendor/Server/Skills/ohos-lib-pr-push/scripts/gitcode-api.cjs +317 -208
- package/vendor/Server/Skills/rn-code-check/SKILL.md +320 -320
- package/vendor/Server/Skills/rn-code-check/references/API-GUIDE.md +234 -234
- package/vendor/Server/Skills/rn-code-check/references/CHECKLIST.md +188 -188
- package/vendor/Server/Skills/rn-code-check/references/REPORT-TEMPLATE.md +246 -246
- package/vendor/Server/Skills/rn-lib-add-example/SKILL.md +198 -198
- package/vendor/Server/Skills/rn-lib-add-example/references/APP_RENAME_GUIDE.md +40 -40
- package/vendor/Server/Skills/rn-lib-add-example/references/SHELL_ADAPTATION_CHECKLIST.md +78 -78
- package/vendor/Server/Skills/rn-lib-add-example/references/TEMPLATE_REPO_GUIDE.md +70 -70
- package/vendor/Server/Skills/rn-lib-add-example/references/VERIFICATION_GUIDE.md +47 -47
- package/vendor/Server/Skills/rn-lib-add-example/scripts/create-example.cjs +252 -252
- package/vendor/Server/Skills/rn-lib-update-doc/SKILL.md +176 -176
- package/vendor/Server/Skills/rn-lib-update-doc/config.json +5 -5
- package/vendor/Server/Skills/rn-lib-update-doc/references/changelog-template.md +35 -35
- package/vendor/Server/Skills/rn-lib-update-doc/references/format-diff.md +137 -137
- package/vendor/Server/Skills/rn-lib-update-doc/references/link-registry.json +33 -33
- package/vendor/Server/Skills/rn-lib-update-doc/references/model-0.4.2-overview.md +40 -40
- package/vendor/Server/Skills/rn-lib-update-doc/references/model-0.4.2.md +429 -429
- package/vendor/Server/Skills/rn-lib-update-doc/scripts/detect-readme-format.cjs +105 -105
- package/vendor/Server/Skills/rn-lib-update-doc/scripts/find-usage-doc.cjs +90 -90
- package/vendor/Server/Skills/rn-lib-update-doc/scripts/link-verify.cjs +213 -213
- package/vendor/Server/Skills/rn-lib-update-doc/scripts/normalize-changelog.cjs +73 -73
- package/vendor/Server/Skills/rn-lib-update-doc/scripts/normalize-package-meta.cjs +258 -258
- package/vendor/Server/Skills/rn-lib-update-doc/scripts/normalize-readme-links.cjs +341 -341
- package/vendor/Server/Skills/rn-lib-update-doc/scripts/resolve-usage-docs.cjs +67 -67
- package/vendor/Server/Skills/rn-library-consistency-check/SKILL.md +299 -299
- package/vendor/Server/Skills/rn-library-consistency-check/assets/REPORT_TEMPLATE.md +293 -293
- package/vendor/Server/Skills/rn-library-consistency-check/references/ARKTS_COMPONENT_CHECK_GUIDE.md +857 -857
- package/vendor/Server/Skills/rn-library-consistency-check/references/ARKTS_TURBOMODULE_CHECK_GUIDE.md +856 -856
- package/vendor/Server/Skills/rn-library-consistency-check/references/CHECKLIST.md +157 -157
- package/vendor/Server/Skills/rn-library-consistency-check/references/CPP_COMPONENT_CHECK_GUIDE.md +801 -801
- package/vendor/Server/Skills/rn-library-consistency-check/references/CPP_TURBOMODULE_CHECK_GUIDE.md +683 -683
- package/vendor/Server/Skills/rn-library-consistency-check/references/FUNCTION_COMPARISON_GUIDE.md +233 -233
- package/vendor/Server/Skills/rn-library-consistency-check/references/NATIVE_COMPONENT_IDENTIFICATION.md +1373 -1373
- package/vendor/Server/Skills/rn-library-consistency-check/references/SCORING_STANDARD.md +220 -220
- package/vendor/Server/Skills/rn-library-consistency-check/scripts/validate-skill.py +245 -245
- package/vendor/Server/Skills/rn-library-documentation-quality-check/SKILL.md +263 -263
- package/vendor/Server/Skills/rn-library-documentation-quality-check/assets/MODEL.md +414 -414
- package/vendor/Server/Skills/rn-library-documentation-quality-check/assets/MODEL_en.md +413 -413
- package/vendor/Server/Skills/rn-library-documentation-quality-check/assets/REPORT_TEMPLATE.md +311 -311
- package/vendor/Server/Skills/rn-library-documentation-quality-check/references/CHECK_CHECKLIST.md +355 -355
- package/vendor/Server/Skills/rn-library-documentation-quality-check/references/CROSS_VALIDATION_GUIDE.md +115 -115
- package/vendor/Server/Skills/rn-library-documentation-quality-check/references/DOC_COLLECTION_GUIDE.md +139 -139
- package/vendor/Server/Skills/rn-library-documentation-quality-check/references/EXAMPLES.md +586 -586
- package/vendor/Server/Skills/rn-library-documentation-quality-check/references/GOTCHAS.md +57 -57
- package/vendor/Server/Skills/rn-library-documentation-quality-check/scripts/validate-skill.py +259 -259
- package/vendor/Server/Skills/rn-library-interface-analyzer/SKILL.md +301 -301
- package/vendor/Server/Skills/rn-library-interface-analyzer/assets/REPORT_TEMPLATE.md +148 -148
- package/vendor/Server/Skills/rn-library-interface-analyzer/references/DOC_ANNOTATION_GUIDE.md +118 -118
- package/vendor/Server/Skills/rn-library-interface-analyzer/references/SCRIPT_USAGE_GUIDE.md +168 -168
- package/vendor/Server/Skills/rn-library-interface-analyzer/references/URL_PARSING_RULES.md +176 -176
- package/vendor/Server/Skills/rn-library-issue-generalizer/SKILL.md +274 -274
- package/vendor/Server/Skills/rn-library-issue-generalizer/assets/case-template.md +45 -45
- package/vendor/Server/Skills/rn-library-issue-generalizer/assets/rule-template.md +25 -25
- package/vendor/Server/Skills/rn-library-issue-generalizer/references/PR_FETCH_GUIDE.md +61 -61
- package/vendor/Server/Skills/rn-library-issue-generalizer/references/RULE_TEMPLATE_GUIDE.md +165 -165
- package/vendor/Server/Skills/rn-library-issue-generalizer/scripts/fetch-pr-info.py +239 -239
- package/vendor/Server/Skills/rn-library-known-issue-check/SKILL.md +263 -263
- package/vendor/Server/Skills/rn-library-known-issue-check/assets/REPORT_TEMPLATE.md +96 -96
- package/vendor/Server/Skills/rn-library-known-issue-check/references/COMPLIANCE_CHECKLIST.md +57 -57
- package/vendor/Server/Skills/rn-library-known-issue-check/references/INSPECTION_GUIDE.md +99 -99
- package/vendor/Server/Skills/rn-library-known-issue-check/references/PRESCREEN_GUIDE.md +64 -64
- package/vendor/Server/Skills/rn-library-known-issue-check/references/SOURCE_PREP_GUIDE.md +74 -74
- package/vendor/Server/Skills/rn-library-known-issue-check/scripts/validate-known-issue-report.py +164 -164
- package/vendor/Server/Skills/rn-library-memory-leak-check/SKILL.md +356 -339
- package/vendor/Server/Skills/rn-library-memory-leak-check/assets/REPORT_TEMPLATE.md +85 -83
- package/vendor/Server/Skills/rn-library-memory-leak-check/references/CHECKLIST.md +905 -859
- package/vendor/Server/Skills/rn-library-memory-leak-check/references/HARMONY_CODE_PATTERNS.md +124 -122
- package/vendor/Server/Skills/rn-library-memory-leak-check/scripts/scan_harmony_code.py +287 -279
- package/vendor/Server/Skills/rn-library-stability-check/SKILL.md +273 -273
- package/vendor/Server/Skills/rn-library-stability-check/assets/REPORT_TEMPLATE.md +129 -129
- package/vendor/Server/Skills/rn-library-stability-check/references/CHECKLIST.md +581 -581
- package/vendor/Server/Skills/rn-library-stability-check/references/SCORING_RULES.md +138 -138
- package/vendor/Server/Skills/rn-library-stability-check/scripts/scan-harmony-code.py +168 -168
- package/vendor/Server/Skills/rn-library-test-coverage-check/SKILL.md +526 -506
- package/vendor/Server/Skills/rn-library-test-coverage-check/references/CHECKLIST.md +321 -280
- package/vendor/Server/Skills/rn-library-test-coverage-check/references/DEMO_LOCATION.md +221 -221
- package/vendor/Server/Skills/rn-library-test-coverage-check/references/REPORT_TEMPLATE.md +220 -209
- package/vendor/Server/Skills/rn-library-test-coverage-check/references/SCRIPT_USAGE.md +325 -325
- package/vendor/Server/Skills/rn-ohos-xts-generator/SKILL.md +279 -279
- package/vendor/Server/Skills/rn-ohos-xts-generator/assets/TEST_FILE_TEMPLATES.md +404 -404
- package/vendor/Server/Skills/rn-ohos-xts-generator/references/BUILD_TROUBLESHOOTING.md +185 -185
- package/vendor/Server/Skills/rn-ohos-xts-generator/references/BUILD_WORKFLOW.md +265 -265
- package/vendor/Server/Skills/rn-ohos-xts-generator/references/COMMON_CONFIG.md +189 -189
- package/vendor/Server/Skills/rn-ohos-xts-generator/references/CONFIG_GUIDE.md +203 -203
- package/vendor/Server/Skills/rn-ohos-xts-generator/references/COVERAGE_ANALYZER.md +147 -147
- package/vendor/Server/Skills/rn-ohos-xts-generator/references/DESIGN_DOC_GUIDE.md +148 -148
- package/vendor/Server/Skills/rn-ohos-xts-generator/references/GENERATION_FLOW.md +165 -165
- package/vendor/Server/Skills/rn-ohos-xts-generator/references/GENERATION_STRATEGY.md +195 -195
- package/vendor/Server/Skills/rn-ohos-xts-generator/references/HARMONY_SCANNER.md +249 -249
- package/vendor/Server/Skills/rn-ohos-xts-generator/references/JEST_FRAMEWORK.md +235 -235
- package/vendor/Server/Skills/rn-ohos-xts-generator/references/PLATFORM_COMPAT.md +161 -161
- package/vendor/Server/Skills/rn-ohos-xts-generator/references/RN_TEST_PATTERNS.md +413 -413
- package/vendor/Server/Skills/rn-ohos-xts-generator/references/SOURCE_PARSER.md +235 -235
- package/vendor/Server/Skills/rn-ohos-xts-generator/references/TEST_CONVENTIONS.md +245 -245
- package/vendor/Server/Skills/rn-ohos-xts-generator/references/TROUBLESHOOTING.md +282 -282
- package/vendor/Server/Skills/rn-ohos-xts-generator/references/USAGE_GUIDE.md +251 -251
- package/vendor/Server/Skills/rnoh-cptf-api-test/SKILL.md +469 -469
- package/vendor/Server/Skills/rnoh-cptf-api-test/assets/CPTF_TEST_TEMPLATE.md +310 -310
- package/vendor/Server/Skills/rnoh-cptf-api-test/assets/DEMO_PAGE_TEMPLATE.md +303 -303
- package/vendor/Server/Skills/rnoh-cptf-api-test/references/CPTF_TEST_GENERATION_GUIDE.md +421 -421
- package/vendor/Server/Skills/rnoh-cptf-api-test/references/DEMO_GENERATION_GUIDE.md +358 -358
- package/vendor/Server/Skills/rnoh-cptf-api-test/references/QUALITY_CHECKLIST.md +114 -114
- package/vendor/Server/Skills/rnoh-cptf-api-test/references/SPEC_PARSING_GUIDE.md +274 -274
- package/vendor/Server/Skills/rnoh-cptf-api-test/scripts/validate-rnoh-cptf-api-test.sh +185 -185
- package/vendor/Server/Skills/rnoh-cptf-demo-test/SKILL.md +191 -191
- package/vendor/Server/Skills/rnoh-cptf-demo-test/assets/TEST_TEMPLATE.md +216 -216
- package/vendor/Server/Skills/rnoh-cptf-demo-test/references/CPTF_SETUP.md +107 -107
- package/vendor/Server/Skills/rnoh-cptf-demo-test/references/DEMO_ANALYSIS.md +120 -120
- package/vendor/Server/Skills/rnoh-cptf-demo-test/scripts/validate-rnoh-cptf-test.sh +108 -108
- package/vendor/Server/Skills/rnoh-cptf-test-run/SKILL.md +237 -237
- package/vendor/Server/Skills/rnoh-cptf-test-run/references/DEBUG_PLAYBOOK.md +261 -261
- package/vendor/Server/Skills/rnoh-cptf-test-run/scripts/validate-run-env.sh +112 -112
- package/vendor/Server/Skills/rnoh-lib-code-adapt/SKILL.md +497 -497
- package/vendor/Server/Skills/rnoh-lib-code-adapt/references/adapt-checklist.md +270 -270
- package/vendor/Server/Skills/rnoh-lib-code-adapt/references/api-migration-checklist.md +164 -164
- package/vendor/Server/Skills/rnoh-lib-code-adapt/references/common-compile-errors.md +556 -556
- package/vendor/Server/Skills/rnoh-lib-code-adapt/references/deveco-cli-tools.md +459 -459
- package/vendor/Server/Skills/rnoh-lib-code-adapt/references/edge-cases.md +65 -65
- package/vendor/Server/Skills/rnoh-lib-code-adapt/references/har-tgz-build.md +512 -512
- package/vendor/Server/Skills/rnoh-lib-code-adapt/references/version-compat-matrix.md +132 -132
- package/vendor/Server/Skills/rnoh-lib-code-adapt/scripts/auto-integrate.sh +424 -424
- package/vendor/Server/Skills/rnoh-lib-demo-coverage/SKILL.md +137 -137
- package/vendor/Server/Skills/rnoh-lib-demo-create/SKILL.md +306 -306
- package/vendor/Server/Skills/rnoh-lib-demo-create/assets/DEMO_PAGE_TEMPLATE.md +485 -485
- package/vendor/Server/Skills/rnoh-lib-demo-create/assets/INDEX_TEMPLATE.md +172 -172
- package/vendor/Server/Skills/rnoh-lib-demo-create/references/DEMO_CODE_RULES.md +422 -422
- package/vendor/Server/Skills/rnoh-lib-demo-create/references/QUALITY_CHECKLIST.md +95 -95
- package/vendor/Server/Skills/rnoh-lib-demo-create/references/SOURCE_EXPLORATION.md +169 -169
- package/vendor/Server/Skills/rnoh-lib-demo-doc/SKILL.md +554 -554
- package/vendor/Server/Skills/rnoh-lib-demo-doc/assets/DEMO_DOC_TEMPLATE.md +106 -106
- package/vendor/Server/Skills/rnoh-lib-demo-doc/references/COVERAGE_INCREMENTAL.md +216 -216
- package/vendor/Server/Skills/rnoh-lib-demo-doc/references/DEMO_DESIGN_RULES.md +413 -413
- package/vendor/Server/Skills/rnoh-lib-demo-doc/references/OUTPUT_FORMAT.md +269 -269
- package/vendor/Server/Skills/rnoh-lib-demo-doc/references/QUALITY_CHECKLIST.md +98 -98
- package/vendor/Server/Skills/rnoh-lib-func-test/SKILL.md +337 -337
- package/vendor/Server/Skills/rnoh-lib-rnt-search/SKILL.md +722 -722
- package/vendor/Server/Skills/rnoh-lib-rnt-search/assets/PORT_REPORT_TEMPLATE.md +219 -219
- package/vendor/Server/Skills/rnoh-lib-rnt-search/references/DEMO_PORTING_GUIDE.md +345 -345
- package/vendor/Server/Skills/rnoh-lib-rnt-search/references/PORT_VERIFICATION.md +269 -269
- package/vendor/Server/Skills/rnoh-lib-rnt-search/references/RNT_PROJECT_STRUCTURE.md +173 -173
- package/vendor/Server/Skills/rnoh-lib-rnt-search/scripts/analyze-demo-deps.cjs +267 -267
- package/vendor/Server/Skills/rnoh-lib-rnt-search/scripts/detect-rnoh-version.cjs +174 -174
- package/vendor/Server/Skills/rnoh-lib-rnt-search/scripts/find-rnt-demo.sh +111 -111
- package/vendor/Server/Skills/rnoh-lib-rnt-search/scripts/port-demo.sh +186 -186
- package/vendor/Server/Skills/rnoh-lib-rnt-search/scripts/read-lib-meta.cjs +150 -150
- package/vendor/Server/Skills/rnoh-lib-test-align/SKILL.md +296 -296
- package/vendor/Server/Skills/rnoh-lib-test-align/references/jest-troubleshooting.md +371 -371
- package/vendor/Server/Skills/rnoh-lib-test-align/references/port-test-demo-guide.md +212 -212
- package/vendor/Server/Skills/rnoh-lib-test-align/references/runtime-troubleshooting.md +345 -345
- package/vendor/Server/Skills/rnoh-lib-test-align/references/unit-test-template.md +223 -223
- package/vendor/Server/Skills/rnoh-lib-test-align/scripts/port-test-demo.sh +1053 -1053
- package/vendor/Server/Skills/rnoh-lib-version-diff/SKILL.md +295 -255
- package/vendor/Server/Skills/rnoh-lib-version-diff/references/OHOS_ADAPTATION.md +82 -77
- package/vendor/Server/Skills/rnoh-lib-version-diff/references/OHOS_API_LOOKUP.md +224 -224
- package/vendor/Server/Skills/rnoh-lib-version-diff/references/script-reference.md +175 -174
- package/vendor/Server/Skills/rnoh-lib-version-diff/scripts/detect-upstream-version.js +967 -967
- package/vendor/Server/Skills/rnoh-lib-version-diff/scripts/generate-diff.js +1798 -1798
- package/vendor/Server/Skills/rnoh-lib-version-diff/scripts/package.json +5 -5
- package/vendor/Server/Skills/rnoh-lib-version-diff/templates/report-template.md +180 -137
- package/vendor/Server/Skills/rnoh-lib-xts-coverage/SKILL.md +145 -145
- package/vendor/Server/Skills/skill-generator/SKILL.md +365 -365
- package/vendor/Server/Skills/skill-generator/assets/SKILL_TEMPLATE.md +183 -183
- package/vendor/Server/Skills/skill-generator/references/COMPLIANCE_CHECKLIST.md +82 -82
- package/vendor/Server/Skills/skill-generator/references/KB_QUERY_GUIDE.md +110 -110
- package/vendor/Server/Skills/skill-generator/references/MCP_TOOL_DISCOVERY.md +122 -122
- package/vendor/Server/Skills/skill-generator/references/SKILL_SPEC.md +121 -121
- package/vendor/Server/Skills/skill-generator/scripts/validate-skill.cjs +259 -259
- package/vendor/Server/Skills/task-analysis/SKILL.md +234 -234
- package/vendor/Server/Skills/task-analysis/assets/OUTPUT_EXAMPLES.md +160 -160
- package/vendor/Server/Skills/task-analysis/references/ACCEPTANCE_CRITERIA_GUIDE.md +190 -190
- package/vendor/Server/Skills/task-analysis/references/ANALYSIS_SCHEMA.md +224 -224
- package/vendor/Server/Skills/task-analysis/references/INTEGRATION_CONTEXT.md +162 -162
- package/vendor/Server/Skills/task-analysis/references/LESSONS_LEARNED.md +55 -55
- package/vendor/Server/Skills/task-analysis/references/REPLAN_CONTEXT_GUIDE.md +126 -126
- package/vendor/Server/Skills/task-analysis/scripts/validate-analysis-output.sh +256 -256
- package/vendor/Server/Skills/task-decomposition/SKILL.md +528 -528
- package/vendor/Server/Skills/task-decomposition/assets/OUTPUT_EXAMPLES.md +451 -451
- package/vendor/Server/Skills/task-decomposition/references/ACCEPTANCE_CRITERIA_GUIDE.md +179 -179
- package/vendor/Server/Skills/task-decomposition/references/DEPENDENCY_GRAPH_GUIDE.md +152 -152
- package/vendor/Server/Skills/task-decomposition/references/INTEGRATION_CONTEXT.md +175 -175
- package/vendor/Server/Skills/task-decomposition/references/REPLAN_CONTEXT_GUIDE.md +195 -195
- package/vendor/Server/Skills/task-decomposition/references/TASK_DECOMPOSITION_TEMPLATES.md +226 -226
- package/vendor/Server/Skills/task-decomposition/scripts/list-available-skills.cjs +263 -263
- package/vendor/Server/Skills/task-decomposition/scripts/skill-blacklist.json +29 -29
- package/vendor/Server/Skills/task-decomposition/scripts/validate-decomposition-output.cjs +376 -376
- package/vendor/Server/Skills/task-decomposition/scripts/validate-param-consistency.cjs +259 -259
- package/vendor/Server/Skills/task-decomposition/scripts/validate-specified-skills.cjs +216 -216
- package/vendor/Server/Skills/task-decomposition/scripts/validate-step-graph.cjs +424 -424
- package/vendor/Server/Skills/tpc-cpp-check/SKILL.md +427 -427
- package/vendor/Server/Skills/tpc-cpp-check/references/CHECKLIST.md +214 -214
- package/vendor/Server/Skills/tpc-cpp-check/references/HPKBUILD_GUIDE.md +235 -235
- package/vendor/Server/Skills/tpc-cpp-check/references/REPORT_TEMPLATE.md +250 -250
- package/vendor/Server/TestCheck/TestCheck.js +775 -775
- package/vendor/Server/TestCheck/index.js +11 -11
- package/vendor/Server/library/McpToolGuide.js +280 -280
- package/vendor/Server/library/StatsService.js +126 -126
- package/vendor/Server/library/cstTime.js +16 -16
- package/vendor/VERSION +3 -3
- package/workflows/pr-check.mjs +105 -105
- package/workflows/pr-push.mjs +192 -192
- package/vendor/Server/Agent/SkillSelector/types.js +0 -63
|
@@ -1,299 +1,299 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: flutter-code-check
|
|
3
|
-
description: 对 Flutter(OpenHarmony 平台适配)三方库的 Pull Request 进行自动化代码检视,包括获取 PR diff、深度分析鸿蒙平台代码(ohos/ 目录 ArkTS/C++)、跨平台一致性对比、Dart 改动必要性审查,生成结构化检视报告与合入建议分级。Use when 需要检视 Flutter 三方库 PR、审查 ohos 平台适配代码、评估 Dart 改动对跨平台的影响。
|
|
4
|
-
license: Apache-2.0
|
|
5
|
-
compatibility: 需要 MCP Gateway 运行中(script_flutter_* 脚本工具 + gitcode_ 工具),需要 curl、git、jq 及访问 GitCode/GitHub/Gitee API 的网络权限
|
|
6
|
-
metadata:
|
|
7
|
-
version: "1.0.0"
|
|
8
|
-
category: code-check
|
|
9
|
-
language: Dart
|
|
10
|
-
---
|
|
11
|
-
|
|
12
|
-
# Flutter 代码检视
|
|
13
|
-
|
|
14
|
-
对 Flutter(OpenHarmony 平台适配)三方库的 Pull Request 进行自动化代码检视,重点关注鸿蒙平台侧实现代码(`ohos/` 目录)。
|
|
15
|
-
|
|
16
|
-
> ⚠️ **核心原则**:鸿蒙适配 PR 的本质是为已有 Flutter 库增加 ohos 平台实现。**Dart 代码(`lib/`)原则上不应修改**;若存在 Dart 变更,必须逐一论证必要性并评估对 android/iOS/Web 的影响。ohos 实现必须与 android/iOS 保持方法路由、参数类型、返回值、错误码一致。
|
|
17
|
-
|
|
18
|
-
---
|
|
19
|
-
|
|
20
|
-
## 任务参数
|
|
21
|
-
|
|
22
|
-
| 参数 | 类型 | 必填 | 说明 |
|
|
23
|
-
|------|------|------|------|
|
|
24
|
-
| `prUrl` | string | ✅ | PR 地址,如 `https://gitcode.com/openharmony-sig/flutter_image_picker/pulls/5` |
|
|
25
|
-
| `outputDir` | string | ✅ | 输出目录绝对路径,如 `/path/to/codecheck/owner_repo_pr5` |
|
|
26
|
-
| `accessToken` | string | ❌ | GitCode / Gitee Personal Access Token;GitCode API 必须携带 |
|
|
27
|
-
| `branch` | string | ❌ | 目标分支名;若提供则优先使用 |
|
|
28
|
-
|
|
29
|
-
---
|
|
30
|
-
|
|
31
|
-
## 输入依赖
|
|
32
|
-
|
|
33
|
-
- **前置 SKILL**: 无(独立任务)
|
|
34
|
-
- **输入文件**: 无(从 `prUrl` 实时获取)
|
|
35
|
-
- **仓库源码**: 检视过程中克隆到 `{outputDir}/source/`
|
|
36
|
-
|
|
37
|
-
---
|
|
38
|
-
|
|
39
|
-
## 工作流程概览
|
|
40
|
-
|
|
41
|
-
```
|
|
42
|
-
Phase 1: 信息收集(解析 PR + 获取元数据 + 下载 patch + 克隆仓库)
|
|
43
|
-
↓
|
|
44
|
-
Phase 2: 差异分析(按路径分类 + 确定优先级)
|
|
45
|
-
↓
|
|
46
|
-
Phase 3: 代码检视(ArkTS/C++/Dart 多维度 + 跨平台一致性 + Dart 必要性)
|
|
47
|
-
↓
|
|
48
|
-
Phase 4: 影响分析(功能/兼容/性能/安全)
|
|
49
|
-
↓
|
|
50
|
-
Phase 5: 生成报告(check-report.md + changed-files.txt + result.txt)
|
|
51
|
-
```
|
|
52
|
-
|
|
53
|
-
---
|
|
54
|
-
|
|
55
|
-
## 工作流程
|
|
56
|
-
|
|
57
|
-
### Phase 1:信息收集
|
|
58
|
-
|
|
59
|
-
#### 1.1 解析 PR 地址
|
|
60
|
-
|
|
61
|
-
从 `prUrl` 提取:`platform`、`owner`、`repo`、`prNumber`。
|
|
62
|
-
|
|
63
|
-
| 平台 | URL 模式 | API 支持 |
|
|
64
|
-
|------|----------|----------|
|
|
65
|
-
| GitCode | `https://gitcode.com/{owner}/{repo}/pulls/{num}` | ✅ 优先 REST API |
|
|
66
|
-
| GitHub | `https://github.com/{owner}/{repo}/pull/{num}` | ❌ patch 下载 |
|
|
67
|
-
| Gitee | `https://gitee.com/{owner}/{repo}/pulls/{num}` | 部分 V5 API |
|
|
68
|
-
| GitLab | `https://gitlab.com/{owner}/{repo}/-/merge_requests/{num}` | ❌ patch 下载 |
|
|
69
|
-
|
|
70
|
-
#### 1.2 获取 PR 元数据
|
|
71
|
-
|
|
72
|
-
```bash
|
|
73
|
-
curl -s "https://api.gitcode.com/api/v5/repos/{owner}/{repo}/pulls/{prNumber}?access_token={accessToken}"
|
|
74
|
-
```
|
|
75
|
-
|
|
76
|
-
提取:`title`、`base.ref`、`head.sha`、`user.name`、`created_at`、`state`
|
|
77
|
-
|
|
78
|
-
#### 1.3 获取 Commit Patch
|
|
79
|
-
|
|
80
|
-
```bash
|
|
81
|
-
curl -s "https://api.gitcode.com/api/v5/repos/{owner}/{repo}/commit/{head.sha}/patch?access_token={accessToken}" -o {outputDir}/pr.patch
|
|
82
|
-
```
|
|
83
|
-
|
|
84
|
-
返回标准 unified diff 格式,包含完整 commit 信息和文件变更。
|
|
85
|
-
|
|
86
|
-
> **注意**:路径使用 `commit`(单数),不是 `commits`
|
|
87
|
-
|
|
88
|
-
#### 1.4 克隆仓库
|
|
89
|
-
|
|
90
|
-
**必须克隆**以获取完整源码上下文:
|
|
91
|
-
|
|
92
|
-
```bash
|
|
93
|
-
git clone --depth=50 --branch {targetBranch} https://{platform}/{owner}/{repo}.git {outputDir}/source/
|
|
94
|
-
```
|
|
95
|
-
|
|
96
|
-
若分支克隆失败,去掉 `--branch` 克隆默认分支。
|
|
97
|
-
|
|
98
|
-
---
|
|
99
|
-
|
|
100
|
-
### Phase 2:差异分析
|
|
101
|
-
|
|
102
|
-
按路径分类修改文件,确定检视优先级:
|
|
103
|
-
|
|
104
|
-
| 路径 | 类型 | 优先级 |
|
|
105
|
-
|------|------|--------|
|
|
106
|
-
| `ohos/src/main/ets/**/*.ets` | ArkTS Plugin | **最高** |
|
|
107
|
-
| `ohos/src/main/cpp/**` | C++ Native | **高** |
|
|
108
|
-
| `ohos/CMakeLists.txt` | 构建配置 | 中 |
|
|
109
|
-
| `ohos/oh-package.json5` | 包依赖 | 中 |
|
|
110
|
-
| `lib/**/*.dart` | Dart 接口层 | 中(需必要性审查) |
|
|
111
|
-
| `example/**` | 示例 | 低 |
|
|
112
|
-
|
|
113
|
-
统计改动规模,从 PR 标题和 commit message 理解改动意图。
|
|
114
|
-
|
|
115
|
-
> **Dart 改动特别关注**:鸿蒙适配 PR 的核心原则是「Dart 代码不应修改,或尽量少修改」。若 PR 包含 `lib/` 目录的 Dart 代码变更,必须进行必要性分析(详见 Phase 3.3)。
|
|
116
|
-
|
|
117
|
-
---
|
|
118
|
-
|
|
119
|
-
### Phase 3:代码检视
|
|
120
|
-
|
|
121
|
-
#### 3.1 多维度检视
|
|
122
|
-
|
|
123
|
-
按以下维度检视,详细检查项参见 [检视清单](references/CHECKLIST.md):
|
|
124
|
-
|
|
125
|
-
| 维度 | 关键点 |
|
|
126
|
-
|------|--------|
|
|
127
|
-
| 非兼容变更 | API 删除/重命名、返回格式变更 |
|
|
128
|
-
| Flutter Plugin 协议 | 生命周期方法、`result.success()` 后必须 `return` |
|
|
129
|
-
| **Dart 改动必要性** | **Dart 代码原则上不应修改,每处变更必须论证必要性,评估对其他平台的影响** |
|
|
130
|
-
| **跨平台一致性** | **ohos 实现与 android/iOS 的方法路由、参数类型、返回值、错误码对比** |
|
|
131
|
-
| ArkTS 约束 | 禁止 `any`、`eval()`、`prototype` |
|
|
132
|
-
| OH API 兼容 | `minAPIVersion`、权限申请 |
|
|
133
|
-
| 异步处理 | `await` + `try/catch`、Promise rejection |
|
|
134
|
-
| 事件监听器 | `on`/`off` 对称性 |
|
|
135
|
-
| C++ Native | NAPI 内存管理、跨线程安全 |
|
|
136
|
-
| 错误处理 | `result.error()` 后 `return`、switch `break` |
|
|
137
|
-
| 安全性 | 无硬编码敏感信息、HTTPS 校验 |
|
|
138
|
-
|
|
139
|
-
#### 3.2 跨平台一致性对比(重点维度)
|
|
140
|
-
|
|
141
|
-
> PR 中仅包含鸿蒙代码提交,需从克隆的仓库中读取 `android/` 和 `ios/` 的已有实现,与 ohos 变更进行对比。
|
|
142
|
-
|
|
143
|
-
**对比方法**:在克隆的 `{outputDir}/source/` 中定位 android/iOS 的 Plugin 实现文件,逐方法对比。
|
|
144
|
-
|
|
145
|
-
| 对比项 | 说明 | 问题等级 |
|
|
146
|
-
|--------|------|----------|
|
|
147
|
-
| 方法路由一致性 | ohos `onMethodCall` 的 case 分支是否与 android `onMethodCall` 完全对应,缺失方法 = 功能缺失 | 🔴 Critical |
|
|
148
|
-
| 参数类型映射一致性 | 同一 MethodChannel 方法,ohos 解析 `call.argument` 的类型是否与 android 一致(如 `bool` vs `string`) | 🔴 Critical |
|
|
149
|
-
| 返回值格式一致性 | `result.success()` 返回的数据结构是否与 android 对齐 | 🟠 Major |
|
|
150
|
-
| 错误码一致性 | `result.error()` 的 code/message 是否与 android 对齐 | 🟠 Major |
|
|
151
|
-
| MethodChannel 名称一致性 | ohos 注册的 Channel 名是否与 android 注册名一致(Dart 侧共用同一 Channel) | 🔴 Critical |
|
|
152
|
-
| 功能实现完整性 | android 有实现但 ohos 为 Stub(返回固定值无实际操作)或完全缺失 | 🔴 Critical |
|
|
153
|
-
| EventChannel 对比 | EventChannel 事件名和数据格式跨平台一致 | 🟠 Major |
|
|
154
|
-
|
|
155
|
-
**对比操作步骤**:
|
|
156
|
-
|
|
157
|
-
1. 从 PR diff 中提取 ohos 变更涉及的所有 `case 'methodName'` 分支
|
|
158
|
-
2. 在 `{outputDir}/source/android/` 中定位对应的 Plugin 实现文件(通常为 `*Plugin.java` 或 `*Plugin.kt`)
|
|
159
|
-
3. 逐方法对比:方法是否存在 → 参数类型是否一致 → 返回值格式是否一致 → 错误处理是否对齐
|
|
160
|
-
4. 对于 EventChannel,对比事件名和事件数据格式
|
|
161
|
-
5. 将对比结果记录到报告的"跨平台一致性"章节
|
|
162
|
-
|
|
163
|
-
---
|
|
164
|
-
|
|
165
|
-
#### 3.3 Dart 改动必要性分析(重点维度)
|
|
166
|
-
|
|
167
|
-
> **核心原则**:鸿蒙适配 PR 的本质是为已有 Flutter 三方库增加 ohos 平台实现,Dart 代码(`lib/` 目录)理论上不应修改。若 PR 包含 Dart 代码变更,必须逐一论证其必要性,并评估对 android/iOS 平台的影响。
|
|
168
|
-
|
|
169
|
-
**分析步骤**:
|
|
170
|
-
|
|
171
|
-
1. **识别所有 Dart 变更**:从 PR diff 中提取 `lib/**/*.dart` 的所有修改点
|
|
172
|
-
2. **逐项分类**:将每处 Dart 变更归入以下类别
|
|
173
|
-
|
|
174
|
-
| 变更类别 | 说明 | 问题等级 | 示例 |
|
|
175
|
-
|----------|------|----------|------|
|
|
176
|
-
| ✅ 必要改动 | 鸿蒙适配必须的改动,无其他实现方式 | 无问题(需记录) | 添加 `Platform.isOHOS` 平台判断以注册 ohos Plugin |
|
|
177
|
-
| ⚠️ 有替代方案 | 可通过 ohos 侧实现避免,不应修改 Dart | 🟠 Major | 修改 `MethodChannel` 方法名以匹配 ohos 实现(应在 ohos 侧适配) |
|
|
178
|
-
| ❌ 不应修改 | 改变已有 API 行为或影响其他平台 | 🔴 Critical | 修改 `invokeMethod` 的参数结构、修改返回值解析逻辑 |
|
|
179
|
-
| 🔍 需评估 | 涉及平台条件分支,需判断影响范围 | 🟡 Minor | 新增 `if (Platform.isOHOS)` 条件分支 |
|
|
180
|
-
|
|
181
|
-
3. **跨平台影响评估**:对每处 Dart 变更,验证以下影响
|
|
182
|
-
|
|
183
|
-
| 影响项 | 检查方法 | 问题等级 |
|
|
184
|
-
|--------|----------|----------|
|
|
185
|
-
| android/iOS 运行时行为是否改变 | 检查变更是否在 `Platform.isOHOS` 条件内 | 🔴 Critical(若影响) |
|
|
186
|
-
| MethodChannel 协议是否破坏 | Dart 侧 `invokeMethod` 参数是否与 android 侧仍兼容 | 🔴 Critical(若破坏) |
|
|
187
|
-
| 公共 API 签名是否变更 | `pubspec.yaml` 版本号是否对应升 major | 🟠 Major(若未升版本) |
|
|
188
|
-
| Platform Interface 默认实现是否受影响 | 默认实现的行为是否被改变 | 🟠 Major(若改变) |
|
|
189
|
-
|
|
190
|
-
4. **Dart 变更替代方案建议**:对于每处「有替代方案」或「不应修改」的变更,给出 ohos 侧的替代实现建议
|
|
191
|
-
|
|
192
|
-
**常见 Dart 改动模式与判定**:
|
|
193
|
-
|
|
194
|
-
| Dart 改动模式 | 判定 | 说明 |
|
|
195
|
-
|--------------|------|------|
|
|
196
|
-
| 新增 `if (Platform.isOHOS)` 条件分支 | ✅ 必要 | 鸿蒙平台需要独立的初始化或注册逻辑 |
|
|
197
|
-
| 新增 `TargetPlatform.ohos` case | ✅ 必要 | 平台枚举扩展 |
|
|
198
|
-
| 修改 `MethodChannel` 名称 | ❌ 不应修改 | 应在 ohos 侧注册与 android 相同的 Channel 名 |
|
|
199
|
-
| 修改 `invokeMethod` 的参数 | ❌ 不应修改 | 应在 ohos 侧适配参数解析 |
|
|
200
|
-
| 修改 `result` 返回值的解析逻辑 | ❌ 不应修改 | 应在 ohos 侧返回与 android 相同格式 |
|
|
201
|
-
| 新增 `import 'dart:io'`(原无) | ⚠️ 需评估 | 若仅用于 `Platform.isOHOS` 可接受,但可能影响 Web 平台 |
|
|
202
|
-
| 修改 Platform Interface 方法签名 | ❌ 不应修改 | 应通过 ohos Plugin 实现来适配 |
|
|
203
|
-
| 新增公共 API 方法 | ⚠️ 需评估 | 若仅 ohos 使用,应考虑是否为通用需求 |
|
|
204
|
-
| 修改 `pubspec.yaml` 依赖 | 🔍 需评估 | 新增依赖是否为 ohos 适配必要 |
|
|
205
|
-
| 删除或注释已有代码 | ❌ 不应修改 | 鸿蒙适配不应删除已有功能代码 |
|
|
206
|
-
|
|
207
|
-
---
|
|
208
|
-
|
|
209
|
-
### Phase 4:影响分析
|
|
210
|
-
|
|
211
|
-
- **功能影响**:是否引入新功能或修改行为
|
|
212
|
-
- **兼容性影响**:Plugin API 是否向后兼容
|
|
213
|
-
- **性能影响**:异步处理是否改善
|
|
214
|
-
- **安全影响**:是否修复或引入安全问题
|
|
215
|
-
|
|
216
|
-
---
|
|
217
|
-
|
|
218
|
-
### Phase 5:生成报告
|
|
219
|
-
|
|
220
|
-
输出文件:
|
|
221
|
-
- `{outputDir}/check-report.md` — 检视报告
|
|
222
|
-
- `{outputDir}/changed-files.txt` — 修改文件列表
|
|
223
|
-
- `{outputDir}/result.txt` — 合入建议分级结果
|
|
224
|
-
|
|
225
|
-
> 报告格式参见 [输出模板](assets/OUTPUT_TEMPLATE.md)
|
|
226
|
-
|
|
227
|
-
### 5.1 合入建议分级标准
|
|
228
|
-
|
|
229
|
-
根据代码检视结果,将合入建议分为四个等级:
|
|
230
|
-
|
|
231
|
-
| 等级 | 标识 | 颜色建议 | 判断标准 | 举例场景 |
|
|
232
|
-
|------|------|----------|----------|----------|
|
|
233
|
-
| **建议合入** | ✅ `PASS` | 绿色 | 无或极少(≤2 Minor)修改引入问题,改动风险低,功能验证充分 | 纯文档更新、配置优化、已验证的 Bug 修复 |
|
|
234
|
-
| **条件合入** | ⚠️ `CONDITIONAL` | 黄色 | 存在少量(≤3 Major)可修复问题,在特定条件下可合入 | 需补充测试用例、需调整文档说明、非核心路径兼容性问题 |
|
|
235
|
-
| **修改后合入** | 🔧 `NEED_FIX` | 橙色 | 存在较多问题(≥1 Critical 或 ≥4 Major),需修复后方可合入 | Plugin API 兼容性破坏、内存泄漏、`result.success()` 后未 return |
|
|
236
|
-
| **不允许合入** | ❌ `REJECT` | 红色 | 存在严重问题(≥2 Critical),影响功能/安全/稳定性,不建议合入 | 安全漏洞、严重 NAPI 内存问题、未验证的重大架构变更 |
|
|
237
|
-
|
|
238
|
-
**分级判断流程**:
|
|
239
|
-
|
|
240
|
-
```
|
|
241
|
-
统计问题数量:
|
|
242
|
-
- Critical 级别问题数
|
|
243
|
-
- Major 级别问题数
|
|
244
|
-
- Minor 级别问题数
|
|
245
|
-
|
|
246
|
-
判断逻辑:
|
|
247
|
-
IF Critical ≥ 2 → ❌ REJECT
|
|
248
|
-
IF Critical ≥ 1 → 🔧 NEED_FIX
|
|
249
|
-
IF Major ≥ 4 → 🔧 NEED_FIX
|
|
250
|
-
IF Major ≥ 1 → ⚠️ CONDITIONAL
|
|
251
|
-
IF Minor ≤ 2 → ✅ PASS
|
|
252
|
-
ELSE → ⚠️ CONDITIONAL
|
|
253
|
-
```
|
|
254
|
-
|
|
255
|
-
### 5.2 result.txt 文件格式
|
|
256
|
-
|
|
257
|
-
`result.txt` 用于存储分级结果和统计信息,格式详见 [输出模板](assets/OUTPUT_TEMPLATE.md) 的 result.txt 生成规范。
|
|
258
|
-
|
|
259
|
-
### 5.3 结构化指标输出(必须执行)
|
|
260
|
-
|
|
261
|
-
执行环境会在任务 prompt 中下发一个**结构化指标文件输出路径**(`stepId{X}_loop{L}_meta_{N}.json`)。
|
|
262
|
-
检视完成后,**必须**将分级结果以下述 JSON 对象写入该路径:
|
|
263
|
-
|
|
264
|
-
```json
|
|
265
|
-
{
|
|
266
|
-
"mergeLevel": "PASS | CONDITIONAL | NEED_FIX | REJECT",
|
|
267
|
-
"critical": 0,
|
|
268
|
-
"major": 0,
|
|
269
|
-
"minor": 0
|
|
270
|
-
}
|
|
271
|
-
```
|
|
272
|
-
|
|
273
|
-
| 字段 | 类型 | 说明 |
|
|
274
|
-
|------|------|------|
|
|
275
|
-
| `mergeLevel` | string | 合入等级:PASS / CONDITIONAL / NEED_FIX / REJECT |
|
|
276
|
-
| `critical` | number | Critical 级别问题数 |
|
|
277
|
-
| `major` | number | Major 级别问题数 |
|
|
278
|
-
| `minor` | number | Minor 级别问题数 |
|
|
279
|
-
|
|
280
|
-
**说明**:该文件由执行框架自动收割入库(前端/历史列表直接读取,无需解析报告文本);详细的统计与理由已在 `result.txt` 中完整记录。若 prompt 未下发该路径则跳过本节。
|
|
281
|
-
|
|
282
|
-
---
|
|
283
|
-
|
|
284
|
-
## 注意事项
|
|
285
|
-
|
|
286
|
-
1. 若 PR 只修改 `android/` 或 `ios/`,无鸿蒙改动 → 注明并 APPROVED
|
|
287
|
-
2. 报告文件名必须是 `check-report.md`
|
|
288
|
-
3. **`result.success()` / `result.error()` 后必须 `return`** — 最常见严重问题
|
|
289
|
-
4. 若同时修改 `ohos/` 和 `lib/` → 交叉验证方法名和参数类型
|
|
290
|
-
5. **Dart 代码修改原则**:鸿蒙适配 PR 中 Dart 代码不应修改;若存在 Dart 变更,必须在报告中单独论证每处变更的必要性和跨平台影响
|
|
291
|
-
6. Dart 侧修改 `MethodChannel` 方法名/参数/返回值格式 → 应改为在 ohos 侧适配,而非修改 Dart 代码
|
|
292
|
-
|
|
293
|
-
---
|
|
294
|
-
|
|
295
|
-
## 参考文档
|
|
296
|
-
|
|
297
|
-
- [检视清单 CHECKLIST.md](references/CHECKLIST.md)
|
|
298
|
-
- [平台 API PLATFORMS.md](references/PLATFORMS.md)
|
|
299
|
-
- [输出模板 OUTPUT_TEMPLATE.md](assets/OUTPUT_TEMPLATE.md)
|
|
1
|
+
---
|
|
2
|
+
name: flutter-code-check
|
|
3
|
+
description: 对 Flutter(OpenHarmony 平台适配)三方库的 Pull Request 进行自动化代码检视,包括获取 PR diff、深度分析鸿蒙平台代码(ohos/ 目录 ArkTS/C++)、跨平台一致性对比、Dart 改动必要性审查,生成结构化检视报告与合入建议分级。Use when 需要检视 Flutter 三方库 PR、审查 ohos 平台适配代码、评估 Dart 改动对跨平台的影响。
|
|
4
|
+
license: Apache-2.0
|
|
5
|
+
compatibility: 需要 MCP Gateway 运行中(script_flutter_* 脚本工具 + gitcode_ 工具),需要 curl、git、jq 及访问 GitCode/GitHub/Gitee API 的网络权限
|
|
6
|
+
metadata:
|
|
7
|
+
version: "1.0.0"
|
|
8
|
+
category: code-check
|
|
9
|
+
language: Dart
|
|
10
|
+
---
|
|
11
|
+
|
|
12
|
+
# Flutter 代码检视
|
|
13
|
+
|
|
14
|
+
对 Flutter(OpenHarmony 平台适配)三方库的 Pull Request 进行自动化代码检视,重点关注鸿蒙平台侧实现代码(`ohos/` 目录)。
|
|
15
|
+
|
|
16
|
+
> ⚠️ **核心原则**:鸿蒙适配 PR 的本质是为已有 Flutter 库增加 ohos 平台实现。**Dart 代码(`lib/`)原则上不应修改**;若存在 Dart 变更,必须逐一论证必要性并评估对 android/iOS/Web 的影响。ohos 实现必须与 android/iOS 保持方法路由、参数类型、返回值、错误码一致。
|
|
17
|
+
|
|
18
|
+
---
|
|
19
|
+
|
|
20
|
+
## 任务参数
|
|
21
|
+
|
|
22
|
+
| 参数 | 类型 | 必填 | 说明 |
|
|
23
|
+
|------|------|------|------|
|
|
24
|
+
| `prUrl` | string | ✅ | PR 地址,如 `https://gitcode.com/openharmony-sig/flutter_image_picker/pulls/5` |
|
|
25
|
+
| `outputDir` | string | ✅ | 输出目录绝对路径,如 `/path/to/codecheck/owner_repo_pr5` |
|
|
26
|
+
| `accessToken` | string | ❌ | GitCode / Gitee Personal Access Token;GitCode API 必须携带 |
|
|
27
|
+
| `branch` | string | ❌ | 目标分支名;若提供则优先使用 |
|
|
28
|
+
|
|
29
|
+
---
|
|
30
|
+
|
|
31
|
+
## 输入依赖
|
|
32
|
+
|
|
33
|
+
- **前置 SKILL**: 无(独立任务)
|
|
34
|
+
- **输入文件**: 无(从 `prUrl` 实时获取)
|
|
35
|
+
- **仓库源码**: 检视过程中克隆到 `{outputDir}/source/`
|
|
36
|
+
|
|
37
|
+
---
|
|
38
|
+
|
|
39
|
+
## 工作流程概览
|
|
40
|
+
|
|
41
|
+
```
|
|
42
|
+
Phase 1: 信息收集(解析 PR + 获取元数据 + 下载 patch + 克隆仓库)
|
|
43
|
+
↓
|
|
44
|
+
Phase 2: 差异分析(按路径分类 + 确定优先级)
|
|
45
|
+
↓
|
|
46
|
+
Phase 3: 代码检视(ArkTS/C++/Dart 多维度 + 跨平台一致性 + Dart 必要性)
|
|
47
|
+
↓
|
|
48
|
+
Phase 4: 影响分析(功能/兼容/性能/安全)
|
|
49
|
+
↓
|
|
50
|
+
Phase 5: 生成报告(check-report.md + changed-files.txt + result.txt)
|
|
51
|
+
```
|
|
52
|
+
|
|
53
|
+
---
|
|
54
|
+
|
|
55
|
+
## 工作流程
|
|
56
|
+
|
|
57
|
+
### Phase 1:信息收集
|
|
58
|
+
|
|
59
|
+
#### 1.1 解析 PR 地址
|
|
60
|
+
|
|
61
|
+
从 `prUrl` 提取:`platform`、`owner`、`repo`、`prNumber`。
|
|
62
|
+
|
|
63
|
+
| 平台 | URL 模式 | API 支持 |
|
|
64
|
+
|------|----------|----------|
|
|
65
|
+
| GitCode | `https://gitcode.com/{owner}/{repo}/pulls/{num}` | ✅ 优先 REST API |
|
|
66
|
+
| GitHub | `https://github.com/{owner}/{repo}/pull/{num}` | ❌ patch 下载 |
|
|
67
|
+
| Gitee | `https://gitee.com/{owner}/{repo}/pulls/{num}` | 部分 V5 API |
|
|
68
|
+
| GitLab | `https://gitlab.com/{owner}/{repo}/-/merge_requests/{num}` | ❌ patch 下载 |
|
|
69
|
+
|
|
70
|
+
#### 1.2 获取 PR 元数据
|
|
71
|
+
|
|
72
|
+
```bash
|
|
73
|
+
curl -s "https://api.gitcode.com/api/v5/repos/{owner}/{repo}/pulls/{prNumber}?access_token={accessToken}"
|
|
74
|
+
```
|
|
75
|
+
|
|
76
|
+
提取:`title`、`base.ref`、`head.sha`、`user.name`、`created_at`、`state`
|
|
77
|
+
|
|
78
|
+
#### 1.3 获取 Commit Patch
|
|
79
|
+
|
|
80
|
+
```bash
|
|
81
|
+
curl -s "https://api.gitcode.com/api/v5/repos/{owner}/{repo}/commit/{head.sha}/patch?access_token={accessToken}" -o {outputDir}/pr.patch
|
|
82
|
+
```
|
|
83
|
+
|
|
84
|
+
返回标准 unified diff 格式,包含完整 commit 信息和文件变更。
|
|
85
|
+
|
|
86
|
+
> **注意**:路径使用 `commit`(单数),不是 `commits`
|
|
87
|
+
|
|
88
|
+
#### 1.4 克隆仓库
|
|
89
|
+
|
|
90
|
+
**必须克隆**以获取完整源码上下文:
|
|
91
|
+
|
|
92
|
+
```bash
|
|
93
|
+
git clone --depth=50 --branch {targetBranch} https://{platform}/{owner}/{repo}.git {outputDir}/source/
|
|
94
|
+
```
|
|
95
|
+
|
|
96
|
+
若分支克隆失败,去掉 `--branch` 克隆默认分支。
|
|
97
|
+
|
|
98
|
+
---
|
|
99
|
+
|
|
100
|
+
### Phase 2:差异分析
|
|
101
|
+
|
|
102
|
+
按路径分类修改文件,确定检视优先级:
|
|
103
|
+
|
|
104
|
+
| 路径 | 类型 | 优先级 |
|
|
105
|
+
|------|------|--------|
|
|
106
|
+
| `ohos/src/main/ets/**/*.ets` | ArkTS Plugin | **最高** |
|
|
107
|
+
| `ohos/src/main/cpp/**` | C++ Native | **高** |
|
|
108
|
+
| `ohos/CMakeLists.txt` | 构建配置 | 中 |
|
|
109
|
+
| `ohos/oh-package.json5` | 包依赖 | 中 |
|
|
110
|
+
| `lib/**/*.dart` | Dart 接口层 | 中(需必要性审查) |
|
|
111
|
+
| `example/**` | 示例 | 低 |
|
|
112
|
+
|
|
113
|
+
统计改动规模,从 PR 标题和 commit message 理解改动意图。
|
|
114
|
+
|
|
115
|
+
> **Dart 改动特别关注**:鸿蒙适配 PR 的核心原则是「Dart 代码不应修改,或尽量少修改」。若 PR 包含 `lib/` 目录的 Dart 代码变更,必须进行必要性分析(详见 Phase 3.3)。
|
|
116
|
+
|
|
117
|
+
---
|
|
118
|
+
|
|
119
|
+
### Phase 3:代码检视
|
|
120
|
+
|
|
121
|
+
#### 3.1 多维度检视
|
|
122
|
+
|
|
123
|
+
按以下维度检视,详细检查项参见 [检视清单](references/CHECKLIST.md):
|
|
124
|
+
|
|
125
|
+
| 维度 | 关键点 |
|
|
126
|
+
|------|--------|
|
|
127
|
+
| 非兼容变更 | API 删除/重命名、返回格式变更 |
|
|
128
|
+
| Flutter Plugin 协议 | 生命周期方法、`result.success()` 后必须 `return` |
|
|
129
|
+
| **Dart 改动必要性** | **Dart 代码原则上不应修改,每处变更必须论证必要性,评估对其他平台的影响** |
|
|
130
|
+
| **跨平台一致性** | **ohos 实现与 android/iOS 的方法路由、参数类型、返回值、错误码对比** |
|
|
131
|
+
| ArkTS 约束 | 禁止 `any`、`eval()`、`prototype` |
|
|
132
|
+
| OH API 兼容 | `minAPIVersion`、权限申请 |
|
|
133
|
+
| 异步处理 | `await` + `try/catch`、Promise rejection |
|
|
134
|
+
| 事件监听器 | `on`/`off` 对称性 |
|
|
135
|
+
| C++ Native | NAPI 内存管理、跨线程安全 |
|
|
136
|
+
| 错误处理 | `result.error()` 后 `return`、switch `break` |
|
|
137
|
+
| 安全性 | 无硬编码敏感信息、HTTPS 校验 |
|
|
138
|
+
|
|
139
|
+
#### 3.2 跨平台一致性对比(重点维度)
|
|
140
|
+
|
|
141
|
+
> PR 中仅包含鸿蒙代码提交,需从克隆的仓库中读取 `android/` 和 `ios/` 的已有实现,与 ohos 变更进行对比。
|
|
142
|
+
|
|
143
|
+
**对比方法**:在克隆的 `{outputDir}/source/` 中定位 android/iOS 的 Plugin 实现文件,逐方法对比。
|
|
144
|
+
|
|
145
|
+
| 对比项 | 说明 | 问题等级 |
|
|
146
|
+
|--------|------|----------|
|
|
147
|
+
| 方法路由一致性 | ohos `onMethodCall` 的 case 分支是否与 android `onMethodCall` 完全对应,缺失方法 = 功能缺失 | 🔴 Critical |
|
|
148
|
+
| 参数类型映射一致性 | 同一 MethodChannel 方法,ohos 解析 `call.argument` 的类型是否与 android 一致(如 `bool` vs `string`) | 🔴 Critical |
|
|
149
|
+
| 返回值格式一致性 | `result.success()` 返回的数据结构是否与 android 对齐 | 🟠 Major |
|
|
150
|
+
| 错误码一致性 | `result.error()` 的 code/message 是否与 android 对齐 | 🟠 Major |
|
|
151
|
+
| MethodChannel 名称一致性 | ohos 注册的 Channel 名是否与 android 注册名一致(Dart 侧共用同一 Channel) | 🔴 Critical |
|
|
152
|
+
| 功能实现完整性 | android 有实现但 ohos 为 Stub(返回固定值无实际操作)或完全缺失 | 🔴 Critical |
|
|
153
|
+
| EventChannel 对比 | EventChannel 事件名和数据格式跨平台一致 | 🟠 Major |
|
|
154
|
+
|
|
155
|
+
**对比操作步骤**:
|
|
156
|
+
|
|
157
|
+
1. 从 PR diff 中提取 ohos 变更涉及的所有 `case 'methodName'` 分支
|
|
158
|
+
2. 在 `{outputDir}/source/android/` 中定位对应的 Plugin 实现文件(通常为 `*Plugin.java` 或 `*Plugin.kt`)
|
|
159
|
+
3. 逐方法对比:方法是否存在 → 参数类型是否一致 → 返回值格式是否一致 → 错误处理是否对齐
|
|
160
|
+
4. 对于 EventChannel,对比事件名和事件数据格式
|
|
161
|
+
5. 将对比结果记录到报告的"跨平台一致性"章节
|
|
162
|
+
|
|
163
|
+
---
|
|
164
|
+
|
|
165
|
+
#### 3.3 Dart 改动必要性分析(重点维度)
|
|
166
|
+
|
|
167
|
+
> **核心原则**:鸿蒙适配 PR 的本质是为已有 Flutter 三方库增加 ohos 平台实现,Dart 代码(`lib/` 目录)理论上不应修改。若 PR 包含 Dart 代码变更,必须逐一论证其必要性,并评估对 android/iOS 平台的影响。
|
|
168
|
+
|
|
169
|
+
**分析步骤**:
|
|
170
|
+
|
|
171
|
+
1. **识别所有 Dart 变更**:从 PR diff 中提取 `lib/**/*.dart` 的所有修改点
|
|
172
|
+
2. **逐项分类**:将每处 Dart 变更归入以下类别
|
|
173
|
+
|
|
174
|
+
| 变更类别 | 说明 | 问题等级 | 示例 |
|
|
175
|
+
|----------|------|----------|------|
|
|
176
|
+
| ✅ 必要改动 | 鸿蒙适配必须的改动,无其他实现方式 | 无问题(需记录) | 添加 `Platform.isOHOS` 平台判断以注册 ohos Plugin |
|
|
177
|
+
| ⚠️ 有替代方案 | 可通过 ohos 侧实现避免,不应修改 Dart | 🟠 Major | 修改 `MethodChannel` 方法名以匹配 ohos 实现(应在 ohos 侧适配) |
|
|
178
|
+
| ❌ 不应修改 | 改变已有 API 行为或影响其他平台 | 🔴 Critical | 修改 `invokeMethod` 的参数结构、修改返回值解析逻辑 |
|
|
179
|
+
| 🔍 需评估 | 涉及平台条件分支,需判断影响范围 | 🟡 Minor | 新增 `if (Platform.isOHOS)` 条件分支 |
|
|
180
|
+
|
|
181
|
+
3. **跨平台影响评估**:对每处 Dart 变更,验证以下影响
|
|
182
|
+
|
|
183
|
+
| 影响项 | 检查方法 | 问题等级 |
|
|
184
|
+
|--------|----------|----------|
|
|
185
|
+
| android/iOS 运行时行为是否改变 | 检查变更是否在 `Platform.isOHOS` 条件内 | 🔴 Critical(若影响) |
|
|
186
|
+
| MethodChannel 协议是否破坏 | Dart 侧 `invokeMethod` 参数是否与 android 侧仍兼容 | 🔴 Critical(若破坏) |
|
|
187
|
+
| 公共 API 签名是否变更 | `pubspec.yaml` 版本号是否对应升 major | 🟠 Major(若未升版本) |
|
|
188
|
+
| Platform Interface 默认实现是否受影响 | 默认实现的行为是否被改变 | 🟠 Major(若改变) |
|
|
189
|
+
|
|
190
|
+
4. **Dart 变更替代方案建议**:对于每处「有替代方案」或「不应修改」的变更,给出 ohos 侧的替代实现建议
|
|
191
|
+
|
|
192
|
+
**常见 Dart 改动模式与判定**:
|
|
193
|
+
|
|
194
|
+
| Dart 改动模式 | 判定 | 说明 |
|
|
195
|
+
|--------------|------|------|
|
|
196
|
+
| 新增 `if (Platform.isOHOS)` 条件分支 | ✅ 必要 | 鸿蒙平台需要独立的初始化或注册逻辑 |
|
|
197
|
+
| 新增 `TargetPlatform.ohos` case | ✅ 必要 | 平台枚举扩展 |
|
|
198
|
+
| 修改 `MethodChannel` 名称 | ❌ 不应修改 | 应在 ohos 侧注册与 android 相同的 Channel 名 |
|
|
199
|
+
| 修改 `invokeMethod` 的参数 | ❌ 不应修改 | 应在 ohos 侧适配参数解析 |
|
|
200
|
+
| 修改 `result` 返回值的解析逻辑 | ❌ 不应修改 | 应在 ohos 侧返回与 android 相同格式 |
|
|
201
|
+
| 新增 `import 'dart:io'`(原无) | ⚠️ 需评估 | 若仅用于 `Platform.isOHOS` 可接受,但可能影响 Web 平台 |
|
|
202
|
+
| 修改 Platform Interface 方法签名 | ❌ 不应修改 | 应通过 ohos Plugin 实现来适配 |
|
|
203
|
+
| 新增公共 API 方法 | ⚠️ 需评估 | 若仅 ohos 使用,应考虑是否为通用需求 |
|
|
204
|
+
| 修改 `pubspec.yaml` 依赖 | 🔍 需评估 | 新增依赖是否为 ohos 适配必要 |
|
|
205
|
+
| 删除或注释已有代码 | ❌ 不应修改 | 鸿蒙适配不应删除已有功能代码 |
|
|
206
|
+
|
|
207
|
+
---
|
|
208
|
+
|
|
209
|
+
### Phase 4:影响分析
|
|
210
|
+
|
|
211
|
+
- **功能影响**:是否引入新功能或修改行为
|
|
212
|
+
- **兼容性影响**:Plugin API 是否向后兼容
|
|
213
|
+
- **性能影响**:异步处理是否改善
|
|
214
|
+
- **安全影响**:是否修复或引入安全问题
|
|
215
|
+
|
|
216
|
+
---
|
|
217
|
+
|
|
218
|
+
### Phase 5:生成报告
|
|
219
|
+
|
|
220
|
+
输出文件:
|
|
221
|
+
- `{outputDir}/check-report.md` — 检视报告
|
|
222
|
+
- `{outputDir}/changed-files.txt` — 修改文件列表
|
|
223
|
+
- `{outputDir}/result.txt` — 合入建议分级结果
|
|
224
|
+
|
|
225
|
+
> 报告格式参见 [输出模板](assets/OUTPUT_TEMPLATE.md)
|
|
226
|
+
|
|
227
|
+
### 5.1 合入建议分级标准
|
|
228
|
+
|
|
229
|
+
根据代码检视结果,将合入建议分为四个等级:
|
|
230
|
+
|
|
231
|
+
| 等级 | 标识 | 颜色建议 | 判断标准 | 举例场景 |
|
|
232
|
+
|------|------|----------|----------|----------|
|
|
233
|
+
| **建议合入** | ✅ `PASS` | 绿色 | 无或极少(≤2 Minor)修改引入问题,改动风险低,功能验证充分 | 纯文档更新、配置优化、已验证的 Bug 修复 |
|
|
234
|
+
| **条件合入** | ⚠️ `CONDITIONAL` | 黄色 | 存在少量(≤3 Major)可修复问题,在特定条件下可合入 | 需补充测试用例、需调整文档说明、非核心路径兼容性问题 |
|
|
235
|
+
| **修改后合入** | 🔧 `NEED_FIX` | 橙色 | 存在较多问题(≥1 Critical 或 ≥4 Major),需修复后方可合入 | Plugin API 兼容性破坏、内存泄漏、`result.success()` 后未 return |
|
|
236
|
+
| **不允许合入** | ❌ `REJECT` | 红色 | 存在严重问题(≥2 Critical),影响功能/安全/稳定性,不建议合入 | 安全漏洞、严重 NAPI 内存问题、未验证的重大架构变更 |
|
|
237
|
+
|
|
238
|
+
**分级判断流程**:
|
|
239
|
+
|
|
240
|
+
```
|
|
241
|
+
统计问题数量:
|
|
242
|
+
- Critical 级别问题数
|
|
243
|
+
- Major 级别问题数
|
|
244
|
+
- Minor 级别问题数
|
|
245
|
+
|
|
246
|
+
判断逻辑:
|
|
247
|
+
IF Critical ≥ 2 → ❌ REJECT
|
|
248
|
+
IF Critical ≥ 1 → 🔧 NEED_FIX
|
|
249
|
+
IF Major ≥ 4 → 🔧 NEED_FIX
|
|
250
|
+
IF Major ≥ 1 → ⚠️ CONDITIONAL
|
|
251
|
+
IF Minor ≤ 2 → ✅ PASS
|
|
252
|
+
ELSE → ⚠️ CONDITIONAL
|
|
253
|
+
```
|
|
254
|
+
|
|
255
|
+
### 5.2 result.txt 文件格式
|
|
256
|
+
|
|
257
|
+
`result.txt` 用于存储分级结果和统计信息,格式详见 [输出模板](assets/OUTPUT_TEMPLATE.md) 的 result.txt 生成规范。
|
|
258
|
+
|
|
259
|
+
### 5.3 结构化指标输出(必须执行)
|
|
260
|
+
|
|
261
|
+
执行环境会在任务 prompt 中下发一个**结构化指标文件输出路径**(`stepId{X}_loop{L}_meta_{N}.json`)。
|
|
262
|
+
检视完成后,**必须**将分级结果以下述 JSON 对象写入该路径:
|
|
263
|
+
|
|
264
|
+
```json
|
|
265
|
+
{
|
|
266
|
+
"mergeLevel": "PASS | CONDITIONAL | NEED_FIX | REJECT",
|
|
267
|
+
"critical": 0,
|
|
268
|
+
"major": 0,
|
|
269
|
+
"minor": 0
|
|
270
|
+
}
|
|
271
|
+
```
|
|
272
|
+
|
|
273
|
+
| 字段 | 类型 | 说明 |
|
|
274
|
+
|------|------|------|
|
|
275
|
+
| `mergeLevel` | string | 合入等级:PASS / CONDITIONAL / NEED_FIX / REJECT |
|
|
276
|
+
| `critical` | number | Critical 级别问题数 |
|
|
277
|
+
| `major` | number | Major 级别问题数 |
|
|
278
|
+
| `minor` | number | Minor 级别问题数 |
|
|
279
|
+
|
|
280
|
+
**说明**:该文件由执行框架自动收割入库(前端/历史列表直接读取,无需解析报告文本);详细的统计与理由已在 `result.txt` 中完整记录。若 prompt 未下发该路径则跳过本节。
|
|
281
|
+
|
|
282
|
+
---
|
|
283
|
+
|
|
284
|
+
## 注意事项
|
|
285
|
+
|
|
286
|
+
1. 若 PR 只修改 `android/` 或 `ios/`,无鸿蒙改动 → 注明并 APPROVED
|
|
287
|
+
2. 报告文件名必须是 `check-report.md`
|
|
288
|
+
3. **`result.success()` / `result.error()` 后必须 `return`** — 最常见严重问题
|
|
289
|
+
4. 若同时修改 `ohos/` 和 `lib/` → 交叉验证方法名和参数类型
|
|
290
|
+
5. **Dart 代码修改原则**:鸿蒙适配 PR 中 Dart 代码不应修改;若存在 Dart 变更,必须在报告中单独论证每处变更的必要性和跨平台影响
|
|
291
|
+
6. Dart 侧修改 `MethodChannel` 方法名/参数/返回值格式 → 应改为在 ohos 侧适配,而非修改 Dart 代码
|
|
292
|
+
|
|
293
|
+
---
|
|
294
|
+
|
|
295
|
+
## 参考文档
|
|
296
|
+
|
|
297
|
+
- [检视清单 CHECKLIST.md](references/CHECKLIST.md)
|
|
298
|
+
- [平台 API PLATFORMS.md](references/PLATFORMS.md)
|
|
299
|
+
- [输出模板 OUTPUT_TEMPLATE.md](assets/OUTPUT_TEMPLATE.md)
|