chore: update task and agents content

AGENTS.md

@@ -1,7 +1,5 @@
 # AGENTS.md
 
-Guide for AI coding agents working in the Devsy repository.
-
 ---
 
 ## Environment Setup
@@ -10,24 +8,18 @@ Devsy is a monorepo with a Go-based CLI and an Electron-based Svelte 5 desktop a
 
 ### Prerequisites and Tooling
 
-- **Go 1.26**: The repository targets Go 1.26 (`go.mod`). With `GOTOOLCHAIN=auto`, an older Go toolchain automatically downloads the required Go version.
-- **NodeJS 24**: Required for building and testing the desktop workspace (`.nvmrc`, CI `node-version`).
-- **Taskfile (go-task)**: All build, test, and setup commands run through `task`. Installation:
-
-  ```bash
-  sudo sh -c "$(curl --location https://taskfile.dev/install.sh)" -- -d -b /usr/local/bin
-  ```
+Toolchain is managed by mise. Install mise using `curl https://mise.run | sh`. Install toolchain dependencies with `mise install`.
 
 ---
 
 ## Common Developer Commands
 
-`task --list` shows all available commands. The most common tasks:
+`task --list` shows developer commands. The most common tasks:
 
 ### CLI (Go) Development
 
 - **Tidy Go modules**: `task cli:tidy`
-- **Lint CLI**: `task cli:lint` (or `task cli:lint:fix` to apply fixes)
+- **Lint CLI**: `task cli:lint` and `task cli:lint:ci` (or `task cli:lint:fix` to apply fixes)
 - **Format CLI**: `task cli:format`
 - **Run unit tests**: `task cli:test` (writes coverage to `dist/profile.out`; a `dist` directory is required, e.g. `mkdir -p dist`)
 - **Build development binary**: `task cli:build:dev` (output under `dist/devsy-dev_linux_amd64_v1/`)
@@ -69,14 +61,12 @@ The agent binary runs inside the (Linux) workspace, so it is a Linux binary (`de
 
 ### Headless / Xvfb Requirements
 
-Desktop tests run inside an Electron browser environment. In headless or container environments (CI, automated sandbox agents), Electron-dependent commands require an `xvfb-run` prefix to emulate a display server:
+Desktop tests run inside an Electron browser environment. In headless or container environments, Electron commands require an `xvfb-run` prefix to emulate a display server:
 
 ```bash
-# Desktop unit tests headlessly
-xvfb-run task desktop:test
+xvfb-run task desktop:test # Desktop unit tests headlessly
 
-# Desktop E2E tests headlessly
-xvfb-run task desktop:test:e2e
+xvfb-run task desktop:test:e2e # Desktop E2E tests headlessly
 ```
 
 ### E2E (Ginkgo) Tests
@@ -93,8 +83,9 @@ Devsy uses [Ginkgo](https://onsi.github.io/ginkgo/) for Go E2E and integration t
 
 ### Go Code Style
 
+- **Idiomatic**: Focus on simplicity, reliability, and efficiency when writing clear, idiomatic Go code.
+- **Style Guide**: Use the Uber style guide https://github.com/uber-go/guide/blob/master/style.md.
 - **Linter**: `golangci-lint` via `task cli:lint` (or `task cli:lint:fix`). Run `task cli:lint:ci` before pushing changes.
-- **Logs**: Log messages and logging strings are lowercase.
 
 ### TypeScript / Svelte Code Style
 
@@ -105,24 +96,7 @@ Biome formats and checks web frontend files.
 ## Pull Request and Commit Guidelines
 
 1. **Contributor License Agreement (CLA)**: All contributors sign the CLA.
-2. **Commit messages**: Conventional Commits, with a concise subject line (50 characters max).
+2. **Commit messages**: Conventional Commits, with a concise subject line.
 3. **Commit signing**: All commits are required to be signed.
 4. **Branch name**: Branch should be named according to the task.
-
-4. **Pre-commit checks**: Linters, checkers, and relevant unit tests run before pushing. `prek` (a pre-commit hook manager) manages them.
-   - Installation (Linux and macOS):
-     ```bash
-     curl --proto '=https' --tlsv1.2 -LsSf https://github.com/j178/prek/releases/latest/download/prek-installer.sh | sh
-     ```
-   - Installation (Windows):
-     ```powershell
-     powershell -ExecutionPolicy ByPass -c "irm https://github.com/j178/prek/releases/latest/download/prek-installer.ps1 | iex"
-     ```
-   - Manual run on all files:
-     ```bash
-     prek run --all-files
-     ```
-   - Git hook installation:
-     ```bash
-     prek install
-     ```
+5. **Pre-commit checks**: Linters, checkers, and relevant unit tests run before pushing. `prek` (a pre-commit hook manager) manages them.

Taskfile.yml

@@ -13,14 +13,6 @@ tasks:
     desc: verify THIRD_PARTY_LICENSES.md is in sync and all licenses are allowed (CI)
     cmd: go run ./hack/licenses --check
 
-  automations:generate:
-    desc: regenerate .agents/agents/<id>/agent.md from hack/automations
-    cmd: go run ./hack/automations
-
-  automations:check:
-    desc: verify generated agent prompts are in sync (CI / pre-commit)
-    cmd: go run ./hack/automations -check
-
   cli:lint:
     desc: lint go code using golangci-lint
     deps: [cli:lint:version]

desktop/e2e/integration.e2e.ts

@@ -105,7 +105,7 @@ test.describe
       // Click Continue to add the provider
       await wizard.getByRole("button", { name: /^Continue$/ }).click()
 
-      // Docker mock has no required options → wizard jumps to init then complete.
+      // Docker mock has no required options -> wizard jumps to init then complete.
       // Wait for the "Done" button on the Complete step.
       await wizard
         .getByRole("button", { name: "Done" })
@@ -269,7 +269,7 @@ test.describe
     })
 
     test("should show new workspace in table", async () => {
-      // Workspace ID from template name: 'Node.js' → 'node-js'
+      // Workspace ID from template name: 'Node.js' -> 'node-js'
       await expect(page.locator("table")).toContainText("node-js", {
         timeout: 10000,
       })
@@ -408,7 +408,7 @@ test.describe
     })
 
     test("should show python workspace in table", async () => {
-      // Template name 'Python' → workspace id 'python'
+      // Template name 'Python' -> workspace id 'python'
       await expect(page.locator("table")).toContainText("python", {
         timeout: 10000,
       })

desktop/src/renderer/src/lib/components/update/channel.test.ts

@@ -21,11 +21,11 @@ describe("channel labels", () => {
 })
 
 describe("isDowngrade", () => {
-  it("treats Preview → Stable as a downgrade", () => {
+  it("treats Preview -> Stable as a downgrade", () => {
     expect(isDowngrade("beta", "stable")).toBe(true)
   })
 
-  it("treats Stable → Preview as not a downgrade", () => {
+  it("treats Stable -> Preview as not a downgrade", () => {
     expect(isDowngrade("stable", "beta")).toBe(false)
   })
 

desktop/src/renderer/src/lib/ipc/mock.ts

@@ -300,7 +300,7 @@ export async function invoke<T>(
   await new Promise((r) => setTimeout(r, 50 + Math.random() * 100))
 
   const result = handler(args ?? {})
-  console.debug(`[mock] ${cmd}`, args ?? {}, "→", result)
+  console.debug(`[mock] ${cmd}`, args ?? {}, "->", result)
   return result as T
 }
 

e2e/tests/ide/browser_returns.go

@@ -92,7 +92,7 @@ var _ = ginkgo.Describe(
 
 				// Run up with a browser IDE. --ide-launch=headless suppresses the
 				// host browser launch (no display available in CI) but still runs
-				// openIDE → startDetachedBrowserTunnel → writes tunnel.json.
+				// openIDE -> startDetachedBrowserTunnel -> writes tunnel.json.
 				// --ide-launch=skip would skip openIDE entirely, which is not what
 				// the test exercises. With the old blocking behavior this would
 				// still hang past SpecTimeout; with the new behavior the CLI
@@ -180,7 +180,7 @@ var _ = ginkgo.Describe(
 					"expected --recreate to spawn a new helper (PID1=%d, PID2=%d)", pid1, pid2)
 
 				// PID1 should now be dead. Use Eventually because the kill is
-				// best-effort SIGTERM → wait → SIGKILL.
+				// best-effort SIGTERM -> wait -> SIGKILL.
 				gomega.Eventually(func() error {
 					return syscall.Kill(pid1, 0)
 				}).WithTimeout(5*time.Second).WithPolling(100*time.Millisecond).

e2e/tests/up/up_behaviors.go

@@ -412,6 +412,6 @@ var _ = ginkgo.Describe("up command behaviors", ginkgo.Label("up-behaviors"), fu
 			"postCreate",
 			"dotfiles-before-postStart",
 			"postStart",
-		}), "lifecycle ordering should be: postCreate → dotfiles → postStart")
+		}), "lifecycle ordering should be: postCreate -> dotfiles -> postStart")
 	}, ginkgo.SpecTimeout(framework.TimeoutShort()))
 })

pkg/agent/delivery/factory_test.go

@@ -153,7 +153,7 @@ func TestNewAgentDelivery_KubernetesDriver_FallsBackWhenNoPodExec(t *testing.T)
 		ExecFunc:                   execFn,
 		DownloadURL:                "https://artifacts.example.test/devsy",
 		KubernetesAgentInstallPath: testKubernetesInstallPath,
-		// PodExec intentionally nil → legacy fallback.
+		// PodExec intentionally nil -> legacy fallback.
 	}
 
 	d := NewAgentDelivery(opts)

pkg/apple/helper_test.go

@@ -124,7 +124,7 @@ func TestParseImageTag(t *testing.T) {
 }
 
 func TestWaitContainerRunningFailsFastOnExit(t *testing.T) {
-	// A container reporting a terminal (stopped→exited) state must error
+	// A container reporting a terminal (stopped->exited) state must error
 	// immediately rather than block for the full poll timeout.
 	stdout := `[{"id":"c1","configuration":{"id":"c1"},"status":{"state":"stopped"}}]`
 	h := &AppleHelper{Command: stubContainer(t, stdout, 0)}
@@ -143,7 +143,7 @@ func TestWaitContainerRunningFailsFastOnExit(t *testing.T) {
 }
 
 func TestEnsureBuilderRunning(t *testing.T) {
-	// Exit 0 (the real CLI's behavior even when already running) → no error.
+	// Exit 0 (the real CLI's behavior even when already running) -> no error.
 	okHelper := &AppleHelper{Command: stubContainer(t, "", 0)}
 	if err := okHelper.EnsureBuilderRunning(context.Background()); err != nil {
 		t.Errorf("exit 0 must succeed, got %v", err)

pkg/devcontainer/config/substitute.go

@@ -327,7 +327,7 @@ func ListToObject(list []string) map[string]string {
 }
 
 // ComputeDevContainerID implements the official devcontainer CLI algorithm:
-// SHA-256(JSON.stringify(labels, sorted keys)) → BigInt → base-32 (0-9a-v) → left-pad to 52 chars.
+// SHA-256(JSON.stringify(labels, sorted keys)) -> BigInt -> base-32 (0-9a-v) -> left-pad to 52 chars.
 func ComputeDevContainerID(labels map[string]string) string {
 	keys := make([]string, 0, len(labels))
 	for k := range labels {

pkg/devcontainer/graph/graph_test.go

@@ -492,9 +492,9 @@ func (suite *GraphTestSuite) TestLargeGraph() {
 }
 
 func (suite *GraphTestSuite) TestSortNodeIDsRoundBased() {
-	// A→B edge means B depends on A.
-	// Round 1: A and C both have in-degree 0 → sorted alpha → [A, C]
-	// Round 2: B (now in-degree 0 after A processed) → [B]
+	// A->B edge means B depends on A.
+	// Round 1: A and C both have in-degree 0 -> sorted alpha -> [A, C]
+	// Round 2: B (now in-degree 0 after A processed) -> [B]
 	// Final: [A, C, B]
 	suite.Require().NoError(suite.graph.AddNode("A", "dataA"))
 	suite.Require().NoError(suite.graph.AddNode("B", "dataB"))
@@ -507,10 +507,10 @@ func (suite *GraphTestSuite) TestSortNodeIDsRoundBased() {
 }
 
 func (suite *GraphTestSuite) TestSortNodeIDsRoundBasedMultiLevel() {
-	// A→B→D, C→D (independent chains with shared sink)
-	// Round 1: A, C (in-degree 0) → [A, C]
-	// Round 2: B (depends on A only) → [B]
-	// Round 3: D (depends on B and C, but C was processed in round 1) → [D]
+	// A->B->D, C->D (independent chains with shared sink)
+	// Round 1: A, C (in-degree 0) -> [A, C]
+	// Round 2: B (depends on A only) -> [B]
+	// Round 3: D (depends on B and C, but C was processed in round 1) -> [D]
 	suite.Require().NoError(suite.graph.AddNode("A", "dataA"))
 	suite.Require().NoError(suite.graph.AddNode("B", "dataB"))
 	suite.Require().NoError(suite.graph.AddNode("C", "dataC"))

pkg/devcontainer/setup/lifecyclehooks_test.go

@@ -762,7 +762,7 @@ func (s *LifecycleHookTestSuite) TestPostCreateHookUsesOnceSemantics() {
 
 	hooks := preAttachPhaseParams(result, env, false)
 
-	// postCreateCommand should have content = Created (non-empty → once semantics).
+	// postCreateCommand should have content = Created (non-empty -> once semantics).
 	var postCreate, postStart hookRunParams
 	for _, h := range hooks {
 		if h.phase == PhasePostCreate {

pkg/git/git.go

@@ -65,7 +65,7 @@ func NormalizeRepository(str string) *GitInfo {
 
 // canonicalizeURL strips the workspace-source "git:" scheme (the form
 // WorkspaceSource.String emits; without this strip, a value that round-trips
-// through workspace list → up becomes "https://git:https://...") and upgrades
+// through workspace list -> up becomes "https://git:https://...") and upgrades
 // bare host[/path] inputs to https://.
 func canonicalizeURL(str string) string {
 	str = strings.TrimPrefix(str, "git:")

pkg/ide/opener/browser_tunnel_test.go

@@ -403,7 +403,7 @@ func relistenAfter(addr string, delay, hold time.Duration) <-chan error {
 }
 
 // checkRelistenOrSkip drains a non-blocking read from listenErr. A non-nil
-// error means the OS reassigned the port in the close→relisten window;
+// error means the OS reassigned the port in the close->relisten window;
 // skip rather than fail to avoid spurious CI failures on busy hosts.
 func checkRelistenOrSkip(t *testing.T, addr string, listenErr <-chan error) {
 	t.Helper()

pkg/provider/version_cache.go

@@ -73,7 +73,7 @@ func LoadProviderVersionCache() (ProviderVersionCache, error) {
 	}
 	cache := ProviderVersionCache{}
 	if err := json.Unmarshal(data, &cache); err != nil {
-		// Corrupt cache → start fresh.
+		// Corrupt cache -> start fresh.
 		return ProviderVersionCache{}, nil
 	}
 	return cache, nil

pkg/provider/versions_test.go

@@ -78,7 +78,7 @@ func TestMarkCurrent_NoTag(t *testing.T) {
 	versions := []ProviderVersion{{Tag: testTagV100}}
 	got := MarkCurrent(versions, "github.com/foo/bar")
 	if got[0].Current {
-		t.Fatal("no pinned tag → none current")
+		t.Fatal("no pinned tag -> none current")
 	}
 }
 

pkg/ssh/server/exec.go

@@ -19,9 +19,6 @@ import (
 // Process group isolation (SysProcAttr) ensures child processes can be
 // properly signaled on shutdown. SSH client signals are forwarded to the
 // process.
-//
-// Loosely modeled after Coder's startNonPTYSession:
-//   - https://github.com/coder/coder/blob/main/agent/agentssh/agentssh.go
 func execNonPTY(sess ssh.Session, cmd *exec.Cmd) (err error) {
 	log.Debugf("execute SSH server command: %s", strings.Join(cmd.Args, " "))
 
@@ -125,13 +122,11 @@ type ptyExecParams struct {
 // SSH signal forwarding. Output is copied on the main goroutine to ensure all
 // buffered data is flushed before process.Wait().
 //
-// DisablePTYEmulation prevents double NL→CRNL translation. The kernel's line
+// DisablePTYEmulation prevents double NL->CRNL translation. The kernel's line
 // discipline already performs this; the gliderlabs/ssh library's own conversion
 // would corrupt terminal escape sequences.
 //
-// Ported from coder/ssh (Coder's fork of gliderlabs/ssh):
-//   - Coder issue:  https://github.com/coder/coder/issues/3371
-//   - Neovim issue: https://github.com/neovim/neovim/issues/3875
+// - Neovim issue: https://github.com/neovim/neovim/issues/3875
 func execPTY(p ptyExecParams) (retErr error) {
 	log.Debugf("execute SSH server PTY command: %s", strings.Join(p.cmd.Args, " "))
 	p.sess.DisablePTYEmulation()

pkg/ssh/server/exit.go

@@ -43,7 +43,6 @@ func exitCode(err error) int {
 		// Map -1 to 255 to match OpenSSH behavior. -1 would be
 		// transmitted as uint32(4294967295).
 		// OpenSSH returns 255 for this case, and the shell does the same.
-		//   - https://github.com/coder/coder/blob/main/agent/agentssh/agentssh.go
 		//   - https://github.com/openssh/openssh-portable/blob/master/session.c
 		code = 255
 	}

pkg/workspace/exec_shared_test.go

@@ -65,7 +65,7 @@ func TestExecWithRunnerExitCode(t *testing.T) {
 }
 
 func TestProbeEnvWithRunner(t *testing.T) {
-	// /proc/self/environ succeeds → NUL-separated parse.
+	// /proc/self/environ succeeds -> NUL-separated parse.
 	run := func(_ context.Context, _ []string, _ io.Reader, stdout, _ io.Writer) error {
 		_, _ = stdout.Write([]byte("PATH=/bin\x00HOME=/root\x00"))
 		return nil
@@ -76,7 +76,7 @@ func TestProbeEnvWithRunner(t *testing.T) {
 		t.Errorf("probed env = %v", env)
 	}
 
-	// Total failure → empty map (documented contract), never a panic.
+	// Total failure -> empty map (documented contract), never a panic.
 	failRun := func(_ context.Context, _ []string, _ io.Reader, _, _ io.Writer) error {
 		return errors.New("exec failed")
 	}