refactor ParsedIntent to use pimpl pattern

[Johnson-zs]

• refactor: convert HighlightOptions to Pimpl pattern
• fix: refactor ParsedIntent to use pimpl pattern

[sourcery-ai]

Sorry @Johnson-zs, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Johnson-zs

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• debian/deepin/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[deepin-ci-robot]

deepin pr auto review

★ 总体评分：100分

■ 【总体评价】

代码完美实现了ParsedIntent和HighlightOptions从裸struct到Pimpl模式的架构重构，所有调用点均准确适配
逻辑正确且无任何安全风险，完全符合Qt隐式共享与ABI稳定性设计规范

■ 【详细分析】

• 1.语法逻辑（完全正确）✓

重构将直接暴露成员变量的struct改为基于QSharedDataPointer的class，并提供了const与非const重载的getter及setter。拷贝/移动构造函数及赋值运算符均正确使用=default，依赖QSharedDataPointer自动实现COW语义与内存管理。所有Extractor、Builder、JsonOutput及测试代码中的属性访问均从点操作符（如intent.timeConstraint.kind）准确替换为getter调用（如intent.timeConstraint().kind），赋值操作准确替换为setter调用（如intent.setKeywords()），无任何遗漏或编译不匹配问题。

• 2.代码质量（良好）✓

头文件中清晰注释了Pimpl模式的设计意图与ABI稳定性保障，类声明遵循Qt命名规范。Pimpl实现文件parsedintent.cpp与highlightoptions_p.h职责划分清晰。在actionextractor.cpp与locationextractor.cpp中存在对同一getter的连续两次调用（如intent.searchDirectories().contains(path)与intent.searchDirectories().append(path)），虽然因QSharedDataPointer的detach机制保证逻辑正确，但略显冗余。
建议：在actionextractor.cpp和locationextractor.cpp中，可通过局部引用变量缓存getter返回值以提升代码可读性，例如QStringList &dirs = intent.searchDirectories();

• 3.代码性能（无性能问题）✓

引入QSharedDataPointer增加了一次指针解引用开销，但对于文件搜索这种IO密集型场景可完全忽略。在单线程解析流程中传递ParsedIntent引用不会触发不必要的深拷贝。测试用例中多次调用timeConstraint()等getter返回的是引用，不涉及大量数据拷贝。

• 4.代码安全（存在0个安全漏洞）✓

漏洞对比统计：新增漏洞 0 个，减少漏洞 0 个，持平 0 个
本次提交为纯架构重构，仅改变了数据成员的访问方式（从直接暴露改为getter/setter），未引入任何新的外部输入解析、命令执行、路径拼接或权限校验逻辑，彻底杜绝了引入新安全漏洞的可能。

■ 【改进建议代码示例】

// src/dfm-search/dfm-search-lib/semantic/extractors/actionextractor.cpp
// 优化连续调用getter的代码片段，提升可读性并避免冗余的detach检查
void ActionExtractor::extract(const QString &input, ParsedIntent &intent)
{
    // ... 前置逻辑省略 ...

        // ── IM received files ──
        if (isImReceive) {
            const QStringList resolvedPaths = resolveImReceivedPaths();
            if (!resolvedPaths.isEmpty()) {
                // 使用局部引用缓存getter返回值，避免重复调用
                QStringList &dirs = intent.searchDirectories();
                for (const QString &path : resolvedPaths) {
                    if (!dirs.contains(path)) {
                        dirs.append(path);
                    }
                }

                MatchSpan span;
                span.start = m.capturedStart();
                span.end = m.capturedEnd();
                span.ruleId = ruleIds[i];
                intent.consumedSpans().append(span);
            }
            continue;
        }

    // ... 后置逻辑省略 ...
}