search

atlas-engineer / cl-electron

Lisp Interface to Electron.
BSD 3-Clause "New" or "Revised" License
16 stars 1 forks source link

Separate results from errors #10

Closed Ambrevar closed 6 months ago

Ambrevar commented 1 year ago

See https://github.com/atlas-engineer/cl-electron/pull/7#issuecomment-1590960631.

Shouldn't be too hard, but then send-message should probably not use read-line (see https://github.com/atlas-engineer/cl-electron/pull/7#issuecomment-1596701751).

Actually it may be OK, since the S-exp is fully controlled and does not contain arbitrary strings.

Alternatively, we could wrap the result / error in JSON: 

{result: ...
 error:
}
jmercouris commented 6 months ago

No longer valid in the current architecture.