chore: internalize unused exports across all packages #283

2026-05-16T09:59:18Z

xiaoju commented

2026-05-16 09:59:18 +00:00

What

从应用入口出发做可达性分析，收敛所有包的公共 API surface。只改 index.ts，不删源码。

Why

大量内部实现细节暴露为公共 export，增加认知负担和 API 契约维护成本。

Method

按 package-api-audit skill 的方法论：从 Worker/CLI/Dashboard 应用入口出发，沿定制链做类型签名的可达性闭包，不可达的符号从 index.ts 移除。

Changes (10 files, +32/-105)

Package	Removed	Notes
workflow-util	7	base32 internals, logger config types
workflow-cas	12	merkle internals, serialization
workflow-execute	24	engine internals, LLM extract
workflow-reactor	4	reactor config/invocation
workflow-register	8	redundant protocol re-exports, internal YAML fns
workflow-runtime	curated	stop full protocol re-export, keep bundle-author subset
workflow-util-agent	5	internal agent helpers
workflow-agent-cursor	1	validateCursorAgentConfig
workflow-agent-hermes	1	validateHermesAgentConfig

Not Changed

workflow-protocol — downstream packages still import symbols via internal paths. Cleanup requires updating workflow-runtime/src/types.ts first (follow-up PR).

Verification

bunx tsc --noEmit and bunx tsc --build both pass.

Ref

Refs #273, #274, #275, #276, #277, #278, #279, #280, #281, #282

## What 从应用入口出发做可达性分析，收敛所有包的公共 API surface。只改 `index.ts`，不删源码。 ## Why 大量内部实现细节暴露为公共 export，增加认知负担和 API 契约维护成本。 ## Method 按 `package-api-audit` skill 的方法论：从 Worker/CLI/Dashboard 应用入口出发，沿定制链做类型签名的可达性闭包，不可达的符号从 `index.ts` 移除。 ## Changes (10 files, +32/-105) | Package | Removed | Notes | |---------|---------|-------| | workflow-util | 7 | base32 internals, logger config types | | workflow-cas | 12 | merkle internals, serialization | | workflow-execute | 24 | engine internals, LLM extract | | workflow-reactor | 4 | reactor config/invocation | | workflow-register | 8 | redundant protocol re-exports, internal YAML fns | | workflow-runtime | curated | stop full protocol re-export, keep bundle-author subset | | workflow-util-agent | 5 | internal agent helpers | | workflow-agent-cursor | 1 | validateCursorAgentConfig | | workflow-agent-hermes | 1 | validateHermesAgentConfig | ## Not Changed `workflow-protocol` — downstream packages still import symbols via internal paths. Cleanup requires updating `workflow-runtime/src/types.ts` first (follow-up PR). ## Verification `bunx tsc --noEmit` and `bunx tsc --build` both pass. ## Ref Refs #273, #274, #275, #276, #277, #278, #279, #280, #281, #282

xiaoju added 1 commit 2026-05-16 09:59:19 +00:00

chore: internalize unused exports across all packages 5bbac3e4f7

Audit public API surfaces using reachability analysis from application
entry points (Worker, CLI, Dashboard). Symbols not reachable from any
application's customization tree are removed from package index.ts files.

Source files and internal usage are untouched — only the public export
surface is narrowed.

Changes by package:
- workflow-util: -7 exports (base32 internals, logger config types)
- workflow-cas: -12 exports (merkle internals, serialization details)
- workflow-execute: -24 exports (engine internals, LLM extract details)
- workflow-reactor: -4 exports (reactor config/invocation internals)
- workflow-register: -8 exports (redundant protocol re-exports, internal YAML fns)
- workflow-runtime: curated re-export subset (stop full protocol re-export)
- workflow-util-agent: -5 exports (internal agent helpers)
- workflow-agent-cursor: -1 export (validateCursorAgentConfig)
- workflow-agent-hermes: -1 export (validateHermesAgentConfig)

Note: workflow-protocol index.ts unchanged — downstream packages still
import removed symbols via internal paths. Protocol cleanup requires
updating workflow-runtime/src/types.ts first (separate PR).

Refs #273, #274, #275, #276, #277, #278, #279, #280, #281, #282