fix: improve file conversion fallback logic for truncated files

[Johnson-zs]

Refactored the file conversion logic to:

1. Split the truncation handling into two separate functions
2. First attempt conversion with original file extension
3. Only fallback to similar extensions if primary conversion fails
4. Removed duplicate error logging and simplified control flow
5. Improved code organization and separation of concerns

The changes ensure proper fallback behavior when processing files with different extensions and maintains cleaner error handling. Previously, the code would unconditionally try similar extensions even when the primary conversion succeeded.

Influence:

1. Test with files having correct extensions that should convert successfully
2. Test with files needing fallback to similar extensions
3. Verify truncation behavior works correctly for both cases
4. Check error cases where neither primary nor fallback extensions work
5. Validate truncation markers are added correctly

fix: 改进截断文件转换的回退逻辑

重构文件转换逻辑：

1. 将截断处理拆分为两个独立函数
2. 首先尝试使用原始文件扩展名进行转换
3. 仅当主转换失败时才回退到类似扩展名
4. 移除重复的错误日志记录并简化控制流
5. 改进代码组织和关注点分离

这些改动确保在处理不同扩展名文件时有正确的回退行为，并保持更清晰的错误处
理。之前的代码会在主转换成功后仍无条件尝试类似扩展名。

Influence:

1. 测试使用应能成功转换的正确扩展名文件
2. 测试需要回退到类似扩展名的文件
3. 验证两种情况的截断行为是否正确
4. 检查主转换和回退扩展名都无效的错误情况
5. 验证截断标记是否正确添加

[deepin-ci-robot]

deepin pr auto review

我来对这段代码变更进行审查：

1. 代码逻辑改进：

• 将原来的单一函数拆分成了两个更小的函数：tryConvertWithTruncation 和 doConvertFileWithTruncation
• tryConvertWithTruncation 现在专注于处理单个后缀的转换，而 doConvertFileWithTruncation 负责处理后缀尝试逻辑
• 这种拆分提高了代码的可维护性和可测试性

2. 代码质量改进：

• 函数职责更加单一，遵循了单一职责原则
• 减少了代码重复，因为现在可以重用 tryConvertWithTruncation 来尝试不同的后缀
• 错误处理更加清晰，每个函数都有明确的返回值约定

3. 性能影响：

• 性能基本保持不变，因为只是重构了代码结构，没有改变核心逻辑
• 可能会有轻微的函数调用开销，但可以忽略不计

4. 安全性改进：

• 没有引入新的安全风险
• 错误处理更加健壮，因为现在可以更好地处理转换失败的情况

5. 建议的额外改进：

• 可以考虑在 tryConvertWithTruncation 中添加对 maxBytes 参数的合理性检查
• 可以在 doConvertFileWithTruncation 中添加对文件是否存在的检查
• 考虑将 similarExtensionMap 作为参数传递，而不是使用静态变量，这样更容易进行单元测试

6. 代码风格改进：

• 代码格式统一，缩进一致
• 注释清晰，函数文档完整
• 变量命名合理，易于理解

总体来说，这是一个很好的重构，提高了代码的可维护性和可读性，同时保持了原有的功能和性能。

[github-actions]

TAG Bot

TAG: 1.0.24
EXISTED: no
DISTRIBUTION: unstable

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Johnson-zs, max-lvs

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