refactor: remove runtime CLI, move to packages/, separate lockfile I/O#510
Merged
refactor: remove runtime CLI, move to packages/, separate lockfile I/O#510
Conversation
- Delete runtime CLI (bin/, src/cli/) and remove commander/dotenv deps - Separate lockfile I/O: move findLockfile/loadLockfile to @perstack/tui - Move @perstack/runtime from apps/ to packages/ (library, not app) - Merge e2e/perstack-runtime/ tests into e2e/perstack-cli/ - Update benchmarks to use apps/perstack/bin/cli.ts - Port CLI tests (get-env, provider-config, perstack-toml, context) to TUI - Update docs and README to reflect single CLI architecture Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Include minRuntimeVersion in expert mapping in resolveRunContext so runtime version validation works correctly via the perstack CLI - Remove --run-id e2e test since perstack CLI doesn't have that option Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add patch changeset for @perstack/runtime and @perstack/tui - Remove unused smol-toml dep from runtime (moved to tui with lockfile I/O) - Remove unused memfs devDep from runtime (lockfile tests moved to tui) - Fix knip entry pattern for apps/perstack Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Replace the old README (which was just the apps/runtime version minus the CLI section) with a focused library README that documents the public API surface, architecture overview, and links to detailed docs. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This was referenced Feb 15, 2026
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
bin/,src/cli/ディレクトリとcommander/dotenv依存を除去。CLI はapps/perstackに一本化findLockfile/loadLockfileを@perstack/runtimeから@perstack/tuiへ移動。runtime には純粋関数getLockfileExpertToolDefinitionsのみ残すapps/runtime→packages/runtime)e2e/perstack-runtime/の全15ファイルをe2e/perstack-cli/へマージ。runRuntimeCli→runCliに統一get-env,provider-config,perstack-toml,contextのテストをpackages/tuiに追加apps/perstack/bin/cli.tsに変更、README から二重 CLI の記述を削除Test plan
pnpm typecheck— 21/21 passpnpm test— 100 files, 1093 tests passpnpm build— 21/21 passpnpm format-and-lint— exit 0 (pre-existing warnings only)🤖 Generated with Claude Code