Skip to content

feat: Add MDX language support#1257

Merged
alanhamlett merged 1 commit intowakatime:developfrom
GeekCornerGH:feature/mdx-support
Jan 8, 2026
Merged

feat: Add MDX language support#1257
alanhamlett merged 1 commit intowakatime:developfrom
GeekCornerGH:feature/mdx-support

Conversation

@GeekCornerGH
Copy link
Copy Markdown
Contributor

@GeekCornerGH GeekCornerGH commented Jan 6, 2026

Proper implementaion of #1253

cf00d5b adds MDX to the map of known languages
7e3cb34 routes any file ending in MDX to the WritingDocsCategory instead of unknown

Development was done against Wakapi, but should work on the official Wakatime instance without any change

@gandarez
Copy link
Copy Markdown
Member

gandarez commented Jan 6, 2026

let's hold this PR a little as we're implementing a new dynamic way to generate language maps that may affect and conflict with this one.

#1256

@GeekCornerGH
Copy link
Copy Markdown
Contributor Author

I see unit tests are failing, but they were failing on a fresh clone too for me, did I broke something by mistake?

@alanhamlett
Copy link
Copy Markdown
Member

@GeekCornerGH it's because @gandarez just refactored our Chroma language detection to auto-generate language mappings yesterday.

@GeekCornerGH GeekCornerGH force-pushed the feature/mdx-support branch 5 times, most recently from f81eb6e to 7e3cb34 Compare January 7, 2026 10:51
@GeekCornerGH
Copy link
Copy Markdown
Contributor Author

GeekCornerGH commented Jan 7, 2026

@alanhamlett @gandarez apologies for the notification spam, Git decided not to include the latest merge commit in my local branch for no reason. Should be good now.

@gandarez
Copy link
Copy Markdown
Member

gandarez commented Jan 7, 2026

Let me rethink the strategy for non-mapped languages in Chroma

@gandarez
Copy link
Copy Markdown
Member

gandarez commented Jan 7, 2026

Just found a pretty solution that enables adding missing languages via comment.

#1258

@alanhamlett alanhamlett merged commit 147a966 into wakatime:develop Jan 8, 2026
21 checks passed
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 53.17%. Comparing base (a1d8211) to head (f5f028b).
⚠️ Report is 2 commits behind head on develop.

@@           Coverage Diff            @@
##           develop    #1257   +/-   ##
========================================
  Coverage    53.17%   53.17%           
========================================
  Files          394      394           
  Lines        11496    11496           
========================================
  Hits          6113     6113           
  Misses        4841     4841           
  Partials       542      542           
Flag Coverage Δ
unittests 53.17% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
pkg/heartbeat/category.go 94.16% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@gandarez gandarez mentioned this pull request Jan 8, 2026
@alanhamlett alanhamlett mentioned this pull request Jan 14, 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.

3 participants