uncaged/workflow
Archived
10
0
Fork 0
Code Issues 6 Pull Requests Actions Packages Projects Releases Wiki Activity

chore: remove deprecated code #114

New Issue
Closed
opened 2026-05-08 02:22:24 +00:00 by xingyue · 0 comments
xingyue commented 2026-05-08 02:22:24 +00:00
Owner
Copy Link
Copy Source

What

Remove all deprecated aliases, compat shims, and dead code across the monorepo.

Items

1. createThreadCas alias (workflow)

  • packages/workflow/src/cas/cas.ts:67 — export const createThreadCas = createCasStore
  • Re-exported in cas/index.ts and src/index.ts
  • Test in cas.test.ts just asserts it equals createCasStore
  • No real consumers — CAS is global, not per-thread
  • Action: delete alias, remove re-exports, delete test

2. formatSkillDoc() legacy compat (cli-workflow)

  • packages/cli-workflow/src/skill.ts:235-237 — deprecated wrapper around formatSkillCli()
  • Only caller: help.test.ts legacy test
  • Action: delete function, update test to use formatSkillTopic('cli')

3. help command (cli-workflow)

  • cli-dispatch.ts:89,103 — already prints deprecation warning pointing to skill
  • Action: remove help from COMMAND_TABLE, users get unknown command (or keep as soft error with suggestion)
  • Decision needed: hard-remove or keep warning for one more release?

4. CLI flat-command deprecated aliases (cli-workflow)

  • DEPRECATED_ALIASES in cli-dispatch.ts: 13 flat commands (add, list, show, remove, ps, kill, pause, resume, threads, fork, gc, history, rollback)
  • All print deprecation warnings and delegate to grouped commands
  • Action: remove aliases, update tests
  • Decision needed: hard-remove or keep warning for one more release?

Scope

Items 1-2 are safe to remove immediately (no external consumers).
Items 3-4 may affect users/agents relying on old command names — discuss timeline.

## What Remove all deprecated aliases, compat shims, and dead code across the monorepo. ## Items ### 1. createThreadCas alias (workflow) - packages/workflow/src/cas/cas.ts:67 — export const createThreadCas = createCasStore - Re-exported in cas/index.ts and src/index.ts - Test in cas.test.ts just asserts it equals createCasStore - **No real consumers** — CAS is global, not per-thread - Action: delete alias, remove re-exports, delete test ### 2. formatSkillDoc() legacy compat (cli-workflow) - packages/cli-workflow/src/skill.ts:235-237 — deprecated wrapper around formatSkillCli() - Only caller: help.test.ts legacy test - Action: delete function, update test to use formatSkillTopic('cli') ### 3. help command (cli-workflow) - cli-dispatch.ts:89,103 — already prints deprecation warning pointing to skill - Action: remove help from COMMAND_TABLE, users get unknown command (or keep as soft error with suggestion) - **Decision needed**: hard-remove or keep warning for one more release? ### 4. CLI flat-command deprecated aliases (cli-workflow) - DEPRECATED_ALIASES in cli-dispatch.ts: 13 flat commands (add, list, show, remove, ps, kill, pause, resume, threads, fork, gc, history, rollback) - All print deprecation warnings and delegate to grouped commands - Action: remove aliases, update tests - **Decision needed**: hard-remove or keep warning for one more release? ## Scope Items 1-2 are safe to remove immediately (no external consumers). Items 3-4 may affect users/agents relying on old command names — discuss timeline.
This repo is archived. You cannot comment on issues.
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 jiashuang (Jia Shuang) jiayi (Jiayi) luming scottwei tuanzi xiaoju xiaomo xiaonuo xingyue
No Assignees
1 Participants
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: uncaged/workflow#114
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?