Skip to content

Setup Codex sandbox#1547

Merged
white238 merged 19 commits intodevelopfrom
feature/white238/codex-sandbox
Mar 11, 2026
Merged

Setup Codex sandbox#1547
white238 merged 19 commits intodevelopfrom
feature/white238/codex-sandbox

Conversation

@white238
Copy link
Member

@white238 white238 commented Feb 23, 2026

This is aimed at a good base for using the Codex Sandbox on LC. I put in effort to allow it to work on non-LC machines but haven't tried it personally. For example, the compute/login node script will default to compute if not on LC and put some guardrails recommended by ChatGPT that might sandbox you outside of a sandbox.

To be very clear, these are here to help but cannot be fully trusted. Only use Agentic AI on repos that you know dont have any work you care to lose since it can still revert/commit/delete.

@btalamini btalamini added the WIP Work in progress label Feb 25, 2026
@white238 white238 force-pushed the feature/white238/codex-sandbox branch from 9705d9e to bf22c8c Compare March 2, 2026 22:49
@white238 white238 added CS Computer science and software engineering ready for review Ready for active inspection by reviewers and removed WIP Work in progress labels Mar 4, 2026
Copy link
Collaborator

@tupek2 tupek2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you are interested in getting into the next level of this agentic stuff (which I haven't done yet), I'd recommend you research custom /slash commands (maybe for build or testing), and also the use of LLM hooks (where, for example, you can make a hook that after the agent makes code changes, it automatically builds the code, runs the tests, makes style, etc). Just something to be aware of that I'm not sure the initial tutorial covered. I'm personally more interested in these things than skills, which are mostly ignored.

2) Configure (recommended wrapper around CMake):

```bash
./config-build.py -bp build -ip install -hc host-configs/<file>.cmake -bt Debug --exportcompilercommands
Copy link
Collaborator

@tupek2 tupek2 Mar 5, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this likely to default to everyone running debug by default? Its pretty common for people to tell me: 'this or that smith problem is slow' only for me to (after a day of work) realize they were default building debug. Now my default response to these this is: just build in release (which they often still ignore). I personally rarely build, test, or debug in debug. Maybe we want a release_build skill, and a debug_build skill.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Swapped to RelWithDebInfo by default unless specifically asked for a debug build.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Release please. I people want to debug, the should probably run in debug.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we should discuss this in the technical meeting. RelWithDebInfo is nearly as fast (-O2 vs -O3) but does get you debug symbols. I was envisioning these files are mainly for out development. Are you expecting them to be used by people outside of our team? I feel this is a healthy middle ground and if someone really cares they can put it in their agents file or just tell it to use release.

Copy link
Member Author

@white238 white238 Mar 10, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I removed the build type from all command except the asan build where i told it to default to debug. I also told it to use the -bt <CMake Build Type> option if asked. I verified it is now finding the skills w/ the new symlink and tested several different variations of prompts and it seems to work like I would expect at this point.

Side thought: I like that I asked codex to make these skills and iterated on them with it.. but it couldn't find them after. It does now.

@btalamini btalamini requested a review from ebchin March 10, 2026 22:25
@tupek2 tupek2 self-requested a review March 10, 2026 22:27
@white238
Copy link
Member Author

dont merge this.. i just found out its not using my skills and how to fix it.

@white238 white238 force-pushed the feature/white238/codex-sandbox branch from 7e31674 to 01451f1 Compare March 10, 2026 22:54
@white238
Copy link
Member Author

dont merge this.. i just found out its not using my skills and how to fix it.

Fixed now.

@white238
Copy link
Member Author

Example of AGENTS.md that defaults to building debug if that is better for you:

# AGENTS.md

## Build Configuration

Unless explicitly instructed otherwise, always configure and build the project in **Debug** mode.

Do not default to Release or RelWithDebInfo builds.

You can also say "build in debug and with GCC". That worked for me.

@white238 white238 merged commit ee6d154 into develop Mar 11, 2026
10 checks passed
@white238 white238 deleted the feature/white238/codex-sandbox branch March 11, 2026 07:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CS Computer science and software engineering ready for review Ready for active inspection by reviewers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants