fix(ops): update hardcoded old-PC paths for new machine - #107
Open
lwalden wants to merge 1 commit into
Open
Conversation
… C:\repos) The 2026-07-24 machine migration moved all repos from D:\Source to C:\repos, and the D: drive no longer exists. The ops scripts' path DEFAULTS still pointed at the old layout, so preflight.ps1 and Check-DailySnapshot.ps1 would fail (or silently check the wrong file of record) when run without explicit -DataDirectory/-SnapshotPath/ -SettingsPath overrides. The paper-validation runbook's copy-paste commands had the same problem. Updated script defaults, the Check-DailySnapshot help text describing the worker cwd/file of record, and the runbook's three D:\Source commands. Decision records are untouched -- they describe the past. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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
The 2026-07-24 machine migration moved all repos from
D:\SourcetoC:\repos, and the D: drive no longer exists on this machine. Ops script defaults and runbook commands still pointed at the old layout.What changed:
tools/ops/preflight.ps1—$DataDirectorydefaultD:\Source\...->C:\repos\TradingSystem\src\TradingSystem.Functions\datatools/ops/Check-DailySnapshot.ps1—$SnapshotPath/$SettingsPathdefaults updated toC:\repos\...; the.DESCRIPTIONhelp text describing the worker cwd and file of record updated to match (with a note that the 2026-06-10 verification predates the path migration)docs/paper-validation-runbook.md— three copy-paste commands updated (cdto the Functions dir, the claude-gateway restart path, and theRegister-GapDayMonitorTask.ps1invocation)Deliberately untouched: DECISIONS.md and other decision-record history — they describe the past. A repo-wide grep confirms zero remaining
D:\Source(or anyD:\) literals in scripts, docs, or config.Test plan
grep -r 'D:\\Source'across the repo returns no hitsdotnet restore/build/testpass on this machine (script/doc-only change; baseline run on this branch)tools/ops/preflight.ps1with no arguments and confirm the data-directory check passes againstC:\repos\TradingSystem\src\TradingSystem.Functions\data🤖 Generated with Claude Code