Skip to content
This repository was archived by the owner on Oct 9, 2026. It is now read-only.
This repository was archived by the owner on Oct 9, 2026. It is now read-only.

Fix exec-harness automation reliability gaps #31

Closed

Description

@shiny-code-bot

Objective

Fix the known exec-harness automation gaps before relying on the harness as the regression gate for Codex Lab-specific behavior.

Finish Line

Known exec-harness automation gaps are fixed and covered by focused validation.

Current Status

State: Active. Next action: create a task branch and reproduce the just local-cleanup-space --apply argument forwarding failure.

Scope

In:

  • Fix just local-cleanup-space --apply argument forwarding.
  • Update .github/workflows/exec-harness.yml path filters to include local helper scripts.
  • Add focused validation for dry-run versus apply behavior where practical.

Out:

  • No broad harness refactor.
  • No feature scenario work beyond what is needed to prove the automation fixes.

Acceptance Criteria

  • just local-cleanup-space remains a dry run.
  • just local-cleanup-space --apply forwards --apply to scripts/local/cleanup-space.sh.
  • Exec-harness workflow triggers when scripts/local/exec-harness-env.sh or scripts/local/cleanup-space.sh changes.
  • PR validation records the exact commands or CI evidence used.

Relationships

Child of #28. This should be completed before new harness scenarios become required gates. Related to #22 for broader fork CI posture.

Validation

Run the focused shell/script tests added for this change, plus any existing local script test target that owns these helpers.

Decisions

This is the first implementation slice after plan review because broken automation can silently invalidate later regression work.

Open Questions

Should scripts/local/** be the path filter, or should the workflow enumerate only the two known helper scripts?

Activity

  1. added
    plan:donePlan completed or superseded
    and removed
    plan:activePlan is actionable now
    on Jun 12, 2026
  2. shiny-code-bot commented on Jun 12, 2026

    @shiny-code-bot
    CollaboratorAuthor

    Completed by PR #38, merged as 32bf457127406added395e271492907c18eab180.

    What landed:

    • just local-cleanup-space now forwards arguments through the repo's {args} just-shell token.
    • exec-harness PR path filters now include scripts/just-shell.py, scripts/local/cleanup-space.sh, and scripts/local/exec-harness-env.sh.
    • Added focused tests for dry-run, apply, and --keep-exec-harness-cache behavior in tools/codex-exec-harness/test_local_cleanup.py.

    Validation recorded in PR #38:

    • python3 -m unittest tools/codex-exec-harness/test_local_cleanup.py
    • python3 -m unittest discover -s tools/codex-exec-harness -p 'test_*.py'
    • just fmt
    • PR CI: 4/4 checks passed before merge.

    Note: local actionlint parsed the workflow but reported the existing custom self-hosted runner label chris-testing-codex as unknown to local actionlint configuration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    planDurable planning issueplan:donePlan completed or superseded

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions