Skip to content

[codex] Minimal IPython edit prompt guidance#99

Merged
samsja merged 1 commit into
mainfrom
codex/ipython-minimal-edit-prompt
Jun 12, 2026
Merged

[codex] Minimal IPython edit prompt guidance#99
samsja merged 1 commit into
mainfrom
codex/ipython-minimal-edit-prompt

Conversation

@samsja

@samsja samsja commented Jun 12, 2026

Copy link
Copy Markdown
Member

Summary

Minimal alternative to the larger IPython prompt PR.

Changes:

  • adds one short reminder that %%bash must be the first line of an IPython code cell
  • adds a compact edit-skill hint only when edit is installed
  • keeps a single safe await edit(...) example using triple-single-quoted variables for texts that may contain triple double quotes
  • adds focused prompt tests

No runtime guard and no IPython tool schema changes.

Validation

  • uv run --frozen ruff check --isolated . => passed
  • uv run --frozen ruff format --check --isolated . => 31 files already formatted
  • uv run --frozen pytest tests/test_prompt.py tests/test_git_block.py -q => 62 passed

Note

Low Risk
Changes are limited to system prompt text and unit tests; no runtime, tool schema, or guard behavior.

Overview
Tightens IPython system-prompt guidance and adds a conditional edit skill hint.

The IPython section now states that %%bash must be the first line of a code cell (no leading comments, blanks, imports, or Python). When edit is among installed skills, the prompt adds a short block recommending await edit(path=..., old_str=..., new_str=...) with triple-single-quoted old/new and exact string matching.

Tests cover the new %%bash wording and assert EDIT_SKILL_PROMPT appears only when edit is installed.

Reviewed by Cursor Bugbot for commit b47e19f. Bugbot is set up for automated code reviews on this repo. Configure here.

@samsja samsja marked this pull request as ready for review June 12, 2026 04:36
@samsja samsja merged commit 1d7831a into main Jun 12, 2026
8 checks passed
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