Skip to content

fix(proving-ground): use local runx evolve skill#159

Merged
auscaster merged 1 commit into
mainfrom
codex/aster-proving-ground-local-evolve
May 13, 2026
Merged

fix(proving-ground): use local runx evolve skill#159
auscaster merged 1 commit into
mainfrom
codex/aster-proving-ground-local-evolve

Conversation

@auscaster

@auscaster auscaster commented May 13, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • invoke the checked-out runx evolve skill by local path in proving-ground
  • assemble optional answers arguments without tripping set -u when no answer file is present

Validation

  • RUNX_ROOT=/Users/kam/dev/runx-oss-thread-story ASTER_ROOT=/Users/kam/dev/runx/aster ARTIFACT_DIR=/tmp/aster-proving-ground-test PROVING_GROUND_PROFILE=minimal bash scripts/proving-ground.sh
    • This runs with no $RUNX_ANSWERS_DIR, so it directly exercises the no-answer-file branch that previously tripped set -u.
  • node scripts/summarize-proving-ground.mjs /tmp/aster-proving-ground-test
  • npm run check
  • git diff --check
  • bash -n scripts/proving-ground.sh

@auscaster

Copy link
Copy Markdown
Collaborator Author

I'd like one more validation detail in the PR body: does the listed proving-ground run actually exercise the no-answer-file path? Since half the change is about avoiding set -u there, please either call out that PROVING_GROUND_PROFILE=minimal hits that branch or add a small targeted check for it.

Head SHA: e070994

@auscaster auscaster merged commit 873bb77 into main May 13, 2026
4 checks passed
@auscaster auscaster deleted the codex/aster-proving-ground-local-evolve branch May 13, 2026 17:27
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