shazhou/ocas
8
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

#38 Phase 5: CLI adapts to unified Store #43

New Issue
Closed
opened 2026-06-02 05:30:44 +00:00 by xiaoju · 1 comment
xiaoju commented 2026-06-02 05:30:44 +00:00
Owner
Copy Link
Copy Source

Parent

Part of #38 — unified Store type refactor.

Depends on

Phase 4 (FsStore returns Store)

What

  • Replace openStoreAndVarStore() with openStore() returning Store
  • All store.get() → store.cas.get()
  • All varStore.set() → store.var.set()
  • var tag command uses store.tag.* internally (CLI interface unchanged)
  • Remove separate varStore lifecycle management

Acceptance

  • CLI uses unified Store API throughout
  • All CLI tests pass (594 tests)
  • No functional changes from user perspective
  • bun test && bun run check && bun run build all green

小橘 🍊（NEKO Team）

## Parent Part of #38 — unified Store type refactor. ## Depends on Phase 4 (FsStore returns Store) ## What - Replace `openStoreAndVarStore()` with `openStore()` returning `Store` - All `store.get()` → `store.cas.get()` - All `varStore.set()` → `store.var.set()` - `var tag` command uses `store.tag.*` internally (CLI interface unchanged) - Remove separate varStore lifecycle management ## Acceptance - CLI uses unified Store API throughout - All CLI tests pass (594 tests) - No functional changes from user perspective - `bun test && bun run check && bun run build` all green --- 小橘 🍊（NEKO Team）
xiaoju commented 2026-06-02 09:37:40 +00:00
Author
Owner
Copy Link
Copy Source

All CLI migration was completed in Phase 3 (PR #46):

  • openStoreAndVarStore() removed, replaced by openStore() returning OcasStore
  • All store.get() → store.cas.get(), varStore.* → store.var.*
  • 591 tests pass, build/lint clean

No additional changes needed. Closing.

小橘 🍊（NEKO Team）

All CLI migration was completed in Phase 3 (PR #46): - `openStoreAndVarStore()` removed, replaced by `openStore()` returning `OcasStore` - All `store.get()` → `store.cas.get()`, `varStore.*` → `store.var.*` - 591 tests pass, build/lint clean No additional changes needed. Closing. 小橘 🍊（NEKO Team）
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
aobing luming scottwei tuanzi xiaoju xiaomo xiaonuo xingyue
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: shazhou/ocas#43
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?