Skip to content

feat(ts): convert CLI, REPL, sessions layer to TypeScript (Phase 5) - #61

Merged
graphanov merged 1 commit into
mainfrom
forge/phase-5-cli-repl-sessions-ts
Jun 29, 2026
Merged

feat(ts): convert CLI, REPL, sessions layer to TypeScript (Phase 5)#61
graphanov merged 1 commit into
mainfrom
forge/phase-5-cli-repl-sessions-ts

Conversation

@graphanov

Copy link
Copy Markdown
Owner

Summary

Scope

  • src/cli.ts, src/cli-output.ts, src/repl.ts, src/sessions.ts, src/status.ts, src/doctor.ts, src/onboard.ts, src/installer.ts, src/update.ts, src/banner.ts, src/ulw.ts, src/agent/context.ts, src/agent/deadline.ts, src/agent/thinking.ts, src/prompt.ts, src/scaffold/handoff.ts. Tests re-pointed to dist/. Docs updated.

Out of scope

  • merge/publish/release; build pipeline (Phase 1); other module conversions (Phases 2-4); version bump/tag/publish; test file conversion.

Verification

  • npm run typecheck (tsc --noEmit) -> clean; npm run build (tsc) -> clean; npm test (node --test test/.test.mjs) -> 445/445 pass, 0 fail; node bin/lazyglm.js --version -> lazyglm 0.1.4; node bin/lazyglm.js doctor -> 8/11 ok; npm pack --dry-run -> 58 files, version 0.1.4; find src -name '.js' -> empty

Risk

  • low — pure .js to .ts mechanical conversion preserving NodeNext .js import specifiers; no runtime behavior changes

Linked issue

Closes #53

Authority boundary

john-lomein did not merge, publish, release, dispatch workflows, change settings, force-push, rewrite history, or touch secrets.

Convert the remaining runtime JavaScript modules to TypeScript, completing
the five-phase TypeScript migration tracked in #28.

Converted modules:
- src/cli.ts, src/cli-output.ts, src/repl.ts, src/sessions.ts
- src/status.ts, src/doctor.ts, src/onboard.ts, src/installer.ts
- src/update.ts, src/banner.ts, src/ulw.ts
- src/agent/context.ts, src/agent/deadline.ts, src/agent/thinking.ts
- src/prompt.ts, src/scaffold/handoff.ts

No runtime .js source remains under src/. NodeNext .js import specifiers
are preserved in source. Tests import from dist/ after build, matching
the package runtime boundary.

Docs updated to reflect the final phase and .ts paths.

Closes #53
@graphanov
graphanov marked this pull request as ready for review June 29, 2026 16:30
@graphanov

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. More of your lovely PRs please.

Reviewed commit: bbb6f01037

ℹ️ 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".

@graphanov
graphanov merged commit 846a011 into main Jun 29, 2026
1 check passed
@graphanov
graphanov deleted the forge/phase-5-cli-repl-sessions-ts branch June 29, 2026 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Phase 5: Convert CLI, REPL, and sessions layer to TypeScript (.js → .ts)

1 participant