Skip to content

fix add more highlight - #2593

Merged
sinelaw merged 4 commits into
sinelaw:masterfrom
asukaminato0721:2556
Jul 12, 2026
Merged

sinelaw merged 4 commits into
sinelaw:masterfrom
asukaminato0721:2556

Conversation

@asukaminato0721

@asukaminato0721 asukaminato0721 commented Jul 7, 2026 •

Copy link
Copy Markdown
Contributor

Add syntax highlighting for more languages and file formats

Adds Sublime-syntax grammars (with test fixtures) for nine additional languages/formats and registers them in the grammar loader.

New grammars

Language / format Extension(s) Fixture
gettext PO .po hello.po
m4 .m4 hello.m4
Xcode pbxproj .pbxproj project.pbxproj
Metal .metal hello.metal
CUDA .cu hello.cu
HIP .hip hello.hip
Fortran .f90 hello.f90
LLVM IR .ll hello.ll
MLIR .mlir hello.mlir

Changes

  • Added grammar definitions under crates/fresh-editor/src/grammars/.
  • Registered the new grammars via build.rs, config.rs, and primitives/grammar/types.rs.
  • Added highlighting fixtures under tests/fixtures/syntax_highlighting/ and extended tests/e2e/syntax_highlighting_coverage.rs to cover them.

Closes

Fortran, LLVM IR, and MLIR grammars are included as additional coverage beyond the linked issues.

@asukaminato0721

Copy link
Copy Markdown
Contributor Author

fix #2618

@sinelaw sinelaw added the in-review label Jul 12, 2026 — with Claude

@sinelaw sinelaw left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

This is an autogenerated comment

  • Reviewed against CONTRIBUTING.md: adds 9 syntect grammars (Metal, CUDA, HIP, Fortran, MLIR, LLVM IR, Gettext PO, M4, Xcode pbxproj) following the exact existing pattern in config.rs/types.rs/build.rs β€” no duplication, no architectural changes.
  • Confirmed no prior support existed for any of these languages/extensions (grepped master) β€” fixes real gaps (#2556, #2557, #2560, #2612, #2618), not duplicate features.
  • Includes unit tests (grammar load + path resolution) and e2e test_highlighting_works! coverage per fixture, satisfying the "reproduce before claiming" rule.
  • cargo check --all-targets passes cleanly (62 embedded syntaxes, up from 53).
  • Manually validated interactively via tmux: opened hello.cu, hello.f90, project.pbxproj, hello.m4, hello.po fixtures in a built binary β€” each is correctly detected in the status bar (CUDA/Fortran/Xcode Project/M4/Gettext PO) and renders distinct token coloring (keywords, strings, comments, punctuation).
  • Approving β€” simple, well-scoped, low-risk data addition.

Generated by Claude Code

@sinelaw sinelaw added reviewed and removed in-review labels Jul 12, 2026 — with Claude
@sinelaw
sinelaw merged commit e9f90be into sinelaw:master Jul 12, 2026
8 checks passed
@sinelaw

sinelaw commented Jul 12, 2026

Copy link
Copy Markdown
Owner

πŸ‘

@asukaminato0721
asukaminato0721 deleted the 2556 branch July 12, 2026 08:43
sinelaw pushed a commit that referenced this pull request Jul 15, 2026
Audited all 60 PRs merged after the v0.4.3 tag and added the user-facing
ones the 0.4.4 section was missing: new/fixed syntax grammars (#2593,
#2553), vi find-char operators (#2441), LSP range formatting (#2605),
monorepo git support (#2369), large wrapped-file edit lag (#2610),
dormant-SSH dock dives (#2570), per-split terminal scrollback (#2595),
and tab-bar scroll-into-view (#2675).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QiCqqGW2WzTNrQPSKetF2W
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

highlight for metal, cuda, hip pbxproj highlight m4 highlight po (Portable Object) file highlight

2 participants