fix(harness): remove hardcoded filesystem paths for portability#1
Open
sixty4bit wants to merge 1 commit intoworkos:mainfrom
Open
fix(harness): remove hardcoded filesystem paths for portability#1sixty4bit wants to merge 1 commit intoworkos:mainfrom
sixty4bit wants to merge 1 commit intoworkos:mainfrom
Conversation
All references to /Users/nicknisi/Developer/case have been replaced
with dynamic resolution so the harness works for any user/fork.
Shell scripts and hooks now derive CASE_REPO from their own location
using the SCRIPT_DIR pattern (matching bootstrap.sh). SKILL.md and
agent prompts use ${CASE_REPO} as a variable resolved by the
orchestrator at invocation time. Each agent's Input contract now
includes CASE_REPO as an explicit parameter.
upload-screenshot.sh now requires the CASE_ASSETS_REPO env var
instead of defaulting to a specific user's repo, with clear setup
instructions on failure.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
6 tasks
nicknisi
added a commit
that referenced
this pull request
Mar 17, 2026
- Remove double-written metrics: drop legacy log-run.sh call from pipeline.ts, writer.ts is the single path (#1) - Await retrospective instead of fire-and-forget so process.exit doesn't kill it (#3) - Fix README claiming retrospective applies changes directly — it proposes amendments, only learnings are applied directly (#6) - Document attended-mode retry semantics: maxRetries is per-attempt, human can re-enter implement indefinitely (#2) - Reconcile --worktree docs: removed from flag list, branch isolation is handled by skill layer before orchestrator dispatch (#4)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
/Users/nicknisi/Developer/casereferences (70+ occurrences across 22 files) with dynamic resolution so the harness works for any user or fork.CASE_REPOfrom their own location using theSCRIPT_DIRpattern (matchingbootstrap.sh).${CASE_REPO}as a variable resolved by the orchestrator at invocation time. Each agent's Input contract now includesCASE_REPOas an explicit parameter.upload-screenshot.shnow requires theCASE_ASSETS_REPOenv var instead of defaulting to a specific user's repo, failing fast with clear setup instructions.Test plan
bash -nsyntax check/casefrom a target repo to verify orchestrator resolvesCASE_REPOcorrectlyupload-screenshot.shfails with helpful message whenCASE_ASSETS_REPOis unset${CLAUDE_PLUGIN_ROOT}🤖 Generated with Claude Code