|
| 1 | +# Contributing |
| 2 | + |
| 3 | +Contributions that make identity selection safer, clearer, or more portable are |
| 4 | +welcome. Please keep changes focused and avoid adding credential storage or |
| 5 | +provider-specific assumptions to the common workflow. |
| 6 | + |
| 7 | +## Before opening an issue or pull request |
| 8 | + |
| 9 | +- Never include credentials, access tokens, authentication files, private |
| 10 | + research data, participant data, or unredacted logs. |
| 11 | +- Report a suspected vulnerability privately according to |
| 12 | + [SECURITY.md](SECURITY.md), not in a public issue. |
| 13 | +- Search existing issues and test the current `main` branch when practical. |
| 14 | + |
| 15 | +## Development setup |
| 16 | + |
| 17 | +Follow [INSTALLATION.md](INSTALLATION.md), then run: |
| 18 | + |
| 19 | +```bash |
| 20 | +./tests/test.sh |
| 21 | +``` |
| 22 | + |
| 23 | +Keep the helper compatible with Bash 3.2, including the system Bash on macOS. |
| 24 | +The default tests must run without credentials or a live provider. |
| 25 | + |
| 26 | +Generated configuration has been checked with Codex CLI 0.145.0. Rerun the |
| 27 | +tests after updating Codex because its settings and provider support can change. |
| 28 | + |
| 29 | +## Making a change |
| 30 | + |
| 31 | +1. Create a short-lived branch from `main`. |
| 32 | +2. Add or update regression tests for behavior changes. |
| 33 | +3. Update user documentation when commands, generated files, risks, or |
| 34 | + prerequisites change. |
| 35 | +4. Run the test suite and Bash syntax checks locally: |
| 36 | + |
| 37 | + ```bash |
| 38 | + bash -n bin/codex-home bin/install-macos-open-with tests/test.sh |
| 39 | + ./tests/test.sh |
| 40 | + ``` |
| 41 | + |
| 42 | +5. Open a pull request explaining what changes for users, how you tested it, |
| 43 | + and any remaining limits. Use synthetic examples and remove sensitive |
| 44 | + information from logs. |
| 45 | + |
| 46 | +For a new API provider or model path, document the exact provider, protocol, |
| 47 | +model, authentication method, and tested tool behavior. A successful model |
| 48 | +listing or plain-text response is not enough to establish full Codex |
| 49 | +compatibility. |
| 50 | + |
| 51 | +## Update the printable cheatsheet |
| 52 | + |
| 53 | +Edit [docs/build-cheatsheet.py](docs/build-cheatsheet.py), then rebuild the PDF |
| 54 | +in a Python environment with ReportLab installed: |
| 55 | + |
| 56 | +```bash |
| 57 | +python3 docs/build-cheatsheet.py |
| 58 | +``` |
| 59 | + |
| 60 | +Keep it to two pages and inspect both pages after rebuilding. Command text |
| 61 | +should remain selectable and fit without clipping. |
0 commit comments