Skip to content

#260 & #49: switch to TypeScript#683

Draft
ntseng wants to merge 4 commits into
mainfrom
ntseng/issue260
Draft

#260 & #49: switch to TypeScript#683
ntseng wants to merge 4 commits into
mainfrom
ntseng/issue260

Conversation

@ntseng
Copy link
Copy Markdown
Member

@ntseng ntseng commented May 30, 2026

Summary

Very unfinished, but leaving the draft here for early feedback and viewing progress over time.

  • change package type from commonjs to module
  • rewrite most files to use TypeScript
  • rework runMode logic
    • add run mode to all logging timestamps
    • check for dev mode instead of prod mode (test mode is "pretending to be" prod mode)
    • make enum for run modes
    • pass isDevMode boolean instead of entire mode enum
  • Fix model configuration so that types populate to instances #49: add property type declarations to models

Local Tests Performed

  • bot still turns on (no build errors, circular dependencies, or start-up errors)
  • Require full regression test for released version

Issue

Closes #260
Closes #49

@ntseng ntseng linked an issue May 30, 2026 that may be closed by this pull request
@ntseng ntseng changed the title #260: switch to TypeScript #260 & #49: switch to TypeScript May 30, 2026
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.

Switch from JavaScript to TypeScript Fix model configuration so that types populate to instances

1 participant