Skip to content

Harden Nightshift from live validation runs#13

Merged
fazxes merged 10 commits intomainfrom
feat/codex-nightshift-runner
Apr 4, 2026
Merged

Harden Nightshift from live validation runs#13
fazxes merged 10 commits intomainfrom
feat/codex-nightshift-runner

Conversation

@HarshyChoc
Copy link
Copy Markdown
Collaborator

Summary

  • harden Nightshift worktree and target-repo validation based on live Codex/tmux runs
  • enforce more of the cycle contract in the runner instead of relying on prompt compliance
  • improve short-run fix selection by steering the agent toward high-signal auth/http/api helper paths

What changed

  • validate and recover broken worktrees before a cycle starts
  • fail fast with a clear error when the target repo is not a valid git checkout
  • reject repo-wide cycle commands like npm run lint, npm run test, and npm run build
  • verify forbidden commands from structured output fields as well as raw Codex JSONL
  • enforce one-fix-one-commit cycle accounting
  • avoid generating blocked npm lockfiles in lockfile-less repos during dependency install
  • add repo-aware high-signal focus hints for short validation runs

Validation

  • full test suite passes locally: 164 passed
  • live Nightshift runs were exercised against a fresh standalone copy of Reclip-Reposter-Hub
  • those runs surfaced runner issues which drove the commits on this branch
  • latest clean target run produced a relevant logged security issue (parse-social-url hostname validation) that matched manual repo review

Notable commits

  • 7883afe validate broken worktrees before cycles
  • d51e0b0 surface invalid repo checkouts early
  • b9e6d5e reject forbidden repo-wide cycle commands
  • 96d1de0 enforce one-commit-per-fix cycles
  • 456065b avoid generating blocked npm lockfiles
  • d85c1f5 steer short runs toward high-signal paths

HarshyChoc and others added 10 commits April 3, 2026 20:03
Both branches added features to the same files:
- PR #13: forbidden cycle commands, high-signal focus paths, worktree
  validation, repo checkout validation, one-commit-per-fix enforcement
- main: backend forcing, test escalation, state injection, learnings

Merged both feature sets, added focus_hints parameter to test helpers.
208 tests passing.
@fazxes fazxes merged commit 43f1645 into main Apr 4, 2026
@fazxes fazxes deleted the feat/codex-nightshift-runner branch April 4, 2026 01:13
fazxes pushed a commit that referenced this pull request Apr 4, 2026
- Simplify high_signal_focus_paths() by removing redundant fallback (#16)
- Broaden _extract_shell_command regex to handle bash/sh/dash wrappers (#17)
- Add docstring to expected_cycle_commits() documenting commit contract (#18)
- Add Python/Go/Rust paths to HIGH_SIGNAL_PATH_CANDIDATES (#19)

https://claude.ai/code/session_011hjFhGSxRM4pWsWN1r4GSM
fazxes pushed a commit that referenced this pull request Apr 4, 2026
- Simplify high_signal_focus_paths() by removing redundant fallback (#16)
- Broaden _extract_shell_command regex to handle bash/sh/dash wrappers (#17)
- Add docstring to expected_cycle_commits() documenting commit contract (#18)
- Add Python/Go/Rust paths to HIGH_SIGNAL_PATH_CANDIDATES (#19)

https://claude.ai/code/session_011hjFhGSxRM4pWsWN1r4GSM
fazxes added a commit that referenced this pull request Apr 4, 2026
Address quality issues from PR #13 review
fazxes added a commit that referenced this pull request Apr 9, 2026
fazxes added a commit that referenced this pull request Apr 9, 2026
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.

2 participants