bwrap starts with an empty environment (0.13.3) #18

Merged
ai-reviewer merged 3 commits from claude-code/bwrap-empty-env into main 2026-10-21 21:23:44 +02:00
Collaborator

See docs/prs/2026-10-20-PR#18-bwrap-empty-env.md.

🤖 Generated with Claude Code

See docs/prs/2026-10-20-PR#18-bwrap-empty-env.md. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
claude-code added 1 commit 2026-10-20 19:29:41 +02:00
PR document: bwrap starts with an empty environment
werkator build succeeded after 00:06
werkator/dependencies build succeeded after 00:05
80f8931597
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
claude-code added 1 commit 2026-10-20 19:30:20 +02:00
werkdock's own environment holds the values of the caller's -e KEY since
werkator PR#166, with names a build may choose; bwrap, a host process,
must not read them, and needs no variable.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
claude-code added 1 commit 2026-10-20 19:30:28 +02:00
PR document: line length
werkator build succeeded after 00:03
werkator/dependencies build succeeded after 00:04
ai-review/assessment human review required; a human approval unlocks the merge; review requested from mi; merge scheduled for when the checks succeed
ai-review/gate build succeeded after 01:12
0bf81cf096
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
claude-code marked the pull request as ready for review 2026-10-20 19:36:25 +02:00
Collaborator

🟡 werkrevisor: YELLOW / HUMAN_REVIEW

Change kind: mixed.

Reasons

  • YELLOW risky paths changed: docs/risk-analysis.md (docs/risk-analysis.md)
  • YELLOW Sandbox isolation according to Jev: internal/engine/bwrap.go (0.93); a human checks it, unless you revert the change because the task does not need it
  • hint 2 changed file(s) that neither weaken the checks, touch security nor show a risk category of jvm-spring-backend according to Jev

Classifier

7 of 7 units passed all questions (1 PR document(s)).
1 answer ignored because of a confidence below [classifier] min_confidence.

Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 25543, calls 1, tokens in 24683 / out 5820 (reasoning included in out), duration 69.7 s, cost 0.0144 USD (reported), classifier `jev-1.13.0` calls 7, tokens in 4983, cost 0.0002 USD, build file judge `jev-1.13.0` calls 3, tokens in 2787, cost 0.0001 USD, phases: path_judge 0.3 s, classifier 0.5 s, context 0.3 s, model_review 68.2 s, config: defaults + host + repository (.werkrevisor.toml at main b226665).
Reviewed head `0bf81cf0960ca9f1eeff2c1f7ef4c4866fd7347b` against base `b2266658f3c00b22460df982acda2dfee6a129b7`, werkrevisor 0.55.0.
<!-- werkrevisor:0bf81cf0960ca9f1eeff2c1f7ef4c4866fd7347b --> <!-- werkrevisor-verdict:YELLOW --> <!-- werkrevisor-blocks:29646a94268962eddcb07584224cc14c,e17be818ad0b0982b45cbba8fd554263,7681636d54bf4966b9b434723b3e5cd7,36d2516ec0fcf089758ed1d0678f2678,05923db194a7b3fb36391f74167d507e,088e236fad9e9bb0a1062ded4fc6fb88,b58d34f8f3946c2816d84ca463fc5c50,b30104c44ed4dbab8b164bf842c07489,68228c605f456906820089d2311a2524 --> ## 🟡 werkrevisor: YELLOW / HUMAN_REVIEW Change kind: `mixed`. ### Reasons - **YELLOW** risky paths changed: docs/risk-analysis.md (docs/risk-analysis.md) - **YELLOW** Sandbox isolation according to Jev: internal/engine/bwrap.go (0.93); a human checks it, unless you revert the change because the task does not need it - **hint** 2 changed file(s) that neither weaken the checks, touch security nor show a risk category of jvm-spring-backend according to Jev ### Classifier 7 of 7 units passed all questions (1 PR document(s)). 1 answer ignored because of a confidence below [classifier] min_confidence. <sub> Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 25543, calls 1, tokens in 24683 / out 5820 (reasoning included in out), duration 69.7 s, cost 0.0144 USD (reported), classifier `jev-1.13.0` calls 7, tokens in 4983, cost 0.0002 USD, build file judge `jev-1.13.0` calls 3, tokens in 2787, cost 0.0001 USD, phases: path_judge 0.3 s, classifier 0.5 s, context 0.3 s, model_review 68.2 s, config: defaults + host + repository (.werkrevisor.toml at main b226665).<br> Reviewed head `0bf81cf0960ca9f1eeff2c1f7ef4c4866fd7347b` against base `b2266658f3c00b22460df982acda2dfee6a129b7`, werkrevisor 0.55.0. </sub>
ai-reviewer scheduled this pull request to auto merge when all checks succeed 2026-10-20 19:37:55 +02:00
ai-reviewer requested review from mi 2026-10-20 19:37:55 +02:00
mi approved these changes 2026-10-21 21:23:43 +02:00
ai-reviewer merged commit 61d53429a0 into main 2026-10-21 21:23:44 +02:00
ai-reviewer deleted branch claude-code/bwrap-empty-env 2026-10-21 21:23:44 +02:00
Sign in to join this conversation.
No Reviewers
No labels
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mi/werkdock#18