Skip to content

#4059 - #4060

Closed
KATO-Hiro wants to merge 2 commits into
AtCoder-NoviSteps:stagingfrom
KATO-Hiro:#4059
Closed

#4059#4060
KATO-Hiro wants to merge 2 commits into
AtCoder-NoviSteps:stagingfrom
KATO-Hiro:#4059

Conversation

@KATO-Hiro

Copy link
Copy Markdown
Collaborator

No description provided.

KATO-Hiro and others added 2 commits September 19, 2026 05:11
- Dockerfile: remove chmod u+s on /usr/bin/bwrap
- compose.yaml: unmask /proc via systempaths=unconfined so bubblewrap
  0.12.0 can mount a fresh procfs without setuid
- .claude/settings.json, .codex/config.toml: consolidate credential
  denies onto directory-level entries to avoid multi-file masking
  issues (openai/codex#43929) and drop the duplicate **/.env.* glob
- .devcontainer/setup-devcontainer.sh: install the pnpm version
  pinned in packageManager before pnpm install, so sandboxed pnpm
  commands don't try to download a mismatched version
- docs/dev-notes: plan/survey/learning for the bwrap setuid fix

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Merge plan.md, survey.md and learning.md into summary.md, keeping the
decisions, trade-offs, root causes of the detours and primary sources.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@KATO-Hiro KATO-Hiro closed this Sep 19, 2026
@coderabbitai

coderabbitai Bot commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Repository: AtCoder-NoviSteps/AtCoderNoviSteps/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 056e590c-c221-476d-85d5-49bfb692ec1a

📥 Commits

Reviewing files that changed from the base of the PR and between 0f761ac and a3a1aff.

📒 Files selected for processing (6)
  • .claude/settings.json
  • .codex/config.toml
  • .devcontainer/setup-devcontainer.sh
  • Dockerfile
  • compose.yaml
  • docs/dev-notes/2026-09-18/fix-bwrap-setuid/summary.md
 _____________________________________________________________
< Oompa Loompa doompadee doo, I've got a code review for you. >
 -------------------------------------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

This branch had an error being deployed

1 failed deployment
preview — a3a1aff7 Deployed Sep 19, 2026 by KATO-Hiro via preview #8727
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.

1 participant