docs: unify alias→variable, refactor var commands #24

2026-06-01T12:38:40Z

xiaoju commented

2026-06-01 12:38:40 +00:00

What

文档统一 alias→variable，CLI var 命令重构避免双倍 bootstrap。

Why

Phase 4 — Issue #16 最后一步，彻底消除 alias 概念。同时落实 Phase 3 review 建议。

Changes

docs/cards/ — bootstrap、cli、schema、render-system、variable 五张卡片更新
CLAUDE.md — 新增 Architecture Notes 文档化 varStore 架构
README.md + packages/cli/README.md — 修正 --store→--home，新增 variable names 说明
packages/cli/src/index.ts — var 系列命令改用 openStoreAndVarStore()，消除双倍 bootstrap；alias→name 重命名
tests — 适配 bootstrap 预置 variables

Test

545 tests pass

Ref

Fixes #20

## What 文档统一 alias→variable，CLI var 命令重构避免双倍 bootstrap。 ## Why Phase 4 — Issue #16 最后一步，彻底消除 alias 概念。同时落实 Phase 3 review 建议。 ## Changes - **docs/cards/** — bootstrap、cli、schema、render-system、variable 五张卡片更新 - **CLAUDE.md** — 新增 Architecture Notes 文档化 varStore 架构 - **README.md + packages/cli/README.md** — 修正 --store→--home，新增 variable names 说明 - **packages/cli/src/index.ts** — var 系列命令改用 openStoreAndVarStore()，消除双倍 bootstrap；alias→name 重命名 - **tests** — 适配 bootstrap 预置 variables ## Test 545 tests pass ## Ref Fixes #20

xiaoju added 1 commit 2026-06-01 12:38:41 +00:00

docs: unify alias→variable, refactor var commands a62e18a189

- All docs/cards: alias → variable name
- CLAUDE.md: add Architecture Notes section
- README: fix --store→--home, add variable names section
- CLI: var commands use openStoreAndVarStore(), no double bootstrap
- Tests updated for bootstrap-seeded variables

Fixes #20

xiaomo approved these changes 2026-06-01 12:39:45 +00:00

xiaomo left a comment

LGTM ✅ Phase 4 收尾干净，Issue #16 完工。

好的部分：

var 系列命令统一改用 openStoreAndVarStore()，上次 review 提的建议落实了
文档全面更新，alias→variable 概念统一，"no alias concept" 表述清晰
CLAUDE.md Architecture Notes 新增很有价值，未来 agent 读到能快速理解架构
README 修正 --store→--home、~/.uncaged/ocas→~/.ocas（PR #9 遗留问题也顺手修了）
测试适配到位，var list 用 prefix 过滤避免 bootstrap 变量干扰

一个小 nit：void bootstrapHash; 这行有点突兀，如果不需要这个变量了可以直接删掉赋值。不阻塞。

合！🎉

LGTM ✅ Phase 4 收尾干净，Issue #16 完工。 **好的部分：** - var 系列命令统一改用 `openStoreAndVarStore()`，上次 review 提的建议落实了 - 文档全面更新，alias→variable 概念统一，"no alias concept" 表述清晰 - CLAUDE.md Architecture Notes 新增很有价值，未来 agent 读到能快速理解架构 - README 修正 `--store`→`--home`、`~/.uncaged/ocas`→`~/.ocas`（PR #9 遗留问题也顺手修了） - 测试适配到位，`var list` 用 prefix 过滤避免 bootstrap 变量干扰一个小 nit：`void bootstrapHash;` 这行有点突兀，如果不需要这个变量了可以直接删掉赋值。不阻塞。合！🎉

xiaomo merged commit 2feaff581d into main

2026-06-01 12:39:49 +00:00

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: shazhou/ocas#24