[agent/claude:opus] Fix CodeRabbit findings. Verify each against current code, f... #4

Merged

Virgil merged 1 commit from agent/fix-coderabbit-findings--verify-each-aga into main

2026-03-17 16:24:04 +00:00

Virgil commented

2026-03-17 16:23:49 +00:00

Member

Task

Fix CodeRabbit findings. Verify each against current code, fix ONLY if valid. Run tests. Commit: fix(coderabbit): address review findings

IMPORTANT: Do NOT change function signatures in dependency packages.

node/bundle.go:303-327 — extractTarball mixes os.OpenFile with coreio abstraction. Use coreio consistently or document why os.OpenFile is needed here.
node/bundle.go — check inline findings.

Agent: claude:opus
Commits: 1
Branch: agent/fix-coderabbit-findings--verify-each-aga

Auto-created by core-agent dispatch system.
Co-Authored-By: Virgil virgil@lethean.io

## Task Fix CodeRabbit findings. Verify each against current code, fix ONLY if valid. Run tests. Commit: fix(coderabbit): address review findings IMPORTANT: Do NOT change function signatures in dependency packages. 1. node/bundle.go:303-327 — extractTarball mixes os.OpenFile with coreio abstraction. Use coreio consistently or document why os.OpenFile is needed here. 2. node/bundle.go — check inline findings. **Agent:** claude:opus **Commits:** 1 **Branch:** `agent/fix-coderabbit-findings--verify-each-aga` --- Auto-created by core-agent dispatch system. Co-Authored-By: Virgil <virgil@lethean.io>

Virgil added 1 commit 2026-03-17 16:23:49 +00:00

fix(bundle): document why os.OpenFile is used instead of coreio in extractTarball

Security Scan / security (pull_request) Successful in 9s

Details

Test / test (pull_request) Successful in 1m25s

Details

b334cb4909

The coreio abstraction hardcodes file permissions (0644) and has no
OpenFile equivalent. os.OpenFile is needed here to preserve tar header
mode bits — executable binaries require 0755.

Co-Authored-By: Virgil <virgil@lethean.io>