| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
fix: add exceptions config for profile tests including cpp with try-catch Co-Authored-By: Agent Signed-off-by: RuiChen_01 <chenrui193@huawei.com> 🤖 AI[100%] 👌 AI Adopted[100%] 🧑 Human[0%] Co-authored-by: opencode (deepseek-v4-flash-free) <ai@local> | 2 个月前 | |
perf(insight_intent): skip delete when bundle has no intent cache Guard both intent delete paths with an in-memory cache check before doing any work. The guard goes through a new CanSkipDelete(bundleName, userId) that is atomic under the cache mutex and only trusts a cache miss when the cache was successfully loaded for that user, so a failed load (which clears the map but keeps the old userId) or a user mismatch never skips the RDB delete and CLI unregister (review finding F1/F2). - InsightIntentEventMgr::DeleteInsightIntentEvent: a never-registered bundle uninstall previously ran a no-op RDB delete, a full BackupRdb file write and an unregister IPC; all skipped now - InsightIntentSysEventReceiver::DeleteInsightIntent: the name-based emptiness probe costs two prefix RDB queries plus JSON transforms; the cache hit now short-circuits them, and the redundant outer HasBundleCache check at the call site is collapsed Also compress multi-line comments in function_call_convert to one-line summaries. Co-Authored-By: Agent Signed-off-by: RuiChen_01 <chenrui193@huawei.com> AI[100%] Human Fixed[0%] Human[0%] AI Adopted[100%] Co-authored-by: claude (glm-5.3) <ai@local> | 1 个月前 | |
refactor(insight_intent): unify conditional delete into DbCache InsightIntentEventMgr::DeleteInsightIntent, the uninstall flow in DeleteInsightIntentEvent and InsightIntentSysEventReceiver::DeleteInsightIntent all carried their own delete logic with slight drift. Route them through the single guarded entry InsightIntentDbCache::DeleteInsightIntentTotalInfo, which now: - applies HasBundleCache (replaces CanSkipDelete, three-layer checks: user mismatch, failed cache load, then map lookup) - deletes in-memory entries and rdb rows only when the cache proves presence, returning whether the delete succeeded - drops the unused single-arg HasBundleCache overload Follow-up work (BackupRdb, function unregister) stays at the call sites. Co-Authored-By: Agent AI[100%] Human Fixed[0%] Human[0%] AI Adopted[100%] Co-authored-by: claude (glm-5.3) <ai@local> Change-Id: Iceea785c76cafe35f451a305c7da9a6c107b6d03 Signed-off-by: RuiChen_01 <chenrui193@huawei.com> | 1 个月前 | |
refactor(insight_intent): unify conditional delete into DbCache InsightIntentEventMgr::DeleteInsightIntent, the uninstall flow in DeleteInsightIntentEvent and InsightIntentSysEventReceiver::DeleteInsightIntent all carried their own delete logic with slight drift. Route them through the single guarded entry InsightIntentDbCache::DeleteInsightIntentTotalInfo, which now: - applies HasBundleCache (replaces CanSkipDelete, three-layer checks: user mismatch, failed cache load, then map lookup) - deletes in-memory entries and rdb rows only when the cache proves presence, returning whether the delete succeeded - drops the unused single-arg HasBundleCache overload Follow-up work (BackupRdb, function unregister) stays at the call sites. Co-Authored-By: Agent AI[100%] Human Fixed[0%] Human[0%] AI Adopted[100%] Co-authored-by: claude (glm-5.3) <ai@local> Change-Id: Iceea785c76cafe35f451a305c7da9a6c107b6d03 Signed-off-by: RuiChen_01 <chenrui193@huawei.com> | 1 个月前 | |
意图、wantAgent模块维测日志增强 Signed-off-by: linjunjie <linjunjie6@huawei.com> | 1 年前 | |
verify foundation uid in insight intent callback stub Add a calling-UID check (FOUNDATION_UID = 5523) in InsightIntentExecuteCallbackStub::OnRemoteRequest so that only the foundation process (AMS) can deliver OnExecuteDone results. Previously the stub relied solely on InterfaceToken, which is a publicly known descriptor string and not a security boundary; a leaked binder reference would let any process forge intent results into the caller's business callback. IsCallingFromFoundation is a non-virtual private method so subclasses cannot weaken the check. Tests exercise both accept and reject paths by mocking IPCSkeleton::GetCallingUid via the standard test/unittest/.../mock/include/ipc_skeleton.h pattern. Signed-off-by: RuiChen_01 <chenrui193@huawei.com> 🤖 AI[0%] 🔧 Human Fixed[0%] 🧑 Human[100%] 👌 AI Adopted[0%] | 2 个月前 | |
fix interactionInfo Co-Authored-By: Agent Signed-off-by: 任国军 <renguojun1@h-partners.com> AI[0%] Human Fixed[0%] Human[100%] AI Adopted[0%] | 1 个月前 | |
want op Co-Authored-By: Agent Signed-off-by: wangzhen <wangzhen416@huawei.com> 🤖 AI[0%] 🔧 Human Fixed[0%] 🧑 Human[100%] 👌 AI Adopted[0%] Change-Id: Ib81fc41ac313075bbb89425580fa08bfdd67cc0f | 2 个月前 | |
fix interactionInfo Co-Authored-By: Agent Signed-off-by: 任国军 <renguojun1@h-partners.com> AI[0%] Human Fixed[0%] Human[100%] AI Adopted[0%] | 1 个月前 | |
fix(insight_intent): guard json::dump against deep-nesting stack overflow nlohmann::json::dump() recurses on nested values; super-deep input causes SIGSEGV stack overflow that try/catch cannot intercept. - Add iterative IsJsonDepthOk walker with JSON_DUMP_MAX_DEPTH (100) bound; SafeDump/SafeDumpTo reject over-depth input before dump(). Definitions live in intent_json_safe_get.cpp, compiled only into the ability_manager innerkit so symbols resolve for libabilityms.so and direct-compiling test/fuzz targets (libabilityms.map hides non-whitelisted symbols). - Switch raw .dump() calls in function_call_convert, execute_result and info_for_query to the safe wrappers; validate inputSchema depth before merge. - Add IsJsonDepthOk boundary tests (under/at/over limit). Co-Authored-By: Agent Signed-off-by: RuiChen_01 <chenrui193@huawei.com> AI[100%] Human Fixed[0%] Human[0%] AI Adopted[100%] Co-authored-by: claude (unknown) <ai@local> Co-authored-by: claude (glm-5.3) <ai@local> Co-authored-by: claude (unknown) <ai@local> Co-authored-by: claude (glm-5.3) <ai@local> Co-authored-by: claude (unknown) <ai@local> Co-authored-by: claude (glm-5.3) <ai@local> Co-authored-by: claude (unknown) <ai@local> Co-authored-by: claude (glm-5.3) <ai@local> Co-authored-by: claude (unknown) <ai@local> | 1 个月前 | |
refactor: extract InsightIntentParamParser, add IntentFilterUtil Move parameter parsing/assembly for ExecuteIntentByFunctionCall out of AbilityManagerService anonymous helpers into a dedicated InsightIntentParamParser class with explicit, unit-testable rules. Expose multi-candidate matcher API (GetMatchedIntentInfos). Drop SA-only check on ExecuteIntentByFunctionCall entry so function calling agents can invoke intents directly. Add IntentFilterUtil for CliTool registration path: callers preprocess intent lists before RegisterInsightIntentFunctions to (1) drop intents not bound to background UIAbility or ServiceExtension, (2) pick entry module first when same intentName spans modules, (3) prefer UIAbility over ServiceExtension when same module has multiple abilities. Co-Authored-By: Agent Signed-off-by: RuiChen_01 <chenrui193@huawei.com> | 3 个月前 | |
fix insight intent function registration and cleanup Co-Authored-By: Agent Signed-off-by: RuiChen_01 <chenrui193@huawei.com> 🤖 AI[0%] 🔧 Human Fixed[0%] 🧑 Human[100%] 👌 AI Adopted[0%] | 2 个月前 | |
fix: add exceptions config for profile tests including cpp with try-catch Co-Authored-By: Agent Signed-off-by: RuiChen_01 <chenrui193@huawei.com> 🤖 AI[100%] 👌 AI Adopted[100%] 🧑 Human[0%] Co-authored-by: opencode (deepseek-v4-flash-free) <ai@local> | 2 个月前 | |
注解类意图静态化和queryEntity Co-Authored-By: Agent Signed-off-by: renjh5496 <renjianhao@h-partners.com> | 5 个月前 | |
fix(insight_intent): fix rdbStore_ data race in BackupRdb InsightIntentRdbDataMgr::BackupRdb was the only public method that did not acquire rdbStoreMutex_ before writing the rdbStore_ shared_ptr member. Concurrent calls from other methods (InsertData / UpdateData / DeleteData* / QueryDataBeginWithKey) could race on the shared_ptr assignment, causing undefined behavior. Add the missing lock_guard. Co-Authored-By: Agent Signed-off-by: RuiChen_01 <chenrui193@huawei.com> | 3 个月前 | |
fix: 意图数据库前缀匹配修复, Co-Authored-By: Agent Signed-off-by: songkeyuan <songkeyuan@huawei.com> AI[47%] Human Fixed[0%] Human[53%] AI Adopted[100%] | 1 个月前 | |
refactor(insight_intent): unify conditional delete into DbCache InsightIntentEventMgr::DeleteInsightIntent, the uninstall flow in DeleteInsightIntentEvent and InsightIntentSysEventReceiver::DeleteInsightIntent all carried their own delete logic with slight drift. Route them through the single guarded entry InsightIntentDbCache::DeleteInsightIntentTotalInfo, which now: - applies HasBundleCache (replaces CanSkipDelete, three-layer checks: user mismatch, failed cache load, then map lookup) - deletes in-memory entries and rdb rows only when the cache proves presence, returning whether the delete succeeded - drops the unused single-arg HasBundleCache overload Follow-up work (BackupRdb, function unregister) stays at the call sites. Co-Authored-By: Agent AI[100%] Human Fixed[0%] Human[0%] AI Adopted[100%] Co-authored-by: claude (glm-5.3) <ai@local> Change-Id: Iceea785c76cafe35f451a305c7da9a6c107b6d03 Signed-off-by: RuiChen_01 <chenrui193@huawei.com> | 1 个月前 | |
注解类意图静态化和queryEntity Co-Authored-By: Agent Signed-off-by: renjh5496 <renjianhao@h-partners.com> | 5 个月前 | |
fix(insight_intent): guard json::dump against deep-nesting stack overflow nlohmann::json::dump() recurses on nested values; super-deep input causes SIGSEGV stack overflow that try/catch cannot intercept. - Add iterative IsJsonDepthOk walker with JSON_DUMP_MAX_DEPTH (100) bound; SafeDump/SafeDumpTo reject over-depth input before dump(). Definitions live in intent_json_safe_get.cpp, compiled only into the ability_manager innerkit so symbols resolve for libabilityms.so and direct-compiling test/fuzz targets (libabilityms.map hides non-whitelisted symbols). - Switch raw .dump() calls in function_call_convert, execute_result and info_for_query to the safe wrappers; validate inputSchema depth before merge. - Add IsJsonDepthOk boundary tests (under/at/over limit). Co-Authored-By: Agent Signed-off-by: RuiChen_01 <chenrui193@huawei.com> AI[100%] Human Fixed[0%] Human[0%] AI Adopted[100%] Co-authored-by: claude (unknown) <ai@local> Co-authored-by: claude (glm-5.3) <ai@local> Co-authored-by: claude (unknown) <ai@local> Co-authored-by: claude (glm-5.3) <ai@local> Co-authored-by: claude (unknown) <ai@local> Co-authored-by: claude (glm-5.3) <ai@local> Co-authored-by: claude (unknown) <ai@local> Co-authored-by: claude (glm-5.3) <ai@local> Co-authored-by: claude (unknown) <ai@local> | 1 个月前 |
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
| 2 个月前 | ||
| 1 个月前 | ||
| 1 个月前 | ||
| 1 个月前 | ||
| 1 年前 | ||
| 2 个月前 | ||
| 1 个月前 | ||
| 2 个月前 | ||
| 1 个月前 | ||
| 1 个月前 | ||
| 3 个月前 | ||
| 2 个月前 | ||
| 2 个月前 | ||
| 5 个月前 | ||
| 3 个月前 | ||
| 1 个月前 | ||
| 1 个月前 | ||
| 5 个月前 | ||
| 1 个月前 |