Skip to content

Refactor build configs and CKEditor types for TypeScript migration#109

Open
myabc wants to merge 6 commits intomasterfrom
code-maintenance/72830-convert-typescript
Open

Refactor build configs and CKEditor types for TypeScript migration#109
myabc wants to merge 6 commits intomasterfrom
code-maintenance/72830-convert-typescript

Conversation

@myabc
Copy link
Copy Markdown
Contributor

@myabc myabc commented Mar 5, 2026

Summary

  • align build tooling (Babel, Jest, webpack, ESLint) with the TypeScript conversion
  • refresh package metadata and scripts to support the new setup, including mention utilities and plugins
  • update CKEditor helpers, plugins, and commonmark processors/types to TypeScript-compatible implementations

Testing

  • Not run (not requested)

myabc added 6 commits March 5, 2026 13:24
Add typed interfaces to op-ckeditor exports so downstream
OpenProject code can replace its local CKEditor type shim.

Configure ESLint to parse TypeScript syntax and keep lint,
tests, typecheck, and build passing.
Introduce src/ckeditor-types.ts and export it via the package
subpath '@openproject/commonmark-ckeditor-build/types'.

Keep existing type exports from the main entrypoint and update
README paths and type usage docs for downstream consumers.
Add a build-time copy step for generated declaration files.

During build, copy types into OPENPROJECT_CORE frontend vendor
ckeditor paths so downstream can import via core-vendor aliases.
Copilot AI review requested due to automatic review settings March 5, 2026 16:59
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@myabc myabc requested a review from Copilot March 5, 2026 17:25
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants