fix: harden repo-local helper launcher resolution#90
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ce4c11c1c7
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| if (result.error.code === 'ENOENT') return false; | ||
| throw result.error; | ||
| } | ||
| process.exit(result.status ?? 0); |
There was a problem hiding this comment.
spawnSync returns status: null and sets signal when the child is terminated, but process.exit(result.status ?? 0) converts that into exit code 0. As a result, node .agents/bin/gsdd.mjs ... can report success even when the underlying gsdd command was killed (for example by SIGTERM/OOM/timeout), which allows workflows to continue after failed file-op/phase-status mutations. Treat signaled exits as failures (or re-emit the signal) instead of coercing them to success.
Useful? React with 👍 / 👎.
| } | ||
|
|
||
| if (platforms.length > 0 || existsSync(join(ctx.cwd, '.agents', 'skills'))) { | ||
| if (platforms.length > 0 || existsSync(join(ctx.cwd, '.agents'))) { |
There was a problem hiding this comment.
This condition now treats any existing .agents/ directory as evidence that GSDD portable surfaces are installed. In repos that use .agents/ for unrelated tooling, gsdd update will unexpectedly generate .agents/skills/gsdd-* and .agents/bin/gsdd.mjs even when gsdd init was never run, causing unintended repository mutations. The trigger should check for explicit GSDD-owned paths (for example .agents/bin/gsdd.mjs or gsdd-* skill folders) rather than the top-level .agents directory.
Useful? React with 👍 / 👎.
1 participant
Summary
.agents/bin/gsdd.mjslauncher as part of the portable runtime surfacegsdd ...gsdd-cli@<version>gsdd updateRoot cause
Generated workflows assumed helper commands could call bare
gsdd ..., butgsdd initonly guaranteed generated files inside the repo, not an ambient PATH install. The first launcher draft still left two gaps: partial.agents/installs were not fully self-repaired bygsdd update, and ambient PATH resolution could drift to the wrong CLI version.Testing
node tests/gsdd.init.test.cjsnode tests/gsdd.manifest.test.cjsnode tests/phase.test.cjsnode tests/gsdd.scenarios.test.cjsnode tests/gsdd.health.test.cjsnode tests/gsdd.cross-runtime.test.cjsnode --test-name-pattern="Phase 18 deterministic CLI guards|G45 - Runtime Surface Freshness Contract|G10 - CLI Module Boundary" tests/gsdd.guards.test.cjsnode tests/gsdd.invariants.test.cjs