chore: update platform repo paths and stale org name - #259
Open
mergesort wants to merge 1 commit into
Open
Conversation
test-pilot-server is merging into Pursuit-Org/platform as server/, and the Pursuit-Assets org was renamed to Pursuit-Org. - 11 scripts load_dotenv() from a sibling test-pilot-server checkout; the paths now point at platform/server. These are absolute paths on one developer's machine, so they need checking against a real checkout. - account-dedupe-and-mirror-plan.md describes the repo layout. - Pursuit-Assets -> Pursuit-Org in JOBS_QUICKSTART, BRANCH_PROTECTION and two task docs. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
mergesort
force-pushed
the
chore/platform-monorepo-and-org-rename
branch
from
August 4, 2026 17:00
fa202e6 to
5437687
Compare
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
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.
Part of the
test-pilot-server+pilot-client→Pursuit-Org/platformmigration, plus adjacent org-name cleanup.Note
@Jacquelinereverand — the first section needs your eyes specifically. It edits hardcoded paths on your machine that I can't verify.
Local
.envpaths (11 scripts)Eleven scripts under
financial_forecasting/scripts/bootstrap credentials from a sibling backend checkout:I applied the mechanical translation —
.../test-pilot-server/.env→.../platform/server/.env— which is right only if the monorepo gets cloned to~/dev/build/platform. If it lands anywhere else, these need a different value.Worth flagging while it's open: these are absolute paths into one person's home directory, so they only work for one developer and break on any re-clone. Reading the path from an env var with a fallback would make the scripts portable. I didn't do that here because it's a behaviour change beyond the migration — happy to follow up if you want it.
Two scripts document the same path as a
--envCLI argument (apply_sql.py,apply_jobs_roles_v2.py); those are relative (../platform/server/.env) and correspondingly less fragile.tasks/account-dedupe-and-mirror-plan.mdThe "Repos:" line described the two-repo layout. Rewritten for the monorepo. Note I deliberately kept the reference to
"/Users/jacquelinereverand/old platform folder/test-pilot-server"— that's a local checkout of the pre-merge backend, and the doc's parenthetical ("current despite name") is still the useful thing to say about it. Also re-rootedtest-pilot-server/controllers/employmentEngineController.js→server/controllers/....Pursuit-Assets→Pursuit-Orgfinancial_forecasting/JOBS_QUICKSTART.md(4, including agit clone),.github/BRANCH_PROTECTION.md(a settings URL),tasks/jac-running-notes.mdandtasks/objects-production-readiness-plan.md(PR links).Severity: low. I verified
Pursuit-Assetsno longer exists as an org and GitHub redirects the old URLs, so the clone command and PR links still work. Hygiene against the redirect lapsing — though the JOBS_QUICKSTART one is worth having right since someone follows it by hand during setup.