fix(opencode): restore cwd-agnostic opencodealt entry - #6
Conversation
Machine-local opencodealt.bat points at index.alterspective.ts, which was never in the repo, so the launcher failed with Module not found. Add the entry so @opentui/solid/preload registers before loading index.ts when cwd is not the package directory, and document the launcher in AGENTS.md. No user-doc impact: developer/fork launcher only; no product surface change.
|
Thanks for your contribution! This PR doesn't have a linked issue. All PRs must reference an existing issue. Please:
See CONTRIBUTING.md for details. |
📝 WalkthroughWalkthroughChangesAlterspective launcher
Estimated code review effort: 1 (Trivial) | ~5 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
|
Thanks for updating your PR! It now meets our contributing guidelines. 👍 |
PR-loop disposition (cycle 1)Reviewers (current head
|
Issue for this PR
Closes #7
Type of change
What does this PR do?
opencodealt(machine-local Nodist launcher for this Alterspective fork) failed immediately withModule not foundforpackages/opencode/src/index.alterspective.ts. That path was hard-coded inopencodealt.batbut the entry file was never committed.This PR adds the cwd-agnostic entry: it registers
@opentui/solid/preload(required because bun only appliesbunfig.tomlpreloads from the process cwd), then dynamically imports./index.ts. Also documents the launcher contract inAGENTS.mdfork-local notes.How did you verify your code works?
%TEMP%:bun run --conditions=browser <path>/packages/opencode/src/index.alterspective.ts --version→local(exit 0)--help→ full CLI helpopencodealt --version/--helpfrom arbitrary cwd succeeds (was Module not found)Screenshots / recordings
N/A — launcher/CLI entry only, no UI change.
Checklist
No user-doc impact: developer/fork launcher only; no product surface change.