Skip to content

Conversation

@yavpungggi
Copy link

@yavpungggi yavpungggi commented Oct 10, 2025

Related GitHub Issue

Closes: #8597

Description

This PR adds support for the .al (AL Language) file extension used in Microsoft Dynamics 365 Business Central development. Since there is no dedicated WASM parser available for AL Language, this implementation uses fallback chunking similar to other languages like VB.NET, Scala, and Swift.

Test Procedure

   pnpm test src/services/code-index/processors/__tests__/parser.vb.spec.ts

Manual Testing

  1. Create a test AL file (e.g., test.al) with sample AL Language code
table 50100 "Customer Extension"
{
    fields
    {
        field(1; "No."; Code[20]) { }
        field(2; "Name"; Text[100]) { }
    }
}
  1. Open the file in the codebase and verify it's properly indexed
  2. Confirm that the code parser processes the file using fallback chunking without errors

Pre-Submission Checklist

  • Issue Linked: This PR is linked to an approved GitHub Issue (see "Related GitHub Issue" above).
  • Scope: My changes are focused on the linked issue (one major feature/fix per PR).
  • Self-Review: I have performed a thorough self-review of my code.
  • Testing: New and/or updated tests have been added to cover my changes (if applicable).
  • Documentation Impact: I have considered if my changes require documentation updates (see "Documentation Updates" section below).
  • Contribution Guidelines: I have read and agree to the Contributor Guidelines.

Get in Touch

AlexTrader82


Important

Add AL Language support by using fallback chunking for .al files, updating extensions and tests accordingly.

  • Behavior:
    • Adds .al extension to fallbackExtensions in supported-extensions.ts for AL Language support.
    • Updates extensions in tree-sitter/index.ts to include .al.
    • Uses fallback chunking for .al files in parser.vb.spec.ts.
  • Tests:
    • Adds .al to fallback chunking tests in parser.vb.spec.ts to verify behavior.
    • Ensures case-insensitive handling of .al extension in tests.

This description was created by Ellipsis for ada6f01. You can customize this summary. It will automatically update as commits are pushed.

roomote and others added 2 commits October 10, 2025 12:53
…chunking

- Added "al" to the extensions array in tree-sitter index
- Added ".al" to fallbackExtensions for length-based chunking
- Enables AL Language (Dynamics 365 Business Central) file support

Fixes RooCodeInc#8596
@yavpungggi yavpungggi requested review from cte, jr and mrubens as code owners October 10, 2025 16:10
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. enhancement New feature or request labels Oct 10, 2025
Copy link

@roomote roomote bot left a comment

Choose a reason for hiding this comment

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

No new issues found - the implementation is clean and follows established patterns. Tests pass successfully. ✅

@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Oct 10, 2025
@daniel-lxs daniel-lxs moved this from Triage to PR [Needs Prelim Review] in Roo Code Roadmap Oct 28, 2025
@hannesrudolph hannesrudolph added PR - Needs Preliminary Review and removed Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. labels Oct 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request PR - Needs Preliminary Review size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

Status: PR [Needs Prelim Review]

Development

Successfully merging this pull request may close these issues.

3 participants