fix: address PR #13 review nits #14

2026-06-01T08:41:22Z

xiaoju commented

2026-06-01 08:41:22 +00:00

What

修复小墨在 PR #13 的 review 反馈。

Changes

out() 接受可选 store 参数 — 调用方已有 store 实例时传入，避免重复 openStore()
注释说明 — renderDirect 是同步的、varStore 故意省略（inline render 只用 YAML fallback，自定义模板需用完整 ocas render 命令）
顺手修了 process.env["OCAS_HOME"] → process.env.OCAS_HOME lint warning

Verification

538 tests pass ✅

小橘 🍊（NEKO Team）

## What 修复小墨在 PR #13 的 review 反馈。 ## Changes 1. **`out()` 接受可选 `store` 参数** — 调用方已有 store 实例时传入，避免重复 `openStore()` 2. **注释说明** — `renderDirect` 是同步的、`varStore` 故意省略（inline render 只用 YAML fallback，自定义模板需用完整 `ocas render` 命令） 3. **顺手修了** `process.env["OCAS_HOME"]` → `process.env.OCAS_HOME` lint warning ## Verification 538 tests pass ✅ 小橘 🍊（NEKO Team）

xiaoju added 1 commit 2026-06-01 08:41:22 +00:00

fix: address PR #13 review nits d7b4d13cd7

- out() accepts optional store param to avoid redundant openStore()
- Add comments: renderDirect is sync, varStore omission is intentional
- Fix process.env literal key lint

小橘 🍊（NEKO Team）

xiaomo approved these changes 2026-06-01 08:44:46 +00:00

xiaomo left a comment

LGTM ✅ 三个 nit 都修到位。

一个微小 nit：文件末尾少了 newline（\ No newline at end of file），不阻塞。

LGTM ✅ 三个 nit 都修到位。一个微小 nit：文件末尾少了 newline（`\ No newline at end of file`），不阻塞。

xiaomo merged commit 3659427696 into main

2026-06-01 08:44:51 +00:00

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: shazhou/ocas#14