copilot.el icon indicating copy to clipboard operation
copilot.el copied to clipboard

Incompatible with latest copilot-node-server

Open DamienCassou opened this issue 1 year ago • 7 comments

https://github.com/jfcherng/copilot-node-server released 1.41.0. The main JS file changed from agent.js to language-server.js in 1.34.0. This makes copilot.el incompatible with the latest version.

DamienCassou avatar Oct 15 '24 09:10 DamienCassou

M-x copilot-install-server doesn't complete at all because of this. :)

chesterroh avatar Oct 16 '24 11:10 chesterroh

M-x copilot-install-server doesn't complete at all because of this. :)

I no longer have the copilot-install-server command available at all 😅

jordangarrison avatar Oct 17 '24 16:10 jordangarrison

~Hello, I submitted a PR to solve this.~ ~You can patch copilot.el by: https://github.com/copilot-emacs/copilot.el/pull/346~ ~By this patch, I tried, it worked on latest node copilot-node-server (1.41.0) https://www.npmjs.com/package/copilot-node-server~

Sad, it not work on latest copilot-node-server: copilot-agent started successfully, but code completion not work.

(:jsonrpc "2.0" :method "textDocument/didChange" :params
  (:textDocument
	(:uri "file:///home/exec/.config/emacs/elisp/copilot.el" :version
	  4)
	:contentChanges
	[(:range
	   (:start (:line 18 :character 0) :end (:line 19 :character 0))
	   :text "")]))
[jsonrpc] e[21:39:59.154] --> getCompletions[5] 
(:jsonrpc "2.0" :id 5 :method "getCompletions" :params
  (:doc
	(:version 4 :tabSize 4 :indentSize 2 :insertSpaces :json-false
	  :path "/home/exec/.config/emacs/elisp/copilot.el" :uri
	  "file:///home/exec/.config/emacs/elisp/copilot.el" :relativePath
	  "copilot.el" :languageId "emacs-lisp" :position
	  (:line 18 :character 0))))
[jsonrpc] e[21:39:59.162] <-- getCompletions[5] 
(:jsonrpc "2.0" :id 5 :error
  (:code -32002 :message "Agent service not initialized."))
[jsonrpc] i[21:39:59.162] [5] error -32002 ignored: Agent service not initialized. ignored

eval-exec avatar Oct 30 '24 10:10 eval-exec

M-x copilot-install-server doesn't complete at all because of this. :)

I no longer have the copilot-install-server command available at all 😅

my problem was just a path problem. Simply emacs didn't find a 'npx' command accordingly. Thanks for claude-sonnet, I was able to solve the problem.

exec-path-from-shell package solved mine.

(use-package exec-path-from-shell :ensure t :if (memq window-system '(mac ns x)) :config (exec-path-from-shell-initialize))

chesterroh avatar Nov 03 '24 04:11 chesterroh

M-x copilot-install-server doesn't complete at all because of this. :)

I no longer have the copilot-install-server command available at all 😅

my problem was just a path problem. Simply emacs didn't find a 'npx' command accordingly. Thanks for claude-sonnet, I was able to solve the problem.

exec-path-from-shell package solved mine.

(use-package exec-path-from-shell :ensure t :if (memq window-system '(mac ns x)) :config (exec-path-from-shell-initialize))

After you use 1.41.0 copilot-node-server, does copilot-completion work well?

eval-exec avatar Nov 03 '24 04:11 eval-exec

M-x copilot-install-server doesn't complete at all because of this. :)

I no longer have the copilot-install-server command available at all 😅

my problem was just a path problem. Simply emacs didn't find a 'npx' command accordingly. Thanks for claude-sonnet, I was able to solve the problem. exec-path-from-shell package solved mine. (use-package exec-path-from-shell :ensure t :if (memq window-system '(mac ns x)) :config (exec-path-from-shell-initialize))

After you use 1.41.0 copilot-node-server, does copilot-completion work well?

didn't try it yet. but I suspect it's mainly because of the script name change? from agent.js to langauge-server.js ? Did you try this?

chesterroh avatar Nov 03 '24 04:11 chesterroh

I tried language-server.js ,copilot completion not work. Sad.

On Sun, Nov 3, 2024, 12:50 Chester Jungseok Roh @.***> wrote:

M-x copilot-install-server doesn't complete at all because of this. :)

I no longer have the copilot-install-server command available at all 😅

my problem was just a path problem. Simply emacs didn't find a 'npx' command accordingly. Thanks for claude-sonnet, I was able to solve the problem. exec-path-from-shell package solved mine. (use-package exec-path-from-shell :ensure t :if (memq window-system '(mac ns x)) :config (exec-path-from-shell-initialize))

After you use 1.41.0 copilot-node-server, does copilot-completion work well?

didn't try it yet. but I suspect it's mainly because of the script name change? from agent.js to langauge-server.js ? Did you try this?

— Reply to this email directly, view it on GitHub https://github.com/copilot-emacs/copilot.el/issues/344#issuecomment-2453295939, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALCAINWOMNBBI637E2TTOD3Z6WTSZAVCNFSM6AAAAABP6VVTFCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJTGI4TKOJTHE . You are receiving this because you commented.Message ID: @.***>

eval-exec avatar Nov 03 '24 05:11 eval-exec