Skip to content

Revert 'Change image to run as non-root' (#387)#416

Merged
St0rmz1 merged 2 commits intomainfrom
revert/kiloclaw-non-root-entrypoint
Feb 20, 2026
Merged

Revert 'Change image to run as non-root' (#387)#416
St0rmz1 merged 2 commits intomainfrom
revert/kiloclaw-non-root-entrypoint

Conversation

@St0rmz1
Copy link
Contributor

@St0rmz1 St0rmz1 commented Feb 20, 2026

No description provided.

@St0rmz1 St0rmz1 marked this pull request as ready for review February 20, 2026 17:31
@kilo-code-bot
Copy link
Contributor

kilo-code-bot bot commented Feb 20, 2026

Code Review Summary

Status: 1 Issue Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 0
Issue Details (click to expand)

WARNING

File Line Issue
kiloclaw/Dockerfile 43-46 Debug/diagnostic RUN commands (ls -l, head, od) left in Dockerfile — should be removed before merge
Other Observations (not in diff)

This PR reverts the non-root user changes from PR #387, meaning the container will run as root. While this is an intentional revert (likely to resolve runtime issues with the non-root approach), running containers as root is a security concern. Consider re-introducing non-root execution in a follow-up once the underlying issues are resolved.

Files Reviewed (2 files)
  • kiloclaw/Dockerfile - 1 issue
  • kiloclaw/entrypoint.sh (deleted) - no issues

Fix these issues in Kilo Cloud

@St0rmz1 St0rmz1 merged commit 42ae9fc into main Feb 20, 2026
12 checks passed
@St0rmz1 St0rmz1 deleted the revert/kiloclaw-non-root-entrypoint branch February 20, 2026 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants