shazhou/united-workforce
8
0
Fork 0
Code Issues 8 Pull Requests 2 Actions Packages Projects Releases Wiki Activity

refactor: remove legacy symlink migration code #25

Merged
xiaomo merged 1 commits from refactor/remove-legacy-symlink-migration into main 2026-06-02 16:28:52 +00:00
Conversation 1 Commits 1 Files Changed 3
+3 -80
xingyue commented 2026-06-02 16:22:37 +00:00
Owner
Copy Link
Copy Source

What

Remove migrateStorageIfNeeded() — the last code referencing uncaged.

Why

This function created symlinks from ~/.uncaged/workflow → ~/.uwf and ~/.uncaged/json-cas → ~/.ocas on every CLI startup. With the migration complete, this is dead code. Users on old paths can do a one-time manual copy.

Changes

  • Removed migrateStorageIfNeeded() from store.ts
  • Removed call from cli.ts
  • Removed symlinkSync import
  • Removed migration tests from store-storage-root.test.ts
  • 3 files, +3 -80

Result

Zero uncaged references in active codebase (only docs/plans/ historical docs remain).

check ✅, 397 CLI tests pass, 0 fail.

## What Remove `migrateStorageIfNeeded()` — the last code referencing `uncaged`. ## Why This function created symlinks from `~/.uncaged/workflow` → `~/.uwf` and `~/.uncaged/json-cas` → `~/.ocas` on every CLI startup. With the migration complete, this is dead code. Users on old paths can do a one-time manual copy. ## Changes - Removed `migrateStorageIfNeeded()` from `store.ts` - Removed call from `cli.ts` - Removed `symlinkSync` import - Removed migration tests from `store-storage-root.test.ts` - 3 files, +3 -80 ## Result **Zero** `uncaged` references in active codebase (only `docs/plans/` historical docs remain). check ✅, 397 CLI tests pass, 0 fail.
xingyue added 1 commit 2026-06-02 16:22:37 +00:00
refactor: remove legacy symlink migration code
CI / check (pull_request) Failing after 7m43s
Details
4306935dbc 
Remove migrateStorageIfNeeded() which created symlinks from
~/.uncaged/workflow → ~/.uwf and ~/.uncaged/json-cas → ~/.ocas.

This was temporary migration support. Users who still have old paths
can run a one-time copy manually.

Zero 'uncaged' references remain in active codebase.
xiaomo approved these changes 2026-06-02 16:28:51 +00:00
xiaomo left a comment
Owner
Copy Link
Copy Source

LGTM ✅ symlink 迁移代码 + 测试删除，-80 行。活跃代码零 uncaged 残留。

LGTM ✅ symlink 迁移代码 + 测试删除，-80 行。活跃代码零 uncaged 残留。
xiaomo merged commit 0d93e56acd into main 2026-06-02 16:28:52 +00:00
xiaomo deleted branch refactor/remove-legacy-symlink-migration 2026-06-02 16:28:52 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
xiaomo
Labels
Clear labels
bug
Something is not working
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
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: shazhou/united-workforce#25
Delete Branch "refactor/remove-legacy-symlink-migration"

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?