Skip to content

fix: permission issues in sandboxes - #256

Merged
stephantul merged 2 commits into
mainfrom
fix-permission-issues
Aug 26, 2026
Merged

fix: permission issues in sandboxes#256
stephantul merged 2 commits into
mainfrom
fix-permission-issues

Conversation

@stephantul

Copy link
Copy Markdown
Contributor

This PR correctly adds the distinction between a config file not existing and a config file not being readable because of other issues. Previously, we assumed that if a config file existed, it was also readable, but this is not true in the case of sandboxes, which caused errors.

Solves #255

@codecov

codecov Bot commented Aug 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ
src/semble/installer/agents.py 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
@greptile-apps

greptile-apps Bot commented Aug 25, 2026

Copy link
Copy Markdown

Confidence Score: 5/5

The PR appears safe to merge with no actionable defects identified.

The permission-aware helper preserves the established OpenCode configuration precedence and intentionally makes best-effort agent detection tolerate sandbox-denied metadata access.

Reviews (1): Last reviewed commit: "fix: permission issues in sandboxes" | Re-trigger Greptile

@stephantul
stephantul merged commit 6bbbd32 into main Aug 26, 2026
16 checks passed
@stephantul
stephantul deleted the fix-permission-issues branch August 26, 2026 04:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant