refactor(docs,e2e): remove multi-runtime and storage backend references#459
Merged
refactor(docs,e2e): remove multi-runtime and storage backend references#459
Conversation
Remove documentation and e2e tests for features being deprecated: - Multi-runtime support (Docker, Cursor, Claude Code, Gemini) - Storage backend abstraction Deleted files: - docs/using-experts/multi-runtime.md - docs/operating-experts/storage-backends.md - e2e/perstack-cli/runtime-selection.test.ts - e2e/perstack-cli/docker-security.test.ts - e2e/perstack-cli/docker-attack-scenarios.test.ts - e2e/perstack-runtime/storage-behavior.test.ts - e2e/experts/docker-security.toml - e2e/experts/docker-attack-scenarios.toml Modified docs to remove Docker runtime references while keeping generic container terminology for deployment guidance. Removed --runtime flags from e2e tests and runtime availability checks from prerequisites. This is phase 1 of the cleanup - implementation changes will follow. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit removes support for multiple runtimes and the storage abstraction layer, simplifying Perstack to only use local runtime. Deleted packages (11): - @perstack/docker - @perstack/cursor - @perstack/claude-code - @perstack/gemini - @perstack/adapter-base - @perstack/runner - @perstack/s3-storage - @perstack/r2-storage - @perstack/s3-compatible-storage - @perstack/mock Key changes: - core: Remove adapter registry, types, and Storage interface - core: Simplify RuntimeName to only "local" - core: Remove runtime/workspace/volume/env CLI options - runtime: Remove PerstackAdapter, call run() directly - CLI: Remove runtime-dispatcher, call perstackRun() directly - filesystem-storage: Remove FileSystemStorage class, keep helpers - TUI: Remove Docker/multi-runtime UI rendering Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Delete apps/e2e-mcp-server - Delete apps/create-expert - Delete .agent directory - Delete .cursorrules - Move packages/storages/filesystem/ to packages/filesystem/ Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Delete codecov.yml - Rewrite SECURITY.md to reflect current architecture (no Docker runtime) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The lazy-init tests referenced the removed apps/e2e-mcp-server package. Replace with a lightweight NDJSON-based MCP server script in e2e/fixtures. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…e removal Remove dead constants, event types, unreachable code paths, and unused files left over from the multi-runtime/storage refactor. Fix stale references in READMEs, docs, benchmarks, and test files. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Delete dead TUI components: progress UI, useErrorHandler, useInputState, barrel files - Remove dead functions from run-manager.ts (getMostRecentRunId, getRunsByJobId, etc.) - Remove useEventStream hook and its tests from @perstack/react - Remove SelectableList and TextInput from @perstack/tui-components - Remove dead exports from @perstack/runtime (getModel, buildDelegateToState, etc.) - Remove dead exports from @perstack/filesystem-storage (storeRunSetting, getEventsByRun, etc.) - Remove hasCustomProviderSkills duplicate from @perstack/core - Deduplicate isPrivateOrLocalIP and filter/formatter constants - Fix all provider READMEs: wrong interface names, missing providerName, reasoning example - Fix docs: event types (finishRun→completeRun), staging URLs, roadmap status - Fix runtime README: state machine diagram, events list, checkpoint statuses - Fix timeout help text (60000→300000) and stale test mocks - Move react from dependencies to peerDependencies in @perstack/react Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…Event The `runtime` field was a leftover from multi-runtime support. After the refactor to local-only, `RuntimeName` is always "local" and no code reads the field. Remove it from: - Checkpoint.metadata.runtime - RuntimeEventPayloads.initializeRuntime.runtime - createNormalizedCheckpoint params - createRuntimeInitEvent params - RuntimeName type export (now unused) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Delete unused runtime-name.ts (no consumers after runtime field removal) - Unexport getEventsByRun (only used internally in filesystem package) - Remove stale ts-dedent and perstack from knip ignore lists Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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
--runtimeflags from e2e testsChanges
Deleted (8 files)
docs/using-experts/multi-runtime.mddocs/operating-experts/storage-backends.mde2e/perstack-cli/runtime-selection.test.tse2e/perstack-cli/docker-security.test.tse2e/perstack-cli/docker-attack-scenarios.test.tse2e/perstack-runtime/storage-behavior.test.tse2e/experts/docker-security.tomle2e/experts/docker-attack-scenarios.tomlModified (22 files)
--runtimeflags and runtime availability checksStats
Test plan
pnpm build- 28/28 packages passpnpm typecheck- 28/28 packages passNext steps
This is phase 1 (docs/e2e only). Implementation changes will follow in a separate PR:
@perstack/runner, storage packages--runtimeCLI option🤖 Generated with Claude Code