Commit f55b19c
committed
refactor: architectural simplification of @browserkit-dev/core
Ten targeted simplifications to reduce complexity, dead code, and duplication:
1. Remove script-runner.ts and quickjs-emscripten dependency (-411 LOC, -1 dep)
2. Split adapter-server.ts god module into wrap-tool-call, browser-tool, mcp-resources
3. Move observability.ts to @browserkit-dev/core/testing subpath
4. DRY config types — add BrowserLaunchFields base for SessionConfig/AdapterConfig
5. Fix CLI logout dead code — wire existing cmdLogout into switch case
6. Remove duplicate minCoreVersion check from defineAdapter (kept in server.ts loadAdapter)
7. Extract buildConnectArgs helper shared by PersistentBackend and StorageStateBackend
8. Extract buildDaemonStatus helper shared by ServerHandle and status sidecar
9. Fix browser tool description — remove duplicate Actions block
10. Split adapter-utils.ts (650 LOC) into selector-, screenshot-, navigation-, auth-, form-utils
All 156 core tests + 37 HN tests + 37 LinkedIn tests remain green.
Made-with: Cursor1 parent c7f50cf commit f55b19c
20 files changed
Lines changed: 1488 additions & 1231 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
41 | 41 | | |
42 | 42 | | |
43 | 43 | | |
44 | | - | |
45 | 44 | | |
46 | 45 | | |
47 | 46 | | |
| |||
Large diffs are not rendered by default.
0 commit comments