Pake icon indicating copy to clipboard operation
Pake copied to clipboard

[Bug] --inject not work on macos

Open morganorix opened this issue 1 year ago • 4 comments

Search before asking

  • [x] 我在 issues 列表中搜索,没有找到类似的内容。 I searched in the issues and found nothing similar.

Pake version

Version of pake cli : 3.1.1

System version

Version of system : MacOs 15.2

Node.js version

Node.js v20.18.2

Minimal reproduce step

pake "https://chat.mistral.ai/chat" --name "MistralAI" --hide-title-bar --icon ./icon_512.icns --debug true --inject ./inject.js 
PakeAppOptions {
  icon: '/Users/morganorix/Documents/apps maison/mistralai/icon_512.icns',
  width: 1200,
  height: 780,
  useLocalFile: false,
  fullscreen: false,
  hideTitleBar: true,
  multiArch: false,
  inject: './inject.js',
  debug: true,
  proxyUrl: '',
  userAgent: '',
  targets: 'deb',
  appVersion: '1.0.0',
  alwaysOnTop: false,
  darkMode: false,
  disabledWebShortcuts: false,
  activationShortcut: '',
  showSystemTray: false,
  systemTrayIcon: '',
  installerLanguage: 'en-US',
  name: 'MistralAI',
  identifier: 'com.pake.d5f4c2'
}
www.npmjs.com latency is 29 ms
✹ Installing package...

up to date, audited 211 packages in 1s

69 packages are looking for funding
  run `npm fund` for details

✔ Package installed!
✼ The first packaging may be slow, please be patient and wait, it will be faster afterwards.
file:///Users/morganorix/Pake/dist/cli.js:572
        if (!inject.every(item => item.endsWith('.css') || item.endsWith('.js'))) {
                    ^

TypeError: inject.every is not a function
    at mergeConfig (file:///Users/morganorix/Pake/dist/cli.js:572:21)
    at async MacBuilder.buildAndCopy (file:///Users/morganorix/Pake/dist/cli.js:656:9)
    at async MacBuilder.build (file:///Users/morganorix/Pake/dist/cli.js:649:9)
    at async Command.<anonymous> (file:///Users/morganorix/Pake/dist/cli.js:1013:5)
`


### What did you expect to see?

I expect the --inject option to work

### What did you see instead?

`pake "https://chat.mistral.ai/chat" --name "MistralAI" --hide-title-bar --icon ./icon_512.icns --debug true --inject ./inject.js 
PakeAppOptions {
  icon: '/Users/morganorix/Documents/apps maison/mistralai/icon_512.icns',
  width: 1200,
  height: 780,
  useLocalFile: false,
  fullscreen: false,
  hideTitleBar: true,
  multiArch: false,
  inject: './inject.js',
  debug: true,
  proxyUrl: '',
  userAgent: '',
  targets: 'deb',
  appVersion: '1.0.0',
  alwaysOnTop: false,
  darkMode: false,
  disabledWebShortcuts: false,
  activationShortcut: '',
  showSystemTray: false,
  systemTrayIcon: '',
  installerLanguage: 'en-US',
  name: 'MistralAI',
  identifier: 'com.pake.d5f4c2'
}
www.npmjs.com latency is 29 ms
✹ Installing package...

up to date, audited 211 packages in 1s

69 packages are looking for funding
  run `npm fund` for details

✔ Package installed!
✼ The first packaging may be slow, please be patient and wait, it will be faster afterwards.
file:///Users/morganorix/Pake/dist/cli.js:572
        if (!inject.every(item => item.endsWith('.css') || item.endsWith('.js'))) {
                    ^

TypeError: inject.every is not a function
    at mergeConfig (file:///Users/morganorix/Pake/dist/cli.js:572:21)
    at async MacBuilder.buildAndCopy (file:///Users/morganorix/Pake/dist/cli.js:656:9)
    at async MacBuilder.build (file:///Users/morganorix/Pake/dist/cli.js:649:9)
    at async Command.<anonymous> (file:///Users/morganorix/Pake/dist/cli.js:1013:5)

Anything else?

No response

Are you willing to submit a PR?

  • [ ] I'm willing to submit a PR!

morganorix avatar Feb 09 '25 18:02 morganorix

我也遇到了这个问题,应该是参数解析成字符串了而不是数组。

ImKujio avatar Feb 10 '25 16:02 ImKujio

Image

I used the same command as you, but the inject parameter parsed by commander is an array. Could this be an issue with the commander library?

jeasonnow avatar Feb 17 '25 04:02 jeasonnow

I've encountered the same problem too.

$ pake https://chat.qwenlm.ai --name QwenChat --icon ~/Downloads/aquv9-bkkex.icns --inject "./1.css" --d
ebug
PakeAppOptions {
  icon: '/Users/zc/Downloads/aquv9-bkkex.icns',
  width: 1200,
  height: 780,
  useLocalFile: false,
  fullscreen: false,
  hideTitleBar: false,
  multiArch: false,
  inject: './1.css',
  debug: true,
  proxyUrl: '',
  userAgent: '',
  targets: 'deb',
  appVersion: '1.0.0',
  alwaysOnTop: false,
  darkMode: false,
  disabledWebShortcuts: false,
  activationShortcut: '',
  showSystemTray: false,
  systemTrayIcon: '',
  installerLanguage: 'en-US',
  name: 'QwenChat',
  identifier: 'com.pake.3a6982'
}
www.npmjs.com latency is 127 ms
✹ Installing package...
✸ Installing package...

added 77 packages, and audited 222 packages in 1s

73 packages are looking for funding
  run `npm fund` for details

✔ Package installed!
✼ The first packaging may be slow, please be patient and wait, it will be faster afterwards.
file:///Users/zc/.nvm/versions/node/v18.20.5/lib/node_modules/pake-cli/dist/cli.js:572
        if (!inject.every(item => item.endsWith('.css') || item.endsWith('.js'))) {
                    ^

TypeError: inject.every is not a function
    at mergeConfig (file:///Users/zc/.nvm/versions/node/v18.20.5/lib/node_modules/pake-cli/dist/cli.js:572:21)
    at async MacBuilder.buildAndCopy (file:///Users/zc/.nvm/versions/node/v18.20.5/lib/node_modules/pake-cli/dist/cli.js:656:9)
    at async MacBuilder.build (file:///Users/zc/.nvm/versions/node/v18.20.5/lib/node_modules/pake-cli/dist/cli.js:649:9)
    at async Command.<anonymous> (file:///Users/zc/.nvm/versions/node/v18.20.5/lib/node_modules/pake-cli/dist/cli.js:1013:5)

Node.js v18.20.5

coderzc avatar Feb 22 '25 13:02 coderzc

pake https://chat.kittizz.com --name ChatKT --icon /Users/kittizz/workspace/pake/chat-kt/chatkt.ico --hide-title-bar --inject "/Users/kittizz/workspace/pake/chat-kt/style.css" ✺ Installing package...

up to date, audited 223 packages in 991ms

73 packages are looking for funding run npm fund for details

✔ Package installed! ✼ macOS icon must be .icns type., but you give .ico ✼ Icon will remain as default. file:///Users/kittizz/Library/pnpm/global/5/.pnpm/[email protected]/node_modules/pake-cli/dist/cli.js:572 if (!inject.every(item => item.endsWith('.css') || item.endsWith('.js'))) { ^

TypeError: inject.every is not a function at mergeConfig (file:///Users/kittizz/Library/pnpm/global/5/.pnpm/[email protected]/node_modules/pake-cli/dist/cli.js:572:21) at async MacBuilder.buildAndCopy (file:///Users/kittizz/Library/pnpm/global/5/.pnpm/[email protected]/node_modules/pake-cli/dist/cli.js:656:9) at async MacBuilder.build (file:///Users/kittizz/Library/pnpm/global/5/.pnpm/[email protected]/node_modules/pake-cli/dist/cli.js:649:9) at async Command. (file:///Users/kittizz/Library/pnpm/global/5/.pnpm/[email protected]/node_modules/pake-cli/dist/cli.js:1013:5)

Node.js v22.14.0

kittizz avatar Mar 04 '25 22:03 kittizz

#968

dawn-lc avatar May 10 '25 13:05 dawn-lc