chore: Phase 4 post-merge cleanup — duplicate types, unused params, test fixes #46

Closed
opened 2026-05-31 06:57:04 +00:00 by xiaoju · 0 comments
Owner

来源

PR #45 post-merge review 发现的 minor issues。

待修复

1. 重复 RenderOptions 类型

  • packages/json-cas/src/render.tspackages/json-cas/src/liquid-render.ts 各定义了一份
  • 应提取到 types.ts 或让 liquid-render 直接 import

2. 未使用的 _globalDecay 参数

  • liquid-render.ts 中有未使用的参数,应清理或使用

3. 测试编号跳号

  • liquid-render.test.ts 测试编号有 gap,不影响功能但不整洁

4. 25 个 CLI 测试 pre-existing 失败

  • 与 Phase 4 无关,但需要排查修复

小橘 🍊(NEKO Team)

## 来源 PR #45 post-merge review 发现的 minor issues。 ## 待修复 ### 1. 重复 `RenderOptions` 类型 - `packages/json-cas/src/render.ts` 和 `packages/json-cas/src/liquid-render.ts` 各定义了一份 - 应提取到 `types.ts` 或让 `liquid-render` 直接 import ### 2. 未使用的 `_globalDecay` 参数 - `liquid-render.ts` 中有未使用的参数,应清理或使用 ### 3. 测试编号跳号 - `liquid-render.test.ts` 测试编号有 gap,不影响功能但不整洁 ### 4. 25 个 CLI 测试 pre-existing 失败 - 与 Phase 4 无关,但需要排查修复 --- 小橘 🍊(NEKO Team)
This repo is archived. You cannot comment on issues.
No Label
1 Participants
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: uncaged/json-cas#46