@jackwener/opencli 1.5.6 → 1.5.7
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/CHANGELOG.md +26 -0
- package/README.md +4 -2
- package/README.zh-CN.md +4 -1
- package/SKILL.md +879 -0
- package/dist/browser/cdp.d.ts +1 -0
- package/dist/browser/cdp.js +30 -27
- package/dist/browser/daemon-client.d.ts +7 -1
- package/dist/browser/daemon-client.js +3 -0
- package/dist/browser/dom-helpers.js +1 -0
- package/dist/browser/dom-helpers.test.js +14 -1
- package/dist/browser/mcp.js +18 -13
- package/dist/browser/page.js +22 -2
- package/dist/browser/page.test.d.ts +1 -0
- package/dist/browser/page.test.js +44 -0
- package/dist/browser/stealth.js +198 -0
- package/dist/browser/stealth.test.d.ts +1 -0
- package/dist/browser/stealth.test.js +134 -0
- package/dist/browser.test.js +1 -1
- package/dist/build-manifest.d.ts +1 -0
- package/dist/build-manifest.js +5 -1
- package/dist/build-manifest.test.js +2 -0
- package/dist/cli-manifest.json +544 -137
- package/dist/cli.js +20 -3
- package/dist/clis/antigravity/serve.d.ts +1 -1
- package/dist/clis/antigravity/serve.js +5 -8
- package/dist/clis/bilibili/subtitle.js +4 -0
- package/dist/clis/bilibili/subtitle.test.d.ts +1 -0
- package/dist/clis/bilibili/subtitle.test.js +48 -0
- package/dist/clis/chatwise/ask.js +0 -2
- package/dist/clis/chatwise/export.js +0 -2
- package/dist/clis/chatwise/history.js +0 -2
- package/dist/clis/chatwise/model.js +0 -2
- package/dist/clis/chatwise/new.js +1 -2
- package/dist/clis/chatwise/read.js +0 -2
- package/dist/clis/chatwise/screenshot.js +1 -2
- package/dist/clis/chatwise/send.js +0 -2
- package/dist/clis/chatwise/status.js +1 -2
- package/dist/clis/ctrip/search.d.ts +13 -0
- package/dist/clis/ctrip/search.js +73 -48
- package/dist/clis/ctrip/search.test.d.ts +1 -0
- package/dist/clis/ctrip/search.test.js +64 -0
- package/dist/clis/douyin/_shared/sts2.js +8 -2
- package/dist/clis/douyin/_shared/sts2.test.d.ts +1 -0
- package/dist/clis/douyin/_shared/sts2.test.js +27 -0
- package/dist/clis/douyin/activities.js +4 -2
- package/dist/clis/douyin/activities.test.js +34 -1
- package/dist/clis/douyin/collections.js +1 -1
- package/dist/clis/douyin/collections.test.js +24 -2
- package/dist/clis/douyin/draft.d.ts +8 -11
- package/dist/clis/douyin/draft.js +302 -185
- package/dist/clis/douyin/draft.test.d.ts +1 -1
- package/dist/clis/douyin/draft.test.js +357 -2
- package/dist/clis/douyin/hashtag.js +9 -2
- package/dist/clis/douyin/hashtag.test.js +35 -2
- package/dist/clis/douyin/profile.js +1 -1
- package/dist/clis/douyin/profile.test.js +36 -1
- package/dist/clis/douyin/videos.js +22 -5
- package/dist/clis/douyin/videos.test.js +45 -2
- package/dist/clis/facebook/search.test.d.ts +5 -0
- package/dist/clis/facebook/search.test.js +60 -0
- package/dist/clis/facebook/search.yaml +4 -3
- package/dist/clis/instagram/download.d.ts +16 -0
- package/dist/clis/instagram/download.js +225 -0
- package/dist/clis/instagram/download.test.d.ts +1 -0
- package/dist/clis/instagram/download.test.js +118 -0
- package/dist/clis/notebooklm/bind-current.d.ts +1 -0
- package/dist/clis/notebooklm/bind-current.js +29 -0
- package/dist/clis/notebooklm/bind-current.test.d.ts +1 -0
- package/dist/clis/notebooklm/bind-current.test.js +35 -0
- package/dist/clis/notebooklm/binding.test.d.ts +1 -0
- package/dist/clis/notebooklm/binding.test.js +44 -0
- package/dist/clis/notebooklm/compat.test.d.ts +3 -0
- package/dist/clis/notebooklm/compat.test.js +16 -0
- package/dist/clis/notebooklm/current.d.ts +1 -0
- package/dist/clis/notebooklm/current.js +28 -0
- package/dist/clis/notebooklm/get.d.ts +1 -0
- package/dist/clis/notebooklm/get.js +37 -0
- package/dist/clis/notebooklm/history.d.ts +1 -0
- package/dist/clis/notebooklm/history.js +25 -0
- package/dist/clis/notebooklm/history.test.d.ts +1 -0
- package/dist/clis/notebooklm/history.test.js +58 -0
- package/dist/clis/notebooklm/list.d.ts +1 -0
- package/dist/clis/notebooklm/list.js +35 -0
- package/dist/clis/notebooklm/note-list.d.ts +1 -0
- package/dist/clis/notebooklm/note-list.js +28 -0
- package/dist/clis/notebooklm/note-list.test.d.ts +1 -0
- package/dist/clis/notebooklm/note-list.test.js +56 -0
- package/dist/clis/notebooklm/notes-get.d.ts +1 -0
- package/dist/clis/notebooklm/notes-get.js +47 -0
- package/dist/clis/notebooklm/notes-get.test.d.ts +1 -0
- package/dist/clis/notebooklm/notes-get.test.js +72 -0
- package/dist/clis/notebooklm/rpc.d.ts +36 -0
- package/dist/clis/notebooklm/rpc.js +189 -0
- package/dist/clis/notebooklm/rpc.test.d.ts +1 -0
- package/dist/clis/notebooklm/rpc.test.js +105 -0
- package/dist/clis/notebooklm/shared.d.ts +87 -0
- package/dist/clis/notebooklm/shared.js +3 -0
- package/dist/clis/notebooklm/source-fulltext.d.ts +1 -0
- package/dist/clis/notebooklm/source-fulltext.js +44 -0
- package/dist/clis/notebooklm/source-fulltext.test.d.ts +1 -0
- package/dist/clis/notebooklm/source-fulltext.test.js +106 -0
- package/dist/clis/notebooklm/source-get.d.ts +1 -0
- package/dist/clis/notebooklm/source-get.js +40 -0
- package/dist/clis/notebooklm/source-get.test.d.ts +1 -0
- package/dist/clis/notebooklm/source-get.test.js +84 -0
- package/dist/clis/notebooklm/source-guide.d.ts +1 -0
- package/dist/clis/notebooklm/source-guide.js +44 -0
- package/dist/clis/notebooklm/source-guide.test.d.ts +1 -0
- package/dist/clis/notebooklm/source-guide.test.js +104 -0
- package/dist/clis/notebooklm/source-list.d.ts +1 -0
- package/dist/clis/notebooklm/source-list.js +30 -0
- package/dist/clis/notebooklm/status.d.ts +1 -0
- package/dist/clis/notebooklm/status.js +31 -0
- package/dist/clis/notebooklm/summary.d.ts +1 -0
- package/dist/clis/notebooklm/summary.js +30 -0
- package/dist/clis/notebooklm/summary.test.d.ts +1 -0
- package/dist/clis/notebooklm/summary.test.js +78 -0
- package/dist/clis/notebooklm/utils.d.ts +37 -0
- package/dist/clis/notebooklm/utils.js +739 -0
- package/dist/clis/notebooklm/utils.test.d.ts +1 -0
- package/dist/clis/notebooklm/utils.test.js +390 -0
- package/dist/clis/substack/utils.d.ts +4 -0
- package/dist/clis/substack/utils.js +8 -2
- package/dist/clis/substack/utils.test.d.ts +1 -0
- package/dist/clis/substack/utils.test.js +46 -0
- package/dist/clis/v2ex/hot.yaml +4 -1
- package/dist/clis/v2ex/latest.yaml +4 -1
- package/dist/clis/v2ex/topic.yaml +6 -1
- package/dist/clis/weixin/download.d.ts +9 -0
- package/dist/clis/weixin/download.js +76 -6
- package/dist/clis/weread/book.js +108 -2
- package/dist/clis/weread/commands.test.js +262 -152
- package/dist/clis/weread/utils.d.ts +10 -0
- package/dist/clis/weread/utils.js +27 -7
- package/dist/clis/xiaohongshu/comments.d.ts +3 -0
- package/dist/clis/xiaohongshu/comments.js +76 -17
- package/dist/clis/xiaohongshu/comments.test.js +70 -9
- package/dist/clis/xiaohongshu/download.d.ts +4 -1
- package/dist/clis/xiaohongshu/download.js +83 -22
- package/dist/clis/xiaohongshu/download.test.d.ts +1 -0
- package/dist/clis/xiaohongshu/download.test.js +75 -0
- package/dist/clis/xiaohongshu/note-helpers.d.ts +12 -0
- package/dist/clis/xiaohongshu/note-helpers.js +23 -0
- package/dist/clis/xiaohongshu/note.d.ts +7 -0
- package/dist/clis/xiaohongshu/note.js +76 -0
- package/dist/clis/xiaohongshu/note.test.d.ts +1 -0
- package/dist/clis/xiaohongshu/note.test.js +136 -0
- package/dist/clis/xiaohongshu/search.js +9 -0
- package/dist/clis/xiaohongshu/search.test.js +10 -4
- package/dist/clis/youtube/search.js +57 -17
- package/dist/clis/zhihu/question.js +19 -17
- package/dist/clis/zhihu/question.test.d.ts +1 -0
- package/dist/clis/zhihu/question.test.js +54 -0
- package/dist/commanderAdapter.js +9 -0
- package/dist/commanderAdapter.test.js +25 -0
- package/dist/commands/daemon.d.ts +9 -0
- package/dist/commands/daemon.js +124 -0
- package/dist/commands/daemon.test.d.ts +1 -0
- package/dist/commands/daemon.test.js +185 -0
- package/dist/completion.js +3 -1
- package/dist/constants.d.ts +2 -0
- package/dist/constants.js +2 -0
- package/dist/daemon.d.ts +1 -1
- package/dist/daemon.js +25 -14
- package/dist/daemon.test.d.ts +1 -0
- package/dist/daemon.test.js +65 -0
- package/dist/discovery.d.ts +9 -0
- package/dist/discovery.js +47 -2
- package/dist/electron-apps.d.ts +29 -0
- package/dist/electron-apps.js +65 -0
- package/dist/electron-apps.test.d.ts +1 -0
- package/dist/electron-apps.test.js +43 -0
- package/dist/engine.test.js +41 -9
- package/dist/execution.js +20 -16
- package/dist/idle-manager.d.ts +19 -0
- package/dist/idle-manager.js +54 -0
- package/dist/launcher.d.ts +36 -0
- package/dist/launcher.js +152 -0
- package/dist/launcher.test.d.ts +1 -0
- package/dist/launcher.test.js +57 -0
- package/dist/main.js +3 -3
- package/dist/registry.d.ts +1 -0
- package/dist/registry.js +31 -3
- package/dist/registry.test.js +13 -0
- package/dist/runtime.d.ts +5 -3
- package/dist/runtime.js +12 -5
- package/dist/serialization.d.ts +1 -0
- package/dist/serialization.js +3 -0
- package/dist/serialization.test.js +17 -1
- package/dist/tui.d.ts +7 -0
- package/dist/tui.js +52 -0
- package/dist/tui.test.d.ts +1 -0
- package/dist/tui.test.js +19 -0
- package/dist/weixin-download.test.js +14 -0
- package/docs/.vitepress/config.mts +1 -0
- package/docs/adapters/browser/notebooklm.md +69 -0
- package/docs/adapters/browser/xiaohongshu.md +19 -10
- package/docs/adapters/index.md +67 -66
- package/docs/guide/browser-bridge.md +12 -0
- package/docs/guide/troubleshooting.md +9 -4
- package/docs/superpowers/plans/2026-03-31-daemon-lifecycle-redesign.md +857 -0
- package/docs/superpowers/specs/2026-03-31-daemon-lifecycle-redesign.md +208 -0
- package/docs/zh/guide/browser-bridge.md +12 -0
- package/extension/dist/background.js +794 -513
- package/extension/src/background.test.ts +202 -2
- package/extension/src/background.ts +174 -10
- package/extension/src/cdp.ts +12 -0
- package/extension/src/protocol.ts +7 -5
- package/package.json +1 -1
- package/src/browser/cdp.ts +24 -17
- package/src/browser/daemon-client.ts +7 -1
- package/src/browser/dom-helpers.test.ts +15 -1
- package/src/browser/dom-helpers.ts +1 -0
- package/src/browser/mcp.ts +18 -13
- package/src/browser/page.test.ts +58 -0
- package/src/browser/page.ts +18 -2
- package/src/browser/stealth.test.ts +153 -0
- package/src/browser/stealth.ts +198 -0
- package/src/browser.test.ts +1 -1
- package/src/build-manifest.test.ts +2 -0
- package/src/build-manifest.ts +6 -1
- package/src/cli.ts +21 -3
- package/src/clis/antigravity/SKILL.md +3 -12
- package/src/clis/antigravity/serve.ts +5 -10
- package/src/clis/bilibili/subtitle.test.ts +60 -0
- package/src/clis/bilibili/subtitle.ts +4 -0
- package/src/clis/chatwise/ask.ts +0 -2
- package/src/clis/chatwise/export.ts +0 -2
- package/src/clis/chatwise/history.ts +0 -2
- package/src/clis/chatwise/model.ts +0 -2
- package/src/clis/chatwise/new.ts +1 -2
- package/src/clis/chatwise/read.ts +0 -2
- package/src/clis/chatwise/screenshot.ts +1 -2
- package/src/clis/chatwise/send.ts +0 -2
- package/src/clis/chatwise/status.ts +1 -2
- package/src/clis/ctrip/search.test.ts +73 -0
- package/src/clis/ctrip/search.ts +97 -47
- package/src/clis/douyin/_shared/sts2.test.ts +31 -0
- package/src/clis/douyin/_shared/sts2.ts +11 -3
- package/src/clis/douyin/activities.test.ts +41 -1
- package/src/clis/douyin/activities.ts +12 -3
- package/src/clis/douyin/collections.test.ts +35 -2
- package/src/clis/douyin/collections.ts +1 -1
- package/src/clis/douyin/draft.test.ts +444 -2
- package/src/clis/douyin/draft.ts +382 -218
- package/src/clis/douyin/hashtag.test.ts +42 -2
- package/src/clis/douyin/hashtag.ts +11 -3
- package/src/clis/douyin/profile.test.ts +43 -1
- package/src/clis/douyin/profile.ts +9 -2
- package/src/clis/douyin/videos.test.ts +52 -2
- package/src/clis/douyin/videos.ts +49 -15
- package/src/clis/facebook/search.test.ts +70 -0
- package/src/clis/facebook/search.yaml +4 -3
- package/src/clis/instagram/download.test.ts +159 -0
- package/src/clis/instagram/download.ts +286 -0
- package/src/clis/notebooklm/bind-current.test.ts +43 -0
- package/src/clis/notebooklm/bind-current.ts +36 -0
- package/src/clis/notebooklm/binding.test.ts +53 -0
- package/src/clis/notebooklm/compat.test.ts +19 -0
- package/src/clis/notebooklm/current.ts +38 -0
- package/src/clis/notebooklm/get.ts +53 -0
- package/src/clis/notebooklm/history.test.ts +70 -0
- package/src/clis/notebooklm/history.ts +36 -0
- package/src/clis/notebooklm/list.ts +40 -0
- package/src/clis/notebooklm/note-list.test.ts +64 -0
- package/src/clis/notebooklm/note-list.ts +42 -0
- package/src/clis/notebooklm/notes-get.test.ts +88 -0
- package/src/clis/notebooklm/notes-get.ts +67 -0
- package/src/clis/notebooklm/rpc.test.ts +126 -0
- package/src/clis/notebooklm/rpc.ts +286 -0
- package/src/clis/notebooklm/shared.ts +98 -0
- package/src/clis/notebooklm/source-fulltext.test.ts +123 -0
- package/src/clis/notebooklm/source-fulltext.ts +69 -0
- package/src/clis/notebooklm/source-get.test.ts +100 -0
- package/src/clis/notebooklm/source-get.ts +60 -0
- package/src/clis/notebooklm/source-guide.test.ts +121 -0
- package/src/clis/notebooklm/source-guide.ts +69 -0
- package/src/clis/notebooklm/source-list.ts +45 -0
- package/src/clis/notebooklm/status.ts +34 -0
- package/src/clis/notebooklm/summary.test.ts +94 -0
- package/src/clis/notebooklm/summary.ts +45 -0
- package/src/clis/notebooklm/utils.test.ts +446 -0
- package/src/clis/notebooklm/utils.ts +893 -0
- package/src/clis/substack/utils.test.ts +54 -0
- package/src/clis/substack/utils.ts +10 -2
- package/src/clis/v2ex/hot.yaml +4 -1
- package/src/clis/v2ex/latest.yaml +4 -1
- package/src/clis/v2ex/topic.yaml +6 -1
- package/src/clis/weixin/download.ts +95 -6
- package/src/clis/weread/book.ts +142 -2
- package/src/clis/weread/commands.test.ts +314 -154
- package/src/clis/weread/utils.ts +33 -4
- package/src/clis/xiaohongshu/comments.test.ts +85 -9
- package/src/clis/xiaohongshu/comments.ts +76 -17
- package/src/clis/xiaohongshu/download.test.ts +96 -0
- package/src/clis/xiaohongshu/download.ts +83 -22
- package/src/clis/xiaohongshu/note-helpers.ts +25 -0
- package/src/clis/xiaohongshu/note.test.ts +164 -0
- package/src/clis/xiaohongshu/note.ts +86 -0
- package/src/clis/xiaohongshu/search.test.ts +11 -4
- package/src/clis/xiaohongshu/search.ts +13 -0
- package/src/clis/youtube/search.ts +57 -17
- package/src/clis/zhihu/question.test.ts +71 -0
- package/src/clis/zhihu/question.ts +27 -15
- package/src/commanderAdapter.test.ts +30 -0
- package/src/commanderAdapter.ts +7 -0
- package/src/commands/daemon.test.ts +238 -0
- package/src/commands/daemon.ts +135 -0
- package/src/completion.ts +2 -1
- package/src/constants.ts +3 -0
- package/src/daemon.test.ts +88 -0
- package/src/daemon.ts +26 -14
- package/src/discovery.ts +52 -2
- package/src/electron-apps.test.ts +50 -0
- package/src/electron-apps.ts +89 -0
- package/src/engine.test.ts +45 -9
- package/src/execution.ts +24 -19
- package/src/idle-manager.ts +60 -0
- package/src/launcher.test.ts +67 -0
- package/src/launcher.ts +185 -0
- package/src/main.ts +3 -2
- package/src/registry.test.ts +15 -0
- package/src/registry.ts +32 -3
- package/src/runtime.ts +13 -7
- package/src/serialization.test.ts +19 -1
- package/src/serialization.ts +2 -0
- package/src/tui.test.ts +23 -0
- package/src/tui.ts +65 -0
- package/src/weixin-download.test.ts +27 -0
- package/tests/e2e/browser-public-extended.test.ts +6 -2
- package/chatwise-opencli.ps1 +0 -82
- package/dist/clis/chatwise/shared.d.ts +0 -2
- package/dist/clis/chatwise/shared.js +0 -6
- package/src/clis/chatwise/shared.ts +0 -8
|
@@ -0,0 +1,65 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* Electron app registry — maps site names to launch metadata.
|
|
3
|
+
*
|
|
4
|
+
* Builtin apps are defined here. User-defined apps are loaded
|
|
5
|
+
* from ~/.opencli/apps.yaml (additive only, does not override builtins).
|
|
6
|
+
*/
|
|
7
|
+
import * as fs from 'node:fs';
|
|
8
|
+
import * as path from 'node:path';
|
|
9
|
+
import * as os from 'node:os';
|
|
10
|
+
import yaml from 'js-yaml';
|
|
11
|
+
export const builtinApps = {
|
|
12
|
+
cursor: { port: 9226, processName: 'Cursor', bundleId: 'com.todesktop.runtime.Cursor', displayName: 'Cursor' },
|
|
13
|
+
codex: { port: 9222, processName: 'Codex', bundleId: 'com.openai.codex', displayName: 'Codex' },
|
|
14
|
+
chatwise: { port: 9228, processName: 'ChatWise', bundleId: 'com.chatwise.app', displayName: 'ChatWise' },
|
|
15
|
+
notion: { port: 9230, processName: 'Notion', bundleId: 'notion.id', displayName: 'Notion' },
|
|
16
|
+
'discord-app': { port: 9232, processName: 'Discord', bundleId: 'com.discord.app', displayName: 'Discord' },
|
|
17
|
+
'doubao-app': { port: 9225, processName: 'Doubao', bundleId: 'com.volcengine.doubao', displayName: 'Doubao' },
|
|
18
|
+
antigravity: { port: 9234, processName: 'Antigravity', bundleId: 'dev.antigravity.app', displayName: 'Antigravity' },
|
|
19
|
+
chatgpt: { port: 9236, processName: 'ChatGPT', bundleId: 'com.openai.chat', displayName: 'ChatGPT' },
|
|
20
|
+
};
|
|
21
|
+
/** Merge builtin + user-defined apps. User entries are additive only. */
|
|
22
|
+
export function loadApps(userApps) {
|
|
23
|
+
const merged = { ...builtinApps };
|
|
24
|
+
if (userApps) {
|
|
25
|
+
for (const [name, entry] of Object.entries(userApps)) {
|
|
26
|
+
if (!(name in merged)) {
|
|
27
|
+
merged[name] = entry;
|
|
28
|
+
}
|
|
29
|
+
}
|
|
30
|
+
}
|
|
31
|
+
return merged;
|
|
32
|
+
}
|
|
33
|
+
let _apps = null;
|
|
34
|
+
function ensureLoaded() {
|
|
35
|
+
if (_apps)
|
|
36
|
+
return _apps;
|
|
37
|
+
let userApps;
|
|
38
|
+
try {
|
|
39
|
+
const yamlPath = path.join(os.homedir(), '.opencli', 'apps.yaml');
|
|
40
|
+
if (fs.existsSync(yamlPath)) {
|
|
41
|
+
const content = fs.readFileSync(yamlPath, 'utf-8');
|
|
42
|
+
const parsed = yaml.load(content);
|
|
43
|
+
userApps = parsed?.apps;
|
|
44
|
+
}
|
|
45
|
+
}
|
|
46
|
+
catch {
|
|
47
|
+
// Silently ignore malformed user config
|
|
48
|
+
}
|
|
49
|
+
_apps = loadApps(userApps);
|
|
50
|
+
return _apps;
|
|
51
|
+
}
|
|
52
|
+
export function getElectronApp(site) {
|
|
53
|
+
return ensureLoaded()[site];
|
|
54
|
+
}
|
|
55
|
+
export function isElectronApp(site) {
|
|
56
|
+
return site in ensureLoaded();
|
|
57
|
+
}
|
|
58
|
+
/** Get all registered apps (builtin + user-defined). */
|
|
59
|
+
export function getAllElectronApps() {
|
|
60
|
+
return ensureLoaded();
|
|
61
|
+
}
|
|
62
|
+
/** Reset loaded apps (for testing). */
|
|
63
|
+
export function _resetRegistry() {
|
|
64
|
+
_apps = null;
|
|
65
|
+
}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
export {};
|
|
@@ -0,0 +1,43 @@
|
|
|
1
|
+
import { describe, it, expect } from 'vitest';
|
|
2
|
+
import { getElectronApp, isElectronApp, loadApps } from './electron-apps.js';
|
|
3
|
+
describe('electron-apps registry', () => {
|
|
4
|
+
it('returns builtin app entry for cursor', () => {
|
|
5
|
+
const app = getElectronApp('cursor');
|
|
6
|
+
expect(app).toBeDefined();
|
|
7
|
+
expect(app.port).toBe(9226);
|
|
8
|
+
expect(app.processName).toBe('Cursor');
|
|
9
|
+
});
|
|
10
|
+
it('returns builtin app entry for codex', () => {
|
|
11
|
+
const app = getElectronApp('codex');
|
|
12
|
+
expect(app).toBeDefined();
|
|
13
|
+
expect(app.port).toBe(9222);
|
|
14
|
+
});
|
|
15
|
+
it('returns undefined for non-Electron sites', () => {
|
|
16
|
+
expect(getElectronApp('bilibili')).toBeUndefined();
|
|
17
|
+
expect(getElectronApp('hackernews')).toBeUndefined();
|
|
18
|
+
});
|
|
19
|
+
it('isElectronApp returns true for registered apps', () => {
|
|
20
|
+
expect(isElectronApp('cursor')).toBe(true);
|
|
21
|
+
expect(isElectronApp('codex')).toBe(true);
|
|
22
|
+
expect(isElectronApp('chatwise')).toBe(true);
|
|
23
|
+
});
|
|
24
|
+
it('isElectronApp returns false for non-Electron sites', () => {
|
|
25
|
+
expect(isElectronApp('bilibili')).toBe(false);
|
|
26
|
+
expect(isElectronApp('unknown-app')).toBe(false);
|
|
27
|
+
});
|
|
28
|
+
it('loadApps merges user config additively', () => {
|
|
29
|
+
const apps = loadApps({
|
|
30
|
+
myapp: { port: 9234, processName: 'MyApp' },
|
|
31
|
+
});
|
|
32
|
+
expect(apps.myapp).toBeDefined();
|
|
33
|
+
expect(apps.myapp.port).toBe(9234);
|
|
34
|
+
// Builtins still present
|
|
35
|
+
expect(apps.cursor).toBeDefined();
|
|
36
|
+
});
|
|
37
|
+
it('loadApps does not override builtin entries', () => {
|
|
38
|
+
const apps = loadApps({
|
|
39
|
+
cursor: { port: 9999, processName: 'FakeCursor' },
|
|
40
|
+
});
|
|
41
|
+
expect(apps.cursor.port).toBe(9226); // Builtin wins
|
|
42
|
+
});
|
|
43
|
+
});
|
package/dist/engine.test.js
CHANGED
|
@@ -1,5 +1,5 @@
|
|
|
1
1
|
import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
|
|
2
|
-
import { discoverClis, discoverPlugins, PLUGINS_DIR } from './discovery.js';
|
|
2
|
+
import { discoverClis, discoverPlugins, ensureUserCliCompatShims, PLUGINS_DIR } from './discovery.js';
|
|
3
3
|
import { executeCommand } from './execution.js';
|
|
4
4
|
import { getRegistry, cli, Strategy } from './registry.js';
|
|
5
5
|
import { clearAllHooks, onAfterExecute } from './hooks.js';
|
|
@@ -71,6 +71,36 @@ cli({
|
|
|
71
71
|
await fs.promises.rm(tempBuildRoot, { recursive: true, force: true });
|
|
72
72
|
}
|
|
73
73
|
});
|
|
74
|
+
it('loads legacy user TS CLI modules via compatibility shims', async () => {
|
|
75
|
+
const tempOpencliRoot = await fs.promises.mkdtemp(path.join(os.tmpdir(), 'opencli-user-clis-'));
|
|
76
|
+
const userClisDir = path.join(tempOpencliRoot, 'clis');
|
|
77
|
+
const siteDir = path.join(userClisDir, 'legacy-site');
|
|
78
|
+
const commandPath = path.join(siteDir, 'hello.ts');
|
|
79
|
+
try {
|
|
80
|
+
await ensureUserCliCompatShims(tempOpencliRoot);
|
|
81
|
+
await fs.promises.mkdir(siteDir, { recursive: true });
|
|
82
|
+
await fs.promises.writeFile(commandPath, `
|
|
83
|
+
import { cli, Strategy } from '../../registry';
|
|
84
|
+
import { CommandExecutionError } from '../../errors';
|
|
85
|
+
|
|
86
|
+
cli({
|
|
87
|
+
site: 'legacy-site',
|
|
88
|
+
name: 'hello',
|
|
89
|
+
description: 'hello command',
|
|
90
|
+
strategy: Strategy.PUBLIC,
|
|
91
|
+
browser: false,
|
|
92
|
+
func: async () => [{ ok: true, errorName: new CommandExecutionError('boom').name }],
|
|
93
|
+
});
|
|
94
|
+
`);
|
|
95
|
+
await discoverClis(userClisDir);
|
|
96
|
+
const cmd = getRegistry().get('legacy-site/hello');
|
|
97
|
+
expect(cmd).toBeDefined();
|
|
98
|
+
await expect(executeCommand(cmd, {})).resolves.toEqual([{ ok: true, errorName: 'CommandExecutionError' }]);
|
|
99
|
+
}
|
|
100
|
+
finally {
|
|
101
|
+
await fs.promises.rm(tempOpencliRoot, { recursive: true, force: true });
|
|
102
|
+
}
|
|
103
|
+
});
|
|
74
104
|
});
|
|
75
105
|
describe('discoverPlugins', () => {
|
|
76
106
|
const testPluginDir = path.join(PLUGINS_DIR, '__test-plugin__');
|
|
@@ -245,21 +275,23 @@ describe('executeCommand', () => {
|
|
|
245
275
|
expect(seen[0].error.message).toBe('boom');
|
|
246
276
|
expect(typeof seen[0].finishedAt).toBe('number');
|
|
247
277
|
});
|
|
248
|
-
it('
|
|
278
|
+
it('uses launcher for registered Electron apps (chatwise)', async () => {
|
|
279
|
+
// Mock the launcher to return a fake endpoint (avoids real HTTP/process calls)
|
|
280
|
+
const launcher = await import('./launcher.js');
|
|
281
|
+
const spy = vi.spyOn(launcher, 'resolveElectronEndpoint')
|
|
282
|
+
.mockResolvedValue('http://127.0.0.1:9228');
|
|
249
283
|
const cmd = cli({
|
|
250
284
|
site: 'chatwise',
|
|
251
285
|
name: 'status',
|
|
252
286
|
description: 'chatwise status',
|
|
253
287
|
browser: true,
|
|
254
288
|
strategy: Strategy.PUBLIC,
|
|
255
|
-
requiredEnv: [
|
|
256
|
-
{
|
|
257
|
-
name: 'OPENCLI_CDP_ENDPOINT',
|
|
258
|
-
help: 'Set OPENCLI_CDP_ENDPOINT before running chatwise commands.',
|
|
259
|
-
},
|
|
260
|
-
],
|
|
261
289
|
func: async () => [{ ok: true }],
|
|
262
290
|
});
|
|
263
|
-
|
|
291
|
+
// CDPBridge.connect() will fail (no actual CDP server), but the launcher
|
|
292
|
+
// should have been called with 'chatwise'.
|
|
293
|
+
await expect(executeCommand(cmd, {})).rejects.toThrow();
|
|
294
|
+
expect(spy).toHaveBeenCalledWith('chatwise');
|
|
295
|
+
spy.mockRestore();
|
|
264
296
|
});
|
|
265
297
|
});
|
package/dist/execution.js
CHANGED
|
@@ -18,6 +18,8 @@ import { getBrowserFactory, browserSession, runWithTimeout, DEFAULT_BROWSER_COMM
|
|
|
18
18
|
import { emitHook } from './hooks.js';
|
|
19
19
|
import { checkDaemonStatus } from './browser/discover.js';
|
|
20
20
|
import { log } from './logger.js';
|
|
21
|
+
import { isElectronApp } from './electron-apps.js';
|
|
22
|
+
import { resolveElectronEndpoint } from './launcher.js';
|
|
21
23
|
const _loadedModules = new Set();
|
|
22
24
|
export function coerceAndValidateArgs(cmdArgs, kwargs) {
|
|
23
25
|
const result = { ...kwargs };
|
|
@@ -147,21 +149,25 @@ export async function executeCommand(cmd, rawKwargs, debug = false) {
|
|
|
147
149
|
let result;
|
|
148
150
|
try {
|
|
149
151
|
if (shouldUseBrowserSession(cmd)) {
|
|
150
|
-
|
|
151
|
-
|
|
152
|
-
|
|
153
|
-
|
|
154
|
-
|
|
155
|
-
|
|
156
|
-
|
|
157
|
-
|
|
158
|
-
|
|
159
|
-
|
|
160
|
-
|
|
161
|
-
'
|
|
152
|
+
const electron = isElectronApp(cmd.site);
|
|
153
|
+
let cdpEndpoint;
|
|
154
|
+
if (electron) {
|
|
155
|
+
// Electron apps: auto-detect, prompt restart if needed, launch with CDP
|
|
156
|
+
cdpEndpoint = await resolveElectronEndpoint(cmd.site);
|
|
157
|
+
}
|
|
158
|
+
else {
|
|
159
|
+
// Browser Bridge: fail-fast when daemon is up but extension is missing.
|
|
160
|
+
// 300ms timeout avoids a full 2s wait on cold-start.
|
|
161
|
+
const status = await checkDaemonStatus({ timeout: 300 });
|
|
162
|
+
if (status.running && !status.extensionConnected) {
|
|
163
|
+
throw new BrowserConnectError('Browser Bridge extension not connected', 'Install the Browser Bridge:\n' +
|
|
164
|
+
' 1. Download: https://github.com/jackwener/opencli/releases\n' +
|
|
165
|
+
' 2. chrome://extensions → Developer Mode → Load unpacked\n' +
|
|
166
|
+
' Then run: opencli doctor');
|
|
167
|
+
}
|
|
162
168
|
}
|
|
163
169
|
ensureRequiredEnv(cmd);
|
|
164
|
-
const BrowserFactory = getBrowserFactory();
|
|
170
|
+
const BrowserFactory = getBrowserFactory(cmd.site);
|
|
165
171
|
result = await browserSession(BrowserFactory, async (page) => {
|
|
166
172
|
const preNavUrl = resolvePreNav(cmd);
|
|
167
173
|
if (preNavUrl) {
|
|
@@ -172,8 +178,6 @@ export async function executeCommand(cmd, rawKwargs, debug = false) {
|
|
|
172
178
|
}
|
|
173
179
|
else {
|
|
174
180
|
try {
|
|
175
|
-
// goto() already includes smart DOM-settle detection (waitForDomStable).
|
|
176
|
-
// No additional fixed sleep needed.
|
|
177
181
|
await page.goto(preNavUrl);
|
|
178
182
|
}
|
|
179
183
|
catch (err) {
|
|
@@ -186,7 +190,7 @@ export async function executeCommand(cmd, rawKwargs, debug = false) {
|
|
|
186
190
|
timeout: cmd.timeoutSeconds ?? DEFAULT_BROWSER_COMMAND_TIMEOUT,
|
|
187
191
|
label: fullName(cmd),
|
|
188
192
|
});
|
|
189
|
-
}, { workspace: `site:${cmd.site}
|
|
193
|
+
}, { workspace: `site:${cmd.site}`, cdpEndpoint });
|
|
190
194
|
}
|
|
191
195
|
else {
|
|
192
196
|
// Non-browser commands: apply timeout only when explicitly configured.
|
|
@@ -0,0 +1,19 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* Manages daemon idle timeout with dual-condition logic:
|
|
3
|
+
* exits only when BOTH CLI is idle AND Extension is disconnected.
|
|
4
|
+
*/
|
|
5
|
+
export declare class IdleManager {
|
|
6
|
+
private _timer;
|
|
7
|
+
private _lastCliRequestTime;
|
|
8
|
+
private _extensionConnected;
|
|
9
|
+
private _timeoutMs;
|
|
10
|
+
private _onExit;
|
|
11
|
+
constructor(timeoutMs: number, onExit: () => void);
|
|
12
|
+
get lastCliRequestTime(): number;
|
|
13
|
+
/** Call when an HTTP request arrives from CLI */
|
|
14
|
+
onCliRequest(): void;
|
|
15
|
+
/** Call when Extension WebSocket connects or disconnects */
|
|
16
|
+
setExtensionConnected(connected: boolean): void;
|
|
17
|
+
private _clearTimer;
|
|
18
|
+
private _resetTimer;
|
|
19
|
+
}
|
|
@@ -0,0 +1,54 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* Manages daemon idle timeout with dual-condition logic:
|
|
3
|
+
* exits only when BOTH CLI is idle AND Extension is disconnected.
|
|
4
|
+
*/
|
|
5
|
+
export class IdleManager {
|
|
6
|
+
_timer = null;
|
|
7
|
+
_lastCliRequestTime = Date.now();
|
|
8
|
+
_extensionConnected = false;
|
|
9
|
+
_timeoutMs;
|
|
10
|
+
_onExit;
|
|
11
|
+
constructor(timeoutMs, onExit) {
|
|
12
|
+
this._timeoutMs = timeoutMs;
|
|
13
|
+
this._onExit = onExit;
|
|
14
|
+
}
|
|
15
|
+
get lastCliRequestTime() {
|
|
16
|
+
return this._lastCliRequestTime;
|
|
17
|
+
}
|
|
18
|
+
/** Call when an HTTP request arrives from CLI */
|
|
19
|
+
onCliRequest() {
|
|
20
|
+
this._lastCliRequestTime = Date.now();
|
|
21
|
+
this._resetTimer();
|
|
22
|
+
}
|
|
23
|
+
/** Call when Extension WebSocket connects or disconnects */
|
|
24
|
+
setExtensionConnected(connected) {
|
|
25
|
+
this._extensionConnected = connected;
|
|
26
|
+
if (connected) {
|
|
27
|
+
this._clearTimer();
|
|
28
|
+
}
|
|
29
|
+
else {
|
|
30
|
+
this._resetTimer();
|
|
31
|
+
}
|
|
32
|
+
}
|
|
33
|
+
_clearTimer() {
|
|
34
|
+
if (this._timer) {
|
|
35
|
+
clearTimeout(this._timer);
|
|
36
|
+
this._timer = null;
|
|
37
|
+
}
|
|
38
|
+
}
|
|
39
|
+
_resetTimer() {
|
|
40
|
+
this._clearTimer();
|
|
41
|
+
if (this._timeoutMs <= 0)
|
|
42
|
+
return;
|
|
43
|
+
if (this._extensionConnected)
|
|
44
|
+
return;
|
|
45
|
+
const elapsed = Date.now() - this._lastCliRequestTime;
|
|
46
|
+
if (elapsed >= this._timeoutMs) {
|
|
47
|
+
this._onExit();
|
|
48
|
+
return;
|
|
49
|
+
}
|
|
50
|
+
this._timer = setTimeout(() => {
|
|
51
|
+
this._onExit();
|
|
52
|
+
}, this._timeoutMs - elapsed);
|
|
53
|
+
}
|
|
54
|
+
}
|
|
@@ -0,0 +1,36 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* Electron app launcher — auto-detect, confirm, launch, and connect.
|
|
3
|
+
*
|
|
4
|
+
* Flow:
|
|
5
|
+
* 1. Probe CDP port → already running with debug? connect directly
|
|
6
|
+
* 2. Detect process → running without CDP? prompt to restart
|
|
7
|
+
* 3. Discover app path → not installed? error
|
|
8
|
+
* 4. Launch with --remote-debugging-port
|
|
9
|
+
* 5. Poll /json until ready
|
|
10
|
+
*/
|
|
11
|
+
/**
|
|
12
|
+
* Probe whether a CDP endpoint is listening on the given port.
|
|
13
|
+
* Returns true if http://127.0.0.1:{port}/json responds successfully.
|
|
14
|
+
*/
|
|
15
|
+
export declare function probeCDP(port: number, timeoutMs?: number): Promise<boolean>;
|
|
16
|
+
/**
|
|
17
|
+
* Check if a process with the given name is running.
|
|
18
|
+
* Uses pgrep on macOS/Linux.
|
|
19
|
+
*/
|
|
20
|
+
export declare function detectProcess(processName: string): boolean;
|
|
21
|
+
/**
|
|
22
|
+
* Kill a process by name. Sends SIGTERM first, then SIGKILL after grace period.
|
|
23
|
+
*/
|
|
24
|
+
export declare function killProcess(processName: string): void;
|
|
25
|
+
/**
|
|
26
|
+
* Discover the app installation path on macOS.
|
|
27
|
+
* Uses osascript to resolve the app name to a POSIX path.
|
|
28
|
+
* Returns null if the app is not installed.
|
|
29
|
+
*/
|
|
30
|
+
export declare function discoverAppPath(displayName: string): string | null;
|
|
31
|
+
/**
|
|
32
|
+
* Main entry point: resolve an Electron app to a CDP endpoint URL.
|
|
33
|
+
*
|
|
34
|
+
* Returns the endpoint URL: http://127.0.0.1:{port}
|
|
35
|
+
*/
|
|
36
|
+
export declare function resolveElectronEndpoint(site: string): Promise<string>;
|
package/dist/launcher.js
ADDED
|
@@ -0,0 +1,152 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* Electron app launcher — auto-detect, confirm, launch, and connect.
|
|
3
|
+
*
|
|
4
|
+
* Flow:
|
|
5
|
+
* 1. Probe CDP port → already running with debug? connect directly
|
|
6
|
+
* 2. Detect process → running without CDP? prompt to restart
|
|
7
|
+
* 3. Discover app path → not installed? error
|
|
8
|
+
* 4. Launch with --remote-debugging-port
|
|
9
|
+
* 5. Poll /json until ready
|
|
10
|
+
*/
|
|
11
|
+
import { execFileSync, spawn } from 'node:child_process';
|
|
12
|
+
import { request as httpRequest } from 'node:http';
|
|
13
|
+
import { getElectronApp } from './electron-apps.js';
|
|
14
|
+
import { confirmPrompt } from './tui.js';
|
|
15
|
+
import { CommandExecutionError } from './errors.js';
|
|
16
|
+
import { log } from './logger.js';
|
|
17
|
+
const POLL_INTERVAL_MS = 500;
|
|
18
|
+
const POLL_TIMEOUT_MS = 15_000;
|
|
19
|
+
const PROBE_TIMEOUT_MS = 2_000;
|
|
20
|
+
const KILL_GRACE_MS = 3_000;
|
|
21
|
+
/**
|
|
22
|
+
* Probe whether a CDP endpoint is listening on the given port.
|
|
23
|
+
* Returns true if http://127.0.0.1:{port}/json responds successfully.
|
|
24
|
+
*/
|
|
25
|
+
export function probeCDP(port, timeoutMs = PROBE_TIMEOUT_MS) {
|
|
26
|
+
return new Promise((resolve) => {
|
|
27
|
+
const req = httpRequest({ hostname: '127.0.0.1', port, path: '/json', method: 'GET', timeout: timeoutMs }, (res) => {
|
|
28
|
+
res.resume();
|
|
29
|
+
resolve(res.statusCode !== undefined && res.statusCode >= 200 && res.statusCode < 300);
|
|
30
|
+
});
|
|
31
|
+
req.on('error', () => resolve(false));
|
|
32
|
+
req.on('timeout', () => { req.destroy(); resolve(false); });
|
|
33
|
+
req.end();
|
|
34
|
+
});
|
|
35
|
+
}
|
|
36
|
+
/**
|
|
37
|
+
* Check if a process with the given name is running.
|
|
38
|
+
* Uses pgrep on macOS/Linux.
|
|
39
|
+
*/
|
|
40
|
+
export function detectProcess(processName) {
|
|
41
|
+
try {
|
|
42
|
+
execFileSync('pgrep', ['-x', processName], { encoding: 'utf-8', stdio: 'pipe' });
|
|
43
|
+
return true;
|
|
44
|
+
}
|
|
45
|
+
catch {
|
|
46
|
+
return false;
|
|
47
|
+
}
|
|
48
|
+
}
|
|
49
|
+
/**
|
|
50
|
+
* Kill a process by name. Sends SIGTERM first, then SIGKILL after grace period.
|
|
51
|
+
*/
|
|
52
|
+
export function killProcess(processName) {
|
|
53
|
+
try {
|
|
54
|
+
execFileSync('pkill', ['-x', processName], { stdio: 'pipe' });
|
|
55
|
+
}
|
|
56
|
+
catch {
|
|
57
|
+
// Process may have already exited
|
|
58
|
+
}
|
|
59
|
+
const deadline = Date.now() + KILL_GRACE_MS;
|
|
60
|
+
while (Date.now() < deadline) {
|
|
61
|
+
if (!detectProcess(processName))
|
|
62
|
+
return;
|
|
63
|
+
execFileSync('sleep', ['0.2'], { stdio: 'pipe' });
|
|
64
|
+
}
|
|
65
|
+
try {
|
|
66
|
+
execFileSync('pkill', ['-9', '-x', processName], { stdio: 'pipe' });
|
|
67
|
+
}
|
|
68
|
+
catch {
|
|
69
|
+
// Ignore
|
|
70
|
+
}
|
|
71
|
+
}
|
|
72
|
+
/**
|
|
73
|
+
* Discover the app installation path on macOS.
|
|
74
|
+
* Uses osascript to resolve the app name to a POSIX path.
|
|
75
|
+
* Returns null if the app is not installed.
|
|
76
|
+
*/
|
|
77
|
+
export function discoverAppPath(displayName) {
|
|
78
|
+
if (process.platform !== 'darwin') {
|
|
79
|
+
return null;
|
|
80
|
+
}
|
|
81
|
+
try {
|
|
82
|
+
const result = execFileSync('osascript', [
|
|
83
|
+
'-e', `POSIX path of (path to application "${displayName}")`,
|
|
84
|
+
], { encoding: 'utf-8', stdio: 'pipe', timeout: 5_000 });
|
|
85
|
+
return result.trim().replace(/\/$/, '');
|
|
86
|
+
}
|
|
87
|
+
catch {
|
|
88
|
+
return null;
|
|
89
|
+
}
|
|
90
|
+
}
|
|
91
|
+
function resolveExecutable(appPath, processName) {
|
|
92
|
+
return `${appPath}/Contents/MacOS/${processName}`;
|
|
93
|
+
}
|
|
94
|
+
async function pollForReady(port) {
|
|
95
|
+
const deadline = Date.now() + POLL_TIMEOUT_MS;
|
|
96
|
+
while (Date.now() < deadline) {
|
|
97
|
+
if (await probeCDP(port, 1_000))
|
|
98
|
+
return;
|
|
99
|
+
await new Promise((r) => setTimeout(r, POLL_INTERVAL_MS));
|
|
100
|
+
}
|
|
101
|
+
throw new CommandExecutionError(`App launched but CDP not available on port ${port} after ${POLL_TIMEOUT_MS / 1000}s`, 'The app may be slow to start. Try running the command again.');
|
|
102
|
+
}
|
|
103
|
+
/**
|
|
104
|
+
* Main entry point: resolve an Electron app to a CDP endpoint URL.
|
|
105
|
+
*
|
|
106
|
+
* Returns the endpoint URL: http://127.0.0.1:{port}
|
|
107
|
+
*/
|
|
108
|
+
export async function resolveElectronEndpoint(site) {
|
|
109
|
+
const app = getElectronApp(site);
|
|
110
|
+
if (!app) {
|
|
111
|
+
throw new CommandExecutionError(`No Electron app registered for site "${site}"`, 'Register the app in ~/.opencli/apps.yaml or check the site name.');
|
|
112
|
+
}
|
|
113
|
+
const { port, processName, displayName } = app;
|
|
114
|
+
const label = displayName ?? processName;
|
|
115
|
+
const endpoint = `http://127.0.0.1:${port}`;
|
|
116
|
+
// Step 1: Already running with CDP?
|
|
117
|
+
log.debug(`[launcher] Probing CDP on port ${port}...`);
|
|
118
|
+
if (await probeCDP(port)) {
|
|
119
|
+
log.debug(`[launcher] CDP already available on port ${port}`);
|
|
120
|
+
return endpoint;
|
|
121
|
+
}
|
|
122
|
+
// Step 2: Running without CDP?
|
|
123
|
+
const isRunning = detectProcess(processName);
|
|
124
|
+
if (isRunning) {
|
|
125
|
+
log.debug(`[launcher] ${label} is running but CDP not available`);
|
|
126
|
+
const confirmed = await confirmPrompt(`${label} is running but CDP is not enabled. Restart with debug port?`, true);
|
|
127
|
+
if (!confirmed) {
|
|
128
|
+
throw new CommandExecutionError(`${label} needs to be restarted with CDP enabled.`, `Manually restart: kill the app and relaunch with --remote-debugging-port=${port}`);
|
|
129
|
+
}
|
|
130
|
+
process.stderr.write(` Restarting ${label}...\n`);
|
|
131
|
+
killProcess(processName);
|
|
132
|
+
}
|
|
133
|
+
// Step 3: Discover path
|
|
134
|
+
const appPath = discoverAppPath(label);
|
|
135
|
+
if (!appPath) {
|
|
136
|
+
throw new CommandExecutionError(`Could not find ${label} on this machine.`, `Install ${label} or register a custom path in ~/.opencli/apps.yaml`);
|
|
137
|
+
}
|
|
138
|
+
// Step 4: Launch
|
|
139
|
+
const executable = resolveExecutable(appPath, processName);
|
|
140
|
+
const args = [`--remote-debugging-port=${port}`, ...(app.extraArgs ?? [])];
|
|
141
|
+
log.debug(`[launcher] Launching: ${executable} ${args.join(' ')}`);
|
|
142
|
+
const child = spawn(executable, args, {
|
|
143
|
+
detached: true,
|
|
144
|
+
stdio: 'ignore',
|
|
145
|
+
});
|
|
146
|
+
child.unref();
|
|
147
|
+
// Step 5: Poll for readiness
|
|
148
|
+
process.stderr.write(` Waiting for ${label} on port ${port}...\n`);
|
|
149
|
+
await pollForReady(port);
|
|
150
|
+
process.stderr.write(` Connected to ${label} on port ${port}.\n`);
|
|
151
|
+
return endpoint;
|
|
152
|
+
}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
export {};
|
|
@@ -0,0 +1,57 @@
|
|
|
1
|
+
import { describe, it, expect, vi, beforeEach } from 'vitest';
|
|
2
|
+
import { probeCDP, detectProcess, discoverAppPath } from './launcher.js';
|
|
3
|
+
vi.mock('node:child_process', () => ({
|
|
4
|
+
execFileSync: vi.fn(),
|
|
5
|
+
spawn: vi.fn(() => ({
|
|
6
|
+
unref: vi.fn(),
|
|
7
|
+
pid: 12345,
|
|
8
|
+
on: vi.fn(),
|
|
9
|
+
})),
|
|
10
|
+
}));
|
|
11
|
+
const cp = vi.mocked(await import('node:child_process'));
|
|
12
|
+
describe('probeCDP', () => {
|
|
13
|
+
it('returns false when CDP endpoint is unreachable', async () => {
|
|
14
|
+
const result = await probeCDP(59999, 500);
|
|
15
|
+
expect(result).toBe(false);
|
|
16
|
+
});
|
|
17
|
+
});
|
|
18
|
+
describe('detectProcess', () => {
|
|
19
|
+
beforeEach(() => {
|
|
20
|
+
vi.restoreAllMocks();
|
|
21
|
+
});
|
|
22
|
+
it('returns false when pgrep finds no process', () => {
|
|
23
|
+
cp.execFileSync.mockImplementation(() => {
|
|
24
|
+
const err = new Error('exit 1');
|
|
25
|
+
err.status = 1;
|
|
26
|
+
throw err;
|
|
27
|
+
});
|
|
28
|
+
const result = detectProcess('NonExistentApp');
|
|
29
|
+
expect(result).toBe(false);
|
|
30
|
+
});
|
|
31
|
+
it('returns true when pgrep finds a process', () => {
|
|
32
|
+
cp.execFileSync.mockReturnValue('12345\n');
|
|
33
|
+
const result = detectProcess('Cursor');
|
|
34
|
+
expect(result).toBe(true);
|
|
35
|
+
});
|
|
36
|
+
});
|
|
37
|
+
describe('discoverAppPath', () => {
|
|
38
|
+
beforeEach(() => {
|
|
39
|
+
vi.restoreAllMocks();
|
|
40
|
+
});
|
|
41
|
+
it.skipIf(process.platform !== 'darwin')('returns path when osascript succeeds', () => {
|
|
42
|
+
cp.execFileSync.mockReturnValue('/Applications/Cursor.app/\n');
|
|
43
|
+
const result = discoverAppPath('Cursor');
|
|
44
|
+
expect(result).toBe('/Applications/Cursor.app');
|
|
45
|
+
});
|
|
46
|
+
it.skipIf(process.platform !== 'darwin')('returns null when osascript fails', () => {
|
|
47
|
+
cp.execFileSync.mockImplementation(() => {
|
|
48
|
+
throw new Error('app not found');
|
|
49
|
+
});
|
|
50
|
+
const result = discoverAppPath('NonExistent');
|
|
51
|
+
expect(result).toBeNull();
|
|
52
|
+
});
|
|
53
|
+
it.skipIf(process.platform === 'darwin')('returns null on non-darwin platform', () => {
|
|
54
|
+
const result = discoverAppPath('Cursor');
|
|
55
|
+
expect(result).toBeNull();
|
|
56
|
+
});
|
|
57
|
+
});
|
package/dist/main.js
CHANGED
|
@@ -12,10 +12,9 @@ if (process.platform !== 'win32') {
|
|
|
12
12
|
cur.add(p);
|
|
13
13
|
process.env.PATH = [...cur].join(':');
|
|
14
14
|
}
|
|
15
|
-
import * as os from 'node:os';
|
|
16
15
|
import * as path from 'node:path';
|
|
17
16
|
import { fileURLToPath } from 'node:url';
|
|
18
|
-
import { discoverClis, discoverPlugins } from './discovery.js';
|
|
17
|
+
import { discoverClis, discoverPlugins, ensureUserCliCompatShims, USER_CLIS_DIR } from './discovery.js';
|
|
19
18
|
import { getCompletions } from './completion.js';
|
|
20
19
|
import { runCli } from './cli.js';
|
|
21
20
|
import { emitHook } from './hooks.js';
|
|
@@ -26,8 +25,9 @@ installNodeNetwork();
|
|
|
26
25
|
const __filename = fileURLToPath(import.meta.url);
|
|
27
26
|
const __dirname = path.dirname(__filename);
|
|
28
27
|
const BUILTIN_CLIS = path.resolve(__dirname, 'clis');
|
|
29
|
-
const USER_CLIS =
|
|
28
|
+
const USER_CLIS = USER_CLIS_DIR;
|
|
30
29
|
// Sequential: plugins must run after built-in discovery so they can override built-in commands.
|
|
30
|
+
await ensureUserCliCompatShims();
|
|
31
31
|
await discoverClis(BUILTIN_CLIS, USER_CLIS);
|
|
32
32
|
await discoverPlugins();
|
|
33
33
|
// Register exit hook: notice appears after command output (same as npm/gh/yarn)
|
package/dist/registry.d.ts
CHANGED
package/dist/registry.js
CHANGED
|
@@ -13,9 +13,11 @@ const _registry = globalThis.__opencli_registry__ ??= new Map();
|
|
|
13
13
|
export function cli(opts) {
|
|
14
14
|
const strategy = opts.strategy ?? (opts.browser === false ? Strategy.PUBLIC : Strategy.COOKIE);
|
|
15
15
|
const browser = opts.browser ?? (strategy !== Strategy.PUBLIC);
|
|
16
|
+
const aliases = normalizeAliases(opts.aliases, opts.name);
|
|
16
17
|
const cmd = {
|
|
17
18
|
site: opts.site,
|
|
18
19
|
name: opts.name,
|
|
20
|
+
aliases,
|
|
19
21
|
description: opts.description ?? '',
|
|
20
22
|
domain: opts.domain,
|
|
21
23
|
strategy,
|
|
@@ -31,8 +33,7 @@ export function cli(opts) {
|
|
|
31
33
|
replacedBy: opts.replacedBy,
|
|
32
34
|
navigateBefore: opts.navigateBefore,
|
|
33
35
|
};
|
|
34
|
-
|
|
35
|
-
_registry.set(key, cmd);
|
|
36
|
+
registerCommand(cmd);
|
|
36
37
|
return cmd;
|
|
37
38
|
}
|
|
38
39
|
export function getRegistry() {
|
|
@@ -45,5 +46,32 @@ export function strategyLabel(cmd) {
|
|
|
45
46
|
return cmd.strategy ?? Strategy.PUBLIC;
|
|
46
47
|
}
|
|
47
48
|
export function registerCommand(cmd) {
|
|
48
|
-
|
|
49
|
+
const canonicalKey = fullName(cmd);
|
|
50
|
+
const existing = _registry.get(canonicalKey);
|
|
51
|
+
if (existing) {
|
|
52
|
+
for (const [key, value] of _registry.entries()) {
|
|
53
|
+
if (value === existing && key !== canonicalKey)
|
|
54
|
+
_registry.delete(key);
|
|
55
|
+
}
|
|
56
|
+
}
|
|
57
|
+
const aliases = normalizeAliases(cmd.aliases, cmd.name);
|
|
58
|
+
cmd.aliases = aliases.length > 0 ? aliases : undefined;
|
|
59
|
+
_registry.set(canonicalKey, cmd);
|
|
60
|
+
for (const alias of aliases) {
|
|
61
|
+
_registry.set(`${cmd.site}/${alias}`, cmd);
|
|
62
|
+
}
|
|
63
|
+
}
|
|
64
|
+
function normalizeAliases(aliases, commandName) {
|
|
65
|
+
if (!Array.isArray(aliases) || aliases.length === 0)
|
|
66
|
+
return [];
|
|
67
|
+
const seen = new Set();
|
|
68
|
+
const normalized = [];
|
|
69
|
+
for (const alias of aliases) {
|
|
70
|
+
const value = typeof alias === 'string' ? alias.trim() : '';
|
|
71
|
+
if (!value || value === commandName || seen.has(value))
|
|
72
|
+
continue;
|
|
73
|
+
seen.add(value);
|
|
74
|
+
normalized.push(value);
|
|
75
|
+
}
|
|
76
|
+
return normalized;
|
|
49
77
|
}
|