Skip to content

Conversation

@rfreel
Copy link
Owner

@rfreel rfreel commented Jan 22, 2026

Motivation

  • Implement the first tranche (Top 10) of the MECE Pareto Bash toolset so the canonical loop can run a single PATCH-ONLY iteration with deterministic contracts.
  • Make the loop safe and CI-enforced by adding gate/invariant tooling and a stub model adapter so CI can validate script contracts and loop invariants.

Description

  • Added executable Ralph scripts and helpers: scripts/ralph/step.sh, scripts/ralph/apply_patch.sh, scripts/ralph/slice.sh, scripts/ralph/summarize_log.sh, scripts/ralph/precommit_gate.sh, scripts/ralph/ci_gate.sh, scripts/ralph/invariants.sh, and scripts/ralph/diff_budget.sh and updated scripts/ralph/loop.sh, scripts/ralph/radius.sh, and scripts/ralph/llm_stub.sh to conform to the required contracts (strict mode, --help, --self-test, deterministic outputs, and exit codes).
  • Wired CI to perform script contract checks and gates by updating .github/workflows/ci.yml to call each script with --help and --self-test, run scripts/ralph/invariants.sh and scripts/ralph/ci_gate.sh, and run pre-commit.
  • Updated .pre-commit-config.yaml to include a fast invariants local hook and kept the radius enforcement hook; updated README.md and AGENTS.md to document the PATCH-ONLY pipeline and one-iteration flow.
  • Updated project metadata to record the tranche: added SHELL-010 to prd.json and appended an entry to progress.txt indicating the top-10 tranche was added.

Testing

  • No automated CI tests were executed in this change set locally; CI has been updated to run scripts/ralph/invariants.sh --self-test, scripts/ralph/invariants.sh, scripts/ralph/ci_gate.sh, and pre-commit run --all-files on PRs and pushes and will validate the script contract checks (--help/--self-test) as configured in .github/workflows/ci.yml.

Codex Task

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 323fcf67f8

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +89 to +90
else
if git ls-files -z | xargs -0 grep -n "^(<<<<<<<|=======|>>>>>>>)" >/dev/null 2>&1; then

Choose a reason for hiding this comment

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

P2 Badge Use extended regex for conflict marker scan

When rg is unavailable, the fallback uses grep -n "^(<<<<<<<|=======|>>>>>>>)", but basic grep treats | literally, so it only matches a line containing the literal string <<<<<<<|=======|>>>>>>> rather than any real conflict marker. In minimal environments without ripgrep, merge conflicts will slip past invariants.sh and CI even though the gate claims to enforce them. Consider grep -E (or escaping \|) so the alternation works as intended.

Useful? React with 👍 / 👎.

@rfreel rfreel closed this Jan 28, 2026
@rfreel
Copy link
Owner Author

rfreel commented Jan 28, 2026

c

@rfreel rfreel deleted the codex/implement-top-10-pareto-scripts-with-ci branch January 28, 2026 00:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants