chore: clean up remaining sf-2 stale-name code comments

Final sweep after the prompt + script + README sweep for stale repo
references. These are pure code comments, not active behavior, but they
mislead readers about what repo this code lives in:

- src/resource-loader.ts: "sf-2 repo's working tree" → "sf-run repo's"
- src/web/safe-import-meta-resolve.ts: example URL hostname
- src/resources/extensions/sf/schemas/parsers.ts: dropped "sf-2 /" prefix
- src/resources/extensions/sf/schemas/validate.ts: same
- scripts/parallel-monitor.mjs: comment about "sf-2 repo itself"

Tests intentionally not touched — the test fixtures use @sf-build as a
generic scope name to exercise the symlink-merge logic, and the test
tmpdir prefixes (sf-2821-, sf-2945-) are just numeric tags from issue
numbers, not repo refs.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
Mikael Hugo 2026-05-02 18:14:22 +02:00
parent 56234b5131
commit b046bc1687
5 changed files with 5 additions and 5 deletions

View file

@ -276,7 +276,7 @@ function extractCostFromNdjson(mid) {
// Auto-detect the SF loader path — works across npm global, homebrew, and local installs
function findSfLoader() {
// 1. Check if we're running from inside the sf-2 repo itself
// 1. Check if we're running from inside the sf-run repo itself
const repoLoader = path.resolve(import.meta.dirname, '..', 'dist', 'loader.js');
if (fs.existsSync(repoLoader)) return repoLoader;

View file

@ -39,7 +39,7 @@ import { compareSemver } from "./update-check.js";
// Resolve resources directory — prefer dist/resources/ (stable, set at build time)
// over src/resources/ (live working tree, changes with git branch).
//
// Why this matters: with `npm link`, src/resources/ points into the sf-2 repo's
// Why this matters: with `npm link`, src/resources/ points into the sf-run repo's
// working tree. Switching branches there changes src/resources/ for ALL projects
// that use sf — causing stale/broken extensions to be synced to ~/.sf/agent/.
// dist/resources/ is populated by the build step (`npm run copy-resources`) and

View file

@ -1,4 +1,4 @@
// sf-2 / Deep planning mode — Markdown → structured object parsers for artifact validation.
// Deep planning mode — Markdown → structured object parsers for artifact validation.
//
// Each parser converts a markdown artifact into a typed object suitable for
// JSON Schema validation. The parsers are intentionally minimal — they only

View file

@ -1,4 +1,4 @@
// sf-2 / Deep planning mode — Artifact validator entry point.
// Deep planning mode — Artifact validator entry point.
//
// Validates PROJECT.md, REQUIREMENTS.md, and per-milestone ROADMAP.md
// against the contract spec in .planning/phases/11-deep-planning-mode/11-CONTRACTS.md.

View file

@ -5,7 +5,7 @@ import { fileURLToPath } from "node:url";
* Derive a package root from an import.meta.url, returning null on failure.
*
* The Next.js standalone build bakes import.meta.url as the CI runner's
* absolute path (e.g. file:///home/runner/work/sf-2/sf-2/src/web/bridge-service.ts).
* absolute path (e.g. file:///home/runner/work/sf-run/sf-run/src/web/bridge-service.ts).
* On Windows, fileURLToPath() rejects this Linux path with
* "File URL path must be absolute".
*