refactor: Remove dockerode dependency from package.json

[sebyx07]

Summary

Removed the dockerode dependency from package.json as part of the migration to Incus-only worker backend. This completes Task 6 of the Docker removal PR group.

• Removed dockerode ^4.0.2 from dependencies
• Updated package-lock.json via npm install (removed 49 packages)
• Verified ESLint passes with no style issues

Files Modified

• package.json - removed dockerode dependency (line 60)
• package-lock.json - automatically updated by npm install

Testing

• ✓ ESLint check passed (npm run lint)
• Note: Integration and E2E tests fail due to tests referencing removed Docker sandbox type - these will be fixed in PR4 (Update Tests)

Context

This is the final task of PR1 (Remove Docker Worker Backend & Backward Compatibility Code). Previous tasks have already:

• Removed DockerManager and DockerBackend classes
• Removed LegacyRedirectRouter
• Updated ConfigValidator to default to Incus
• Updated CLI commands to use IncusBackend
• Updated public exports

🤖 Generated with Claude Code

Greptile Overview

Greptile Summary

Completes the Docker-to-Incus migration by removing the dockerode dependency and all Docker-specific code. The PR successfully removes 8,775 lines including DockerManager, DockerBackend, and the legacy redirect router, while updating all references to use IncusBackend instead.

Key changes:

• Removed dockerode package dependency (49 packages cleaned up)
• Deleted DockerManager (645 lines) and DockerBackend (160 lines) implementations
• Removed LegacyRedirectRouter (158 lines) for /api/bots backward compatibility
• Updated CLI commands to instantiate IncusBackend instead of DockerManager
• Changed default sandbox type from docker to incus in ConfigValidator
• Removed dockerAccess field from sandbox configuration
• Updated error handling to use container category instead of docker
• Cleaned up backward compatibility code throughout orchestrator and container pool

Critical issue found:

• src/core/orchestrator.js:323 still references BACKEND_TYPES.DOCKER which was removed, causing a runtime error on startup

Confidence Score: 2/5

• This PR contains a critical runtime bug that will cause the application to crash on startup
• While the refactoring is comprehensive and well-executed (removing 8,775 lines across 43 files), the reference to the non-existent BACKEND_TYPES.DOCKER constant in orchestrator.js:323 will cause immediate failure when the orchestrator initializes. This is a blocking issue that must be fixed before merge.
• Pay close attention to src/core/orchestrator.js - the bug on line 323 must be fixed before merging

Important Files Changed

Filename	Overview
package.json	Removed dockerode dependency (line 60) as part of Docker to Incus migration
src/core/orchestrator.js	Critical bug: line 323 still references BACKEND_TYPES.DOCKER which no longer exists; removed legacy router and backward compatibility code
src/execution/container-pool.js	Removed DockerBackend integration, DockerManager references, and docker-specific logic; simplified to Incus-only backend
src/index.js	Updated exports: replaced DockerManager with IncusBackend and IncusClient
src/cli/DevCommand.js	Replaced DockerManager with IncusBackend in both botManagerFactory and messageProcessorFactory
src/cli/StartCommand.js	Replaced shared DockerManager with IncusBackend in factory functions
src/config/ConfigValidator.js	Updated sandbox schema: removed docker type and dockerAccess field; changed default to incus; removed legacy channel validation

Flowchart

flowchart TD
    A[Application Start] --> B[Orchestrator.init]
    B --> C{Check sandbox type}
    C -->|Line 323| D[BACKEND_TYPES.DOCKER ❌]
    D --> E[Runtime Error: undefined]
    
    C -->|Should be| F[BACKEND_TYPES.INCUS ✓]
    F --> G[Create IncusBackend]
    G --> H[ContainerPool]
    H --> I[Bot Execution]
    
    style D fill:#ff6b6b
    style E fill:#ff6b6b
    style F fill:#51cf66
    style G fill:#51cf66

Loading

_{Last reviewed commit: 575bb15}

[greptile-apps]

_{31 files reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

[greptile-apps]

Additional Comments (1)

src/core/orchestrator.js
BACKEND_TYPES.DOCKER no longer exists after removal from container-backend.js

      const defaultSandboxType = this.config?.defaults?.sandbox?.type || BACKEND_TYPES.INCUS;

Prompt To Fix With AI

This is a comment left during a code review.
Path: src/core/orchestrator.js
Line: 323:323

Comment:
`BACKEND_TYPES.DOCKER` no longer exists after removal from `container-backend.js`

```suggestion
      const defaultSandboxType = this.config?.defaults?.sandbox?.type || BACKEND_TYPES.INCUS;
```

How can I resolve this? If you propose a fix, please make it concise.

[greptile-apps]

_{43 files reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

[greptile-apps]

Additional Comments (1)

src/cli/ProjectInitializer.js
Docker reference in project template not updated to Incus

            sandbox: { type: 'incus' },

Prompt To Fix With AI

This is a comment left during a code review.
Path: src/cli/ProjectInitializer.js
Line: 285:285

Comment:
Docker reference in project template not updated to Incus

```suggestion
            sandbox: { type: 'incus' },
```

How can I resolve this? If you propose a fix, please make it concise.

[greptile-apps]

_{43 files reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

[greptile-apps]

Additional Comments (1)

src/core/orchestrator.js
BACKEND_TYPES.DOCKER no longer exists after removing Docker support

      const defaultSandboxType = this.config?.defaults?.sandbox?.type || BACKEND_TYPES.INCUS;

Prompt To Fix With AI

This is a comment left during a code review.
Path: src/core/orchestrator.js
Line: 323:323

Comment:
`BACKEND_TYPES.DOCKER` no longer exists after removing Docker support

```suggestion
      const defaultSandboxType = this.config?.defaults?.sandbox?.type || BACKEND_TYPES.INCUS;
```

How can I resolve this? If you propose a fix, please make it concise.